fop: 2.6 -> 2.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/fop/versions.
meta.description for fop is: "XML formatter driven by XSL Formatting Objects (XSL-FO)"
meta.homepage for fop is: "https://xmlgraphics.apache.org/fop/"
meta.changelog for fop is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/wlv1q60l6c7iwsm8d6bdyj5p5zm3sz4r-fop-2.7/bin/fop had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 2.7 with grep in /nix/store/wlv1q60l6c7iwsm8d6bdyj5p5zm3sz4r-fop-2.7
found 2.7 in filename of file in /nix/store/wlv1q60l6c7iwsm8d6bdyj5p5zm3sz4r-fop-2.7

Rebuild report (if merged into master) (click to expand)

50 total rebuild path(s)

28 package rebuild(s)

28 x86_64-linux rebuild(s)
22 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
asciidoc-full
asciidoc-full-with-plugins
clevis
disorderfs
fop
hal-hardware-analyzer
igraph
kakounePlugins.rep
luksmeta
python310Packages.explorerscript
python310Packages.igraph
python310Packages.kmapper
python310Packages.skytemple-dtef
python310Packages.skytemple-files
python310Packages.skytemple-ssb-debugger
python39Packages.explorerscript
python39Packages.igraph
python39Packages.kmapper
python39Packages.scikit-tda
python39Packages.skytemple-dtef
python39Packages.skytemple-files
python39Packages.skytemple-ssb-debugger
rep
skytemple
spring
springLobby
tang
udiskie

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/wlv1q60l6c7iwsm8d6bdyj5p5zm3sz4r-fop-2.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fop https://github.com/r-ryantm/nixpkgs/archive/fe5ca118bf107c61f2843c09bd3d5437bbd57257.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/wlv1q60l6c7iwsm8d6bdyj5p5zm3sz4r-fop-2.7
ls -la /nix/store/wlv1q60l6c7iwsm8d6bdyj5p5zm3sz4r-fop-2.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @bjornfor for testing.

building spring manually after applying this PR works, so I guess spring is a false positive from nix-review


Fix isAvailableSchedule function

Fix root cause for clearing cart items when selecting timeslot

Thanks for the pull request. Can I have some understanding as to what this fixes, with an error trace if possible?

The function was incorrect (always return as FALSE) when start_date & end_date are empty (null or '0000-00-00 00:00:00')
Timeslot changed will remove an item in cart if there is not match with any mealtimes.
By default, menu is associate with no mealtime.


fix(i18n): update Chinese translation of apis and microservices projects

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

@ZhichengChen I rebased due to Travis failing.


⚠ Bofrost.nl has degraded performance

In 7b8c4a8, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 2797 ms

Resolved: Bofrost.nl performance has improved in 0169cc4.


Update api-overview.md for zh

This PR is going to fix #18809 .
update api-overview.md for zh.
/language zh

Thanks for the PR. Please hold the change when there are ongoing revision to the English content. The reason is that the extensions group is gone in 1.18.
/hold

@huchengze Please help rebase on the master and see if there are different texts to update/translate?

Sorry，I‘m a little busy these days.
I will close this pr and finish it in the end of this month.


Still draws the line if there are no replies

Great library!
But if the root comment has no replies, the lib will draw the line to the next root comment. But if there are replies, the line end with the last reply, won't draw to the next root comment.
I think if there are no replies, it shouldn't draw the line at all.
Thanks.

agree

Did you find any solution to this issue?

Did you find any solution to this issue?

I updated the root_comment_widget myself, passed in a boolean value called hasReplies, if it is true, do not draw anything inside paint function of RootPainter.
This project seems out of maintaining, so I didn't bother submitting the PR.

i did a similar thing and replaced the line color to Transparent when there's no replies and that worked

        treeThemeData: TreeThemeData(
          lineColor: Colors.green[500]!,
          lineWidth: comment.replies!.isEmpty ? 0 : 3,
        ),

@westlinkin
@raduungurean
hi, do you know how can we connect root avatars together? like with another timelines


http3: Fix a crash on READ_READY event for QPACK

I think this fixes #11115. It seems like #10851, which is nothing wrong, woke up a sleeping bug.
The event handler in QPACK uses the second parameter data as a pointer for VIO, but if you look at the event handler in Http3App, it uses data->cookie instead. This is probably the reason the VIO has odd values.
https://github.com/apache/trafficserver/blob/36ff81f909aa026c0cedd021d1a6d3b4052ef716/src/proxy/http3/Http3App.cc#L131-L153

[approve ci rocky]

[approve ci centos]

[approve ci autest]

[approve ci autest]

[approve ci centos]

[approve ci autest]

Cherry-picked to v10.0.x


Bump version to 0.2.97

I'd like to get a new release published, since I want to make use of https://github.com/rust-lang/libc/pull/2228 in https://github.com/rust-analyzer/rust-analyzer/pull/9204

r? @JohnTitor
(rust-highfive has picked a reviewer for you, use r? to override)

Could you also update the libc dependency version on libc-test?

Sure, done

Thanks! @bors r+

:pushpin: Commit 83842baa85995705195f6922aba0db513840cc28 has been approved by JohnTitor

:hourglass: Testing commit 83842baa85995705195f6922aba0db513840cc28 with merge 1c66799b7b8b82269c6bff0eab97d1a30e37fd36...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing 1c66799b7b8b82269c6bff0eab97d1a30e37fd36 to master...

Published on crates.io: https://crates.io/crates/libc/0.2.97

Great, thanks!


merge does not merge book-ended features

As opposed to the documentation, my book-ended features are not merged without any error using bedtools v2.25.0.
For example using following file
$ cat ~/merge_test.bed
chr2L	7529	8116	FBgn0031208:1	.	+
chr2L	8117	9484	intron_FBgn0031208:1_FBgn0031208:2	.	+

I get
$ bedtools merge -s -i ~/merge_test.short.bed
chr2L	7529	8116	+
chr2L	8117	9484	+

although it should be
chr2L	7529	9484	+

Also, as noted already here, the strand is reported without using -c and -o.

Note that since BED is zero-based, half-open, those intervals are not book-ended.
These are:
$ cat ~/merge_test.bed
chr2L   7529    8116    FBgn0031208:1   .   +
chr2L   8116    9484    intron_FBgn0031208:1_FBgn0031208:2  .   +

That explains a lot of course. Thanks!


mix.version() adds 'public' to path and fails

Laravel Mix Version: 2.1.11
Node Version: 8.9.4
NPM Version: 6.1.0
OS: Windows 7 x64

Description:
mix.version() seems to be confusing paths to sass files for mapping. It is adding 'public' to the middle of the correct path to the scss file.
In my webpack.mix.js file (project root):
let combinedFile = __dirname + '/resources/assets/sass/combined.scss';

mix.combine(sassFiles, combinedFile);

mix.sass(combinedFile, 'public/css/app.css');

mix.version(['public/js/app.js', 'public/css/app.css']);

Results in:
Error: ENOENT: no such file or directory, open 'E:\Projects\laravel-slab\public\resources\assets\sass\combined.scss'
Commenting out the version() seems to resolve the issue. But I would like to have the file versioned.
Steps To Reproduce:
I'm not sure how to reproduce this. I am combining the main sass file, and sass files from sub-directories (working on a modular cms). The paths in these sub-directory files are relative as if they were in the same folder as combined.scss However this seems unrelated, as it's not having an issue locating node packages, it's having an issue finding the file it literally just got done processing.
Any ideas what might be going on here?

I had the same problem!
Check out this comment on issue #1333. It worked for me :)


EIP Bot should not require changes to the EIP file to modify EIP asset files

See https://github.com/ethereum/EIPs/pull/5213
Example error:

file assets/eip-4907/contracts/ERC4907.sol is associated with EIP 4907 but there are no changes being made to corresponding EIP itself. To assure that the change is authorized by the relevant stake-holders, you must also make changes to the EIP file itself for the asset changes to be eligible for auto-merge

Question:  is there a determined list for "normal rules for changing files in assets directory" ?

is there a determined list for "normal rules for changing files in assets directory" ?

No, except for "the EIP has to be modified too."

To make this clear:
If any EIP-asset file is modified the EIP-Bot fails and requests authors to modify EIP-File itself ?
Is there some one else working on it ?

No, nobody else it working on it. Go ahead!

Roger that, I will take a look on it..

Nope, I'm not working on this.


GetCurrentWindow in BrowserWindow X effecting BrowserWindow Y

So I have a main window which starts when the app begins, then you can load many more windows from each window.
Anyway the problem comes in, if I was to call remote.getCurrentWindow().toggleDevTools() on window Y, the dev tools would appear on window X.
There seems to be very little information on how to handle multiple first class windows, so could anyone let me know how the best way to handle each windows scope?
(I have thought about trying to keep track of the browser window instances somewhere, or just having ipc events on windows but cannot find out much about if IPC is isolated to window/renderer or are global)
Any info around this area would be great.

Can you provide a full example? This is related to where and how you call remote.getCurrentWindow, and we didn't see similar issues in Atom which uses Electron and creates multiple windows.

Here is a overly simplified example of what is happening, it also shows another issue I run into where it deletes the window context of another window.

Here is the example file if you want it:
https://dl.dropboxusercontent.com/u/98595151/menu-test.rar

You are creating a new application menu in your second window, so the callback of "Toggle DevTools" is ran in the second window even though you click it in the first window.
You should either only set menu for current window in each page, or set the menu in the main process. And the latter way is preferred under most cases.

Sorry to be a simpleton but what do you mean exactly? I need each window to have a menu and function as its own application, so are you saying that if I set it once in the main application then never set it again it will carry over or something?
A brief code example of how to best handle this scenario would be very much appreciated.

setApplicationMenu is equivalent to call setMenu for every window on Windows, in your case you can call remote.getCurrentWindow().setMenu instead of setApplicationMenu to fix it.

Thanks, will give this a try.

@zcbenz, according to the docs setMenu is only available on OS X, is that incorrect?

It did seem to work for me, and I am on windows.

@zcbenz, according to the docs setMenu is only available on OS X, is that incorrect?

We made a mistake there, it is actually not available on OS X.

So if setMenu doesnt work on OS X what is the desired approach in that environment to achieve the same thing?

I reckon you can replace the application menu when ever a window comes into focus.

ok so do set application window every time the window is selected?

Yes. I would probably just hold menu objects on the browser window that uses it, so when you get a browser-window-focus app event you just call Menu.setApplicationMenu(window.mainMenu).
If you encounter some performance issues, you can optimize the code to switch menus only when needed, so if your window lost focus because the user opened another app, coming back to the same window, won't switch the menu (you can do this easily by keeping track of the window id).


Fix autodoc on rtd

Addresses #122
Test build

@mpacer merging so that the docs are displaying autodoc. We can finesse the formatting in a future PR.


Auto branch docs

Closes #3396
NOTE: I have made version changes (2.2 --> 2.3)  to the .RSTs in 3 places :

docs/iris/src/_templates/index.html
docs/iris/src/_templates/layout.html
docs/iris/src/whatsnew/index.rst

In future, to make this scheme work properly (so branch docs are properly readable, especially the whats-new), we really need to move these on just after a new release, when we bump the version in master.
That is, exactly as we do to the iris version string.

Thanks @lbdreyer , I hope I've addressed those points now.
Mostly : I have added a 'special' mode to the aggregation as suggested, and it produces a temporary whatsnew page with a temporary non-version name.
See : https://scitools-docs.github.io/iris/auto_branch_docs/whatsnew/index.html

Looking good! Thanks @pp-mo !


Fix some z-fighting for lava generator, basic generator, and steam addon

Slightly deflated some block shapes to fix z-fighting issues.

Great improvement, those were also on my todo list. Really appreciate it


README does not contain steps to actually run the driver

While the README does tell you how to navigate to the build directory in step 6, it does not inform the user how to run the binary. However, step 7 assumes the user has ran the binary somehow.
The code suggests that there is some calibration procedure too. That isn't described either.

Done, thanks! Calibration is described in the application


Weighted search not working?

For example, if I pass keys like this:
keys: [{ name: 'title', weight: 0.3 }, { name: 'parent', weight: 0.6 }, { name: 'grandparent', weight: 0.9 }]

I get this error:
fuse.js:238 Uncaught TypeError: path.indexOf is not a function

It looks like path is receiving a full object instead of a string.

Which version of Fuse are you using? I'm unable to replicate this with the current version 😄

I'm so sorry. I hadn't updated my dependancies. Works great with the latest.

I am getting the same error on version 3.2.0.

Can you provide an example script so we can reproduce the error?

While putting together a snippet I discovered the issue. When only providing a list of keys and one of the elements in the array is a function, fuse will not throw an error. When providing keys and weights however fuse will throw an error if one of the keys is a function.


Multiple project Creation, Status not change after creation project

While creating multiple projects, change status to Not Started, but after create project status not change, project goes to Lead

After creation of project, this error message appeared

The issue over here is just that projects are created on lead even if user has specified some status while creating job.

 @mitulpanch

Background-color changed for Status Change
Status not Applied to Projects are not coming was the Error. User must apply status to project in order to Created project with desired Status


.net driver NEVER reuses a connection

Hello all, my first post.
I am using neo4j to create a framework based on nodes and relations.
Opened my system today for a demonstration, for the 1000th time.
But this time the .net driver refuses to ever reuse a connection. So after 100 queries, the pool
has no available connections, and it is full, so after a while it says that:
"Failed to obtain a connection from pool within 00:01:00"
This is even a problem if I use:
WithMaxConnectionLifetime(TimeSpan.FromSeconds(10)) in the driver settings.
The connections just never become available again. Something locks them.
I can use both Session.Run and Transaction.Run with using. No difference. I use the simple, not async
version because it was easier to find info about that when I wrote the code.
I have checked the neo4j browser, and yes, all 100 connections are there. And stays there until I have closed my c# application. I can change the number of connections in the pool, to say 10, but the behavior stays the same. But after 10 connections, that is.
The code has been working for years. And today it decided that, nope, won't work.
( of course during a presentation )
I have updated the driver and neo4j server to latest version.
I didn't change anything. It worked yesterday. But not today. And I have restarted everything. Nope. Won't work.
There is no exception, the queries that does get through work as they should. The database is not corrupted. Everything works as it should in the Neo4j browser.  The database log says nothing.
Any ideas why connections gets magically locked would be much appreciated.
/eric

Hi @vikingivar thanks for reporting this issue.
Which version are/were you using, just trying to work out when it stopped working for you.
Which server version & configuration (i.e. cluster)?

Hi @thelonelyvulpes,
Today I started my computer and ... it worked again!, but... there is now another bug. Should be related.
When my program execute:
using var tx = session.BeginTransaction();
there is an StackOverflowException thrown. And alas, not much info there:

So one wonder why these weird errors creep up.

That's very odd, Do you have loop in the transaction's metadata?


Hardening artifact collection on multiple nodes

Hardened artifact collection so that if collection fails on one node, collection is still attempted on the other nodes.

Second round of changes are to the Pipelines YAML. Specifically:

Consolidated the three different steps for regular, no storage acct, and byovnet deployments into one.

These are controlled with new Pipelines variables (VFXT_DEPLOY_NEW_VNET and VFXT_DEPLOY_WITH_BLOB).
The new deploy step just uses true/false combinations to run the appropriate variant.
The false/false combination is not yet implemented (no storage account AND byovnet).

Added a "check for common error messages" step after the deploy step (to run only if deploy failed).

This step will succeed if a common error message was found.
This step will fail if a common error message was not found (i.e., need to look into the error further).
First set of common error messages are for quota and GUID hash conflict.

Cleaned up descriptions a bit to prefix them with something like "TEST," "CHECK," etc.

PR build failed on purpose to check that the quota error message was being picked up in the common error messages step. New build manually initiated.

This passed: https://dev.azure.com/averevfxt/vfxt-github/_build/results?buildId=1217
Submitting.


Visible steps

Add a parameter to the main stepper, that will make all steps open/visible but disable those that are not completed.
For example:
There are 5 steps:
1st is visible and editable -> 2nd, 3rd, 4th, 5th are visible but disabled
After clicking next -> 1st and 2nd steps are visible and editable but others are disabled and etc.

 This would be useful to you?


raftstorev2: avoid unnecessary snapshot after split

What is changed and how it works?

Issue Number: Close #14787
What's Changed:

raftstorev2: avoid unnecessary snapshot after split

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Integration test

Release note
None

What about follower is pretty slow? Without the PR, the splitted regions can begin to work after received snapshot from the leader even the parent region on its store has not applied the snapshot.
Maybe we can set a deadline?

So the idea is to block the first prevote or vote message if it's uninitialized. But it essentially would defeat the purpose of the v2's improvement that the un-initialized peer can still do the election so that the new region can serve traffic quickly (reduce the missing region)
Is it possible to delay the snapshot sending in leader side. The key improvement of this PR is to reduce the cross AZ traffic. If we can somehow avoid the snapshot sent in leader, then we can keep the current design's improvement.

/merge


goreleaser: 1.10.3 -> 1.11.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/goreleaser/goreleaser/releases.
meta.description for goreleaser is: "Deliver Go binaries as fast and easily as possible"
meta.homepage for goreleaser is: "https://goreleaser.com"
meta.changelog for goreleaser is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.11.0 with grep in /nix/store/czpm2xgf69cycnl6ms31cqlghm5r3lb8-goreleaser-1.11.0
found 1.11.0 in filename of file in /nix/store/czpm2xgf69cycnl6ms31cqlghm5r3lb8-goreleaser-1.11.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
goreleaser

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/czpm2xgf69cycnl6ms31cqlghm5r3lb8-goreleaser-1.11.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A goreleaser https://github.com/r-ryantm/nixpkgs/archive/36e7d6e432f81eca5e3911f8e88c184f2127a9bf.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/czpm2xgf69cycnl6ms31cqlghm5r3lb8-goreleaser-1.11.0
ls -la /nix/store/czpm2xgf69cycnl6ms31cqlghm5r3lb8-goreleaser-1.11.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

goreleaser

Maintainer pings
cc @c0deaddict @endocrimes @sarcasticadmin @techknowlogick for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/goreleaser/goreleaser/releases.
meta.description for goreleaser is: "Deliver Go binaries as fast and easily as possible"
meta.homepage for goreleaser is: "https://goreleaser.com"
meta.changelog for goreleaser is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.11.1 with grep in /nix/store/1yyni3vvpknch7gk240bl2vf055sn0a6-goreleaser-1.11.1
found 1.11.1 in filename of file in /nix/store/1yyni3vvpknch7gk240bl2vf055sn0a6-goreleaser-1.11.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
goreleaser

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1yyni3vvpknch7gk240bl2vf055sn0a6-goreleaser-1.11.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A goreleaser https://github.com/r-ryantm/nixpkgs/archive/46b41ad3f55600cad420721dd89daef3b0fec2cf.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1yyni3vvpknch7gk240bl2vf055sn0a6-goreleaser-1.11.1
ls -la /nix/store/1yyni3vvpknch7gk240bl2vf055sn0a6-goreleaser-1.11.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

goreleaser

Maintainer pings
cc @c0deaddict @endocrimes @sarcasticadmin @techknowlogick for testing.

Duplicate of: https://github.com/NixOS/nixpkgs/pull/188989


Lazy-load menu items on launch, add exception handling

Rather than loading & checking that every class in the menu is a valid EuroPiScript, only check the script that's been selected from the menu.  Rely on CI to make sure the main branch only ever contains valid scripts, and if the user makes changes then the onus is on them to make sure they're writing good code.
The main rationale for this is to reduce the memory footprint created by the menu. Some classes, e.g. PamsWorkout, are pretty big. By only loading & checking what's actually going to run we should get a comparable amount of safety (e.g. if the script isn't going to run, why do we care if it's valid or not?), but should also avoid running out of RAM.
Some very basic exception handling has also been added to show an error message if the selected script is invalid (should never be seen by a normal user, but may be useful for developers), as well as a kernel panic/BSOD style error if the selected script exits with an unhandled exception. This should (hopefully) be preferable to an exception just making the module freeze up with no explanation. Not it freezes up with an explanation, which is at least mildly more helpful.

No problem!


Does this liberary contain context parallel to train long-context models?

We're looking into it and this might be added in the future ! For now xformers library allows context parallel only for forward pass so we're still missing the necessary primitives for backward pass. @lw

Delegating this issue to future versions of xformers, but feel free to reopen.


UI improvements

[x] Alerts
[x] Navbar
[x] Logo
[x] favicon
[x] Form validation
~[ ] name ellipses~ No need for ellipses expansion and tooltip since dataset names seems to be wrapping around nicely
[x] summary stats
[x] toasts
[x] Button loading
[x] non-white background color of the tool group buttons on hover

:rocket: Issue was released in v0.1.0 :rocket:


Default argument of closure is inferred as throwing even if the default value doesn’t throw

Consider the following example:
func foo(body: () throws -> Void = {}) rethrows {}

foo(body: {})
foo() // error: Call can throw but is not marked with 'try'

IMO the second call to foo should also be valid because the default value of body does not throw and thus foo the call to foo also shouldn’t throw because foo is marked as rethrows.
rdar://104781983

cc @etcwilde


whois 5.2.11

Merged in https://github.com/Homebrew/homebrew-dupes/commit/c88728ed068ea17e19f05f5b93f6d4760615afbe. Thank you for another contribution to Homebrew @Habbie! :smiley_cat:


Move Django files to reflect the suggestions of the Django documentation

Resolves #51

Move the Django app from /var/www/html to /home/apel_rest_interface as per the documentation.
Move static files from var/www/html/static to /var/www/static as per the documentation for non app specific static files.
Remove site-packages from WSGIDaemonProcess in the apache conf as the path is wrong and no needing according to the documentation.

This PR also updates Dockerbuild, the docker/run_on_entry.sh script and the developer documentation to refelct these chnages.

Have you checked that there are no permissions issues with the new location?

yep, The docker container seems to run fine.


Typo on line 89

OVERRIDE_NUMBER_MATCHING_WITH_TOP  -> OVERRIDE_NUMBER_MATCHING_WITH_OTP

@mjsantani

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#please-close @AlexCesarini thanks so much! This fix is complete via the private repo.


Support Squirrel Update Endpoint

According to Squirrel documentation https://github.com/Squirrel/Squirrel.Mac#server-support, we need to provide Update endpoint with correct response.
Acceptance Criteria

Support /version/update endpoint
Response Squirrel payload

Done #25


UsePythonVersion@0 does not configure correctly Python 3.4 and 3.5

2018-09-17T20:14:15.5777113Z py -3.5 -c "import sys; print(sys.executable)"
2018-09-17T20:14:15.6419862Z ##[command]"C:\Windows\system32\cmd.exe" /D /E:ON /V:OFF /S /C "CALL "D:\a\_temp\15c4da66-e82e-4a75-ab7d-6d7f0803f389.cmd""
2018-09-17T20:14:16.2417178Z Requested Python version (3.5) not installed
2018-09-17T20:14:16.4035016Z ##[error]Cmd.exe exited with code '103'.

See tests inside https://github.com/tox-dev/tox/pull/995.

That’ll be the lack of PEP 514 registry entries. Python 3.6 works because it’s finding a different install than the one installed by this task.
The expected use would be to run "python" rather than "py", but I suspect it wouldn’t be too hard to get the UsePythonVersion task to also update the registry? That might not play well on not-our-machines though. Still, probably better than the vague error message we're seeing right now.

@zooba is right, the registry entries aren't set on the hosted Windows machines.  They should probably be set at image creation when installing Python rather than by the Use Python Version task.  That will be easier, it won't mess with self-hosted agents, and it will let people use py instead of Use Python Version if they're so inclined.
The fix will be straightforward but will need to roll out with the next hosted image update.  @gaborbernat , is this blocking you?

Thanks 😎 I can work around it in the meantime, just noticed that this part does not work for the Python usage.

(Going back through Python issues)
This issue is for setting up Python versions in the hosted tools cache to work with the Py launcher on Hosted VS2017.  The work required would be to use the Windows Python installer during image generation.


Create documentation for k3s testing.

Describe the solution you'd like
Addition to the Contributing.md which describes when and how to create and run tests in the k3s repo.

Additional context

Backporting

[ ] Needs backporting to older releases

Close as this is both documentation and testing only.


refactor(esl-media): event resubscribtion and resize target

:tada: This PR is included in version 5.0.0-beta.17 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.17.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


use upstream log-dump script to gather logs during e2e test

The kubernetes/test-infra repo had a log collection script they use in the various prow jobs to test kubernetes clusters.  It handles a lot more than we really need, windows node/gke/etc, but I was thinking it would be easier to use it straight up than take on the work off modifying for our use case and maintaining.  I have left the license intact and added a short readme explaining what it is.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Fixed the non-inclusive language and added a note to the readme if we ever need to update from upstream and reproduce the changes.


Mac 13 M1 and reloadium 9.0 installs x64 version

file ~/.reloadium/package/3.10/reloadium/corium.cpython-310-darwin.so
~/.reloadium/package/3.10/reloadium/corium.cpython-310-darwin.so: Mach-O 64-bit dynamically linked shared library x86_64
file ~/Downloads/reloadium4/lib/reloadium-0.9.0/META-INF/wheels/reloadium-0.9.5-cp310-cp310-macosx_12_0_arm64/reloadium/corium.cpython-310-darwin.so
~/Downloads/reloadium4/lib/reloadium-0.9.0/META-INF/wheels/reloadium-0.9.5-cp310-cp310-macosx_12_0_arm64/reloadium/corium.cpython-310-darwin.so: Mach-O 64-bit dynamically linked shared library arm64
I tried to manually install with
pip install ~/Downloads/reloadium/lib/reloadium-0.9.0/META-INF/wheels/reloadium-0.9.5-cp310-cp310-macosx_12_0_arm64.whl

Manually copying the folder works, now there is a brand new error
Unable to load extension pydevd_plugins.extensions.types.pydevd_plugin_pandas_types
Similar to
https://github.com/microsoft/vscode-jupyter/issues/10057 but reloadium seems to work

Fixed in reloadium 0.9.6 and PyCharm plugin 0.9.1


Restore messages from s3

…t of records

How it works.
Based on topics to restore allow and deny list we get list of TopicPartitionToRestore objects. For each there will be run one worker in thread pool with its own transactional producer.
Worker obtains list of json files with backup data and tries to send content of each batch in one transaction. After sending each message it's offset is saved in memory to prevent duplicates - before sending message we check if it's offset has already ben produced.
After committing transaction we read metadata of produced records to update new offset that each of the messages got.
So we end up with map of old-new offset for each restored message.
It is possible to force producer to use transaction for one produced record - to do this one needs to change RestoreMessageProducer.PRODUCER_BATCH_SIZE to 1.
Please note that we produce each record with timestamp we have from backup. Looking at documentation HERE I would argue that we can always give timestamp to produced message but for CreateTime type we can get some messages not being restored if restoring old messages. On the other hand if message.timestamp.difference.max.ms is same as retention.ms it should make no difference for us because older messages would also disappear from source cluster.

Please note that we produce each record with timestamp we have from backup. Looking at documentation HERE I would argue that we can always give timestamp to produced message but for CreateTime type we can get some messages not being restored if restoring old messages. On the other hand if message.timestamp.difference.max.ms is same as retention.ms it should make no difference for us because older messages would also disappear from source cluster.

💯


Update composer command in docs

composer must require parental version 0.6.1 to successfully install under Laravel 5.8

Thanks @deadpunk !


Version checking

Initial stab at some kind of version check, so daktari doesn't explode with some meaningless python exception when you try to run an old version on a project.
It's a little hacky - I've used a regex to parse the version out of the config file because trying to interpret it as python is not necessarily safe if the daktari version is too old. An alternative I considered but rejected was having a separate config file to specify the version - that seemed more sucky from a user perspective (who wants multiple config files?).
Open to other suggestions :grin:

It's a little hacky - I've used a regex to parse the version out of the config file because trying to interpret it as python is not necessarily safe if the daktari version is too old. An alternative I considered but rejected was having a separate config file to specify the version - that seemed more sucky from a user perspective (who wants multiple config files?).

Yeah, the regex is an entirely reasonable fudge IMO.
Could we update the README to include daktari_version = ... as part of the example config (and perhaps it can be part of the list of files that bump-version manages?)


Fix tarbomb.

When extracting the Linux .tgz file from the releases page, it is a tarbomb.  This can be very detrimental to anyone extracting the application.

A tarbomb, also sometimes written as tar bomb, is a tarball whose contents appear to explode into the current directory or some other existing directory containing a large number of items when untarred rather than into a new directory created by the tarball specifically for such contents.

@jackmcguire1 - can you elaborate here? Which version/what is expanding into? Looking into the tarball, it shows:
-rw-r--r--  0 root   root    11361 Jan 12  2021 LICENSE
-rw-r--r--  0 root   root     2063 Jul 27 17:20 README.md
-rwxr-xr-x  0 root   root 10988504 Dec  8 18:25 twitch

after running tar -tvf twitch-cli_1.1.5_Linux_x86_64.tar.gz
Which matches with what I expect.

The issue is that the tarball extracts into the current directory without creating a sub directory to extract itself into.
For example, if I'm working in a directory with a preexisting README.md and I extract your tarball, it will overwrite my README.md with the one from your package, destroying work.
This is, at the very least, bad form in the *nix communities I frequent.


Elwalid recover table

Proposition for recover_table(fully_qualified_table, timestamp)

Thanks @ewibrahim for the input.
However, I think this function should be able to work without snapshot enabled using bigquery time travel.
https://cloud.google.com/bigquery/docs/time-travel#query_data_at_a_point_in_time
create or replace my_project.my_dataset.my_table as
select * from
my_project.my_dataset.my_table
FOR SYSTEM_TIME AS OF timestamps;

should make the job.
(but adding the correct table options, description, etc)
Maybe we could combine both time travel and snapshot

Thank you for your answer,
What do you mean by table options ? should i add optional parameters in the
function ? If yes which ones apart from description would you want ?
Le lun. 5 juin 2023 à 12:14, Unytics @.***> a écrit :

Thanks @ewibrahim https://github.com/ewibrahim for the input.
However, I think this function should be able to work without snapshot
enabled using bigquery time travel.
https://cloud.google.com/bigquery/docs/time-travel#query_data_at_a_point_in_time
create or replace my_project.my_dataset.my_table as
select * from
my_project.my_dataset.my_table
FOR SYSTEM_TIME AS OF timestamps;
should make the job.
(but adding the correct table options, description, etc)
Maybe we could combine both time travel and snapshot
—
Reply to this email directly, view it on GitHub
https://github.com/unytics/bigfunctions/pull/113#issuecomment-1576518189,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARQYCCEOFJLFC45NDZGRZZ3XJWWQ3ANCNFSM6AAAAAAY2XWGKI
.
You are receiving this because you were mentioned.Message ID:
@.***>

partition config should be kept for instance as well as any other configuration.
as seen here, ddl to create the table is in information schema:
https://cloud.google.com/bigquery/docs/information-schema-tables#schema

maybe we could use it at the same sytem_time

Thanks @ewibrahim, I'll have a look.
Could you join the slack so that it faciliates conversation?
Thansk

It seems that for now, it is not possible to use time travel on a deleted table with sql (while it is possible with bq command line).
So I close this pull request that I may re-open the day when it becomes possible.


.gitignore files are not respected in the project search and ctrl+p

VSCode Version: Code 1.13.0 (376c52b955428d205459bea6619fc161fc8faacf, 2017-06-08T16:41:25.571Z)
OS Version: Linux x64 4.4.0-78-generic
Extensions:

Extension
Author
Version

beautify
HookyQR
1.1.1

vscode-docker
PeterJausovec
0.0.15

ReactSnippetsStandard
TimonVS
1.1.0

jsx
TwentyChung
0.0.1

html-css-class-completion
Zignd
1.7.0

path-intellisense
christian-kohler
1.4.2

vscode-eslint
dbaeumer
1.2.11

auto-close-tag
formulahendry
0.4.2

vue-snippets
hollowtree
0.1.1

spacegray-vscode
ionutvmi
1.2.3

theme-chromodynamics
lei
1.0.8

vue
liuji-jim
0.1.5

Go
lukehoban
0.6.61

ruby-rubocop
misogi
0.3.5

HTMLHint
mkaufman
0.3.3

atom-keybindings
ms-vscode
2.0.1

latex
ms-vscode
0.1.3

Ruby
rebornix
0.12.1

guides
spywhere
0.7.0

code-spell-checker
streetsidesoftware
1.1.0

vscode-todo-highlight
wayou
0.5.5

change-case
wmaurer
1.0.0

JasmineSnippets
xabikos
0.2.1

JavaScriptSnippets
xabikos
1.4.0

ReactSnippets
xabikos
1.3.0

The files I have excluded using my .gitignore are coming up in project search and ctrl+p but they are not shown in the diff. This causes my system to slow down when searching and also show unwanted results.
Can some please tell me how to fix this. I can provide more details as required.

using
"search.useIgnoreFilesByDefault": true,

in my user settings solved this.

I'm using vscode Version 1.14.2.
Setting "search.useIgnoreFilesByDefault": true, in my user settings AND workspace settings is NOT ignoring files inside gitignored directories when using cmd+p.  It IS however ignoring files in global search.

I take that back.  My behavior is very inconsistent.  In one of my projects, the gitignore file is being honored for searches only; in another project, the gitignore file is not being honored at all.
I've double and triple checked that "search.useIgnoreFilesByDefault": true is set in my user settings.

ctrl+p isn't able to respect gitignore files yet. In search, the gitignore file is only respected if you toggle the button for it in the advanced search options (under the ... button).
What the "search.useIgnoreFilesByDefault": true setting does is determine whether that gitignore button should be toggled on by default for new workspaces - after you do a search in the workspace, it remembers whether the button is toggled on and will persist however you set it.
Slightly confusing, but we couldn't enable it by default for everyone because I didn't want to confuse people and cause them to lose results.

#24030

@roblourens thanks a lot for the clarification.  Makes sense.  Adding to "files.exclude" and "search.exclude" will have to do for now.
Thanks!

Indeed, this is the item to track for gitignore support with quickopen, but I won't have time in the near future: https://github.com/Microsoft/vscode/issues/24074


Only 1 active Download?

Hey yasharrashedi,
Why only one file is active at a time even if I click resume all.
The uGet download manager is capable of running multiple downloads simultaneously using aria2 plugin.
Is there any trick to achieve that?

you can set max concurrent downloads in aria2.conf
max-concurrent-downloads=1

The documentation says:
Set the maximum number of parallel downloads for every queue item.
So it is for per queue item.
It's not controlling how many different downloads can be active at one time.

after i changed the max-xonxurrent-downloads to >1 , multiple files were able to download simultaneously.
Looks like I misinterpreted the documentation.


On updating DA with Attribute, save as draft. The existing DA in privacy dashboard, will update with Attribute name

On updating DA with Attribute, save as draft. The existing DA in privacy dashboard, will update with Attribute name

@kashishmalik  This is updating from API.
@georgepadayatti For your attention.

Please raise an issue in the API repository ?


Add spell checker for docstrings

From<EMAIL_ADDRESS>on 2010-08-24T07:21:12Z
Hello,
there is a library that supports spellchecking of docstrings: http://www.language-binding.net/pydsc/pydsc.html Would be nice to have this feature in Spyder.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=311

From pierre.raybaut on 2010-10-17T13:06:10Z
Labels: -Type-Defect -Priority-Medium Type-Enhancement Priority-Low

From pierre.raybaut on 2012-03-18T14:08:44Z
Labels: -Type-Enhancement Type-Enh

From pierre.raybaut on 2011-05-16T01:09:22Z
Labels: Cat-Editor

From pierre.raybaut on 2011-10-29T04:30:54Z
Status: ContributorNeeded

From pierre.raybaut on 2012-03-18T14:16:30Z
Status: HelpNeeded

The link to the library seems to no work, I do not think this is going to happen.


Add Shattered Pixel Dungeon and some of its forks

There's another FOSS game called Bombsquad but you might want to take a look at the 1st 2 questions in the FAQ to decide if it can be added to this list or not.

Anything not directly contained in this repository, however, even if automatically downloaded by build scripts, is still proprietary and cannot be redistributed without explicit consent. This includes assets and prebuilt libraries/binaries.


Changed paths to change Referrer

Seems this was copied from pfSense

It's simple: we moved it. It's difficult to embed paths of the menu into the strings, as we can never guarantee consistency unless we do an "automatic" path conversion. Maybe we could do that, but for now let's merge this, thanks! :)


Update installation instruction for probe-rs

The binary probe-rs tools has been moved to it's own crate: probe-rs-tools. Therefore, the current instructions are outdated and fail:
$ cargo install probe-rs
    Updating crates.io index
  Downloaded probe-rs v0.24.0
  Downloaded 1 crate (1.7 MB) in 0.35s
error: there is nothing to install in `probe-rs v0.24.0`, because it has no binaries
`cargo install` is only for installing programs, and can't be used with libraries.
To use a library crate, add it as a dependency to a Cargo project with `cargo add`.

This commit updates the installation instruction.
For more information, see the probe-rs' changelog.

Sigh. Thanks much for the timely update. We'll get it merged shortly.

Sure, here you go.

Appreciate your help with this.


Navigation not work on Windows 10 1809/RS5 with T10.2

I realized today that after the upgrade 1809 the navigation via NavigationView doesn't work anymore. I haven't installed RS5 sdk yet, this should be interesting, if it works again with the new target.

I have not upgraded yet. I will probably this weekend. What's the error?

If I click on NavigationViewItem, the navigation is apparently not executed. But as I've seen through the debugging, the navigated event doesn't fire, it doesn't seem to matter which sdk you have as your target. The navigation itself seems to work fine, only the assignment of the new context in the navigation view does not work.


secp256k1_ecdsa_signature_parse_der support

Hi,
would it be possible to add support for secp256k1_ecdsa_signature_parse_der function?
Regards,
Martin

https://github.com/MeadowSuite/Secp256k1.Net/pull/8

Updated package with these changes published to nuget https://www.nuget.org/packages/Secp256k1.Net/0.1.54
Thanks for the contribution!


Change the UniqueDimension's concept of a 'label' to a different word

Using the term label is confusing when a dimension has a label field for a human readable definition of the dimension, but also a label_ field for filtering by the label e.g. a keyword name. Maybe this label_field could be called display_field.

https://github.com/kayak/fireant/pull/22


Revert the daemonset selector to the one used previously

Changing the label selector will break upgrades, because it's not possible to change the selector in daemonsets, effectively not rolling up the new metallb.
Here we roll back before releasing a new metallb-operator version.

LGTM , can u open an issue for this while we are at it so we don't forget the details

LGTM , can u open an issue for this while we are at it so we don't forget the details

Filed https://github.com/metallb/metallb/issues/1193 as this also involves the helm chart


isEqualTo on date can lead to unexpected failures when switching time zones

Describe the bug
The behaviour of isEqualTo(String) when asserting a java.util.Date can fail if you switch the default time zone during a test run.

assertj core version: 3.24.2
java version: 21
test framework version: JUnit 5
os (if relevant):

Test case reproducing the bug
Add a test case showing the bug that we can run
public class Test {
 @Test
    void testWithIsEqualTo() {
//      TimeZone.setDefault(TimeZone.getTimeZone("CET"));
//      assertThat(Date.from(Instant.parse("2024-03-01T00:00:00.000+01:00")))
//          .describedAs("Using CET time zone")
//          .isEqualTo("2024-03-01");
      TimeZone.setDefault(TimeZone.getTimeZone("WET"));
      assertThat(Date.from(Instant.parse("2024-03-01T00:00:00.000+00:00")))
          .describedAs("Using WET time zone")
          .isEqualTo("2024-03-01");
    }

    @Test
    void test() {
//      TimeZone.setDefault(TimeZone.getTimeZone("CET"));
//      assertThat(Date.from(Instant.parse("2024-03-01T00:00:00.000+01:00")))
//          .describedAs("Using CET time zone")
//          .hasYear(2024)
//          .hasMonth(3)
//          .hasDayOfMonth(1);
      TimeZone.setDefault(TimeZone.getTimeZone("WET"));
      assertThat(Date.from(Instant.parse("2024-03-01T00:00:00.000+00:00")))
          .describedAs("Using WET time zone")
          .hasYear(2024)
          .hasMonth(3)
          .hasDayOfMonth(1);
    }
}

Run this test with the code commented out. The test will be ok.
Now enable the code in comments and run the test again. The test testWithIsEqualTo now fails, but on the part that was already active before.
(I found the issue with using @DefaultTimeZone, but I can reproduce it without it, so I believe it might be an AssertJ issue).

Thanks for reporting this @wimdeblauwe, we will have a look at it shotly
.

AssertJ uses various date format to get parse the date from the given string, in the test case, it is the yyyy-MM-dd date pattern (as expected).
AssertJ delegates the parsing to SimpleDateFormat parse method, there is not much AssertJ can do at this point, it's in the hand of the java runtime.
My understanding is that the parsing has to use the default timezone because 2024-03-01 is not a Date but a LocalDate, to make it a Date it has to be interpreted within a timezone which can be no other than the default timezone.
Passing string instead of date was added to avoid building dates by hand which was pain before java 8, it's now less useful with the new date/time API which provides a bunch of factory methods.
@wimdeblauwe what behavior would you have liked to see her  do you have a suggestion here

My understanding is that the parsing has to use the default timezone because 2024-03-01 is not a Date but a LocalDate, to make it a Date it has to be interpreted within a timezone which can be no other than the default timezone.

I agree, but shouldn't it keep working when you switch the default timezone ? It feels like some kind of caching is done and the updated default timezone is not taken into account.

Indeed this is bug in AssertJ, we store default date formats as SimpleDateFormat, they are created only once at the first date assertion, they are stateful and store the timezone in their underlying calendar.
The problem arises when you change the timezone, the default date formats have already been created with the previous timezone and thus interprets incorrectly the dates as string.
well spotted @wimdeblauwe !

Thanks @joel-costigliola . That is great with open-source, you can get a peek "inside" to guess what might be issue. Actually solving it is usually a step harder :-)


Adding support to create insecure TLS

feature
How to use
insecure mode is disabled by default, and the property to enable is .insecure=true
TlsUtils.createTrustManagerFactory method creates an instance of trust manager factory according to the insecure mode.

Codecov Report
Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (9321eab) 61.76% compared to head (0a4c98e) 0.00%.
Report is 1 commits behind head on master.

Files
Patch %
Lines

...n/java/org/apache/pinot/common/utils/TlsUtils.java
0.00%
6 Missing :warning:

...java/org/apache/pinot/common/config/TlsConfig.java
0.00%
4 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12416       +/-   ##
=============================================
- Coverage     61.76%    0.00%   -61.77%
=============================================
  Files          2436     2360       -76
  Lines        133098   129358     -3740
  Branches      20617    20059      -558
=============================================
- Hits          82214        0    -82214
- Misses        44838   129358    +84520
+ Partials       6046        0     -6046

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
0.00% <0.00%> (-61.66%)
:arrow_down:

skip-bytebuffers-false
0.00% <0.00%> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <0.00%> (-61.77%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Changing Query.all() timesScanned to timesQueried

Changing Query.all() timesScanned to timesQueried, since we are querying instead of scanning.
This PR is based on PR #285, so that PR should probably be approved first

Coverage remained the same at 79.032% when pulling 0e128e617583395f41fa3fb0564197840cfabbf3 on fishcharlie:queryall_timesqueried into 933016b244ae28807107c0a0ecd43fabc713eb05 on automategreen:master.


BUG-Unable to change the Icon post launch

Description
Unable to change the icon that appears on the browser

Also, any way to add headers & header validations for the GETing of the html page for security purposes?

You can change the icon by specifying it in the run.
<Gui>.run(favicon="path/to/your/icon.png")
I will let the R&D respond to the other question. We should maybe create another issue. @FredLL-Avaiga @FabienLelaquais

if you specify your taipy page as HTML, the head section is kept

@EVAUTOAI Did you solve your favicon issue?
We are currently designing a solution for the second question that will be part of another issue.


docs: Create 2023-11-rename-sshrv(d).md

- What I did

Added an Engineering DL for renaming sshrvd to drop the "ssh" part

- How I did it
- How to verify it
- Description for the changelog
docs: Create 2023-11-rename-sshrv(d).md

@XavierChanth Another option might be to eliminate the tcp and/or socket bit entirely; e.g. use npnr and npnrd (no ports network rendezvous [daemon]). However my instincts are that it will be much cleaner for us to have separate binaries for tcp vs udp / etc in the future so I would lean towards something like tcpr/tcprd, or npsr/npsrd (no ports socket rendezvous)

To vote for 1. thumbs up this comment

To vote for 2. thumbs up this comment

@XavierChanth let's set a deadline for the vote to Fri 2359UTC once that's passed you can update the decision and we can review/approve

Sounds good thanks

@XavierChanth let's set a deadline for the vote to Fri 2359UTC once that's passed you can update the decision and we can review/approve

@XavierChanth @gkc @cpswan - I kept this open an extra week, seems like generic to sockets is the agreed upon naming scheme. I am inclined to either go with
srv(d) or sr(d)... but srv is already used by DNS, and SR is the marketing name for the Socket Rendezvous. So I am inclined to go with sr(d).

@cconstab @gkc @cpswan - I kept this open an extra week, seems like generic to sockets is the agreed upon naming scheme. I am inclined to either go with srv(d) or sr(d)... but srv is already used by DNS, and SR is the marketing name for the Socket Rendezvous. So I am inclined to go with sr(d).
But, we also have shrd which is split-horizon-root-daemon... awfully close to srd...
naming things is fun.

We should rename shrd
I like sr(d)

Updated the DL with information based on these votes.


[docs] add Virtuoso endless scrolling example to list

Adds a simplified endless scrolling example to the list documentation page, using React Virtuoso.
For a live (more complex) example and step-by-step explanations, check this blog post.

[x] I have followed (at least) the PR section of the contributing guide.

Thanks for your hard work!

@petyosi Thanks for sharing, I wish we talked about it before as it's not obvious to me that we should accept the change. Our users value the fact that we make educated decisions for them regarding what the go-to solution for their problem is. Proposing too many options creates friction: What the best option for my use case? Will I miss out if I go with the wrong option? That's the paradox of choice.
So here is my question, why should we document react-virtuoso? We already document react-window and react-virtualized.

These are valid concerns. Of course, you are not obliged to accept this pull request. react-window is very popular solution.
My motivation for the Virtuoso project is to make a virtualization engine that's easy to use, figures out as much as possible for you and does not require a bunch of manual component assembly for common use cases. Good examples for that are automatic item height, native keyboard navigation, and endless scrolling - all of those come out of the box or super easy to configure.
This PR documents scenario which I consider interesting (without being overly complex) while being a common use case when dealing with Lists. It complements the existing react-window one. Can it increase ambiguity or confuse users? I hope not.
I would love to get some suggestions on how this can be helpful to the Material-UI community. Thank you for your time!

When it comes to integration with third-parties, I don't think that Material-UI should be that opinionated. the choice should be left to end-users. Out-of-the-box solution from material-ui/core/lab/styles ..., should be opinionated, they shouldn't be friction there.
So maybe, the list Demo is not the right place to put it, maybe in an integrated demo section or something...
just my 2 cents...

I would be fine with the following: we link react virtualized at the end of the react-window demo, explaning that it's better suites for advanced use cases. We link virtuoso as a possible alternative, we don't add any live demo.
What do you think?


Reliability: staying connected despite changing IP address / connection changes

I'm running lnd v0.13.1-beta on a RaspberryPi 4B (32-bit) with bitcoind (master). The installation itself works fine, but unfortunately the Internet connection sometimes gets reset and a new IP is assigned. The node is Tor only, so no connections over clearnet (but Tor connections to clearnet-only nodes).
I've noticed that this can lead to a subset of channels—let's between 1 and 3—staying "Inactive" for several hours, despite the actual "outage" (no Internet) only lasting for a second or two. This can typically be fixed by manually connecting to the respective nodes via the CLI.
I have not spotted a clear pattern, yet, but a hypothesis is that clearnet channels are more likely to stay inactive (to be confirmed). This would somewhat make sense given the Tor node needs to initiate the connection, and maybe it doesn't.

Do you have any hypotheses where to start looking for the culprit(s)? E.g., is this more likely to be an lnd or a tor problem? Something else?

Is there a painless way to reliably trigger reconnects if an inactive channel is detected? The "brute force" solution would be to just lncli connect to all nodes in a while true; do lncli connect [...]; sleep 600; done loop, but that probably has unintended side effects.

Let me know if I skipped any key info.

let's between 1 and 3—staying "Inactive" for several hours, despite the actual "outage" (no Internet) only lasting for a second or two.

 It's actually a config argument (--maxbackoff) so you can tweak things if things are taking too long to connect out manually.

Ahh. Nice one. I kept the default value and forgot this setting even existed. I've now set it to 10 minutes instead of 1 hour.
Will report back, but feel free to close this if you want to keep things tidy.

Seems like you have the same problems as I experience as well.
I also tried manually reconnect with lncli connect but it was just a matter of time until everything went wrong there resulting into other connections also failing.
Do you have these problems as well when restarting lnd? I am currently a bit frustrated since there are some channels which just won't come back online from time to time because i cannot connet to their peer. Looking them up on 1ml and the lightning terminal they appear all up and running. I do not know if this is related to lnd or the peer lightning software...

I don't really know if there's been an improvement... I noticed it last time because someone complained. Whenever I check lntop, there are some inactive channels ... earlier today I saw two inactive channels, and when I try to connect manually, both show me unknown error unknown code: 240.
$ lncli connect $KEY_A@$ONION_A:9735
[lncli] rpc error: code = Unknown desc = socks connect tcp <IP_ADDRESS>:9050->$ONION_A:9735: unknown error unknown code: 240

$ lncli connect $KEY_AB$ONION_B:9735
[lncli] rpc error: code = Unknown desc = socks connect tcp <IP_ADDRESS>:9050->$ONION_B:9735: unknown error unknown code: 240

lncli disconnect $KEY_A
[lncli] rpc error: code = Unknown desc = unable to disconnect peer: peer $KEY_A is not connected

lncli disconnect $KEY_B
[lncli] rpc error: code = Unknown desc = unable to disconnect peer: peer $KEY_B is not connected

What's this error? Are they maybe just down? Is there a way to get more verbose output? @Roasbeef

It's kinda sad, but I execute this after every detected Internet connection reset now ... will try to debug properly when I find enough time.
#! /usr/bin/env -S bash -x

DELAY_PRELOOP=20
DELAY_LOOP=90
DELAY_INTRAPEER=0.5

NICE=19

PEER_LIST="\
key1@onion1.onion:9375
[...]
keyN-1@onionN-1.onion:9375
keyN@onionN.onion:9375
"

## restart tor
sudo systemctl restart tor && sleep "${DELAY_PRELOOP}"

## reconnect to lnd peers
for i in {1..3}; do
    for peer in ${PEER_LIST}; do
        echo -en "\nConnect: ${peer}\n"
        nice -n${NICE} lncli connect --perm "${peer}"
        sleep "${DELAY_INTRAPEER}"
    done
    sleep "${DELAY_LOOP}"
done

Closing this for now, as it appears that the peers are just down themselves.
That script you have won't help if you already have the peer established as a persistent connection ("already have 1 persistent connection attempt...").
You may want to also try restarting the Tor daemon, then also restarting lnd as well if you're just generally running into issues w/ Tor as it can be more unstable at times.
This PR should also help in the case where the remote peer changes it's tor address or IP address between reconnection attempts.

In the example the peers were down and there was a persistent connection attempt, thus the message and the script would not have helped.
However, since I posted the examples I've observed other inactive channels, and some of the counterparty nodes were again not down. However, I haven't done enough testing to know if the script truly fixes channels that would have stayed inactive otherwise, or if it just gives lnd a nudge and accelerates the connection process a little bit.
Besides, I'm no longer sure if the problem is really the connection reset, or if the connectivity problems with certain nodes are independent of that. I've now observed inactive channels to major nodes (which shouldn't be down) even after the IP has been stable for several days.
Need to do more pattern recognition.


Setup Hypershift development environment

Summary

Change Type

Check List Before Merging

[ ] This PR passes all pre-commit hook validations.
[ ] This PR is fully tested and regression tests are included.
[ ] Relevant documentation has been updated.

Additional Information

Is blocked by package-operator/cardboard#71

Codecov Report
Attention: 87 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (cardboard@234a76f). Click here to learn what that means.

Files
Patch %
Lines

cmd/build/dev.go
0.00%
72 Missing :warning:

cmd/build/cluster.go
0.00%
15 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             cardboard       #2   +/-   ##
============================================
  Coverage             ?   51.97%
============================================
  Files                ?      176
  Lines                ?    10401
  Branches             ?        0
============================================
  Hits                 ?     5406
  Misses               ?     4434
  Partials             ?      561

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Superseded by package-operator/package-operator#959


Error: Config must be specified for the HighchartsChart component

import React from 'react';
import highcharts from 'highcharts';
const ReactHighcharts = require('react-highcharts'); // Expects that Highcharts was loaded in the code.
class  Chart1 extends  React.Component {
componentDidMount() {
let chart = this.refs.chart.getChart();
chart.series[0].addPoint({x: 10, y: 12});
}
render() {
const config = {
/* HighchartsConfig */
};
return ;
}
}
I am getting "Config must be specified for the HighchartsChart" error? Can you please help me out?

Closing and there has been no activity in this issue for a while.
if this is still relevant, please feel free to reopen


Update icon to use GitHub style

This is an update to the SVG icon + hovering behavior that we use. Now that GitHub has its own copy button auto-added, we might be able to reduce cognitive load on users by mimicking their UX. This PR is an attempt at that!
It also adds in a background color (neutral gray) rather than trying to inherit the background from its parent element, which I think was introducing some variability.
Curious what @pradyunsg thinks
GitHub behavior:

This PR behavior:

It is hard to get the behavior totally consistent across themes etc, and I had a hard time getting the heights to quite line up, so please make suggestions if you think there are improvements to make there (or elsewhere).

I’m ambivalent on this. The only concerns I personally have with this change is:

Is this sensitive to the font size of the code block in any way?
How does it look on Furo in dark mode? 🙃

None the less, I think this is a good idea overall — I’m happy to add a bunch of CSS to Furo to make it look right in dark mode, if necessary.

Two quick things:

Maybe more padding to be closer to the GH one?
I think the green border is actually important because somehow enforces the "success" idea at the time to copy

Thanks for the feedback all - here's what it looks like now:

@pradyunsg - it does scale with the rem of the div (though, it did before as well). Part of my goal in having a grey background was to make this work in a more standard way across themes...do you think there are other improvements to make there?

rem of the div (though, it did before as well)

1rem in a div is definitionally equal to 1rem in another div (regardless of the font size in those individual divs).
(Aside into CSS units: 1 rem is equal to the font size on the root HTML node (html) and is consistent throughout the document. I read rem as “root em”. Within the same document, it’s possible for 1em to be equivalent to 16px in one place and 12px in another. 1 rem however will always be a constant value throughout — say equivalent to 16px.)
All this to say: I think we should use em everywhere in this plugin, such that the button’s size is mapped to the current element’s font size, and not use rem at all. It’s not a “problem” introduced in this PR, so we can also fix this separately.
How do you feel about leaning in all the way and completely mimick GitHub’s styling for this; using the exact same colours and using the exact same padding ratios?

You have taught me something i didn't know today 🙂
Re: totally mimicking github, IMO that is reasonable. They almost certainly have put more thought into ui ux than i have haha

I've updated this to use em instead of rem, though didn't notice a huge difference in the result. @pradyunsg I also added Furo to our tox dependencies, to make it a bit easier to test out w/ that theme.

Here's the latest commit - I made the background a little bit lighter by default so that it's less obtrusive when just hovering over the code cell.

I think I am pretty happy with where it is right now. @damianavila @pradyunsg anything else you'd like to see updated or improved here? I think there are other improvements to make to get it closer to GitHub style (e.g., arrows on the tooltips) but perhaps this is a nice step forward?

I am enough happy with the latest version you posted above.

This looks good to me! Now, to throw a wrench... how do you feel about making this configurable? :3

@pradyunsg making what configurable? Also, I thought you didn't like making things configurable 😅

@pradyunsg making what configurable?

The style of the copy button -- using different svgs and general style. In case someone wants to use the current aesthetic and don't like the GitHub copy button style.

I thought you didn't like making things configurable 😅

It isn't a blanket rule. Each of these is a choice where I'd like to think we'd consider the tradeoffs and what the configuration options' effects will be. 🙃

Yeah i agree with that sentiment - was just giving you a hard time 😉
I think making the icon configurable would be pretty reasonable. For the styling etc I'd imagine we could just document the CSS rules to use and how to overwrite them manually. Does that make sense?
Is this something you think needs to be in this PR? If not, want to open an issue so we can track it?

Is this something you think needs to be in this PR? If not, want to open an issue so we can track it?

It doesn't need to done here. Let's do it separately, and only if someone complains that they want the old style back. :)

ok cool - that makes sense. I opened up https://github.com/executablebooks/sphinx-copybutton/issues/134 to track that one, let me know if it looks OK to you :-)


Use an ID Attribute to Style an Element

Challenge Use an ID Attribute to Style an Element has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.103 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

<form action="/submit-cat-photo" id="cat-photo-form">
  <input type="radio" name="indoor-outdoor" checked> Indoor
  <input type="radio" name="indoor-outdoor"> Outdoor
  <input type="checkbox" name="personality" checked> Loving
  <input type="checkbox" name="personality"> Lazy
  <input type="checkbox" name="personality"> Energetic
  <input type="text" placeholder="cat photo URL" required>
  <button type="submit">Submit</button>
</form>

@ddgil1124 Please read the instructions on the left. You have given the id cat-photo-form to the <form> element, but in your css, you use #cat-photo-element, which is why it doesn't work for you.

Thanks @Manish-Giri.
@ddgil1124 Please check Help I have found a bug.
and reachout to Help Chat Room, for assistance to coding.
Thanks.

Hello All,
I think I am experiencing a problem on this challenge even though I have the id properly entered in the style sheet and the id name properly entered in the form.
Appreciate assistance as time permits;
Moving on to the next item.
Bill

CatPhotoApp
Click here for cat photos.

Things cats love:

cat nip
laser pointers
lasagna

Top 3 things cats hate:

flea treatment
thunder
other cats

 Indoor
 Outdoor
 Loving
 Lazy
 Energetic

Submit

thanks @Manish-Giri

CatPhotoApp

Click here to view more cat photos.

Things cats love:

cat nip
laser pointers
lasagna

Top 3 things cats hate:

flea treatment
thunder
other cats

 Indoor
 Outdoor
 Loving
 Lazy
 Energetic

Submit

This issue is locked, as resolved. If you require any assistance with the freeCodeCamp curriculum, feel free to search for help, or open a topic on the forum.
Happy coding 😃


Reproduced Results

Hi, @bwconrad ,
Great work.
I want to know how to reproduce the results of your teaser.

An image is splited to 2*2, and the accuracy is 84.4%.
Best,
Guopeng.

What this figure is visualizing is the process of patchifying an image with different patch sizes using pi-resizing and having a much less significant performance drop compared to naive approaches.
The image being split 2x2 and 3x3 is just for visualization purposes. It represents the idea of, for example, using a patch size of 16 and 32 on the same image to adjust the number of tokens being processed.
I am not an author of this work, this is simply a reimplementation of the architecture. If you want to try and reproduce the results just follow the hyperparameters listed in the paper and use your own classification training script. The accuracy numbers from the figure look to be for a ViT-L at patch sizes of 24 and 10 according to Table 2 in the appendix.


[no-shadowed-variable] Allow generic types parameters of same name

Bug Report

TSLint version: 5.9.1
TypeScript version: 2.8.1
Running TSLint via: gulp-tslint

TypeScript code being linted
export type Partial<T> = {
	[P in keyof T]?: T[P];
};

export interface PartialArray<T> extends Array<Partial<T>> {}

export type PartialWithArrays<T> =
	T extends Array<infer U> ? PartialArray<U> :
	T extends ReadonlyArray<infer U> ? PartialArray<U> :
	Partial<T>;

with tslint.json configuration:
{
  "rules": {
    "align": false,
    "no-shadowed-variable": true,
  }
}

Actual behavior
ERROR: (no-shadowed-variable) [9, 32]: Shadowed name: 'U'

Expected behavior
No error.

💀 It's time! 💀
TSLint is deprecated and no longer accepting pull requests other than security fixes. See #4534. ☠
We recommend you instead use typescript-eslint to lint your TypeScript code with ESLint. ✅
👋 It was a pleasure open sourcing with you!

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


Incorrect response from the server

everything worked fine yesterday, today like this
v.2

Have the same problem...


Fix #364

Use the default value for line if set, otherwise use 'auto'.

The only failure is the diffs...which from the output looks like auto is different? I'm having ha hard time running this on my machine with node 6.10...

Yeah, tests more or less require node 0.10. Though, last time I made a big change against master, I just tested against travis, haha

Would be nice to see this PR merged


Get union of enum values

🤔 Question
Describe your question
Hi. Here is any way to get union of enum values?
Tried many things, none of them is worked

Hi @sanex3339, the enum itself kind of works like a union:
enum E {
    A,
    B,
    C,
    D,
    X,
}

type test0 = Exclude<E, 0> // E.B | E.C | E.D | E.X

https://www.typescriptlang.org/docs/handbook/enums.html
Did this solve your problem?

I'm want something different.
enum SomeOptionValue {
   Bar = 'bar',
   Baz = 'baz
}

type Options = {
    someOption: EnumValues<SomeOptionValue> // 'bar' | 'baz', not 'SomeOptionValue.Bar | SomeOptionValue.Baz'
};

It's a reference, you can't change it/do that.

You should try to replace your enum with plain objects, make a helper library if you really need this.


Question about the PSRO.

Dear authors,
I am trying to find some examples of applying DRL as an oracle to the generalized PSRO framework. However, I found that in examples related to PSRO (e.g generalized PSRO and RNR) only an evolutionary-strategy oracle is being used for simplification. So I am wondering if there is an example in which PSRO is combined with DRL as experiments in the corresponding papers. Moreover, I notice that there are implemented actor-critic and DQN in the library. Is there a simple way to combine them with PSRO? Thank you.

Hi @wyz2368 ,
We unfortunately do not have the full PSRO as described in the paper with Deep RL oracles (as described in the paper).
It is very much something we would like to have in the code base, though. And yes, there should be a simple way to combine them with PSRO, but we have not had the time to do it.
It would be a very welcome contribution that I could help advise (maybe with help from the original author), if you're interested. It would involve making a custom RL environment that would in the fixed policies of the other players that DQN could train on.

Hi @lanctot,
Thank you for your reply. In fact, I am currently writing the code and trying to make the policy gradient algorithms (the version in OpenSpiel) an oracle for PSRO. I don't know how general the code is going to be because the DRL codes (DQN, A2C, etc.) in OpenSpiel are customed to a multi-agent setting. What I am doing is that suppose PSRO starts with a uniform strategy for each player. In each iteration, I keep the DRL agent in a training model for the current player who is training while fixing other players' agents by setting the flag is_evalution to be True in agent.step(time_step, is_evaluation=True). Obviously, my code is highly specific to the RL code in OpenSpiel.
Moreover, I found that the Nash solver in OpenSpiel now can only solve two-player zero-sum games. Another thing I am doing is to add a more general Nash solver to OpenSpiel, which can be used in the double-oracle algorithm or others that need Nash equilibrium.

That sounds great, and it's exactly what I had in mind. I'm not worried if the PSRO implementation here would be highly specific to the RL code in OpenSpiel; it's in OpenSpiel for that reason, provided as an example. If people want to generalize it to other RL algorithms or frameworks, they are welcome to.
BTW, there is an example of solving a (2-player) general-sum matrix game here (using nashpy): https://github.com/deepmind/open_spiel/blob/master/open_spiel/python/examples/matrix_nash.py but it seems like we never put that into a general library. I agree, it would be good to have that as a general function we can call.
Would be great to submit a PR with these additions when you think it's ready!

Hi @wyz2368 , as we have been discussing over email, we are also working on this internally (and by 'we', I mean almost entirely @paulfmmuller!!) Just updating the thread here to let others know that we plan to release the code, but might take several weeks for review since it's a large change.

Quick follow-up, thanks to the work of @PaulFMMuller, Monday's update will include all of his implementations. Quoting an email from him:
"It will introduce a number of breaking changes ; we have for the moment identified the following ones :
- Oracles now take in a list of list of dictionaries specifying training parameters, and therefore, all former oracles are now deprecated (But new ones have been added)
- EvolutionaryOracle has been removed.
- (minor) file name changes, helper functions moved to new files, function name changes
In counterpart, it will introduce:
- New oracles (Exact Best Response, RL Oracle supporting Policy Gradient and DQN)
- Self-Play training
- Symmetric game optimizations
- A Policy Aggregator working with Joint Probabilities
- An actual example running on Kuhn, Leduc, ..., and computing exploitabilities
- Misc. refactorizations and code improvements."
In addition, he has reproduced the original results from Lanctot et al. 2017 on Leduc (even improved it down to NashConv of 0.46!

Hi there !
The internal reviews have taken longer than anticipated, and unfortunately, the whole of the update hasn't made it to this Monday's code update ; only small parts have been included.
Nevertheless, things are drawing to a close, and we're anticipating the update to soon be submitted, apologies if you were hoping to start working on it today :/

I believe all of Paul's implementations are in the code base now, so closing this. Please re-open if you want to discuss further! Thanks again, Paul!

Thanks! @lanctot @PaulFMMuller  :)))


Port & mix last 3D elixirs

Will this close #219?

I think so, yes. But just to be sure, we could also close #219 on merging #267.


upgrade to lodash4, move lodash to deps instead of peerDeps

resolves #16 #20
@rafaelchiti can I get a sanity-check on this one?

besides the small comment, :+1:


Fix - Plymouth - NixOS logo (watermark.png) not showing when using systemd initrd.

Description of changes
This is with the default "bgrt" theme (which in turn depends on the default "spinner" theme).
The plymouth installation on the root (non-initrd) filesystem /etc/plymouth/themes had un-patched *.plymouth files which referenced a store path (pkgs.plymouth) that does not contain watermark.png. This image is added in the pkgs.runCommand "plymouth-logos" derivation, and the *.plymouth files need their "ImageDir=...` updated tor the output path.
I have now used the existing runCommand "plymouth-initrd-themes" derivation for both root and initrd plymouth installations, and now the logo displays. This is also necessary for splash screen on shutdown, lock, etc. which don't use the initrd filesystem.
However, this is revealing - plymouth doesn't seem to be loading its themes (or at least, this one file watermark.png) from the initrd filesystem. I might investigate why - but for now, this is better than what was there before.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@DasJ Do you have time to review? It's quite a simple change, and is strictly better than the previous version (although I think it's only papering over an existing issue)


Reflect spec changes to the gRPC plugin

The ocgprc package is not compatible with https://github.com/census-instrumentation/opencensus-specs/blob/master/stats/gRPC.md anymore. Implement the recent changes on the spec.

Isn't this a duplicate of https://github.com/census-instrumentation/opencensus-go/issues/687 for which there is https://github.com/census-instrumentation/opencensus-go/pull/689?

Yes, it is a duplicate and is closed via https://github.com/census-instrumentation/opencensus-go/pull/689.


[GSSOC'21] [IDEA] 2048 Game

Is your feature request related to a problem? Please describe.
I would like to develop 2048 Game using Javascript
Describe the solution you'd like
Develop a 2048 game using javascript
Features

Different coloured tiles for different numbers
A score indicator
Best score display

Additional context
NA

Status update : project is almost done - will upload in a day or two


Use wrapper scripts to create default user.

Signed-off-by: Drew Moseley<EMAIL_ADDRESS>(cherry picked from commit 071a62b2c7572467f7c1fb2141b8a7004d17aec9)

@GregorioDiStefano can you review and merge?

looks good!


[Automated] Update eventing-istio-eventing nightly (#289)

Update eventing nightly
Signed-off-by: Knative Automation<EMAIL_ADDRESS>

OCF Webhook is merging this PR


WebGL 2 and WebGL Extensions Roadmap

Extensions in Canary

[x] WEBGL_debug_shaders and WEBGL_debug_renderer_info - no longer require privileged access.
[ ] OES_element_index_uint - already used by createVertexArrayFromGeometry, but could also be used by billboards and polylines that use vertex buffer directly.
[x] OES_texture_half_float
[x] OES_texture_float_linear and OES_texture_half_float_linear for completeness.
[x] WEBGL_compressed_texture_s3tc - for glTF.
[x] EXT_frag_depth (still draft) - opaque ray-traced geometry, e.g., ellipsoids, sensors, etc.

Draft WebGL Extensions

[ ] WEBGL_color_buffer_float and EXT_color_buffer_half_float for HDR (I think).
[ ] WEBGL_compressed_texture_atc and WEBGL_compressed_texture_pvrtc - for glTF
[ ] WEBGL_draw_buffers - first for combing color and pick passes in a single pass.  Later, with the post-processing framework, we can combine other passes like glow and velocity.  Shadow and reflection passes will still be separate since they are rendered with different view parameters.
[ ] WEBGL_shared_resources - for multiple canvases, but not web workers?

Proposed WebGL Extensions

[ ] ANGLE_timer_query, for profiling tools.
[ ] WEBGL_compressed_texture_etc1 - for glTF.
[ ] WEBGL_dynamic_texture - for video textures.

OpenGL ES 3.0 Features

[ ] Multisampled Renderbuffers - AA without using FXAA.
[ ] Half-float vertex attributes, i.e., GL_OES_vertex_half_float.
[ ] Uniform buffer objects - for backing uniforms with buffers based on update frequency.  Also see #649.
[ ] Sampler objects - the Cesium abstractions already look like this to some extent.
[ ] Immutable textures - reduce driver validation (EXT_texture_storage).
[x] Instancing, i.e., ANGLE_instanced_arrays, for batching geometry and models.
[ ] Primitive restart - batch render line strips instead of individual lines.
[ ] Transform feedback - perhaps for particle systems.
[ ] Program binaries - to reduce shader compile times. (Not in WebGL 2.0)

More on ES 3.0

OpenGL ES 3.0 at SIGGRAPH 2012
OpenGL ES 3.0 – Halti has arrived

More on WebGL 2.0

What's coming in WebGL 2.0
WebGL 2 Specification
Mozilla Platform/GFX/WebGL2

WebGL 2 Demos

http://toji.github.io/webgl2-particles/

Done

[x] OES_vertex_array_object - #766.

Cesium Implementation Status (after #3094)

OES_standard_derivatives is disabled. dFdx, dFdy and fwidth are not recognized when compiling the shader.

There should be one test failure for materials because the PolylineArrowMaterial uses standard derivatives.

EXT_frag_depth is disabled. gl_FragDepth is not recognized when compiling the shader (neither is gl_FragDepthEXT).
OES_texture_float is disabled. Could not create a texture with PixelDatatype.FLOAT.
WEBGL_depth_texture is disabled. Could not attach a depth-stencil texture to a FBO.

WebGL 2.0 talk from SIGGRAPH Asia 2015 - https://docs.google.com/presentation/d/1Orx0GB0cQcYhHkYsaEcoo5js3c5-pv7ahPniIRIzzfg/edit?usp=sharing

It will be easy enough to do by hand, but here's a tool to convert WebGL 1 to WebGL 2 shaders: https://github.com/Jam3/glsl-100-to-300/

WebGL 2 Samples

https://github.com/WebGLSamples/WebGL2Samples
https://floooh.github.io/oryol-webgl2/

http://webgl2fundamentals.org/

WebGL 2 articles on the RTR blog:

WebGL 2 Basics
WebGL 2: New Features

Texture Array: we could use TEXTURE_2D_ARRAY in some cases where texture atlases are used.  It will be simpler and great for dynamic updates when the size doesn't change, #2319

MDN has doc on WebGL 2 - see the Firefox 51 release

WebGL 2 lands in Firefox

WebGL 2 from WebGL 1
4.1 Backwards Incompatibility

WebGL 2 Algorithm Examples

WebGL 2 Webinar Slides

Hardware PCF for soft shadows

Mipmapping restrictions gone. No longer need to upscale in Model.js
Point cloud should use integer attributes for certain things
3D Tiles batch table can avoid uploading a pick texture. https://github.com/AnalyticalGraphicsInc/cesium/blob/3d-tiles/Source/Scene/Cesium3DTileBatchTable.js#L1371-L1374. This is also doable now but won't work for pick ids that exceed 16777217 (last int floating point can represent... would this ever happen?).

For FXAA, use textureLod and textureLodOffset like the original shader. Consider using textureSize instead of a uniform.

A minimal, WebGL 2-only rendering library https://tsherif.github.io/picogl.js/

Use native uint32 to avoid the pick texture for 3D Tiles, see createPickTexture in Cesium3DTileBatchTable.js

WebGL 2.0 Masterclass from Web3D 2017

Likely run our existing GLSL through modernizeShader, #5623, to generate GLSL ES 3.0, and then have a bit of a DSL (domain-specific language) with marcos/etc to handle differences like varying vs in/out.  I'm sure it will not be that simple and some thing will have to be GLSL ES 3.0 specific, but we do not want to limit ourselves to only using GLSL ES 1.0 over the long haul, and I believe that engines like Unity do exactly this.

Reaching the Largest Gaming Platform of All: The Web. WebGL, WebVR and glTF. GDC 2017

NanoGL - thin WebGL 2 wrapper

See https://github.com/AnalyticalGraphicsInc/cesium/pull/6314 which enabled WebGL 2 by default but hasn't been tested enough yet and had gone stale.

Possible terrain loading bug in WebGL 2: https://github.com/AnalyticalGraphicsInc/cesium/issues/6756

For anyone trying to use CesiumJS with WebGL 2, note this is not currently officially supported, but is on the roadmap. I'm consolidating open issues with running CesiumJS with WebGL 2 here.
For reference, you can activate WebGL 2 by initializing the viewer as follows:
var viewer = new Cesium.Viewer('cesiumContainer', {
    contextOptions: {
        requestWebgl2: true
    }
})

Known issues with WebGL 2

Classification example https://github.com/CesiumGS/cesium/issues/8629
Post processing https://github.com/CesiumGS/cesium/issues/7952
Incorrect depth testing https://github.com/CesiumGS/cesium/issues/7942
Error when using clipping planes https://github.com/CesiumGS/cesium/issues/7712
HDR https://github.com/CesiumGS/cesium/issues/7647

The original issue description is very out of date, if we're going to use this to track WebGL2 implementation, someone should update it with updated thoughts and plan of action.

@mramato 👍
That'll be on me an @YoussefV once we start the WebGL 2 / MSAA work.

In WebGL 2 the camera zooms towards the globe instead of towards the geometry that the mouse is hovering over. This seems like a bug in either ScreenSpaceCameraController or scene.pickPosition.
Sandcastle
WebGL 1

WebGL 2

@lilleyse
When ScreenSpaceCameraController needs a pickPosition, it calls both scene.pickPositionWorldCoordinates and globe.pickWorldCoordinates and chooses the closer of the two, assuming both are defined.
In our case, scene.pickPositionWorldCoordinates returns undefined and so ScreenSpaceCameraController uses the globe pick instead.
I played around a little and the problem seems to be that the depth texture is empty.
With the help of this error, I tried hard-coding a NEAREST filtering and it worked alright (even the error was gone). So I guess it's some problem with linear minification/magnification filtering.
I tried it with the changes from 8969, so that can also be a factor.

@dennisadams awesome, that fixes it for me as well. Just to confirm you added nearest filtering to the depth-stencil texture?
  globeDepth._depthStencilTexture = new Texture({
    context: context,
    width: width,
    height: height,
    pixelFormat: PixelFormat.DEPTH_STENCIL,
    pixelDatatype: PixelDatatype.UNSIGNED_INT_24_8,
    sampler: Sampler.NEAREST // Fixes the zoom
  });

@dennisadams @lilleyse That also fixes the Silhouette post processing as well. Also I look it up, and in WebGL 2, it seems depth texture cannot have linear filtering. That may be a potential problem with other depth textures used by Cesium as well

Just to confirm you added nearest filtering to the depth-stencil texture?

I went on a safe bet - hard-coded the Texture constructor.

Labels look a bit different in WebGL 2. Click the images to see the difference.
WebGL 2

WebGL 1

Sandcastle

The AO post process has artifacts: Local sandcastle.

Polyline arrows start to get squashed. Local sandcastle

Cesium Inspector is extremely show when "Show Frustums" is checked. Local sandcastle

Contour lines are much thinner in WebGL 2. Local sandcastle.
First thing I would look into is GL_OES_standard_derivatives which is another built-in define that's part of WebGL 1 but maybe not WebGL 2 (I haven't checked). GL_EXT_frag_depth is another one we should investigate.

Alright that's all I've uncovered so far by going through each sandcastle example with WebGL2 enabled. @baothientran I recomend starting with https://github.com/CesiumGS/cesium/issues/797#issuecomment-646920630 and seeing what other bugs are fixed in the process.

@baothientran since the number of bugs fixes was starting to grow too much I consolidated them into a single CHANGES.md entry: https://github.com/CesiumGS/cesium/commit/65757f8b7dce74d1ffbf9d61ea10d6e4508b86f2

Also I updated the checklist above: https://github.com/CesiumGS/cesium/issues/797#issuecomment-617189407

[ ] modernizeShader is too slow. This is something we should address before defaulting to WebGL 2.

[ ] This may mean upgrading all shaders to GLSL ES 3.0 with some DSL (aka @pjcozzi's comment https://github.com/CesiumGS/cesium/issues/797#issuecomment-316990165)
[ ] Or see https://github.com/CesiumGS/cesium/issues/9201 for a workaround

When implementing EXT_mesh_features in CesiumJS, one annoyance we came across is the lack of gl_VertexID.
Implicit feature ID attributes can be computed by offset + floor(gl_VertexID / repeat). However, in WebGL 1, we have no choice but to do this computation on the CPU and then upload an extra attribute

Another WebGL 1 limitation that limits our implementation of EXT_mesh_features: In this table in the MDN docs, it seems that even a highp int is only required to have 17 bits. So for feature IDs, I'll have to use a highp floatto squeeze out a couple more bits.  Also there's nouint type. In a shader,uint32` would be more ideal, but only guaranteed in WebGL 2

CloudCollection and voxels would benefit from 3D textures

Point cloud properties (e.g. classification) may have precision artifacts in WebGL 1 if accessed in the frag shader. WebGL 2 may fix this with the flat qualifier. See https://github.com/CesiumGS/3d-tiles-samples/pull/55#issuecomment-1218332158 for more details.

您的邮件我已收到，会尽快回复。亲

Given the work done in https://github.com/CesiumGS/cesium/pull/10894, that should cover most of the actionable items in this issue. I'm going to close this and any specific items should be listed in their won issue, such as https://github.com/CesiumGS/cesium/issues/649 and https://github.com/CesiumGS/cesium/issues/630.

您的邮件我已收到，会尽快回复。亲


Please invite me to the Recode-Hive GitHub Community Organization

Name
sneha056
Discord Username (if applicable)
No response
Additional Context
No response

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


Changing function arguments doesn't change HLIL output

Version and Platform (required):

Binary Ninja Version: 3.0.3238-dev Enterprise
OS: macOS
OS Version: 12.2.1

Bug Description:
In the malware sample linked in this discussion thread, the function call at 0x00402eff exhibits a whole bunch of odd behaviors. Here are my random notes from a Slack conversation about it:

The function call is through a memory access (call dword[ebp-0x30])
HLIL says, initially, int32_t eax_5 = eax_1(...), which is confusing, but not wrong (eax is moved into ebp-0x30 earlier in the function)
Re-typing the function is weird because it initially just shows as void* eax_1 despite it showing as taking 3 arguments in the HLIL view
Afterward, there is no change to the output whatsoever (aside from eax_1 being renamed to var_34_1)
There are clearly 5 arguments to the function, although one of them ("HTTP") is being pushed to an address instead of the stack which is kinda weird
Hitting y on var_34_1 shows the type as int32_t var_34_1 and not the type I gave it before (function with 5 arguments)
Changing the type to the same exact thing I had before just changes the variable name to eax_1, but this time, it shows the type correctly while the HLIL output still only shows 3 arguments despite the type now showing 5

Steps To Reproduce:
See above. The type I was trying to give to the function was void* eax_1(LPCSTR, DWORD, LPCSTR, LPCSTR, DWORD).
Expected Behavior:
As mentioned above, I expected HLIL to show 5 arguments for the function after changing its prototype instead of continuing to show 3. I also didn't expect the type for the variable eax_1 to continually be shown incorrectly throughout the process.

So, turns out I'm bad at computers at the type should have been void* (*eax_1)(LPCSTR, DWORD, LPCSTR, LPCSTR, DWORD). This works just fine if you don't do the wrong thing!
But: Shouldn't this be an error if you try to do this..? I can't think of an instance where applying a function type to a variable would be something you actually intend to do.

Here's the dialog you will get for this as of 3.2.3801-dev:


Upgrade minimum Vagrant version from 1.1 to 1.5

@smashwilson / @mdarby
WIP but ready for review. I want to get your review and feedback on the changes so far, but check w/ me before merging so I can do one last review and cleanup.
Fixes #125.

@maxlinc :shipit: