Rename KEY_RETURN to KEY_ENTER and KEY_ENTER to KEY_KP_ENTER

Closes #7695

Wait, why? "Return" is the canonical name of the main "insert a carriage return and/or validate entry" key, and "Enter" is typically written on the Enter key of the keypad (while the Return key typically shows a return symbol).

@akien-mga ~ Quoting @reduz on https://github.com/godotengine/godot/issues/7695#issuecomment-320502308:

despite the proper name, I also think it's less confusing to rename to KEY_ENTER and KEY_KP_ENTER . We are still in time if you want to change this before 3.0. Any volunteers?

Well I don't agree, but if you guys prefer it this way, I won't fight :)

What do you think about KEY_RETURN and KEY_KP_ENTER as a fully explicit,
everyone-happy solution?
Or even prefixing all numpad keys with KEY_KP_. By the way, does 'KP' stand
for 'key pad'?
English is not my primary language, but if that's the meaning, maybe NUMPAD
(or the shorter, and uglier, NP) would be better. To me, num-pad is not an
explicit reference to the numeric pad/keyboard.
El 7 ago. 2017 11:33 a. m., "Rémi Verschelde"<EMAIL_ADDRESS>escribió:

Well I don't agree, but if you guys prefer it this way, I won't fight :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/10123#issuecomment-320616758,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALQCtkh_sfqNKBXRgHNE9gbTjuz4W9t6ks5sVtnNgaJpZM4Ouu0I
.

KEY_KP is a more-or-less universal way that engines refer to numpad keys (UE4, pygame, libtcod, bearlibterminal, unsure about Unity) so I'm +1 for prefixing all numpad keys with KEY_KP.

If KP is universal, then following the trend is something good. Nothing against.

@akien-mga to me the key name is "return" too, but for the new generations it's "enter". A lot of people got confused by this, and this change (for bad or good), makes Godot more user friendly

IBM Labelled it "Enter" since their first IBM PC models, it's their fault..

https://ux.stackexchange.com/questions/54461/should-i-use-the-name-enter-or-return-key

Wow, I don't feel that old, I was born in 1991 :P
Let's go with what pleases the millennials then ;)


make the web version to http

(Mixed Content: The page at 'https://ps4rps.vercel.app/' was loaded over HTTPS, but requested an insecure XMLHttpRequest endpoint 'http://<IP_ADDRESS>:90/ps4/'. This request has been blocked; the content must be served over HTTPS.)

Browser used: Edge, Chrome and Firefox

Try this http://ps4rps.njzydark.com

unfortunately it still have issue regarding CORS
Access to XMLHttpRequest at 'http://<IP_ADDRESS>:90/ps4/' from origin 'http://ps4rps.njzydark.com' has been blocked by CORS policy: The request client is not a secure context and the resource is in more-private address space "private".
I guess well be having issues with this if the website is hosted over the internet where the webdav hosted on the NAS is under local IP address.
I did work around on it though but this involves basically fetch the website data of that site and host it locally on my NAS and now it works.

unfortunately it still have issue regarding CORS
Access to XMLHttpRequest at 'http://<IP_ADDRESS>:90/ps4/' from origin 'http://ps4rps.njzydark.com' has been blocked by CORS policy: The request client is not a secure context and the resource is in more-private address space "private".

This is because the server(<IP_ADDRESS>:90) not enable cors.  On my NAS, I use this docker image to create webdav server: https://github.com/hacdias/webdav#cors

ill probably just close this issue since this is no longer a related, unfortunately this may be a issue on some NASes, I have already tested this on several instances of docker containers with CORS definitely enabled on the webserver and/or different NAS with their own webserver apps but they all have the same issue, not unless I disabled the CORS on my browser which is a very bad idea or use the public sided IP address which im not doing and pointless.
this may actually end up like a "it works for me" kind of situation.

it seems to work only on firefox, already tried several browser like Chrome, Brave, Edge, Chromium, all have the same CORS error.

On the issue of CORS, the easiest way is to open the developer tools and check whether the response header has a cors response header, the correct cors setting should be like this


Directory upload is not recursive

I suppose this is a deliberate choice, but the fact that uploading a directory only uploads the files in it (and not the subdirectories) is rather inconvenient - especially since it's not made clear.
Ideally, there should be a recursive mode but, failing that, I think that the behavior should be explicit.

Agreed, this just burnt me. Took ages to figure out what was happening.

Same for me here, took me a while to figure out. A recursive mode would be much appreciated.

Recursive upload has been added in 1.4.0.

Great! Will try this out soon.

Brilliant!


Unable to integrate latest 10.0.1

Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository file a Github issue.

If you are filing an issue about FCM in the background make sure to read #4 and #89 first!

For general technical questions, post a question on StackOverflow tagged appropriately.
For general Firebase discussion, use the firebase-talk google group
For help troubleshooting your application that does not fall under one of the above categories, reach out to the personalized Firebase support channel

Step 2: Describe your environment

Android device: _____any device
Android OS version: _____any OS
Google Play Services version: _____3.0.0
Firebase/Play Services SDK version: _____9.2.1 works but not 10.0.1

Step 3: Describe the problem:
Unable to add compile 'com.google.firebase:firebase-messaging:10.0.1'
Always throws not found though i have updated the Google Play Repository as expected, so able to build only wiht 9.2.1
Steps to reproduce:

Observed Results:

What happened?  This could be a description, logcat output, etc.

Expected Results:

What did you expect to happen?

Relevant Code:
// TODO(you): code here to reproduce the problem

@MohamedImthiaz can you try following these steps: http://stackoverflow.com/a/37310513/324977

@MohamedImthiaz due to the nature of this issue and the lack of updates I assume you have solved it?  Closing, but please add a note or re-open if this is still happening.


Assignment_1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Changed assignment_1.ipynb
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist

[X] I can confirm that my changes are working as intended

A1 missing written responses

Hi Amanda,
I've added missing written responses to the A1.  @amanda-ng518
Thanks,
George


Support nested fragments.

Mirror PR of https://github.com/clentfort/urql-custom-scalars-exchange/pull/23

Superseeded by #5


Add select-ts to ecosystem

📖 Documentation
I am building a package select-ts for fp-ts for  behaviors of type "if this condition is true, then do that". It is based on:

Mokhov, Andrey, et al. "Selective applicative functors." Proceedings of the ACM on Programming Languages 3.ICFP (2019): 1-29. https://dl.acm.org/doi/pdf/10.1145/3341694

I think it could be useful for implementing Ramda-like conditionals even if probably goals are different from those in fp-ts-ramda.
It is still very experimental. What do you think? Does it make sense to you?

FWIW, I had myself a try at "fp-ts and Selective" some time ago

it could be useful for implementing Ramda-like conditionals even if probably goals are different from those in fp-ts-ramda.

I didn't look at the code in select-ts yet, but if it is indeed an encoding of selective functors for fp-ts, it might be good to reimplement some related ramda functions in fp-ts-ramda based on it

Hey @giogonzo I haven't seen your pr!
Yes, it is an encoding for selective functors in fp-ts. As in the original paper, you can build one from a Monad (getSelectM) or from an Applicative (missing). The execution semantics is different in the two cases.
For example, when is very Ramda-ish. You can find other Ramda-like operators in the project: cond, unless, ifElse...
Imo also retry-ts can be rewritten using a selective functor. I have already implemented a while operator, it shouldn't be much different.


ocamlPackages.camlp5: 7.03 -> 7.05

Motivation for this change
Support for OCaml 4.06.1
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build  ocamlPackages.camlp5

Failure on aarch64-linux (full log)
Partial log (click to expand)
error: attribute 'camlp5' in selection path 'ocamlPackages.camlp5' not found

Failure on x86_64-linux (full log)
Partial log (click to expand)
error: attribute ‘camlp5’ in selection path ‘ocamlPackages.camlp5’ not found

Failure on x86_64-darwin (full log)
Partial log (click to expand)
error: attribute 'camlp5' in selection path 'ocamlPackages.camlp5' not found

@GrahamcOfBorg build ocamlPackages.camlp5_strict ocamlPackages.camlp5_transitional

Failure on aarch64-linux (full log)
Partial log (click to expand)
make[3]: *** [../../config/Makefile:24: versdep.cmo] Error 127
make[3]: Leaving directory '/build/source/ocaml_src/lib'
make[2]: *** [Makefile:164: library_cold] Error 2
make[2]: Leaving directory '/build/source'
make[1]: *** [Makefile:35: boot/camlp5] Error 2
make[1]: Leaving directory '/build/source'
make: *** [Makefile:150: world.opt] Error 2
builder for '/nix/store/y57dzhha3nhk7rs3bvdacsmyfz4cr5sa-camlp5_transitional-7.05.drv' failed with exit code 2
builder for '/nix/store/c7i80l4r67b2ybkhws5lvbd14dzamk7d-camlp5-7.05.drv' failed with exit code 2
error: build of '/nix/store/c7i80l4r67b2ybkhws5lvbd14dzamk7d-camlp5-7.05.drv', '/nix/store/y57dzhha3nhk7rs3bvdacsmyfz4cr5sa-camlp5_transitional-7.05.drv' failed

Failure on x86_64-linux (full log)
Partial log (click to expand)
/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash: /bin/rm: No such file or directory
make[3]: *** [../../config/Makefile:24: versdep.cmo] Error 127
make[3]: Leaving directory '/tmp/nix-build-camlp5_transitional-7.05.drv-0/source/ocaml_src/lib'
make[2]: *** [Makefile:164: library_cold] Error 2
make[2]: Leaving directory '/tmp/nix-build-camlp5_transitional-7.05.drv-0/source'
make[1]: *** [Makefile:35: boot/camlp5] Error 2
make[1]: Leaving directory '/tmp/nix-build-camlp5_transitional-7.05.drv-0/source'
make: *** [Makefile:150: world.opt] Error 2
builder for ‘/nix/store/zp2id8g6xpfhy1f0dr1sm2fwy1fyl054-camlp5_transitional-7.05.drv’ failed with exit code 2
error: build of ‘/nix/store/k5m1x2a014glpqyv87hd1l4mz7svl7xv-camlp5-7.05.drv’, ‘/nix/store/zp2id8g6xpfhy1f0dr1sm2fwy1fyl054-camlp5_transitional-7.05.drv’ failed

@GrahamcOfBorg build ocamlPackages.camlp5_strict ocamlPackages.camlp5_transitional

Success on x86_64-linux (full log)
Partial log (click to expand)
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type

Success on aarch64-linux (full log)
Partial log (click to expand)
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
wrong ELF type
/nix/store/58z618vnszmy31650nw927q12870nc39-camlp5-7.05
/nix/store/z8gdr0nxvnmjzqzn8ahvajim94icg7nn-camlp5_transitional-7.05


Unable to upload large pdf files for tram

Hi, I was trying to upload some documents, however, it seems there are space limitations. Am I able to change this value to a larger value?

@fishzengyu What size limit are you seeing? I am unable to upload any file. I am running the application on Docker on Windows 10 Home, and trying to upload the file from elsewhere on the Windows 10 machine.

I would echo @duttad comments, the more information you can provide us the better to replicate and fix:

What is your host OS?
Are you downloading the Docker image, building the image locally from the Dockerfile,  or running the project directly on the host OS?
How big in filesize: 5MB, 10MB, 50MB? How long is the document: 10, 50, 100 pages?
Are you able to provide any output from TRAM logs?

Running from this commit https://github.com/center-for-threat-informed-defense/tram/commit/7c9d875c3b6628014a98a9cbb835bddc70368f80 I cannot replicate the issue on host. Uploaded the documents via the Web UI. They got accepted, the UI displayed Queue for the submission until done. The test document details:

Size
Pages
ML Processing Time

~6MB
1602
26 minutes

~25MB
56
2 minutes

I think we will need more evidence to replicate the issue.

@fishzengyu and @duttad, could you try the new published release v1.1.3 to check if the issue still persists?

@emmanvg Sure I plan to give that a try. Do I replace image: ghcr.io/center-for-threat-informed-defense/tram:latest with image: ghcr.io/center-for-threat-informed-defense/tram:v1.1.3 in the docker-compose.yml file?

No need to change it, with image: ghcr.io/center-for-threat-informed-defense/tram:latest it will automatically pull the latest version for you.

Hi, I confirm that it's not possible to upload reports (I checked Pdf format) that are bigger than 2MB. I suppose that it's reason of Nginx settings. The error I have is: “client intended to send too large body”. I' ve tried this solution: https://www.jamescoyle.net/how-to/1608-nginx-error-client-intended-to-send-too-large-body , but it is not working! Could you tell how to repair this issue and enlarge maximum size?
I use TRAM build in docker container from the Dockerfile.

I'm having the same issue.  Any pdf over 2MB and TRAM won't upload it.  It won't even start uploading it.

I'm having the same issue. Any pdf over 2MB and TRAM won't upload it. It won't even start uploading it.

me too

Hello, I also experienced this issue.
I rectified it in the containers live as they were running.
There are two problems. The first is as @mehaase explains the nginx server has a default client_max_body_size of 1MB. This can be changed easily. I simply put client_max_body_size 20M; in (the tram-nginx container) /etc/nginx/conf.d/default.conf
In the live container I did this with: sudo docker exec -it [container id] vi /etc/nginx/conf.d/default.conf
However, Django also has a limit on file upload so you need to set that as well. To do this you need to set DATA_UPLOAD_MAX_MEMORY_SIZE and FILE_UPLOAD_MAX_MEMORY_SIZE in /tram/src/tram/settings.py. It needs to be set in bytes so FILE_UPLOAD_MAX_MEMORY_SIZE = 2097152 for 20MB. Unfortunately the tram container does not have a text editor (not even vi) so I modified the settings.py file, putting the two settings just under the database configuration, then used sudo docker cp settings.py [container id]:/tram/src/tram/settings.py. If the file doesnt overwrite you can rename the settings.py in the container with sudo docker exec -it [container id] mv /tram/src/tram/settings.py /tram/src/tram/settings-old.py then copy the file again. After this I reboot the containers, but not using docker-compose because that would wipe the config.
I'd suggest putting those changes into the published containers.
Remy

Hello, I also experienced this issue.
I rectified it in the containers live as they were running.
There are two problems. The first is as @mehaase explains the nginx server has a default client_max_body_size of 1MB. This can be changed easily. I simply put client_max_body_size 20M; in (the tram-nginx container) /etc/nginx/conf.d/default.conf
In the live container I did this with: sudo docker exec -it [container id] vi /etc/nginx/conf.d/default.conf
However, Django also has a limit on file upload so you need to set that as well. To do this you need to set DATA_UPLOAD_MAX_MEMORY_SIZE and FILE_UPLOAD_MAX_MEMORY_SIZE in /tram/src/tram/settings.py. It needs to be set in bytes so FILE_UPLOAD_MAX_MEMORY_SIZE = 2097152 for 20MB. Unfortunately the tram container does not have a text editor (not even vi) so I modified the settings.py file, putting the two settings just under the database configuration, then used sudo docker cp settings.py [container id]:/tram/src/tram/settings.py. If the file doesnt overwrite you can rename the settings.py in the container with sudo docker exec -it [container id] mv /tram/src/tram/settings.py /tram/src/tram/settings-old.py then copy the file again. After this I rebooted the containers, but not using docker-compose because that would wipe the config.
I'd suggest putting those changes into the published containers.
Remy

Can you let me know where exactly in settings.py we need to change DATA_UPLOAD_MAX_MEMORY_SIZE and FILE_UPLOAD_MAX_MEMORY_SIZE. I'm unable to see any such fields.

Hello
You need to add those fields. Because they are not specified it defaults to a very small allocation. So you add a line for each and set your values.

In my testing, DATA_UPLOAD_MAX_MEMORY_SIZE is not needed because the upload() handler does not access reqeust.body or request.POST (it uses request.FILES). See this doc. Also FILE_UPLOAD_MAX_MEMORY_SIZE is not needed since that affects spill over to disk but does not throw exceptions.
Setting it in nginx allows me to upload a 27MB test PDF. I'll open a PR for this.


小城复选框checkbox未选中

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

在项目其他页面中，尝试后偶尔也会出现未选中的现象，请问checked这个属性使用的时候是有背景条件吗


Soycode fixtests

Fix #47

Going to merge as at least running the tests is a good thing - most pass, some of the TCP tests regressed (not as many as it looks - xit-ing out the first two and last one failures cause all other tests to pass, so it's asynciness that is leaking tests). Feedback still welcome if you have any thoughts, thanks.


feat: use new getPackage method from parcel-bundler

This commit aims to remove the deprecated message:
"asset.package is deprecated. Please use
await asset.getPackage() instead." when bundling.
The bundled event function is now async and it awaits the getPackage() method calls or falls back into former package property if it doesn't exist.

Looks good to me.
Im going to merge it after I have done a manual test on my app.
Thanks for this pr :)

Cool! After the release, don't forget to publish the new version on NPM.


KeyError: 'short_answers' - octodns-dump

[8036670976] INFO  Manager dump: using custom YamlProvider
Traceback (most recent call last):
  File "../venv/bin/octodns-dump", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "../venv/lib/python3.11/site-packages/octodns/cmds/dump.py", line 48, in main
    manager.dump(
  File "../venv/lib/python3.11/site-packages/octodns/manager.py", line 805, in dump
    source.populate(zone, lenient=lenient)
  File "../venv/lib/python3.11/site-packages/octodns_ns1/__init__.py", line 905, in populate
    data = data_for(_type, record)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "../venv/lib/python3.11/site-packages/octodns_ns1/__init__.py", line 740, in _data_for_SPF
    values = [v.replace(';', '\\;') for v in record['short_answers']]
                                             ~~~~~~^^^^^^^^^^^^^^^^^
KeyError: 'short_answers'

Hello,
Getting KeyError when using octodns-dump to import records from NS1

Dependency
Version

ns1-python
0.18.0

octodns
1.0.0rc0

octodns-ns1
0.0.4

Hrm. Must be something non-standard/expected in the record that's tripping things up, or NS1 has changed something. It's likely going to require a bit of manual poking around to figure out what's up with the record.
Can you try adding the following around line 739 of ../venv/lib/python3.11/site-packages/octodns_ns1/__init__.py, just before the exception:
from pprint import pprint
pprint(record)

That dump of the record details (feel free to replace any sensitive names) should help to figure out what's up and what needs changed.

This is what the short answer look like. Data is randomly generated
'short_answers': ['k=rsa; '
                   'p=crcv8e0nsqwghckdsqjimxse0557s4zv86i9y37m0jrof77czjoi4tsnf2q68ulnjz36gegabn4c5o9i5qqlnqfe3skrsahn9q1w1ropq0vh0pbx8y0qd03ucmwidn4uo4yn9bjqcf6yoyo69zo2i1bpfc6kfgvdsnkdzkoim3nyvu01dqolywethpp1wo429p0g70zg5n9ezzi1y8g234j1;']

Hrm. Was that the last one printed right before the exception? Since that's a short_answers key and value I'm guessing it's not the one causing the exception. Sorry wasn't explicit about the last one part. Will also need the other data from the record dump as it may provide clues as to what's up.


Release wheels for macOS M1, win32, and linux aarch64

@pzelasko
Please release a new version to trigger the CI.

Thanks, just triggered it as the 0.7.2 release.


remove/unset startup script variables

On current master the use of too many variables leads to strange startup issues. This is an attempt to fix this for the moment.
@NosDE can you please check the mikrokopter support? Am I right that we can just depend on the mixer variable? Also if this works the mikrokopter wiki entry needs to be updated.

Mixers have been renamed on master.

In rcS you have removed extras.txt which is needed for mkblctrl to work.
The MK_MODE was used to activate the addresstranslation, if you are using a stock mikrokopter frame.
This can't be set automatically by mixer depencies. You never know, which order of the motors is used by the user.

@NOSDE ah so you can fly in the + configuration with an original mikrocopter x layout? From https://pixhawk.org/firmware/apps/mkblctrl it seems that the transformation between motor numbers is always the same though?

In rcS you have removed extras.txt which is needed for mkblctrl to work.

It mas moved, not removed.
From the wiki "They must be set in etc/config.txt file on SD card."  Why is extras.txt needed as well?

From https://pixhawk.org/firmware/apps/mkblctrl it seems that the transformation between motor numbers is always the same though?

Ok, if one also looks at hexa and octo configurations this is not true

I would prefer if this translation would depend on a parameter in the driver and can be moved out of the startup script. It's easier to use and makes more sense.

Thanks Thomas! Merging.


Create .gitattributes

I'm not sure if this works like I think it does...I think I'm reading that it will force Unix-style line-endings to be used when text files are added to the repo? (Recent fuse.sh and vice.sh files had DOS line-endings CR/LF instead of just LF.) If this doesn't work it may also require something like:
* text eol=lf

Also not sure if or how it affects existing files. Vice and fuse may still need fixed manually.

(Recent fuse.sh and vice.sh files had DOS line-endings CR/LF instead of just LF.)
what ending are you talking about

I think this is working. Took some head-bashing to make git do what I wanted so I could test it, but I got there eventually.
I'm working off the branch with the CRLF line endings. It doesn't seem to be affecting existing files so I tried creating a new one. I copied fuse.sh to test.sh and confirmed it was CRLF with dos2unix -i *.sh.
Then I did git add . and it said
warning: CRLF will be replaced by LF in scriptmodules/emulators/test.sh.
The file will have its original line endings in your working directory

As it said, the file still had its original line endings in my working directory. But, then I committed the change with git commit -mtest (still DOS line endings), then switched to "master" and back to my test branch (git checkout master && git checkout test-attrib) -- and they were UNIX!

what viewer are you using to see these so easyly, im using notepad ++ and all i see is a blank line (which i deleted once you pointed it out)
and how will this work? is it automatic or is there something i would need to run to check the files.

what viewer are you using to see these so easyly, im using notepad ++ and all i see is a blank line (which i deleted once you pointed it out)

I'm using dos2unix:
sudo apt install dos2unix
dos2unix -i
(The -i option gives info about the named file/s; if you leave it off, they will be converted):
$ dos2unix -i *.sh
      80       0       0  no_bom    text    fuse.sh
       0      76       0  no_bom    text    gearboy.sh
       0      56       0  no_bom    text    kat5200.sh
       0      31       0  no_bom    text    mpv.sh
       0      71       0  no_bom    text    openbor.sh
       0      59       0  no_bom    text    pico8.sh
       0      46       0  no_bom    text    pokemini.sh
       0     106       0  no_bom    text    supermodel.sh
       0      80       0  no_bom    text    test.sh
     126       0       0  no_bom    text    vice.sh
$ dos2unix vice.sh
dos2unix: converting file vice.sh to Unix format...
$ dos2unix -i *.sh
      80       0       0  no_bom    text    fuse.sh
       0      76       0  no_bom    text    gearboy.sh
       0      56       0  no_bom    text    kat5200.sh
       0      31       0  no_bom    text    mpv.sh
       0      71       0  no_bom    text    openbor.sh
       0      59       0  no_bom    text    pico8.sh
       0      46       0  no_bom    text    pokemini.sh
       0     106       0  no_bom    text    supermodel.sh
       0      80       0  no_bom    text    test.sh
       0     126       0  no_bom    text    vice.sh

and how will this work? is it automatic or is there something i would need to run to check the files.

Looks like it's automatic when files are added via command-line, at least. That's what I was testing above. Make a file with DOS endings (I just copied an existing file), commit it to the repo, and Unix endings are substituted in the version that's in the repo.
I'm not quite sure how to test for when the files are created directly on the github.com website; pretty sure that in the past I've seen them come out both ways and I'm not sure what decides it, which style endings they have. For all I know it's just a little tick-box somewhere that I overlooked.


Removal of the extension via podman desktop and reinstallation ends up with an error during activation

Install sandbox extension
Stop extension
Install extension again
Assert: Extension is installed and in active state
Actual: Installation happens, but during activation the state of extension is failed with error:

Error: ENOENT: no such file or directory, open '/Users/podmanqe/.local/share/containers/podman-desktop/plugins/ghcrioredhatdeveloperpodmandesktopsandboxext/node_modules/tar/index.js'
Stack trace:
Error: ENOENT: no such file or directory, open '/Users/podmanqe/.local/share/containers/podman-desktop/plugins/ghcrioredhatdeveloperpodmandesktopsandboxext/node_modules/tar/index.js'
    at Object.readFileSync (node:fs:453:20)
    at t.readFileSync (node:electron/js2c/node_init:2:9771)
    at Module._extensions..js (node:internal/modules/cjs/loader:1413:18)
    at Module.load (node:internal/modules/cjs/loader:1214:32)
    at Module._load (node:internal/modules/cjs/loader:1030:12)
    at c._load (node:electron/js2c/node_init:2:13672)
    at _module._load (/Applications/Podman Desktop.app/Contents/Resources/app.asar/packages/main/dist/index.cjs:146:33447)
    at Module.require (node:internal/modules/cjs/loader:1242:19)
    at require (node:internal/modules/helpers:176:18)
    at Object.<anonymous> (/Users/podmanqe/.local/share/containers/podman-desktop/plugins/ghcrioredhatdeveloperpodmandesktopsandboxext/node_modules/@kubernetes/client-node/dist/cp.js:7:34)

@odockal I traced all the calls related to tar and file system calls and have not found any issues that could trigger this. Cannot replicate that on my Windows 10 or Mac M1. Was it happening on bare metal or virtual host? I've seen similar issues with ssh mounted filesystems.

@odockal still cannot replicate it. Should we close it?

@dgolovin tested with 1.13.0-next and 0.0.4 (sandbox) and all works now.


update node and npm versions to latest LTS

ref: https://github.com/adobe/brackets/issues/11748
I've tried doing this, but I get the following error which is not comprehensible for me:
>> Error: Command failed: C:\WINDOWS\system32\cmd.exe /s /c "bash scripts/fix-msvc.sh"
>>       2 [main] bash 3416 C:\Program Files\Git\usr\bin\bash.exe: * fatal error in forked process - fork: can't reserve memory for parent stack 0x600000 - 0x800000, (child has 0x400000 - 0x600000), Win32 error 487
>>     513 [main] bash 3416 cygwin_exception::open_stackdumpfile: Dumping stack trace to bash.exe.stackdump
>>       2 [main] bash 8080 fork: child -1 - forked process 3416 died unexpectedly, retry 0, exit code 0x100, errno 11
>> scripts/fix-msvc.sh: fork: retry: No child processes
>> 1004878 [main] bash 10932 C:\Program Files\Git\usr\bin\bash.exe: * fatal error in forked process - fork: can't reserve memory for parent stack 0x600000 - 0x800000, (child has 0x400000 - 0x600000), Win32 error 487
>> 1005294 [main] bash 10932 cygwin_exception::open_stackdumpfile: Dumping stack trace to bash.exe.stackdump
>> 1100847 [main] bash 8080 fork: child -1 - forked process 10932 died unexpectedly, retry 0, exit code 0x100, errno 11
>> scripts/fix-msvc.sh: fork: retry: No child processes
>> 3108069 [main] bash 8804 C:\Program Files\Git\usr\bin\bash.exe: * fatal error in forked process - fork: can't reserve memory for parent stack 0x600000 - 0x800000, (child has 0x400000 - 0x600000), Win32 error 487
>> 3108730 [main] bash 8804 cygwin_exception::open_stackdumpfile: Dumping stack trace to bash.exe.stackdump
>> 3207844 [main] bash 8080 fork: child -1 - forked process 8804 died unexpectedly, retry 0, exit code 0x100, errno 11
>> scripts/fix-msvc.sh: fork: retry: No child processes
>> 7224872 [main] bash 11240 C:\Program Files\Git\usr\bin\bash.exe: * fatal error in forked process - fork: can't reserve memory for parent stack 0x600000 - 0x800000, (child has 0x400000 - 0x600000), Win32 error 487
>> 7226544 [main] bash 11240 cygwin_exception::open_stackdumpfile: Dumping stack trace to bash.exe.stackdump
>> 7316396 [main] bash 8080 fork: child -1 - forked process 11240 died unexpectedly, retry 0, exit code 0x100, errno 11
>> scripts/fix-msvc.sh: fork: retry: No child processes
>> 15320804 [main] bash 11272 C:\Program Files\Git\usr\bin\bash.exe: * fatal error in forked process - fork: can't reserve memory for parent stack 0x600000 - 0x800000, (child has 0x400000 - 0x600000), Win32 error 487
>> 15321198 [main] bash 11272 cygwin_exception::open_stackdumpfile: Dumping stack trace to bash.exe.stackdump
>> 15409163 [main] bash 8080 fork: child -1 - forked process 11272 died unexpectedly, retry 0, exit code 0x100, errno 11

also, why do we need npm?

I don't exactly see why we need npm. Perhaps @jasonsanjose recalls why we need it.

btw npm is only part of node-win so it makes me wonder why there is npm for windows and not for other envs

That's a good question to ask. What happens if you don't download it? Are you able to build and run the tests?

Whew. That took some digging but there's the answer https://github.com/adobe/brackets-shell/commit/c913ec882c24051f17495490a91296f0fc7850ba. We used npm to install a dependency on ws for websocket support.
In https://github.com/adobe/brackets-shell/pull/209 we moved that code from @joelrbrandt's git repo into the brackets-shell repo, but we never removed the npm dependency.
Looks safe to remove to me.

Great! Thanks for digging this up @jasonsanjose. @zaggino do want to remove it in this PR?

Yes, because npm package is no longer on the nodejs server and the format of the tar in npm registry is a bit different.

@zaggino @ingorichter Is this blocked on something?

@ficristo Have you tested if

the existing file watching mechanism is properly woking with updated node engine?
installing extensions from the extension manager works.
Unit tests relating to extension pass
instant search is working
Unit tests relating to instant search is working.
Once all these are verified, I could run some test cases with updated node.

Oh wait! it was @zaggino 's PR :sweat_smile: Wrongly addressed to @ficristo .

This is not finished, I've run into fatal error in forked process and wasn't able to get around it. Compiling brackets works fine on my machine but not with this PR.

if anyone wants to try to compile the shell with this PR, go ahead ... maybe something is wrong with my machine, I'll test this on Mac when I get a chance

Some days ago I tryed this PR and had some problems.
I tryed again today and seems to run fine on Windows... :confused:

I tried this today and it failed to download the node binary. The current download URL is http://nodejs.org/dist/v4.4.4/node-v4.4.4-darwin-x86.tar.gz, but this version is no longer available. Only the x64 version is available for OSX. So, this needs to be updated and then the question is: does it compile and work with the x64 version?

I'd be great if someone could tackle this...
https://github.com/zaggino/brackets-eslint/issues/36

what is blocking this?

@AdriVanHoudt first things that come to my mind:

I left some comments that should be answered.
I think the version for OSX should be changed. See @ingorichter comment.
Verify all steps mentioned by @abose.
What we will do for Linux since is building from another branch?

I tried again last night and we have to change to the x64 variant. Changing it to x64 makes the download and build succeed. I haven't tested yet if Brackets launches and behaves properly (my machine had some thermal issues and shutdown down :-/). I can test it tonight on a different machine.

I've changed Mac to the 64 bits; I've updated to the latest node (as there's little point to upgrading to v4 now that v6 is out); Removed some remaining npm references;
I'd love if someone from Adobe could try to build this on the machines that they use to build releases as my current Windows machine stopped being able to compile brackets-shell for whatever reason.
If I have a valid build of shell, I can start working on a PR of making https://github.com/adobe/brackets work with it.

Guys, what about not porting fsevents_win and using https://github.com/paulmillr/chokidar for watching file system changes? If it's good for others, it should be good for Brackets, no? @ficristo @ingorichter

@zaggino I'm totally in favor: https://github.com/adobe/brackets/pull/12190

Ok, I'm gonna try building this shell and run tests using https://github.com/adobe/brackets/pull/12190 branch, we'll see what happens. Anyone else is welcome to do the same :)

@zaggino I like the idea. Frees some resources for us. I'll give it a try tonight

Now that https://github.com/adobe/brackets/pull/12647 has landed in master, there're no other native dependencies that would block this from merging.
Please review/merge @ingorichter @ficristo
I'd love to see this land for 1.8 to solve https://github.com/zaggino/brackets-eslint/issues/36 and also https://github.com/adobe/brackets/issues/11748#issuecomment-228580962

To test this I've run only the test suite.
I see some consistent failures:

Integration \ FindInFiles
MainView \ MainViewManager
Extensions \ Code Folding
Extensions \ HealthData

but I doubt any of these is related to this PR.
For example I'm sure the FindInFiles is on master too, but I didn't test master to check the others.

I tryed both on Windows10 and OSX and see some consistent failures on Integration \ FindInFiles and Extensions \ HealthData but I'm sure they are on master too.
@zaggino have you checked @abose comment?
Otherwise I think this is ready.

@ficristo @abose

yes, file watching was updated to support node 6 and that's already in master
yes, installing extensions from the extension manager works.
yes, unit tests relating to extension manager pass
yes, instant search is working
yes, unit tests related to instant search are working

running this branch and latest master, no unit test is failing, from integration tests there are two failures which are definitely not related to this

@zaggino and everyone else on this thread. Thanks for your amazing efforts in taking up this herculean task. As always, I am in awe :+1:


[QUESTION] How do i import table from excel file with matching headers in the middle of the file

X
I have uploaded the file using headingRow and returning the line number
X
###Current Versions

PHP version: 7.1
Laravel version: 5.6
Package version: 3.1

I am uploading file that is generated from a software it contains some headers and tables on the top and the main data (table with headers) is after it or can say in the middle of the file,
what i want to is get the desired table by matching column name , columns name will be static .
so it will match the table in the middle of the file and get its data by matching column names as the header
Another query i have is , i couldn't find out code, how to restrict the column and rows
Previously i was using version

PHP version: 7.1
Laravel version: 5.6
Package version: 2.1

I have uploaded sample file of excel
Masonry 0.1.xlsx

I am trying to extract data of table with these matching headers
Sr. No. | Wall Name |   | Length | Breadth | Depth | Area | Quantity | Unit

Hello @AbdulBasit510
Please check the following parts of the documentation:
https://docs.laravel-excel.com/3.1/imports/heading-row.html
and:
https://docs.laravel-excel.com/3.1/imports/mapped-cells.html
With these two techniques combined, you should be able to import the precise information you need from your spreadsheets.

thank you very much Sir
Regards,
Abdulbasit
On Thu, Aug 22, 2019 at 2:28 AM jochemfuchs<EMAIL_ADDRESS>wrote:

Hello @AbdulBasit510 https://github.com/AbdulBasit510
Please check the following parts of the documentation:
https://docs.laravel-excel.com/3.1/imports/heading-row.html
and:
https://docs.laravel-excel.com/3.1/imports/mapped-cells.html
With these two techniques combined, you should be able to import the
precise information you need from your spreadsheets.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Maatwebsite/Laravel-Excel/issues/2331?email_source=notifications&email_token=AG6YNULVQCCPOQWMRKJW3NLQFWT6BA5CNFSM4INWPKNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD43DKTI#issuecomment-523646285,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG6YNUPGEHONSGNYONXOAETQFWT6BANCNFSM4INWPKNA
.


Integrate the newer xslt package in favor of the current one

Just got wind of this: https://github.com/jeroenooms/xslt. This will be a more popular project than the one I'm currently using and it looks like the installation is easier.

Fixed in 10df03c04ab8d996f3137b4890e1dc35ef79b205


Support user input for python tests in TH

Add support for getting user input during the execution of Python Tests in TH UI.
Update TC-RVCOPSTATE-2.4 as an example.

The other tests that need user input need to be updated to use this new wait_for_user_input function created in this PR.
Related TH backend PR:

https://github.com/project-chip/certification-tool-backend/pull/73

@cecille can you take a look at this and help getting it merged? This is to support TH ability to run python tests that require user action and ideally should be released to SVE participants.
CC @raju-apple


Select Storage Classes in Workbench creation

Goal
Implement storage class selection in workbench creation
Dependency issue
No response
Itemized goals

Add customHook to fetch storage classes and CR
Add the customHook to the Project Context
Ability to select  available storages in workbenches

List all the available storage classes
Auto-select the default one
If only one storage class available, disable the dropdown

Aspects continued elsewhere
No response
Mocks

https://www.sketch.com/s/f6fa37fc-0c25-49e3-8b24-ecaf981433a8/a/EeqZzD2
https://www.sketch.com/s/f6fa37fc-0c25-49e3-8b24-ecaf981433a8/a/j4VakGz

Migrated to:  https://issues.redhat.com/browse/RHOAIENG-1109


Inline experiment

Throwing this up for thoughts. by itself it's not really a win, but maybe it's worth adding some additional options that only enable it in the simple cases, e.g. no spreads to deal with?
I mostly wrote it so that when ppl say "MEH THERE IS ACTUALLY A RUNTIME" i can point them to this :P

i mean, it's definitely neat.
if you wanted to do something more robust, you could possibly use something like the existing JSX transform to convert all the props into a single props object, then look at the resulting props.className and props.style.
like, IIFE style?
const __props = { ...spread, other, foo };
return { __props, style: { __props.style, ... } };
})()}```

ooo that's a cool idea.


pyglet.graphics.vertexbuffer's documentation suggests to use create_buffer(), which was removed

pyglet.graphics.vertexbuffer's documentation suggests to use create_buffer() which was removed long time ago in 25ac7f4425eb9dd88afc5413076a6a1f245dc16c.

This is updated in the latest doc build. Thanks for opening the ticket!


Master-Master replication errors with large Redis 4.0.9 dump.rdb file.

OS: Ubuntu 18.04.5 LTS
KeyDB Version: 6.0.18-1-bionic1 from APT
DB's in Redis dump.rdb: 14
Dataset: hash and lists
Master-A: 2 x 8-core Xeon w/ 192GB RAM
Master-B: 1 x 8-core Xeon w/ 8GB RAM

Tested that servers replicate by starting them both up with blank DB's, doing several SET's and GET's.
Issues occur when I copy over my 2.5GB Redis 4.0.9 dump file into /var/lib/keydb. I start first server (Master-A) file loads fine and log shows loading the 4.0.9 dump file. I can view keys, no errors on this server.
I then start second server (Master-B) it connects up and starts to sync. Then Master-A spits out some CRITICAL "This replica is sending an error to its master: 'Invalid MVCC Tstamp' after processing the command 'KEYDB.MVCCRESTORE'" errors. Master-B will then going into a continuous resync loop. However if I connect to Master-B with keydb-cli I can see the data has loaded. It appears the two servers can't agree on the data being valid even though Master-B received a duplicate copy from Master-A. I tested this with a smaller subset of my data by doing a FLUSHDB on a copy of the redis data on 12 of the DB's. Thus only having 2 DB's worth of data (about 32MB worth) and this seemed to sync okay.

MASTER-A Config changes to default keydb.conf:
port 6390
save 900 1
# save 300 10
save 60 10000
masterauth testpass123
replica-read-only no
requirepass testpass123
server-threads 4
multi-master yes
active-replica yes
replicaof <Master-B IP> 6379

MASTER-B Config changes to default keydb.conf:
masterauth testpass123
replica-read-only no
requirepass testpass123
server-threads 4
multi-master yes
active-replica yes
replicaof <Master-A IP> 6390

Log files from both servers:
http://<IP_ADDRESS>/downloads/keydb-server-logs.tar.gz
I can give you a copy of my data to verify the issue if needed.

Some tuning in my /etc/sysctl.conf that could impact tcp-backlog and syncing:
# Increase Linux autotuning TCP buffer limits
# Set max to 16MB for 1GE and 32M (33554432) or 54M (56623104) for 10GE
# Don't set tcp_mem itself! Let the kernel scale it based on RAM.
net.core.rmem_max = 16777216
net.core.wmem_max = 16777216
net.core.rmem_default = 16777216
net.core.wmem_default = 16777216
net.core.optmem_max = 40960
net.ipv4.tcp_rmem = 4096 87380 16777216
net.ipv4.tcp_wmem = 4096 65536 16777216
# Increase max connections for Redis
net.core.somaxconn = 65365

# Make room for more TIME_WAIT sockets due to more clients,
# and allow them to be reused if we run out of sockets
# Also increase the max packet backlog
net.ipv4.tcp_max_syn_backlog = 30000
net.ipv4.tcp_max_tw_buckets = 2000000
net.ipv4.tcp_tw_reuse = 1
net.ipv4.tcp_fin_timeout = 10
net.ipv4.tcp_keepalive_time = 600
net.ipv4.tcp_keepalive_intvl = 60
net.ipv4.tcp_keepalive_probes = 20
net.core.netdev_max_backlog = 60000
net.core.netdev_budget = 60000
net.core.netdev_budget_usecs = 6000

# If your servers talk UDP, also up these limits
net.ipv4.udp_rmem_min = 8192
net.ipv4.udp_wmem_min = 8192

# Change Congestion Control Algorithm to BBR
net.core.default_qdisc=fq
net.ipv4.tcp_congestion_control=bbr

Don't believe this will be an issue but it's the only additional items I can think of that might impact this. I do run UFW on these servers.

#237
Is this related? I do see a large amount of the MVCC Timestamp errors in the logs when it attempts to replicate.

So found the issue to this replication problem.
Error in log file shows:
8368:8377:S 29 Apr 2021 18:33:09.191 * RDB memory usage when created 5004.23 Mb
8368:8377:S 29 Apr 2021 18:33:26.462 # Client id=18 addr=<IP_ADDRESS>:36701 fd=34 name= age=252 idle=0 flags=S db=0 sub=0 psub=0 multi=-1 qbuf=0 qbuf-free=32768 argv-mem=0 obl=0 oll=249 omem=269195096 tot-mem=269256560 events=rw cmd=replconf user=default scheduled to be closed ASAP for overcoming of output buffer limits.
8368:8377:S 29 Apr 2021 18:33:26.465 # Connection with replica <IP_ADDRESS>:6390 lost.

You can see the error as "default scheduled to be closed ASAP for overcoming of output buffer limits". This is telling us that the client-output-buffer-limit for replica is too small. The default is:
client-output-buffer-limit replica 256mb 64mb 60

The format is:
client-output-buffer-limit <class> <hard limit> <soft limit> <soft seconds>

You can view your used memory with:
keydb-cli -p 6379 info memory

Initially I set the client-output-buffer-limit for replica to "0 0 0". This seemed to work. After testing I was able to set this limit to "4096mb 2048mb 60". Though I might have been able to tune this more. However I am curious if this will work with say a 10GB or even larger RDB file. Some adjustments need to be made to the default keydb.conf to handle large dumb.rdb files. I am sure others will run into this issue when trying to add additional masters to their already running configuration.

Looks like you've already solved this issue. Just to add, setting the client-output-buffer-limit for replica to "0 0 0"


[Fix] Minor typo in prefer-default-export.md

I'm not sure this is a typo; it's correct (and the same) both before and after your change.

Yes, but I thought it might be unintended because eslint doesn't recommend anonymouse default export and const bar is never used.
https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-anonymous-default-export.md
I'm sorry I didn't write the explanation above.
Thank you for your time.


[Question]: How can I specify the ability(metadata) of llm model?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
The llm like DASHSCOPE has the ability of function call,but the package named llms in the llamaindex set the is_function_calling_model of metadata of all the DASHSCOPE model for False default so that it make the error "ValueError: Model name qwen-max does not support function calling API. "
So how can i  fix this problem such as specifying the is_function_calling_model  variables to True.(Attetion ,is_function_calling_model
is read-only decorating by @property)

@912100012 the LLM needs to implement specific function calling logic (i.e subclassing FunctionCallingLLLM, implementing a few new methods)
It is not as simple as flipping a metadata value


Send traces to both Honeycomb and Jaeger in staging

To provide backward compatibility with what we currently have, we want our otel-collector to keep sending traces to Jaeger while also sending them to Honeycomb.
This should be done in two steps (separate PRs)

Deploy the otel-collector in staging with a configuration that sends traces to both Jaeger and Honeycomb.
Change Gitpod to send traces to the otel-collecor

@ArthurSens Changed the scope of this so it is only for staging initially.

This repository in particular has no knowledge of any existing environment. So the context here doesn't really care if it's staging/prod/preview who is going to send traces to Honeycomb or Jaeger.
What we need to do is add the possibility to configure this, our ArgoCD apps that consume this repository are the ones deciding if traces go to Honeycomb or Jaeger 🙂

@ArthurSens That makes sense, I have changed the title and will create a new issue for enabling tracing in staging and configuring gitpod to send traces to the otel-collector.


which part of the code update pseudo label

Thank you for releasing the code! I've been trying to read your code but I haven't found the part that you generate or update the pesudo label.

Hi, I am also curious about how the pseudo-labels of unlabeled data are generated? According to the codebase, it seems that the targets of unlabeled data are always loaded from the dataloader?  Look forward to the author response, thanks~
https://github.com/jianlong-yuan/SimpleBaseline/blob/c30f2fa6813ef831c8193fdbd7c38e30b2e89797/tools/train_semi.py#L153


add simplified Chinese version

Organization: xitu/gold-miner
Translators: Nicolas(Yifei) Li, MAYDAY1993, DeadLion, lovelyCiTY, sqrthree

Thanks for putting this together. Much appreciated. Perhaps rather than a header in bold letters, we could instead write something like, "Also available in Chinese" in a smaller font at the top of the guide?

@drduh OK, I'll change it at once.


Cannot achieve the various metrics described in the paper after running the code.

Hello, your code cannot achieve the various metrics described in the paper after running. Could you please provide the hyperparameters you set before running, and the training log after running.

Hello, PengPeixi. Thank you for your interest. We will provide hyperparameters and training logs soon. In the meantime, for which tasks in the paper were you unable to achieve metrics? We would appreciate it if you could leave a specific issue.

For example, for the downstream task of medical report generation, after running the file generation_decode.py on the openi dataset, the metrics obtained by the MedViLL model are:
{'bleu 1': 0.15519337010503506, 'bleu 2': 0.09391290091144913, 'bleu 3': 0.05791537394797654, 'bleu 4': 0.03430614826591653, 'best_bleu1': 0.15519337010503506, 'best_bleu2': 0.09391290091144913, 'best_bleu3': 0.05791537394797654, 'best_bleu4': 0.03430614826591653}
It doesn't reach the level of the paper.

Hello, PengPeixi. Thanks for you patience. Due to code refactoring work, it has been delayed to answer. Note that the models released in this GitHub repository are pre-trained models. Therefore, to reach the performance of our paper, the pre-trained model needs to be a fine-tuning process. We will release our refactored code v.2 soon, and we will additionally reveal the performance achieved when using multiple CNN backbones soon. Please stay tuned!


add linked workspace endpoints(Not ready for review yet)

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

Though we introduced new APIs and properties in this PR, we don't want to bump the api-Version right now, because:

Those are not breaking changes, client which not aware of these new APIs won't call the new endpoints, and the new properties in the JSON response will be safely ignored by the JSON deserializer.
These new items are for a new feature that planning to be private preview in May, we want to bump the api-Version by then.
Thank you and looking forward your approval and merge!

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk


Owner is not actionable

When I try to activate the owner role

I get an error
2024/05/10 15:05:24 ERROR: The role assignment validation failed with status 'PendingEvaluation'

Thanks for the report! It looks like the role you're trying to activate requires evaluation (i.e. approval) from another party (e.g. a manager) - could that be the case? (ref. https://learn.microsoft.com/en-us/entra/id-governance/privileged-identity-management/pim-resource-roles-approval-workflow#view-pending-requests)
The current solution attempts to validate the request, and expects the status of the request to be 'Granted' before requesting the actual role activation. Seeing as the status returned by your request is 'PendingEvaluation', it fails. None of the roles I have available require approval, so I have never encountered this issue before. I will create a PR with a fix for this issue!

@Flukas88 I've issued a fix in #14 which I believe should resolve this issue. Please give it another go with the latest version, and let me know if your issue persists :smile:

will check asap and get back to you


Ran into a problem while getting bots.

When I run the module "pflood" It reads: ">This module hasn't been configured yet. Would you like to do so now? [Yes/No]: "
Then shortly after that it asks for the path to the file containing minecraft account credentials, which I have no idea where to find. Where can I find this?

You need to have a list of minecraft premium accounts inside of a file, separated by line. (format: email/user:password)
For example:
steveminecraft@email.com:p4ssw0rd
abccraft@cmail.com:w0rdl1st

Then, you can enter the path to that file and the program will load the accounts automatically.
Note: if you have no premium account(s), check out the 'cflood' module, it works with cracked accounts/servers.


ThirdStats logo on stats page

Is your feature request related to a problem? Please describe.
This is just a small thing, but there is no permanent logo of ThirdStats on the stats page.
Describe the solution you'd like
The logo next to the heading.
Describe alternatives you've considered
None.
Additional context

It now looks like this:


 It’s only used in Urdu. Calling it arabic period would be confusing.

Fixed

I see this is now periodurdu but there is another 'fullstop' for Ethiopic; should that be 'period'?


choose_present_mode chooses lowest priority when available

I think the function choose_present_mode is selecting the lowest priority of present_mode_priority_list if request_present_mode is not found.
https://github.com/KhronosGroup/Vulkan-Samples/blob/ae6e4a3712cebde49a29e5996a3bd2f403566890/framework/core/swapchain.cpp#L85-L99
A comment suggests that the first element of present_mode_priority_list has highest priority.
https://github.com/KhronosGroup/Vulkan-Samples/blob/ae6e4a3712cebde49a29e5996a3bd2f403566890/framework/core/swapchain.h#L158
Am I understanding this correctly, and is this intended?

Thank you for pointing this out!
Agree, looks like we are missing a break here? Created https://github.com/KhronosGroup/Vulkan-Samples/pull/409, please let me know what you think


recoverable should be configurable to not expose whether an email exist as user

When recoverable is "activated", and email-addresses are used as users account names, visitors to the page can find out information about who is a potential user of the page by filling in known email addresses to the "Reset Password" form.
When an email address is entered that belongs to a User (I assume defaults here), the messages between "valid email" and "invalid email" differ.
For my application I want to protect users privacy as much as possible. I do not want parents or colleagues (or companies, goverments, hackers ...) to be able to find out whether somebody with a given email address registered for my service or not (although the service itself is not illegal or whatever).
I would like to be able to let the devise controller behave the same irrespective of the presence of a given email address (redirect to login page with a flash notice ~ "If that email is registered, password reset instructions will be mailed to it within the next minutes").

Isn't that what config.paranoid mode is for? :)

Oh thanks josevalim and god, it is :)


How to set a maximum MultiPart Form Data size?

I know MultiPartFeature is provided by the jersey-media-multipart library but I was wondering if it is possible to set MultipartConfig from the Dropwizard Environment object or in any other way?

@cgendreau If you're talking about @MultipartConfig, that's not being used by JAX-RS/Jersey/Dropwizard directly as it's for Servlets and not JAX-RS resources.

My understanding is that I should create my own Servlet with @MulipartConfig and register it with environment.admin().addServlet(...) ?
My main goal is to set a limit on the InputStream I may receive:
@POST
@Consumes(MediaType.MULTIPART_FORM_DATA)
public void uploadFile(@FormDataParam("file") final InputStream inputStream,
                                    @FormDataParam("file") final FormDataContentDisposition contentDispositionHeader){ ...}

@cgendreau You're mixing up pure Servlets (for which @MultipartConfig might work in the right environment) and JAX-RS resources.
There is no equivalent for @MultipartConfig in Jersey, as far as I know. You can check the Content-Length request header yourself and/or use a length limited input stream to read the request body:

[ByteStreams#limit(…)](https://google.github.io/guava/releases/19.0/api/docs/com/google/common/io/ByteStreams.html#limit(java.io.InputStream, long))
BoundedInputStream

ah yes sorry, there were 2 different things.

This is what I was referring to:

MultipartConfigElement mce = new MultipartConfigElement("/tmp", 10, 1, 10);
environment.getApplicationContext().addServlet(FileUploadServlet.class, "/upload").getRegistration().setMultipartConfig(mce);

FileUploadServlet is my own class.

This is indeed part of the JAX-RS resource I had and it is replaced by a plain FileUploadServlet in what I mentioned above. It was more a wish to be able to keep the JAX-RS resource but configure the underlying Servlet. This is becoming more a Jersey thing than Dropwizard. Thanks for your suggestions with limited/bounded InputStream, that should definitely work even I have to admit I'm not sure what will happen with the "rest" of the stream once we reach the limit (do we need to consume it or we can just return) but, this is again outside of the Dropwizard scope.

Thanks for you help @joschi

I have just tested a JAX-RS resource combined with a wrapped stream from ByteStreams#limit(…) and it works perfectly with a proper Exception handling/mapping.
For the records, Apache commons-fileupload can also work directly on a HttpServletRequest so it can be easily combined with a JAX-RS resource.
Sorry for the noise, at least it makes some kind of documentation for that.

@cgendreau

Sorry for the noise, at least it makes some kind of documentation for that.

Additional documentation in form of a pull request would be welcome. 😉

One potential problem with ByteStreams#limit(…): It called only after jersey persist full file on disk in something like /var/xxx/MIMEyyy.tmp file. So for big files it will be time consuming and maybe not safe(if server disk not very big)

Related feature request https://github.com/eclipse-ee4j/jersey/issues/5570


Problems during building a docker image from nix configuration

OS
NixOS 24.05.2411.706eef542dec (Uakari)
Versions
sancho-5.1.0 (revision: https://github.com/IntersectMBO/cardano-db-sync/releases/tag/sancho-5.1.0 7dd3f60bf12c1d2853baf44e267a8dc3cb52bccf)
Problem Report
The documentation for building the Docker image using Nix suggests the following commands:
nix build .#dockerImage
docker load -i ./result

However, the following issues were encountered:

No such flake target as dockerImage:❯ nix build .#dockerImage
warning: ignoring untrusted substituter 'https://cache.iog.io', you are not a trusted user.
Run `man nix.conf` for more information on the `substituters` configuration option.
error: flake 'git+file:///home/placek/Projects/IntersectMBO/cardano-db-sync' does not provide attribute 'packages.x86_64-linux.dockerImage', 'legacyPackages.x86_64-linux.dockerImage' or 'dockerImage'

Build process using other available target (cardano-db-sync-docker) is failing:❯ nix build .#cardano-db-sync-docker
warning: ignoring untrusted substituter 'https://cache.iog.io', you are not a trusted user.
Run `man nix.conf` for more information on the `substituters` configuration option.
error: builder for '/nix/store/gmhpdji3gnwxyf7nqs6a86v4vjyxjks1-docker-layer-cardano-db-sync-base-env.drv' failed with exit code 139;
       last 10 log lines:
       > qemu-kvm: falling back to tcg
       > cSeaBIOS (version rel-1.16.2-0-gea1b7a073390-prebuilt.qemu.org)
       >
       >
       > iPXE (http://ipxe.org) 00:03.0 CA00 PCI2.10 PnP PMM+1EFD0F00+1EF30F00 CA00
       >
       >
       >
       > Booting from ROM...
       > Probing EDD (edd=off to disable)... oc./run-vm: line 5:    13 Segmentation fault      (core dumped) /nix/store/81syc34kn8xwvrfzm08xhwf87l3nm3lf-qemu-host-cpu-only-8.1.0/bin/qemu-kvm -cpu max -nographic -no-reboot -device virtio-rng-pci -virtfs local,path=/nix/store,security_model=none,mount_tag=store -virtfs local,path=/build/xchg,security_model=none,mount_tag=xchg -drive file=./image/disk-image.qcow2,if=virtio,cache=unsafe,werror=report -kernel /nix/store/ng20x62170qvivq1ym2gbcxq1s8vpl1d-linux-6.1.53/bzImage -initrd /nix/store/ijp66m0xyb3b9h998w3i7c9in9invw72-initrd/initrd -append "console=ttyS0 panic=1 command=/nix/store/0qgga5n54adny309857n7lwaz90hnbg1-vm-run-stage2 out=/nix/store/6hg5phgzxijbz92srdmqfl37qv59bkgr-docker-layer-cardano-db-sync-base-env mountDisk= loglevel=4" -m 512 -smp cpus=16
       For full logs, run 'nix log /nix/store/gmhpdji3gnwxyf7nqs6a86v4vjyxjks1-docker-layer-cardano-db-sync-base-env.drv'.
error: 1 dependencies of derivation '/nix/store/j7w1kibymia7d8wdysiy9c3j84dgczk2-docker-image-cardano-db-sync-base-env.tar.gz.drv' failed to build
error: 1 dependencies of derivation '/nix/store/hwbii13frdrkyksv2pf7qf1q3ingb92x-docker-image-cardano-db-sync-env.tar.gz.drv' failed to build
error: 1 dependencies of derivation '/nix/store/ch6sn2kwnyav475wqbh3pny63iwv236s-docker-image-cardano-db-sync.tar.gz.drv' failed to build

Additional Log Information:
The detailed logs for second part indicate the following issues:
Formatting './image/disk-image.qcow2', fmt=qcow2 cluster_size=65536 extended_l2=off compression_type=zlib size=1073741824 lazy_refcounts=off refcount_bits=16
Could not access KVM kernel module: No such file or directory
qemu-kvm: failed to initialize kvm: No such file or directory
qemu-kvm: falling back to tcg
cSeaBIOS (version rel-1.16.2-0-gea1b7a073390-prebuilt.qemu.org)

iPXE (http://ipxe.org) 00:03.0 CA00 PCI2.10 PnP PMM+1EFD0F00+1EF30F00 CA00
Press Ctrl-B to configure iPXE (PCI 00:03.0)...

Booting from ROM...
Probing EDD (edd=off to disable)... oc./run-vm: line 5:    13 Segmentation fault      (core dumped) /nix/store/81syc34kn8xwvrfzm08xhwf87l3nm3lf-qemu-host-cpu-only-8.1.0/bin/qemu-kvm -cpu max -nographic -no-reboot -device virtio-rng-pci -virtfs local,path=/nix/store,security_model=none,mount_tag=store -virtfs local,path=/build/xchg,security_model=none,mount_tag=xchg -drive file=./image/disk-image.qcow2,if=virtio,cache=unsafe,werror=report -kernel /nix/store/ng20x62170qvivq1ym2gbcxq1s8vpl1d-linux-6.1.53/bzImage -initrd /nix/store/ijp66m0xyb3b9h998w3i7c9in9invw72-initrd/initrd -append "console=ttyS0 panic=1 command=/nix/store/0qgga5n54adny309857n7lwaz90hnbg1-vm-run-stage2 out=/nix/store/6hg5phgzxijbz92srdmqfl37qv59bkgr-docker-layer-cardano-db-sync-base-env mountDisk= loglevel=4" -m 512 -smp cpus=16

The build process fails with a segmentation fault, indicating an issue with the QEMU virtual machine setup. Additionally, it seems that KVM kernel module is not accessible.

Build process using other available target (cardano-db-sync-docker) is failing:

This is the correct target (dockerImage was removed some time ago)

The build process fails with a segmentation fault, indicating an issue with the QEMU virtual machine setup. Additionally, it seems that KVM kernel module is not accessible.

I have been happily building cardano-db-sync-docker on NixOS unstable on a regular basis for a long time without issues (as has CI). I wonder if making yourself a trusted user will make any difference. In any case, I think it would at least allow you to download the cached artifact.

I wonder if making yourself a trusted user will make any difference. In any case, I think it would at least allow you to download the cached artifact.

And how to do that? Or is it restricted for IntersectMBO organisation memners only?

In my NixOS config, I have:
nix.settings = {
  trusted-users = ["root" "@wheel"];
};

Can this close?


how to deploy a rocketqa model in paddle serving?

I can't find any instruction on how to deploy rocketqa model in paddle serving. I think this is very basic need for using rocketqa. So could someone give me some advice?
Thanks!

In examples/faiss_example/rocketqa_service.py, you can see an example of developing a web service of rocketqa.


investigate memory leak

Got a report of slowly growing mem usage. We should confirm this and track down the source.

Hi, I've been looking through the code base to learn more about Ethereum.
If I saw correctly you don't have any unsafe code anywhere so unless one of the crates you're using has a memory leak there shouldn't be one.
I haven't measured anything but on  the Node struct you keep all the finalized_payloads in memory, so they grow over time. I don't know how large these are though. Could that be the problem?

Oh that may actually be the issue. Thanks for taking a look.

Recent refactoring should have fixed this.


Keep running out of memory when trying rmsprop for matrix factorization

Description
(A clear and concise description of what the bug is.)
Hello, I'm new to djl and currently trying rmsprop for matrix factorization. My code is basically written according to the rmsprop code in chapter 11 of the "Dive Into Deep Learning". The problem is whenever I try to run the code, the program would fastly consume up my ram (or my gpu memory when using gpu). But when I run my tensorflow implementation in python the program worked fine. I've tried to switch from cpu to gpu, or switch the engine from mxnet to pytorch. But none of them worked.
Expected Behavior
(what's the expected behavior?)
Maybe the program shouldn't keep consuming my memory since basically the cost will be calculated once and discarded in every iteration.
Error Message
Exception in thread "main" ai.djl.engine.EngineException: [enforce fail at ..\..\c10\core\CPUAllocator.cpp:75] data. DefaultCPUAllocator: not enough memory: you tried to allocate 29333056 bytes. Buy new RAM!
	at ai.djl.pytorch.jni.PyTorchLibrary.torchBackward(Native Method)
	at ai.djl.pytorch.jni.JniUtils.backward(JniUtils.java:1340)
	at ai.djl.pytorch.engine.PtGradientCollector.backward(PtGradientCollector.java:46)
	at ai.djl.pytorch.engine.PtGradientCollector.backward(PtGradientCollector.java:31)
	at MFRMSProp.mfRMSProp(MFRMSProp.java:78)
	at MFRMSProp.main(MFRMSProp.java:86)

How to Reproduce?
public class MFRMSProp {
    private NDArray M;
    private NDArray W;
    private NDArray H;
    private NDArray squareW;
    private NDArray squareH;
    private int nodeNum = 2708;
    private int maxIter = 5000;
    private float baseEps = (float) 1e-10;
    private int vecSize = 100;
    private float learnRate = (float) 0.015;
    private float decay = (float) 0.9;

    private float[][] MfromFile() throws IOException {
        BufferedReader bufferedReader = new BufferedReader(new FileReader("./M.txt"));
        String line = null;
        float[][] arrayM = new float[nodeNum][nodeNum];
        int iter = 0;
        while ((line = bufferedReader.readLine()) != null) {
            String[] vals = line.split(" ");
            for(int j = 0; j < nodeNum; j++) {
                arrayM[iter][j] = Float.parseFloat(vals[j]);
            }
            iter++;
        }
        return arrayM;
    }

    private void initMatrix() throws IOException {
        NDManager manager = NDManager.newBaseManager();
        M = manager.create(MfromFile());
        W = manager.randomNormal(0f, 1f, new Shape(nodeNum, vecSize), DataType.FLOAT32);
        H = manager.randomNormal(0f, 1f, new Shape(vecSize, nodeNum), DataType.FLOAT32);
        squareW = manager.zeros(new Shape(nodeNum, vecSize), DataType.FLOAT32);
        squareH = manager.zeros(new Shape(vecSize, nodeNum), DataType.FLOAT32);
    }

    private void rmsProp(NDList params, NDList states) {
        float gamma = decay;
        float eps = baseEps;
        for (int i = 0; i < params.size(); i++) {
            NDArray param = params.get(i);
            NDArray state = states.get(i);
            // Update parameter and state
            // state = gamma * state + (1 - gamma) * param.gradient^(1/2)
            state.muli(gamma).addi(param.getGradient().square().mul(1 - gamma));
            // param -= lr * param.gradient / sqrt(s + eps)
            // param.subi(param.getGradient().mul(learnRate).div(state.add(eps).sqrt()));
            param.sub(param.getGradient().mul(learnRate).div(state.add(eps).sqrt()));
        }
    }

    public void mfRMSProp() throws IOException {
        initMatrix();
        System.out.println("matrix initialized");
        W.setRequiresGradient(true);
        H.setRequiresGradient(true);
        NDArray cost;
        NDList params = new NDList(W, H);
        NDList states = new NDList(squareW, squareH);
        for(int i = 0; i < maxIter; i++) {
            System.out.println(i);
            try (GradientCollector gc = Engine.getInstance().newGradientCollector()) {
                cost = M.sub(W.dot(H)).pow(2).sum().add(W.norm()).add(H.norm());
                gc.backward(cost);
            }
            rmsProp(params, states);
        }
    }

    public static void main(String[] args) throws IOException {
        MFRMSProp mfrmsProp = new MFRMSProp();
        mfrmsProp.mfRMSProp();
    }
}

Steps to reproduce
(Paste the commands you ran that produced the error.)

Just paste the above code to your IDE and run it, the problem will pop out after a few epochs.

What have you tried to solve it?

Changed the engine from mxnet to pytorch
Tried the same code on win10 and linux
Tried both the cpu and gpu versions
The problem still exists even when I remove the gc.backward(cost)

Environment Info
Please run the command ./gradlew debugEnv from the root directory of DJL (if necessary, clone DJL first). It will output information about your system, environment, and installation that can help us debug your issue. Paste the output of the command below:
------------------ CUDA -----------------
[DEBUG] - Found cudart: C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v11.0\bin\cudart64_110.dll
GPU Count: 1
CUDA: 110
ARCH: 75
GPU(0) memory used:<PHONE_NUMBER> bytes

----------------- Engines ---------------
Default Engine: MXNet
[DEBUG] - Using cache dir: C:\Users\xuduox\.djl.ai\mxnet
[DEBUG] - Loading mxnet library from: C:\Users\xuduox\.djl.ai\mxnet\1.8.0-cu110mkl-win-x86_64\mxnet.dll
Default Device: null
PyTorch: 2
[DEBUG] - Using cache dir: C:\Users\xuduox\.djl.ai\pytorch
[WARN ] - No matching cuda flavor for win found: cu110.
[DEBUG] - Loading pytorch library from: C:\Users\xuduox\.djl.ai\pytorch\1.9.0-cpu-win-x86_64\0.13.0-SNAPSHOT-cpu-djl_torch.dll
[INFO ] - Number of inter-op threads is 6
[INFO ] - Number of intra-op threads is 12
MXNet: 0
XGBoost: 10
TensorFlow: 3

--------------- Hardware --------------
Available processors (cores): 12
Byte Order: LITTLE_ENDIAN
Free memory (bytes): 153743080
Maximum memory (bytes):<PHONE_NUMBER>
Total memory available to JVM (bytes): 266338304
Heap committed: 266338304
Heap nonCommitted: 34013184

@hoodiney
Please try the following:
    public void mfRMSProp() throws IOException {
        initMatrix();
        System.out.println("matrix initialized");
        W.setRequiresGradient(true);
        H.setRequiresGradient(true);
        NDArray cost;
        NDList params = new NDList(W, H);
        NDList states = new NDList(squareW, squareH);
        for(int i = 0; i < maxIter; i++) {
            System.out.println(i);
            try (GradientCollector gc = Engine.getInstance().newGradientCollector();
                   NDManager temp = NDManager.newBaseNDManager()) {
                params.tempAttach(temp);
                states.tempAttach(temp);
                cost = M.sub(W.dot(H)).pow(2).sum().add(W.norm()).add(H.norm());
                gc.backward(cost);
                temp.ret(params);
                temp.ret(states);
            }
            rmsProp(params, states);
        }
    }

@frankfliu Thanks very much for the timely response! I couldn't find the newBaseNDManager() method in the 0.12.0 version of djl's api, so I used newBaseManager() instead. My code lasted more epochs than before but eventually still run out of memory.

@hoodiney, ooh, I missed NDArray M, I have update the above code. Please try it again.

@frankfliu It worked! Thanks a lot. May I ask where I can learn about the cause of the bug and have a better understanding of the tempAttach and ret functions, so that I can fix the problem by myself next time? I took a look at the Javadoc but still a bit confused. Thanks again for the help!

@hoodiney
You find memory management document: http://docs.djl.ai/docs/development/memory_management.html

@frankfliu Thanks very much for the kind help! That indeed helped me a lot.


[WIP] Implement XBM

Closes #163.
I'm adding commits after small refactoring and polishing comments. Will request a review afterwards.

It's zero-size tensor (scalar), so regular equal and not-equal operators should work --we use torch.allclose and the similar to compare vectors and embeddings to avoid ambiguous meaning of comparing them.

trying to compare xbm_loss and regular loss:
torch.random.manual_seed(42)
embeddings = torch.rand(64, 100)
groups = torch.randint(0, 10, (64,))

loss = TripletLoss(margin=1.0, mining="hard")

print(loss(embeddings, groups))
print(loss._compute_xbm_loss(embeddings, groups, embeddings, groups))

If I understand the approach right,  loss and _compute_xbm_loss should give same result in this case, but in fact I got
tensor(1.5443)
tensor(50.5432)

Order of magnitude different. I am not sure how the implementation was validated and if this is an expected result, but in this case xbm weight constant is especially tricky to select.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #746 (fbb8505) into main (16851d4) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #746   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17


Fix admin page

This branch fixes two things wrong with the admin pages:

Fixes inline permissions which caused shipments and asyncjobs to be unable to load
Fixes Historical shipment detail by removing route_link from it's fieldsets

@mlclay should route_link be excluded from fieldsets? Sounds like it's breaking things but I'm honestly not familiar enough with the Django admin stuff to understand what's wrong from just glancing at it.

The route_link was not removed as including it was causing the BaseModelHistory part of the HistoricalShipmentAdmin to crash. I was unable to get the route working as expected even when placing the route_link definition there so this seemed to be the best course of action.

We made the decision to not support historical Routes.  So the route_link does not exist on the historical model.  It's fine to not include that on the historical view.

Were there any other changes that needed to be made before merging?


Fixed small things in preprocess_audio to make it work.

Fixed the save call for torchaudio which with version 2.3.1 from requirements.txt doesn't have a write call

Wow, you're fast :) I only edited this on Friday to move towards the dataset release. Until the preprocessing script is properly referenced from the README, please consider it as a "sneak preview" with rough edges, maybe we'll also rewrite it.
In any case, you're correct, it should be save, not write, and it should also be atleast_2d, not atleast_1d. Your changed return statements also make sense, but the collected data is not used anywhere any more, so we may get rid of it altogether. I will address this when touching the preprocessing script again. Thank you for the heads up!
Out of curiosity, did you run this over the GTZAN dataset? Let me know if you need help, or need urgent access to the preprocessed data.

Ok, I updated the script. Sorry for bypassing your PR instead of editing and merging it, it was just easier this way. We will continue in the release_data branch to not further confuse people and merge back when it's ready. Again, let me know if you need urgent access to the data!


Title Alpha Issues

[x] Melee should be highlighted. The secondary object that acts as the highlight has a 0 alpha.
[ ] The top and bottom gray bars are missing.

The TExpSimplify causes the gray bars to disappear. Specifically, if either one of the first two functions is commented out, they reappear. Having double-checked the logic of those 100 times over previously, I'm content with commenting one out for now.


Remove .DS_Store file

I noticed a .DS_Store file that had been added.  This PR is to delete it, since I'm guessing that it was added accident.

Thank you for noticing it.


Personal Portfolio of Subhajit added

Good work. I am merging your PR.
Do give us a star by hitting the star button of this repo!


CMIP7 DataRequest

Adds CMIP7 DataRequest parsing
Features

CMIP6DataRequest can be constructed from git address

Blockers

[ ] #81 needs to go in first

@mandresm: The failing formatter check on https://github.com/esm-tools/pymorize/pull/84/commits/50be3092ffed7a0a7d0a6231bb14414b20615c91 is linked to the non-centralised structure for isort/black. I've added it by hand for now, but that might cause a merge conflict later on


Moved Matrix4i to TeraMath

Seems to be used in GrammarSystem only.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/364/
Hooray Jenkins reported success with all tests good!

This should complete #2019.

Thanks :-)
Can confirm, no apparent compilation issues with the main lineup. We'll need to do a new TeraMath release + bump the engine dependency to use it. Since nothing depends on it currently that's a whenever thing tho, I'll leave a reminder somewhere.


Long names overflow in the "author card" component

When long names are added to the "author card" component they extend beyond their container. Long names also change the shape of the author image from a circle to a narrow ellipse.

This issue is similar to #510

Hi @theadamparker. I am having issues recreating this issue.
https://user-images.githubusercontent.com/61438849/161057057-3631c94e-251b-4a8f-a59e-7667919deb76.mp4

@devjgigas I'm seeing the same as you—this is the correct behavior. If everything seems to be in good working order we can close this issue.


Split periodic workflows

Separates the frozen materialization workflow (which is long running) and updating the live database workflow (runs faster, contains the update root_id and new annotation ingestion workflows).
This allows for updating the live database more often while running the materialization at low usage points in time.

This looks worth testing on ltv4, i didn't see any problems reviewing the code by eye


Dark theme of meshery

Description
This PR fixes #
Notes for Reviewers

Here is a list of already found existing issues in meshery that I am currently working on, if you are reviewing this PR, Ignore these sections.
https://docs.google.com/document/d/1Q1txcF99Z4OzW7W-PUaZySSo94JYXUL5asmCY9RWM58/edit?usp=sharing
Signed commits

[x] Yes, I signed my commits.

No updates this past week, @Harshit-Dandriyal?

No updates this past week, @Harshit-Dandriyal?

I am updating it time to time.

conflicts...

Fixing some things here..

Is there a flag that we can use to enable this mode without exposing the UI control to users? @Harshit-Dandriyal @MarioArriaga92 ?

https://user-images.githubusercontent.com/56961488/205129163-c087f3a6-2117-450f-9b1b-2153ba2868a2.mp4
@leecalcote I placed it under popover, so, It cannot be exposed much

@Harshit-Dandriyal ok. +1 from me.

However, this is what I see when I build this PR:

All service mesh logos seem to be missing in the menu.

However, this is what I see when I build this PR:

changed..

All service mesh logos seem to be missing in the menu.

This should not happen, let me see :thinking:

Each of the Application icons (k8s, docker-compose, helm) need light version of their logos. While you're at it, confirm that all other logos have a light and dark mode, too.

All service mesh logos seem to be missing in the menu.

I think this is not because of this PR, I am fixing it though, but this is happening in the latest master branch also.

@Lee < charcoal buttons :smile:

Merge conflict.

@Harshit-Dandriyal can we have a toggle instead?

@Harshit-Dandriyal can we have a toggle instead?

Yup, There can be.. Great suggestion

@Harshit-Dandriyal what @leecalcote means is some kind of environment variable we can set to false so the Dark Mode option/checkbox isn't rendered at all

@Harshit-Dandriyal what @leecalcote means is some kind of environment variable we can set to false so the Dark Mode option/checkbox isn't rendered at all

Right on. Yes, that's right. I will suggest this potentially simpler path, though: If the mode switcher is buried in the user menu, this is probably hidden enough (as a last menu item under "Logout").

https://docs.google.com/document/d/1Q1txcF99Z4OzW7W-PUaZySSo94JYXUL5asmCY9RWM58/edit?usp=sharing

Please merge the issues doc into this doc: https://docs.google.com/document/d/1ZZz_FRfnyHWqaGfufPY188z88BploZnW0XT2k7Hnq1s/edit#heading=h.c5hnflbvp66z
and add it into PR description.

@Harshit-Dandriyal Dark mode looks awesome. :sunglasses: . I just tried it.
I guess this one is missed.

Snackbars also need to reference themes and change colors. Reference: https://notistack.com/features/customization

@Harshit-Dandriyal Integration and lint checks are failing - MuiThemeProvider is defined but never used. https://github.com/meshery/meshery/actions/runs/3711142815/jobs/6291306432#step:5:532

@Harshit-Dandriyal the "do-not-merge" label is still present on this PR.
There are three files with merge conflicts.

3 merge conflicts to resolve

@Harshit-Dandriyal A lint error to fix. MuiThemeProvider

Another conflict cropped up

@Harshit-Dandriyal Would you resolve the merge conflicts finally?

Merge conflicts...


Smart extend keyboard short cut conflicts with default Move Line Up/Move Line Down

I am not sure if you noticed, but by default, Move Line Up is bound to Alt + UpArrow and Move Line Down is bound to Alt + DownArrow.
Given how incredibly useful Move Line Up/Down are, I would expect that everyone would be complaining that p42 just installed a conflicting set of keyboard bindings.
I am removing the bindings that p42 added. I am sure there is a better set of keys to bind the smart expand to.

@gregveres thanks for the bug report! I agree, this should not happen - will put out a fix soon.

I've released v1.142.2, which removes the smart selection keybindings. Thanks again for the bug report!


MWS AUTH Token

How to access all the API's through MWS_AUTH_TOKEN without providing access_key and secret_key, as I have studied in amazon document that with out key's you can access the api's with the help of MWS_AUTH_TOKEN
Can you help me out with small example, as i have never seen in you existing code with such functionality as you have already saved the access_key and secret_key. I don't want to use the both

You can't. You must have misunderstood the documents. Access and secret key are developer's related credentials, wether auth token is seller's token. You cannot use amazon mws without a seller/user token. Should be closed.


ARTEMIS-4546 modernize Validators, etc.

No semantic changes here. Just some code gardening.

looks way better without the clutter


Jedis Cluster JedisPool getResource   throw exception :Software caused connection abort: socket write error

that my whole steps
when I use JedisCluster
the JedisSlotBasedConnectionHandler.getConnection()  throw an exception which said that " JedisConnectionException: no reachable node in cluster"
so I debug the method invoke stack
1 jedis = pool.getResource();
2 internalPool.borrowObject();
3 validate = factory.validateObject(p); at line 492 in class GenericObjectPool
4 jedis.ping()
5 client.getStatusCodeReply();
and I get a new exception "Software caused connection abort: socket write error"
I don't know why I get this
I use redis-cli  -c  at the same machine and it works

@moonlightheng you're running redis-cli -c from the same machine where the Java code is running?.

Closed due to inactivity

How will we disable the 'jedis.ping()' in JedisSlotBasedConnectionHandler.getConnection()?
Most of connection pools tend to have an option to set 'seconds to trust a healthy connection' in order to eliminate the connection test during ''seconds to trust" ...


Types: Time scales with string values aren't recognized

Expected behavior
I expected TypeScript to accept options: { scales: { x: { type: "time", min: "2021-01-01", max: "2021-12-01" } } }.
Current behavior
Compiler errors:
Type '"time"' is not assignable to type '"timeseries" | undefined'.
Type 'string' is not assignable to type 'number | undefined'.
Reproducible sample
https://www.typescriptlang.org/play?#code/JYWwDg9gTgLgBAbzgYQBYENZwL5wGZQQhwDkAxhrAHQBWAziQNwCwAUKJFuZTPQLToAJujAwAplD7DxfPADsGLVmwA2Y+GRgAPOAF44giGQCuIMXJhUA5uoCiasxYBCATwCSggBQAiEC7SYMN4AlHDodHAAEgAqALIAMsjocgBu4fZijjBKavB+AVj6cmIA7ig8nppaADSIbHBwMC5gYgBccN4qwMXe1fUG6DDo7Qj9DdLh6nTtANpjDXWsC8twKugARmIq7d4ACsBivfPLE7PHK0haOwBMAAzXAIx8t08vvXAu7QDMOH1LK8tLjd7q9rs8Hu9PnAHr9zoC4FcOndHs8vuDIe0ACyw-4AhpApEg56Y9G1KEw7B-PELAneZGvACspI+7QZOOp+IRwJRtwAbMyoQBOdkc2n054AdgF7WuIupYqJtwAHNK4Ly5XiFTzBaq2ZS4TSuYSUQ9bqqfvrcZqjXSiQ9XhCyayNQCtU8HmC3k64EqcAaALpUvHraCCCTICAqaA7ADEt3jCaOVuW63QZAA1lZCMY5III1GoLH4wAxYvx7znbDzf39S0NCCiYAQBQjeZ0MjoNTTRYAxGjZMLJotHYwUCHIMAkDdbkO0kGkDoRG2k2eiGVicLKH96lDtodLrFTBJ6lTuTtW4b5YLxGmyvzKv-B-YYJKIA
Optional extra steps/info to reproduce
Here's a codepen demonstrating that this works at runtime: https://codepen.io/StephanTLavavej/pen/RwjooGY
The Time Cartesian Axis > Date Formats documentation explains how string values are adapted:

When providing data for the time scale, Chart.js uses timestamps defined as milliseconds since the epoch (midnight January 1, 1970, UTC) internally. However, Chart.js also supports all of the formats that your chosen date adapter accepts.

And then it lists min and max as number | string.
Possible solution
No response
Context
I'm converting the microsoft/STL Status Chart, powered by Chart.js, to use TypeScript. :smile_cat:
chart.js version
v3.7.0
Browser name and version
Chrome 98.0.4758.81
Link to your project
No response

Thanks! This one is interesting since we thought we typed it correctly. https://github.com/chartjs/Chart.js/blob/master/types/index.esm.d.ts#L3179-L3180. it appears the types aren't merging well with the min/max options on CartesianScaleOptions. Should be a simple fix 😄


An attempt at fixing #635

Hi,
this patch would emit something that works for the simple example in #635.
What do you think?

I don't think it's an appropriate fix; it addresses just one way the issue manifests itself and it adds Quartus-specific hacks to the RTLIL backend.

OK no problem, I know that my understanding of nmigen internals is very limited.


Typo in explanation why 'using namespace' is a root of all evil

In

should be mp instead of mt (the fourth line)

Fixed on Wiki, but i don't have Markdown to PDF tools installed ATM, so will generate PDF later. Thanks.


Update ravenjs to 3.22.1, add releases configuration

Purpose
Summary of Changes
Side Effects / Testing Notes
Ticket
https://openscience.atlassian.net/browse/EOSF-
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md (don't have initial release yet)

Coverage remained the same at 6.481% when pulling 42d436b71e5302a4436cb426f48e2a76b4062d04 on binoculars:feature/update-raven into f22c801da5422ffa9f2a9af244ff0a972034a528 on CenterForOpenScience:develop.


Is there any way to add user specific info to result of algolia?

Hi,
I need to append some info related to authenticated user to the result returned by algolia algoliasearchHelper. I tried to make a ajax request inside helper.on method
helper.on('result', function(data) {
        renderHits(data);
        myfunction(data);
});

but I always get this error: "Uncaught TypeError: Illegal invocation".
Do you have any kind of suggestion?
Regards

Hi, did you figure out how to do it @fab10m?

Hi Vincent, I closed the issue because I realized that the error was caused by my trivial mistake :-)
I did big steps forward, but I still have some doubts about which is the best approach:

append to the objects contained in data variable, the extra attribute retrieved throgh tha ajax request; then execute the function renderHits that compile the template with hogan.js.
render the page executing the function renderHits and then execute my function to modify the html

Do you have any kind of suggestion?
Regards

Is the data you want to add to hits related to the results you got from Algolia? Or can you get this data independently?

The data is only related to the objectID. Pratically, I need to add the info that tells whether a user is following or not an object.

There's a formatData function on the hits widget that let's you rework the hits.
Unfortunately it is not yet asynchronous compatible, we could add that maybe by just supporting returning promises.
In the meantime what I would do is just put some

 in your hit template.
Then fill it after each search, like this:
search.on('render', function() {
  var more = document.querySelectorAll('.more-data');
  // loop through all the more data elements, find the object id, do the requests, fill in the html
});

What do you think?

I did something similar to what you suggest, but definitely less elegant :-)
I retrieved the objectID in this way
var services_id = $.map(data.hits, function(o){
                return o.objectID;
 }).join();

While we are in chat, could tell me how can I update an attribute of the algolia's indexed object?

Can you open another issue with a more descriptive content? It's a lot easier to deal with and for people to refer to in the future, thanks!


[WIP] initial curand implementation for model init

as an alternative to the multi-threaded model init, this uses curand to generate initial weights directly on the GPU.
It is still work-in-progress, needs error-checking, and I dislike the cudamalloc inside the loop, but it should be enough to measure how much speed-up we would get.

Perhaps a stupid question, but could the "fill_kernel" be replaced with a "cudaMemsetAsync"?

cudaMemset works on the level of individual bytes; we want to set floats or bf16s here; for zero, conveniently repeating the bit-pattern 0 also implies the number zero, so you can use it to initialize floats to 0.0; but unless your desired float has the same bits in every byte, it isn't going to work for other values.


Release v0.2.5

Going to start a develop branch and do proper releases.
Resolves #274

@white238 -- can you please add links to the commits in each release, as we do on Axom's release-notes?
The links look like this:

Text: https://github.com/LLNL/blt/compare/v0.1.0...v0.2.0
Link: https://github.com/LLNL/blt/compare/v0.1.0...v0.2.0

See: https://github.com/LLNL/axom/blob/develop/RELEASE-NOTES.md

Appveyor has gone off the deep end. It's reporting the wrong SHA test results to github (the cancelled one) and the correct SHA build is sitting queued.  This PR doesn't have code changes so I am going to merge even though the tests haven't run. I feel dirty.....


Status bar colour is not getting changed when SAP Business App Studio reloads with project open in workspace

Tutorial: https://developers.sap.com/tutorials/appstudio-fioriapps-create.html
Hi Colleagues,
In "Step 2  Open the project's folder" of this tutorial, at Step 4. it is mentioned as "The status bar color changes to blue, indicating that a workspace or folder is open."
Actual Result: But there status bar colour is not getting changed.
Screenshot in the Tutorial:

Actual result in the UI:

Labels to be added:
Scenario label: SCPTest-fbizapp
Sprint Label: SCPTest-2308A
Landscape Label: Trial3 (AP Region)
Priority: Low
Many thanks & regards,
Baskar

Hi Baskar,
I changed this - as soon as Joshua accepts the changes, they will become visible.
Best regards,
Birgit

Hi Baskar,
this has been updated. Thus closing this issue.
Best regards,
Birgit


[Extensions] to allow a connector feature to connect components

Current Behavior

Desired Behavior

Screenshots/Logs

Environment

Host OS: Mac Linux Windows
Meshery Server Version: stable-v
Meshery Client Version: stable-v
Platform: Docker Kubernetes

Contributor Guides and Handbook

📚 Meshery Build and Release Strategy
🎨 UI Design For Meshery Extensions Figma in Figma (open invite)
📚 Meshery Extensibility Providers
⌨ Meshery API Docs
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

// @aabidsofi19


[Backup] Create backup on iCloud keychain

This function is complete in the dev mode IOS version 2.0.4(2). The vision of test phone is IOS 17.1.2
The whole process is attached below. User can backup on iCloud keychain which process is same as UI design, after that user can view all backup information in their backup page.
https://github.com/Outblock/FRW-iOS/assets/87978227/e1d0bfba-6637-4b32-8446-4fab392b6ccb


bundler under npm install fails to execute (wrong ruby version)

if I run npm install the script dies with
Your Ruby version is 2.0.0, but your Gemfile specified 2.4.0

however if I run bundle install --binstubs ./examples/ios && bundle exec pod install --project-directory=./example/ios/ all goes fine
my env:
$ ruby -v
ruby 2.4.0p0 (2016-12-24 revision 57164) [x86_64-darwin16]

$ bundler -v
Bundler version 1.14.6

Any idea? (sorry, not versed with ruby :(  )

Hi @rborn, try running this:
curl -sSL https://get.rvm.io | bash -s stable --ruby=2.4.0

Then open a new terminal tab and rvm will change the ruby version to 2.4.0 automatically.
If it doesn't, run rvm use ruby-2.4.0.

@brunolemos no love 😞  I must have a borked installation on my machine. I'll keep trying and let you know. Thanks for the help!

@brunolemos I found it (not fixed yet though) - it seems that the npm script uses the default bundle on the system instead of the one set by rvm.
I need to find how to force npm to use my settings


bump dependencies

closes https://github.com/helm/helm/pull/10580
closes https://github.com/helm/helm/pull/10581

@bacongobbler need rebese.


Investigate failure crate

askalono uses a lot of Box<Error>. The failure crate has been making a buzz: https://github.com/withoutboats/failure
This is likely a good error management story for askalono. I haven't done any research into actually implementing this; it's likely not too difficult to do.

Steve Klabnik and Ashley Williams made a good point on Twitter. Libraries that use custom derive (like Failure or Structopt) allow people to have write super-declarative code, where it's feasible to define a struct or enum, add an annotation, and do your thing. It's really nice to not worry about imperative builders anymore!

Done in PR #12, thanks!


Acrostic should conflict with specific length option

(External report)
if we mix -a with -n - the app should just stop with an error.
mixing these 2 makes no sense...
artur@elitebook ~ $ xkcdpass -V -a Leon -n 15
The supplied word list is located at /usr/lib/python3/dist-packages/xkcdpass/static/default.txt.
Your word list contains 38271 words, or 2^15.22 words.
A 15 word password from this list will have roughly 228 (15.22 * 15) bits of entropy,
assuming truly random word selection.
Labrador essence ovoidal narcotic

it made 4 word password, and told me entropy for 15-word one...
(mixing -n with -a makes no sense, but app should get it and throw some error perhaps)

Fixed in #102