text:
v2.0 Extend Line

Please see image above - Ideally, I want 10000 to be my zero line with a line going across as If it was zero, so I can change thickness and colour. I've tried many things, but can't get this to work
Alternatively, if I can get a line drawn across like in this example: http://jsfiddle.net/7a4hhzge/4/ then this should be enough for my requirements.
I have got as far as extending the Line (see below), but the code would only draw the dots and line and not lines to link up the dots. The line I commented out I thought would draw the lines(commented out as I haven't got syntax right - might not be there?) and then the code that follows is additionally drawn.
Chart.elements.Line = Chart.Element.extend({
        draw: function() {
                //Chart.elements.Line.prototype.draw.apply(this, arguments);  //draw existing lines?

               //my additional line to draw
                ctx.beginPath();
                ctx.moveTo(0,0);
                ctx.lineTo(300,150);
                ctx.stroke();
            }
        });

Any help much appreciated. Keep up the good work.

@andi-b the grid lines are drawn by the scale. The base class for all scales is in src/core/core.scale.js. To style an particular grid line, you could modify the scale so that the following properties are arrays
gridLines: {
    color: , // currently a string. if given as as array of strings, map array to ticks
    lineWidth, // currently a number. if given as an array of numbers, map to ticks
}

You should use Chart.helpers.getValueAtIndexOrDefault to make your life easier. You could modify that function to allow a sparse vector which means that not every grid line needs to be styled in the array. The new getValueAtIndexOrDefault would be:
function(value, index, defaultValue) {
	if (value === undefined || value === null) {
		return defaultValue;
	}

	if (helpers.isArray(value)) {
        // Check for undefined to allow a sparse vector
		return index < value.length && value[index] !== undefined ? value[index] : defaultValue;
	}

	return value;
},

The biggest downside of this method is that it requires knowledge of the number of ticks (since the array index is used). Alternatively you could build a mapping of value -> line settings. I'd be more than happy to merge a PR adding this functionality.

Thanks for the response - I tried colouring the lines with an array, but didn't seem to work, although the second part of your response has led me on a better track. I amended the following in Chart.js to get me closer:
FYI: this.options.gridLines.negative = 10000;
// Draw the chart area
if (this.options.gridLines.drawOnChartArea) {
	this.ctx.moveTo(chartArea.left, yLineValue);
	this.ctx.lineTo(chartArea.right, yLineValue);
	if(label==this.options.gridLines.negative){
		this.ctx.strokeStyle = "red";
		this.ctx.lineWidth = 2;
	}
}

But for some strange reason it colours each horizontal line that follows:

@andi-b I think you need to reset the canvas style after drawing the red line. Changing the if as follows should do it. The scale was written to not add lots of resets of strokeStyle and lineWidth when nothing is changing.
if (label ==  this.options.gridLines.negative) {
    this.ctx.strokeStyle = 'red';
    this.ctx.lineWidth = 2;
    setContextLineSettings = true; // reset next time
}

Brilliant - I didn't realise I was so close :)
Many thanks

Closing since resolved.

authors: andi-b etimberg


text:
Auto set min/max axis values for line graph

If you have line graph with very large numbers that vary slightly (say 10,000 to 10,100) the line graph always looks like a straight line because it is scaled from 0 to 10,100 and all the variation happens in the top few pixels. I know that we can calculate the scale ourselves and override this using the "scaleOverride" option so that the x-intercept isn't zero.
It would be nice if there was an automatic setting to do this that would cause Chart.js to calculate the min/max and scale it appropriately so that I don't have to do this math. It would be a handy convenience.

@offsky what did you set the scaleBeginAtZero option to?

This is the first thing I tried, but both "false" and "true" gave the same graph.

@offsky Thanks for this report, could you please post two JSBINs showing the issue here

I think this might be related to #831

This may be related to #655 with sparse data. I have added a small example (http://jsfiddle.net/uLvxw0jx/).

This is the sparse data issue, closing as duplicate of #655

source: gharchive/issue

authors: dtslvr etimberg fulldecent offsky

license: mit

license_type: permissive

license_source: bigquery


text:
Need to support the better gradient clipping method.

It might be better to support the gradient clipping more systemetical way.

network graph has global norm with 1:1:1:clip_weights.size()
during calcgradient, if gradient clipping is enabled, then compute gradient norm and update the global norm.
after finishing the calcgradient, calcderivatives, network graph compute the global_norm which is used in apply gradient.
we do the apply gradient after applying clipping, and run apply gradient.

In order to do these, it might be better to use this below order in a layer.
base_count = graph.size() *4;
applyGradient (n) --> forwarding (n+1) --> calcGradient (base_count-(n+1)*2)  --> calcDerivative (base_count-(n+1)*2+1)
Normally, we do use,
max_count = graph.size()*4;
forwarding(n) --> calcGradient( max_count-(n+1)*3) --> calcDerivative(max_count-(n+1)*3+1) --> ApplyGradient(max_count-(n+1)*3+2)
In order to do it, we might need to change the execution order in NeuralNetwork::backwarding() and NeuralNetwork::forwarding() w.r.t training and non-training mode.
We also need to think which one is better considering the memory swapping

:octocat: cibot: Thank you for posting issue #2085. The person in charge will reply soon.

authors: jijoongmoon taos-ci

repo: nnstreamer/nntrainer

license_source: github-api


text:
Fixed incorrect notation of k-NN

Signed-off-by: brainer jeansanghyun@gmail.com

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #368. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

INFO: You can read if there are misspelled characters at our misspelling check report. Please read http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/368-202007272011540.65856599807739-f6584eb7eeb87077bdf39f85b1344b4c104e2eb8/report/misspelling_result.txt.

:octocat: cibot: @brainer3220, One of the module of prebuild group is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnstreamer/nntrainer/wiki/.

You can update the commit by pushing to the same personal branch with -f option.

You need commit messages. not just title.

source: gharchive/pull-request

authors: brainer3220 jijoongmoon myungjoo taos-ci


text:
[wait for #754] [neuralnet] Skip backwarding for non-trainable layers

This patch skips the backwarding for the non-trainable layers.
Further, the last trainable layer skips calcDerivative as well.
This results in much fewer calculations as well as more importantly, reduced memory.
See also #732
Self evaluation:
1. Build test: [x]Passed [ ]Failed [ ]Skipped
2. Run test: [x]Passed [ ]Failed [ ]Skipped
Signed-off-by: Parichay Kapoor pk.kapoor@samsung.com

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #755. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:octocat: cibot: @kparichay, One of the module of prebuild group is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnstreamer/nntrainer/wiki/.

With this patch, the peak memory requirement for CIFAR10 transfer learning application (mentioned in #732) is reduced to 38252KB. Below is the analysis of this memory requirement :

weight of the fc layer (there is only one layer in the model) - 62720 (input size) x 10 (number of outputs) x 4 (size of float) = 2450 KB
gradient of the FC layer (same size as weight) = 2450 KB
adam optimizer weights (two tensors with same shape as weight) - 2 x 62720 x 10 x 4 = 4900 KB
input for the layer - 32 (batch size) x 62720 (input size) x 4 (float size) = 7840 KB

Total bare minimum memory requirement (sum of the above) = 17640 KB
Peak heap memory required by nntrainer (evaluated using heaptrack = 26.52 MB.This includes the next input buffer of size 7840KB which has been pre-fetched to be executed next for the best training performance. Excluding this pre-fetched buffer memory requirement brings the Peak heap memory to 19316 KB. This difference of 1676 KB is under investigation.
The total peak memory requirement is high because that value includes nntrainer shared object and its dependencies.

This patch has been updated to apply this logic to graph than to neural network class.

authors: kparichay taos-ci


text:
[Admin] Branching Option

[x] Require branches to be up to date before merging
This ensures pull requests targeting a matching branch have been tested with the latest code. This setting will not take effect unless at least one status check is enabled (see below).

You have enabled that option, which requires to rebase EVERY PR whenever a PR is merged. Do you really need that option enabled?

:octocat: cibot: Thank you for posting issue #239. The person in charge will reply soon.

Screenshot:

So.. do you really need it? (why?)

I have enabled this feature to test a PR with latest code as an experiment even though it is impractical in case of one branch structure. So it will be continued for a while.

Ok. I've got it.

authors: leemgs myungjoo taos-ci

repo: nnsuite/TAOS-CI


text:
Update contributing.md

[Template] PR Description
In general, github system automatically copies your commit message for your convenience.
Please remove unused part of the template after writing your own PR description with this template.
$ git commit -s filename1 filename2 ... [enter]

Summarize changes in around 50 characters or less

More detailed explanatory text, if necessary. Wrap it to about 72
characters or so. In some contexts, the first line is treated as the
subject of the commit and the rest of the text as the body. The
blank line separating the summary from the body is critical;
various tools like `log`, `shortlog` and `rebase` can get confused
if you run the two together.

Further paragraphs come after blank lines.

**Changes proposed in this PR:**
- Bullet points are okay, too
- Typically a hyphen or asterisk is used for the bullet, preceded
  by a single space, with blank lines in between, but conventions vary here.

Resolves: #123
See also: #456, #789

**Self evaluation:**
1. Build test: [ ]Passed [ ]Failed [*]Skipped
2. Run test: [ ]Passed [ ]Failed [* ]Skipped

**How to evaluate:**
1. Describe how to evaluate in order to be reproduced by reviewer(s).

Add signed-off message automatically by running **$git commit -s ...** command.

$ git push origin <your_branch_name>

:memo: Version: 1.20180820. Thank you for submitting PR #331. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://invain.mooo.com/.

To contributor, We have used 'Signed-off-by:' notation by default to handle the license issues, that result from contributors. Note that 'Is there a Signed-off-by line?' is important because lawyers tell us we must have to it to cleanly maintain the open-source license issues even though it has nothing to do with the code itself.

INFO: You can read if there are misspelled characters at our misspelling check report. Please read http://invain.mooo.com/TAOS-CI/ci/repo-workers/pr-checker/201810251041260.94167399406433-331-020873f9f2a47ac284b8b39231fa814cfa109c44/report/misspelling_result.txt.

:octocat: cibot: @JBROAD0706, One of the format checkers is failed. If you want to get a hint to fix this issue, please go to https://github.com/nnsuite/TAOS-CI/wiki/.


text:
🛑 Switch Homebrew Guide is down

In e88973f, Switch Homebrew Guide (https://switch.homebrew.guide) was down:

HTTP code: 403
Response time: 163 ms

Resolved: Switch Homebrew Guide is back up in a9aab30.

license: MIT


text:
Web socket timeout for "Listen on"

https://github.com/node-red/node-red/blob/1419729458b8423484e70c64fb7f762c6ff6a1ca/packages/node_modules/%40node-red/nodes/core/network/22-websocket.js#L186-L198
This would be a neat feature to have.
I tested it with a hardcoded timeout of 30000 ms and it works well for me if i also remove the
clearInterval(node.heartbeatInterval); calls.
https://github.com/node-red/node-red/blob/1419729458b8423484e70c64fb7f762c6ff6a1ca/packages/node_modules/%40node-red/nodes/core/network/22-websocket.js#L128
https://github.com/node-red/node-red/blob/1419729458b8423484e70c64fb7f762c6ff6a1ca/packages/node_modules/%40node-red/nodes/core/network/22-websocket.js#L144
The node already has a property called heartbeat which is only used for "Connect to" nodes
I Think he same property could be used for "Listen On" nodes as well
It would require a little bit of copy&pasting from here:
https://github.com/node-red/node-red/blob/1419729458b8423484e70c64fb7f762c6ff6a1ca/packages/node_modules/%40node-red/nodes/core/network/22-websocket.html#L193-L201

Hi @Lackmann1994
For future reference, please use the issue templates we provide.
We prefer that feature requests are raised on the forum or slack rather than raised as issues. Issues are primarily for bugs in the code.
If you wanted to work up a PR that implements this feature, we'd be happy to take a look.

Start node-red in safe mode
node-red -safe
Then make changes in flow, and remove unnecessary properties in config.
restart you node-red normal

@hishamudeen what has that got to do with this feature request?

authors: Lackmann1994 hishamudeen knolleary

repo: node-red/node-red


text:
Typescript is not evil

I reviewing all the code and you should using typescript ... (thats my opinion)
Because you can target if you want compatible with ES5 or without Ecmascript and bundle only ".js" and ".d.ts" files as your will.
javascript lol<"use strict"<typescript lowest configuration < tolerate null/undefined and any  < no any tolarate, no quiproquo between null/undefined.
I'm using everyday the medium typescript set up:  "tolerate null/undefined and any" and do well the job !
I see a lot of "this", I hope figure out what is "this" in the story => putting variable in "window" is highly ugly rather than class.
If you dont care about this, close this issue please,
Sometimes I fork code and re-write it, thats why I share my opinion, hope it helps.

extract from dumper.js
function compileMap() {
  let result = {
    scalar: {}, sequence: {}, mapping: {}, fallback: {},
    multi: { scalar: [], sequence: [], mapping: [], fallback: [] }
  }, index, length;

  const collectType = (type) => {
    if (type.multi) {
      result.multi[type.kind].push(type);
      result.multi['fallback'].push(type);
    }
    else {result[type.kind][type.tag] = result['fallback'][type.tag] = type;}
  }
  for (index = 0, length = arguments.length; index < length; index += 1) {arguments[index].forEach(collectType);}
  return result;
}

//later
result.compiledTypeMap  = compileMap(result.compiledImplicit, result.compiledExplicit);

Can you explain WTF is going on ? you create a function without paremeter and later call it with parameter ...
I fixed ~20 issues like this but here I really cannot fiix this ...

Can you explain WTF is going on ? you create a function without paremeter and later call it with parameter ...

It's classic ES3-style variadic function (we can't use ES6 variadics for backward compatibility reasons).
In case you aren't familiar with javascript, here's the typescript alternative you can use:
function compileMap(..._lists: any[]) {}

ok, ...
why in the code on this repo it is like ????????????
function compileMap(/*..._lists: any[]*/) {}

extract for the repo
function compileMap(/* lists... */) {
  var result = {
        scalar: {},
        sequence: {},
        mapping: {},
        fallback: {},
        multi: {
          scalar: [],
          sequence: [],
          mapping: [],
          fallback: []
        }
      }, index, length;

  function collectType(type) {
    if (type.multi) {
      result.multi[type.kind].push(type);
      result.multi['fallback'].push(type);
    } else {
      result[type.kind][type.tag] = result['fallback'][type.tag] = type;
    }
  }

  for (index = 0, length = arguments.length; index < length; index += 1) {
    arguments[index].forEach(collectType);
  }
  return result;
}

function Schema(definition) {
  return this.extend(definition);
}

Schema.prototype.extend = function extend(definition) {
  var implicit = [];
  var explicit = [];

  if (definition instanceof Type) {
    // Schema.extend(type)
    explicit.push(definition);

  } else if (Array.isArray(definition)) {
    // Schema.extend([ type1, type2, ... ])
    explicit = explicit.concat(definition);

  } else if (definition && (Array.isArray(definition.implicit) || Array.isArray(definition.explicit))) {
    // Schema.extend({ explicit: [ type1, type2, ... ], implicit: [ type1, type2, ... ] })
    if (definition.implicit) implicit = implicit.concat(definition.implicit);
    if (definition.explicit) explicit = explicit.concat(definition.explicit);

  } else {
    throw new YAMLException('Schema.extend argument should be a Type, [ Type ], ' +
      'or a schema definition ({ implicit: [...], explicit: [...] })');
  }

  implicit.forEach(function (type) {
    if (!(type instanceof Type)) {
      throw new YAMLException('Specified list of YAML types (or a single Type object) contains a non-Type object.');
    }

    if (type.loadKind && type.loadKind !== 'scalar') {
      throw new YAMLException('There is a non-scalar type in the implicit list of a schema. Implicit resolving of such types is not supported.');
    }

    if (type.multi) {
      throw new YAMLException('There is a multi type in the implicit list of a schema. Multi tags can only be listed as explicit.');
    }
  });

  explicit.forEach(function (type) {
    if (!(type instanceof Type)) {
      throw new YAMLException('Specified list of YAML types (or a single Type object) contains a non-Type object.');
    }
  });

  var result = Object.create(Schema.prototype);

  result.implicit = (this.implicit || []).concat(implicit);
  result.explicit = (this.explicit || []).concat(explicit);

  result.compiledImplicit = compileList(result, 'implicit');
  result.compiledExplicit = compileList(result, 'explicit');
  result.compiledTypeMap  = compileMap(result.compiledImplicit, result.compiledExplicit);

  return result;
};

really dont understand why it is worked ...

repo: nodeca/js-yaml


text:
Update the Chinese FAQ to stay in line with the latest.

Mainly break the "what" section into "what" and "why", and updated the "Why did you break with the Node.js version numbering system?" section.

#357 #372 #361

@nodejs/iojs-cn

LGTM +1

LGTM :+1:

Merged, thank you!

authors: coderhaoxin fhemberger laosb xudafeng

repo: nodejs/website


text:
CH340G is 5V powered, not 3.3V

CH340G is 5V powered and thus pushing 5V level (through R12 ~4.3V)  into ESP12 on RXD0 instead of 3.3V level.

This would explain why my 0.9 boards are unstable and difficult to flash.

Argh, lovely...

Where is the software for these drivers?  The one on here drives me nuts as it keeps disconnecting and connecting.

authors: AkosLukacs MrAda abl michal-muhlpachr

repo: nodemcu/nodemcu-devkit


text:
Translations failing on second fetch applicationDidBecomeActive

The translations are correctly loading in the first open of the app, but when the permission for notifications is called, this causes applicationDidBecomeActive to be called which makes the translations update. This then fails to get the translations.

Fixed in #34.

repo: nodes-ios/NStackSDK


text:
An Event has many years?

At the moment, an Event has a name attribute and a year attribute, e.g. Event.new(name: RubyConf, year: 2017).
I wonder if we should change this to be an event has a name and then many years (or occurrences) attached to it, e.g. RubyConf is an Event, it happened in 2017, 2016, 2015, etc..
This will make it more interesting to view the kinds of submissions that, for example, RubyConf has accepted and rejected over the years. We can also add more information to a particular event, for example, something like: 'RandomConf has a different group of reviewers each round'.
I suspect that the event view will be more interesting than the speaker view for a more general audience.
(You could also then look on a per year basis across a range of events and perhaps see what themes were popular or unpopular among speakers, for example.)
What are people's thoughts on this?
We'd have to change the flow of adding a new event to the platform as well.

I think that would make a lot of sense; it's kind of how I think of conferences--the con is abstract, and the event is once per unit of time. I'm guessing I'm not the only one.
I'd be tempted to rename the existing Event model to Cohort or EventYear (not sure just Year is as clear as it could be), and make it belong to a new Event record. That way existing proposals still refer to the same record.
Changes that would need to be made:

rename existing Event model to Cohort or Year
create new Event model
set up ActiveRecord associations between Event and Cohort
update Proposal to point to renamed Cohort resource
update views to refer to the Event's name, not the Cohort's name, effectively sidestepping the danger of having an event cohort called "Bath Ruby Bath Ruby 2015", instead of just "Bath Ruby 2015"

would it make sense to allow the updated Cohort model to have a subtitle? Do conferences do this? "Bath Ruby 2018: the Bathening?"

Models might look like
# new event model
class Event
  validates :name
  validates_uniqueness_of :name

  has_many :cohorts # or :years
  has_many :submissions, through: :cohorts
end

# renamed from Event
class Cohort
  belongs_to :event
  validates :year
  validates_uniqueness_of :year, scope: :event

  has_many :submissions

  def name_and_year
   "#{event.name} #{year}"
  end
end

Enh. Event as a name for a new resource is a bad idea. EventSeries makes assumptions that it will repeat though. I'm not coming up with anything better atm.

Hey @flyinggrizzly — thanks for your thoughts.
Will have a proper read-through at some point in the upcoming week! :-)

FYI, @flyinggrizzly — going through a busy time right now so this will take me a bit longer to look at.

@nodunayo no worries--I'm tinkering with it already to see how my idea plays out. If you get curious, take a look!
About to disappear to the US for Christmas though, so no promises as to how much I'll be doing over the next few weeks. Happy to pick it up again in the new year if the help is still wanted/needed!

Ooh, I'll definitely take a look some time soon.
Let's pick up this conversation in the New Year (at the latest) for sure. I'm about to move flat and lots of change is happening at work at the moment. Then it's family time/holidays, etc.
I'd love to keep working with you in the New Year. It's been a pleasure!
Happy holidays if we don't speak before then.

@nodunayo this is more than I expected (which in retrospect, was maybe a little obvious...), but I'm still working on it 😅
Trying to figure out how to wire the two models together so you can select or name new Event Series from the Cohort screen r/n

Hey! Hope you've had good holidays.
Looking forward to reading all of this and digging into your work soon. 😁

Should we close this issue?

Yes!! 😊

authors: flyinggrizzly nodunayo

repo: nodunayo/speakerline


text:
Github API Browser

create a github api browser that serializes the received data into a class that also sanitizes.

marked this issue as blocking #115

marked this issue as blocked by #58

marked this issue as related to #68

marked this issue as blocked by #138


text:
feat: support Xamarin, MAUI

Hi @nogic1008 thanks for your effort with this repo!  I tried to use it in xamarin context and found that I need to do a few changes, maybe you would like to accept them in the main repo.

There was exception accessing file on configuration reload so I added stream.Close()
Refactored IWritableOptions to inherit from IOptionsMonitor so we can access both CurrentValue and OnChange from an instance of IWritableOptions
Added extension method for configuring JsonWritableOptions with a simple path in the form of string for cases where IHostEnvironment is not available

I kind of neglected that there were tests! Will add them later.

@anpin
Could you please provide the Xamarin example?
I want to add it to /sandbox directory.

@anpin Could you please provide the Xamarin example? I want to add it to /sandbox directory.

Should I rather add a sample for MAUI instead of Xamarin?

xamarin sample is also added. it relies on the added extension with explicit filepath as well as creates the file if it doesn't exist.

@anpin
Thank you for your contribution!!
I'll ship this changes on Next Major release because adding interface is breaking change.
ex.

Breaking Change Rules
A definitive guide to API-breaking changes in .NET

@nogic1008 thanks for considering the changes and pointing me in the direction on how to improve my style of coding!


text:
Compressed ACIR format

This will be needed for those who want to put the circuit representation on-chain

This should be moved to the acvm repo

On the topic of moving issues, we can use "Transfer issue" at the bottom of the sidebar:

Thanks for informing me of that :) I'm on GitHub mobile so I can't currently do it -- just adding the relevant labels, then moving once I get on my laptop again

authors: kevaundray phated

repo: noir-lang/noir


text:
new flex-layout-module

Please add a small layout module by using flex-layout. For example, "l-flex-hor", "l-flex-vert". Actually, the "F7-layout-module" ("layout-grid") uses "calc-prop".
Would be nice to add something like "layout-flex"-module in F7.
F7 uses already a much of flex-props spread all over the F7-stylesheet. These props can be isolated within a new module "layout-flex" which can also be used internally by F7 for layouting and externally be the user.

You're very welcome to create a PR :-)

I didn't get the issue. What should be added, what is flex-layout? Layout grid in F7 already uses flex

Closed due to no response after 8 days


text:
Responsive scroll container with only one slide

Hello,
I have an image 4252x686px inside a scrollcontainer slider in freemode.
But I dont like my solution cause it was more like trial and error.
The Markup is:
div#bench-wrapper > div.swiper-container >  div.swiper-slide > img
Options are:
var options  = {
                scrollContainer: true,
                freeMode: true,
                momentumRatio: 0.5,
                grabCursor: true,
                resizeReInit: true,
                cssWidthAndHeight: true,
                scrollbar: {
                    container: ".scrollbar",
                    hide: false
                }
            };

And to resize it I use this code:
var win = $(window);
var swiperFullWidth = bench.wrapper.offsetWidth,
      swiperFullHeight = bench.wrapper.offsetHeight;
var toBeResized = $(".swiper-wrapper, .swiper-slide");
win.on("resize", function(){
      var w = win.innerWidth(),
            h = win.innerHeight(),
            wrpH = toBeResized.css("height"),
            wrpW = toBeResized.css("width"),
            targetHeight, targetWidth, ratio;
            console.log("onresize");
      if( w < 650 || h < swiperFullHeight*0.5){
            ratio = 0.25;
      }
      else if( w < 850 || h < swiperFullHeight*0.75){
            ratio = 0.5;
            console.log("w < 850 ");
      }
      else if( w < 1150 || h < swiperFullHeight){
            ratio = 0.75;
            console.log("w < 1150 ");
      }
      else{
            ratio = 1;
       }
      targetWidth = Math.round(swiperFullWidth*ratio);
      targetHeight = Math.round(swiperFullHeight*ratio);
      if( wrpH == targetHeight || wrpW == targetWidth ){
            return;
      }
      toBeResized.css({
            height: targetHeight+"px",
            width: targetWidth+"px"
      });
            bench.resizeFix(true);
});
win.trigger("resize");

Is there an easier or better way?

Didn't really understand what you are trying to achieve, do you have a live link so i can take a look?


text:
Orientation change

Hi,
I am using this for main banner and it's set as responsive style.
The problem is when I change screen orientation, it seems " .swiper-container"  doesn't change it's height.
Image cuts off(overlap by next ul element) when it's on landscape mode and image stays too far (.swiper-container extended)  from the next ul element when it's back from the landscape mode.
Normal View
ooooooooooooo   <-- banner
ooooooooooooo
<-- 20px padding
ululululululululul   <-- listview
ululululululululul
ululululululululul
Landscape View
ooooooooooooooooooooooooooo   <-- banner
ulululululululululululululululululululul   <-- listview covered banner
ulululululululululululululululululululul
ulululululululululululululululululululul
Back to Normal View
ooooooooooooo   <-- banner
ooooooooooooo   <--container still remains landscape mode height
empty space
empty space
empty space
ululululululululul   <-- listview
ululululululululul
ululululululululul
please advise
Thank you

Sorry to summon this, but I've just had this same issue and fixed it by calling swiper.update() on a 50ms timeout (as suggested by @nolimits4web).
Swiper as all iDangerous projects is great. Thanks!


text:
Getting "Hydration failed because the initial UI does not match what was rendered on the server" error when using Swiper Element on NextJs

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/p/sandbox/dreamy-benz-et8xf6?file=%2Fpages%2Findex.tsx&selection=[{"endColumn"%3A2%2C"endLineNumber"%3A9%2C"startColumn"%3A2%2C"startLineNumber"%3A9}]
Bug description
Using <swiper-container> returns the error:
Error: Hydration failed because the initial UI does not match what was rendered on the server.
This error is from the dev console:

Refer to: https://nextjs.org/docs/messages/react-hydration-error
As seen in the example on codesandbox, there is no  element anywhere in the source code. But apparently, one  element is automatically added inside <swiper-container> when rendered.

Is there a way to disable this  element from being rendered or to change the element  to a different element just to avoid this issue?
Expected Behavior
No response
Actual Behavior
No response
Swiper version
9.0.5
Platform/Target and Browser Versions
macOS, Windows, Google Chrome 110.0.5481
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

You can disable a11y

I'm having the same issues here, it happens when I pass both of the breakpoints and loop props.
by deleting any of those props, the problem does not show anymore.

I'm having the same issues here, it happens when I pass both of the breakpoints and loop props.
By deleting any of those props, the problem does not show anymore.

I'm having the same issues here, it happens when I pass both of the breakpoints and loop props. By deleting any of those props, the problem does not show anymore.

I am having this exact issue. Did you ever find a way to get it to work without having to remove breakpoints and/or loop?

@nachtfunke you can actually check if dom is loaded or not and render swiper package accordingly.
Example :
const Index = () => {
  const [domLoaded, setDomLoaded] = useState(false);

  useEffect(() => {
    setDomLoaded(true);
  }, []);

  return (
    <>
      {domLoaded && (
        <Swiper>

Test

        </Swiper>
      )}
    </>
  );
};

export default Index;

Reference : Stackoverflow answer

@nachtfunke you can actually check if dom is loaded or not and render swiper package accordingly.
Example :
const Index = () => {
  const [domLoaded, setDomLoaded] = useState(false);

  useEffect(() => {
    setDomLoaded(true);
  }, []);

  return (
    <>
      {domLoaded && (
        <Swiper>

Test

        </Swiper>
      )}
    </>
  );
};

export default Index;

Reference : Stackoverflow answer

Works perfect in Next.JS!!! Thanks

authors: adarshaacharya alimokhlsi hsnmrd marcelo-albuquerque nachtfunke nolimits4web xvnds


text:
Swiper miscalculates center

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/p/sandbox/swiper-react-mousewheel-control-forked-rpg0qw?file=%2Fsrc%2Fstyles.css&selection=[{"endColumn"%3A1%2C"endLineNumber"%3A54%2C"startColumn"%3A2%2C"startLineNumber"%3A41}]
Bug description
When using centeredSlides, loop and slidesPerView={"auto"} simultaneously, the slider doesn't calculate the correct center of the slides. The pagination also jumps to the incorrect slide when clicked, and the mousewheel on MacOS jumps multiple slides during the same swipe.
Expected Behavior

Slides should be centered
Pagination should go to correct slide
Mousewheel shouldn't jump two slides in a row

Actual Behavior

Slides position after changing slides is unpredictable
Pagination doesn't go to the right slide
Mousewheel jumps multiple slides on same swipe

Swiper version
9.0.0
Platform/Target and Browser Versions
macOS Ventura 13.2.1 / Chrome 112.0.5615.49
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Demo you posted uses old Swiper version, update to latest first


text:
🛑 mastodon.macsnet.cz is down

In b132a7b, mastodon.macsnet.cz (https://mastodon.macsnet.cz/nodeinfo/2.0) was down:

HTTP code: 0
Response time: 0 ms

Resolved: mastodon.macsnet.cz is back up in e553197.

repo: nolog-it/mastodon-uptime


text:
只有neteasemusic.start一个命令可以执行，别的命令都找不到无法执行

执行neteasemusic.start后弹出NeteaseMusic标签页，上面只显示 Please preserve this webview tab ，点击无反应，输入其他命令会提示没有匹配的命令，请问是什么问题？最新版的vscode版本1.42.1，我看其中自带了ffmpeg.dll，出现上述问题，我去下载了对应版本的ffmpeg.dll进行替换还未解决

vscode 所有版本都自带 ffmpeg.dll 但是是不完整的，需要替换
你的问题不是 ffmpeg 的问题
是没有网络或者请求失败了的问题，参考 #67 #81
因为启动过程中会请求登录状态、签到状态、红心歌单数据，如果请求失败了就启动不了
目前应该还是不支持带 authentication 的代理
可以关掉插件设置里的证书校验，或者关掉 https 请求试一下

明白了，谢谢大大

repo: nondanee/vsc-netease-music


text:
remove the client_noobaa.log logrotate in NC env ?

Environment info

NooBaa Version: VERSION
Platform: Kubernetes 1.14.1 | minikube 1.1.1 | OpenShift 4.1 | other: specify

ODF 4.15.0 d/s build 0228
Actual behavior

Do we need to have the following in the /etc/logrotate.d/noobaa/logrotate_noobaa.conf file, we are not creating this file at all or what/when does this create
/var/log/client_noobaa.log
{
daily
size 100M
start 1
missingok
rotate 10
compress
create 644 root root
sharedscripts
postrotate
/bin/kill -HUP cat /var/run/syslogd.pid 2> /dev/null 2> /dev/null || true
/bin/kill -HUP cat /var/run/rsyslogd.pid 2> /dev/null 2> /dev/null || true
endscript
}
Expected behavior

Checking if we need to remove these in the logroate.conf file , if the file is not required for the NC NSFS
Steps to reproduce

NA
More information - Screenshots / Logs / Other output

@rkomandu We are not creating a client_noobaa.log for now and log rotate configured in such a way that even if client_noobaa.log is missing it wont create any issue(missingok). I am not sure are there any standalone noobaa products generating client_noobaa.log or not. I dont have any issue with that.
@guymguym @romayalon @tangledbytes WDYT?

tagging @dannyzaken who worked on this area?

As @naveenpaul1 wrote - we do not create this log file for nsfs non-containerizes, so the logrotate config with missingok is expected to skip. This way we don't need to split our logrotate conf to 3 files and can deploy just one. Closing as not planned. Thanks

authors: guymguym naveenpaul1 rkomandu romayalon

repo: noobaa/noobaa-core


text:
E-mail template

Email body is encoded, so it's look like this:
Thank you for signing up

Please click the link below to activate your account:

http://demo.int/account/activate/isuGuop8aLLRVhiyGbKNFHRY36sKfK7Y

In YiiMailSender class:
public function send(array $config = [])
{
    return Yii::$app->mailer->compose()
        ->setTo($config['to'])
        ->setFrom($config['from'])
        ->setSubject($config['subject'])
        ->setHtmlBody($config['body'])
        ->send();
}

setTextBody method should be swapped to setHtmlBody.

Thanks for noticing this. Any chance that you could create a PR that changed to use setHtmlBody?

Sure! Here it is: https://github.com/nordsoftware/yii2-account/pull/12


text:
Warn users when map structure indicates unintended inline map in YAML

With the {name} and {{name}} bind point formats, it is quite easy for a user to use
yaml:
  op:
    field: {bindingname}

when they actually need
yaml:
  op:
    field: "{bindingname}"

or
yaml:
  op:
    field: |
     {bindingname}

The issue is that YAML parses {word} as a JSON map with a null value for the key word. (YAML is a superset of JSON)
There is no valid construction yet in which a null-valued key is meaningful for op templates. For now, it should be considered an error for this to occur, with a helpful warning indicating to the user that bind points must be in quotes or in a block value context.

complete

repo: nosqlbench/nosqlbench


text:
Best approach for adding a slider into menu?

Love the class, great work!
I have implemented it with great success. Makes a much better impression that the default popup-menu.
I recently had an request for adding a slider into the menu.

What is the best approach to add an PullDownMenuItem.slideable item?

Hi @large
Apple (at least as of date) have not introduced a slider item for their menus, so there might not be any "official" implementation any time soon by me.
As of best approach to add any custom item, there is only one - your slideable item widget should extend PullDownMenuEntry. That's pretty much it.
You can also ignore any overrides for PullDownMenuEntry (just use some placeholder values) since those would be removed in the next update.
Really quick example:
Code
class SliderExample extends PullDownMenuEntry {
  const SliderExample(
    this.initialValue,
    this.onChanged, {
    super.key,
  });

  final double initialValue;
  final ValueChanged<double> onChanged;

  @override
  Widget build(BuildContext context) => _Slider(
        initialValue: initialValue,
        onChanged: onChanged,
      );

  // Can be ignored
  @override
  double get height => 0;

  // Can be ignored
  @override
  bool get isDestructive => false;

  // Can be ignored
  @override
  bool get represents => false;
}

class _Slider extends StatefulWidget {
  const _Slider({
    required this.initialValue,
    required this.onChanged,
  });

  final double initialValue;
  final ValueChanged<double> onChanged;

  @override
  State<_Slider> createState() => _SliderState();
}

class _SliderState extends State<_Slider> {
  late double value = widget.initialValue;

  void onChanged(double v) {
    setState(() => value = v);

    widget.onChanged(v);
  }

  @override
  Widget build(BuildContext context) => Material(
        type: MaterialType.transparency,
        child: Slider(
          value: value,
          onChanged: onChanged,
        ),
      );
}

It is unfortunately slightly ugly aproach since PullDownMenuEntry extends StatelessWidget, but it should work

Hi @notDmDrl ,
Great feedback, I fully understand that a scrollbar will be little "outside" the scope of this.
With your tip for implementation I would add this slider https://pub.dev/packages/cupertino_range_slider_improved so it fall into the design.
Thank you for the help :)

authors: large notDmDrl


text:
Update the branch policies for the repository

After reviewing the branch policies for the repository, it seems that the main and release branches can benefit with more strict push and PR review policies. I would like to propose updating those as follows:

Follow naming convention release/<release-number> for release branches. Example release/rc.1
Use the following rules for main and release/* branches:

Require PR before merging

Require 3 approvals
Dismiss stale PR approvals when new commits are pushed
Require review from Code Owners

Require status checks to pass before merging
Require conversation resolution before merging
Require signed commits
Do not allow bypass the above settings

Please add your comments and proposals for additional changes to this issue.

LGTM but IANAM

LGTM but IANAM

Updated with checkboxes to indicate the current state as several of the rules were already in place.

LGTM but IANAM

Closed as all polices enforced

authors: SteveLasker iamsamirzon priteshbandi toddysm yizha1

repo: notaryproject/notaryproject


text:
Notation Alpha Library

This notation library is based on the prototype-3 of the notary v2 library, which is based on prototype-2.
The underlying artifact specification and registry referrer API are alighed to oras-project/artifacts-spec.

LGTM as this is based on prototype 2 and JWS refactoring will bring in more changes.

Hello! 👋
This PR seems to implement a fair bit of the registry protocol in Go, in addition to implementing the new not-yet-standard bits necessary to store signatures. I happen to have some experience writing and maintaining a fairly widely used registry protocol client in Go, which you might be interested in reusing: go-containerregistry; specifically the low-level HTTP transport in pkg/v1/remote/transport, which is well-contained and doesn't depend on any specific API data types.
This package ends up being used by a number of tools, including buildpacks, kaniko, ko, Tekton, Knative, and probably a dozen others. This means it's pretty well battle-tested in real world scenarios, and with more users it should have any bugs or vulnerabilities discovered and fixed more quickly than new single-use clients.
This comment shouldn't block this PR, but I think it'd be worth discussing reusing an existing client instead of writing and maintaining your own, and (call me biased) I think go-containerregistry would be a good one. 😄

@imjasonh that's great feedback and we should consider reusing an existing registry client. I believe a lot of changes are done into oras-go to support the new signature manifest format with local cache store similar to how helm is working and also has dependency on and quite heavily used. Another aspect is [move] (https://github.com/oras-project/oras-go/pull/8) of the entire graph of signatures.

Tracking refactoring in #8

Tracking signature format update in #9

authors: imjasonh sajayantony shizhMSFT

repo: notaryproject/notation-go-lib


text:
Nothing Changed after I installed notion enhancer (windows 10)

"changed 24 packages, and audited 25 packages in 5s
3 packages are looking for funding
run npm fund for details
found 0 vulnerabilities"
That what appeared to me when I entered this code (npm i -g notion-enhancer)
And nothing changed in notion after this
please help me

see https://github.com/notion-enhancer/desktop/issues/762

repo: notion-enhancer/desktop


text:
Update cats-effect to 3.5.0

About this PR
📦 Updates org.typelevel:cats-effect from 3.2.9 to 3.5.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

💡 Applied Scalafix Migrations

org.typelevel:{cats-effect,cats-effect-laws}:3.3.0 (created no change)

github:typelevel/cats-effect/v3_3_0?sha=series/3.x

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "cats-effect" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.typelevel", artifactId = "cats-effect" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, scalafix-migrations, commit-count:1

Superseded by #458.

repo: notxcain/aecor


text:
Adding new polkadot parachains

Efinity

Spoiler!

Centrifuge

Spoiler!

HydraDX
This network is blocking connection to wss://rpc-01.hydradx.io by Origin header which must equal https://polkadot.js

Spoiler!

Interlay

Spoiler!

Interlay tokens:
https://github.com/interlay/interbtc/blob/master/primitives/src/lib.rs#L473
Interlay existential deposit:
https://github.com/interlay/interbtc/blob/34c24961384208d8a22cef3446fea22d585c3ef7/parachain/runtime/interlay/src/lib.rs#L942

Interlay uses orml

fixed!

Interlay uses orml

Could you resolve your change request?

repo: nova-wallet/nova-utils


text:
Basket. Parachain transfers. Can't handle submit operation in block

Description
When send not main asset on parachain - unending submitting state, author_extrinsicUpdate - inblock the latest status, but its still shown as submitting.
The latest status for relaychain is finalized

Steps to Reproduce

Add few not main asset in parachain into basket
Submit few operation from basket

Expected Behavior
Submitting status can be changed on success or failure
Actual Behavior
Unending submitting status.
This will cause transactions that have already been sent to not be deleted from the Basket.
Environment
https://github.com/novasamatech/nova-spektr/commit/f3287cf5439fb28de44822ff6fb2a64a18f87b22

Status verification: ✅

repo: novasamatech/nova-spektr


text:
Governance. The referenda list won't update until app will restart

Description
When user run Spektr app and load the Governance page, this list won't be updated.
Cases when data is updated:

Create new referenda
Finish ongoing referenda

Steps to Reproduce

Go to Governance page
Switch to test network
Create some test referenda

Expected Behavior
Spektr should subscribe on storage, when new referenda is created it should appear in the list.
Actual Behavior
Referenda list won't be updated until restart of the app
Environment
https://github.com/novasamatech/nova-spektr/commit/285a866ace2a82902e4c1017492b05e98a657e6f

Status verification: ✅


text:
Remove allocations by serialize

I remembered that there was a last part of the code that still had unnecessary allocations. The solution was a bit complicated because I wanted to keep the main code clean. No changes to the user-facing API.

This is neat! I don't have super strong opinions about this, so I will defer to you on the best practices here!

authors: daxpedda kevinlewi

repo: novifinancial/voprf


text:
fix: center small graphs

Center-aligns graphs in 1:1 mode.  Fixes #110.

Last time I tried this, content got clipped (I think large trees)

Last time I tried this, content got clipped (I think large trees)

Are you referring to this PR or something else you/I tried previously?
This diff should only change the behavior in the specific case of 1:1 zoom where the graph is known to not need scrollbars.

authors: broofa fregante

repo: npmgraph/npmgraph


text:
added email to output for vkontakte

By some reason when 'email' scope is asked for vkontakte, email is not in output of 'getUser'. But it is returned when you get token. Given it is only way to get user email from vkontakte, at least for now, it makes sense to add email to output.

Can you address any issues that came up with Travis CI?  It failed and I can't check it out right now.
Thanks,

Sorry, fixed it

Thanks for your contribution :-)

repo: nraboy/ng-cordova-oauth


text:
Is "AF2 initial guess" supposed to have access to the standard AF2 databases?

Hello,
My installation seems to be running as expected and it produced some beautifully-folded peptide binders with low pae_interaction scores for my project. My question is how is the "AF2 initial guess" pipeline able to generate a predicted structure without access to all the same databases that are normally required to run AF2? i.e.:
    bfd/                                   # ~ 1.8 TB
    mgnify/                                # ~ 64 GB
    params/                                # ~ 3.5 GB
    pdb70/                                 # ~ 56 GB
    pdb_mmcif/                             # ~ 206 GB
    uniclust30/                            # ~ 87 GB
    uniref90/                              # ~ 59 GB

Did I miss an important part of the installation? Is the predict.py script somehow running the full AF2 installation on my machine? Or am I just ignorant to how your implementation of AF2 works?
I just want to make sure that my dl_binder_design installation is configured properly before I put too much trust into the pae_interaction scores and spend money on peptide orders.
Thank you,
Robert Szabla

I have the same question.

Thank you, that is very helpful!

authors: johnny-rodriguez rszabla


text:
Usability fix: Allow Return Key in export dialog

very simple but anoying for a keyboard user :)

Thanks :)

This was one of the annoying things I did not yet get around to tackle when working on #314. Thanks for trying to fix this!
However, it does not work in Chromium for me, where it even breaks for mouse users: Starting at an URL with an empty query component, clicking "Export route" in the export dialog redirects to /?format=gpx#....
(In Firefox the issue does not manifest itself, since there the /?format=gpx request is listed as "blocked".)

I added a event.preventDefault() to stop the default submit action and it seems to fix the download and ?format=gpx URL query in Chromium.
Don't know why the return key does not work in Chromium. From a quick search the criteria for implicit submission is a form tag and non-submit buttons having a type="button" attribute which both seems to be the case.

@nrenner Thanks, working great now.

authors: bagage nrenner rkflx tbsmark86

repo: nrenner/brouter-web


text:
[nrf fromtree] logging: Changed log template to consider default log …

…level
Added using LOG_DEFAULT_LEVEL kconfig option in log module template for assigning the default log level to newly created module instance, instead of assuming always INFO level.
Signed-off-by: Kamil Kasperczyk kamil.kasperczyk@nordicsemi.no
(cherry picked from commit 6fe83ea810ca35ca21bc3bc460dbc0fde332a559)

Do you want me to cherry-pick it in #1096?

Hmm, if there is no problem with that, it's always nice to have it on main branch sooner than later. However I assumed that it will not go in, until upmerge ends, as it's not a bug fix.

@jfischer-no btw I don't know if sdk-nrf CI will pass on this PR, so maybe let's wait to not break upmerge PR unnecessarily.

repo: nrfconnect/sdk-zephyr


text:
Custom Session Handler with predis

Since it is not possible to setup the native PHP Session handler to write to remote redis master and read from local redis slave (https://github.com/phpredis/phpredis/issues/646), would it be possible to adjust your example (https://github.com/nrk/predis/blob/v1.0/examples/session_handler.php) for such scenario? Thank you.

Since Predis\Session\Handler's constructor takes an instance of Predis\ClientInterface you can just configure the client to work in a master/slave setup just like in the following example:
$parameters = [
    'tcp://127.0.0.1:6379?alias=master',
    'tcp://127.0.0.1:6380?alias=slave',
];

$client = new Predis\Client($parameters, ['replication' => true, 'prefix' => 'sessions:']);

$handler = new Predis\Session\Handler($client);
$handler->register();

I've never tested this but it should work.

authors: nrk skynet

repo: nrk/predis


text:
How do I create  '~/.passctlrc

Hello,
I installed the package but I couldn't find any ifno on how to create a '~/.passctlrc file with my Team Manager login details. Please advise.

Hello,
Cool that you found this project! I will provide you with a sample config file asap

Thanks

Hi @SalviaLover,
I have just uploaded a new version 0.1.1 to pypi that include instructions on how to create a configuration file:
You can view it by typing:
$ passctl help

I have also added the same instructions to the README.rst file in the git repository here:
https://github.com/nrocco/team_password_cli
% cat ~/.passctlrc
[passctl]
baseurl = https://passwords.example.com/index.php
username = my-username@example.com
password = xxxxxxxxxxxxxxxxxxx

I am curious if it works for you! Any feedback is highly appreciated.

authors: SalviaLover nrocco


text:
Local packages are newer than in a repository

After switching to flexo I start seeing such warning messages during system upgrades:
warning: python-more-itertools: local (8.10.0-1) is newer than community (8.9.0-1)
warning: python-poetry: local (1.1.10-1) is newer than community (1.1.9-1)
warning: python-traitlets: local (5.1.0-1) is newer than community (5.0.5-2)

The list of warnings changes if I re-run an upgrade. Sometimes it mentions a few packages, sometimes it mentions a lot of packages. I also noticed that so far those warnings appear only for a particular database (extra or community).
As far as I understood, there is currently no way to tell flexo to prioritize more up-to-date mirrors. I can manually select mirrors for the predefined list, but I'm concerned that it'll require manual maintenance in the future.
Is it make sense to amend the logic of selecting mirrors to make it to pay attention to delay values?
UPD:
I checked the mirror (https://mirror.moson.org/arch/) that was used last time and emitted the warnings for community database.
The delay value for it is 0:04 which is quite small already. However, I'm not sure that this particular mirror was used to download community database. In the logs I see that there was an attempt to download community.db.sig file from it, that's why I thought that it was also used to download the database file too.
Any ideas what could be the problem?

For now I set predefined mirrors generated with reflector -f5 --delay 0.1 --sort age -p https. I'll give it a try for some time to see if there are any issues.

Thanks for reporting this!

Is it make sense to amend the logic of selecting mirrors to make it to pay attention to delay values?

I think so. Flexo already makes use of the score attribute to filter out unusable mirrors, but it seems that this does not always exclude out of date mirrors. For example, I have just now (2021-09-26T09:10:00Z) examined the mirrorlist from https://archlinux.org/mirrors/status/json/ and found the following mirror:
{
    "url": "https://arlm.tyzoid.com/",
    "protocol": "https",
    "last_sync": "2021-09-26T03:08:30Z",
    "completion_pct": 0.759493670886076,
    "delay": 114,
    "duration_avg": 0.50416832168897,
    "duration_stddev": 0.2286094097319996,
    "score": 1.0065184574820543,
    "active": true,
    "country": "United States",
    "country_code": "US",
    "isos": true,
    "ipv4": true,
    "ipv6": true,
    "details": "https://archlinux.org/mirrors/arlm.tyzoid.com/1028/"
}

Notice that the score is very decent (lower is better), but the last_sync is already a few hours old, much older than most mirrors. So this is definitely something that needs to be improved in Flexo, it should not be using mirrors that are much older than most mirrors.

I checked the mirror (https://mirror.moson.org/arch/) that was used last time and emitted the warnings for community database.

According to https://archlinux.org/mirrors/moson.org/1566/, this mirror was always up-to-date in the last few days, so I'm not entirely sure what to make out of this. Maybe it was another mirror that Flexo has used to serve the database file. But in any case, using the last_sync attribute for mirror selection is a good idea, so I'm going to implement this and see if it solves the issue.

Thank you for the great tool and for the support!

According to https://archlinux.org/mirrors/moson.org/1566/, this mirror was always up-to-date in the last few days, so I'm not entirely sure what to make out of this. Maybe it was another mirror that Flexo has used to serve the database file.

Yes, in the logs on info level it's not explicitly said from which mirror a particular *.db file was downloaded. But we can see attempts to download *.db.sig files from particular mirrors. That's why I made the assumption regarding the source or community.db. In this context I have a question: shouldn't *.db.sig files be downloaded from the same mirror as corresponding *.db files? Otherwise, I can imagine a situation when two mirrors are not in sync and getting *.db from one mirror and *.db.sig from the other will lead to signature validation error.

In this context I have a question: shouldn't *.db.sig files be downloaded from the same mirror as corresponding *.db files? Otherwise, I can imagine a situation when two mirrors are not in sync and getting *.db from one mirror and *.db.sig from the other will lead to signature validation error.

Yes, that's correct, but it's not a problem at this moment because the mirrors don't provide db.sig files anyway. With or without Flexo, pacman just receives 404's and ignores them when it attempts to fetch those files.
But I need to put some thought into how to improve the downloads of those database files to avoid those kinds of errors. I currently imagine something like a "mirror-stickiness" for database files where it chooses one primary mirror to fetch all database files (and db.sig files), and then doesn't change the mirror unless there's a good reason to do so.

I was also able to reproduce this issue by running pacman -Syu twice. I guess I never had this issue before because I update my system only once a week or so.

authors: Zebradil nroi

repo: nroi/flexo


text:
Passing token from iOS app returns 401

After several days of debugging and even nuking my repo and starting over, I still can't seem to get any authorization with a JWT token I'm passing from an iOS app authenticated with Auth0 Lock. After updating to Knock 1.5, I've narrowed it down to self.from_token_payload not creating a new user. Any assistance would be greatly appreciated!
Here's my repo: https://github.com/brandonmanson/happening-api

It would be helpful if you could provide a failing request spec for this issue.
Also, could you link to the documentation explaining which claims are passed through the payload in your case?
It seems like you expect the payload to contain a field :email. Are you sure this is valid? When reading this, it seems like the user id is passed through the subject claim (i.e. payload["sub"]).

Auth0 allows you to specify claims in this documentation. Here's what my payload looks like:
"payload: {"email"=>"mansonbrandon@gmail.com", "email_verified"=>true, "iss"=>"https://brandonmanson.auth0.com/", "sub"=>"auth0|57843c212f0e1ca74ee67b60", "aud"=>"PGQxxDeLQBkABM5Pg6v1lOZgz2tLcTlU", "exp"=>1469081229, "iat"=>1469045229}"
I was trying to use find_or_create_by with something other than payload["sub"] since I hadn't gotten that to work either.
I don't have a failing request spec yet. I am passing a valid token in an HTTP request from XCode Simulator. I'll come up with a way to mock that.

I'll close this issue as it does not seem related to Knock directly but you can keep posting here and I'll be happy to help once we have a failing spec :)

👍 thank you!

authors: brandonmanson nsarno

repo: nsarno/knock


text:
Cell cleanup: typos

Description:
Corrected some misspelled words in the documentation.
Reviewer(s):
@nskinkel

Coverage remained the same at 35.89% when pulling 21b286fc8d252743620cf83aa9ed475ae88ab1e0 on CivBase:clean_up_typos into 44441db07463583f47d7cc8887d22a0798d7b80d on nskinkel:master.

Thanks! merged.

authors: CivBase coveralls nskinkel

repo: nskinkel/oppy


text:
Optimize session creation

closes #446

Updated PR, and added one more cache for ready to sing tokens.

Reworked session management. Regular cache stores fully signed tokens (with verb, container,etc). Each client has separate field to store the basic session token, with the public key. It will be used to generate fully signed tokens

repo: nspcc-dev/neofs-sdk-go


text:
Hydrogen seems to run, but no output

Hydrogen logs no apparent errors, but fails to produce any results
1: installed hydrogen via Atom GUI
2: installed python kernel with
pip3 install ipykernel --user
3: Start up Atom, see notice about finding python 3 kernel
When attempting to run a single line (a print statement, nothing else in the file) I receive the following in the console log
Hydrogen: CellManager: Cell [start, end]: Array[2]0: Point1: Pointlength: 2__proto__: Array[0] cursor: Point {row: 1, column: 0}
Nothing else shows up. If I add a bubble, I see a lightning bolt next to my code. No amount of left/right-clicking on the bubble yields anything. If I open a standalone tab for the Hydrogen output, I get nothing. If I select "start local kernel" and see my only kernel, python3. I start and I see the following log:
Hydrogen: Selected kernel: Object {display_name: "Python 3", language: "python", argv: Array[5]}
But when I choose to connect to an existing kernel, Hydrogen tells me there are no running kernels.
Fedora 27
Hydrogen 2.1.1
Atom 1.22.1
Edit: Debugging has been enabled since install as well

This looks like a serious misconfiguration with Jupyter. Despite having installed under python3, the kernel is set to call python2 (/usr/bin/python). Updating this manually in kernel.json solved the issue.

repo: nteract/hydrogen


text:
How to set argv parameter in Hydrogen?

Description:
How to set the argv parameter in Hydrogen?
Steps to Reproduce:

Try "Hydrogen:Run" to run the Python source below:

from sys import argv
script, inputfilename = argv
print("The script is called:",script)
print("The input file name is:", inputfilename)

I get the following error:

ValueError                                Traceback (most recent call last)
 in ()
1 from sys import argv
2
----> 3 script, inputfilename = argv
4
5 print("The script is called:",script)
ValueError: too many values to unpack (expected 2)
Versions:
OS: Windows 10
Python: 3.6
Atom: 1.24.0
Hydrogen: 2.2.3
Logs:
NA

I don't think it's possible to set argv, because Hydrogen isn't running through a terminal.

authors: kylebarron zhangliqun


text:
Make keyboard shortcuts configurable

In my current scenario, I often hit cmd+Enter on my mac to create a new line under my current cursor, which conflicts with default strategy to run the current cell in a Hydrogen enabled Python file.
Thus I have to remove the preset keyboard shortcuts in the source, which is quite inconvenient to repeat after every update.

Try adding
'atom-text-editor, atom-text-editor:not([mini])':
  'cmd-enter': 'editor:newline-below'

to your keymap.cson (Atom->Keymap...) file.
This should overwrite Hydrogens default.

FYI for atom 1.10 you need to use .platform-darwin atom-text-editor:not([mini]) as the selector instead. (However, I believe that this is fixed as of atom 1.12)

Fortunately 1.12 is already stable

Great! 🎉

authors: lgeiger nikitakit plumps


text:
Styling

I have a dark website. Is there any way to style the buttons?
I am using the react version of your buttons.
Tried with styled components = no luck, styledButton and add a className='style'
On initial page load it used my styling then the buttons loaded from github and back to white.
They are really cool like they are.
I am using them and withUtterances for GitHub Issues as comments

Same, I'm having trouble styling the buttons too. I ended up putting it in a div and styling the div, it worked for padding styling at least

I'm planning to have theme option for buttons (light & dark). However, there are a few technical complications here.
First, the inability to customize css is by design, and here is why: The buttons use native DOM API to render stuff, the react / vue components are merely just a wrapper. The buttons are encapsulated with shadow DOM or iframe in a way such that it's isolated from main page's render context to prevent css / js pollution on the page. It's also a little bit more privacy friendly this way - that our buttons do not have any tracking code, and it's intentionally made impossible for user to inject click tracking code unless creating a fork and modify the code base. However, keep in mind that GitHub / repository owner can still track button click via GitHub's traffic analytics if page referral is turned on in the browser.
It's not very hard to implement a dark option. However, Apple's OS level dark mode make things complicated. With Apple's leading the implementation, we now have a new CSS media query @media prefers-color-scheme, this means it's possible that for web developers to have the following 4 difference design choice instead simply just light or dark:

light color theme on both light mode and dark mode.
dark color theme on both light mode and dark mode.
light color theme on light mode; dark color theme on dark mode.
dark color theme on light mode; light color theme on dark mode. (high contrast, not sure if this would be actually useful)

I'm drafting an implementation that would also include this new media query support. I'll share more details once I have something.

authors: donaldboulton mr-islam ntkme

repo: ntkme/github-buttons


text:
Inline components

Is there a way to use components inline with []-syntax in Markdown?
Like
# Title

text [component-name] stuff...

More text [.class-name] etc.

[clock] 00:00 - some time

Currently no, but would make sense. This requires an understanding of Marked internals. This thread may offer a solution.

Should be supported on dev branch

authors: nobkd tipiirai

repo: nuejs/nue


text:
🛑 Secret Site BIS is down

In 799f3d4, Secret Site BIS ($SECRET_SITE) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Secret Site BIS is back up in 59d0b2e after 21 minutes.


text:
🛑 Università e ricerca is down

In 51037cf, Università e ricerca (https://www.mur.gov.it/it) was down:

HTTP code: 403
Response time: 433 ms

Resolved: Università e ricerca is back up in cdf6022.


text:
🛑 Agricoltura, Sovranità alimentare e Foreste is down

In e58de8c, Agricoltura, Sovranità alimentare e Foreste (http://www.politicheagricole.it/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Agricoltura, Sovranità alimentare e Foreste is back up in 354c826.


text:
Inclusion and numericality checks

Is it possible to add the checks to schema.rb?

I think you should probably be using structure.sql if you are using the advanced DB features from rein.

authors: jorge-marques nullobject

repo: nullobject/rein


text:
Centre reprojected image on longitude

Is your feature request related to a problem? Please describe.
When using the --nocrop option the created image seems to be centred on where the full-disk images are. I would like to be able to make an image centred on 0° lon even when some images are unavailable.
Describe the solution you'd like
A command line option to centre on a specified longitude.
Additional context
I'm sorry if I missed an option that can already do this, but I could not find one.

Hi Thomas, sure thing. I am in the throes of moving house this month so may not have much time to look at this for a bit.

Thank you

I can't reply to the other issue but.
I tried searching for this github bug but I couldn't find anything, so I assumed it was on my end.  I've never seen this happen before, Would you like me to upload the images here, or is the text enough?

I raised a bug against Github - will see what they say. Can you email the
PNG to @.***? I'm still in the throes of house moving so I may
not be able to look at it until early next month.
On Fri, 19 May 2023 at 08:54, Thomas @.***> wrote:

I can't reply to the other issue but.
I tried searching for this github bug but I couldn't find anything, so I
assumed it was on my end. I've never seen this happen before, Would you
like me to upload the images here, or is the text enough?
—
Reply to this email directly, view it on GitHub
https://github.com/nullpainter/sanchez/issues/95#issuecomment-1553631160,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANG565YFTGIUTUKTBQWD23XG2EBLANCNFSM6AAAAAAXUKOPSA
.
You are receiving this because you were assigned.Message ID:
@.***>

--
Matt Painter
@.***
+64 21 115 9378

I think github has redacted your email address, should I upload the PNG to this issue? Or I could upload them to my website and send a link.  Thank you for helping.

What a nuisance. Sure, your website would work. Thanks!
On Fri, 19 May 2023 at 10:16, Thomas @.***> wrote:

I think github has redacted your email address, should I upload the PNG to
this issue? Or I could upload them to my website and send a link. Thank you
for helping.
—
Reply to this email directly, view it on GitHub
https://github.com/nullpainter/sanchez/issues/95#issuecomment-1553731637,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANG564NRKEZXUW3PWBQP23XG2NU7ANCNFSM6AAAAAAXUKOPSA
.
You are receiving this because you were assigned.Message ID:
@.***>

--
Matt Painter
@.***
+64 21 115 9378

JPG: https://i3b.co/803626547E21B066A6576FF8445D641F.jpg
PNG: https://i3b.co/B6C1EC11D360B63673084C4F8360838C.png

Oh my! Thank you. How bizarre.
On Fri, 19 May 2023 at 10:21, Thomas @.***> wrote:

JPG: https://i3b.co/803626547E21B066A6576FF8445D641F.jpg
PNG: https://i3b.co/B6C1EC11D360B63673084C4F8360838C.png
—
Reply to this email directly, view it on GitHub
https://github.com/nullpainter/sanchez/issues/95#issuecomment-1553735097,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANG56Z6QK5DBUHN7WTILM3XG2OHJANCNFSM6AAAAAAXUKOPSA
.
You are receiving this because you were assigned.Message ID:
@.***>

--
Matt Painter
@.***
+64 21 115 9378

Hi Thomas - have you tried using --lon -180:180?

I could have been sure I tried that, but yes that works. Thank you!
I'm not sure why I missed that before, maybe I was using the --lat option instead?

Perfect!

authors: nullpainter thomasdouwes

repo: nullpainter/sanchez


text:
SDXL model release, any update？

Hi, thanks you for your hard work to implement brushnet in comfyui!!!
The SDXL model has been release, any update to compatible with SDXL model? or I can just download the SDXL model and use it as usual?
https://github.com/TencentARC/BrushNet

I am working on it

@DENG-XC @nullquant
Is it implemented already?

authors: DENG-XC aartikov nullquant

repo: nullquant/ComfyUI-BrushNet


text:
feat: add support for (exact) class attributes

lua-language-server version 3.7.0 introduced exact class attributes.

@numToStr (sorry for the ping). Any chance of getting this merged? :smile:
I've been using this for generating vimdoc, and would love to be able to make use of the new (exact) classes.

repo: numToStr/lemmy-help


text:
compression on the wire and at rest

Summary
There are 2 major limiting factors for increasing the TPS

40 MB/s limitation imposed by JS for a given stream
inbound and outbound network bandwidth of the node on which the pod is scheduled

The easiest way to improve the TPS is by reducing the data transfer bytes by compressing the data.
Compressing the data before writing to the ISB will reduce the data at rest size, which will improve the performance of JS node restart.

https://github.com/klauspost/compress

We didn't see a lot of improvements due to size reduction.

authors: vigith whynowy

repo: numaproj/numaflow


text:
Fix librosa

Seems like the dependencies for librosa are not correct, use this PR to figure out why.

So, librosa is green with this one: it came down to:

when installing ffmpeg you need to use -c conda-forge ffmpeg as the last command before installing librosa -- otherwise the depedencies of ffmpeg such as lame will be superseeed by a higher priority channel and then the packages won't mix.

the libsamplerate0 package must be installed using the system package manager as no suitable / trustworthy conda  package seems to exist.

I've mangled the history to retain a single patch that does what is needed.

I think we may want to get this in anyway, even though it's a big hack, it makes librosa work again so that is 💯 worth it.

@sklam thanks for the merge!

repo: numba/numba-integration-testing


text:
dtypes are not constructible in cfuncs

In [6]: from numba import cfunc, int64

In [7]: import numpy as np
   ...:
   ...: @cfunc(int64())
   ...: def f():
   ...:     dt = np.dtype('int8')
   ...:     return 1

gives
UntypedAttributeError: Failed at nopython (nopython frontend)
Unknown attribute 'dtype' of type Module(<module 'numpy' from '/Users/cpcloud/miniconda3/envs/py35/lib/python3.5/site-packages/numpy/__init__.py'>)
File "<ipython-input-5-47bd311bf29d>", line 5
[1] During: typing of get attribute at <ipython-input-5-47bd311bf29d> (5)

@jit works as expected.

You don't need to construct the dtype in the cfunc. Just construct it outside of the cfunc and use the global variable there.

Note: @jit works "as expected" simply because it falls back to object mode, which will make your code roughly as slow as pure Python :-)

Ah, ok. Thanks!

authors: cpcloud pitrou

repo: numba/numba


text:
SmartArray.init(shape, dtype, order) is broken

As of numba 0.38.1, in the SmartArray(shape, dtype, order) constructor:

where='host' never works
where='gpu' only works if order=None

>>> SmartArray(shape=(2, ), dtype=np.float64, where='host')

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-34-6b2b2d21fe6a> in <module>()
----> 1 a = SmartArray(shape=(2, ), dtype=np.float64, where='host')

~/anaconda3/lib/python3.6/site-packages/numba/smartarray.py in __init__(self, obj, copy, shape, dtype, order, where)
     69         self._host = self._gpu = None
     70         self._host_valid = self._gpu_valid = False
---> 71         self._allocate(where, obj, dtype, shape, _o2s(dtype, shape, order), copy)
     72         if where == 'host':
     73             self._host_valid = True

~/anaconda3/lib/python3.6/site-packages/numba/smartarray.py in _allocate(self, where, obj, dtype, shape, strides, copy)
    170                 self._host = np.array(obj, dtype, copy=copy)
    171             else:
--> 172                 self._host = np.empty(shape, dtype, _s2o(dtype, shape, strides))
    173         else:
    174             # Don't import this at module-scope as it may not be available

~/anaconda3/lib/python3.6/site-packages/numba/smartarray.py in _s2o(dtype, shape, strides)
     36     s2 = _o2s(dtype, shape, order)
     37     if strides != s2:
---> 38         raise ValueError('strides do not correspond to contiguous data layout')
     39     return order
     40

ValueError: strides do not correspond to contiguous data layout

>>> SmartArray(shape=(2, ), dtype=np.float64, where='gpu', order='C')

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-44-7716b217b260> in <module>()
----> 1 a = SmartArray(shape=(2, ), dtype=np.float64, where='gpu', order='C')

~/anaconda3/lib/python3.6/site-packages/numba/smartarray.py in __init__(self, obj, copy, shape, dtype, order, where)
     69         self._host = self._gpu = None
     70         self._host_valid = self._gpu_valid = False
---> 71         self._allocate(where, obj, dtype, shape, _o2s(dtype, shape, order), copy)
     72         if where == 'host':
     73             self._host_valid = True

~/anaconda3/lib/python3.6/site-packages/numba/smartarray.py in _o2s(dtype, shape, order)
     19     for i in range(len(shape), 0, -1):
     20         strides.append(itemsize)
---> 21         itemsize *= shape[i - 1]
     22     if order in ('C', None):
     23         strides.reverse()

TypeError: unsupported operand type(s) for *=: 'getset_descriptor' and 'int'

where='host', order='C' also gives the same error as above.
Additional problems:

dtype is not properly wrapped with the numpy.dtype constructor; e.g. dtype='int32' or dtype=float do not work.
there is no control whatsoever that order must be C, F, or K. One can write order='yourmom' and no ValueError will be raised.

Thanks for the report, I can reproduce most of what you describe. It should be noted that SmartArray was/is experimental, though these construction issues should probably have been caught http://numba.pydata.org/numba-doc/dev/cuda/memory.html?highlight=smartarray#smartarrays-experimental

SmartArray is deprecated as of 0.44 http://numba.pydata.org/numba-doc/latest/reference/deprecation.html#deprecation-of-smartarray. Closing as "wontfix".

authors: crusaderky stuartarchibald


text:
call_conv holds a reference to a different CPUContext object

While working on #8134, I notice on BaseLower that the CPUContext context and context.call_conv.context are not referencing the same object. I believe this is due to context.call_conv being implemented as a @cached_property and not a @property.
https://github.com/numba/numba/blob/0b75a0053a72b2ebd85a7bde4d12c3c010fe13fd/numba/core/lowering.py#L34
https://github.com/numba/numba/blob/0b75a0053a72b2ebd85a7bde4d12c3c010fe13fd/numba/core/cpu.py#L111-L113
The problem that this brings is that both contexts can have the same attribute with different values (i.e. NRT)
Reproducer:
diff --git a/numba/core/lowering.py b/numba/core/lowering.py
index 251851d70..6abe61a14 100644
--- a/numba/core/lowering.py
+++ b/numba/core/lowering.py
@@ -34,6 +34,9 @@ class BaseLower(object):
         self.blocks = utils.SortedMap(func_ir.blocks.items())
         self.func_ir = func_ir
         self.call_conv = context.call_conv
+        print(context is context.call_conv.context)  # False
+        print(context.enable_nrt)  # True
+        print(self.call_conv.context.enable_nrt)  # False
         self.generator_info = func_ir.generator_info
         self.metadata = metadata
         self.flags = targetconfig.ConfigStack.top_or_none()

I wonder if the root cause of https://github.com/numba/numba/issues/8036 is a similar problem.

I have a fix for it in #8134, which just replace @cached_property by @property.

Thanks for the pointer - turns out that doing that alone doesn't resolve #8036. Maybe there's something else going wrong too, but it feel like that's a step in the right direction.

@guilhermeleobas is this now fixed through #8134 ?

Yes!

Yes!

Thanks for confirming, closing!


text:
Support slicing tuples with a constant slice

This allows e.g. some_tuple[0::1] but not some_tuple[x:y]. Note this works for all kinds of tuples (homogenous and heterogenous).

Merge conflicts with master now...

authors: pitrou seibert


text:
Fix IndexError when accessing the "-1" element of dummyarray

This PR fixes the following error:
>>> import numpy as np
>>> from numba import cuda
>>> ary = np.arange(10)
>>> d_ary = cuda.to_device(ary)
>>> d_ary[-1]
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/leofang/conda_envs/mpi4py_dev/lib/python3.6/site-packages/numba/cuda/cudadrv/devices.py", line 225, in _require_cuda_context
    return fn(*args, **kws)
  File "/home/leofang/conda_envs/mpi4py_dev/lib/python3.6/site-packages/numba/cuda/cudadrv/devicearray.py", line 497, in __getitem__
    return self._do_getitem(item)
  File "/home/leofang/conda_envs/mpi4py_dev/lib/python3.6/site-packages/numba/cuda/cudadrv/devicearray.py", line 507, in _do_getitem
    arr = self._dummy.__getitem__(item)
  File "/home/leofang/conda_envs/mpi4py_dev/lib/python3.6/site-packages/numba/dummyarray.py", line 225, in __getitem__
    dims = [dim.__getitem__(it) for dim, it in zip(self.dims, item)]
  File "/home/leofang/conda_envs/mpi4py_dev/lib/python3.6/site-packages/numba/dummyarray.py", line 225, in <listcomp>
    dims = [dim.__getitem__(it) for dim, it in zip(self.dims, item)]
  File "/home/leofang/conda_envs/mpi4py_dev/lib/python3.6/site-packages/numba/dummyarray.py", line 76, in __getitem__
    raise IndexError
IndexError

Thanks for the PR. Seems like CI is currently failing to due flake8 violations https://dev.azure.com/numba/numba/_build/results?buildId=3744&view=logs&j=687448c2-4012-5c9e-43b3-3520c9635a30&t=dba03d4a-57a7-5e59-a47a-79fe530f297a&l=78

@stuartarchibald I silenced it in ba4476a. Is it OK?

@stuartarchibald I silenced it in ba4476a. Is it OK?

Could the result just not be assigned to a variable opposed to silencing the error about not assigning the result, or is that not possible here?

We could do either of these if you prefer not to do noqa:

Have a global placeholder a = None declared on top, and overwrite it repeatedly, or
Take away the assignment and just have a single line like darr[10]

I usually don't use the second approach, but it does the job.

We could do either of these if you prefer not to do noqa:
1. Have a global placeholder `a = None` declared on top, and overwrite it repeatedly, or

2. Take away the assignment and just have a single line like `darr[10]`

I usually don't use the second approach, but it does the job.

I usually just do 2. Is there some problem you know with this?

I usually just do 2. Is there some problem you know with this?

Pure aesthetic reason 🤣 Let me force push.

@stuartarchibald Done. Looks like flake8 is happy now. PTAL, thanks!

btw, any chance this fix could go into Numba 0.48 RC?

Passed internal CI

Thanks for the update, the patch is now failing flake8, https://dev.azure.com/numba/numba/_build/results?buildId=3841&view=logs&j=90fddb40-a9bd-5e17-109d-19517de6dba5&t=a765cc85-fe2d-5487-1056-2dc2d92f2a0a&l=78
You may find it useful to conda install flake8, then run flake8 <files you changed> to check whether it passes locally before pushing. There are also pre-commit hooks available, setup instructions are here:
http://numba.pydata.org/numba-doc/dev/developer/contributing.html#coding-conventions

Sorry, I am traveling and do not have access to my laptop, so I edited on the web interface. I will fix it when I can.

PTAL

btw, I rechecked this on the internal CUDA CI, and it passes.

Thanks guys!

authors: leofang seibert stuartarchibald


text:
use sys.executable to retrieve python path

Fixes #760 and #759.
Thanks!

Thanks!

authors: MaanasArora christianp

repo: numbas/editor


text:
add (users): nikarashihatsu.github.io

This PR add my personal site to this https://blowfish.page/users/ page.

Sure thing, let me change that for a moment.

repo: nunocoracao/blowfish


text:
codecrunch

hi!!! last 20 mins for us to submit and codecrunch as usual failing on us :((( will we be penalised for late submission due to this issue :((
also I would like to say that we've come a long way struggling with this project!!! so good job everyone!!!

Have the same issue too!

hi, have you tried logging out of codecrunch and logging back in? it worked for me

hi! if you really need to maybe can email the prof!

authors: ca-leb jodytng rexlionz t11-20


text:
Exercise 4 level 5 accessing Grid2D get() method

hi, I am trying to implement the move() method, but I got the "cannot find symbol" Error. can anyone explain to me why is this the case? for information, I implemented my constructor for puzzle with this.puzzle = new Grid<Integer>(list, n)

If you did the previous levels correctly, your implementation should be such that Puzzle has-a Grid2D. The Grid2D Class is the one with the .get() method, not Puzzle, which is why when you try to call .get() on puzzle, it can't find the method.
If you want to use p.get(), you would need to define a method in Puzzle itself that calls Grid2D's .get() method, although I think that is not necessary since you only need to create move() and isSolved().

If you did the previous levels correctly, your implementation should be such that Puzzle has-a Grid2D. The Grid2D Class is the one with the .get() method, not Puzzle, which is why when you try to call .get() on puzzle, it can't find the method.
If you want to use p.get(), you would need to define a method in Puzzle itself that calls Grid2D's .get() method, although I think that is not necessary since you only need to create move() and isSolved().

sorry I still don't really understand. Is this this.puzzle = new Grid<Integer>(list, n) not correct? what am I missing?

this.puzzle = new Grid<Integer>(list, n) is fine.
But in your jshell, p refers to an instance of Puzzle and not the property puzzle that each Puzzle instance is expected to have. So what you're doing here is Puzzle.get(), whereas what you probably wanted to do is Puzzle.puzzle.get().

Use Puzzle.puzzle.get() since the get() method is defined in Grid2D class and you have assigned it to this.puzzle. I would suggest to rename this.puzzle to this.grid2D, etc to differentiate since it may cause some minor confusion. It is also more intuitive to name it to this.grid2D since the Puzzle has-a Grid2D instead of Puzzle has-a puzzle.

alright thx! I see where's the problem


text:
Mock PA2 testcase 5_1

anyone knows what could be the error here?

Hey guys, i tried to reduce it in my calculations, all good thanks!


text:
[T2A3][T16-A2]Chan Khai Ern Edwin

Ready for Review

Sorry for the delay, been delayed by CNY stuffs

closing this PR as there is nothing much to review!

authors: EdwinChanSingapore okkhoy


text:
[T4A3][F12-A3]Wang Yu

…rent class called Contact.

@WangYu-g
Some comments added. Please ack & close the PR after reading comments.

authors: WangYu-g okkhoy


text:
[W5][W14-1]Chang Lei

Change the find command to the case-insensitive search in AB2, with user guide, JUnit tests and I/O tests updated.

Consider incrementally coding and having more commits
Good attempt at refactoring and removing unnecessary lines

authors: jeraldtsy lallanachang