skypilot: 0.6.0 -> 0.6.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for skypilot is: Run LLMs and AI on any Cloud
meta.homepage for skypilot is: https://github.com/skypilot-org/skypilot
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.6.1 with grep in /nix/store/dyp6mv899di508mvzzak0p7nj516qpal-skypilot-0.6.1
found 0.6.1 in filename of file in /nix/store/dyp6mv899di508mvzzak0p7nj516qpal-skypilot-0.6.1

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

skypilot

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/e2143121ce8b6078da637c41c9104c31a57dea89#skypilot

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

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

2 packages built:

skypilot
skypilot.dist

Maintainer pings
cc @seanrmurphy for testing.

Result of nixpkgs-review pr 330575 run on x86_64-linux 1

3 packages built:

nixpkgs-manual
skypilot
skypilot.dist


fix: type error in options parameter of mdxOptions

What:
Change the type of the options parameter in the mdxOptions function from the original ProcessorOptions to Option, which is the CompileOptions type.
Why:
After upgrading the @mdx-js dependency to version 3.0, there have been some changes in the Options defined in @mdx-js/esbuild/lib. This change has made the previously used ProcessorOptions invalid, causing a type error for the options parameter when using mdxOptions in TypeScript.
How:
Read the doc about ProcessorOptions and CompileOptions and find that need to use CompileOptions here.
Checklist:

[x] Documentation
[ ] Tests
[ ] Ready to be merged

@mdx-js/esbuild/lib exports CompileOptions and an equivalent type called Options. I'm not sure which one is more appropriate to use here. In this commit, I have chosen Options.

Looks like this failed to typecheck: https://github.com/kentcdodds/mdx-bundler/actions/runs/7547192375

Thank you for your reply!
It seems that remark-mdx-images v2.0 references unified v10, causing the Plugin type error. Upgrading remark-mdx-images to v3.0 will fix this error, as it will reference unified v11, which uses the new Plugin type.
The original ProcessorOptions type does not exist in @mdx-js/esbuild/lib, which causes the type of remarkPlugins to be any. I believe the new Option type would be more appropriate.

Ah, would you like to try again with another PR?


Added more attributes

as requested in #28:

added last update reason
added condition based services
added parking lights

@gerard33 please review

Concerning the Enums in general:
I would favor going through the pains of finding all the values so we eventually have a complete list. This would make it much easier for the users of the library to understand what the values might be. It will raise a lot of exceptions in the beginning, but eventually we will get there...

@ChristianKuehnel I agree it would be great if we have all the values, on the other hand the use case I see is to show this value as a attribute of a (binary) sensor or the lock.
If that's the only use of this value, then I would favor to just get the value from the API and show it in the attributes.

Good point. We should decide per case if it makes sense to use type string or type Enum.
In that case you're right, let's use string to be more flexible. I pushed the change directly to master.


Add register view and add input fields for username and password aswell.

https://github.com/sopra-fs23-group-40/Client/commit/bf7a776aca7576e26961980efc35b73fe7521cdc


allow operation definition in service declaration

Disabling an operation is quite a painful process: you have to redeclare every call, and then inject them via the initItemOperations or inicCollectionOperations.
I added a way to specify the methods you want in the collection or the item calls.
I added two keys in the properties service definition. It is kind of a dirty way, but the tag definition do not allow array declaration and I do not think there is another way to do that ?
Moreover I did not understand the test class on the ResourcePass, if you can help me with that ?
Thank you

I like the idea and it will ease many things, but maybe that defining everything directly as service is now too limited. I took this way initially because it was easier to bootstrap this bundle but a cleaner way would be to have a traditional configuration handled by the extension that will register services.
Example:

# app/config.yml

dunglas_api:
    resources:
        product:
            class: AppBundle\Entity\Product
            methods: [ GET, POST ] # Can be empty
            operations: [ my_operation_service_name ] # Can be empty
            normalization_groups: [ a, b ]
            denormalization_groups: [ a, b ]
            validation_groups: [ a, b]

And of course the extension will reuse the existing infrastructure to register services.
What do you think?

Doing this way will allow another thing I want to do: setting normalization, denormalization and validation groups on a per operation basis instead of a per resource basis.

@dunglas far better idea IMO :+1:

Enhanced config sample taking #131 into account:
# app/config/config.yml

dunglas_api:
    resources:
        product:
            resource_class: Dunglas\ApiBundle\Resource # Optional
            entry_class: AppBundle\Entity\Product # Required
            short_name: MyProduct # Optional, guessed if not set
            methods: [ GET, POST ] # Optional, default to GET, POST, PUT, DELETE
            operations: [ my_operation_service_name ] # Optional
            normalization_groups: [ a, b ] # Optional, default to null
            denormalization_groups: [ a, b ] # Optional, default to null
            validation_groups: [ a, b] # Optional, default to null
            pagination: # Optional, default values if not set
                    default:
                        enabled: true
                        items_per_page: 30
                    client:
                        can_enable_or_disable: false
                        can_change_items_per_page: false
                        parameters:
                          page: page
                          items_per_page: itemsPerPage

And bonus: Resource can now be really immutable (without the init* methods trick).

@dunglas :tada:

@dunglas : Actually, we're already exposing a configuration instead of defining services for one of our project, using a compiler pass to generate the resources.
If it could be a native feature it would be simply awesome :sparkles: :sparkles: :+1:

I totally agree with the "classic" configuration. I just added this to the
actual configuration.
Le 17 juil. 2015 21:06, "Maxime Steinhausser"<EMAIL_ADDRESS>a
écrit :

@dunglas https://github.com/dunglas : Actually, we're already exposing
a configuration instead of defining services for one of our project, using
a compiler pass to generate the resources.
If it could be a native feature it would be simply awesome [image:
:sparkles:] [image: :sparkles:] [image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/dunglas/DunglasApiBundle/pull/174#issuecomment-122375014
.

@jdeniau are you interested in adding the "classical" configuration instead of this PR?

Yes no problem. I will push you another PR and then delete this one.

Great!

I didn't to work a lot on this. I made this branch : https://github.com/jdeniau/DunglasApiBundle/tree/jd-feat-better-config?files=1
Now I am on holiday, I'll be back in two weeks. If you want to work on it no problem. Anyway I will finish this when I an back

I updated the current configuration reference (+examples) we currently have in our projet in #185 .
I don't know neither when I will take some time to port the implementation to the bundle, however. :/
Some of the features or the way they are implemented are specific to our needs, and might not be 100% suitable in a third-party bundle context ; like the operations definition for instance: it'll use the default operation factory, and we can easily imagine a factory option to make it configurable. But there is no interface describing the factory for now. On purpose ?


Bug Report in KE 1.8: Songs ending early

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
done lol
If you are playing a downloaded version of the game, what operating system are you using?
Windows 10 64-bit, my computer specs are all up-to-date(last upgrade was past 3 years).
What version of the game are you using? Look in the bottom left corner of the main menu.
I'm using Kade Engine version 1.8 stable that just came out a few hours ago.
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
1) Play a song, I'll be using Satin Panties for this example cause it's a good song.
I'm not fully sure if this issue occurs as well for Story-mode, but I accessed the song from the Free-play menu and that's where the bug happened.

2) Play until there's 7 seconds left, bug should happen.
Please describe your issue. Provide extensive detail and images if possible.
The song that's being played ends and goes to the judgements screen earlier than it should, not allowing you to finish it in its entirety.
I managed to screen-record it, here you go:
https://user-images.githubusercontent.com/87498840/138617269-c9e19810-a5eb-49e9-ac42-67561c4768e8.mp4
As you can see, it ends with 7 seconds left and brings you to the judgements screen.

Already did pull request to merge to Master #2858


System.IndexOutOfRangeException: Index was outside the bounds of the array. (in v3.0.331)

Describe the bug
Error when loading save.
To Reproduce
Steps to reproduce the behavior:

Load save, nothing else.

System Information

OS: Tested on both EndeavourOS linux and on a windows 10 21H2 vm.
Game version: Steam cs:13779. It's a cracked version to test if the game works on linux, and it does, so I'll be buying it on steam soon and I can test there too if it would make a difference
Version of the editor: v3.0.331

Additional context
svd_fmt_5_11.sav.tar.gz
It's possible I have missing dependencies as it isn't very clear what needs to be installed. Wine comes with practically nothing and it is a fresh windows vm. On wine, I installed dotnet48. On windows, I installed the .net 6.0 desktop runtime, which also appears to include the same thing I installed on wine.

I have the same issue with a new save on windows 10.
did they change the save format in the last patch or the previous one ?

Dupe of #114


First PR of the new era

New things that work so far. Meanwhile I'm working on the next bit.

README describes current usage (briefly)
Make sure git status is happy before running, then use git commands where they make sense (it turns out that without git mv it is painful to understand what this tool has done)
Runs on single site A2 project in current working directory, with the beginnings of support for other scenarios
Rename lib/modules to modules
Rename project level of known apostrophe core modules
Support require statements that use destructuring
Fix bug causing crash with event handlers

Since I need to work linearly and not on a forest of features right now, I'm going to open new catch-up PRs consecutively as the previous one gets approved. If we can keep a cadence of reviews and approvals going regularly then they won't get unmanageable to read.
I am currently testing against the apostrophecms-website project.

eslint is broken in this project at the moment. When I fix it, I see that it will make a great many changes, which would frustrate viewing PRs. So I'll do it after this PR is approved, at least.


add clWaitForEvents to data_movers.c to avoid possible GC/OpenCL race

We always force a clWaitForEvent with enqueued data moves (read and write) whilst we are pinning the array.
After the unpin GC is free to move the buffer, potentially before written (enqueueWrite), or after buffer has moved (enqueueRead)

This will force the JNI code to be blocking for copies (in & out). We can force clEnqueueReadBuffer and clEnqueueWriteBuffer to be blocking instead in the JNI code, and avoid the clWaitForEvents.
Additionally, IMO since we are currently using a single command queue per device, this would not make too much of a difference in performance.
@grfrost , what do you think?

@jjfumero we could absolutely block (by passing true arg to clEnqueueRead/Write).
I was merely taking the lead from the current code.  One patch of the if(PRINT_DATA) branch calls getTime (or some such).  A side effect of which is that we call clWaitfOrEvent().  I merely balanced the if with an else.
Let me know which you prefer.

Ok. sure. Let's take this path (adding clWaitForEvents). Since I am developing a new design for events. it will be easy to modify this way.
Thanks @grfrost

Out of curiosity, did the error with ROCm driver go away with this patch?

Out of curiosity, did the error with ROCm driver go away with this patch?

They went from 'always' to 'intermittent' on my newer device.  Always still on the older device.  There is something weird in this codebase that ROCM does not like. Neither handcrafted C++/OpenCL nor Aparapi solutions trigger these errors.     I am hopeful that  when we implement a strategy for releasing events vs clReleaseEvent() (when we call ContextDriver.sync()/flush()) these may well go away.  All the AMD_ROCM flag did was essentially make all event queues block (as if we called clWaitForEvents after every enqueueXXX), so there maybe some other enqueuing issues that I have not seen yet.  Will neet to monitor as we make incremental fixes to event handling.


Your License

HI,
I am not expert regarding Licenses, but why don't you use the same License as freecad?
They told me that if you ever wish to join your WB to FreeCAD you have to have the same License.
What do you think?

What I think? about typos? I don't plan make more typos so typo-snake might be disappointed ;-) From law perspective it is not true you need to have LGPL to be part of AppImage.
FreeCAD is patchwork with about 1000 different licenses. Unpack AppImage and search for license file so you will see how many different licenses there are, many of them are MIT. Personally, I don't like the LGPL attitude. MIT allow for much more than LGPL but still keep ownership and being author of the code.

ok


Upload download

Codecov Report

Merging #36 into failures_testsuite will decrease coverage by <.01%.
The diff coverage is n/a.

@@                  Coverage Diff                  @@
##           failures_testsuite     #36      +/-   ##
=====================================================
- Coverage               58.91%   58.9%   -0.01%
=====================================================
  Files                     209     209
  Lines                   24372   24372
=====================================================
- Hits                    14358   14356       -2
- Misses                   8475    8478       +3
+ Partials                 1539    1538       -1

Impacted Files
Coverage Δ

cmd/admin-heal-ops.go
51.15% <0%> (-1.54%)
:arrow_down:

cmd/xl-v1-common.go
100% <0%> (+5.55%)
:arrow_up:

Continue to review full report at Codecov.

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


lxml should depend on icu 58

Otherwise you get errors like     ImportError: dlopen(/Users/aaronmeurer/anaconda3/lib/python3.5/site-packages/lxml/etree.cpython-35m-darwin.so, 2): Library not loaded: @rpath/libicui18n.58.dylib (e.g., using icu from defaults).

Or maybe it should libxslt, the full error is
    ImportError: dlopen(/Users/aaronmeurer/anaconda3/lib/python3.5/site-packages/lxml/etree.cpython-35m-darwin.so, 2): Library not loaded: @rpath/libicui18n.58.dylib
      Referenced from: /Users/aaronmeurer/anaconda3/lib/libxslt.1.dylib
      Reason: image not found

It is libxml2 that depends on icu and libxml2 is used to build both libxslt and lxml.
The CIs are doing the right thing, see https://travis-ci.org/conda-forge/lxml-feedstock/jobs/239789593#L482
Can you check if you are getting any package from defaults?

I am getting icu from defaults (I mentioned that above). That's why the dependency in the package should specify a version, so that it installs the right one.

(and more generally, it would be nice if conda-forge automatically or semi-automatically detected these version dependencies from the linkages and added them to the packages)

Apparently libxslt isn't even depending on icu at all. I just happened to have it installed. See https://github.com/conda-forge/libxslt-feedstock/issues/8.This should probably be closed in favor of that issue, since lxml doesn't link against icu directly:
$conda inspect linkages lxml
lxml
----

libxml2-2.9.4-0:
    libxml2.2.dylib (lib/libxml2.2.dylib)

libxslt-1.1.29-3:
    libexslt.0.dylib (lib/libexslt.0.dylib)
    libxslt.1.dylib (lib/libxslt.1.dylib)

zlib-1.2.11-0:
    libz.1.dylib (lib/libz.1.dylib)

system:
    libSystem.B.dylib (/usr/lib/libSystem.B.dylib)

not found:


added welcome text, hero image, and mission statement.

making an update to the reference to the css folder location


Support for FormData / multipart/form-data?

Until HTML JSON forms become standard, submitting form data via JSON payloads to a REST API is non-trivial.
With FormData support, sending a form to the endpoint from vanilla JavaScript would be as simple as:
var formElement = document.getElementById("myFormElement");
var request = new XMLHttpRequest();
request.open("POST", "http://localhost:3000/api/collection");
request.send(new FormData(formElement));

Does this make sense and would it be within the scope of Restivus?

This is something I would definitely be interested in supporting in Restivus (as well as simple:rest, via middleware). I'll look into it within the next few days and continue the discussion here. I'm no expert on FormData, so I just need some time to get an idea of what it would take to support this in Restivus. If you have any suggestions, I'm always happy to hear them. Thanks for the really great suggestion! This will be a nice feature to add.

Hi  is it  a way now to support FormData from a request to an endpoint ?


Batch will not run in terminal.

I can not get the batch file to run in the built in terminal it will just open a separate CMD prompt and run it there.

Make sure you have the setting batch-runner.runBatchIn set to "Terminal".
or are you trying to run the batch file as administrator? If so VS Code must be started with admin privileges to be able to run in the terminal.
Cheers,
Nils

Hi,
Going to close this issue now due to a lack of activity.
Feel free to re-open this issue with more information if you're still experiencing this issue.


pueue: 3.4.0 -> 3.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Nukesor/pueue/releases.
meta.description for pueue is: A daemon for managing long running shell commands
meta.homepage for pueue is: https://github.com/Nukesor/pueue
meta.changelog for pueue is: https://github.com/Nukesor/pueue/blob/v3.4.1/CHANGELOG.md
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.4.1 with grep in /nix/store/daidhbfigvpylv3n7d0s3yqa4p55pkqg-pueue-3.4.1
found 3.4.1 in filename of file in /nix/store/daidhbfigvpylv3n7d0s3yqa4p55pkqg-pueue-3.4.1

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

pueue

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/e596c6c30a43ccbdbfcd4cfbae8d9ae08491a31b#pueue

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

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

1 package built:

pueue

Maintainer pings
cc @sarcasticadmin for testing.

Result of nixpkgs-review pr 317340 run on x86_64-linux 1

1 package built:

pueue


Indentation rules are case sensitive

VHDL is not case sensitive, but the indentation rules are. I made a local change to language-configuration.json that fixes this.
Old:
    "indentationRules": {
        "increaseIndentPattern": "\\b(begin|is|port|else|loop|generate|then)\\b",
        "decreaseIndentPattern": "^\\s*(elsif|else|end|begin)\\b"
    }

Updated:
    "indentationRules": {
        "increaseIndentPattern": {
            "pattern": "\\b(begin|is|port|else|loop|generate|then)\\b",
            "flags": "i"
        },
        "decreaseIndentPattern": {
            "pattern": "^\\s*(elsif|else|end|begin)\\b",
            "flags": "i"
        }
    }

I learned about the flags option here in case you are wondering: https://github.com/microsoft/vscode/issues/74493.
Perhaps this is something that would be useful for others as well?
Side note: the only detail that does not really work to my liking is WHEN statements, but that is more of a personal preference, I was trying to get them to format like this, but I don't see how that can be done with the available indentationRules handholds in vscode.
CASE selector IS
    WHEN "00" =>
        out <= option0;
    WHEN "01" =>
        out <= option1;
    WHEN "10" =>
        out <= option2;
    WHEN OTHERS =>
        out <= option3;
END CASE;

I could be mistaken, but I don't think this has anything to do with VHDL_LS. Language configuration is a built-in feature of VS Code. This repository is for a third-party VHDL language server. If you want a tool that can do VHDL formatting, I suggest that you look at this https://vhdl-style-guide.readthedocs.io/en/3.21.0/overview.html

For a moment, I thought I had my tabs crossed, but it is actually here, so I think I've posted this in the right place (note the rust_hdl_vscode vs rust_hdl):
https://github.com/VHDL-LS/rust_hdl_vscode/blob/master/language-configuration.json

I stand corrected, apologies!

This looks like a nice addition. Could you open a Pull Request with your change?

Here's the pull request: https://github.com/VHDL-LS/rust_hdl_vscode/pull/83

Closed with #83


Fix doc generation script

make docs was failing for me due to to line 129 of util/generate_docs.py:
html = markdown.markdown(contents, ['def_list', 'codehilite', 'smarty'])

The only markdown modules I could find had markdown.markdown taking two arguments: text, and **kwargs. The required kwarg in this was extensions=.

This seems to be based on your other PR. Might be good to separate it out so that just the documentation generation changes (generate_docs) are in this one.

Oops, you're completely correct, I wasn't even thinking! Should I just git revert <commit-id> and then squash the commits?

You could do. Or you could reset to master and then cherry-pick the commit.

Alright, I think I did that right.
Cherry picking is a nice trick, I'll have to remember that one!


App provides www.expensify.com link instead of new.expensify.com link in login in SMS

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the app
If logged in then log out
Enter your contact number (one which you have access to SMS) for login
Observe SMS received for magic code

Expected Result:
App should have new.expensify.com link in SMS for login as we have in email
Actual Result:
App provides www.expensify.com link in SMS for login instead of new app link
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/f3e3503b-9b88-4e9b-a22e-be10d0aa4cd8
https://github.com/Expensify/App/assets/93399543/7ee7be40-a415-4d27-92c8-d5df0404ab03
Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1684770585988739
View all open jobs on GitHub

I'm conflicted on this one. @dhanashree-sawant and @kbecciv:
When you tap that link, does it properly route you? If so this may be more of a New Feature than a Bug.

Closing in favor of https://github.com/Expensify/App/issues/19929, I am going to fix both there.

Sure 👍

Noted and added to the GH ^


decompress success but image data is wrong

                int xdim = header[ASTC_HEADER_MAGIC];
                int ydim = header[ASTC_HEADER_MAGIC + 1];
                texture->w = astcGetWidth(header);
                texture->h = astcGetHeight(header);
                dataLen = dataLen - ASTC_HEADER_SIZE;
                const unsigned char * ptrdata = data + ASTC_HEADER_SIZE;
                //         bool decompress(uint8_t* pDst, const uint8_t* data, bool isSRGB, int blockWidth, int blockHeight);
                uint32_t size_rgba = texture->w * texture->h * 4;
                uint8_t* pDst = new uint8_t[size_rgba];
                bool loadSucceed =  basisu::astc::decompress(pDst,ptrdata,false,xdim, ydim );

the image shows error
The display is different every time it is run
How to solve this problem？

Hi - I didn't write this decoder. It's part of Android's testing framework. I suggest you try ARM's decoder in astc-encoder:
https://github.com/ARM-software/astc-encoder
If you can send me the .KTX or .ASTC file, I'll check it out.


The code_dir storage error log is not consistent with size

Organization Name:
microsoft
Short summary about the issue/question:
When User code_dir_size is 0. There will be an error log says that dir is larger than 200MB.
[INFO] User code_dir_size is 0
hdfs://<IP_ADDRESS>:9000/qinsu/qinsu-test2_04320b0b is larger than 200MB, exit.

The related bug location:
https://github.com/Microsoft/pai/blob/0f80208482eade027af88c61b4d58b78bf39b57a/src/rest-server/src/templates/yarnContainerScript.mustache#L262

https://github.com/Microsoft/pai/pull/2388


python311Packages.agate-sql: 0.7.0 -> 0.7.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.agate-sql is: Adds SQL read/write support to agate.
meta.homepage for python311Packages.agate-sql is: https://github.com/wireservice/agate-sql
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.7.2 with grep in /nix/store/jlzqprkfxl2maqsyyd6fz6rr06lx8x8r-python3.11-agate-sql-0.7.2
found 0.7.2 in filename of file in /nix/store/jlzqprkfxl2maqsyyd6fz6rr06lx8x8r-python3.11-agate-sql-0.7.2

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
csvkit
python311Packages.agate-sql

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jlzqprkfxl2maqsyyd6fz6rr06lx8x8r-python3.11-agate-sql-0.7.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/dfc573e79b77fb5ac9c48d8d17b3b232e263272f#python311Packages.agate-sql

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jlzqprkfxl2maqsyyd6fz6rr06lx8x8r-python3.11-agate-sql-0.7.2
ls -la /nix/store/jlzqprkfxl2maqsyyd6fz6rr06lx8x8r-python3.11-agate-sql-0.7.2/bin

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

2 packages built:

csvkit
python311Packages.agate-sql

Maintainer pings
cc @vrthra for testing.

@ofborg eval


Implement Data Loading Step in ETL Data Pipeline

The task has been completed and issues have been resolved.


[RESOLVED] High ICMP ping response time

Problem started at 08:19:29 on 2023.06.26
Problem name: High ICMP ping response time
Host: HP_iLO
Severity: Warning
Operational data: Value: 716ms
Original problem ID: 29893
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23268&eventid=29893

Problem has been resolved in 4m 0s at 08:23:29 on 2023.06.26
Problem name: High ICMP ping response time
Host: HP_iLO
Severity: Warning
Original problem ID: 29893


Resolving binding issues

Closes #264
Related #260

Coverage increased (+0.02%) to 96.594% when pulling 57d703de7d24b901bcb0e191002e1409da95d748 on dkamburov/fix-binding-issues into f5415885c716181b3b2c3f272529b6d94c99f8c0 on master.


Missing exact information about whom at customer side is contacted in case of abuse detected.

In the "Notification and Action" part. Can we specify who at the customer's side will get notified in case of abuse. Is the Subscription owner, or someone else?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 568d17ea-0c67-d7c6-1c20-6e708d772382
Version Independent ID: 4dbb773b-12c2-628f-604b-66a804e89c66
Content: Azure OpenAI Service abuse monitoring - Azure OpenAI
Content Source: articles/ai-services/openai/concepts/abuse-monitoring.md
Service: cognitive-services
Sub-service: openai
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@chente92
Thanks for your feedback! We will investigate and update as appropriate.

@chenge92
Thanks for your Question!!
you can specify who at the customer's side will get notified in case of abuse. By default, the subscription owner will receive notifications about abuse incidents. However, you can configure additional notification recipients by creating an action group in Azure Monitor.
Here are the steps to create an action group for abuse notifications:

Sign in to the Azure portal
Navigate to your OpenAI resource.
Click on the Alerts tab.
Click on the Manage actions button.
Click on the + New action group button.
Enter a name for the action group.
Under Actions, click on the + Add action button.
Select the type of action you want to add (e.g., email, SMS, webhook).
Enter the details for the action (e.g., email address, phone number, URL).
Click on the OK button.
Click on the Create button to create the action group.

Once the action group is created, you can configure your abuse alerts to use the action group for notifications. When an abuse incident is detected, notifications will be sent to the subscription owner and any additional recipients specified in the action group.

@chenge92
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


add fresco progress drawable

Hi, I make some changes to make work https://github.com/facebook/fresco with your library! I try to minimum change your code.
I add ability to set size of the circle because fresco does not allow to set up progress bar size.

I will review the changes along this week, and see if it can be safely incorporated, thank you!

http://frescolib.org/docs/progress-bars.html#_ link to see how library can be used.


Handle Share off for events in OwnTracks app

Description:
Fix exception if share is turned off in OwnTracks app
Trace error message
Related issue (if applicable): #
Raised in gitter
Checklist:
If code communicates with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Pretty sure my code didn't break the failing sunrise / sunset test!


Installation of fhempy peer on Pi Zero W -> VERY Very Slow and breaks sometimes

Hey there,
since the newest update my zero W seems to do not like the fhempy stuff anymore :-)
Is there some heavy change?

It takes ages.
And stopped working totally yesterday - so i reinstalled completely.
*Edit -> still standing on
"Building wheels for collected packages: zeroconf
Building wheel for zeroconf (PEP 517) ... |"
for 21 minutes O_o
System = fresh Raspian from Rapsian Imager.

Looks like the zeroconf wheel package is missing for Pi Zero. If the package is missing, it's getting built which might take some hours. Please test again in a few days, sometimes it takes some time until the packages are provided.

Mhh...

Can you please try
sudo -u fhem bash
cd
source .fhempy/fhempy_venv/bin/activate
pip3 install zeroconf
Please post the output of the commands.

Waited now a day for:
Building wheels for collected packages: zeroconf   Building wheel for zeroconf (PEP 517) ... done
:-D
pi@Zerow:~ $ systemctl status fhempy ● fhempy.service - fhempy      Loaded: loaded (/etc/systemd/system/fhempy.service; enabled; vendor preset: enabled)      Active: active (running) since Sun 2024-02-18 19:59:03 GMT; 44s ago    Main PID: 4708 (fhempy)       Tasks: 4 (limit: 414)         CPU: 35.102s      CGroup: /system.slice/fhempy.service              ├─4708 /home/pi/.fhempy/fhempy_venv/bin/python3 /home/pi/.fhempy/fhempy_venv/bin/fhempy              └─4717 /home/pi/.fhempy/fhempy_venv/bin/python3 -m pip install --quiet construct==2.10.70 --upgrade
So would like to not kill it again yet.
But will check on that idea if needed in the future.

Closed due - worked out after long waiting ;-)

Can you please try
sudo -u fhem bash
cd
source .fhempy/fhempy_venv/bin/activate
pip3 install zeroconf
Please post the output of the commands.

Now I did it ;-)

pi@Zerow:~ $ source .fhempy/fhempy_venv/bin/activate
(fhempy_venv) pi@Zerow:~ $ pip3 install zeroconf
Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple
Requirement already satisfied: zeroconf in ./.fhempy/fhempy_venv/lib/python3.9/site-packages (0.130.0)
Requirement already satisfied: async-timeout>=3.0.0 in ./.fhempy/fhempy_venv/lib/python3.9/site-packages (from zeroconf) (4.0.3)
Requirement already satisfied: ifaddr>=0.1.7 in ./.fhempy/fhempy_venv/lib/python3.9/site-packages (from zeroconf) (0.2.0)


Create relative effects matrix

See

http://beyene-sigma-lab.com/code/analysis_Diabetes.R
http://www.ncbi.nlm.nih.gov/pmc/articles/PMC4277278

###### Report relative effect estimates ###########################################################

# Relative treatment effects (log-odds ratios) of all treatments compared to placebo (treatment # 6)
summary(relative.effect(mtcresults,"6"))

# Odds ratios and corresponding 95% credible intervals of all treatments compared to placebo (treatment # 6)
# To produce the 'gemtc' component of Table 6 of manuscript:
ORandCIs <- round(exp(summary(relative.effect(mtcresults,"6"))$summaries$quantiles[1:5,c("50%", "2.5%", "97.5%")]), 2)
paste(ORandCIs[, 1], "(", ORandCIs[, 2], ", ", ORandCIs[, 3], ")", sep="")

# Forest plot for a specific reference treatment.
# To produce Figure 4 of manuscript:
forest(relative.effect(mtcresults,"6"))

Scratch pad:
###### Report relative effect estimates ###########################################################

# Relative treatment effects (log-odds ratios) of all treatments compared to placebo (treatment # 6)
# summary(relative.effect(mtcresults,"6"))
summary(relative.effect(results, "Warfarin"))
rownames(((summary(relative.effect(results, "Warfarin")))$summaries)$quantiles)

# Odds ratios and corresponding 95% credible intervals of all treatments compared to placebo (treatment # 6)
# To produce the 'gemtc' component of Table 6 of manuscript:
getOR <- function (comparator) {
  require(data.table)
  D <- summary(relative.effect(results, comparator))$summaries$quantiles
  D <- data.table(treatment = rownames(D),
                  comparator = comparator,
                  D)
  D <- D[grep("^sd", treatment, invert=TRUE)]
  D <- D[, x := gsub(sprintf("d.%s.", comparator), "", treatment)]
  D <- D[, c("treatment", "comparator", "50%", "2.5%", "97.5%"), with=FALSE]
  setnames(D, c("50%", "2.5%", "97.5%"), c("median", "boundLower", "boundUpper"))
  D <- D[,
         `:=` (median = exp(median),
               boundLower = exp(boundLower),
               boundUpper = exp(boundUpper))]
  return(D)
}
quantiles <- rbindlist(list(getOR("Apixaban_5_mg"),
                            getOR("Dabigatran_110_mg"),
                            getOR("Dabigatran_150_mg"),
                            getOR("Edoxaban_30_mg"),
                            getOR("Edoxaban_60_mg"),
                            getOR("Rivaroxaban_15_mg"),
                            getOR("Rivaroxaban_20_mg"),
                            getOR("Warfarin")))
quantiles <- quantiles[grep("^sd", rownames(quantiles), invert=TRUE), ]
ORandCIs <- exp(quantiles[, c("50%", "2.5%", "97.5%")])
ORandCIs
paste(ORandCIs[, 1], "(", ORandCIs[, 2], ", ", ORandCIs[, 3], ")", sep="")

# Forest plot for a specific reference treatment.
# To produce Figure 4 of manuscript:
forest(relative.effect(mtcresults,"6"))

See commit a6c7bda19471ba243ce822ffa362f03157506474


matrix-synapse: fix startup

Motivation for this change
Currently, setuptools isn't propagated automatically to python
packages[1] which causes the following error when starting
matrix-synapse:
Traceback (most recent call last):
   File "/nix/store/xxkds7821mrahfx75az0sq3ryf69m612-matrix-synapse-1.3.1/bin/.homeserver-wrapped", line 39, in <module>
     import synapse.config.logger
   File "/nix/store/xxkds7821mrahfx75az0sq3ryf69m612-matrix-synapse-1.3.1/lib/python3.7/site-packages/synapse/config/logger.py", line 27, in <module>
     from synapse.app import _base as appbase
   File "/nix/store/xxkds7821mrahfx75az0sq3ryf69m612-matrix-synapse-1.3.1/lib/python3.7/site-packages/synapse/app/__init__.py", line 18, in <module>
     from synapse import python_dependencies  # noqa:E402
   File "/nix/store/xxkds7821mrahfx75az0sq3ryf69m612-matrix-synapse-1.3.1/lib/python3.7/site-packages/synapse/python_dependencies.py", line 19, in <module>
     from pkg_resources import (
 No module named 'pkg_resources'

[1] https://github.com/NixOS/nixpkgs/pull/68314
Things done

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

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

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

backported in 8050566cea3b6ffce82810f86e8f8b0b3a402ad6


Using recv in js inside an eval on desktop blocks all js if it doesnt have any data to fetch

Problem
Currently when you do eval("await dioxus.recv();") on desktop and dont send a message it blocks all js from running, including devtool console and events.
Steps To Reproduce
Heres the code i used to test it
#![allow(non_snake_case)]

use dioxus::{desktop::use_window, prelude::*};
use log::LevelFilter;

fn main() {
    // Init debug
    dioxus_logger::init(LevelFilter::Info).expect("failed to init logger");

    dioxus::launch(App);
}

#[component]
fn App() -> Element {
    // Build cool things ✌
    let windows = use_window();
    use_hook(|| windows.devtool());
    let mut signal = use_signal(|| false);
    use_effect(move || {
        if signal() {
            eval(
                r#"
                    window.alert("1");
                    await dioxus.recv();
                    window.alert("2");
                "#,
            );
        }
    });

    let onclick = move |_| signal.with_mut(|v| *v = !*v);
    rsx! {
        button {
            onclick,
            "locks js"
        }
        {format!("{}", signal())}
    }
}

Environment:

Dioxus version: 0.5.1
Rust version: 1.76.0
OS info: MacOS
App platform: desktop

Questionnaire

[ ] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

as far as i understand it the offending code is the infinite loop in the timeout here in query.rs
recv: function () {
        return new Promise((resolve, _reject) => {
            // Ever 50 ms check for new data
            let timeout = setTimeout(() => {
                let __msg = null;
                while (true) {
                    let __data = _message_queue.shift();
                    if (__data) {
                        __msg = __data;
                        break;
                    }
                }
                clearTimeout(timeout);
                resolve(__msg);
            }, 50);
        });
    },

the above timeout should likely be changed to an interval without the inner loop instead

not directly related but it should probably be cleaned up when dropping UseEval as well


In the training set, the content of the picture is a house far away from the photographer, so will it cause a big error to use a close-range picture of the house to predict in test time?

The CosPlace model automatically learns to pay more attention to nearby static objects, while paying less attention to far away objects (e.g. a tall building in the distance) or moving objects (cars and pedestrians).
This is true also for previous methods, such as NetVLAD.

Thank you for your answer.But how to understand this sentence  【The CosPlace model automatically learns to pay more attention to nearby static objects, while paying less attention to far away objects 】 in terms of network model structure or mathematics?

I don't really understand what you mean. All I want to say is that the model's descriptors depend mostly on things that can help to understand the camera position, which usually are buildings not too far away.

How does this model do this 【The CosPlace model automatically learns to pay more attention to nearby static objects, while paying less attention to far away objects 】？

That's what the loss is teaching the model. To group together (close in features space) images with the same orientation within a 10x10 meter cell, and keep their features far from other images (belonging to other cells), the only way is to focus on nearby static objects

Wow, I finally understand.The features of the objects close to each other have strong correlation。Does that mean that when we make training data sets, these images should be close-range and static  photos?

When we collect datasets, the images will contain some close-range static objects for sure. It is very rare for a photo taken outdoor to have absolutely no nearby static objects.

Think you for your help


Cut release

:tada: This PR is included in version 5.0.0-rc.20 :tada:
The release is available on:

npm package (@rc dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


After yarn release minor version doesn't increase

Hello!
Im trying to increase my changelog with my conventional commits.
Configured from: https://commitlint.js.org/#/guides-local-setup?id=install-commitlint
Always increase last numbers 0.1.XX
I want to get X.X.XX
Environment

npx create-next-app@latest --typescript
commit-and-tag-version 10.0.1
@commitlint/cli
@commitlint/config-conventional
husky

The commit message after yarn release is always the same: chore release and 0.1.xx... It doesn't matter if you commit a feat or breaking change.

Hello!
Im trying to increase my changelog with my conventional commits.
Configured from: https://commitlint.js.org/#/guides-local-setup?id=install-commitlint
Always increase last numbers 0.1.XX
I want to get X.X.XX
Environment

npx create-next-app@latest --typescript
commit-and-tag-version 10.0.1
@commitlint/cli
@commitlint/config-conventional
husky

The commit message after yarn release is always the same: chore release and 0.1.xx... It doesn't matter if you commit a feat or breaking change.

WELL, if it starts from 1.0.0 works. But the commit message keep printing chore(release): 1.1.1

Well, if it starts from 1.0.0 works! But the commit message keep printing 'chore(release): 1.1.1'

Below version 1, semver has no definition for what the version number means. I can't see any breaking changes in the screenshot you linked - but I think the behaviour is for the minor version to increase on breaking changes, and for patch versions to increase for everything else.
From version 1.0.0 onwards, semver should apply and you should see the version number increasing correctly.

But the commit message keep printing 'chore(release): 1.1.1'

I'm not sure what you mean by this - that looks right to me. Can you give more details on your setup and what you're expecting?


Added Travis_retry to db setup and test

Coverage remained the same at 74.251% when pulling 4f47277fa6686c062659da801ca1b049b0c595c2 on Hadis-Fard:Travis_retry into d8515c388d511c02a91432e686974704949d1e49 on Microsoft:dev.


Import ignores new templates

Hi,
I'm trying to sync between 2 projects. The source projects has new page & link templates.
The import will ignore the new templates.  I have to manually import them to FirstSpirit, is there any way to tell the Importer to create the new templates during an import?

Hi Frank,
no, there's no such thing that you want. I think the official workflow would be to add the templates in your firstspirit project and export it first, like I think you already did. Further information about the supported development workflows can be found in the official product documentation for "External
Synchronization".
Regards,
Hannes


In
flo-freezer/src/test/java/com/spotify/flo/freezer/PersistingContextTest.java
https://github.com/spotify/flo/pull/90#discussion_r195025904:

final Exception exception1 = new Exception("foo", bar);

exception1.addSuppressed(new FoobarException());
final Exception exception = exception1;

final Path exceptionFile = temporaryFolder.getRoot().toPath().resolve("exception");
PersistingContext.serialize(exception, exceptionFile);

final Exception deserialized = PersistingContext.deserialize(exceptionFile);

// Verify that stack trace can be accessed and printed and seems to be correct

deserialized.printStackTrace();

final StringWriter sw = new StringWriter();
final PrintWriter pw = new PrintWriter(sw);
deserialized.printStackTrace(pw);

I mean we do this anyway here so why printing to stdout at line 55?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spotify/flo/pull/90#pullrequestreview-128308524, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYAScPEs6vHsJ95QiRE60M2rme3YcI6ks5t8ORHgaJpZM4UlmAS
.

👍


tt-rss efficient update of new feed contents

Newsboat version (copy the output of newsboat -v or the first line of git show): 2.21.0
Config file (copy from ~/.newsboat/config or ~/.config/newsboat/config):
urls-source ttrss
ttrss-login "me"
ttrss-password "mypass"
ttrss-url "https://my-server.org/tt-rss/"

Steps to reproduce the issue:

Log in, update everything (slow)

Wonder what's new.  newsboat laboriously polls everything all over again

Other info you think is relevant:
The tt-rss API has a mode for getHeadlines which can just tell you what's new.  For people with a large number of feeds, this would make checking for new contents many, many times faster.

I guess you're referring to the since parameter. I don't think we can use that; please see https://github.com/newsboat/newsboat/issues/16#issuecomment-341112285 for background info.
If that's not what you meant, or I'm mistaken and you think we can speed it up with some better queries, please tell so! I don't promise to work on it myself, but if we document the idea, someone else could pick it up.

Thank you for that pointer (I searched before submitting this, but didn't spot that!).  tt-rss is clearly a cloud service, and I see now that Newsboat sees itself as a host-centric application.  So it's OK to mark something read and never tell tt-rss, because the app is the point.  Two unintended consequences come to mind--the need to bulk refresh to re-sync with a cloud service of unknown coherence.  But also if you read on one device--say, Newsboat--and then on tt-rss's native web interface, or some other client like FeedMonkey?  Newsboat's private state becomes apparent.  Most likely you'll see dups, perhaps lots of them.
I'm interested in the problem.  But if you, as the maintainer, think this moves your app in a direction which demands too much effort?  Or is in general not a priority for now?  Then no harm, no foul.  Thanks for the response!


Refactor the whole thing

Key events
Make a layer on top of whole editor that will be used for getting focus. By default none editor should be active, it should become active only when a command is called, eg. $('id').notepad('focus'). That can be triggered when click happens on that on-the-top layer (and after that it should dissapear).
Also, when many notepad instances on the site then only one can be active. That means when one gets activated the others get inactive.
Key events are attached to document.body so there is no need to handle these events more than once. That will require some variables inside jsNotepad to mark if keys were already loaded.
Id of initalized instance should also be added as we'll need them to disactivate.
As a first step, let's add ctrl, alt, shift and arrow keys. We need to properly test that with leaving the tab/window and going back. When user leaves the page, all jsNotepad instance should be disactivated. Once they're back they should activate it by clicking the top layer (or that should be handled outside of jsNotepad).

Commands
Everything that happens with the text, clipboard and cursor should be happening through command. It might be nice to execute via $('id').notepad('command').


Release/13.0.x

Any chance https://github.com/input-output-hk/cardano-db-sync/pull/1210 can make it to release so we are not patching ;-)  It's already been tested by Erik and myself :)


python38Packages.aiohue: 2.6.3 -> 3.0.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.aiohue is: "Python package to talk to Philips Hue"
meta.homepage for python38Packages.aiohue is: "https://github.com/home-assistant-libs/aiohue"
meta.changelog for python38Packages.aiohue is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.0.1 with grep in /nix/store/i5qiq1p8cz2726npvhq24k90yhr6nqif-python3.8-aiohue-3.0.1

found 3.0.1 in filename of file in /nix/store/i5qiq1p8cz2726npvhq24k90yhr6nqif-python3.8-aiohue-3.0.1

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

5 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
home-assistant
python38Packages.aiohue
python39Packages.aiohue

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/i5qiq1p8cz2726npvhq24k90yhr6nqif-python3.8-aiohue-3.0.1
ls -la /nix/store/i5qiq1p8cz2726npvhq24k90yhr6nqif-python3.8-aiohue-3.0.1/bin

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

1 package failed to build:

home-assistant

2 packages built:

python38Packages.aiohue
python39Packages.aiohue

Maintainer pings
cc @fabaff for testing.

This breaks home-assistant:
       > ImportError while loading conftest '/build/source/tests/components/hue/conftest.py'.
       > tests/components/hue/conftest.py:6: in <module>
       >     from aiohue.groups import Groups
       > E   ModuleNotFoundError: No module named 'aiohue.groups'

closing, as stale


chore: Clean up markdown codeblock formatting

Description
Cleans up markdown fenced code blocks by using yaml syntax highlighting.

Current dependencies on/for this PR:

main

PR #36   👈

This comment was auto-generated by Graphite.


Syntax Highlighting

Can you please add support for syntax highlighting using highlight.js in code blocks?

You can do this by writing a custom codeBlock output rule, like:
const SimpleMarkdown = require('simple-markdown');
const hljs = require('highlight.js');

// Make a rules object that is a modified version of the default rules:
const rules = Object.assign({}, SimpleMarkdown.defaultRules, {
    codeBlock: {
        // Assuming you're using html rather than react, here's a custom
        // html outputter for highlighted code blocks:
        html: function(node, output, state) {
            // First we call highlightjs with our codeblock's language (node.lang)
            // and our codeblock's code (node.content):
            let hljsResult = hljs.highlight(node.lang, node.content, true);
            let highlightedCode = hljsResult.value;

            // Then we return a div wrapping highlightjs's formatted result.
            // Feel free to customize the wrapping element to be whatever you need:
            return (
                '
' +
                    highlightedCode +
                '
'
            );
        },
    },
});

// Then turn that rules object into parse, output, and markdownToHtml functions:
let parse = SimpleMarkdown.parserFor(rules);
let highlightingHtmlOutput = SimpleMarkdown.outputFor(rules, 'html');
let markdownToHighlightedHtml = function(markdown) {
    return highlightingHtmlOutput(parse(markdown));
};

// And optionally export those via module.exports or export {};
module.exports = {
    parse: parse,
    highlightingHtmlOutput: highlightingHtmlOutput,
    markdownToHighlightedHtml: markdownToHighlightedHtml,
};

I don't generally add features like this directly to the library itself, because everyone has different needs, but instead try to support easy extensibility like the above.
Would something like that work for you?

// Assuming you're using html rather than react

I'm using react... Would something like this work? (I'm unable to test it now)
codeBlock: {
    html: function(node, output, state) {
        const hljsResult = hljs.highlight(node.lang, node.content, true);
        const highlightedCode = hljsResult.value;

        return SimpleMarkdown.reactElement(
            'div',
            state.key,
            {
                className: `highlighted-code ${SimpleMarkdown.sanitizeText(node.lang)}`,
                children: highlightedCode
            }
        );
    },
},

Cool! I think to get your example to work, you'd want:
codeBlock: {
    // need 'react' key here instead of html:
    react: function(node, output, state) {
        const hljsResult = hljs.highlight(node.lang, node.content, true);
        const highlightedCode = hljsResult.value;

        return React.createElement(
            'div',
            {
                key: state.key,
                className: `highlighted-code ${SimpleMarkdown.sanitizeText(node.lang)}`,

                // and we need dangerouslySetInnerHTML here, since highlightedCode is a string:
                // https://reactjs.org/docs/dom-elements.html#dangerouslysetinnerhtml :
                dangerouslySetInnerHTML: highlightedCode,
            }
        );
    },
},

But you could probably use something like react-highlight.js to do this more easily:
import SimpleMarkdown from 'simple-markdown';
import Highlight from 'react-highlight.js';

// Make a rules object that is a modified version of the default rules:
const rules = Object.assign({}, SimpleMarkdown.defaultRules, {
    codeBlock: {
        react: function(node, output, state) {
            return (
                <Highlight language={node.lang}>
                    {node.content}
                </Highlight>
            );
        },
    },
});

(I haven't tested either of these either yet)
And either of these would need outputFor(rules, 'react') instead of outputFor(rules, 'html')

Interesting :thinking:... I'll give react-highlight.js a try. Thanks!

 Let me know if you have trouble with it

This doesn't seem to work due to inlineCode.

I forgot to mention, I'm using three backticks for code blocks.

Test link: https://regex101.com/r/ki1o3D/1

match: blockRegex(/^```(\w*)\n+([\s\S]+)\n+```$/),

I forgot to mention, I'm using three backticks for code blocks.

Test link: https://regex101.com/r/ki1o3D/1

match: blockRegex(/^```(\w*)\n+([\s\S]+)\n+```$/),

Looks like changing it to not use blockRegex works.
match: function (source) {
  return /^```(\w*)\n+([\s\S]+)\n+```$/.exec(source);
},


+WheelFalling, LastTurboLevel, ReactorFinalTimer

I found some more values inside of CSceneVehicleVisState that are not exposed by default, at least for TM2020.

For each wheel, I also found an EPlugSurfaceGameplayId type (special effect touching the wheel), but getting these values in the same manner is slightly trickier as the offsets are not a multiple of 4 (only +1 from FLGroundContactMaterial etc.)


new adapter ioBroker.minuaru

Result of Adapter Checker:

[E506] "admin/jsonCustom.json" not found, but custom support is declared
[E300] Not found on travis. Please setup travis

both false positive:

jsonCustom.json is there
please add travis.com to checker together with travis.org

RE-CHECK!

Automated adapter checker
No changed adapters found
Add comment "RE-CHECK!" to start check anew

Hi @Apollon77
here some review comments:
Please add adapter testing also to the travis job (or better go to Github Actions?) - right now only package testing is done
yes, is planned before adding to stable. Can you give me some example of testing admin/custom.json ?
Is the adapter type storage really making sense
for me it is the same like "ioBroker.history", storing and displaying alarm-messages
What other type would you suggest ???
Please remove admin from the "dependencies" list , minimum js-controller 3.3 as dependency
done
data-directory ( now .../iobroker-data/minuaru.0) changed, please check if config
https://github.com/minukodu/ioBroker.minuaru/blob/bde7c341d48c60561fdce6d27f10b785feae7ed5/io-package.json#L127
is correct for backup
Thank you for this good support

@SVallant
1.) Testing custom ... in fact you can create an object with the relevant custom entries before adapter get started and such. In fact we have very detailed testing for history and such (eg. https://github.com/ioBroker/ioBroker.history/blob/master/test/testAdapter.js#L100-L116), but this uses the "legacy testing scripts" and not the new testing lib. But basically "the same" is possible there too...
2.) I would see it more "alarm" or "misc-data" dependeing on what you want to focus on. I think the main focus is more about "doing something based on actions" and not "dump storage of data"
3.) Backup/Data Dir: seems fine.. maybe test it :-) you can check that it restored on "restore" or that it is contained in the backup file

Additionally for testing: even if you do not add real custom testing it is better then nothing to just proof that install and start basically works. But yes you can also go deep into functional testing ;-)

adapter's type is "misc-data" now, since alarm-adpters or mostly "security"-Adapters
Backup works like you said
wirting tests will be a big challenge next weeks ... ;-)

For tests: As said: For the repo addition it would be enought to just see adater starting basically (so execute the integration tests - done). More is optinal after that

@SVallant Anything we can support with? Just poke when ready for re-review.

No, thanks, just no available time at the moment .... other projects are more important

@SVallant if it is only about the testing thenthis would just be requirement that "it runs and starts the adapter", custom testing is formally not required

@Apollon77 Adapter is ready for latest now. Do you a new PR?

resolve the conlfucts or do a new PR (but please add a linkto here) ... up to you :-)


Expose postcss config

Maybe expose postcss config. TDB.

Duplicate of https://github.com/bcomnes/siteup/issues/38


Restore local artifact server.

This partially reverts #5370.
Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.

Waiting on BeamFilesystemArtifactService instead.


Write output to buffer instead of file

Instead of writing the generated docx contents to a temporary file,
we now just create a string buffer with the file content.
This makes it easier to use the gem on a system where the application might not
have write access to the filesystem. It also ensures that the gem doesn't leave
behind a lot of temporary files, since there was no system for cleaning them up
after use.
This is also the recommended way of using Rubyzip for modifying docx files:
https://github.com/rubyzip/rubyzip#modify-docx-file-with-rubyzip

@anitsirc check this out

@tobiashm Next time, make the buffering part as an optional parameter. This change might affect others using the gem that expects it to actually generate a file and not just a buffered string.


Get file Size before downloaded it

Hi everybody,
my plugin version is
"react": "16.0.0",
"react-native": "0.51.0",
"react-native-fetch-blob": "^0.10.8"
I want to get File Size before downloaded it but I don't have any Idea  about this
Please help me in this subject.

Do you find a way to get the size of the file now?

No, I can't find anyway and I am waiting and searching :|


Another neo4j 3.0 compatibility patch

I believe more will follow as they surface so eventually we'll get them all covered.

and yes it's again relted to #108


Add wget fallback and option

Add fallback to wget, Add option to force use of wget and make sure we get json object and not the webpage.
Example openwrt does not ship with curl, but it ships with wget.

@mariogrip how about removing the force wget option to keep the options simple, or do you see a need for this option?


Remove confirmation on "infrakit manager change -c .."

Currently, the command requires verification via stdin; the presence of the "-c" argument is sufficient to commit the change.
Signed-off-by: Steven Kaufer<EMAIL_ADDRESS>

An example of the output without the commit arg:
# infrakit manager change -v Groups/workers/Properties/Properties/Allocation/Size=3
Proposed changes, hash=c60105499b21d9783e23add7de04d4c4
Groups:
  workers:
    Plugin: group-stateless
    Properties:
      ID: workers
      Properties:
        Allocation:
          Size: 13
        Flavor:
          Plugin: flavor-swarm/worker
...
#

An example of the output with the commit arg:
# infrakit manager change -v Groups/workers/Properties/Properties/Allocation/Size=3 -c
Committing changes, hash=c60105499b21d9783e23add7de04d4c4
Groups:
  workers:
    Plugin: group-stateless
    Properties:
      ID: workers
      Properties:
        Allocation:
          Size: 13
        Flavor:
          Plugin: flavor-swarm/worker
...
#

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "mgr-commit-verification" git@github.com:kaufers/infrakit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354030360
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.


username does not get filled into the template on unix environments

From<EMAIL_ADDRESS>on 2011-11-09T04:06:01Z
What steps will reproduce the problem?

create a new file in editor (from template) 2. 3. What is the expected output? What do you see instead? the username does not get addded on osx (and probably any other unix).

I hava a patch that fixes this as well as allow access to all environment variables:
diff -r 94d4028b995d spyderlib/plugins/editor.py
--- a/spyderlib/plugins/editor.py	Tue Nov 08 21:02:47 2011 +0100
+++ b/spyderlib/plugins/editor.py	Wed Nov 09 10:04:24 2011 +0100
@@ -1335,8 +1335,17 @@
if encoding_match:
enc = encoding_match.group(1)
try:

       text = text % {'date': time.ctime(),

                      'username': os.environ.get('USERNAME', '-')}

       #make username work on linux as well

       if sys.platform == 'darwin' or \

           sys.platform == 'linux':

           username = os.environ.get('USER','-')

       elif sys.platform == 'win32':

           username = os.environ.get('USERNAME', '-')

       #make the entire os.environ useable

       replacements = {'date': time.ctime(),

                      'username': username}

       replacements.update(os.environ)

       text = text % replacements
   except:
       pass
   create_fname = lambda n: unicode(_("untitled")) + ("%d.py" % n)

Original issue: http://code.google.com/p/spyderlib/issues/detail?id=830

From<EMAIL_ADDRESS>on 2011-11-10T03:47:58Z
This issue was updated by revision f61a1a3b563e .
I didn't copy all environment variables into template vars, because it's not cross-platform and will lead to KeyError exceptions on different systems. A different bug report may be a better place to discuss this feature separately.
Status: Fixed

From<EMAIL_ADDRESS>on 2011-11-09T16:17:06Z
Ubuntu/Fedora: both USERNAME and USER
Ubuntu/Debian/Fedora through SSH/Shell: USER only
I guess Spyder doesn't run from shell, so it's save to leave USERNAME for linux (although I'd like to see some docs to reference to).

From<EMAIL_ADDRESS>on 2011-11-10T02:45:46Z
OK, maybe linux is not affected (although I could imagine running it from a shell) but OS X definitely is.
Labels: OpSys-OSX

From<EMAIL_ADDRESS>on 2011-11-10T03:49:29Z
Thanks for the patch, BTW. )

From<EMAIL_ADDRESS>on 2011-11-16T08:45:25Z
You are welcome ;-).
Since USER is ususally not quite as 'human readable' as USERNAME, I have meanwhile figured out a way to get something similar to the USERNAME on OS X. The patch builds on my original modifications but I am sure it is easy to get this into the current version:
--- a/spyderlib/plugins/editor.py	Wed Nov 09 09:33:10 2011 +0100
+++ b/spyderlib/plugins/editor.py	Wed Nov 16 17:38:38 2011 +0100
@@ -1336,9 +1336,12 @@
enc = encoding_match.group(1)
try:
#make username work on linux as well

       if sys.platform == 'darwin' or \

           sys.platform == 'linux':

           username = os.environ.get('USER','-')

       if sys.platform == 'darwin':

+#                username = os.environ.get('USER','-')

           import os

           f = os.popen("dscl . -read /Users/%s RealName"%os.environ['USER'])

           s = f.read()

           username = s.splitlines()[1].strip()
       elif sys.platform == 'win32':
           username = os.environ.get('USERNAME', '-')
       \#make the entire os.environ useable

Cheers.

From<EMAIL_ADDRESS>on 2011-11-17T14:02:48Z
It looks like there is also a way to extract full name on Windows [1] and Linux [2]. I think it is better to leave login name as-is and provide full name as a separate variable. It would be easier to track this feature request in a different issue from this one.

http://msdn.microsoft.com/en-us/library/ms724435(VS.85).aspx 2. http://stackoverflow.com/questions/833227/whats-the-easiest-way-to-get-a-users-full-name-on-a-linux-posix-system


Question: Generating a list of Standard Library global functions

How can I use SwiftDoc to generate a list of global functions in the Swift Standard Library, similar to what's on https://swiftdoc.org?
I'd like to use this list to update https://github.com/highlightjs/highlight.js/blob/master/src/languages/swift.js
Thanks!

@svanimpe Apologies for not responding sooner. One idea for getting a list of symbols would be to open a Swift project in Xcode, command-click on a standard library type, and copy the generated interface into a new file. You could then run something like DeclarationCollector on the interface to get a list of symbols.

Alternatively, you could try this approach: https://forums.swift.org/t/is-there-a-way-to-dump-the-list-of-all-standard-library-symbols-on-linux/48571


iOS port: remove Java Runtime specifics

This PR is opened in order to create a branch the work on graphhopper-ios https://github.com/graphhopper/graphhopper-ios/pull/47  can depend on for the time being. Please do not close the PR right now.

My intention was that this Pull request will create a branch on graphhopper this way.... At least I didn't found it ...

I'll create a new "ios_compatibility" branch (or something) and integrate your changes.

Ok, once we merge this the test should be green.
If you like we can additionally add you as a contributor. We would do this on master branch and for that we would also need our CLA signed (just to confirm the changes are under Apache License): see this document. Let me know your email (you can contact us here)


A question about how to judge the performance of the prompts after running "run_fsc.py" file?

Dear Author:
I am very interested in your research in article RLPrompt: Optimizing Discrete Text Prompts With Reinforcement Learning , after carefully learning your code, I have a question and would like to ask you for it.
After running the "run_fsc.py", a lot of json files containing 'output_tokens' and 'scores' are obtained. I would like to ask you about how to judge the performance of prompts? Should I run "run_eval. py" once for each prompt in all  json files, and see the performance of the prompt from the Acc, or can I simply think that the output_tokens with high scores has better performance?
Wishing for your replying!

Thanks for reaching out. When you first ran the script, were you prompted to set up your wandb account? Our code logs the training progress, e.g., accuracy, through this website by default.
If you are more comfortable with other trackers like TensorBoard, you can modify the code in the Trainer class to log to your preferred tracker.
If you don't want to use any tracker at all, you can also simply configure the PromptedClassificationReward class to directly print any quantity you want, but it'll be more difficult to visualize and spot trends.
Let me know if this makes snese. I'm closing this issue because it's a clarification question.

That is also an interesting question! If you want to pick some prompts, it is okay for you to pick prompts which show decent performance on 16-shot dev set and 16-shot training set. Usually, I prefer to picking prompts which show some high reward values on training set, and the reward values are much more prominent than simple accuracy scores, since accuracy scores can be biased towards few-shot examples.

Yes, I agree with you a lot and I will try to follow your suggestion to select the prompts, then use these prompts on test set to see the performance of the these prompts.


Google Cloud - PIA Update for CAS

Describe the issue
Work with program area to complete draft of PIA Update.
Which Sprint Priority is this issue related to?
Google CAS POC Milestone
Additional context
None
Definition of done
Draft of PIA complete and ready for approval route or fully signed (ideally).

Connected with ENV Ministry Privacy Officer and determined that no update to the original PIA is needed. Nothing is changing to the project/program itself and no personal information is being introduced.


CMake: Add travis and update lib file

Added .travis.yml config to pick this example for Travis CI build
Update mbed-os.lib to pick always latest master

@hugueskamba @0xc0170

@paul-szczepanek-arm Could you review this PR


WebAssembly

Emscripten has WebAssembly backend. Have you tried what is the performance?
https://github.com/kripken/emscripten/wiki/WebAssembly

Yes please !

FWIW Webassembly should not bring us much: it's just a new bytecode format.
I expect the main improvement will be in page load time. However it comes at the huge cost of browser compatibility.
I'll give it a try at some point, but for now I don't even have a browser installed that supports it. If someone else wants to do a comparison feel free.

Well bypassing js is bound to be faster no ?

Well bypassing js is bound to be faster no ?

asm.js (which lua.vm.js uses) is already specially processed by browsers.

For test purposes I did build to WebAssembly.
 The js object is not populated yet.
But it works if anyone is interested.

@giann It looks like WebAssembly is now stable, and it would be awesome to get formal support here. I'm hoping to build a cross-platform game using Lua as a scripting language, and being able to write the bulk of my logic (everything except rendering, that is) in WebAssembly calls would be awesome.
Have you tried doing this recently? I think WebAssembly was stabilized earlier this week, so I'm guessing not yet. If you don't have a chance, I may play with it this weekend. I wonder how it'll compare to native JavaScript.
As for this project, is this the proper place to propose merging patches to make it work with wasm?

being able to write the bulk of my logic (everything except rendering, that is) in WebAssembly calls would be awesome.

Why? (I'm not sure what benefit this brings)

I may play with it this weekend. I wonder how it'll compare to native JavaScript.

Let me know how you go.

As for this project, is this the proper place to propose merging patches to make it work with wasm?

Yes.

PS, @giann started the project https://github.com/giann/fengari at my suggestion after discussing this thread elsewhere.

Webassembly should not bring us much

Have you seen the WebAssembly GC/DOM integration page?

Webassembly should not bring us much
Have you seen the WebAssembly GC/DOM integration page?

Looks like they've added to it a bit since I saw it last; but it's still all just unconfirmed plans with no implementations.This is in addition to the fact that these new features won't ever work in legacy (or current) browsers.
In 3 years when the code exists we can start work on an implementation. In 7-8 years there should be enough browser penetration that we might be able to use it for non-demo purposes.


DNM

Avoid ghcr.io (issue 622)

same issue
  error: failed to solve: public.ecr.aws/docker/library/nginx:1.19-alpine: failed to do request: Head "https://public.ecr.aws/v2/docker/library/nginx/manifests/1.19-alpine": dial tcp: lookup public.ecr.aws on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:57540-><IP_ADDRESS>:53: i/o timeout
        time="2022-01-25T09:43:55Z" level=fatal msg="unrecognized image format"
        time="2022-01-25T09:43:55Z" level=fatal msg="error while building image nerdctl-compose-test672377653_web: exit status 1"

https://cirrus-ci.com/task/5419024245850112?logs=rootless_test#L224


[run] improve UX by not requiring '--'

Summary
Inspired by @jimmidyson's awesome post I thought we could remove the need for -- every time you want to run something with devbox that has flags.
This change assumes that when a user runs devbox run the first non-flag arg after "run" is part of the command or the script name the user wants to run. This relies on run never having subcommands (which I think will be true forever). So we transform:
devbox run python --version
into
devbox run -- python --version
If command already has -- then we do nothing.
This change is fully backward compatible.
@LucilleH I'm somewhat sure I fixed the issues from demo. PTAL
How was it tested?
devbox add python
devbox run python --version
Python 3.11.4
devbox run -c examples/development/python/pip python --version

Just FYI the comment re devbox run in my big post wasn't about the need for --, I actually like that too demarcate the command, but about needing devbox run at all - I couldn't figure out a way to massage the environment so PATH, etc was configured properly to use the devbox installed tools.

thanks for the clarification @jimmidyson!
We chatted about that as well and I think @LucilleH has some ideas to make the action better.
Not sure if it helps your particular use case, but if you do:
eval "$(devbox shellenv)" in a directory with a devbox.json it's the equivalent of turning the current shell into the devbox environment. I think in Github actions the environment is not persisted between steps so not sure if this is helpful, we may need to make a chance to the action itself.


Does not work in remote SSH mode?

Describe the bug
I am using VSCode in "Remote SSH" mode and have installed this extension in remote mode as well. Pressing Ctrl+Alt+; does not seem to work in this case. It works fine in a local mode (when VSCode and extensions are working with local files), so it's not an issue of the keyboard binding misconfiguration.
To Reproduce
Steps to reproduce the behavior:

Install https://marketplace.visualstudio.com/items?itemName=ms-vscode-remote.remote-ssh
Set up remote Linux machine following the instructions from #1 (it installs VSCode server under the hood)
Open a repository in SSH mode.
Install SmartOpen extension in remote VSCode as well (relaunch VSCode after)
Open any file in the explorer, scroll away from it and press Ctrl+Alt+; - the explorer window is focused but not scrolled to the opened file

Expected behavior
Similar to the local behavior - pressing Ctrl+Alt+; opens and scrolls Explorer to the currently opened file.
Screenshots
N/A
Desktop (please complete the following information):

OS: Windows VSCode client, Linux VSCode remote
Version - VSCode 1.43.2, SmartOpen 1.6.4

Hi, thanks for submitting the issue report.
Unfortunately, I couldn't make the SSH working with VSCode on my machine. I thinks there is a way around if you simply want use the shortcut. Check this if it is work for you.
The shortcut calling one of the existing command in the vscode, you can try following step to directly setup "keyboard shortcut".

In vscode, open Keyboard Shortcuts
Search for workbench.files.action.showActiveFileInExplorer
Set a shortcut. Try if it works for the SSH.

If it is still not working, then I couldn't do anything because it is not a customizable thing. Internally, the Ctrl+Alt+; calls this command

I don't know what changed, but suddenly today the shortcuts started working even in remote mode. Apologies for the churn!

Maybe other extension updated. I see the remote SSH is still in beta, maybe some issue there. Enjoy your coding👍


added link to codemonk

codemonk is an hackerearth initiation that provides CP questions topic wise covering every data structure and algorithms

Thanks for adding this!
Take some more time and see if you can find a couple more interesting resources to add here. Then we'll merge them all under this PR.

for sure sir !

@himanshusanecha do you have more resources to add to this repo

no sir all the other resources are already present if i would find some links to free courses i would add them as soon as possible
Thank you

Cool! Merging this then.


GetPositionAsync() kept throwing exception on iOS

This is not an issue in Android
Bug Information
Version Number of Plugin: <IP_ADDRESS>-beta
Device Tested On: iPhone 8 plus 256 gb
Simulator Tested On:
Version of VS: 2015
Version of Xamarin: <IP_ADDRESS>0
Versions of other things you are using: Plugin.Permision 2.1.0
Steps to reproduce the Behavior - Code snippets
async Task LocationPermissionCheck() {
    var permissionStatus = await CrossPermissions.Current.CheckPermissionStatusAsync(Permission.Location);
    if (permissionStatus != PermissionStatus.Granted) {
        var reqPerStatus = await CrossPermissions.Current.RequestPermissionsAsync(Permission.Location);
        if (reqPerStatus[Permission.Location] != PermissionStatus.Granted) {
            await Acr.UserDialogs.UserDialogs.Instance.AlertAsync("Location permission is required....");
            CrossPermissions.Current.OpenAppSettings();
            return;
        } else {
            StartPositionTimer();
        }
    }
    else {
        StartPositionTimer();
    }
}

async Task StartPositionTimer() {
    if (await IsLocationAvailable()) {
        while (true) {
            await CheckPosition();
            await Task.Delay(System.TimeSpan.FromMinutes(3));
        }
    } else {
        Acr.UserDialogs.UserDialogs.Instance.Alert("GeoLocation is not available or enabled. Please enable location access and try again.");
        return;
    }
}

async Task CheckPosition() {
    Plugin.Geolocator.Abstractions.Position position = null;
    try {
        var locator = CrossGeolocator.Current;
        locator.DesiredAccuracy = 100;

        var temp = await locator.GetPositionAsync(TimeSpan.FromSeconds(30), null, true);
        position = temp;

    } catch (Exception ex) {
        App.CurrentLatitude = "";
        App.CurrentLongitude = "";
        Xamarin.Insights.Report(ex);
        return;
    }

    App.CurrentLatitude = position.Latitude.ToString();
    App.CurrentLongitude = position.Longitude.ToString();
}

Expected Behavior
var temp = await locator.GetPositionAsync(TimeSpan.FromSeconds(30), null, true);

is supposed to run at least 30s before being timed out but an exception is thrown immediately there saying:
On iOS 8.0 and higher you must set either NSLocationWhenInUseUsageDescription or NSLocationAlwaysUsageDescription in your Info.plist file to enable Authorization Requests for Location updates!

Actual Behavior
An exception is thrown immediately there saying:
On iOS 8.0 and higher you must set either NSLocationWhenInUseUsageDescription or NSLocationAlwaysUsageDescription in your Info.plist file to enable Authorization Requests for Location updates!

I have all the permissions needed in info.plist
<key>NSLocationWhenInUseUsageDescription</key>
<string>Location access is required .....</string>
<key>NSLocationAlwaysAndWhenInUseUsageDescription</key>
<string>Location access is required .....</string>

The prompt shows up when app launches first time, asking for permission and I tapped allow. Checked in general setting and it shows location access permission correctly.
Screenshotst

Please read through: https://jamesmontemagno.github.io/GeolocatorPlugin/GettingStarted.html

@jamesmontemagno : I already read it and setup my info.plist as I stated above:
<key>NSLocationWhenInUseUsageDescription</key>
<string>Location access is required .....</string>
<key>NSLocationAlwaysAndWhenInUseUsageDescription</key>
<string>Location access is required .....</string>

I'm not sure what I'm missing here

The guide said

So I put only those 2 above. Eventually I tried adding the following:
<key>NSLocationAlwaysUsageDescription</key>
<string>Location access is required .....</string>

Now info.plist looks like this
<key>NSLocationWhenInUseUsageDescription</key>
<string>Location access is required .....</string>
<key>NSLocationAlwaysUsageDescription</key>
<string>Location access is required .....</string>
<key>NSLocationAlwaysAndWhenInUseUsageDescription</key>
<string>Location access is required .....</string>

Now it's working.

That is what exactly the guide says... you can use one OR the other... and then under iOS 11 it tells you you have to put all 3

I guess I misread this part somehow:
...(If your app supports iOS 10 and earlier, the NSLocationAlwaysUsageDescription key is also required.)...

My apology


Revert "improve process state management"

Reverts apple/swift-tools-support-core#203
This seems to have broken the Windows build:
https://compnerd.visualstudio.com/3133d6ab-80a8-4996-ac4f-03df25cd3224/_apis/build/builds/49569/logs/35

@swift-ci please test

cc @tomerd

@compnerd i'd be okay reverting temporarily since its an improvement only, but think its something we want to bring back once we understand the windows breakage. can you tell how it broke the windows build / give guidance on how i can test it for windows via ci?

@tomerd the failure is in the log above (https://compnerd.visualstudio.com/3133d6ab-80a8-4996-ac4f-03df25cd3224/_apis/build/builds/49569/logs/35); The easiest way to test on Windows is to setup a local build sadly as we do not have GHA for this.  I can kick off a one-off though if you have a PR.

okay sounds good, i am ooo till monday but will look then. feel free to revert if more urgent then that

I think its good to revert for the time being - we haven't had a good build of the development tools in a while now (and this would also block toolchain updates).  Can someone accept the change so that it can be merged?


Automatically deploy snapshot releases to Sonatype Nexus from BuildHive

Need to find a solution because this doesn't seem to be possible at the moment without making Nexus credentials publicly visible, see here.

@jgodi This can be closed now with the new Travis CI build.

Thank you for combing through the issues.


Remove site object from System

something like this can be put back in as and when the spec gets around to defining it.

tests are using System.site too. Looks like there's quite a lot of changes needed to the test suite to get it in line with the new spec.

Let's remove the sites tests then? Or perhaps move the implementation of sites into a hook in the tests if the tests rely on it too heavily?

the tests are currently using site. for 2 things: in system.normalize.spec.js, and to set babel and traceur locations. As both of these have to be changed anyway (normalize no longer exists, and Babel has to change for Babel 6 - if we decide to go with a special build with only import/export transpile, then package.json will have to be changed to reflect that), I don't see much point in doing anything to the tests atm. ISTM the better approach is to decide first what needs to be done to match the polyfill to the spec and the new environment (such as Babel/Traceur), and to ignore the tests until that's done.

site. is also used by index.js for setting path for Babel/Traceur

might as well close this for the moment as well


[Snyk] Security upgrade cypress from 10.11.0 to 13.0.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

646/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 6.5
Server-side Request Forgery (SSRF) SNYK-JS-CYPRESSREQUEST-5871337
Yes
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Server-side Request Forgery (SSRF)

👷 Deploy Preview for localhostorange processing.

Name
Link

🔨 Latest commit
0576f5ddcb82ba6dd101262efb01133f34136f26

🔍 Latest deploy log
https://app.netlify.com/sites/localhostorange/deploys/64ef6de564476a00088b0bc9


cluster manager: implement CDS API

Fixes https://github.com/lyft/envoy/issues/144
The only remaining piece that is missing is more graceful initialization
of CDS clusters on boot. (Basically we should only start listening once
we have done an initial init pass on any loaded CDS clusters). I will do
this in a follow up a this change is enough for folks to start testing basic
functionality.

@lyft/network-team @timperrett @lizan


React support

@itsjavi any samples please?

Sorry your issue description is empty and there are no plans to add react support, but PRs are welcome.


Fix Sticker Mule link

Hello! 👋 This PR fixes the Sticker Mule link the sponsors section.
The screenshot below shows the link it goes to currently.

https://github.com/JUET-Builds/jaypee-hacks/commit/af37ae07c0060d334150756942d652e32dc3d9b2 appears to have fixed it


RuntimeError: self and mat2 must have the same dtype

---------pip package version
transformers-4.30.0.dev0
accelerate              0.20.0.dev0
bitsandbytes            0.39.0
peft-0.3.0.dev0
----------python cmd
python qlora.py --model_name_or_path /home/bmb/models/facebook/opt-125m
---------error
Traceback (most recent call last):
File "/home/bmb/projects/qlora/qlora.py", line 766, in
train()
File "/home/bmb/projects/qlora/qlora.py", line 728, in train
train_result = trainer.train(resume_from_checkpoint=checkpoint_dir)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/transformers/trainer.py", line 1696, in train
return inner_training_loop(
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/transformers/trainer.py", line 1972, in _inner_training_loop
tr_loss_step = self.training_step(model, inputs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/transformers/trainer.py", line 2786, in training_step
loss = self.compute_loss(model, inputs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/transformers/trainer.py", line 2818, in compute_loss
outputs = model(**inputs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/peft/peft_model.py", line 678, in forward
return self.base_model(
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
output = old_forward(*args, **kwargs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/transformers/models/opt/modeling_opt.py", line 945, in forward
outputs = self.model.decoder(
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
output = old_forward(*args, **kwargs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/transformers/models/opt/modeling_opt.py", line 703, in forward
layer_outputs = torch.utils.checkpoint.checkpoint(
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/torch/utils/checkpoint.py", line 249, in checkpoint
return CheckpointFunction.apply(function, preserve, *args)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/torch/utils/checkpoint.py", line 107, in forward
outputs = run_function(*args)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/transformers/models/opt/modeling_opt.py", line 699, in custom_forward
return module(*inputs, output_attentions, None)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
output = old_forward(*args, **kwargs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/transformers/models/opt/modeling_opt.py", line 331, in forward
hidden_states, self_attn_weights, present_key_value = self.self_attn(
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
output = old_forward(*args, **kwargs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/transformers/models/opt/modeling_opt.py", line 174, in forward
query_states = self.q_proj(hidden_states) * self.scaling
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/home/bmb/anaconda3/envs/qlora/lib/python3.10/site-packages/peft/tuners/lora.py", line 565, in forward
result = F.linear(x, transpose(self.weight, self.fan_in_fan_out), bias=self.bias)
RuntimeError: self and mat2 must have the same dtype

pip install -q -U git+https://github.com/huggingface/peft.git
peft version should be 0.4.0.dev0

pip install -q -U git+https://github.com/huggingface/peft.git peft version should be 0.4.0.dev0

I used to install  0.4.0.dev0 and I meet the following problem: lora weights are not saved correctly
https://github.com/artidoro/qlora/issues/41

pip install -q -U git+https://github.com/huggingface/peft.git peft version should be 0.4.0.dev0

I used to install 0.4.0.dev0 and I meet the following problem: lora weights are not saved correctly #41

maybe you can try this: https://github.com/artidoro/qlora/pull/44

peft version should be 0.4.0.dev0

Thank you.Now the checkpoint can be saved correctly


waffle.io Badge  http://example.com

Merge this to receive a badge indicating the number of issues in the ready column on your waffle.io board at https://waffle.io/mbulkeley/KanbanSwimlanes
This was requested by a real person (user mbulkeley) on waffle.io, we're not trying to spam you.

http://example.com


Geolocation

Geolocation (not automatic). It should be in the burger menu.

If need - create pagination and optimize the request, with cache (client side)