Cannot set Elvish shell as login shell

Issue description
When setting the login shell to elvish in configuration.nix, I get the error: error: /nix/store/fw7gq9fwhbj1kykhgykkpl64dizzllyf-elvish-0.11-bin is not a shell package
Output with --show-trace
~/.git-configs
❱ sudo nixos-rebuild --show-trace switch
warning: Nix search path entry '$HOME/.nix-defexpr/channels' does not exist, ignoring
building Nix...
warning: Nix search path entry '$HOME/.nix-defexpr/channels' does not exist, ignoring
warning: Nix search path entry '$HOME/.nix-defexpr/channels' does not exist, ignoring
warning: Nix search path entry '$HOME/.nix-defexpr/channels' does not exist, ignoring
building the system configuration...
warning: Nix search path entry '$HOME/.nix-defexpr/channels' does not exist, ignoring
error: while evaluating the attribute 'activationScript' of the derivation 'nixos-system-rens-pc-nixos-18.09pre144558.b7c97e6267b' at /nix/store/7xqi0ahj5z666qqzjz7w41isasfk5clh-nixos/nixos/pkgs/stdenv/generic/make-derivation.nix:170:11:
while evaluating the attribute 'system.activationScripts.script' at /nix/var/nix/profiles/per-user/root/channels/nixos/nixos/modules/system/activation/activation-script.nix:62:9:
while evaluating 'textClosureMap' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/strings-with-deps.nix:70:35, called from /nix/var/nix/profiles/per-user/root/channels/nixos/nixos/modules/system/activation/activation-script.nix:83:18:
while evaluating 'id' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/trivial.nix:46:8, called from undefined position:
while evaluating the attribute 'text' at /nix/var/nix/profiles/per-user/root/channels/nixos/nixos/modules/system/activation/activation-script.nix:9:5:
while evaluating the attribute 'text' at /nix/var/nix/profiles/per-user/root/channels/nixos/lib/strings-with-deps.nix:77:38:
while evaluating the attribute 'text' of the derivation 'users-groups.json' at /nix/store/7xqi0ahj5z666qqzjz7w41isasfk5clh-nixos/nixos/pkgs/stdenv/generic/make-derivation.nix:170:11:
while evaluating 'toShellPath' at /nix/var/nix/profiles/per-user/root/channels/nixos/nixos/lib/utils.nix:21:17, called from /nix/var/nix/profiles/per-user/root/channels/nixos/nixos/modules/config/users-groups.nix:389:17:
/nix/store/fw7gq9fwhbj1kykhgykkpl64dizzllyf-elvish-0.11-bin is not a shell package
Exception: sudo exited with 1
[tty], line 1: sudo nixos-rebuild --show-trace switch

Normally, you can add /bin/elvish to /etc/shells, then chsh -s /bin/elvish, but this doesn't work in NixOS.
Steps to reproduce
I set users.users.<name>.shell = pkgs.elvish;.
Then do a rebuild sudo nixos-rebuild switch.
With pkgs.fish or pkgs.bash this works fine.
Technical details
 - system: `"x86_64-linux"`
 - host os: `Linux 4.14.52, NixOS, 18.09pre144558.b7c97e6267b (Jellyfish)`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.0.4`
 - channels(root): `"nixos"`
 - channels(rensk): `"nixos-18.09pre144558.b7c97e6267b"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Actually, maybe scratch this.
I just tried using Elvish as a login shell, but this causes all sorts of problems, as it cannot source /etc/profile, which causes you to have an empty environment, and it would probably be best if you would be unable to actually set it as a login shell.
For now I am falling back on setting bash as login shell, and then using exec elvish

There is an elvish module to setup nix env, which is also mentioned on the wiki page.


order sources and sets by created_at

On sets#show order sources by their created date (earliest created should appear first).
On sets#index, order sets by their created date (earliest created should appear first).
This addresses ticket #8078

I will investigate this failing test as soon as I can.

Test is passing.  Ready for review.

Looks simple enough.  I'm noticing the change to index.html.erb_spec.rb. Does that test check whether they are ordered?

It does not check that they are ordered, it just changes the value of @source_sets in the spec from an Array to an ActiveRecord::Relation (in the real app, the value of @source_sets will be an ActiveRecord::Relation).  The Array was doing a good job of emulating the true value of @source_sets before b/c they are both enumerables.  But, Array does not respond to the order method, so it was causing the test to fail.

@markbreedlove do you think I should add a test that checks for order?

No, not now. I'll go ahead and merge it if it's rebased.


Add macos testing

Description
Related to #53 where we describe our desire to expand to macos testing. 🤞 that tests work!
What is the nature of your change?

[ ] Bug fix (fixes an issue).
[x] Enhancement (adds functionality).
[?] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] This change requires a documentation update.

Checklist
Please ensure that all boxes are checked before indicating that a pull request is ready for review.

[x] I have read the CONTRIBUTING.md guidelines.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] I have deleted all non-relevant text in this pull request template.

Hmm, it seems the PR tests passed but the merging tests did not again https://github.com/cytomining/CytoTable/actions/runs/5730141075/job/15528300183


Fix typo in example IP requirements

@mkostersitz Could you review the proposed update? Note that the same sentence structure is used in line 160:

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Fixed in both instances. Thanks for confirming @mkostersitz. I'll merge the PR.
Thanks, all.


[ERROR:Pooled connection observed an error;]

Query/Question
I find that every time I download a file with BlockBlobClient, an error log will appear in the background later. This error does not affect my application,it work well, but I think it will have some impact on operation team.
I used the sample code BasicExample.java
My Question is Is this a problem, and how to avoid it..
Log:
2020-04-19 11:03:51,280 ERROR [reactor-http-nio-4] (Loggers.java:300) (reactor.util.Loggers$Slf4JLogger:error) - [id: *********, L:/*********** - R:***********.blob.core.windows.net/***********:443] Pooled connection observed an error; [   ] java.io.IOException: 既存の接続はリモート ホストに強制的に切断されました。 	at java.base/sun.nio.ch.SocketDispatcher.read0(Native Method) 	at java.base/sun.nio.ch.SocketDispatcher.read(SocketDispatcher.java:43) 	at java.base/sun.nio.ch.IOUtil.readIntoNativeBuffer(IOUtil.java:276) 	at java.base/sun.nio.ch.IOUtil.read(IOUtil.java:233) 	at java.base/sun.nio.ch.IOUtil.read(IOUtil.java:223) 	at java.base/sun.nio.ch.SocketChannelImpl.read(SocketChannelImpl.java:358) 	at io.netty.buffer.PooledUnsafeDirectByteBuf.setBytes(PooledUnsafeDirectByteBuf.java:288) 	at io.netty.buffer.AbstractByteBuf.writeBytes(AbstractByteBuf.java:1132) 	at io.netty.channel.socket.nio.NioSocketChannel.doReadBytes(NioSocketChannel.java:347) 	at io.netty.channel.nio.AbstractNioByteChannel$NioByteUnsafe.read(AbstractNioByteChannel.java:148) 	at io.netty.channel.nio.NioEventLoop.processSelectedKey(NioEventLoop.java:648) 	at io.netty.channel.nio.NioEventLoop.processSelectedKeysOptimized(NioEventLoop.java:583) 	at io.netty.channel.nio.NioEventLoop.processSelectedKeys(NioEventLoop.java:500) 	at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:462) 	at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:897) 	at java.base/java.lang.Thread.run(Thread.java:834)
Why is this not a Bug or a feature Request?
A clear explanation of why is this not a bug or a feature request?
Setup (please complete the following information if applicable):

OS: [e.g. Windows10]
IDE : [e.g. STS]
JDK: zulu 11
azure-storage-blob V12

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[Yes] Query Added
[Yes] Setup information Added

Decoding the error message:
Pooled connection observed an error; [ ] java.io.IOException: Existing connection forced to the remote host
One possibility is service sending a RST packet, there is no clean way for us to know if a connection is in this state (where the service has sent an RST packet) without actually trying to read or write data into the connection/channel. An attempt to write|read on such connection result in "An existing connection was forcibly closed by the remote host". There could be various reason service decide to close (i.e. sending RST), such as service timeout set on connections.
I believe such exceptions are retried by creating a new connection or picking a healthy connection to the same host from the pool. Due to the retry underneath, this retryable may not surface to the application, which explain why your application expected behavior is not impacted.
\\cc @rickle-msft

@anuchandy
Thanks for your comments.
I tried to set the max-retires to 0, But the same error happened.

Maybe  this problem due to the version of netty.
azure-blob-storage sdk(v12) has no problem.
my springboot version is 2.1.1. and the nested netty's version is too old. when I change to 2.2.x, it never occurs any more.
@anuchandy thank you again.
I will close this issue..


Add action to build and upload package to anaconda

Thanks @Skealz to do the publish for this repo!


Expense-Delete and replace options displayed for receipt in archived workspace offline

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

Version Number: 9.0.47-1
Reproducible in staging?: Y
Reproducible in production?: Y
Issue was found when executing this PR: https://github.com/Expensify/App/pull/50395
Issue reported by: Applause Internal Team
Action Performed:

Open the app and log in
Navigate to a workspace chat
Disable internet connection
Create an expense with a receipt
Navigate to the list of workspaces and delete the workspace
Open the archived workspace chat
Select the expense > tap 3 dots

Expected Result:
The "Delete" and "Replace" options should not be present
Actual Result:
The "Delete" and "Replace" options are displayed when the workspace is deleted offline
Workaround:
Unknown
Platforms:

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/ab81d261-f065-4c24-807a-176a8fecba71

View all open jobs on GitHub

We think that this bug might be related to #wave-collect - Release 1

@dylanexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

ongoing

Hey @johncschuster! I'm heading out on parental leave so reassigning this! TY! 🙇♂

@Krishna2323 how's this one going?

@Krishna2323 how's this one going? Were you able to get a PR up for this?

Thanks for your comment, @Krishna2323! Let me review this with fresh eyes in the morning.

I didn't get to this one today. I will return to this comment tomorrow.

No update today. Still catching up from a shorter week last week.

Ok! I'm back. @Krishna2323 I'm not quite sure if it's intended to delete and replace the receipt image for an archived expense. I'll need to check on that on Monday.

I didn't get to dig into this today. I should have some more time tomorrow!

No update on this one. I'll need to push this one forward tomorrow.

I'm working a shorter week this week and couldn't fit this in today

I'm catching up from being OOO for the holiday last week. Will get this slotted in tomorrow.

I couldn't action this today. I will check it out this weekend.

No update. I'll bring this to Slack tomorrow for discussion on the intended behavior.

I didn't get to this today. I will write up the discussion over the weekend and post it Monday

No update


On Ubuntu 18.04, your wallet fails to launch (wallet from your released binaries).

On both Ubuntu 16.04 and 18.04 the compilation fails even after all the dependencies have been installed. The only way to compile is by using the switch --without-gui which then means no gui wallet.

Due to the previous errors, it seems that the wallet can not be run on Ubuntu (the daemon works fine if compiled with the above switch).

On Ubuntu 18.04, unless the switch --disable-tests is used with ./configure, the compilation fails.

New version it is updated check


Fixed TimescaleDB License Information

Fixed the licensing for the timescale package to TSL (Timescale License) since it was built with the TSL code base enabled and split the Apache License only version into its own package
Description of changes
The licensing information for the existing TimescaleDB package was wrong. It was built with the TSL (Timescale License) licensed components, but was identified as Apache License (missing the APACHE_ONLY parameter when compiling).
A new package is introduced with only the Apache Licensed elements. That means upgrades of existing installation would not miss any of the features they were potentially using, but the license changed.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Not sure if the existing package should be fixed to only provide the Apache licensed components and a new TSL package should be provided (with full features) or the way I did it now. I thought it's more important to provide all the features that were provided beforehand, but this implies a license change on upgrades.
Happy to flip the style.

please update it as per IndeedNotJames suggestion

Applied the requested changes and changed the commit message. Not sure about the meta  attributes though 🤔
Guess sign-off isn't necessary, is it?

@ofborg build postgresqlPackages.timescaledb postgresqlPackages.timescaledb-apache

Are we good here, or is there anything else to fix / do for me? :)


Feature/rusu1/rc/visit 3324 reverse cli docs generation mechanism

Description
Resolves #3324
This PR changes the way cli documentation is processed. Previously, documentation was created in MethodDoc.C. A script would then transform that documentation into an .rst files for RTD. That order has been reversed as such:

Developer runs /src/doc/sphinx_cli_extractor.py to generate updates for events.rst and attributes.rst (same as before)
Developer modifies functions.rst with the desired documentation.
Developer runs /src/doc/functions_to_method_doc.py. This populates MethodDoc.[Ch].

So MethodDoc.[Ch] are now auto-generated files.  Functions.rst is easier to modify than MethodDoc.[Ch], so this should make the documentation process easier, and since the documentation gets added directly to the .rst file, there are no extra steps to port the documentation into a sphinx-friendly format.
Changes to look out for:

I've added a pre-commit hook that ensures any changes to documentation in functions.rst is coordinated with changes in MethodDoc.[Ch].
Table row formats have changed in attributes.rst
sphinx_cli_extractor.py no longer auto-generates functions.rst, so that support has been stripped from the script.
CLI documentation aliasing has been turned off. MethodDoc.[Ch] and visitmodule.C reflect this.
Functions.rst now accurately presents the argument and return types for almost every function (See #3324 for a list of functions that need informed knowledge).
Functions.rst uses sphinx tables instead of markdown tables because sphinx tables are easier to manually manage.
I manually updated most aliasing functions in functions.rst so that the description only talks about that specific function.
EvalCubic, EvalCubicSpline, EvalLinear, and EvalQuadratic were previously missing from functions.rst. I've added them.
Removed the doc for Pick because visitmodule.C showed that Pick was not supported (All the other pick functions are still there)
MethodDoc.C is now missing the short description under the function name (see diff for example). These are gone because they were not ever generated for functions.rst, so they did not exist when I was doing the reverse generation. I have a way to put these back, just let me know if I should.

Type of change

[X] Other (please describe below)
Reverse generation of cli documentation.

How Has This Been Tested?

Meticulously compared old and new MethodDoc.[Ch] to ensure documentation matches. There were a few places in the old MethodDoc.C that had some issues, so I updated those places.
Tested all possible paths through hook logic to ensure everything worked as planned.

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[x] I have updated the release notes
[X] New and existing unit tests pass locally with my changes
[X] I have assigned reviewers (see VisIt's PR procedures for more information).

Very cool! I'm on my other project until next Monday, so my review time will be limited until then, but I'll try and squeeze some time in this week.


fixed some unstable tests

Description
#4871 moved the mark_deploy_if_done call to a background task. These two tests weren't updated to account for that.
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[ ] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Merged into branches master in 6a836afc2e6b5ecc47057d2a4bf2aa3b4c201aec


Adding SequenceFileAsBinaryInputFormat iterator for hadoop mappers

I'm currently running some Hadoop jobs on image data. They are typedbytes in a SequenceFile.
I set the job's -inputformat to org.apache.hadoop.mapred.SequenceFileAsBinaryInputFormat, which worked great to pass in typedbytes to the mapper.
However, I found that JobTask._map_input always splits on newlines. However, newlines occur in the middle of my image files, which causes failures.
Here's a janky fix that reads in the keys and values and their lengths, and splits the input stream based on that. I have a few questions before I would consider this worthy of merging, but since I'm already working on it I figured it should probably go in a PR.

The method by which I access (and check) the input_format is really janky. Is there a better way to set this up? Maybe when job_runner().run_job(job) is called, we can set job.jobopts to be a dict of all the options passed in to job_runner.

How would you suggest testing this? It looks like test/hadoop_test.py seems to be mostly tests of the "run jobs, then check the output" flavor. Would it be sufficient to just have a "_binseqfile_iter(stdin) takes binary input and outputs key-value pairs properly" test?

Closing to minimize noise. Open up again if you want to iterate on this.


fix(slugs): escape special characters in slugs

Summary
Fixes #4212. Removes special characters from slugs to prevent errors in loading pages at invalid routes.
Test plan
Added unit tests to ensure the additional characters are being properly removed from the slugs.

This doesn't seem like an issue with the CMS, but with the Bitbucket API.
’ is a valid char for file paths and saving an entry with it works in GitHub and GitLab:
https://github.com/erezrokah/netlify-cms-reproductions/pull/240/files
https://gitlab.com/erezrokah/netlify-cms-reproductions/-/merge_requests/3/diffs
In Bitbucket the PR is created with a valid filename:
https://bitbucket.org/erezro/netlify-cms-reproductions/pull-requests/5/create-post-test-test/diff
But using the following API call to read it fails:
curl -i https://api.bitbucket.org/2.0/repositories/erezro/netlify-cms-reproductions/src/8dd93bd12411ff42cbf1d9da60647afd02c46a2b/content/posts/test%E2%80%99-test.md

{"type":"error","error":{"message":"No such file or directory: content/posts/test’-test.md"}}

Regardless, the other chars added to sanitisation in this PR are already handled in:
https://github.com/netlify/netlify-cms/blob/9217e10e0610ba807b8e657d3885bb62c5aeed22/packages/netlify-cms-core/src/lib/urlHelper.ts#L83


Send Bad Response for missing fields

In the logIn, signUp and authGoogle functions, present in controllers > auth.controller.js, check if any field is missing in the request and return a 400 response if so.
API structure: API_endpoints.md

Make sure to pull before starting your work.
For any doubts or difficulty, you can ping me on discord, I'll try to help you out as soon as possible.
All the Best!!

claim

Assigned to @SanyamAgrawal07. All the Best!!


♻Megamenu - Add The New Menu

Cc:

Hi @Harry-Ross
Pain

As per https://github.com/SSWConsulting/SSW.Website/issues/1560, I have added the new megamenu to the https://github.com/SSWConsulting/SSW.MegaMenu repo, and is ready to be published to NPM once refinements have been made.
What code could be improved?

Tasks
### Tasks
- [ ] Add v4.0.0 of the megamenu to rules once it has been published to NPM

More Information

Screenshots

Figure: The new (v4) mega menu on the v3 site

Figure: Before - the v3 mega menu on SSW Rules

Figure: After - the current prototype v4 working on SSW Rules locally as per https://github.com/SSWConsulting/SSW.Website/issues/1560
Thanks!

Hey Team,

Please test this one staging.ssw.com.au

Done - This is now live 🚀


Normalize slashes in a file path to be posix/unix-like forward slashes

On windows, pagesPath and this.resourcePath is using \ instead of /.
If there aren't _app.js. App will not be translated because this.resourcePath.includes('node_modules/next/dist/pages/_app') is always false on windows.
This PR may fixes #455.

The alternative solution is using https://www.npmjs.com/package/normalize-path instead of .replace(/\\/g, '/')

The alternative solution is using https://www.npmjs.com/package/normalize-path instead of .replace(/\\/g, '/')

Thanks for this contribution @vimutti77

Thanks for this contribution @vimutti77

I prepublished 1.0.2-canary.7 with this change

I prepublished 1.0.2-canary.7 with this change


Unit tests for gradio UI endpoint

Description
Unit tests for gradio UI endpoint
(showing various possibilities of pytest and patch modules)
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[x] Unit tests

Testing

make test-unit

/approve
/lgtm


Clarification on sparse keyframe in-betweening

Hi,
Thanks for this great work and your codebase!
In the sparse keyframe setting of the inbetweening experiments (refering to Table 5 in the paper), how are keyframes exactly selected? Do you select the first frame and then, select the next frame every t = {5, 10, 15, 20} frames? This way, when t = 5 and motion length is some arbitrary length, the mask will look like this: [1,0,0,0,0,1,0,0,0,0,1,...]
I'm asking this as I'm trying to compare my own inbetweening model with yours and I want to make sure I'm exactly replicating your setup.
Thanks :)

Hi, thanks for your interest in our work! You can find the keyframe selection code in our application.py:
https://github.com/c-he/NeMF/blob/79918430970fd138ae730510459c8f34893a3f86/src/application.py#L440-L442
We fix the first and last frames as they are required for SLERP, and the rest of the keyframes are sampled every $t$ frames.


main.rs is empty

Would love a command line PagerDuty client 😃

Hey @wezm,
I'd love to pick up some work here again soon. Are there any resources that are most compelling that you'd like to see in the CLI first? Are you looking at viewing and/or modifying resources? Would you like interactions with the events api for triggering/ack/resolving incidents?
Happy to prioritize some work depending on what people would find most useful.

Most useful thing/thing I was looking to do was bulk resolve incidents.
So a pd-cli incendent resolve all would be excellent.
On Wed, Aug 9, 2017, at 10:48 AM, Gary M. Josack wrote:

Hey @wezm[1],

I'd love to pick up some work here again soon. Are there any resources
that are most compelling that you'd like to see in the CLI first? Are
you looking at viewing and/or modifying resources? Would you like
interactions with the events api for triggering/ack/resolving
incidents?> Happy to prioritize some work depending on what people would find
most useful.> — You are receiving this because you were mentioned. Reply to this
email directly, view it on GitHub[2], or mute the thread[3].>

Links:

https://github.com/wezm
https://github.com/ZeroCostGoods/pdctl/issues/1#issuecomment-321121060
https://github.com/notifications/unsubscribe-auth/AABVG0NJl6WJb7ZHz18i-hC6kpEE6yCsks5sWQHOgaJpZM4Oxbi3


Is there a way to switch php version?

Some wordpress plugins are not compatible with php7, so i would use php5. I see a php5 folder into puppet modules, but how could i use it?

Edit the puppet/manifests/init.pp file and comment PHP7 and uncomment PHP5.


fixes #2584 -- Add Seq.getOption(int) and Seq.getOrElse(int, T)

i believe the PR is correct, however I am annoyed by the code duplication between get, getOrElse and getOption, especially for List, Stream and Queue.
However I think it's unavoidable if we don't want to create "unnecessary" intermediary objects at runtime, which would enable code reuse (or worse, catching exceptions as part of normal operation). Since you suggested getOrElse especially for performance, I'm guessing you'd rather not to that... Unless if we did that only for List, Stream and Queue, which have less trivial get() implementations.. and List and Stream are not very performance oriented.

That does mean double bounds check on getOption & getOrElse but then if you want to do that in a loop, there are better ways than looping on an index.. I'll have the change soon.

I've applied the changes. I agree that for Stream we can reuse getOption for getOrElse, and maybe we could even share the implementation for get().

Codecov Report

Merging #2594 into master will increase coverage by 0.01%.
The diff coverage is 92.85%.

@@             Coverage Diff              @@
##             master    #2594      +/-   ##
============================================
+ Coverage     92.84%   92.86%   +0.01%
- Complexity     5338     5357      +19
============================================
  Files            89       89
  Lines         12739    12752      +13
  Branches       1609     1611       +2
============================================
+ Hits          11828    11842      +14
+ Misses          723      722       -1
  Partials        188      188

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/io/vavr/collection/LinearSeq.java
96.77% <ø> (+1.02%)
18.00 <0.00> (ø)

src/main/java/io/vavr/collection/Stream.java
98.49% <90.00%> (-0.17%)
270.00 <6.00> (+6.00)
:arrow_down:

src/main/java/io/vavr/collection/List.java
99.44% <100.00%> (+<0.01%)
268.00 <3.00> (+3.00)

src/main/java/io/vavr/collection/Queue.java
96.46% <100.00%> (+0.01%)
205.00 <3.00> (+3.00)

src/main/java/io/vavr/collection/Seq.java
94.00% <100.00%> (+0.25%)
44.00 <4.00> (+4.00)

src/main/java/io/vavr/collection/IndexedSeq.java
95.83% <0.00%> (+0.83%)
38.00% <0.00%> (+3.00%)

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 4126534...eabd63b. Read the comment docs.

What if for instance in vector, we'd call directly true.getIndex, and similar in other collections, to avoid double bounds checking?

but you want to have a public apply method without bounds check? I wouldn't expect that to be public, as it has no guarantee as to the type of exceptions it'll throw, right?
But appart from the 'public' part, it sounds good, on it.

@emmanueltouzery yeah, that's right. In fact, we already have a public method T apply(Integer) on Seq because Seq implements PartialFunction.
A PartialFunction f works exactly that way:

f.isDefinedAt(t) states whether the function is defined at a specific value t
f.apply(t) is defined when, and only when f.isDefinedAt(t) == true, otherwise the behavior isn't predictable (a call may throw or may return an arbitrary value).

Please go ahead!

done. I didn't manage to do a javadoc link to isDefinedAt :blush:, otherwise it's starting to look nice!


stage 3 shape/dimension issues

Edit: Okay - I have zero clue why this is happening but at this point can only assume that it has to do with using CUDA 11 and sparse attention. I've disabled literally everything - including FusedAdam and cpu_offload, and I even just reinstalled deepspeed but this issue persists.
Wouldn't be the first time I had strange errors because of a somehow borked CUDA install either...
On the other hand, the issue does at least seem quite coincidentally related to the linked deepspeed issue. So I guess I'll leave this up until I know more.
Original post:
Perhaps it has to do with this:
https://github.com/microsoft/DeepSpeed/issues/828

When enabling deepspeed stage 3 and using their "FusedAdam" optimizer (the super fast CPU one) instead of passing in the normal Adam optimizer, I get the following stacktrace:
Traceback (most recent call last):
  File "train_dalle.py", line 331, in <module>
    loss = distr_dalle(text, images, return_loss = True)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 889, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/opt/conda/lib/python3.8/site-packages/deepspeed/runtime/engine.py", line 914, in forward
    loss = self.module(*inputs, **kwargs)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 889, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/root/DALLE-pytorch/dalle_pytorch/dalle_pytorch.py", line 459, in forward
    image = self.vae.get_codebook_indices(image)
  File "/opt/conda/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
  File "/root/DALLE-pytorch/dalle_pytorch/vae.py", line 152, in get_codebook_indices
    _, _, [_, _, indices] = self.model.encode(img)
  File "/root/.local/lib/python3.8/site-packages/taming_transformers-0.0.1-py3.8.egg/taming/models/vqgan.py", line 54, in encode
    quant, emb_loss, info = self.quantize(h)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 889, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/root/.local/lib/python3.8/site-packages/taming_transformers-0.0.1-py3.8.egg/taming/modules/vqvae/quantize.py", line 42, in forward
    torch.sum(self.embedding.weight**2, dim=1) - 2 * \
IndexError: Dimension out of range (expected to be in range of [-1, 0], but got 1)

The input_mask that we currently support in our implementation must have this shape (batch_size, 1, 1, sequence_length). This means that the mask can be different for different input.

In the issue - they are working on a solution which allows you to pass in a custom shape. I think that might help. Alternatively - is it possible to work with the expected shape somehow?

All I know for sure is that deepspeed stage 3 with cpu offload (and literally nothing else) worked for me - and then somehow deepspeed stage 3 broke itself in a permanent way.
I've tried several things:

merge the pull request mentioned in the deepspeed issue into my PR
is it a dataset problem? switch to CUB because many have successfully trained with it
am i certain my training code is identical otherwise? git checkout . from main and add back in just this config:

    "zero_optimization": {
        "stage": 3,
        "cpu_offload": True,
    },

But the problem persists - even though IT WORKED - this exact same config worked before I ever decided to enable FusedAdam. After that - it won't go away at all until I just straight up git checkout train_dalle.py to disable zero_optimization alltogether.
I haven't yet tried the most obvious fix - use microsoft's main repo with a version of CUDA 10.1 instead - but I unfortunately don't have the ability to reconfigure my install like that and would have to just set up a new instance just to test this problem which may in fact just sort of go away for whatever reason.
So if anyone can try running stage 3 with FusedAdam on CUDA 10.1 on a V100 - have at it. I'll just say, at this point, I'm glad we're supporting horovod as well.

I have at least verified that the problem was indeed caused by running "fused_adam" on the CPU with stage 3.
Switch to stage 2 with cpu_offload = True results in a CUDA out of memory error - not sure why.
However, if you still want to use FusedAdam (it should still be a good deal faster than torch.optim.Adam - correct?) then you can disable cpu_offload and stick with stage 2:

deepspeed_utils.check_batch_size(BATCH_SIZE)
deepspeed_config = {
    "zero_optimization": {
        "stage": 2,
    },
    "optimizer": {
        "type": "Adam",
        "params": {
            "lr": LEARNING_RATE,
        },
    },
    'train_batch_size': BATCH_SIZE,
    'gradient_clipping': GRAD_CLIP_NORM,
    'fp16': {
        'enabled': args.fp16,
    },
}

(distr_dalle, opt, dl, scheduler) = deepspeed_utils.maybe_distribute(
    args=args,
    model=dalle,
    optimizer=None,
    model_parameters=dalle.parameters(),
    training_data=ds if args.deepspeed else dl,
    lr_scheduler=scheduler if LR_DECAY else None,
    config_params=deepspeed_config,
)
avoid_model_calls = args.deepspeed and args.fp16

May also be related to external parameters in DeepSpeed.
I've opened an issue asking about these but haven't gotten an answer that.
Since you're getting an error, I'm assuming these are problems with external parameters finally showing up. :)

DeepSpeed 0.3.15 now automatically detects external parameters but I'll #207 adds manual external parameter registration anyway.


Mark big strong contributors as Collaborators

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Description
In an attempt to highlight people who keep this application going, mm2pl and I discussed bumping each-other to Collaborator(s).
This prompted me to include those who are currently active Contributors who have contributed a lot, and who have added themselves to this list prior.
If you are a contributor who has not yet added themselves to the list, but would likely fall under the Collaborator label, open a PR as normal and I'll bump you in that PR
If you see this PR in the future and are a former-active contributor who has contributed a lot and would like to be bumped, open a PR and I'll approve+merge it.

@dnsge You get to be the one person I ping on Github, as you are a still-active contributor but not very active in chat
Would you like to be bumped to Collaborator? (feel free to say no if you don't want to, it's not an issue at all 🙂)

Just one thing I noticed and think would be better for the PR title
- Collaberators
+ Collaborators

I'll fix on merge with REFINED GITHUB

@kornes It's come to my attention that you were never given the copy pasta so I'll ping you in this, would you like to be added to the contributors list? As you'd fall under the bump-to-collaborator 🙂

@kornes It's come to my attention that you were never given the copy pasta so I'll ping you in this, would you like to be added to the contributors list? As you'd fall under the bump-to-collaborator 🙂

fine with me, just text without avatar

@Wissididom [2023-01-16 13:28:35] #pajlada pajlada: $notify wissididom you can add yourself to contributors in a PR btw, idk if I posted the copypasta in any of your PRs

You didn't but I also wasn't sure if I should because I only contributed to things outside of the program code. Will do later I think.


UI/UX Research

Want to collect also in this project collection about UI and UX about search:

https://design4users.com/design-search-in-user-interfaces/

Part of readme now


feat(move-stable): tag the deployments

do-not-merge
it will require additional changes with regards to getting the latest version during RPM build, since it breaks both hatch and setuptools…


Dual pane view!

Implements #1674

The changes to remove the App.CurrentInstance (thanks @duke7553!) allow to implement this with minimum changes.
What's missing:

[ ] The tab title reflects the working directory of the pane that was last navigated (should the title be like "Documents | Downloads" instead?)
[ ] The resume where you left off option reopens only one pane with the working directory of the pane that was last navigated before closing
[x] Maybe an option to always open the right pane when opening a new tab
[ ] An option to open a folder or a sidebar item in the right pane
[ ] An button to open the right pane from the navigation bar menu
[x] An option to disable this feature

@gave92 Nice work! How does the dual pane work with the sidebar?

@yaichenbaum the sidebar always controls the left pane. Seems better since the sidebar is visually connected to the left pane only

@yaichenbaum the sidebar always controls the left pane. Seems better since the sidebar is visually connected to the left pane only. What do you think? At most I'd add a "Open in the right pane" context menu option to the sidebar items

An "Open in new pane" option would be helpful. However, being that this feature is for power users and many users would prefer to hide these options, I would recomend adding a setting to enable/disable the dual pane feature.

WOOOOOOOOOOOOOOOOOOO
POGCHAMP
:shipit: :shipit: :shipit:

The tab title reflects the working directory of the pane that was last navigated (should the ta title be like "Documents | Downloads" instead?)

Maybe we indicate to user (somehow) the selected pane and update path for it when it's focused.

@gave92 Instead of two panes, could it support variable size panes?

@Jaiganeshkumaran yeah it could support any number of panes (the panes are already resizable).
I don't know if I'll manage to include that in this PR, also it would overlap a lot with ColumnView #2500

Do we have any progress report on #2500?

@gave92 This is great to see! Looking forward to finally addressing user feedback with this PR.

/azp run

bruh

@gave92 This feature looks great! Here are a couple of things I noticed so far

The second pane looks like it's not the same width as the first pane
The focus indicator should only show up if more than one pane is open
The "New Pane" menu item should be hidden when two panes are already open
The shadow between the panes should be removed in favor of a thin border

Not entirely comfortable with there being two panes for a single tab, with two totally unrelated locations.  Do you need two tab bars, or does the tab UI change to indicate two panes side by side?  Could the second pane be a flyout on the right that can be brought into view temporarily for file transfers or comparisons, then dismissed?  So a transient UI and not a full IconView in the same way as the tab would navigate to?
There is also a question to be asked with future support for dual-screened devices.  Assuming Files plans to be ready for 10X, how will the dual screen form factor be supported?  This feature could complicate that.

Not entirely comfortable with there being two panes for a single tab, with two totally unrelated locations. Do you need two tab bars, or does the tab UI change to indicate two panes side by side? Could the second pane be a flyout on the right that can be brought into view temporarily for file transfers or comparisons, then dismissed? So a transient UI and not a full IconView in the same way as the tab would navigate to?

@mdtauk This dual pane feature is commonly found in other file managers and its targeted towards power users who want to work with more than one directory in the same tab. You can think of it as split screen, but the difference is that it's inside the tab and you can even restore your session when restarting the app. A transient UI is an interesting idea, but the dual pane view is something that users have been asking for us to add for quite some time and the functionality is different.

There is also a question to be asked with future support for dual-screened devices. Assuming Files plans to be ready for 10X, how will the dual screen form factor be supported? This feature could complicate that.

As of now Files does not run on Windows 10X but if we can get it working in the future, the dual pane feature will give the user more choices in how to use the app.

@mdtauk This dual pane feature is commonly found in other file managers and its targeted towards power users who want to work with more than one directory in the same tab. You can think of it as split screen, but the difference is that it's inside the tab and you can even restore your session when restarting the app. A transient UI is an interesting idea, but the dual pane view is something that users have been asking for us to add for quite some time and the functionality is different.

So I think what people have wanted, is the ability to drag a tab to the side of the window, and dock it as a split view.  That is a bit different to how you are implementing the feature.
I think an option to have two tab views, split into two columns, would solve the problem.  Or visually distinguishing the second pane as a flyout (without light dismiss behaviour, but the ability to close it) so there is no confusion for navigation or tab contents.
Having split tab bars, would also solve the problem with supporting Dual Screens.
Then the only thing you may need to implement is a toggle for the Sidebar, so you can close it to give you more space on both screens.

@mdtauk That use case is already covered by the built-in split screen in Windows. I think that both of them are valid use cases but as of now, we have only gotten feedback regarding the use case being implemented here.

@mdtauk That use case is already covered by the built-in split screen in Windows. I think that both of them are valid use cases but as of now, we have only gotten feedback regarding the use case being implemented here.

Except that is not two windows side by side, but a single window split over both screens.

@mdtauk is your idea close to what you get if you open two instances of the app and stick them to the screen edges? You get two independent tab bars and you can drag tabs from one to the other.

@mdtauk is your idea close to what you get if you open two instances of the app and stick them to the screen edges? You get two independent tab bars and you can drag tabs from one to the other.

That should already be possible, I think I am talking more about the best way to have a split pane layout for the app.
The idea as demonstrated in the screenshot, is a single window, but the two icon views are under one tab.
I think this could be confusing to users, as two views should be two tabs.  But the purpose is for dragging between views, or visually comparing right?  So rather than present the second icon view in the same way, if it were in a right hand sidebar/flyout - it would look distinctive enough to not need a second tab.
But if you are thinking ahead to Windows 10X and Dual Screen form factors, then two tab bars beside each other would work better.

@gave92 This feature looks great! Here are a couple of things I noticed so far

The second pane looks like it's not the same width as the first pane
...
@yaichenbaum everything in the list is now implemented 👍


[Web] Fix dlopen threading issues by pre-opening all GDExtensions

Calling dlopen from a thread cause stalling on some browsers (notably firefox: https://github.com/godotengine/godot/issues/86382), the only valid workaround seems to be to dlopen all GDExtensions from the main thread as soon as possible.
We then keep track of the pre-opened libraries, and return the correct stored when a thread try to dlopen.
This means still no editor+thread+dlink support yet (since we don't know which libraries to open at startup in that case).
Draft status:

Requires way more hacks than I'd like (having 3 main functions, 4 callbacks, and a fake loop is not great :/ )
Only tested in 4.2 yet (via https://github.com/Faless/godot/tree/web/4.2_fix_dlopen_by_preloading)

Instead of using "fake" and "real" terminology, why not instead use "phase1" and "phase2"? Phase 1 loads phase 2, so we would have phase1_loop() (with a comment // Does nothing.), and a phase2_loop.

As a workaround, this seems fine.
It would be nice if we could make OS::open_dynamic_library() asynchronous, so that its work could be deferred to the main thread. Since it's internal and not exposed to users, that's theoretically possible, but since exposed APIs that depend on it (like GDExtensionManager::load_extension()) are synchronous, I don't think we could easily do that. I suppose we could add a LOAD_STATUS_DEFERRED along with a signal to GDExtensionManager? But that's not something we should really try to do for Godot 4.3.

Is this one superseded by PR https://github.com/godotengine/godot/pull/93143 now?

Is this one superseded by PR #93143 now?

By the look of it, yes, it seems so.

Is this one superseded by PR #93143 now?

Yes it is, closing.


How to upload a file using eden treaty?

While the documentation made clear that how to use get and post simple json, there are no example on how to post a file using it, and its not clear on how to approach that. Using formdata as body failed for me. So a better documentation depicting how to make it work would be great.

I'm not sure you're supposed to use eden treaty for that. For comparison it doesn't seem like tRPC supports that

You can pass either File or FileList from value of `, see Treaty File Example


Fix: TypeError: expected string or bytes-like object, got 'list'

sub expects a string for string masking, but currently it can receive any type. Therefore, we should convert the message into a string beforehand.
2024-01-27T21:12:43.276792243Z   File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 256, in _async_update_data
2024-01-27T21:12:43.276804451Z     return await self.update_method()
2024-01-27T21:12:43.276813478Z            ^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-01-27T21:12:43.276821534Z   File "/config/custom_components/toyota/__init__.py", line 102, in async_get_vehicle_data
2024-01-27T21:12:43.276830029Z     _LOGGER.debug(vehicle_informations)
2024-01-27T21:12:43.276837955Z   File "/usr/local/lib/python3.11/logging/__init__.py", line 1477, in debug
2024-01-27T21:12:43.276868291Z     self._log(DEBUG, msg, args, **kwargs)
2024-01-27T21:12:43.276877968Z   File "/usr/local/lib/python3.11/logging/__init__.py", line 1634, in _log
2024-01-27T21:12:43.276890185Z     self.handle(record)
2024-01-27T21:12:43.276902704Z   File "/usr/local/lib/python3.11/logging/__init__.py", line 1644, in handle
2024-01-27T21:12:43.276912209Z     self.callHandlers(record)
2024-01-27T21:12:43.276920238Z   File "/usr/local/lib/python3.11/logging/__init__.py", line 1706, in callHandlers
2024-01-27T21:12:43.276929011Z     hdlr.handle(record)
2024-01-27T21:12:43.276937179Z   File "/usr/local/lib/python3.11/logging/__init__.py", line 974, in handle
2024-01-27T21:12:43.276945496Z     rv = self.filter(record)
2024-01-27T21:12:43.276957643Z          ^^^^^^^^^^^^^^^^^^^
2024-01-27T21:12:43.276970582Z   File "/usr/local/lib/python3.11/logging/__init__.py", line 830, in filter
2024-01-27T21:12:43.276979936Z     result = f.filter(record)
2024-01-27T21:12:43.276987794Z              ^^^^^^^^^^^^^^^^
2024-01-27T21:12:43.276995922Z   File "/usr/local/lib/python3.11/site-packages/mytoyota/utils/logging/log_filters.py", line 16, in filter
2024-01-27T21:12:43.277008170Z     record.msg = self.mask_sensitive_data(record.msg)
2024-01-27T21:12:43.277020769Z                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-01-27T21:12:43.277030495Z   File "/usr/local/lib/python3.11/site-packages/mytoyota/utils/logging/log_filters.py", line 23, in mask_sensitive_data
2024-01-27T21:12:43.277039476Z     msg = compiled_pattern.sub("****", msg)
2024-01-27T21:12:43.277047483Z           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-01-27T21:12:43.277055417Z TypeError: expected string or bytes-like object, got 'list'
2024-01-27T21:12:43.277063292Z 2024-01-27 22:12:43 DEBUG Finished fetching toyota data in 1.984 seconds (success: False)

Hey @GitOldGrumpy can you please take a look and approve if it's ok with you. This solves a problem when setting up the Home Assistant integration when it has updated to MyToyota 1.3.0 in the background.


Issues with Update create/publish Webhooks Events

Motivated by https://github.com/opencollective/opencollective/discussions/6437, I was looking at the web-hook events related to the update create/publish activities. It seems that,

These events don't trigger web-hook events currently.
These events somehow is limited to collectives; although we can have updates for organizations and events and projects as well.

Could you clarify whether it's only impacting Slack/Discord (in this case, we know these don't support all event types yet) or all webhook events (in this case, it's a bug)?

Could you clarify whether it's only impacting Slack/Discord (in this case, we know these don't support all event types yet) or all webhook events (in this case, it's a bug)?

Yes that's correct. I've updated the title and description. 👍🏽


Consider moving beacon-state-transition/fast to separate package

Create a new package beacon-state-transition-fast and move the fast specific code there.
Other options
Use a more structured fs in a single package beacon-state-transition

naive
fast
util

Bump @wemeetagain

See https://github.com/ChainSafe/lodestar/pull/1873#discussion_r542619731

https://github.com/ChainSafe/lodestar/pull/1886#issuecomment-745083813
wdytk @dapplion @wemeetagain ?

Your suggestion of namespacing phase0.fast.processBlock would remove the need to import from deep inside lib which was the may driver behind this issue. The main difference between both solutions is to separate code inside the same package or in different packages. I don't have a strong opinion on either direction and would love to know your pros and cons of each path

Your suggestion of namespacing phase0.fast.processBlock would remove the need to import from deep inside lib which was the main driver behind this issue. The difference between both solutions is to separate code inside the same package or in different packages. I don't have a strong opinion on either direction and would love to know your pros and cons of each path

I guess biggest reason is that it would be easier to import because if we have phase0 namespace in both packages everywhere you use both original utilities and fast stuff you would have to manually rename import (import {phase0 as phase0Fast} or something).

#1886 (comment)

yeah I prefer this setup to have both the original and fast state transition stay inside lodestar-beacon-state-transition, they are all about state transition in the end. Imagine if we have another type of state transition, I prefer to have it stay inside lodestar-beacon-state-transition than having another separate package for it.


price list,rate and tax column values are overlapping/ congestedly placed under estimate,sales order and sales return

issue fixed !


Require plants to be harvestable before sampling

About the PR
Clipping a plant now only produces seeds if a plant is harvestable. Suggested by @GreyMario in https://github.com/space-wizards/space-station-14/issues/24591#issuecomment-1912566099_.
Why / Balance
In plant genetics (PR https://github.com/space-wizards/space-station-14/pull/11407), random mutations have a high probability of making a mutated plant seedless or non-viable (i.e. will die after a while). This was intended to make it more difficult to make a very strong plant.  You could get a very potent plant if you were willing to put in the work.
However, it appears that players have been circumventing the intended gameplay loop by:

Planting a seed
Mutating it a lot, without caring if it becomes non-viable or seedless
Clipping the plant to get the seed
Repeat

This change:

Makes a lot of sense because seeds come from fruits, and if a plant isn't ready to harvest it can't have seeds
Brings very potent mutations back into balance, because plants need to be viable in order for you to clip their seeds
Slows down seed spamming, because you have to wait until the first harvest before you can get seeds

For fun, clipping a seed early shows a message indicating that you knew the harvesting was premature, but you do it anyway and it applies the health change effect.

also this still has Sampled bool so the "lol get fucked" rng can kick in at just 1 seed, after you waited 5 minutes for something to grow

I apologize for my poor English.
I want to point out that this PR is not particularly logical and threatens to kill botany in general. After all, at least such a substance as Robust Harvest removes the opportunity to collect seeds from the fruits of the plant. It is also worth noting the fact that in real life a new plant can be obtained not only from seeds, but also by budding, which is currently used in Space Station 14

I want to point out that this PR is not particularly logical and threatens to kill botany in general.

because harvesting 300 seeds by repeatedly clipping and shoveling seedlings is logical

anyway candidate for speedmerging

i think we should find a solution that lets you get seeds from ungrown plants but prevents you from just mass-duplicating seeds

because harvesting 300 seeds by repeatedly clipping and shoveling seedlings is logical

Why not just allow seeds to be harvested at a certain stage without the influence of the seedlessness gene? It appears EXTREMELY often, which forces you not to use robust or mutagen

In addition, I should note that such restrictions will force you to store 300+ bags of seeds in order to quickly respond to the requests of cooks, cargo and med

In addition, I should note that such restrictions will force you to store 300+ bags of seeds in order to quickly respond to the requests of cooks, cargo and med

No it won't? You should have a healthy backup supply of seeds regardless. This change just forces you to beave realistically and harvest your seed stock at a realistic time and in a realistic manner, rather than shoving a seed into a hydro tray and clipping the resulting seedling to death before shoveling it and repeating.

No it won't? You should have a healthy backup supply of seeds regardless. This change just forces you to beave realistically and harvest your seed stock at a realistic time and in a realistic manner, rather than shoving a seed into a hydro tray and clipping the resulting seedling to death before shoveling it and repeating.

Well, I think it's pretty realistic that eggs grow on trees, you can collect seeds from eggs and they grow in a couple of minutes

Besides, I don't think this is a good option because the chefs will have to wait 10-15 minutes waiting for plain wheat.  I'll be fine with growing a bunch of plants for seeds if the extractor actually guarantees a lot more seeds than you put in

Botany is honestly slow enough as is, mutations are a pain to work with, obscure and hard to learn, adding more waiting and potential problems to the pile isn't gonna be making anyone have a better time playing botanist.

I'm open to other suggestions for what to do. I don't think the current meta where botanists can spam large amounts of seeds is okay. Seed management is part of the game, like water, pests, the growing cycle, etc. There's a reason there's botany gameplay besides a "click to make wheat grow instantly" button.
Some thoughts:

I had intended that there would be more round start hydro trays/plots than there is now
Bumping up seed extractor output sounds fine
Perhaps limit the number of times you can clip seeds

more trays means more seeds that are needed to fill it
i think it would be fine if it was just a stage or 2 of growth, so you cant instantly farm everything but at the same time its not waiting 5 minutes to expand your farm by 1 plant

Seeds management is indeed one part of botany, unfortunately there is not a lot we can control right now because we miss 50% of the content botany have everywhere on 13, mainly a mean to keep genbank for the different seeds AND able to supply themselves with the biogenerator.
I would argue to not make any changed up until Botany hit a refactor, because the code need some love


Let's get our Travis CI tests to pass again

Fixes #
Checklist

[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream master branch.
[ ] I have added necessary documentation (if appropriate)

Changes proposed in this pull request:

@ParthS007 Could I please get your reviews on these changes?  It would be good to have our automated tests passing.


[Http] PSR-7 contract violation

spec
// MessageSerializer.php

/**
 * Retrieves all message header values.
 *
 * The keys represent the header name as it will be sent over the wire, and
 * each value is an array of strings associated with the header.
 *
 *     // Represent the headers as a string
 *     foreach ($message->getHeaders() as $name => $values) {
 *         echo $name . ": " . implode(", ", $values);
 *     }
 *
 *     // Emit headers iteratively:
 *     foreach ($message->getHeaders() as $name => $values) {
 *         foreach ($values as $value) {
 *             header(sprintf('%s: %s', $name, $value), false);
 *         }
 *     }
 *
 * While header names are not case-sensitive, getHeaders() will preserve the
 * exact case in which headers were originally specified.
 *
 * @return string[][] Returns an associative array of the message's headers. Each
 *     key MUST be a header name, and each value MUST be an array of strings
 *     for that header.
 */
public function getHeaders();

problem
When calling $message->getHeaders(), wandu/http only returns an array of arrays if the header is multi-valued. This breaks constructs like the examples above, since in wandu/http $values could be either a string or an array. Here is a sample script reproducing the issue:
// wandu.php

<?php

use Wandu\Http\Psr\Request;
use Wandu\Http\Psr\Uri;

require_once __DIR__ . '/vendor/autoload.php';

$headers = [
    'Connection' => 'keep-alive',
    'Accept-Encoding' => ['gzip', 'deflate'],
];

$req = new Request('GET', new Uri('http://example.com/index.html'), '1.1', $headers);

var_dump($req->getHeaders());
// actual:
// array(2) {
//   'Connection' =>
//   string(10) "keep-alive"
//   'Accept-Encoding' =>
//   array(2) {
//     [0] =>
//     string(4) "gzip"
//     [1] =>
//     string(7) "deflate"
//   }
// }

// expected:
// array(2) {
//   'Connection' =>
//   array(1) {
//     [0] =>
//     string(10) "keep-alive"
//   }
//   'Accept-Encoding' =>
//   array(2) {
//     [0] =>
//     string(4) "gzip"
//     [1] =>
//     string(7) "deflate"
//   }
// }

why I care about this
I am the author of an HMAC HTTP authentication library that aims to work out of the box with any PSR-7 implementation, so in my use case the whole spec is a very hard dependency. ATM I need special logic in some parts of the library to support wandu/http.

Sorry for the delay. Thanks for the issue.
This bug has already been fixed in v3.0(related issue). It is my fault that i did not considering 1.x version users.
If you raise it to v1.6.4 or higher, it will be solved automatically.


how to connect gird

I did not found the config with config.remote to set the grid hub address like in selenide.  Do I have any way to do that?

Right now, you don't have a config for that, if you need to set your tests to run remotely, you have to create the instance of WebDriver on your own and pass it to the Selenide, like this:
from selene.support.shared import browser
myDriver = ...
browser.config.driver = myDriver

Check the full example here: https://github.com/yashaka/python-web-test/blob/b8d4ab9d68aa8fb8e6b5323cfcb6d32b5dcce3e0/tests/conftest.py#L52
P.S.
We may add the same option like in Selenide, but still thinking on the best way to do this...


only one host thread launch CUDA kernel in 20.03

Hi, I used trtis(19.11) and triton(20.03) to do some inferences with multi instances.
However, when I profiled the sever process with Nsight System, I found that in 19.11, there are multi host pthreads that perform CUDA kernel launch (#pthread equal to #instance), and there is only one host pthread that launch CUDA kernel in 20.03. It seems to have a bad influence on the overall performance.
And I didn't find any cmdl options or model configures that control it. ( I checked triton doc and part of source code)
Did I miss anything?
And I wonder how to use multi host pthreads to perform kernel launch in 20.03.
Thanks a lot!

The intuition is that using multiple threads to launch CUDA kernels will introduce CUDA contention, thus we changed it to let one thread to launch the CUDA works. Can you share your results regarding the bad influence?

We used qa_container built from Dockfile.QA, and used resnet50 as an example.
TensorRT generates serialized models.
The max batch size is set in {1,2,4,8,...,256}, and batch size is equal to MBS in each test.
And the #inst is set in {1,2,3,...,8,10,...,16}
In most situations, 20.03 outperforms 19.11.
But when MBS lower than 8, and #inst larger than 6 or 8, 19.11 has better performance.
Here is part of the results.

The intuition is that using multiple threads to launch CUDA kernels will introduce CUDA contention, thus we changed it to let one thread to launch the CUDA works. Can you share your results regarding the bad influence?

Thanks for replying, and we found there still are the same number of host threads as the number of instances in the server process. It seems these host threads only launch cudaEvenSynchronize and incur pthread_cond_wait, which brought overhead. Is this optimizable?

The host threads that perform only cudaEventSynchronize is the completion thread, not the issuing thread. There is one completion thread for each instance as cudaEventSynchronize will block the host thread which is not desired for other approach of using one completion thread for all instances (need to spin-looping each instance's event state in that case).
Can you share the profiling result (i.e. nsys report) on the overhead you found? I don't think the completion threads will incur overhead as the workflow in completion threads is lightweight and cudaEventSynchronize is quite responsive to when the event is completed.
Also what is the procedure for you to generate the above diagrams? There are many components in between sending requests and receiving responses and I can't tell if the overhead is solely due to how CUDA kernels are being launched simply from the diagrams. It will also be helpful if you can run the two Triton versions with perf_client which will show the breakdowns of time spent in each phase of the pipeline.

The 20.03 implementation should not require a large number of model instances to get the best performance. Usually just 1 or 2 instances will give the maximum throughput. Do you have cases where the maximum thoughput found with 19.11 is significantly greater than the maximum throughput of 20.03, using the best configuration for each?

No, 20.03 has better performance in most of my tests. In some cases, 19.11 has slight advantages.
However, in my nsight system profile results, I notice the interval between the launches of the batches is quite long (around 1.5ms). The separation of kernel launches seems to hurt performance, especially in small models. How can we reduce this space?


OODT 823

Made changes to pom.xml to activate doclint only for jdk 1.8

Looks like this commit has two files included. That is a mistake. The only file associated with this issue is core/pom.xml.


feat: implement upsert function

[x] tests and linter show no problems (npm t)
[x] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)
[x] description of changes is added in CHANGELOG.md
[ ] update .d.ts typings

https://github.com/metarhia/metasql/issues/115

Do you need help to add typings?

@tshemsedinov I'm pushing types in different branch
noticed that metasql.d.ts has empty class, so I branched and added declarations

@tshemsedinov https://github.com/metarhia/metasql/pull/158

@tshemsedinov added types

waiting for https://github.com/metarhia/metasql/pull/167


[BUG] background script reloads losing states

What happened?
Hello.
Background script keeps loose state due to reloads.
I save some data in static vars or in classes parameters but because of reload it get's lost.
If I use production build everything works. With dev I have problems.
Is there a way to disable hot reload?
Thank you.
Version
Latest
What OS are you seeing the problem on?
MacOSX
What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
(OPTIONAL) Contact Details
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

public static readonly csReadyListeners: Set = new Set;
so basically it get's reloaded in the middle of logic and looses it's state
if I don't use dev env and use prod logic works and state is preserved

One more thing.

Hi @belarusrulez, thanks for your detailed report! Apologies for the delay, it's been quite a busy week for us.
Are you using MV2 or MV3? With MV3 you shouldn't have any expectations that the background service worker will stay alive and therefore it should be stateless.
Even without HMR, the background service worker has a fixed lifetime and will die when you don't expect it to, in production.
However, this isn't the case with MV2, so we'll need to figure out a good way to deal with that.
For the time being, if you don't want HMR, please continue to use the prod bundle.
We'll likely build a feature to disable HMR in dev but don't have that in yet. If you'd like to send a PR to add that functionality into plasmo, we'd greatly appreciate it. Something like plasmo build --no-hmr

Hey. Its MV3. It actually dies in the middle of code. It’s not idle when it’s get reloaded.

Oh, that's not great. Thanks for the context. We'll take a look soon

I found if I just remove selected "auto-reload.12a1e065.js" from manifest I don't have problems.
BTW I was trying to get the code and fix it contribution guidelines https://github.com/PlasmoHQ/plasmo/blob/main/.github/CONTRIBUTING.md and I was not able to pull. I got the code in zip and was not able to build it using guidelines.

I found if I just remove selected "auto-reload.12a1e065.js" from manifest I don't have problems.
BTW I was trying to get the code and fix it contribution guidelines https://github.com/PlasmoHQ/plasmo/blob/main/.github/CONTRIBUTING.md and I was not able to pull. I got the code in zip and was not able to build it using guidelines.

Can you paste the command you used to clone? To build the whole project, you will need to get the sub-module as well (since we hoist all our lockfiles to the top)

I tried
git clone git@github.com:belarusrulez/plasmo.git --recurse-submodules
and
git clone git@github.com:PlasmoHQ/plasmo.git --recurse-submodules
It gives
Cloning into 'plasmo'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
also I tried the same with https:
Failed to clone 'packages/permission-ui'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/prettier-plugin-sort-imports'...
remote: Enumerating objects: 1584, done.
remote: Counting objects: 100% (403/403), done.
remote: Compressing objects: 100% (194/194), done.
remote: Total 1584 (delta 270), reused 301 (delta 206), pack-reused 1181
Receiving objects: 100% (1584/1584), 2.12 MiB | 13.11 MiB/s, done.
Resolving deltas: 100% (964/964), done.
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/puro'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/puro.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/packages/puro' failed
Failed to clone 'packages/puro'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/rps'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/rps.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/packages/rps' failed
Failed to clone 'packages/rps'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/storage'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/storage.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/packages/storage' failed
Failed to clone 'packages/storage'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/use-hashed-state'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/use-hashed-state.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/packages/use-hashed-state' failed
Failed to clone 'packages/use-hashed-state'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/utils'...
remote: Enumerating objects: 60, done.
remote: Counting objects: 100% (60/60), done.
remote: Compressing objects: 100% (46/46), done.
remote: Total 60 (delta 26), reused 47 (delta 13), pack-reused 0
Receiving objects: 100% (60/60), 11.24 KiB | 1.61 MiB/s, done.
Resolving deltas: 100% (26/26), done.
Cloning into '/Users/belarusrulez/Downloads/plasmo/templates/qtt'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/qtt.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/templates/qtt' failed
Failed to clone 'templates/qtt'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/examples'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/examples.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/examples' failed
Failed to clone 'examples' a second time, aborting```

also I tried the same with https:
Failed to clone 'packages/permission-ui'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/prettier-plugin-sort-imports'...
remote: Enumerating objects: 1584, done.
remote: Counting objects: 100% (403/403), done.
remote: Compressing objects: 100% (194/194), done.
remote: Total 1584 (delta 270), reused 301 (delta 206), pack-reused 1181
Receiving objects: 100% (1584/1584), 2.12 MiB | 13.11 MiB/s, done.
Resolving deltas: 100% (964/964), done.
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/puro'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/puro.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/packages/puro' failed
Failed to clone 'packages/puro'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/rps'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/rps.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/packages/rps' failed
Failed to clone 'packages/rps'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/storage'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/storage.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/packages/storage' failed
Failed to clone 'packages/storage'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/use-hashed-state'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/use-hashed-state.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/packages/use-hashed-state' failed
Failed to clone 'packages/use-hashed-state'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/utils'...
remote: Enumerating objects: 60, done.
remote: Counting objects: 100% (60/60), done.
remote: Compressing objects: 100% (46/46), done.
remote: Total 60 (delta 26), reused 47 (delta 13), pack-reused 0
Receiving objects: 100% (60/60), 11.24 KiB | 1.61 MiB/s, done.
Resolving deltas: 100% (26/26), done.
Cloning into '/Users/belarusrulez/Downloads/plasmo/templates/qtt'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/qtt.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/templates/qtt' failed
Failed to clone 'templates/qtt'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/examples'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/examples.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/examples' failed
Failed to clone 'examples' a second time, aborting```

If the src of the iframe in the tab is changed, the background will also be reload.
The content of the current page will become invalid.
The reason is also caused by autoreload.js built automatically.
npm run build no problem

also I tried the same with https:
Failed to clone 'packages/permission-ui'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/prettier-plugin-sort-imports'...
remote: Enumerating objects: 1584, done.
remote: Counting objects: 100% (403/403), done.
remote: Compressing objects: 100% (194/194), done.
remote: Total 1584 (delta 270), reused 301 (delta 206), pack-reused 1181
Receiving objects: 100% (1584/1584), 2.12 MiB | 13.11 MiB/s, done.
Resolving deltas: 100% (964/964), done.
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/puro'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/puro.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/packages/puro' failed
Failed to clone 'packages/puro'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/rps'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/rps.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/packages/rps' failed
Failed to clone 'packages/rps'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/storage'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/storage.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/packages/storage' failed
Failed to clone 'packages/storage'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/use-hashed-state'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/use-hashed-state.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/packages/use-hashed-state' failed
Failed to clone 'packages/use-hashed-state'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/packages/utils'...
remote: Enumerating objects: 60, done.
remote: Counting objects: 100% (60/60), done.
remote: Compressing objects: 100% (46/46), done.
remote: Total 60 (delta 26), reused 47 (delta 13), pack-reused 0
Receiving objects: 100% (60/60), 11.24 KiB | 1.61 MiB/s, done.
Resolving deltas: 100% (26/26), done.
Cloning into '/Users/belarusrulez/Downloads/plasmo/templates/qtt'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/qtt.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/templates/qtt' failed
Failed to clone 'templates/qtt'. Retry scheduled
Cloning into '/Users/belarusrulez/Downloads/plasmo/examples'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:PlasmoHQ/examples.git' into submodule path '/Users/belarusrulez/Downloads/plasmo/examples' failed
Failed to clone 'examples' a second time, aborting```

BTW sorry for late reply - basically you will need to setup a SSH keypair and associate it with ur github account, as our git submodules uses ssh: https://docs.github.com/en/authentication/connecting-to-github-with-ssh

Regarding the losing state and invalidated context of this issue - do people really want their background script to be static thorough development, or do they want the logic to change? - the idea of invalidating the context on development is so that folk can test their new background script logic, otherwise you would have to uninstall/re-install.
On production, it will be a single-spawned background script which won't be reloaded at runtime.
While evaluating this issue, I'm not sure how to fix it completely. Basically during development there're times when the bgsw need to be invalidated for new instance and require hard reset of extension. What's the expected DX for background service worker?

Note that in MV3, it's now background service worker. As a worker, it's prone to die (after 5mins or so) - thus, extension developer must account for the nature of worker and make it short-lived and ensure there's an "entry" logic to the bgsw - e.g who's invoking these background services? - otherwise, do not treat them as permanent, long-living worker as in prod they're not afaik - chrome kills them after certain amount of time see #3

I have hotfix that works for me perfectly. I just delete auto-reload.js file from manifest and refresh the extension. (It doesn't affect reload if code is changed and works as expected)
it would be ok if it would die in 5 minutes of inactivity but it refreshes immediately after message to popup or context was sent and code is doing nothing but waiting for response.

@belarusrulez I see, this would likely go hand-in-hand with #215

resolved in #255


v1.11 broke application menus

Prerequisites

[x] Can you reproduce the problem with Debug -> Reload Without Extensions?
[x] Did you perform a cursory search to see if your bug or enhancement is already reported?
[x] Did you read the Troubleshooting guide?

Description
After updating to the latest version of Brackets, my menus and submenus look like:

After downgrading to Brackets v1.10 the problem went away.
Versions
OS: Ubuntu 17.10 x64
Brackets: Release 1.11 build 1.11.0-17524 (release 86b5e2d4e)

Known issue at #13738, please look at #13731 for resolution.
In essence, to solve it please install libpangoft2-1.0-0 >= 1.40.5 or unpack the libs from the package to path of Brackets installation.

Thanks!


add codeclimate configuration

https://codeclimate.com/github/gagatust/nd4j
Changed:
file-lines: 250 -> 5000
method-count: 20 -> 500
method-lines: 25 -> 50
return-statements: 4 -> 20
It's needed to add nd4j to codeclimate dashboard before.

cc @saudet

Sure, I guess, @sshepel?

@gagatust, @saudet I'm ok with these changes, but just want to note that, in the near future I'm going to add SonarCloud integration with our Jenkins (Sonar has better integration with Jenkins).


Make company-go an interactive Emacs function

The convention for company backends is to make them interactive, so
users can manually invoke them. This is particularly useful when
setting up a backend, to allow the user to explicitly test their
setup.

Merged it without testing.

Thanks :)


Set breakpoints in px, contract mobile break width

Picked up in the recent QPixel changes.
Breakpoints are generally better set in pixels than in em; pixels are a device measurement, whereas em is based on font size, which may change.
We should also expand the "mobile" level break. Moving into a hamburger menu on a tablet is okay, but should not be happening on just a small desktop window. I'd suggest we don't consider a page "mobile" (and thus hamburger-ify it) until around ~850px. QPixel can go down to 820px before there's not enough space for the stuff in the header.

Completed with 850px for header breakpoint; can be changed in _config.scss.


Added support for Zod nativeEnum

Added support for Zod nativeEnum so you don't have to define your enum values in two places.  Added a test to structured.test.ts which includes an Enum definition StateProvinceEnum.  It would be better if there was an enum already in langchainjs that could be referred to, but this at least makes it clear what is going on.

Thanks!

Glad to contribute!  Any chance this will get included in next release?  Hey- can you open Discord DM's to @Baloo#8072?


Fix: Table Wrapping in IE11

Resolves #1063
Proposed Changes

Changed flex-basis property from 15% to 12%, which doesn't change appearance in any browsers except IE11

How to Test

View any tables with drawers in IE11

The Threads Table looks like this to me:

Maybe this might be fixed by my PR which wraps this table in a 100% height DIV.

@spmcbride1201 Should be fixed!


Upon app launch on macOS, key press does not register until user clicks within window

Describe the bug
Upon app launch, key presses do not register until user clicks within window.
To Reproduce
Steps to reproduce the behavior:

Open F3D app
Press "H" to open cheatsheet menu
F3D does not register key press. System error beep sound plays.

Expected behavior
Upon app launch, key presses are registered without user having to click within the window.
System Information:

OS: macOS Sonoma 14.1.2

F3D Information
F3D - A fast and minimalist 3D viewer
Version: 2.3.0-72-gd1952fda.
Build date: 2024-02-27 22:40:01.
Build system: Darwin 64-bits.
Compiler: AppleClang <IP_ADDRESS>00100.
External rendering module: OFF.
Raytracing module: OFF.
VTK version: 9.2.6.
Additional Information

macOS issue

Sounds like a focus issue. Maybe related: https://developer.apple.com/documentation/appkit/nsapplication/activationoptions/1525559-activateignoringotherapps
The fix should probably be in VTK directly, but we might be able to test it easily by putting this call in https://github.com/f3d-app/f3d/blob/7bc53ec1c839012a11fc1709e6b2505911f315b1/application/F3DNSDelegate.mm#L37
See https://stackoverflow.com/questions/5089598/set-nswindow-focused#comment37953271_5089809

@kathleenhang since you are on mac, you should be able to check that easily :)

Thanks! I added the fix  [[NSApplication sharedApplication] activateIgnoringOtherApps:YES]; but it still runs into the same issue.
It seems like the app does gain focus because I can drag the app around my desktop from its mac header bar. However, I still am not able to use the hotkeys.
The hot keys only work once I click within the window.

Fixed here: https://gitlab.kitware.com/vtk/vtk/-/merge_requests/11084

VTK updated here: https://github.com/f3d-app/f3d/pull/1394

Lets wait for the next nightly before closing this.

Checked the nightly, it's fixed


added nemesis

The Pull Request resolves Issue #107
Description:
added nemesis

You need to capitalize the file name, you can check the test logs by clicking the "details" link on the "Checks" window below


[Bug]: Cannot push commits: "Obsidian Git did not send all necessary objects" (push works through terminal)

Describe the bug
Hi :)
I suddenly cannot push commits through Obsidian git anymore after having used it since 2021 with no problems. It still makes commits, but it won't push them - I get exactly the same error message every time (see next box). I had 52 commits at some point that the plugin had not pushed. I can push through the terminal with absolutely no problem.
The error message seems to indicate that I have several branches open (main 2), but I only have one branch which is called main.
Thanks in advance!
Relevant errors (if available)
fatal: bad object rets/remotes/origin/main 2
error: httos://github.com/MinaAlmasi/reponame-
Obsidian.git did not send all necessary objects

Steps to reproduce

I use the command "create backup"
I see the message "X files committed"
I get the error message immediately after (the error message does not change at all)

Expected Behavior
No response
Addition context
I have the latest version of Obsidian (version 1.0.0) but the issue was also prevalent on version 0.15.9
Operating system
macOS
Plugin version
2.8.0

I had the same issue but it seems to have gone away in 2.9.0, although I haven't tested it rigorously yet

After some more digging, it seems like the problem arises when there are extra branches in .git/refs/remote/origin/. I took a peek and found an extra main 2 that was created yesterday (around when the problems began arising). Deleting that fixed the issue for me.
I got my info from here:
https://stackoverflow.com/questions/8788975/bitbucket-git-error-did-not-send-all-necessary-objects/70957667#70957667

Thank you!!! Deleting main 2 worked !!!

Hm that's strange. Where did that branch come from? Maybe from another sync service? I'm closing this now as it seems solved for you.

I have no other sync device or anything. The branch did not appear when writing git branch -r and I could not remove it with git rm (nor with git symbolic-ref -d). Was able to find it with git prune and remove it by writing git prune origin. But everything works now! Thank you!

@Vinzent03 I'm using Obsidian Git concurrently with iCloud (probably a bad idea, but I like having backup systems). I'm guessing iCloud saw 2 different mains and decided to add in an extra one. The weird thing is that I don't recall pulling from my repo, so I'm not sure exactly what's going on.


Why don't use softmax before cross entropy in multibox loss?

Hi,
I'm recently doing some modifying on your code. I found that you didn't use softmax before cross entropy in multibox loss which has been used in other SSD implements and alse mentioned in the paper.
However, I tried change it and re-train model without any pre-trained model on VOC 2007. The loss log confused me that softmax makes it converge slowly and seems worse than no-softmax. Can you explain why you remove softmax?  Thx.

you can see here, torch.nn.functional.cross_entropy() combines log_softmax and nll_loss in a single function.


Update Master branch with test-patch

Remodeling of blast
First test for project management
Data management and database management.

:wink:


Newer scalameta-parsers npm artifacts

https://www.npmjs.com/package/scalameta-parsers (added in https://github.com/scalameta/scalameta/pull/949) is currently stuck at 4.4.17, making https://astexplorer.net/ not as useful as it could be for Scala 3.
Bumping & publishing seems manual. I am not familiar with npm but it looks like publishing could be automated by:

running some smoke tests (the example from https://github.com/scalameta/scalameta/blob/main/scalameta/parsers/js/npm/README.md could be a good start I guess).
on tagged release, having a script populate the package.json and publishing it

I managed to run AST explorer with scalameta 4.5.5 locally
scalameta

bumped scalameta version in https://github.com/scalameta/scalameta/blob/main/scalameta/parsers/js/npm/package.json
npm publish; npm pack -> tgz

AST explorer

followed https://github.com/fkling/astexplorer#build-your-own-version-for-development
yarn add <path/to/tgz>

@gabro any idea what would be the best workflow here? Did you publish it all manually before?

I confirm it's been all a manual process so far, and I agree it can be automated with a GitHub action, if someone has the time and will.
Also, the AST Explorer upgrade process is quite trivial, but the release is usually slow since the main maintainer is busy and the process is manual as well.
I'm happy to provide pointers if needed, but I won't have much time myself to work on this task

This should be closed by https://github.com/scalameta/scalameta/pull/2966, let's check during the next release.

The new version of scalameta-parsers is uploaded to npm :tada: https://www.npmjs.com/package/scalameta-parsers, thanks to https://github.com/scalameta/scalameta/pull/2966 we'll publish to npm for every scalameta releases.
Also, I submitted a PR to AST-explorer https://github.com/fkling/astexplorer/pull/685
I'm going to close this issue as completed, but feel free to re-open if there's something left :+1:

Thank you @tanishiking !


Improve search flexibility for results with lower "uniqueness"

Currently in Version 2 the search sometimes does not return results where there are too many results that are partially relevant to the query text. This is particularly relevant for searching for persons by title, first, and last name.
There are many people that would match each of those fields, so in some cases the MultiMatch with CrossFields is not going to give enough of a score to people with very common first and last names to get them returned in the results.
This PR adds another MultiMatch module with the MostFields query type - this gives higher scores to records which have many fields matching.
It is important to balance this carefully so as not to provide less relevant records when single fields do match, so the boost has been set to 2 - much lower than the 6 for MultiMatchSingleField. From local testing this seems to give reasonable results

Idk... I feel like the solution is to exclude titles from search queries and search itself. What value does titles search really bring?

This isn't a problem specific to titles - it's a general problem where the more fields they included, the less relevant the results were, until they stopped appearing completely. This PR makes it so adding more fields gives more relevant results which is how it should be


Token gallery collection filtering options

When the token gallery is expanded, enable options to filter by collection:

Collection drop down menu
Visibility (show/hide) at collection level

Screenshots

/gcbrun

/gcbrun

/gcbrun


KinematicCharacterController.setMaxSlopeClimbAngle is not working

I have a character controller and a trimesh as terrain. The terrain is mostly flat but has some heals and walls
The character controller climbs on top of even the steepest hills, and the setMaxSlopeClimbAngle value does not change a thing.
rapier3D, v0.13.1, javascript

Hi! Can you provide more information on your setup? In particular, the piece of code where you initialize and call the character controller’s methods, and how you apply it to the corresponding rigid-body. What is the geometric shape of your character?

here's the code where I create and move the character controller
setting any value to setMaxSlopeClimbAngle has no effect: the character climbs any slops that less than 90 degrees
Geometric shape is a capsule.
create() {
        const RAPIER = pc.app.rapier;
        const offset = 0.01;

        this.controller = this.world.createCharacterController(offset);
        this.controller.setMaxSlopeClimbAngle(0 * Math.PI / 180);

        const rigidBodyDesc = new RAPIER.RigidBodyDesc(
            RAPIER.RigidBodyType.KinematicPositionBased,
        );

        this.body = this.world.createRigidBody(rigidBodyDesc);
        const position = this.entity.getPosition();
        this.body.setTranslation(position);

        const colliderDesc = RAPIER.ColliderDesc.capsule(0.8, 0.8);

        colliderDesc.setCollisionGroups(GROUPS.PLAYER);
        this.collider = this.world.createCollider(colliderDesc, this.body);

        this.collider.userData = { userID: this.app.userInfo.userID, isPlayer: true, isSelf: true }
        this.entity.rigidBody = this.body;
        this.world.character = this;
    },
        move(input, dt = 0.01666) {
        if (!this.body) return;
        const direction = input.inputMove;
        const current = this.body.translation();

        this.vector.x = direction.x * input.speed * dt;
        this.vector.y = 0;
        this.vector.z = direction.z * input.speed * dt;

        this.controller.setMaxSlopeClimbAngle(0.5);
        this.controller.computeColliderMovement(this.collider, this.vector, null, GROUPS.BULLET | GROUPS.PLAYER);

        const correctedMovement = this.controller.computedMovement();

        this.vector.x = current.x + correctedMovement.x;
        this.vector.y = current.y + correctedMovement.y;
        this.vector.z = current.z + correctedMovement.z;

        this.body.setNextKinematicTranslation(this.vector);
        const newPosition = this.body.translation();
        this.entity.setPosition(newPosition.x, newPosition.y, newPosition.z);
    },

Small update. I was able to reproduce the issue. This is mainly caused by the fact that there is no gravity being applied this.vector.y = 0. But I do consider this a bug that should be fixed.

I have rapier simulations on both the server and the client. I try to run them in sync with each other. Determinism helps, but when I apply gravity, I start having issues - the collision resolution with the ground seems very erratic and unpredictable, and the more gravity you apply, the more the two simulations diverge from each other. So it required me to apply too much server reconciliation, which was not ideal. I found the only solution was to stop applying gravity, but then this bug happened. I kinda solved it by raycasting the ground and applying gravity only if the character was getting higher, but yeah, that sucks. Would be great if it is fixed and I could use setMaxSlopeClimbAngle to handle this.

Same result with @khudiiash, continuous apply gravity simulate into KCC make rigid body movement erratic and unpredictable (sometimes pass through the ground, loss collision event when 2 obstacles shared same edge and unacceptable is even when the ground flat, character sometimes jump a little bit and fall down again...)
Hence, i must stop apply gravity when character is standing on obstacle for more stable, but it make an issue that when character running down slope, it jerky horizontal movement followed by a fall


It seems that new android studio version has removed lint in build script

lint' not found in root project 'abc'. Some candidates are: 'init'


feat(sdk): add login and logout sdk

Description
Modules

[x] Python-SDK

Checklist

[x] run code format and lint check
[x] add unit test
[ ] add necessary doc

Is the concept of session more friendly to multi-user operations? Or optimize later

Is the concept of session more friendly to multi-user operations? Or optimize later

no session plan. The current login information is isolated by the Linux Account.  @goldenxinxing


favicon related changes

Changes pertaining to favicon addition and slight related responsive tweaks

I added the favicon and the logo beside sigmaMind to the about page just like in index.html. I'm not able to understand what you are referring to