Controls sub-menu scene and keyboard controls

I just pushed a new branch
Could you have a look? I'm trying to implement a proper main menu but somehow neither up/down nor my controller is reacting and nothing is ever visibly selected.
When I do pick the second option with the mouse, the controls scene pops up but is somehow displaced by one or two pixels from the top.
I also found a bug. The enemies start spawning prior to the game actually starting (before any input was ever given).
My Godot editor keeps crashing whenever I work on the scene tree, so I'll need to reboot everything and change a kernel parameter (been having weird issues ever since I upgraded to 96GB of RAM).

I also found a bug. The enemies start spawning prior to the game actually starting (before any input was ever given).

I fixed it in a last commit

I also found a bug. The enemies start spawning prior to the game actually starting (before any input was ever given).

I fixed it in a last commit

Oh. Must've missed that. Thanks!

Could you have a look? I'm trying to implement a proper main menu but somehow neither up/down nor my controller is reacting and nothing is ever visibly selected.

Look at this. It's work fine for me. I cant switch by buttons, but they work and controlls fine. Actually i think we should remove this button and make this hint by pick-upable book as you offered

Could you have a look? I'm trying to implement a proper main menu but somehow neither up/down nor my controller is reacting and nothing is ever visibly selected.

Lookad at this. It's work fine for me. I cant switch by buttons, but they work and controlls fine. Actually i think we should remove this button and make this hint by pick-upable book as you offered

The book is unrelated to this. This is for the main menu, although I get what you're saying.
I just want to make sure there's a proper main menu, with the options "New Game", "Controls" and "Credits".
BTW: I'm gonna create the credits page over my lunch break. What should I credit you as? Your username or your real name? I'm putting you as "Code and level design: " since you did most of the code anyways.

Sure. You can write me as Kirill "Ken".

I added main menu and control scheme too
You can write control righ on a PNG or i can do it by title.
Check it


Rewrite how detecting trackers works

Currently we only support one tracker for each role, and don't support generic trackers, distinction between hand and hmd, etc.
My current plan is to iterate over all devices (how?) calling UpdateActionState, varying ulRestrictedToDevice. Then, whatever role is detected, can be assigned to that device.
Handle one device per tick, refilling the list every time it empties. This should spread it out over time, avoiding any sudden spikes.

thank you SDraw, https://github.com/SDraw/openvr_dumper
can use GetSortedTrackedDeviceIndicesOfClass to get my list of devices.

okay, i've hit a lot of dead ends with this regarding device indicies. My current plan is to keep things the way they currently are, and use the data from GetSortedTrackedDeviceIndicesOfClass to supplement, but without role information. That way, trackers will still show up even with a bad configuration, and then maybe users will be clued in that something is weird configuration-wise instead of thinking that the feeder app is broken.

I think I fixed this in #4 but never tied that PR to this issue. oops?


The Spyder Blog's RSS feed is gone

Until recently, I could get notifications of Spyder blog posts by subscribing to https://www.spyder-ide.org/blog/feed.xml with a feed reader. Unfortunately, this URL now gives Error 404.
I could not detect a replacement on the blog page. Is there one?

@conradolandia, could take a look at this one when you have time? Thanks!


Added favicon from https://favicon.io/emoji-favicons/books

nice books favicon from https://favicon.io/emoji-favicons/books
prevents the code from running past the file exists check in public/index.php because favicon does not exist

https://github.com/akrabat/slim-bookshelf/blob/16984915687943c15caf5c85a7d11d299d03796f/public/index.php#L4-L12

Needs an attribution at the bottom of the README to comply with the license.

Good point, I'm a bit at loss here. Maybe you can give me a hand:

The emoji graphics are from the open source project Twemoji. The graphics are copyright 2020 Twitter, Inc and other contributors. The graphics are licensed under CC-BY 4.0. You should review the license before usage in your project.

How should I put it?

Would it works, if I just copy paste that exact same line in your opinion?

Would it works, if I just copy paste that exact same line in your opinion?

Yes. I think that would work in a Credits section at the bottom of the README.


Added labs tag

@perja12, could you please add it to tags.yml? We have a limited set of tags that we use.


Select2 triggers onchange when init

When the page loads, the ui.select2 will trigger a change event even though the select wasn't touched. It seems to be this line's culprit: https://github.com/angular-ui/ui-select2/blob/v0.0.5/src/select2.js#L127
What does this do ?The comment above it doesn't really help ...

+1 for this!

ui-select2 is deprecated, you should use https://github.com/angular-ui/ui-select instead. It is very unlikely that this issue will be fixed.


Fixes missing auction artwork detail

Reported on #incidents channel.
The file has been changed in this commit. Still figuring out if it was intentional.
Before:

After:

I don't think this is quite right @sepans since I believe this metadata is supposed to appear under the artist name directly. I think we can just remove this line of logic and keep the details where they are in other cases.


local clone should not leave an empty-but-not-empty folder if it fails

This behaviour is confusing for the user
➜  ~ ~/git/lakeFS/lakectl local clone  lakefs://quickstart/main/images ./images2
download duckdb-editor-02.png failed: Get "http://minio:9000/quickstart/data/glpo6pv3chgc777qq2q0/cjdnv7f3chgc777qq2ug?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=mi
nioadmin%2F20230815%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20230815T165437Z&X-Amz-Expires=900&X-Amz-SignedHeaders=host&X-Amz-Signature=a7c743b4467e4b297ace94566d1a8741
1d70305e0f0a5ea75c607e5903c385ba": dial tcp: lookup minio: no such host
Error executing command.

➜  ~ ~/git/lakeFS/lakectl local clone  lakefs://quickstart/main/images ./images2
directory '/Users/rmoff/images2' exists and is not empty
Error executing command.

How can it fail (I got an error) but not fail (it's not re-runnable because the failed checkout left a not-empty folder) ?
(rhetorical question - I know why, but it's confusing nonetheless)
Suggestion: if an action fails it should rollback so that it can be re-run by the user.
Leaving a bunch of zero-byte files around locally is no help to man nor beast

@rmoff the clone operation is composed from an init and pull steps.
If the pull fails we are left with an init'ed directory (contains the lakefs index file). To fix this state you can either pull force or checkout the directory.
Regarding rollback - this is a larger issue than just clone, it pertains to all sync operations and can be complex to implement and prone to bugs. In my opinion we should not try to handle these states but rather let the user decide how to proceed using the their knowledge of the previous, current and desired state and with the help of the local commands.
There's an issue discussing how we can "invalidate" a directory in case of a failure mid flow

Thanks for the explanation of how it works.
BTW I looked for force but couldn't see it as an option
$ lakectl local clone --help
Clone a path from a lakeFS repository into a new directory.

Usage:
  lakectl local clone <path uri> [directory] [flags]

Flags:
      --gitignore         Update .gitignore file when working in a git repository context (default true)
  -h, --help              help for clone
  -p, --parallelism int   Max concurrent operations to perform (default 25)
      --pre-sign          Use pre-signed URLs when downloading/uploading data (recommended) (default true)

Global Flags:
      --base-uri string      base URI used for lakeFS address parse
  -c, --config string        config file (default is $HOME/.lakectl.yaml)
      --log-format string    set logging output format
      --log-level string     set logging level (default "none")
      --log-output strings   set logging output(s)
      --no-color             don't use fancy output colors (default when not attached to an interactive terminal)
      --verbose              run in verbose mode

Clone doesn't support a force flag. Only pull. Once the folder already went through init there's no need to clone again. Either pull (force) or checkout will do


Ask Questions / Write Articles not working!!

Hi, I deployed the lastest version locally following http://cookiecutter-django.readthedocs.io/en/latest/developing-locally.html , signed up , verified email using admin panel and logged in. But when I am trying to write articles / ask Questions this error comes up:
render() got an unexpected keyword argument 'renderer'
Error during template rendering
In template C:\Users\kaust\Desktop\globetrotter\bootcamp\env\lib\site-packages\crispy_forms\templates\bootstrap4\field.html, error at line 28
Any Idea why is this happening?

It looks you have a problem with the context variables passed through the crispy-forms package, but you're running this on Windows and I don't use Windows so I can't be of much help. I'm sorry for that.
Can you test it on Linux? Can you test it using the docker-compose scripts? To be honest, running development environments on Windows is hard for almost any software development language, and for Python it makes things even harder.

Hi @kaustav1996 do you have updates on this? Have you found something with this issue?

Hi @sebastian-code what I did was I removed the renderer argument from the function within crispy-forms in my env. I don't know what damage it will cause in the future.


Fix TypeScript definitions

The useTrackPlayerProgress hook can take null as the second argument
to trigger polling regardless of the player's state, but the type
definition for the hook does not allow null. This change adds null as
an allowed value to the type definition.
The TypeScript definitions do not currently include the STATE_CONNECTING
player state. This change adds that constant to the definitions.

Thanks for this @elliotdickison I think this is no longer necessary on v2 :)


Set exit status 126 if warnings like "Please run Mix again." happens

Sometimes Laravel mix requires additional dependencies, which installs automatically with console message: "Please run Mix again."
The problem is that it exists with a default exit code=0, so it looks like command npm run production completed successfully and can be deployed to production (so it was already happened).
The PR set exit code to 126 "Command invoked cannot execute" and it can be catched by a CI correctly as fail.

Good catch! One minor thing to fix and we can merge this in.

Good catch! One minor thing to fix and we can merge this in.

Fixed

Fixed

Thanks!

Thanks!


Yes, there are not enough "make Web Components Better"  tools

https://webcomponents.dev/blog/all-the-ways-to-make-a-web-component/

Please don't use issues to complain about not liking the concept of a project. This is a place for actual bugs and suggestions, not griping about the premise itself - no one is forcing you to use Facet if you prefer a different library's take on making web components less of a clunky mess of boilerplate. In fact, I'm happy to recommend some of my favorites:

Catalyst
Lit
Slim.js


[FLOC-2458] Add EMC links to Install doc

Fixes https://clusterhq.atlassian.net/browse/FLOC-2458
Merge #1677 into release/flocker-1.0.0 branch

changes look good, thankyou @wallrj


ContentTypes names html encoded twice

In ContentType admin, content type names are html encoded twice.

I think there is an issue with LocalizedString class :

Text, ToString() and ToHtmlString() returns the same result, it should not ?
Text and ToString() should return an unencoded string
ToHtmlString() should return the html encoded one
When using format, arguments are encoded but the translation is not...

Issue #5359 refers to the same.

We have the same issue on /Admin/Contents/Create : ContentType name is encoded twice in Layout.Title.


Weighted distribution across multiple variables

Could be a useful addition to your library. As an example, I'm interested in getting stats on race and gender in a group over time. Something like:
data_by_year = data.groupby(['year'])
race_gender_demographics = calc.distribution(data_by_year, ['race', 'gender']).round(3)

Hi @soooh! You should actually be able to do this with the current code. It'll depend on what, exactly, you're looking to calculate. But lets say you're looking for the weighted distribution of race, by gender and over time. In that case, this should work:
grouped = data.groupby([ "year", "gender" ])
dist = calc.distribution(grouped, "race").round(3)

Does that work? Are you aiming for something slightly different?

Ah, so what that does is give me the racial demographics of women and men separately. E.g., of all the women, 20% are white, 10% are black, and so on. What I want is something like, 10% of the group is white men, 8% white women, etc. Does that make sense?

Ah, so what that does is give me the racial demographics of women and men separately. E.g., of all the women, 20% are white, 10% are black, and so on.

Yep.

What I want is something like, 10% of the group is white men, 8% white women, etc. Does that make sense?

Ah, sounds like I misunderstood the goal. In that case, the easiest way might be like so:
data["race_x_gender"] = data[[ "race", "gender" ]].apply(" x ".join, axis=1)
dist = calc.distribution(data.groupby("year"), "race_x_gender").round(3)

Does that achieve your goal? (It assumes that race and gender are strings.)
I'll also think about ways I could incorporate a generic feature like this into the library itself. Thanks for the suggestion!

Ah yes, that is actually what I am doing! 😄
I thought it could be a useful feature, though, which is why I suggested it.


not able to download dataset in step 1

Hi,
Thanks for your work and sharing the dataset.
I am not able to download the dataset at all with following errors, could I get some help on what I have missed. Thanks.
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   233  100   233    0     0    272      0 --:--:-- --:--:-- --:--:--   272
gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now

Hi @bragilee
Thank you for your interest in our dataset.
We are now using a new website for our data!
I updated the link in the repo and it should be working now.
Please let me know if you need anything else.


refactor(payment_methods): Add task to PT for revocation of connector mandate details

Type of Change

[ ] Bugfix
[ ] New feature
[ ] Enhancement
[x] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description
Add task to PT for revocation of connector mandate details
Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Checklist

[ ] I formatted the code cargo +nightly fmt --all
[ ] I addressed lints thrown by cargo clippy
[ ] I reviewed the submitted code
[ ] I added unit tests for my changes where possible

Can you add some test cases @Aprabhat19


CDN url broken

Hello,
The<EMAIL_ADDRESS>url is broken, no "dist" folder ?

I think all that's needed is a new deployment to npm and unpkg will mirror it.

Is it much work to get a new version deployed to npm?

This seems to now be fixed after the 4.11.0 release.


VPC Flow Logs

What:
Ronak has contacted us as access logs for management vpc on prod seems to be broken. VPC is below:
vpc-0de4ba808005983d2 / Management VPC

I had a look at the VPC flow log issue in management VPC the reason it was not working as the referenced role in the flow log config does not exists. As this is not amendable I have re-created a vpc flow subscription with the new role I have created. It should start collecting logs from now on.
https://eu-west-2.console.aws.amazon.com/cloudwatch/home?region=eu-west-2#logsV2:log-groups/log-group/$252Faws$252Fvpc-flow-log$252Fvpc-0de4ba808005983d2


2 Suggestions

I suggest is an option to remove the first and second guidelines, in C# is namespace and class.
The second suggestion is an option to set the minimum size in lines that a guideline appears. An if with 1, 2 or 3 lines don't need a guideline.
Great extension!

Hi @onolox. Thanks for the suggestions. I believe the suggestions are about the "structure guidelines" feature that's built into Visual Studio. Because of that, please close this issue and open up a feedback ticket through Visual Studio's built-in feedback mechanism (see screen snipping below)


 The "Right click action" setting seemed to be what I wanted, but it only ever does the expected behavior for the "Cancel drag" never "Delete waypoint". Conversely, the "Create waypoint" seems to be working fine.

"Delete Waypoint" only removes waypoints that have been placed during the current drag. Waypoints in the movement history cannot be removed using the "Delete Waypoint" option. If you want to reset the movement history for a token, you can right-click the token in the combat tracker as GM and select "Reset movement history".

Then I guess this is a feature request to have an option to "delete most recent waypoint" on right click. (maybe even another option for a reset on right click as well)

I am also interessted in a way for players to remove their waypoint history. Ideally automatically moving to the last location each time until they reach their start.

Removing single waypoints can already be done by anyone (including players). Foundryvtt allows undoing a token movement by pressing Ctrl+Z. When doing so, Drag Ruler will remove the corresponding waypoint from the movement history.

I just tested it and it does work. I just want to add for people not using a QWERTY layout, that you need to press the key, where the Z would be. So it's Ctrl+Y for german layouts.
Don't you use a german layout as well, @manuelVo?

Yes, that is a bug in Foundry core. The Keyboard Layout module allows you to specify your keyboard layout, which fixes the keyboard shortcuts.


Dependency issue with "make test" and libcrypto.so not re-built from newer sources

I just switched from an older branch (I think I built last from commit d29d7a7ff2) to the latest master commit dce7272d08.
Trying to run  make -j4 -s test TESTS="test_verify" I got
make: *** No rule to make target 'Configurations/common.tmpl', needed by 'configdata.pm'.  Stop.

Thus I had to re-configure, and did so using ./configdata.pm -r.
Then I ran make -j4 -s test TESTS="test_verify" again and got a crash:
apps/openssl verify -Using HARNESS_JOBS=4c", "", ["bad-othername-namec-inter"], [], "-partial_chain"
00-prep_fipsmodule_cnf.t .. skipped: FIPS module config file only supported in a fips build
Files=1, Tests=0,  1 wallclock secs ( 0.02 usr  0.01 sys +  0.26 cusr  0.00 csys =  0.29 CPU)
Result: NOTESTS
25-test_verify.t .. 112/156
../../util/wrap.pl ../../apps/openssl verify -auth_level 1 -partial_chain -trusted ../../test/certs/bad-othername-namec-inter.pem ../../test/certs/bad-othername-namec.pem => 139
not ok 134 - Name constraints bad othername name constraint
25-test_verify.t .. 135/156 ----------------------------------------------------
25-test_verify.t .. Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/156 subtests

Test Summary Report
-------------------
25-test_verify.t (Wstat: 256 Tests: 156 Failed: 1)
  Failed test:  134
  Non-zero exit status: 1
Files=1, Tests=156,  6 wallclock secs ( 0.11 usr  0.01 sys +  5.52 cusr  1.18 csys =  6.82 CPU)
Result: FAIL

which is more apparent with
apps/openssl verify -auth_level 1 -partial_chain -trusted test/certs/bad-othername-namec-inter.pem test/certs/bad-othername-namec.pem
Segmentation fault

and
gdb --args apps/openssl verify -auth_level 1 -partial_chain -trusted test/certs/bad-othername-namec-inter.pem test/certs/bad-othername-namec.pem
GNU gdb (Debian 8.2.1-2+b3) 8.2.1
...
(gdb) run
Starting program: apps/openssl verify -auth_level 1 -partial_chain -trusted test/certs/bad-othername-namec-inter.pem test/certs/bad-othername-namec.pem
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/usr/lib/x86_64-linux-gnu/libthread_db.so.1".

Program received signal SIGSEGV, Segmentation fault.
0x00007ffff7d3f0fd in nc_email_eai (eml=0x0, base=0x50ba70) at crypto/x509/v3_ncons.c:603
warning: Source file is more recent than executable.
603	    const char *baseptr = (char *)base->data;
(gdb) bt
#0  0x00007ffff7d3f0fd in nc_email_eai (eml=0x0, base=0x50ba70) at crypto/x509/v3_ncons.c:603
#1  0x00007ffff7d3eedd in nc_match_single (gen=0x507030, base=0x511ea0) at crypto/x509/v3_ncons.c:524
#2  0x00007ffff7d3ed71 in nc_match (gen=0x507030, nc=0x5061a0) at crypto/x509/v3_ncons.c:485
#3  0x00007ffff7d3e900 in NAME_CONSTRAINTS_check (x=0x52a7b0, nc=0x5061a0) at crypto/x509/v3_ncons.c:288
#4  0x00007ffff7d531e5 in check_name_constraints (ctx=0x5078f0) at crypto/x509/x509_vfy.c:738
#5  0x00007ffff7d51e55 in verify_chain (ctx=0x5078f0) at crypto/x509/x509_vfy.c:232
#6  0x00007ffff7d52154 in X509_verify_cert (ctx=0x5078f0) at crypto/x509/x509_vfy.c:294
#7  0x0000000000480ac4 in check (ctx=0x511990, file=0x7fffffffe03f "test/certs/bad-othername-namec.pem", uchain=0x0,
    tchain=0x503b20, crls=0x0, show_chain=0, opts=0x0) at apps/verify.c:290
#8  0x00000000004807cc in verify_main (argc=1, argv=0x7fffffffdbd0) at apps/verify.c:229
#9  0x000000000044cc48 in do_cmd (prog=0x50a740, argc=7, argv=0x7fffffffdba0) at apps/openssl.c:414
#10 0x000000000044c877 in main (argc=7, argv=0x7fffffffdba0) at apps/openssl.c:295
(gdb) quit

This must have been due to inconsistent object files, which is also somewhat indicated by one of the GDB output lines:
warning: Source file is more recent than executable.

It turned out that crypto/x509/libcrypto-{sh.}lib-v3_ncons.{d,o} were freshly built from crypto/x509/v3_ncons.c,
and also libcrypto.ld, but not all other libcrypto.* including libcrypto.so.3.
Also apps/openssl was not fresh.
A second call to make -j4 -s test TESTS="test_verify" correctly re-built r libcrypto.* as well as apps/openssl and the tests passed.
So some dependencies of the test target must be wrong in the Makefile.
I suspect the problem does not arise if a plain make (without test) is called before.

After switching forth and back and forth from commit 17213b2ad03fe577c03a3e77142242adde41fa9f to the current master (commit dce7272d08601929a494b9367f4e70163c524cb3),
on make -j4 -s test TESTS="test_verify" I even got
../../apps/openssl: error while loading shared libraries: ../../util/../util/../libcrypto.so.3: file too short

Calling make -j4 -s test TESTS="test_verify" again lead to a successful test run.

When jumping between branches that differ in their setup, these things happen.  This is where I usually do make clean; ./configdata -r

Perhaps we should somehow force full rebuild after ./configdata.pm -r ?

When jumping between branches that differ in their setup, these things happen. This is where I usually do make clean; ./configdata -r

Doing such a workaround is awkward and getting a crash just because of such an inconsistency can be pretty misleading.
make testetc. should take care of that (or at least warn if they can't).

Perhaps we should somehow force full rebuild after ./configdata.pm -r ?

As per my first comment, I got build inconsistencies w.r.t. apps/openssl also without reconfiguring.

Doing such a workaround is awkward and getting a crash just because of such an inconsistency can be pretty misleading.
make testetc. should take care of that (or at least warn if they can't).

I think you're expecting magic where there isn't magic to be had.  What essentially happens is that files that are collected when configuring - and that are the foundation that Makefile builds upon - are being moved around, the Makefile gets out of sync with reality.
As per my first comment, I got build inconsistencies w.r.t. apps/openssl also without reconfiguring.  This error tells you exactly that:
make: *** No rule to make target 'Configurations/common.tmpl', needed by 'configdata.pm'.  Stop.

This particular error message is a sure tell tale sign that the Makefile is incorrect and should get regenerated.
What you're asking for is that Makefile become aware of how files have moved around during development.
Quite frankly, I find that a big ask.

As per my first comment, I got build inconsistencies w.r.t. apps/openssl also without reconfiguring.
This error tells you exactly that:

make: *** No rule to make target 'Configurations/common.tmpl', needed by 'configdata.pm'.  Stop.

This particular error message is a sure tell tale sign that the Makefile is incorrect and should get regenerated.

This is not what I meant in my first comment, but:
There are also cases where switching between branches does not require reconfiguring,
(or at least the Makefile does not not warn about this), and still make test goes wrong,
while make && make test would have succeeded.

I think you're expecting magic where there isn't magic to be had.

Whatever the meachnism is that is already in use when doing make build_sw,
I can't believe it would require magic to make that happen also for make test.

I can't believe it would require magic to make that happen also for a direct make test.

Actually, it does.  The thing that triggers an issue here is that you use parallel making (-j 4).  All you have to do is look at this line from the produced Makefile:
_tests: build_programs_nodep build_modules_nodep link-utils run_tests

What happens is that run_tests gets to run in parallel with build_programs_nodep, build_modules_nodep and link-utils, so there's a high chance that some things aren't done building in time to run the diverse test programs that expect the building part to be done.  That's a sure recipe to get all kinds of pretty weird errors.
I currently have my head elsewhere and no cycles to spend on this in depth.  The quick fix is to revert c612c7a455d9b3ea602c87fe720d09535f1f6e37

The thing that triggers an issue here is that you use parallel making (-j 4). All you have to do is look at this line from the produced Makefile:
_tests: build_programs_nodep build_modules_nodep link-utils run_tests

Ah, that's an interesting explanation!

Do out-of-tree builds. Use ccache and "make clean" Lots of options are simpler and cleaner than trying to nail down everything for a non-common case.

I think attempting to directly do a make test is not uncommon.
And it almost does the right thing, while occasionally (e.g., when the -j option is involved) it can move into weird states leading to crashes, which is no good style, at least when this can happen without warning.
Making make test more solid should not be too hard (at least for the -j use case; I'll take care of that),
and I see various types of users that will benefit from this: inexperienced, impatient, and busy ones :)

@DDvO did you ever open a PR for this?  If so please link it here and I'll mark this as a community issue, otherwise, it seems it should be closed

It's possible that I fixed it. Why I didn't refer to this issue, I cannot recall...

@levitte I expect it has something to do with the fact that we have 2000+ issues to keep in our heads  :0
@DDvO, I expect you may want to check on current behavior before considering a PR

@DDvO did you ever check on the current behavior here?


Unit testing the rules logic

Hi All, This JSON based rules engine with lambda expression support is great. I can understand that we will be to write Integration tests for the rules configurations. But how can we have Unit tests for the Lamba expression which we write in the rules expression ?. If not unit tested, would not be error prone. ?

Hi @ramkrivas , wouldn't running RulesEngine with the actual rules and test inputs suffice for unit testing? Asserting whether a rule returns success, failure or exception can be a good candidate for unit testing.
We have multiple such test cases to ensure RulesEngine behaves as expected. You could look at it for recipe for your own unit testing.
Let me know if this is what you are looking for 😄

This is a great project. I have a need for a rules engine and after starting to roll my own I stumbled upon this.
I'm not sure if what @ramkrivas is asking is the same as what I'm curious about, but maybe he's asking about Lambda expression validation? I've tried putting garbage values into the expression that reference properties that do not exist in my input object and the rule compilation succeeds. The evaluation of the rule returns False. Ideally what I'd like is to be able to throw a sample object into a rule and see if the lambda expression is valid given the input object.
For example, here my input object specifies first and last name, but the expression is using "surname" instead of "lastName." Is this logic error detectable by some means? If my question and @ramkrivas are completely different please let me know and I'll start my own thread. Thanks.
// Input object
{
    "firstName": "John",
    "lastName": "Smith"
}

// Rule lambda expression
input1.surname == "Smith"

Hi @michaelstaszewski ,
RulesEngine by default swallows the exceptions in Rule parsing as shows them as ErrorMessage in result.
If you need it to throw exception, you can set EnableExceptionAsErrorMessage to false in reSettings
https://github.com/microsoft/RulesEngine/blob/77a570ca91be6bd1e501f6fe948e641df7c4a85a/src/RulesEngine/Models/ReSettings.cs#L16
This will make it easier for you to catch any Error in rule parsing.

Perfect. That's exactly what I was looking for, thanks!

@ramkrivas i am closing this issue due to inactivity. If you still have further questions on this, feel free to reopen this issue


Handle overloaded forward function declarations.

Enhance CqlPreprocessor visitor to compile function headers (AKA "preCompile")
Introduce a new common base class for CqlPreprocessor and Cql2ElmVisitor
Extract common methods from Cql2ElmProcessor to the new superclass
Store expression definition as a hash generated from the semantic function definition instead of a function String
Move loading of the system library from Cql2ElmProcessor to CqlPreprocessor
Move loading of the model from Cql2ElmProcessor to CqlPreprocessor
Note that errors will now be compiled in CqlPrerprocessor as they are in Cql2ElmProcessor instead of letting Exceptions get thrown
Introduce a new ForwardInvocationValidator that will a CallContext derived fro the calling function's representation of the called function versus an eligible function derived from a list of functions found by searching from a method name. This includes comparing each side's list of parameter DataTypes as well as taking into account implicit conversions (ex FHIRHelpers)
Add new unit tests for various scenarios including dealing with overloads with normal types, generic types, as well as implicit conversions and namespace collisions (ex: System.Quantity vs FHIR.Quantity)

Closes: https://github.com/cqframework/clinical_quality_language/issues/1191

@lukedegruchy - I merged master into this branch and fixed a minor typo. However, that exposed an issue:

The new test for a better type error message is failing because it reports the same error twice.
It reports the same error twice because visitNamedTypeSpecifier is executed twice.
VisitNamedTypeSpecifier is executed twice because it's in base ElmCommonVisitor and the parse tree is visited in its entirety by both the PreProcessor and the Cql2ElmVisitor

One potential fix for that is to update the PreProcessor to do all that type resolution itself, store that information on the library builder, and then override the Cql2ElmVisitor to no-op for visiting TypeSpecifiers.


Pull fedora 35 image from fedora registry

Fedora 35 image has been removed from quay.io/fedora/fedora, use fedora registry, later we should upgrade to newer version.

+1 LGTM


How to view container state in react-dev-tool ?

I find it's hard to know what is the current state of the container, is there a way ?

You can inspect the state using React DevTools:

Is that not enough?

You can also track progress on how hooks are displayed in React DevTools here: https://github.com/facebook/react-devtools/issues/1215


Add support for Python 3.7

Bumps Web3.py version to 4.5.0 for Python 3.7 support. Small change to gitignore to handle multiple env* folders.

Merged.


MissingMetadataException at System.Linq.Expressions.Expression`1.Compile

Given this extended HelloWorld-Program:
    public class Person
    {
        public string Name { get; set; }
    }

    class Program
    {
        static void Main(string[] args)
        {
            Expression<Func<Person, string>> e = x => x.Name;

            var f = e.Compile();

            var p = new Person() { Name = "Joe" };

            Console.Write($"Hello, my name is '{f(p)}'.");

        }
    }

from Microsoft.DotNet.ILCompiler -v 1.0.0-alpha-26430-01 onwards this programm fails:
Unhandled Exception: System.Reflection.MissingMetadataException: This operation cannot be carried out because metadata for the following object was removed for performance reasons:\n\n  <unavailable>\n\nNo further information is available. Rebuild in debug mode for better information.\n\n
   at HelloWorld!<BaseAddress>+0xa532c
   at HelloWorld!<BaseAddress>+0x1587e0
   at HelloWorld!<BaseAddress>+0x41d28
   at HelloWorld!<BaseAddress>+0xa6753
   at HelloWorld!<BaseAddress>+0xa6066
   at HelloWorld!<BaseAddress>+0xa64d6
   at HelloWorld!<BaseAddress>+0xa54f6
   at HelloWorld!<BaseAddress>+0xc1faf
   at HelloWorld!<BaseAddress>+0xc3c2f
   at HelloWorld!<BaseAddress>+0xc3dfd
   at HelloWorld!<BaseAddress>+0xb97d1
   at System.Linq.Expressions.Expression`1.Compile(Boolean) + 0x23
   at HelloWorld.Program.Main(String[]) + 0xbc
   at HelloWorld!<BaseAddress>+0x6460fa

with-v 1.0.0-alpha-26428-01 and earlier the result is:
Hello, my name is 'Joe'.

Code and rd.xml: HelloWorld.zip, to reproduce, follow the instructions in README.md

It fails for on 1.0.0-alpha-26428-01 as well.
When the following added to .csproj
  <ItemGroup>
    <IlcArg Include="--stacktracedata" />
  </ItemGroup>

to get full stacktraces, the exception is:
Unhandled Exception: System.Reflection.MissingMetadataException: 'System.Linq.Expressions.ExpressionCreator<System.Func<Person,System.String>>' is missing metadata. For more information, please visit http://go.microsoft.com/fwlink/?LinkID=392859
   at System.Reflection.Runtime.TypeInfos.RuntimeConstructedGenericTypeInfo.ConstructedGenericTypeTable.Factory(RuntimeConstructedGenericTypeInfo.UnificationKey) + 0x1d5
   at System.Collections.Concurrent.ConcurrentUnifierWKeyed`2.GetOrAdd(K) + 0xe7
   at System.Reflection.Runtime.TypeInfos.RuntimeConstructedGenericTypeInfo.GetRuntimeConstructedGenericTypeInfo(RuntimeTypeInfo, RuntimeTypeInfo[], RuntimeTypeHandle) + 0x70
   at System.Reflection.Runtime.TypeInfos.RuntimeConstructedGenericTypeInfo.GetRuntimeConstructedGenericTypeInfo(RuntimeTypeInfo, RuntimeTypeInfo[]) + 0x58
   at System.Reflection.Runtime.General.TypeUnifier.GetConstructedGenericType(RuntimeTypeInfo, RuntimeTypeInfo[]) + 0x24
   at System.Reflection.Runtime.TypeInfos.RuntimeTypeInfo.MakeGenericType(Type[]) + 0x3d4
   at System.Linq.Expressions.Expression.CreateLambda(Type, Expression, String, Boolean, ReadOnlyCollection`1) + 0x100

System.Linq.Expression does not just work with CoreRT right now. It requires additional hints in rd.xml file, like: https://github.com/dotnet/corert/blob/master/samples/WebApi/rd.xml#L61

Out of interest, why isn't --stacktracedata turned on by default, it seems like a useful thing to have available?
Is the problem that it increases the size of the metadata by too much? It exposes internal details that aren't meant to be publicly visible? Or something else?

It is just that it increases the size of the metadata. It is not that much, but definitely noticeable.
We had endless discussion within the team about how defaults like these should be tuned. Should the defaults be optimized for size, to get binary sizes or compilation times that are more competitive with other AOT environments; or should the defaults should be optimized for more .NET-like developer experience at the cost of larger binary sizes? It is a mix right now.

@jkotas thanks, i didn't knew --stacktracedata.
Unfortunatly I can not resolve this issue, even with the additional information.
I already had the ExpressionCreator in the rd.xml:
<Directives>
    <Application>
        <Assembly Name="HelloWorld" Dynamic="Required All" />

        <Assembly Name="System.Linq.Expressions">
            <Type Name="System.Linq.Expressions.ExpressionCreator`1[[System.Func`2[[HelloWorld.Person,HelloWorld],[System.String,System.Private.CoreLib]],System.Private.CoreLib]]" Dynamic="Required All" />
        </Assembly>

    </Application>
</Directives>

The new errormesage with --stacktracedata  is:
Unhandled Exception: System.Reflection.MissingMetadataException: 'System.Linq.Expressions.Interpreter.FuncCallInstruction<HelloWorld.Person,System.String>' is missing metadata.

I tried to add:
<Type Name="System.Linq.Expressions.Interpreter.FuncCallInstruction`2[[HelloWorld.Person,HelloWorld],[System.String,System.Private.CoreLib]]" Dynamic="Required All" />

but still the same errormessage.
With a version from two weeks ago (dotnet add package Microsoft.DotNet.ILCompiler -v 1.0.0-alpha-26428-01) this code works fine.

@danmosemsft do you know who should be monitoring CoreFX UWP test runs and filing bugs? Is there anyone who could have a look at this? From the logs it seems like the compilation succeeded and there is some kind of infra issue preventing the tests from running and it has been like this for a while.

@MichalStrehovsky I look at the CoreFX UWP results periodically but primarily to make sure they are not getting significantly worse, and typically the breaks are due to changes that have not been ported to ProjectN.  The tests are not broken consistently, for example I do see results from 510:
https://mc.dot.net/#/product/netcore/prodconmaster/source/prodcon~2Fcorefx~2Fmaster~2F/type/test~2Ffunctional~2Filc~2F/build/20180510.01/workItem/System.Linq.Expressions.Tests/wilogs
It does seem like the runs are flaky though. I'll check whether there is anyone looking at that.

In the past changes incompatible with N would fail in CI. I definitely remember that happening, so I assumed that N is fine with this change. Otherwise I'd exclude the optimizations when targeting AOT. -
That is why the feature flags are used. I had a feeling there might still be some reasons that AOT would not want this.

The perf difference between delegate invoke and reflection invoke are enormous (in a 100x range, I believe). I doubt 4x in N would be enough to make it insignificant.
Reflection invoke also allocates intermediate argument buffers on every call . I am not sure if N fixes that, but when every invocation also allocates, it could translate into lots of garbage.
Using expression tree lambdas  (or C# dynamic) as a way to avoid per-call costs of reflection invoke on hot paths is a very popular "trick". When the "trick" stops working, the penalty to the app/library that relies on it could be very severe. I have seen bug reports where this perf hit was mentioned as a reason to avoid UWP entirely.
It is a bit unfortunate that these optimizations could not be used in UWP.


Add extension method to configure for FunctionsStartup

Is it possible to have an extension for the v3 FunctionsStartup which works with the new configure method public override void Configure(IFunctionsHostBuilder builder) as opposed to the old IWebJobsStartup.
If not, are you aware of how to configure these side-by-side?

@Tim-Hodge Hi Tim,
Yep, it's possible, though it's kinda hacky.
IFunctionsHostBuilder does not support "AddExtension" method (now)
that allows to Bind Extensions.
So, underneath, I still need to build ServiceProvider and create an instance of IWebJobsBuilder.
            var wbBuilder = builder.Services.AddWebJobs(x => { return; });

Also, it's possible to have 2 startup classes:
    public class Startup : FunctionsStartup
    {
        public override void Configure(IFunctionsHostBuilder builder)
        {
            // all the DI
        }
    }

    public class WebJobsStartup : IWebJobsStartup
    {
        void IWebJobsStartup.Configure(IWebJobsBuilder builder)
        {
           /// All the extensions
        }
    }

Anyway, I've added 3.2.0-beta version Nuget, that supports IFunctionsHostBuilder.
Please, try it.

Hi @vitalybibikov - can confirm  3.2.0-beta works with IFunctionsHostBuilder, thank you very much!


fix(ci): fix multi-line command

Pull Request Test Coverage Report for Build 508421563

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 49.416%

Totals

Change from base Build 508413498:
0.0%

Covered Lines:
1354

Relevant Lines:
2740

💛  - Coveralls

Pull Request Test Coverage Report for Build 508421563

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 49.416%

Totals

Change from base Build 508413498:
0.0%

Covered Lines:
1354

Relevant Lines:
2740

💛  - Coveralls


Sorting channel name search suggestions based on relevance

When I'm looking up a channel by name, and type in a few letters, the suggestions that pop up don't seem to have any particular system or order to them. For channels with overlapping names, sometimes this makes the unwanted results show up first. It is most obvious when searching for 'korone' or 'gura'.
I suggest, if possible, reordering the results in the 'channel' category:

Channels from the currently selected organization should appear before all other channels (Inugami Korone vs Korone Pochi)
Talent channels should maybe appear before clipper channels ('Coco' ch. vs Felix172 ['Coco' clips only])
Full word match should maybe go before partial word match ('Mori' Calliope vs Hime'mori' Luna)

Fixed in #182


[Bug] Fix names in structs!

Before submitting a new issue, please verify and check:

[x] The issue is specific to Raylib-cs and not raylib
[x] I checked there is no similar issue already reported
[x] My code has no errors or misuse of Raylib-cs

Issue description
Hello, i saw variable names that not agree with the C# rules.
Example:
Yours:
    public partial struct Transform
    {
        /// <summary>
        /// Translation
        /// </summary>
        public Vector3 translation;

        /// <summary>
        /// Rotation
        /// </summary>
        public Quaternion rotation;

        /// <summary>
        /// Scale
        /// </summary>
        public Vector3 scale;
    }

How it should look like:
    public partial struct Transform
    {
        /// <summary>
        /// Translation
        /// </summary>
        public Vector3 Translation;

        /// <summary>
        /// Rotation
        /// </summary>
        public Quaternion Rotation;

        /// <summary>
        /// Scale
        /// </summary>
        public Vector3 Scale;
    }

Environment
*Raylib-cs

@ChrisDill What you think about it?

Open to trying this out and Transform seems like a good place to start with this.

This will require all existing code using this struct to be updated. Are we going to limit this to the next major release?

Its not that big deal, but lets talk to chrisdill

@MrScautHD It is a breaking change so I agree with @9ParsonsB that it should go into a major release. Considering creating a separate branch to track and test breaking changes like this.

@MrScautHD It is a breaking change so I agree with @9ParsonsB that it should go into a major release. Considering creating a separate branch to track and test breaking changes like this.

ok, yea then lets do a own branch for it.

Can you create one, so we can do pull requests for it?

@MrScautHD Done, dev branch created. Can you fix the conflicts then I can merge it in.

thx, yea sure

@MrScautHD Done, dev branch created. Can you fix the conflicts then I can merge it in.

I resolved the conflicts, i even made a few mistakes by the names that are fixed now to :)

@ChrisDill is there anything that need todo?

I have added a dev branch to the examples to test the changes and check for any issues. Looks good so far!
Nothing specific at the moment. Thinking about other design changes I may make but have not decided yet.

I have added a dev branch to the examples to test the changes and check for any issues. Looks good so far!
Nothing specific at the moment. Thinking about other design changes I may make but have not decided yet.

i made some checkes too, and it looks fine for me.

Yes Please!
I always hated how raylib-cs didn't use C# naming conventions.

I saw you renamed the Rlgl methods what about we remove from the methods the rl

@MrScautHD Initially I changed them back to get it working with the examples. Now that is done we can try removing the prefix and see how that looks. Needs the EntryPoint to be set so the name can be different.

@MrScautHD Initially I changed them back to get it working with the examples. Now that is done we can try removing the prefix and see how that looks. Needs the EntryPoint to be set so the name can be different.

yea i forgot about the EntryPoint...
Instead of:
[DllImport(nativeLibName, CallingConvention = CallingConvention.Cdecl)]
public static extern void rlEnd();

Lets do:
[DllImport(nativeLibName, EntryPoint = "rlEnd", CallingConvention = CallingConvention.Cdecl)]
public static extern void End();

If you agree with that i could do that for the full class

        // GL blending functions/equations
        public const int RL_FUNC_ADD = 0x8006;
        public const int RL_MIN = 0x8007;
        public const int RL_MAX = 0x8008;
        public const int RL_FUNC_SUBTRACT = 0x800A;
        public const int RL_FUNC_REVERSE_SUBTRACT = 0x800B;
        public const int RL_BLEND_EQUATION = 0x8009;
        public const int RL_BLEND_EQUATION_RGB = 0x8009;
        public const int RL_BLEND_EQUATION_ALPHA = 0x883D;
        public const int RL_BLEND_DST_RGB = 0x80C8;
        public const int RL_BLEND_SRC_RGB = 0x80C9;
        public const int RL_BLEND_DST_ALPHA = 0x80CA;
        public const int RL_BLEND_SRC_ALPHA = 0x80CB;
        public const int RL_BLEND_COLOR = 0x8005;

Should we remove the RL before the names to?

it would looks way cleaner... i removed it with my latest pull request.

And i would suggest to rename the Raylib_cs namespace to Raylib

Fixed https://github.com/ChrisDill/Raylib-cs/commit/23ed54cc2455f6527be72ff86f90f2b8bdc8c5af


dynamic import on summary upload manager in odsp driver.

Description
Testing dynamic import on summary upload manager in odsp driver.

This is not giving me any gain but I see that the odspSummaryUploadManager module is wrapped in separate package. So trying to see what is the reason.

@NicholasCouri Please take a look when you get time. Thanks


Run button disabled while indexing

What steps will reproduce the problem?

do something like delete node_modules then a yarn install on a large project that will cause a lot of indexing to kick off.
Try to click the "run" button for a configured script
What is the expected result?
Should be able to run things even though WS is still indexing files.
What happens instead?
Run button is disabled until indexing finishes. This wasn't the case in previous versions of WS.

Details can be found here: https://youtrack.jetbrains.com/issue/WEB-33228

Fixed for the next update.


feat(docs-website): Adding quickstart guides on Python SDK / GraphQL

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Description
Demo
You can check out how it's gonna be look like via this link :
https://640911dcbddab40008b96a0d--elegant-zuccutto-a03114.netlify.app/docs/dev-guides/tutorials/why-we-are-doing-it
Changes
Adding quickstart guides on Python SDK / GraphQL.
Questions I have at the moment :

are we going to include more use cases?
are we going to include more methods?
where should this be located?

Things to do before merge
These files should be modified as it was.

github/workflows/documentation.yml
docs-website/docusaurus.config.js

@maggiehays  This should've been merged into the datahub repo, not this one

@hsheth2 😢 How can we handle this nicely? merge this one with the main datahub one?

I think we can open the same PR on the datahub side, and then we'll have to resolve the conflicts when merging datahubproject/datahub into acryldata/datahub


Remove top level links if configured

Looks like links can be both top level and resource.  This removes the top level as well if configured.

Thank you @snewcomer 😁
@jeregrine any thoughts?

Just pinging this again.  Thanks for the reviews!

Thank you @snewcomer 👍   Until @jeregrine can cut a new release you'll need to use master:
{:jsonapi, github: "jeregrine/jsonapi"}

Sorry for delay on this. I hope to jump back in and cut a new release next
week. Thanks for covering Sean!
On Tue, Dec 19, 2017 at 5:29 PM, Sean Callan<EMAIL_ADDRESS>wrote:

Merged #88 https://github.com/jeregrine/jsonapi/pull/88.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jeregrine/jsonapi/pull/88#event-1395171744, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGKFqLA9mHq_srDStsucAk82WEY4AIsks5tCEbegaJpZM4RA8-j
.

You got it @jeregrine!


Barycentric Lagrange interpolation

Companion PR required for the SABER PR https://github.com/JCSDA-internal/saber/pull/63

It is strange: tests are all passing on my workstation with GNU 9.3.0, starting from a fresh build. Could anyone try to run the tests of this branch and update the test references accordingly?

Replaced by #531.


Question

Hey there i was wondering if there was a way to contact you, i'd like to ask some questions privately
my discord is: NotZyo#1032
and my email is<EMAIL_ADDRESS>

Hey there i was wondering if there was a way to contact you, i'd like to ask some questions privately
my discord is: NotZyo#1032
and my email is<EMAIL_ADDRESS>
email<EMAIL_ADDRESS>discord: Hype#5026


Electrolytic separator not working with energy upgrades.

Electrolytic separators, despite saying they are compatible with energy upgrades, don't gain full functionality from them.
You can add the normal eight and the amount of power it stores goes up, but the energy usage does not go down at all.
Using
Mekanism-1.7.10-<IP_ADDRESS>1
MekanismGenerators-1.7.10-<IP_ADDRESS>1
MekanismTools-1.7.10-<IP_ADDRESS>1

it is intended that energy upgrades not work on it, to not abuse hydrogen generated power

Ahhh, ok then.
I was just lamenting that it meant ore multiplication required now ridiculous sums of energy to manage with the volume of oxygen required.
Thank you.

Tho, thinking about it, if people wanted to make oxygen energy efficiently, for ore purification,
It could be made that an electrolytic separator, with energy upgrades, has the energy reduction bonus, but either makes half the default per energy upgrade of hydrogen, or no hydrogen at all?

This would not be realistic anymore because the results are not equal anymore to the H2O molecule.

Well then perhaps in increasing the energy efficiency of oxygen production means the hydrogen can no longer be efficiently captured and collected?

Could simply say that it is more energy effecient because it is using more and more of the hydrogen it produces to power itself.

get a new block which only gewnerates oxygen capturing from the air directly. like the one from galacticraft. that way you can raise oxygen production to its limits without abusing hydrogen

That would be great, or possible even something you have to pipe organic matter in, in order to generate oxygen in large quantities.


Attempting to decode images with extra data after EOI marker fails

Seems like some JPEG encoders stuff extra data after EOI marker (see e.g. https://exiftool.org/forum/index.php?topic=4374.0). We bumped into this in real world usage (with react-pdf).
Would it be possible to end decoding at EOI marker and ignore rest of the data?

This is a patch that can be used to fix this, although doesn't feel like the ideal way:
diff --git a/node_modules/jay-peg/src/index.js b/node_modules/jay-peg/src/index.js
index ba77471..d19a573 100644
--- a/node_modules/jay-peg/src/index.js
+++ b/node_modules/jay-peg/src/index.js
@@ -7,9 +7,9 @@ import DRIMarker from "./markers/dri.js";
 import EndOfImageMarker from "./markers/eoi.js";
 import EXIFMarker from "./markers/exif.js";
 import JFIFMarker from "./markers/jfif.js";
-import SOSMarker from "./markers/sos.js";
 import StartOfFrameMarker from "./markers/sof.js";
 import StartOfImageMarker from "./markers/soi.js";
+import SOSMarker from "./markers/sos.js";

 const UnkownMarker = {
   length: r.uint16be,
@@ -46,6 +46,12 @@ const Marker = new r.VersionedStruct(r.uint16be, {
   0xffe1: EXIFMarker,
 });

+Marker.process = function (stream) {
+  if (this.version === 0xffd9) {
+    stream.pos = stream.length; // finish reading
+  }
+};
+
 const JPEG = new r.Array(Marker);

 const decode = (buffer) => {

Can be something like that if you parse manually the array:
diff --git a/src/index.js b/src/index.js
index ba77471..4ec1c9f 100644
--- a/src/index.js
+++ b/src/index.js
@@ -46,10 +46,23 @@ const Marker = new r.VersionedStruct(r.uint16be, {
   0xffe1: EXIFMarker,
 });

-const JPEG = new r.Array(Marker);
-
 const decode = (buffer) => {
-  const markers = JPEG.fromBuffer(buffer);
+  let pos = 0;
+  const markers = [];
+  let marker;
+
+  do {
+    marker = Marker.fromBuffer(buffer.slice(pos));
+    markers.push(marker);
+
+    const markerLength = marker.length ?? 0;
+    pos += r.uint16be.size() + markerLength;
+
+    if (marker.name === "SOS") {
+      pos += marker.data.length;
+    }
+  } while (marker.version !== 0xffd9);
+
   return markers.map(({ version, ...rest }) => ({ type: version, ...rest }));
 };

This is a patch that can be used to fix this, although doesn't feel like the ideal way:
diff --git a/node_modules/jay-peg/src/index.js b/node_modules/jay-peg/src/index.js
index ba77471..d19a573 100644
--- a/node_modules/jay-peg/src/index.js
+++ b/node_modules/jay-peg/src/index.js
@@ -7,9 +7,9 @@ import DRIMarker from "./markers/dri.js";
 import EndOfImageMarker from "./markers/eoi.js";
 import EXIFMarker from "./markers/exif.js";
 import JFIFMarker from "./markers/jfif.js";
-import SOSMarker from "./markers/sos.js";
 import StartOfFrameMarker from "./markers/sof.js";
 import StartOfImageMarker from "./markers/soi.js";
+import SOSMarker from "./markers/sos.js";

 const UnkownMarker = {
   length: r.uint16be,
@@ -46,6 +46,12 @@ const Marker = new r.VersionedStruct(r.uint16be, {
   0xffe1: EXIFMarker,
 });

+Marker.process = function (stream) {
+  if (this.version === 0xffd9) {
+    stream.pos = stream.length; // finish reading
+  }
+};
+
 const JPEG = new r.Array(Marker);

 const decode = (buffer) => {

I can confirm that I was facing the same issue. I had an image that just didn't load in react-pdf, but this change fixed the problem.


List of modded trees that leave floating blocks

Biomes O' Plenty:

Palm trees (#31) (leaves are marked persistent)
Mystic grove oak trees (https://www.curseforge.com/minecraft/mc-mods/treechop?comment=514)

Oh The Biomes You'll Go:

Mangroves (#43)
Redwoods (#32)
Baobabs (#85)

BetterEnd

See https://www.curseforge.com/minecraft/mc-mods/treechop?comment=244

Regions Unexplored

Sculkwood trees (leaves are marked persistent)

TerraFirmaCraft

Lots of trees

~Autumnity:~

Fixed! ~All of 'em (waiting on https://github.com/team-abnormals/autumnity/issues/46)~

~Atmospheric:~

Fixed! ~All of 'em (waiting on https://github.com/team-abnormals/atmospheric/issues/81)~

Cultural Delights
Avocado Trees leave behind Avocado Bundles (culturaldelights:avocado_bundle) and don't show their stripped variants while chopping them down.

Sorry for not replying @thimblebird, I'll look into it.

Ah it seems you already know about the mangrove issue with OTBYG. I hope you fix it soon, it is really painful to manually break down the tops of white mangrove trees.

Just wanted to give a heads up, it seems like most of the trees from TerraFirmaCraft often leave behind floating blocks (playing via the TerraFirmaGreg modpack).

Pam's Harvestcraft 2 fruits. They float when a tree of that mod is chopped :(

@FirinMaLazors Thanks, I've added it to the list.
@Sandrituky I'll look into it, but in the meantime, if you're on Minecraft 1.18+, you could try editing the treechop-common.toml config and adding the block IDs to the list of choppable blocks. I think it's in a "tree-detection" section.

@FirinMaLazors Thanks, I've added it to the list.
@Sandrituky I'll look into it, but in the meantime, if you're on Minecraft 1.18+, you could try editing the treechop-common.toml config and adding the block IDs to the list of choppable blocks. I think it's in a "tree-detection" section.

I've been trying for hours to get the tag of the fruits and nuts that hangs from the trees in PamHarvestcraft, unsuccessful :(
Exploring inside the mod, I think there are no tags for them, only for the item (not hanging block).
I could add blockID one per one, but... I wanted to try to add them by group.
All these blocks I want to include starts by "pamhc2treespam*"
I wonder if there are any syntax in toml that can filter by patterns like * or %
Something like this:
blocks = ["pamhc2trees:pam*"]
Anyways, I've already added some specific IDs for testing and... btw, they work perfectly if I add them as blocks considered as leaves!
I will add block by block, but I'd like to know about filtering with toml.
Thank you for the previous reply!

Ahh, yeah, it's tough when things aren't in block tags. Maybe ask the HarvestCraft devs to use block tags?

I wonder if there are any syntax in toml that can filter by patterns like * or %

Not at the moment, but it's a very nice idea! I'll add it to the to-do list :)

With redwoods problem still actual.

arboria trees also leaves floating blocks even though its just a datapack

Apple trees of „oh the biomes you will go“ also leave massive leaves and wood logs behind

Ars Nouveau has its logs and leaves left behind, similar to the quoted message below:

arboria trees also leaves floating blocks even though its just a datapack
This is a fantastic mod though and I'd love to see more out of it!

To add to the list if it's not already being worked on. Terralith's trees don't work with the mod.


[Turborepo] Provide a command to determine affected workspaces

Which project is this feature idea for?
Turborepo
Describe the feature you'd like to request
As developer I want a graph API to know which worspaces would be affected under certain conditions in order to distribute the execution of a certain tasks.
Sometimes it's interesting to manually distribute task executions among different services. As an example, given my monorepo and a git commit, I want to know which tasks would be executed to run unit tests considering changes since that specific commit.
If I was working in a local environment, this could be easilly accomplished using the turborepo CLI. What happens if I want to distribute this work among several workers processing my CI pipeline? If the foo and bar workspaces are the only ones which contains test:unit tasks, it makes sense to me to request to launch two workers and request turborepo to launch, for each worker, the test:unit task for a single workspace. Without knowing beforehand which projects are affected, I cannot easily accomplish this.
Describe the solution you'd like
Thinking in a general purpose solution, I think it would be great to add an affected:workspaces (I don't really care about the specific name) command line to the CLI.
It would be similart than the run cli command, without options which makes no sense in this case. Instead of running all the affected tasks, it would return a stringified JSON object (maybe additional output options would be a good choice) with all the workspaces affected.
Example:
turbo affected:workspaces --filter=my-pkg --filter=my-app --filter=[HEAD^1]

would return ["my-pkg"] if and only if my-pkg is the name of the only workspace which contains tasks which would normally be executed by the same run command (turbo run --filter=my-pkg --filter=my-app --since=[HEAD^1])
Describe alternatives you've considered
I considered several workarounds but I didn't like any of those.

Launching a worker for each workspace and trying to run the test:unit task for that workspace is not a good choice, I'm consuming resources in order to do nothing in most of the projects.

I also considered other possible solutions

Requesting run to add a flag to actually not run tasks seems sort of a nonsense in my mind since the run command is supposed to be used to tun tasks, not running tasks would be, at least, counter intuitive.
Requesting a task graph (the same output which is already returned by turbo run --graph) is also a really interesting possibility, if we want to go this way I just strongly suggest to fix the bug which adds all possible pipeline task nodes since that would totally mess the workspaces detection

Can’t you use —dryrun=json for this?

Can’t you use —dryrun=json for this?

Yes, I can't believe a missed that option. Thank you so much and sorry for the noise


Linker don't see inline functions

I'm trying to bridge https://github.com/sprang/Inkpad. I'm already hide Dropbox, NSUndoManager and other unimplemented references. For now I meet problem that linker don't see inline functions
Error   LNK2001 unresolved external symbol _WDGLFillCircle  Inkpad (Inkpad\Inkpad)
How should I solve it?

@ad1Dima Can you please share with is the build log file?

@mnithish here you are
Inkpad.txt

@ad1Dima Thanks. It looks like our compiler defaults inline functions to static. To unblock yourself I would recommend adding an "extern" keyword to the inline functions in the source file for WINOBJC. I am following up with the compiler guys to see if we can find a permanent solution
#ifdef WINOBJC
extern
#endif
inline void WDGLFillCircle(CGPoint center, float radius, int sides)

thanks, It was helpful
Had to replace static inline with extern inline too
#ifdef WINOBJC
extern
#else
static
#endif
inline float WDMagnitude(CGPoint point)

And what about permanent solution in compiler?

@weswmsft @mnithish any news from compiler team?

Here is the explanation from the compiler team:
Clang/C2 is supposed to follow what CL does. In both cases, I’m not seeing inline functions marked as static in my test. However, if the function is marked inline, we are not required to emit it. If we do decide to emit it, it will be extern select any (so the linker will just pick any one of the versions of the inline function from one of the translation units).
This means that an inline function should be defined everywhere it is used (or an extern one should be available). It seems like the open source code mentioned in the bug is using inline in an incorrect way.
“A function where all the declarations (including the definition) mention inline and never extern. There must be a definition in the same translation unit. The standard refers to this as an inline definition. No stand-alone object code is emitted, so this definition can't be called from another translation unit.”
http://www.greenend.org.uk/rjk/tech/inline.html
http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1570.pdf
“Second, in case of raw inline (ie without static or extern) to provide an inline definition as an alternative to an external one, which has to be present in a different translation unit. Not providing the external one is undefined behaviour, which will normally manifest as linking failure.”
http://stackoverflow.com/questions/16245521/c99-inline-function-in-c-file/16245669


Docs: invalid-hereafter upper limit

This idea pertains to the aiken documentation's What I Wish I Knew section, specifically the Validity Intervals sub-section.
There is a maximum upper-bound allowed for validity intervals of 1.5 days (129600 slots) past the current slot. If you set an invalid-hereafter to be later than this bound, you will get a PastHorizon error message when submitting the transaction. It took me a while to figure this out and I wish I knew it before :sweat_smile:
The value of 1.5 days is calculated by: stability_window_network_parameter * 2 (source). From the comment in the code (a few lines below the linked source code line):
-- | Zone in which it is guaranteed that no hard fork can take place

Hardforks can alter the slot <--> POSIXTime conversions and so the node rejects conversions that are too far into the future.
Users can still set expirations (in datums) for things that exceed the current 1.5 days bound; but when actually specifying the validity interval for a transaction, the upper bound must currently bemin(129600 + current_slot, expiration_slot).
An example use case is a loan's expiration: the upper-bound can be used to prove the loan's expiration has not passed yet, but loans can last for months. So when trying to prove the loan is not expired, the transaction must use current_slot + 129600 for the upper bound until the actual expiration is within 1.5 days of the current slot.

Indeed! Good idea. And it's even more complicated than that as this constraint only exists for transactions that carry Plutus scripts! Transactions without scripts can have arbitrarily far away TTL since the problem really only occurs when a conversion to POSIX time is needed (for Plutus scripts execution).
Also, the window happens to be larger near hard-fork boundary as well. As soon as the global parameter for the next era are known, the stability windows is extended by another window (since no hard fork can happen immediately after the current one). but I am not sure this edge case is worth mentioning 😅 ...

Also, the window happens to be larger near hard-fork boundary as well. ...

I was not aware of this edge case. Thanks for mentioning it! Considering the window is currently 1.5 days and the typical lifetime of a transaction in a mempool is a few hours at most, I don't think most users will ever need to take advantage of the extra window around hardfork events. Perhaps the edge case could still be mentioned in a sentence or two within a footnote. That way it is there if someone actually does need that information.


[build] update cirque base image version

Problem
The chip-build-cirque is still in 0.4.11 version.

Summary of Changes
Update the version. We also need to upload 0.4.12 chip-build-cirque image to dockerhub.

BTW can we refer to https://docs.docker.com/docker-hub/builds/ to setup automated builds for the chip-build-xxx images so that we don't need to ask authorized users to manually upload the image?

Close since all the changes contained in https://github.com/project-chip/connectedhomeip/pull/3241


Many of the links within the OpenAPI docs do not work

I have been looking at https://github.com/OAI/OpenAPI-Specification/blob/main/versions/3.1.0.md and noticed many of the links don't navigate, such as https://github.com/OAI/OpenAPI-Specification/blob/main/versions/3.1.0.md#pathsObject.
This is just one example of many for this document.
The Firefox browser (version 118.0.1) on Windows 11 has been used to check the links.

Dup of #3197 (see last comment there for recent update)


chore(release): v2.3.2

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #541 (c0ef833) into main (1154020) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #541   +/-   ##
=======================================
  Coverage   88.68%   88.68%
=======================================
  Files          68       68
  Lines        2528     2528
  Branches      732      732
=======================================
  Hits         2242     2242
  Misses        277      277
  Partials        9        9

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1154020...c0ef833. Read the comment docs.


Added the AI section and IP address for FF

@tsyplyaev : Thanks for your contribution! The author, @gsacavdm, has been notified to review your proposed change.

Thanks!


Framework/Test Targets for macOS

Added a target which builds Runtime.framework for macOS,
added a test target RuntimeTestsMac w/ the tests,
and I disabled the class construction stuff except on
iOS.
This builds fine, and all the enabled tests run through (that is, no class-construction on x86).

Note: This also seems to build fine on Linux (but your Package.swift doesn't run the tests anymore, created issue #7).


Batches with multiple images implementation

Hi,
First of all, thanks a lot for your useful repository!
For a project I'm working on, I need your code to support batches with multiple images.
Did you try to implement multiple images extraction at some point? If so, what were your difficulties when trying to do so?

The faster r-cnn implementation, which I'm using, has this single batch restriction. You can check this issue for possible solution.

Got it. Thanks for your help!


Introduce Deming/Orthogonal regression

Deming/Orthogonal regression, least-Squares fitting the points in the 2D dataset (x,y) to a line
   a*x + b*y + c = 0
For delta equal 1 (the default value), this is performing orthogonal regression, minimizing the sum of squared perpendicular distances from the data points to the regression line.
Discussed/suggested here:
https://github.com/mathnet/mathnet-numerics/discussions/786
Questions:

Is Fit.Line2D a good name?
Where to put StandardLineToYxLine?

Is Fit.Line2D a good name?
I think that Fit.Deming would be more appropriate. Line2D seems like it performs the "usual" least squares fitting

Where to put StandardLineToYxLine?
This is used purely for testing purposes, I think it is more appropriate in the testing class FitTests.StandardLineToYxLine

I like the Line in the name, because it is very similar to the SimpleRegression. But I am fine with Deming as well, so I have updated the Fit.Line2D to Fit.Deming. And moved the StandardLineToYxLine to the FitTests - I will then just keep my own line tools alive.


npm start does not work for me

I just tried out getting a local copy of your quickstart. I ran into some problems. The install went fine but when I ran npm start I got errors. I have attached the npm.debug.log. Do you have any ideas why it does not start?
Attached you can find the output of the log.

Hi @mhafren ,
what's your ng and node version?
Here's mine:
angular-cli: 1.0.0-beta.14
node: 6.9.1

Hi,
I am using node 6.5 (will upgrade). However I got it working now, seems I had another program occupying the port 4200 that caused this issue. Sorry for the invalid issue report and thank you a lot for the response!

Glad it worked :)


any way to include non-compiled assets for inject manifest plugin?

Library Affected:
workbox-webpack-plugin.
Browser & Platform:
Latest
Issue or Feature Request Description:
I have 2 javascript files. Seems like I don't want webpack to compile them so I never include them in any js file. So the way I am doing it is I wrote my own plugin which has the very early hook such as thisCompile hook which does some things and then moves some files to /dist folder. Then, I know that injectManifest plugin uses make/emit hooks and gathers all things from there that pass the validation and creates manifest depending on it. The thing is the way workbox-webpack-plugin does that is it uses compilation object. Since my files that I moved in thisCompile hook of my plugin are not included in compilation assets, inject manifest won't know about these files and i am left with no manifest for these 2 files.
I got some ideas, but maybe you can shoot the best practice about injectManifest plugin. I am using its latest version.

The recommended approach for most folks would be to use https://github.com/webpack-contrib/copy-webpack-plugin to handle copying the existing assets over to the output directory. That should be enough to get the assets into webpack's build pipeline, and workbox-webpack-plugin can pick them up from there.
If you can't do that, for some reason, you can add them in to the final manifest in Workbox v5 by using a configuration like
{
  additionalManifestEntries: [{
    url: '/path/to/my/asset.js', revision: 'YOUR_HASH_HERE',
  }. {
    url: '/path/to/my/versioned-asset.1234abcd.js', revision: null,
  }],
  // ...other Workbox configuration options...
}

If your assets don't contain hashes in their URLs (the first example), it's up to you to make sure you generate a unique hash of the file's contents and add it in as the revision parameter. If your assets do already contain hashes, you can safely set revision: null, as in the second example.

The bad thing is that before copying, i have to do some transformation for the files which require some loops and custom logic. That's why I am using my custom plugin with thisCompile hook. but this way, workbox-webpack-plugin won't take these files as they won't exist in compilation's assets, but still will be in /dist folder. makes sense?

Sure. Given that, you can use the second option, with additionalManifestEntries. If you need more flexibility, another option is to use the manifestTransforms option, which lets you specify one or more functions that can manipulation the precache manifest's contents. Inside that function, you can read from the filesystem, refer to the output created by your other plugin, etc.

Awesome. could you quickly write how i can use it? couldn't find it in the docs.

 new InjectManifest({
            swSrc: './src/js/service-worker.js',
            swDest: './service-worker.js',
            exclude: [
                /\.map$/,
                /^manifest.*\.js$/,
                /^.htaccess/
            ],
            manifestTransforms: [
                test(entries, compi) {
                    console.log(manif, ' ah')
                }
            ]
        })

this results in an error.

@jeffposnick , do you recommend transforming manifest so that we remove revision for those urls which include hashes ? if yes/no, why?
Thanks.

In general, using revision: null when a url already contains revision info (i.e. a hash) will lead less data being transferred via the network during the preaching process—assuming that you also are setting long-lived Cache-Control headers on your server for those URLs.
(The easiest way to ensure that revision: null is set for those URLs is to use the dontCacheBustURLsMatching parameter when configuring your Workbox plugin. This will only work for manifest entries that are derived from the webpack asset list, which obviously won't apply to the assets that only your custom plugin knows about.)

@jeffposnick  Thanks for the info, Appreciate it.
Meanwhile, I saw your answer on stackoverflow which states why skipWaiting is risky. https://stackoverflow.com/questions/51715127/what-are-the-downsides-to-using-skipwaiting-and-clientsclaim-with-workbox
I think the first scenario you're describing is not that much of a huge risk. This risk only happens if we have more than 1 tab open. And when an update to a service worker is uploaded, we should refresh one tab to install/activate it. Now, the problem is in the first tab,  if you click the link, view-one.hashbefore.js won't be there in a cache as second tab's refresh removed it. Now, what will happen is view-one.hashbefore.js 's request will get made to the network, which will return 404 as it's removed from the server.
So to sum up, skipWaiting is a problem when one tab refreshes while new service worker update is on the server, and there's another tab which we click on links or whatever and the first tab's results happen to be broken as they don't exist on the server?
Agree?

Here's an article how to scan the output folder and add anything you see fit into the additionalManifestEntries attribute:
Precaching pages with next-pwa - DEV Community


Class hash does not exist

I keep getting
In Container.php line 752:
  Class hash does not exist

When I run behat on a laravel 5.5 project
I have php 7.1, ubuntu 16.04 and  "behat/behat": "^3.5",.
My coworkers don't seem to have this problem, only me. A pc specific issue?
I tried composer dump-autoload and  php artisan cache:clear, nothing works

It seems to be an issue with your app. Behat does not have Container.php.

interestingly if I run
vendor/bin/behat --tags=Ready
I see no errors


Visualizing gradients via TensorBoard

I'd like to visualize weight grad distributions in TensorBoard. I tried to add grads to TB using tf.summary.histogram in TensorBoard class as follows (see full commit here):
diff --git a/keras/callbacks.py b/keras/callbacks.py
index 49567f9..f605e7d 100644
--- a/keras/callbacks.py
+++ b/keras/callbacks.py
@@ -630,6 +630,9 @@ class TensorBoard(Callback):

                 for weight in layer.weights:
                     tf.summary.histogram(weight.name, weight)
+                    grads = model.optimizer.get_gradients(model.total_loss,
+                                                          weight)
+                    tf.summary.histogram('{}_grad'.format(weight.name), grads)
                     if self.write_images:
                         w_img = tf.squeeze(weight)
                         shape = w_img.get_shape()

But it seems targets and sample_weights are defined as placeholders, so they should be specified in the evaluation of grads. Is there a way to do this and add grad visualization to Keras?
I get the following trace when training starts:
Caused by op 'output_sample_weights', defined at:
  File "sample.py", line 9, in <module>
    learning_rate=0.001, reconstruct=False, activation='relu')
  File "api.py", line 24, in autoencode
    epochs=epochs, **kwargs)
  File "train.py", line 38, in train
    model.compile(loss=loss, optimizer=optimizer)
  File "/home/goekcen_eraslan/Code/keras-gokcen/keras/engine/training.py", line 858, in compile
    name=name + '_sample_weights'))
  File "/home/goekcen_eraslan/Code/keras-gokcen/keras/backend/tensorflow_backend.py", line 351, in placeholder
    x = tf.placeholder(dtype, shape=shape, name=name)
  File "/home/goekcen_eraslan/miniconda3/lib/python3.6/site-packages/tensorflow/python/ops/array_ops.py", line 1502, in placeholder
    name=name)
  File "/home/goekcen_eraslan/miniconda3/lib/python3.6/site-packages/tensorflow/python/ops/gen_array_ops.py", line 2149, in _placeholder
    name=name)
  File "/home/goekcen_eraslan/miniconda3/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 763, in apply_op
    op_def=op_def)
  File "/home/goekcen_eraslan/miniconda3/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 2327, in create_op
    original_op=self._default_original_op, op_def=op_def)
  File "/home/goekcen_eraslan/miniconda3/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1226, in __init__
    self._traceback = _extract_stack()

InvalidArgumentError (see above for traceback): You must feed a value for placeholder tensor 'output_sample_weights' with dtype float
         [[Node: output_sample_weights = Placeholder[dtype=DT_FLOAT, shape=[], _device="/job:localhost/replica:0/task:0/gpu:0"]()]]

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[x] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[ ] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[x] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Run examples/mnist_mlp.py from my Keras repository upon installing patched keras.

Follow this: https://github.com/fchollet/keras/issues/2226#issuecomment-259004640

@ltrottier thanks, I figured it out, sending a PR now.

I think "sample weights" in that url of "fchollet#2226" could be a problem, @gokceneraslan  did you solve this in the way you want?


about volume path

why i set volume path with: "-v sqlvolume:/var/opt/mssql"
but the Actual volume path is "/var/lib/docker/volumes/sqlvolume/_data/data"

@q409640976 -- thank you for your feedback; we appreciate it. To make sure we have all the information needed to triage your issue, please let us know which topic you’re referencing, or provide more context around the general task.

I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.


Error in the formula's symbols

Description
Error in the formula's symbols
(Optional:) Please add any files, screenshots, or other information here.

(Required) What is this issue most closely related to? Select one.
Choose One
Internal issue ID
0a3a62ab-15a7-420e-9800-2e87af42dbb3
Paper URL
https://arxiv.org/html/2407.00088v1
Browser
Chrome/<IP_ADDRESS>
Device Type
Desktop

It looks like latexml has missing ligatures for » and «, thank you for reporting these.


Cannot display cascade folders in coverage report when using "collectCoverageFrom" config

Do you want to request a feature or report a bug?
bug
What is the current behavior?
the coverage report table should display cascade folders(here is controllers) but it didn't:

If the current behavior is a bug, please provide the steps to reproduce and
either a repl.it demo through https://repl.it/languages/jest or a minimal
repository on GitHub that we can yarn install and yarn test.
just add a config to collect report from some files:
"jest": {
    "collectCoverageFrom": [
      "controllers/**/*.js",
      "middlewares/**/*.js",
      "services/**/*.js",
      "utils/**/*.js"
    ]
  },

What is the expected behavior?
it should display cascade folders just like the result without config:

Please provide your exact Jest configuration
in package.json:
"jest": {
    "bail": true,
    "collectCoverageFrom": [
      "controllers/**/*.js",
      "middlewares/**/*.js",
      "services/**/*.js",
      "utils/**/*.js"
    ]
  },

Run npx envinfo --preset jest in your project directory and paste the
results here
System:
    OS: macOS High Sierra 10.13.3
    CPU: x64 Intel(R) Core(TM) i5-6267U CPU @ 2.90GHz
  Binaries:
    Node: 8.10.0
    Yarn: Not Found
    npm: 5.7.1
  npmPackages:
    jest:
      wanted: ^22.4.2
      installed: 22.4.2

Can you provide a repo with a repro so we can investigate?

@thymikee here is the reproduced repo, and the problem might be caused by the collectCoverageFrom config.

Does it happen on jest@beta? There have been 2 PRs regarding collectCoverageFrom merged recently: https://github.com/facebook/jest/pull/5601 and https://github.com/facebook/jest/pull/5914 (not sure if included in this tag though)

<EMAIL_ADDRESS>performs the same


Remove use of ProviderBuilder by repo handler

Relates to #2845

Move the bulk provider instantiation for remote repo listing behind the provider manager interface.
Change the repo creation method in the repo service to instantiate its own provider. The handler still queries the table of providers to search for an appropriate provider. I have decided to not put this behind the ProviderManager at this time since I feel like the right place to do this sort of logic may change in subsequent PRs.

Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[x] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[x] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[x] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 50.359% (-0.3%) from 50.625%
when pulling 954e5895a66665358e1b583c39e9bebe44f3317e on cleanup-repo-endpoints-providers
into b9daa6103da6e5fe678d4119a9808e6fff8c429d on main.


TRex wrapper script for testpmd workload

This is required for - TestPMD workload https://github.com/cloud-bulldozer/benchmark-operator/pull/510

Can one of the admins verify this patch?

/rerun all

Results for SNAFU CI Test

Test
Result
Runtime

snafu/hammerdb
PASS
00:06:22

snafu/smallfile_wrapper
PASS
00:06:36

snafu/scale_openshift_wrapper
PASS
00:10:06

snafu/sysbench
PASS
00:01:52

snafu/ycsb_wrapper
PASS
00:05:37

snafu/trex_wrapper
FAIL
00:02:53

snafu/stressng_wrapper
PASS
00:03:34

snafu/fio_wrapper
PASS
00:19:36

snafu/upgrade_openshift_wrapper
PASS
00:00:00

snafu/uperf_wrapper
PASS
00:27:03

snafu/iperf
PASS
00:06:54

snafu/vegeta_wrapper
PASS
00:05:45

snafu/fs_drift_wrapper
PASS
00:06:12

snafu/pgbench_wrapper
PASS
00:04:15


[Downloader] ERROR: unable to open for writing: [Errno 29] Illegal seek

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2022.04.08 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
To reproduce this issue:

create a fifo:
mkfifo /tmp/test

run yt-dlp:
python3 yt-dlp -v -f 140 -o /tmp/test http://www.youtube.com/watch?v=WSRWo4ZhnHM

run a player instance to play the tmp fifo:
mplayer -ao null -vo null /tmp/test
EDIT:
you can also use:
cat /tmp/test > /dev/null
to just read from the fifo, no player needed to reproduce this issue.

yt-dlp crashes with the given verbose log.
I can note, that an older version which I tested is still working fine. The latest version crashes with "ERROR: unable to open for writing: [Errno 29] Illegal seek".
An old version tested:
yt-dlp version 2021.10.10 (zip) <-- which is working
yt-dlp version 2022.04.08 [7884ade65] (zip) <-- fails
Can someone tell if something fundamental changed here? First time seeing this.
I am currently trying to figure out, which newer version started breaking it, because my older version which still works is outdated.
Maybe someone can help.
Verbose log
python3 yt-dlp -v -f 140 -o /tmp/test http://www.youtube.com/watch?v=WSRWo4ZhnHM
[debug] Command-line config: ['-v', '-f', '140', '-o', '/tmp/test', 'http://www.youtube.com/watch?v=WSRWo4ZhnHM']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, err utf-8, pref UTF-8
[debug] yt-dlp version 2022.04.08 [7884ade65] (zip)
[debug] Python version 3.7.3 (CPython 64bit) - Linux-4.9.0-13-amd64-x86_64-with-debian-10.12
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 4.1.8-0, ffprobe 4.1.8-0
[debug] Optional libraries: certifi, sqlite3
[debug] Proxy map: {}
[debug] [youtube] Extracting URL: http://www.youtube.com/watch?v=WSRWo4ZhnHM
[youtube] WSRWo4ZhnHM: Downloading webpage
[youtube] WSRWo4ZhnHM: Downloading android player API JSON
[debug] Loading youtube-sigfuncs.js_596ef930_105 from cache
[debug] Loading youtube-nsig.596ef930 from cache
[debug] [youtube] Decrypted nsig T0SxpikMkB9eNsRln => OFaNYkq205V1WQ
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, codec:vp9.2, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), acodec, lang, proto, filesize, fs_approx, tbr, vbr, abr, asr, vext, aext, hasaud, id
[info] WSRWo4ZhnHM: Downloading 1 format(s): 140
[debug] Invoking downloader on "https://rr4---sn-4g5lzner.googlevideo.com/videoplayback?expire=1650997027&ei=w-JnYrq6Bobj7gOO3JbYDg&ip=<IP_ADDRESS>&id=o-AN8w2oHcOdM1ZLbAoB_GOhd6JZRw9S8Ik-PvK0rlxf2O&itag=140&source=youtube&requiressl=yes&mh=nl&mm=31%2C26&mn=sn-4g5lzner%2Csn-f5f7lnel&ms=au%2Conr&mv=m&mvi=4&pl=16&initcwndbps=2270000&vprv=1&mime=audio%2Fmp4&gir=yes&clen=1896926&dur=117.167&lmt=1650896162842003&mt=1650974936&fvip=1&keepalive=yes&fexp=24001373%2C24007246&c=ANDROID&txp=5432434&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIhAIGGR2ggBWF-gL7_23XC2cZXpe0daxuLszCB99afjdjjAiACAeX5W6V3rN5Y7rb_S1-HF5Hc6TEK1G-WWduGlUYE0A%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRAIgNqe4SSki3jjtFCcG5VB7oivzTAlbF_S7CvfZjiJuFfwCIA_Mmj4fbfS9WioJzIKf-wnN6AhggYfE6MlatCxDYxuu"
ERROR: unable to open for writing: [Errno 29] Illegal seek
Traceback (most recent call last):
  File "yt-new-broken/yt_dlp/downloader/http.py", line 261, in download
    ctx.tmpfilename, ctx.open_mode)
  File "yt-new-broken/yt_dlp/downloader/common.py", line 218, in inner
    return func(self, *args, **kwargs)
  File "yt-new-broken/yt_dlp/downloader/common.py", line 235, in sanitize_open
    f, filename = sanitize_open(filename, open_mode)
  File "yt-new-broken/yt_dlp/utils.py", line 619, in sanitize_open
    stream = locked_file(filename, open_mode, block=False).__enter__()
  File "yt-new-broken/yt_dlp/utils.py", line 2004, in __enter__
    self.f.truncate()
OSError: [Errno 29] Illegal seek

Most likely, either a3125791c7a5cdf2c8c025b99788bf686edd1a8a or fcfa8853e41ca04714a7aa28a783e2804c184375 is the culprit
cc @jakeogh

I am currently trying to figure out, which newer version started breaking it, because my older version which still works is outdated.

Depending on, if this gets fixed or if I have to patch a custom version for myself in the future?

Ah interesting. For the special case of writing to a FIFO we could either catch the OSError on truncate (better option I think, need to test), as named pipe's don't support seek, or skip the truncation when the fd is a fifo. I'll test and send a pull today.

How does io.open deal with this? We should do exactly what Python does, so that we avoid other edge cases like this

@jakeogh have you found a solution for this?


New version: StaticNumbers v0.3.0

Registering package: StaticNumbers
Repository: https://github.com/perrutquist/StaticNumbers.jl
Created by: @perrutquist
Version: v0.3.0
Commit: bced6a63b321b97cdefded6f4148e85ea87c5c74
Reviewed by: @perrutquist
Reference: https://github.com/perrutquist/StaticNumbers.jl/commit/bced6a63b321b97cdefded6f4148e85ea87c5c74#commitcomment-36620872

I had expected Registrator to update this PR, but instead it created #7369.
Somebody please close this one, as the other one will be auto-merged.


Plugins

I needed to run environment specific behavior before an app is run. I did this by adding a simple, queued, plugin system and a start() method that will wait for plugins to finish before booting an app.
The other big breaking change here is that application state is not calculated until start is called. In addition to reducing some complexity, this also means undo history can anchor itself upon this initial start value.

Coverage remained the same at 100.0% when pulling 7cc76dfc1cccd1dda4e0bf60bbe24a35d7eadec0 on plugins into 8e3c5e8fc82f771686c8ce55176b7a7dfda843ad on master.

:+1:

Also worth noting, this feature has been specifically designed to help solve two real problems:

For Pearl, I want to prefetch data for an application on the server only, background loading new content in the client
On Colonel Kurtz, I want the ability to add a "data migration" plugin that will automatically update old schemas.

¯\(°_o):thumbsup:


For some reason "File" --> "Open" asks you for the file twice

Please let me try to add some info.

MacOS 10.13.3 - Firefox 59.0.1 (64-bit): I get one dialog. It just imports the selected SVG
MacOS 10.13.3 -  64.0.3282.186 (Official Build) (64-bit): I get two dialogs. Whatever I choose first is ignored, the second choice is imported
MacOS 10.13.3 - 11.0.3: I get one dialog. The chosen file is not imported


Report errors when projection operation fails

Referencing closed feature request 721
When a projection fails pyproj returns an inf due to upstream proj. There is seemingly a flag that can change this to a NaN, if the user has dug deep enough into the notes to understand what is happening.
It would be very worthwhile if, on a projection operaiton failure, that an error message is presented. Some users have expressed that they do not want a message as they find them annoying, so perhaps this could be addressed by adding a suppress-errors flag of some variety. As it stands locating the source of an inf in the output, particularly when it's an intermittent result, can be infuriating. This has taken up multiple days of development time to hunt down in the past.
Does proj throw any information that can be captured and forwarded on?

There is an errcheck argument for the transform: https://pyproj4.github.io/pyproj/stable/api/transformer.html#pyproj.transformer.Transformer.transform
Also see: https://pyproj4.github.io/pyproj/stable/advanced_examples.html#debugging-internal-proj

I believe this has been addressed. If not, please comment.