vault: 1.5.0 -> 1.5.2

Motivation for this change
Things done

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

[X] NixOS
[X] macOS
[ ] other Linux distributions

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

tested with nix-review, vaul nixos test runs successfully.


Fix StreamMetadata segfault

I've updated this PR with some change notes and added an extra comment preceding the member variables in the stream-related classes. If this can be approved by the end of this week, I'll cherry-pick this change back into a 6.2 branch and initiate the release on Monday.


Implement articleRepliesFrom filter for ListArticles

Fixes #165

Implements articleRepliesFrom in #165

Drops appId because userId should be sufficient according to userId / appId management proposal
I cannot think of any useful use case filtering article-reply's appId yet

Fixes categoryIds and hasArticleReplyWithMorePositiveFeedback, which unions its results when used with other filters and is incorrect

Similar to #171, please see #171 for explanation.

Coverage increased (+0.4%) to 86.933% when pulling<PHONE_NUMBER>e2bbdead400cfd8e234e7cafad3b20 on user-filter into 1ef3790da93aec2ee26426d5561766ff1708ae13 on master.


Add some tests for formatDateRangeWithMessage; fix a bug with the last day of exhibitions

This was a precursor for getting Moment out of this function, but it also uncovered a bug -- because isBetween() does an exclusive range (i.e. a < x < b rather than a ≤ x ≤ b), we'd show "Now on" for an exhibition that was on its final day.  This was caught by the test, and is now fixed.
This is rather important as we have an exhibition closing on Sunday!
Who is this for?
People with an inability to plan in advance.
What is it doing for them?
Making sure they know it's their last chance to see an exhibition.

Part of https://github.com/wellcomecollection/wellcomecollection.org/issues/7831


entity framework  support

https://github.com/lanfengqi/Nut.NET.Skeleton.git

Thanks for creating a Skelton app! We'll try and keep this skeleton app down to the basics not tied to any specific implementation out of the box. I'm curious if you are using our repositories with entity framework?


Entire screen considered dirty

I've been working on a pixelflut client that uses evdi to present a virtual monitor onto which a user can drag whatever content they'd like. The contents of this virtual monitor are sent to a running pixelflut instance.
evdi seems like a perfect DRM driver quickstart for this sort of thing, but when I request a framebuffer update, the dirty rectangles array always comes back with a single element indicating that the whole screen is dirty. Updating the whole display is simply too slow for the pixelflut protocol, which is inefficient by design.
I've tried a couple different EDIDs, and both values of the module option enable_cursor_blending. I've disabled my compositor and redshift. I have not tried different window managers.
I've even instrumented the monitorsim program from evdicpp to see if its behavior is the same, and it is.
Could you point me in the right direction? Does the driver have support for dirty rectangles smaller than the entire screen? If so, is there anything else that might be interfering on my machine?
Thanks!

Hi @qguv,
the base idea is that evdi keeps track of dirty rectangles and sends them to the
userspace. Note that there is assumption for max 16 rects in buffer - we try to
merge two into one if the overal size is smaller or we will simply callapse all
into one if the number still exeeds the buffers limit (it is done in
evdi_painter_mark_dirty).
That is the idea but it heavily depens on how we get the pixels.

FB - it handles three callbacks:

evdi_fb_fillrect
evdi_fb_copyarea
evdi_fb_imageblit
The dirty rect is whatever we get from the userspace there. This is old one
and probably not in heavy use anymore.

dirty_fb - one callback:

evdi_user_framebuffer_dirty
Same as FB this one will use dirty rects which comes from userspace.
Note: from kernel 5.0 there is upstream version of it atomic helper for this one:
drm_atomic_helper_dirtyfb

atomic - new one, most important callback here would be probably:

evdi_plane_atomic_update
Here, if we get new framebuffer from kernel we mark fullscreen dirty.
It will also be called if you get dirty_fb callback.
The official drm_atomic_helper_dirtyfb supports damage rects which we
could use. If you are falling into this one it might be possible to improve (still
depends on the use userspace).

Can you check what is the source of dirts in evdi? dirty_fb, atomic or (not
likely) FB. Can you also say form is your environment (OS, display manager,
etc) and kernel version?

Thanks for the info @displaylink-dkurek, I'll instrument some of the evdi callbacks and get back to you.
I'm running Arch on kernel 5.0.13 (with this Kconfig) with evdi version 1.6.0 (commit 09e6dd50c5615d2ac08432c372134d042ae4f7a5). My window manager is dwm 6.1 (with a couple patches).

Hi @qguv
Have you any luck with this?
I'm not familiar with the pixelflut protocol but is setting pixel by pixel the only way of
sending image?

I'm currently on holiday but I will get back to you soon about the results of instrumenting those calls.
Yes, pixelflut is a pixel-by-pixel protocol used to control a display at hacker events. It's simple and inefficient to simplify client implementation and encourage people to learn more about how their TCP/IP stack works and how it can be configured. The inefficiency also means smaller images perform better.

I'm currently on holiday but I will get back to you soon about the results of instrumenting those calls.
Yes, pixelflut is a pixel-by-pixel protocol used to control a display at hacker events. It's simple and inefficient to simplify client implementation and encourage people to learn more about how their TCP/IP stack works and how it can be configured.


[gui error report] Error: Initializing daemon...

👉 Please describe what you were doing when this error happened.
Specifications

OS: win32
IPFS Desktop Version: 0.20.6
Electron Version: 18.0.3
Chrome Version: 100.0.4896.75

Error
Error: Initializing daemon...
go-ipfs version: 0.12.2
Repo version: 12
System version: amd64/windows
Golang version: go1.16.15
Found outdated fs-repo, migrations need to be run.
Looking for suitable migration binaries.
Need 1 migrations, downloading.
Downloading migration: fs-repo-11-to-12...
Fetching with HTTP: "https://ipfs.io/ipfs/QmdaCHYBDHEhXCMoynH5UcohEay6m1XayZCcxWZzKAHNVN/fs-repo-11-to-12/versions"
Fetching with HTTP: "https://ipfs.io/ipfs/QmdaCHYBDHEhXCMoynH5UcohEay6m1XayZCcxWZzKAHNVN/fs-repo-11-to-12/versions"
Fetching with HTTP: "https://ipfs.io/ipfs/QmdaCHYBDHEhXCMoynH5UcohEay6m1XayZCcxWZzKAHNVN/fs-repo-11-to-12/versions"
Error fetching: exceeded number of retries. last error was http.DefaultClient.Do error: Get "https://ipfs.io/ipfs/QmdaCHYBDHEhXCMoynH5UcohEay6m1XayZCcxWZzKAHNVN/fs-repo-11-to-12/versions": dial tcp <IP_ADDRESS>:443: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
Fetching with IPFS: "fs-repo-11-to-12/versions"
Error fetching: context deadline exceeded
could not get latest version of migration fs-repo-11-to-12: 2 errors occurred:
	* exceeded number of%

Do you have an internet connection while starting ipfs-desktop? It needs to fetch a program to update your local storage – which failed 3 times in a row.

I see Windows – @skins2000 are you running any firewall  / antivirus which could be blocking p2p traffic?


jnp.vectorize causes FloatingPointError: invalid value (nan) when there is no nan returned.

Description
Hi,
My vectorized function complains FloatingPointError: invalid value (nan) encountered in jit(pow). when jax_debug_nans is set to True. The function returns no nan if this environment variable is turned off. Is there a potential bug causing the issue?
Below is the sample code --
import jax
import jax.numpy as jnp

jax.config.update("jax_debug_nans", True)

@jnp.vectorize
def func(x_e):
    return jax.lax.cond(
            x_e > 0,
            lambda: jnp.power(1.0 + x_e, -0.75),
            lambda: jnp.power(1.0 + x_e, -2.0),
        )

x = jnp.array([-3.6927774, 1.5781837])

# Would print "[0.13791107 0.49148992]" if jax_debug_nans is turned off
print(func(x))

It will throw out the error --
--------------------

The above exception was the direct cause of the following exception:

jax.errors.SimplifiedTraceback: For simplicity, JAX has removed its internal frames from the traceback of the following exception. Set JAX_TRACEBACK_FILTERING=off to include these.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Users/jian449/Library/CloudStorage/OneDrive-PNNL/Codes/jax-watershed/examples/debug_nan.py", line 112, in <module>
    func(x)
  File "/Users/jian449/anaconda3/envs/jax-canoak/lib/python3.10/site-packages/jax/_src/numpy/vectorize.py", line 321, in wrapped
    result = vectorized_func(*squeezed_args)
  File "/Users/jian449/anaconda3/envs/jax-canoak/lib/python3.10/site-packages/jax/_src/numpy/vectorize.py", line 138, in wrapped
    out = func(*args)
  File "/Users/jian449/Library/CloudStorage/OneDrive-PNNL/Codes/jax-watershed/examples/debug_nan.py", line 104, in func
    return jax.lax.cond(
FloatingPointError: invalid value (nan) encountered in jit(pow). Because jax_config.debug_nans.value and/or config.jax_debug_infs is set, the de-optimized function (i.e., the function as if the `jit` decorator were removed) was called in an attempt to get a more precise error message. However, the de-optimized function did not produce invalid values during its execution. This behavior can result from `jit` optimizations causing the invalid value to be produced. It may also arise from having nan/inf constants as outputs, like `jax.jit(lambda ...: jax.numpy.nan)(...)`.

Thanks in advance!
Peishi
System info (python version, jaxlib version, accelerator, etc.)
System information --
jax:    0.4.26
jaxlib: 0.4.26
numpy:  1.26.4
python: 3.10.0 | packaged by conda-forge | (default, Nov 20 2021, 02:27:15) [Clang 11.1.0 ]
jax.devices (1 total, 1 local): [CpuDevice(id=0)]
process_count: 1
platform: uname_result(system='Darwin', node='WE46779', release='23.6.0', version='Darwin Kernel Version 23.6.0: Mon Jul 29 21:14:30 PDT 2024; root:xnu-10063.141.2~1/RELEASE_ARM64_T6000', machine='arm64')

When you vectorize lax.cond it becomes lax.select which evaluates both branches and discards one, but in your case it would still create a nan as an intermediate value due to the fractional power of a negative number

When you vectorize lax.cond it becomes lax.select which evaluates both branches and discards one, but in your case it would still create a nan as an intermediate value due to the fractional power of a negative number

Thanks for the reply! The conversion to jax.lax.select when using vectorize or vmap is likely the reason. Is there a way to get around this? Note that I intentionally used the conditioning to avoid calculating the fractional power of negative values.

Usually some version of the "double where trick": https://jax.readthedocs.io/en/latest/faq.html#gradients-contain-nan-where-using-where

Usually some version of the "double where trick": https://jax.readthedocs.io/en/latest/faq.html#gradients-contain-nan-where-using-where

Thank you for the suggestion. In a similar spirit, I can now resolve this issue by changing the codes as --
import jax
import jax.numpy as jnp

jax.config.update("jax_debug_nans", True)

@jnp.vectorize
def exp_func(x_e):
    return jax.lax.cond(
        x_e > 0,
        lambda: -0.75,
        lambda: -2.0
    )
def func(x):
    return jnp.power(1.0+x, exp_func(x))

x = jnp.array([-3.6927774, 1.5781837])
print(func(x))


Document how to login as guest user

The README mentions one can check a usage of this plugin if they login as the "guest user" on build.gocd.org. Can you please add a small sentence (or link) to how that can be done?

Hello! There is usually a plugin on build.gocd.org, which adds a "Guest" button, but it seems to be gone ... maybe because it is built with a wrong version of Java or something. For now, you can log in using the username "view" and password "password" (it is meant to be accessible, so the password is no big secret and mentioned previously).
I think, in this documentation, it can just show an image of that pipeline group. Once you see that group in build.gocd.org, you'll see what I mean. I think we can replace that reference to build.gocd.org in the README with an image.

@arvindsv I still like the possibility of logging in and playing around with GoCD's UI, checking the dashboards and pipelines, etc. If view/password are valid credentials (and remain so), could we instead alter the README to mention these instead of the "guest mode"?

That's fair. I tried to see why the guest login plugin did not show up. Need to look deeper. The username/password combination should continue to be valid, but the button should be easier for everyone.

I see that the guest login button is back! Closing this. :)
PS: I didn't fix it. Someone / upgrade must have fixed it.

Thanks - even though I still think providing an alternative in the README (if this ever happens again) could prevent others from opening similar issues.


[android] enable Thread 1.2 backbone router multicast routing for Android

Close since we don't do this for all OTBR builds


warning for React.PropTypes validation

I get this warning using it...
Warning: You are manually calling a React.PropTypes validation function for the formAPI prop on Form. This is deprecated and will not work in production with the next major version. You may be seeing this warning due to a third-party PropTypes library. See https://fb.me/react-warning-dont-call-proptypes for details.
Does someone can help me with that?

Are you calling the prop type validation yourself?
On Sat, Jan 14, 2017 at 6:30 PM sabatsu<EMAIL_ADDRESS>wrote:

I get this warning using it...
Warning: You are manually calling a React.PropTypes validation function
for the formAPI prop on Form. This is deprecated and will not work in
production with the next major version. You may be seeing this warning due
to a third-party PropTypes library. See
https://fb.me/react-warning-dont-call-proptypes for details.
Does someone can help me with that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-form/issues/27, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AFUmCeUVitqWlO4oLPTJBoNQTcDoboYuks5rSXbRgaJpZM4Ljy7-
.

Closing for now.  If you have more questions, let's continue the discussion in the closed issue.

No i am not.

Hey @tannerlinsley ...
could you please help me with that?
I am getting this warning -
Warning: You are manually calling a React.PropTypes validation function for the zoom prop on ReactCropper. This is deprecated and will not work in production with the next major version. You may be seeing this warning due to a third-party PropTypes library. See https://fb.me/react-warning-dont-call-proptypes for details.

Hi @sabatsu.  React-form doesn't utilize any prop-types anywhere in it's source code, and it's dependencies are also free of its usage as well.  I can't be sure, but it sounds like you are using a component that does.  https://facebook.github.io/react/warnings/dont-call-proptypes.html for more information.
If you could please reproduce the problem in this webpack-bin, that would extremely helpful in figuring out the solution :)


fix ticker leak

Now the throughput won't decrease gradually.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Ubuntu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


What dataset the model was fine-tuned on?

The results look more like artistic / computer graphics generated images than natural images. Can you briefly describe what datasets were used to fine-tune this model?

Was trained on LAION-Aesthetics V2 6+ https://laion.ai/blog/laion-aesthetics/


Tami4 Light Toggle

Great plug-in!
Is it possible to add a light toggle to the device?

Hi! Thanks :)
Do you mean this configuration on the app?

If so, then yes. Everything that you can configure through the app can be called through the API.
Do you have the knowledge to do it yourself and open a PR?


Add Joda time logical type conversion.

Motivation
After upgrade to Apache Avro 1.9.x, the default time conversion changed to JSR-310. For forwarding compatibility, we'd better add the Joda time conversion.
related to #5938
Modifications
Add joda time conversions
Verifying this change
New integration test added
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

Does this pull request introduce a new feature? (no)

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


native: more empty array checks on insertGroups

More prevention against running array based DB operations when the array is empty. In particular, this targets insertGroups which is a spot where we're seeing crashlytics start sync issues. Not positive this will resolve those, but I think this is worth including in the next build and should be a low risk change.
Fixes TLON-2224 (hopefully)

Could be the move. Thanks for tracking this down.


Add language - Expression 2

Had massive problems trying to figure out how to do positive lookaheads, just couldn't make it work. Ended up coloring all untokenized text as purple (function color), and tokenize not just all keywords and typenames, but also all operators and symbols. Might be a little expensive when code gets large, but at least it works.

@revosw what's this "relevance" field for? prioritizing?
If so, you may be able to use that instead of lookahead

@revosw
First of all, truly sorry for the long delay in anyone responding to this PR.
Thank you so much for desiring to contribute and writing a language grammar.  Unfortunately, we currently are no longer accepting new languages into the core repository.  The existing maintainers simply don't have time to maintain new languages (fixing bugs, dealing with issues, etc).  We'd still love to host your language as a 3rd party module though - if you're willing to help maintain it.
It's easy to turn your language into a module that anyone who likes can easily integrate with Highlight.js using a single line of code:
<script src="/path/to/highlight.min.js"></script>
<!-- they'd simply add your language modules -->
<script src="/path/to/highlightjs-expression2.js"></script>
<script>hljs.initHighlightingOnLoad();</script>

It's also possible for users to rebuild the highlight.js library to include specific 3rd party modules, all in a single file (vs loading a bunch of 3rd party languages one at a time).
If you'd like to become the maintainer for this language then we're happy to create a new repository in the highlightjs organization and host the repository there.  That assumes you'd be open to reviewing issues and fixing bugs as they arise.  You're also welcome to host the repo yourself and we'll link to it from our README.
There isn't much in the way of how to docs on this yet, but there are sample repositories and it's honestly pretty easy.  We're still fleshing out an official "spec" for 3rd party modules, but even after that's done it should require minimal changes from the examples we have already.  IE, if you wanted to get started now there isn't any big reason to wait.
My own robots.txt grammar repository is a great, simple example:
https://github.com/highlightjs/highlightjs-robots-txt
If you have any questions feel free to ask.

Had massive problems trying to figure out how to do positive lookaheads, just couldn't make it work.

There were quite a few bugs/issues with lookahead support, all of the known issue have been fixed in the past month or two though - so if you try again you might have better luck with lookaheads.
Let me know if I can help.

Closing for inactivity; (also finally cleaning all these old PRs out for the new year).
As mentioned above if you're interested in becoming the maintainer of a 3rd party module, please respond [whenever you find time] and we'd be happy to help you get started.


Update Dockerfile dependencies for local development and CI

Issues:

Dockerfile chromium and nss versions were no longer supported.
Missing gcompat library for running on a Silicon chip.

Changes

[x] Add latest working chromium and nss versions to Dockerfile.dev and Dockerfile.test
[x] Add gcompat library needed for Apple Silicon development
[x] Commit updated Gemfile.lock and package-lock.json for dummy

Notes

Should I open this against main, develop or 3_2_0_beta?

@aaron-contreras do you know wh the ruby 2.6 ci containers faill?

@aaron-contreras do you know wh the ruby 2.6 ci containers faill?

I can pick up from the failures that there's a clash between the puma and capybara versions used in those Gemfiles. But since they're unversioned and there's no lock file for it, I'm unsure as to how to solve this.

2.6 is EOL https://endoflife.date/ruby
We should simply remove these test containers and add ones for newer ruby versions 3.1 and 3.2 instead? @aaron-contreras what is your opinion?

2.6 is EOL https://endoflife.date/ruby
We should simply remove these test containers and add ones for newer ruby versions 3.1 and 3.2 instead? @aaron-contreras what is your opinion?

@jonasjabari Agreed. I'll also track down sections throughout the documentation where we reference which Ruby/Rails versions the gem is tested against and update accordingly.

@jonasjabari Coverage for Ruby 3.1 and 3.2 added. Some Gemfile dependencies updated as well as upgraded the ruby versions for local development. The docs portion listing the Ruby/Rails versions the gem is tested against has been updated as well.


Revolver showing as deagle

this.
https://i.imgur.com/ZNa5CCF.jpg
Also, why is the E uppercased checked through the code saw no way to change it there prob is but I might just be dumb.

did u pull this update?
https://github.com/MarkHC/CSGOSimple/commit/552c2758e9789544b088d3cb288a45deed4b98f5

Yes, I used that update still shows as a deagle.

the guy that updated the item definition enum fucked up, i already made a pull request for an updated one

Thanks.


package: update dependencies and add core-js

Fixes #11
/cc @onitake @GabrielBrascher - can you test it, it works on Ubuntu

Thanks for the PR @rhtyd, I will be testing it soon!

Works for me!
I haven't tested everything, however. Only loading, login and a bit or resource browsing.


Use R2 for Relay

We're switching from S3 to R2.

@tillkruss You could redirect requests to S3 to R2 and keep the bucket for 1 more year.
https://docs.aws.amazon.com/AmazonS3/latest/userguide/how-to-page-redirect.html

@szepeviktor: Sadly won't work in our case.


[CI] Jenkins is down

The Jenkins master node (xgboost-ci.net) has been down (*) for a few days, according to @trivialfis. Impact: Substantial amount of test suite is rendered unavailable. I'm re-building the master from scratch. Sorry for inconvenience caused. @dmlc/xgboost-committer
PS. This seems to be a good opportunity to document all steps it takes to set up the Jenkins master node from scratch. See hcho3/xgboost-devops#6. Also #4958
(*) I cannot either ping or SSH into the master instance.

Travis seems also down as there is a permission error.  Will try to resolve it later.

Travis is fixed in #5062

@trivialfis Thanks for fixing Travis! I'll get Jenkins back up very soon.

Update: Linux pipeline is now back up. I still need to work on the Windows pipeline.

Okay.  I messed up a little.

@trivialfis I updated https://github.com/hcho3/xgboost-devops/issues/6 to include some details about the Windows pipeline. I'm also closing this issue, as Jenkins is now fully back up.

Sorry, looks like I called it too early. Windows pipeline is still broken.

Fixed in #5078


Create Insta_reel_downloader(Hactoberfest)

Thanks for your Contribution!
Keep Contributing!
Gove a Star to this project if you like this!
Happy Hacktober!


config: Retire me from ti-community-infra admin

@Mini256 @sykp241095 I'm very happy that I've been involved in these efforts in the past, and I probably won't be involved in too many decisions in the next period, but I should keep contributing code. But I will do my best to do a good job of handing over the work.

/lgtm

/assign @Mini256
PTAL~ Thanks!

/lgtm

/merge


Add live image updates for current status of Repo

Pull Request
This PR is for a new functionality as suggested in and to close #269
This runs a bunch of R scripts on each merge into master, and displays this on the README (and in the future the website).
(Sorry for the loads of commits, this took a lot of trial and error and still don't undersand commit squashing/rebase...)

LGTM.
You can remove the install_load_packages.R file that is no longer needed

Keeping it in for now as it might be useful if people wanted to generate the images themselves but aren't comfortable with conda (I'll write some documentation for it in the future though).


jQuery load method is deprecated

Issue Summary
The jQuery load method is deprecated per jQuery docs and is in use here.

Thanks for the report @anselmbradford. Could you also detail why you consider this a bug – is there anything broken in one of your project?
As for a fix, as far as I understand all it would take is to replace this shortcut method with .on( 'load', handler). Could you provide a PR fixing this?

Fixed in #3744

Thanks @gasman!


Scenario issues

High Level Description
This is on the merge_exit map. Expected waypoints to behave like that shown here. But got the below issues in chronological order:

In on-ramp, agent sees three waypoint paths. [ok]
In on ramp, three waypoint paths extends across the junction. [ok]
In junction, agent sees single waypoint path which jumps around. [probably scenario issue]
After merge in highway, waypoint paths end abruptly. [probably scenario issue]
In highway, three short and thick lines appear. [probably scenario issue]
In highway, at the very end of the gif, obs.waypoint_paths is empty, causing the episode to crash. [probabaly scenario issue]

Version
branch: master
version: v1.0.11
Steps to reproduce the bug
Replay the `merge_exit` scenario and visualize the waypoints.

System info
No response
Error logs and screenshots
No response
Impact (If known)
The merge_exit scenario is unusable.

@Adaickalavan The issues are mostly due to overlapped edges around the junctions, if the edges are overlapped, the waypoints are overlapped as well, which leads to:

confusion on picking the right waypoints path (jump around issue)
dense, crowded waypoints in a small edge (three thick lines)

To fix this,

modified the geometry endpoints for most of the edges to minimize the overlapping around the junction so that the waypoints are generally clearer and smoother.
extend edges/junctions a bit for same reason above, and also can give more room/time to access the upcoming waypoints in the observations

Please take a look at the following gif and let me know if this is acceptable, I have already pushed this map onto the repo, thanks!

More scenario issues on straight map.

Use entry tactic method to ensure lead vehicle appears before ego, rather than map design methods.
Check and rectify all scenarios for similar issues.

The modified merge_exit map seems to use junctions with lane-change connections to specify the locations where the lead vehicle could change lane, albeit the decision to change lane is random. Since the above gif uses an EndlessMission agent, the same set of waypoints will be seen by the ego agent. Essentially, the modified maps reveal a degree of privileged info to the ego on where potential lane changes by the lead vehicle could happen, in contrary to the idea of the task where ego should not have any prior knowledge of the leader's trajectory.

More scenario issues on straight map.

Use entry tactic method to ensure lead vehicle appears before ego, rather than map design methods.
Check and rectify all scenarios for similar issues.

We haven implemented such entry tactic yet, i believe it's in progress on PR#1965.

On the modified merge_exit map, we see the following:

The lead vehicle waits too long at the merge such that the episode terminates due to reaching max_steps_per_episode.
The background traffic seems to always change lane to left-most lane prior to the merge location. This severely reduces any traffic interaction this scenario presents to the ego.
Background traffic vehicles and lead vehicle seems to akwardly gain speed between the newly introduced consecutive lane-change junctions in the main road.

On the modified merge_exit map, we see the following:

The lead vehicle waits too long at the merge such that the episode terminates due to reaching max_steps_per_episode.
The background traffic seems to always change lane to left-most lane prior to the merge location. This severely reduces any traffic interaction this scenario presents to the ego.
Background traffic vehicles and lead vehicle seems to akwardly gain speed between the newly introduced consecutive lane-change junctions in the main road.

re 1): There can be two possible solution: 1) reduce the traffic density 2) give the on-ramp higher road priority, which would lead to all traffic vehicle stops at the intersection when the leader is merging in. 3) put a traffic light on that junction.

Please let me know which one is prefered, imp, it depends on how we see this scenario as, if we see this as a hiway scenario, then reducing density might be appropriate. if it's just a urban road with some merging junctions, we might want to add traffic light or modify the road right priority instead.

re 2): I dont see this happens on my end, it's pretty random as shown below
re 3): I reported this to @Gamenot already, we checked every speed config/info in those files, the cause is unknown and might be a sumo issue.

I have pushed several map changes and tested them, please take a look and let me know if any more change is required, thanks @Adaickalavan !

I reduced the traffic density a bit, the current traffic looks reasonable to me and doesnt block the leader to merge in


Possible bug introduced with PHP-7 compat

Re #229
It's either this PR or #230 which is correct course of action. Please advise and merge accordingly. Need immediate release for this :(
Thanks.

Tested this, and does not work. I get the following error:
Cannot instantiate Twig extension 'TwigBridge\Extension\Laravel\String': Class TwigBridge\Extension\Laravel\String does not exist'
As explained in https://github.com/rcrowe/TwigBridge/pull/230 this solution does not fix the problem.

Have merged #230


Removes rails 3 dependencies, 4.2.0 instead

Hello,
With the move to version 0.4.0 I would like to suggest removing any rails 3.x dependencies.
I suggest 4.2.0 in this Pull Request but I'm happy to change it to other version.
I hope it makes sense.
Thank you

@steveklabnik is there anything you would like me to do in this PR ?
Thank you

I'm not sure what ditching Rails 3 buys us, really.

@steveklabnik I think that everyone still has the chance of using rails-api on rails 3 by using the previous version.
I don't see why you would maintain two or three different versions of rails in this codebase, which is what's happening, there's code for rails 3, rails 4 and already conditionals for rails 5.
By removing what is old you are just avoid maintaining an unnecessary debt, that's why in my opinion we should remove that dependency. In the end of the day that's why Gemfile exists, to easily update old gem versions or keep in a specific one for any kind of reason.
Answering your question: The benefit in my opinion is less code to take care and maintain :smiley:
Thank you

+1. I believe that Rails' 3.2 lifespan is nearing the end. And this is nice step towards removing clutter.

You have committed a swap file by the way.

Thanks @EduardoBautista, I've just removed them.

I'm thinking about this and I will close the PR for now until we conclude something.


Update progress bars in PGDs and FBF adversarial trainer

Signed-off-by: Beat Buesser<EMAIL_ADDRESS>Description
This pull request updates nested progress bars in all PGD implementations and the Fast is Better than Free adversarial trainer.
Fixes #488
Type of change
Please check all relevant options.

[x] Improvement (non-breaking)
[ ] Bug fix (non-breaking)
[ ] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

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
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Codecov Report

Merging #603 into dev_1.4.0 will decrease coverage by 7.42%.
The diff coverage is 33.50%.


[GLUTEN-4942][VL] refine vcpkg package script

What changes were proposed in this pull request?
this patch refines the vcpkg package script
How was this patch tested?
no tests required

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_5900_time.csv
log/native_master_05_29_2024_588faae35_time.csv
difference
percentage

q1
33.61
33.56
-0.047
99.86%

q2
21.89
23.85
1.956
108.94%

q3
36.21
37.37
1.164
103.21%

q4
33.09
32.42
-0.669
97.98%

q5
69.97
69.83
-0.134
99.81%

q6
7.39
7.54
0.149
102.01%

q7
83.53
81.75
-1.779
97.87%

q8
87.25
86.57
-0.680
99.22%

q9
121.79
118.35
-3.443
97.17%

q10
45.44
44.16
-1.283
97.18%

q11
21.64
22.23
0.586
102.71%

q12
26.08
26.72
0.637
102.44%

q13
54.75
53.60
-1.148
97.90%

q14
18.18
17.59
-0.594
96.73%

q15
33.19
32.89
-0.295
99.11%

q16
12.84
13.41
0.562
104.38%

q17
103.98
103.62
-0.359
99.65%

q18
144.05
144.53
0.483
100.34%

q19
14.86
13.59
-1.275
91.42%

q20
32.11
29.66
-2.458
92.35%

q21
267.90
260.43
-7.477
97.21%

q22
12.22
13.84
1.622
113.27%

total
1281.98
1267.50
-14.481
98.87%


Documentation Page

I was wondering if there are plans to add a documentation page like sphinx-documentation to make it easier to navigate the different pieces of the library? If not and it is something you are interested in, I could open a PR :)

Hi @nilsleh, thank you for opening this. It's very high on our priority list to set up documentation. Don't worry about a PR just yet - given that I've written most of the code so far and things are starting to stabilise, I'm planning to finish writing docstrings ASAP so we can set up sphinx. It will then be simpler for others to start contributing, either to the code itself or the documentation.

Thank you so much for offering help, @nilsleh. There's some work towards this taking place in the add-documentation branch.

@nilsleh We have an early first look at the documentation live at https://tom-andersson.github.io/deepsensor. We're planning to add some information about how to contribute to the documentation as well, and I think I speak for both myself and Tom when I say that we would really appreciate contributions here. Example notebooks/scripts/tutorials is definitely something we'd appreciate. Is that something you'd be interested in helping with?

@nilsleh We have an early first look at the documentation live at https://tom-andersson.github.io/deepsensor. We're planning to add some information about how to contribute to the documentation as well, and I think I speak for both myself and Tom when I say that we would really appreciate contributions here. Example notebooks/scripts/tutorials is definitely something we'd appreciate. Is that something you'd be interested in helping with?

@kallewesterling Yes, certainly, I'd be happy to help with anything you deep interesting in that space! So just let me know :)

Closing as we now have the documentation page up and running here: https://tom-andersson.github.io/deepsensor/
Still need to write the introductory pages and short notebook demonstrators! Our first notebook-rendered example is here: https://tom-andersson.github.io/deepsensor/getting-started/tutorials/quickstart.html


Release a version with the new Neos namespace

After migrating to the Neos namespace it will be great with a tag/release that can be referred to in the composer.json file (perhaps alongside the release of Flow 4.0)

Will be done with a 4.0 branch

Fixed with #35


how to demo with webcam?

can this code to demo with webcam?

Deepstream is suited for this purpose.


Handle checkpoints in the absence of database activity

It seems that in the absence of database activity, where only timeout-driven checkpoints are occurring,  bottledwater is not well-behaved as a logical replication stream consumer and can cause its restart_lsn (according to pg_stat_replication_slots) to remain static, most likely causing the server's disk to fill up.
/cc @samstokes

From chat:

See http://www.postgresql.org/docs/current/static/app-pgrecvlogical.html
"The server will occasionally request the client to perform a flush and report the flush position to the server. This setting is in addition to that, to perform flushes more frequently."
The streaming replica is obligated to report a flushed-up-to point using replication protocol messages.
(In addition to wrote-up-to messages)

I think the problem is that BW does so, but only after successfully sending a message to Kafka, but obviously a checkpoint wouldn't result in sending a message.
@ept does this ring any bells?  I should have bandwidth to work on this soon, but I'm less familiar with the semantics of the replication protocol client, so want to check I know what I'm doing before blundering in.

This was closed inadvertently.


Create hi.c

hello world
What the program do ?
it prints hello world
In what programming language it is written?
c language

This pull request do not contains files in specific folders. Please fix it and reopen


New methods/features

Add support for HTTP/1.3 and HTTP/2
Add the NULL Method (Null UserAgent and ...)
Add HTTPS-SPOOF (My method that can bypass CF, DDoS Guard, Etc)
Add Split Args (Example: cfpro https://example.com 1000 30)
Add methods for Layer 4 (TCP, UDP, ACK, RST, XMAS, DVR, ARD, Etc)
Note: I do not force you to add everything I asked you since that is the owner's decision (You could also add features little by little)

I invite u collaborator.
If u not want, declined it.
I'll add method next update


[BUG] System Chart repos failed to syncing after upgraded from 2.7.1 to 2.7.2+

Rancher Server Setup

Rancher version: 2.7.1
Installation option (Docker install/Helm Chart):

Helm Chart v2.7.1 on generic Kubernetes v1.18.20

Proxy/Cert Details: Air-Gapped

Information about the Cluster

Kubernetes version: v1.18.20
Cluster Type (Local/Downstream): Local

Describe the bug

After upgraded from v2.7.1 to v2.7.2+, System Chart repos are in error with message like
Git -C /var/lib/rancher-data/local-catalogs/v2/rancher-charts/4b40cac650031b74776e87c1a726b0484d0877c3ec137da0872547ff9b73a721 fetch origin -- 176424fce4f19ab8b01141dcec2d4453f075aad2 error: exit status 128, detail: error: Server does not allow request for unadvertised object 176424fce4f19ab8b01141dcec2d4453f075aad2
Git -C /var/lib/rancher-data/local-catalogs/v2/rancher-partner-charts/8f17acdce9bffd6e05a58a3798840e408c4ea71783381ecd2e9af30baad65974 fetch origin -- d3ad2ded37158ac75e44db651fc5ad223e3587d6 error: exit status 128, detail: error: Server does not allow request for unadvertised object d3ad2ded37158ac75e44db651fc5ad223e3587d6

While on rancher servers, got logs like:
2023/06/28 09:13:57 [ERROR] error syncing 'rancher-partner-charts': handler helm-clusterrepo-ensure: git -C /var/lib/rancher-data/local-catalogs/v2/rancher-partner-charts/8f17acdce9bffd6e05a58a3798840e408c4ea71783381ecd2e9af30baad65974 fetch origin -- d3ad2ded37158ac75e44db651fc5ad223e3587d6 error: exit status 128, detail: error: Server does not allow request for unadvertised object d3ad2ded37158ac75e44db651fc5ad223e3587d6
, requeuing
2023/06/28 09:13:57 [ERROR] error syncing 'rancher-charts': handler helm-clusterrepo-ensure: git -C /var/lib/rancher-data/local-catalogs/v2/rancher-charts/4b40cac650031b74776e87c1a726b0484d0877c3ec137da0872547ff9b73a721 fetch origin -- 176424fce4f19ab8b01141dcec2d4453f075aad2 error: exit status 128, detail: error: Server does not allow request for unadvertised object 176424fce4f19ab8b01141dcec2d4453f075aad2
, requeuing

When downgraded to 2.7.1, they return to active state.
To Reproduce

Result
Expected Result

The repos can sync normally.
Screenshots

Additional context

I experienced the same thing filed as https://github.com/rancher/rancher/issues/41878 - unfortunately for us a simple downgrade to 2.7.1 didn't work

Interestingly, in one 2.7.5 cluster it happens and in the other not.
Also, I have no way to downgrade (to any version) since the EKS cluster the chart is installed on is already at 1.26.

Interestingly, in one 2.7.5 cluster it happens and in another it does not. Also, I have no way to downgrade (to any version) since the EKS cluster the chart is installed on is already at 1.26.
The problem in my case was a missing CRD, something related to etcdsnapshot.
The logs in the bad cluster complained about a nonexistent resource, which I initially mistook as a different issue.
IIRC, the logs said something with kubernetes.io while the actual CRD was cattle-something.
How did I find out about this? Well, out of desperation I tried to diff every aspect of the clusters.
Eventually I ended up k get crding and diffing the outputs, and the only one missing from the bad cluster was apparently the one that fixed (get crd yaml from the good cluster and apply on the bad one) both the complaints in the logs and the repo sync issue.

It seems the commit rancher/charts@176424fce4f19ab8b01141dcec2d4453f075aad2 which 176424fce4f19ab8b01141dcec2d4453f075aad2 refer to not in branch release-v2.7.

I try remove all existing System Chart repos and upgrade to v2.7.10 or v.2.8.2.
The Git error no longer appears, instead the repos stays in In Progress.

bump
issue still persists on 2.8.5, rollback doesn't resolve the issue. Along with etcd snapshots not being viewed in the webui, not sure if that's related in any way.

This is not a organic solution and should be fixed, however I fixed it:

Start an interactive shell in the cluster-agent pod:
kubectl exec -it cattle-cluster-agent-blabla -n cattle-system -- sh

Go to the repositories. They are located in /var/lib/rancher-data/local-catalogs/v2. You can also find them using find:
find / -name config -type f -path '*/.git/config'

Perform a git pull in the directories of the repo.
example:
sh-4.4# cd /var/lib/rancher-data/local-catalogs/v2/rancher-charts/4b40cac650031b74776e87c1a726b0484d0877c3ec137da0872547ff9b73a721/
sh-4.4# git pull

Do a refresh of the repos again in the web UI or via the terminal with:

for i in $(kubectl get clusterrepo -o=jsonpath="{.items[*]['metadata.name']}"); do kubectl patch clusterrepo $i --type='json' -p="[{'op': 'replace', 'path': '/spec/forceUpdate', 'value':'$(gdate +'%Y-%m-%dT%H:%M:%SZ' -d '5 seconds ago')'}]"; done


Fixes to auction series

Fixes to auction series as per @bucanero comments on #2183

Changes are last two commits, this can be squashed into one


echo not requiring braces

Long time since I've been here, but I've been following the project closely.
One thing that I have not been a fan of, is the exception with echo not requiring braces.
It's inconsistent and I'm not quite sure I understand the purpose of it?

echo is a keyword and not a function. It's synonymous to a macro for:
print(x)
print('\n')

since this is a very common use case.

I'd much prefer println() and print(),
echo just feels out of place.

I think we can have a middle ground here. We can enable println built-in function and gradually fade out echo. But we'll need more people to vote on this before we can start.

That is fair


Example MacawOSX error

I'm interested in this library for a MacOS app. I cloned the library and had to clear up some issues.

I had to alter the UIBezierPath code.
I had to remove pods
I then I imported the MacawOSX.Framework target (I have to change all Macaw imports to MacawOSX).
At that point I was able to build and run the example but am getting an error in the EventsExampleController line: macawView?.node = loadScene() I'm getting the error:
Thread 1: EXC_BAD_ACCESS (code=2, address=0x7fff94aa7320)

I do know that the loadScene() func completely runs through all of it's code and receives this error when the return value is loaded into the node
Ideas?

Never mind. This time I installed pods as per instructions and the example works.


Wrong key size and labels

Source:

http://www.keyboard-layout-editor.com/#/samples/commodore-vic20
Result:

yes


Test: Add unit tests (utils and components behavior)

Skipped tests (related to date and time)
Due to the fact that the server transform the date received, store it in the database and then, return it to the client, I decided to use the UTC format to automatically transform that info into the client local UTC. However, if I do that, the tests start to fail because they transform the date and time automatically.


Fix the build

Recent changes broke the build - this fixes it by addressing the new timestamp data type.

@fbriden It broke the build because the patch that it depends on was not merged on the other repo.


Windows Defender/Security's CFA(Controlled Folder Access) prevents registry backup

After an uninstall process, if we do registry cleanup with backup, Windows Defender/Security's CFA(Controlled Folder Access) prevents it and causes an error saying something like '<full_file_path> not found/specified'.
Disabling CFA and retrying works. Not sure but seems like Windows Security side problem as other software like CCleaner, Malwarebytes, etc. also seem to have same issue.

I can't really do much about this, it's entirely on the user to properly configure CFA and add exclusions for things they trust. Looks like there are ways I could circumvent CFA but I'd rather not risk getting labelled as malware.


fail to identify decoding function

sample: 64a38967b32c71575cd1da2b039a4a87
function: 0x100028c0
granted, there are lots of protocol parsing routines in this binary, so detection is difficult.

gets identified now, but emulation fails


Default not being respected

weekend_days = JSONField(         blank=True,         help_text="Days of the week which are considered weekend days (format is DAY_NAME:1, eg. sunday:1)",         schema={             "type": "dict",             "keys": {                 "monday": {"type": "number", "default": 1},                 "tuesday": {"type": "number", "default": 0},                 "wednesday": {"type": "number", "default": 0},                 "thursday": {"type": "number", "default": 0},                 "friday": {"type": "number", "default": 0},                 "saturday": {"type": "number", "default": 1},                 "sunday": {"type": "number", "default": 1},             },         },     )
In the above scenario, the weekend_days when complied sets to null, instead of 0, so when I query I get
` {
            "monday": 1,
            "tuesday": null,
            "wednesday": null,
            "thursday": null,
            "friday": null,
            "saturday":1,
            "sunday":1,
   }`

I use Django 4.1, postgres 13, django-jsonform 2.10.1

Hi, thank you for reporting this issue.
I've found the cause of the bug, it's an easy fix. However, it won't be released until a few days later. I'm already working on a new version, so I'll release the fix along with that (hopefully by the end of this week). I hope you don't have any urgent requirements.

Thanks @bhch Looking forward to the new version !! Cheers

Fixed in v2.11.0. Please upgrade.


outstanding returns needs to be reorganized so it is separated by outcome

there is a new feature for the trading page where users are show which positions have claimable proceeds:

claimable proceeds needs to be returned from getWinningBalance organized by outcome and claimTradingProceeds needs to be position specific, and also keep the previous functionality of claiming all proceeds.

claimTradingProceeds takes market and share holder, all the winning shares will be traded in. There is no reason to have a claim proceeds per position, only one for the whole market.
I think we need to re-think this design.

There is no reason to have a claim proceeds per position, only one for the whole market.

Correct

I agree with Tom. We were originally thinking that only one position would be a winning position, but after talking with Phoebe it seems this is not correct in some places.
@matt-bullock We should consider dropping that "claim proceeds" button on the position row and perhaps move it to where the other market action buttons (Report/Finalize/etc) go.

👍 Perfect!

Just a thought. User will have to click on the claim proceeds button in order to go to the confirmation modal.
There they will determine if its worth it to claim or if they want to wait for lower gas prices.
Do we want to put how much ETH they will be claiming? Or is the button good enough and the user can go to positions page to get the amount of ETH to be claimed?

@bthaile What is the confirmation modal? Is it something we show, or is like a MM popup? If it's the first, we can show that info there, but if it's the second we'll need to think about how to address this (since people using hardware wallets and Edge won't get a similar popup).

This confirmation modal.


Resort heap of documets / photos by Year / Month / Day

Description of the new feature / enhancement
I have large heaps of old documents, photos and it's hard to navigate even with search over the folders with more than 10000 files.
I'd love to have an option to resort files into subfolders like YEAR / MONTH/ DAY
Some customization would be great. For example do not create DAY subfolder if total month items less than 100 (for example)
Scenario when this would be used?
Sorting out large heaps of photos, documents.
For example, I have a WD Mycloud drive and it have a phone photos backup function. It works fine but all files stored in one folder.
Supporting information
ACDSee Photo have similar functionality for import photos function.

@DimonSmart thank you for the feedback!
For the photo sorting, you may try out https://www.mjbpix.com/automatically-move-photos-to-directories-or-folders-based-on-exif-date/. I don't see us integrating such kind of functionality into the PowerToys any time soon.

Hello, @taras-janea I almostd done with logical part of the sorter, and it is just needed a UI to be done.
Please take a look at the repo: https://github.com/DimonSmart/PhotoSorter
May be pre-ready project helps with photo/video sorting function.


FINCN-148 build with travis-ci.com and upload artifacts to Artifactory

This PR looks good to me.

Ok. Merge it when ready @Anh3h

Hi!
Could anyone please move on with reviewing and merging my pull requests.
All the PR-s are listed here: https://issues.apache.org/jira/browse/FINCN-148
Thanks!

Merged @aasaru . Thanks for your contributions.


fix "respect" to match main.json

The current translation for "respect" types "suspect"
Other possible translations:
respect: R-PT, PEBGT, SPWEBGT, RE/PEBGT, RE/SPEBGT, RAOE/SPEBGT
Local hint screenshot:

Thanks! I don't know if that changed at some point, or if... the hints largely came from some logs that Mirabai shared, so sometimes her writing didn't match main.json.


calculateIrradianceThroughCoverDeSoto should include anti-reflective coatings

The calculateIrradianceThroughCoverDeSoto function (lib_pv_incidence_modifier.cpp) does not take into account anti-reflective (AR) coatings, and should be modified to do so. The code submitted in #315 could be added to this function to correct that.

Update: accounting for anti-reflective glass was added to the calculateIrradianceThroughCoverDeSoto function with the addition of PVWattsV7. We currently assume all modules have AR glass in both PVWattsV7 and PVWattsV8, and I think we should also update pvsamv1 to make this same assumption. Double-checking that with a few NREL experts before I make the change.


displayed a smaller number of characters than we use per line

A smaller number of characters than what we use per line is displayed in the documentation. Could be the entire lines displayed there?
Here is an example where I noticed this:
https://www.tlomodel.org/_modules/tlo/methods/contraception.html#StartInterventions

At the moment it is possible to see the entire line in fixed width content like the source code but it is a bit of a pain to do so as you have to manually drag to the right by selecting text, or scroll to the bottom of the content and use the horizontal scroll bar there.
To fix this, we would either need to make the font size smaller in pre elements used to display code or make the whole content column wider. From a readability perspective probably the latter is better? If so will need to add some custom CSS like the following
@media screen and (min-width: 1280px) {
  div.wy-nav-content {
    max-width: 960px;
  }
}

which should make sure this only applies on screens with sufficient width to not need horizontal scroll bars (side navigation column is 320px wide), in a file under docs/_static, for example similar to the docs/_static/theme_overrides.css that was recently removed and some content in conf.py similar to
https://github.com/UCL/TLOmodel/blob/b04fafde5c0ad29c881d4cc34ce9f530e59d73f0/docs/conf.py#L71-L75


Fix: RedisTokenBucketRateLimiter does not refill and block randomly anymore

Fixes #46
This PR reworks the Lua script used by the RedisTokenBucketManager in a few ways. The primary goal is to fix the issues mentioned in #46. Summary of the changes:

The current time now is generated by Redis now, which is safe due to the use of redis.replicate_commands(). By letting Redis take the time itself, all requests are clocked by the same system, guaranteeing best accuracy.
All timestamps are in milliseconds now, which is more accurate and allows working with sub-second TimeSpans.
redis.call('GET', ...) calls have been replaced with redis.call('MGET', ...) to safe a round-trip to the database.
Some variables were renamed in order to give them a clearer name.
The new current_tokens together with the timestamp are only persisted, if the request is allowed.
The TTL for the persisted data is now calculated from the periods_until_full multiplied by the period (duration). This fixes the first issue described in #46.
The persisted timestamp is not now anymore but the time_of_last_replenishment. This fixes the second issue described in #46.

Besides the changes to the Lua script, the PR also includes the following other changes:

The parameters passed to Redis in IRedisDatabase.ScriptEvaluate() are now explicitly flagged as (RedisKey) or (RedisValue) respectively. This ensures that the parameters are mapped to the KEYS[] and ARGS[] arrays correctly, which is important for correct script execution (especially in clusters).
The Lua script does now calculate retry_after (in milliseconds), which is 0 if the request was allowed or the milliseconds until the next token is generated if not allowed. This value is returned by the rate limiter in rounded up seconds.

The failed build is most likely due to the workflow not having access to the Redis connection string secret. The reason for this is explained in this GitHub blog post, where they describe that pull requests could potentially get access to secrets when using the pull_request workflow trigger type. If the pull_request_target workflow trigger type is used instead, the build runs against the build definition of the pull request target branch and secrets are made available automatically.

@Namoshek thanks for your contribution!


Hyperlinks

Being able to incorporate hyperlinks into templates!!

so like just big long text tokens? if no links then one big single text token. while scanning, if it finds the link pattern, stop and end the current text token, make a link token, then start a new text token
ya, loop looking for : and have 1 character of lookahead.
psuedo code for parsing rules in the raw template user input:
create token list
make a new text token
while !atEnd()
char = advance()
if char == : &&  isAlpha(peek(1))
end current text token
add current text token to list
makelink()
start new text token
else
add char to text token
end current text token
add current text token to list
atEnd()
peek(1) == eof
TODO!    still need to make the name "here" in if ::url:
makeLink()
displayText = ""
while(isAlpha(peek(1))
displayText += advance()
if peek(1) == :
advance()
url = ""
while (peek(1) !== : )
url += advance()
make link token
add displayText and url
added link token to token list
pseudo code for adding template to the screen
finalText = ""
for token in tokenList
if token is text token
finalText += ""
finatlText += token.text
finalText += "<//p>"
if token is link token
finalText += "<a href=/""
finalText += token.url
fintalText += "/">"
finalText +=token.displayText
finalText += "<//a>"
finalText

how do I store these tokens in the data base?
does mongo allow arrays of objects in a schema?

YES
{
"_id" : ObjectId("5ec55af811ac5e2e2aafb2b9"),
"name" : "Working with Arrays",
"user" : "Database Rebel",
"desc" : "Maintaining an array of objects in a document",
"content" : "some content ...",
"created" : ISODate("2020-05-20T16:28:55.468Z"),
"updated" : ISODate("2020-05-20T16:28:55.468Z"),
"tags" : [ "mongodb", "arrays" ],
"comments" : [
{
"user" : "DB Learner",
"content" : "Nice post.",
"updated" : ISODate("2020-05-20T16:35:57.461Z")
}
]
}

Version one is pushed


Using generator (function*) as data providers

Is this possible?
   var chart = c3.generate({
        data: {
            columns: [
                ['data2', 200, 130, 90, 240, 130, 220],
                ['data3', 300, 200, 160, 400, 250, 250],
                ['data1', 30, 20, 50, 40, 60, 50],
                (function () {var i=0; var ar = ["lab", 1,2,3]; return function*() {yield ar[i++];}})()
            ]
        }
    });

based on http://c3js.org/samples/data_columned.html

@arjamizo I have absolutely no idea, am thinking not, but somehow being able to use generators is an awesome idea.


| Middle of the Linked List | C++

Title - Solution for Middle of the Linked List
What will change - Adding solution for 876 Middle of the Linked List in C++
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code

Programming Language
Please add/delete options that are not relevant.

[x] C++

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Please assign this to me!

Please assign this to me

@SSKale1 Please assign me this!

please assign me this:)


Log plugin does not work

Log plugin does not work. How does it work?

Have you create the 'storage' folder?
Have you made this folder writeable?

Created a storage folder in plugin / and granted permissions.

I'm in a hurry and I'd appreciate it if you could do it sooner.

Created a folder in php-proxy-app/storage and granted permissions.

the storage folder should in the same folder with the index.php

I'm only getting a partial log, why is that?

config.phpに'Log',を追加することで解決しました


Detect flow of nullable back to target module

This PR is built upon GH-54 GH-54 is landed.
This PR enables AutoAnnotator to detect flow of @Nullable values back to target module from downstream dependencies due to changes in public methods of target module.

@lazaroclapp Thank you for the review, I resolved all comments, will land this once CI passes. 🚀


Visualization problems

Hello, could you please provide me with the visual code? Thanks

Same issue as https://github.com/uzh-rpg/RVT/issues/9
If you are interested how visualization is done in the code: https://github.com/uzh-rpg/RVT/blob/5774d5ca645102407b927619abe491e7f7961ba2/callbacks/detection.py#L76-L80


next版本自定义列格式化bug🐛

Package
Version

@antv/s2
2.0.0-next.3

@antv/s2-react
2.0.0-next.3

@antv/s2-vue
2.0.0-next.3

Sheet Type

[x] PivotSheet
[ ] TableSheet
[ ] GridAnalysisSheet
[ ] StrategySheet
[ ] EditableSheet

🖋 Description
使用meta进行数据格式化的时候，把标题也给格式化了

:tada: This issue has been resolved in version @antv/s2-v2.0.0-next.6 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version @antv/s2-react-v2.0.0-next.5 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Handle JSON Exception for incoming data

For example, https://github.com/AutoPacker-OSS/AutoPacker/blob/a4f7d6fcf07d0c5afd2215a54023db2b4974295c/user-service/src/main/java/no/autopacker/userservice/controller/RESTController.java#L66
It will crash if someone sends an empty JSON {}.
Need to go through all the code and clean this issue.

@ANicholasson I would like to take this up.
Before we create new user, a simple validate of input payload is needed, which checks for mandatory fields.

@chander97 Sure👍

@ANicholasson before I raise PR, just want to know, shall we have simple validation of whether both username etc are present in body
(or)
shall I do schema validation. Ex of schema:
{
"username": "string",
"password": "string"
}

@chander97 sure. Tbh, this project is the result of a bachelor project. So there are a lot of flaws here and there. So we are open for changes, ideas and such as well. I can create a dummy user so you can test the application and make yourself familiar if you want.


Inter-service connectivity between Eureka clients on K8s

backend-2-api-gateway in K8s env complains:
[…] 500 Server Error for HTTP GET "/api/e3selector/e3-courses/"
io.netty.channel.ConnectTimeoutException: connection timed out: ⏎
  sc-elas-official-backend-1-service-registry.elas-official-edge.svc.cluster.local/<IP_ADDRESS>:8004

I have observed that there are some changes already made to the file, and I have the following suggestion below, based on the documentation:
try:
    EUREKA_HOST_NAME = os.environ.get("HOST")
    EUREKA_PORT = os.environ.get("EUREKA_PORT")
    INSTANCE_HOST = os.environ.get("INSTANCE_HOST")
    eureka_client.init(
        eureka_server=f"http://{EUREKA_HOST_NAME}:{EUREKA_PORT}/eureka",  # type: ignore
        app_name="ELAS-AUTH",
        instance_port=int(os.environ.get("DJANGO_PORT", "8002")),
        instance_host=INSTANCE_HOST,  # type: ignore
    )

The instance_ip=socket.gethostbyname(EUREKA_HOST_NAME) is not necessarily required, which can be removed. However, the instance_host property should be provided. If we are using the "host" networking driver, we can set the instance_host to the hostname of the host machine. However, if we are using the "bridge" driver (or a custom network), we should set the instance_host to the IP address of the container within the Docker network.
We have to keep in mind that we need to retrieve the IP address of the container and pass it to the application every time the container is started, and I guess in order to automate this process, we could modify the Dockerfile or our container orchestration configuration to automatically retrieve the container's IP address and set the INSTANCE_HOST environment variable.


Make Class and all metaclasses non-abstract

Resolves #11132. Resolves #9959. The following code will now fail to compile:
abstract class A
end

class B < A
  def self.foo
  end
end

class C < A
  def self.foo
  end
end

x = B || C
x.foo # Error: undefined method 'foo' for A.class (compile-time type is A+.class)

This is because the value of x can also be A, so a definition for A.foo must be present. Currently, if x is indeed A, calling x.foo leads to undefined behaviour whenever multiple target defs are available, because virtual dispatch on x does not produce a branch for A.class.
Does not affect #3835, because .new is still implicitly available on all metaclasses (and .allocate on all class metaclasses). Also related to #5956.
Making Class non-abstract doesn't seem to have any significant consequences yet.

I believe the example in the OP should be abstract class A, right? Without abstract, the code already does not compile (dito first spec).

Looks good, but CI is failing and we don't have access to the logs anymore. I will close and open it again to re-run the checks (🤷 )

The problem is in this spec, which now incorrectly fails saying that Foo.new expects no argument.
      abstract class Foo
      end

      class Bar < Foo
        def initialize(@x : Int32)
        end

        def x
          @x
        end
      end

      ptr = Pointer(Foo.class).malloc(1_u64)
      ptr.value = Bar
      bar = ptr.value.new(1)
      bar.x

There is also this on the Windows CI now:
In src\resolvers\resolver.cr:31:21

 31 | "#{self.class.key}: #{source}"
                    ^--
Error: undefined method 'key' for Shards::Resolver.class (compile-time type is Shards::Resolver+.class)

I believe both of these cases indeed should be rejected, but it looks like we have to guard this behind a flag now.
@asterite what do you think? Is there a way to make only .new work?

I think here we should be a bit more relaxed. The code should compile, but if it ends up calling that method in the base abstract type then a runtime error is produced. Then reason is that it's unlikely that will actually happen.

This indeed couldn't happen via self.class, so a different approach is needed. Maybe we need an internal type for all subclasses of a metaclass but not the metaclass itself...


Fix wrong CPU config

Signed-off-by: Helene Durand<EMAIL_ADDRESS>What this PR does / why we need it:
I got why changing in the MD ProviderSpec in the KKP Kubevirt e2e tests the CPU to 100m/1 did not work.
Documentation explaining our issue : https://kubevirt.io/user-guide/virtual_machines/dedicated_cpu_resources/
Several issues:

It has to be a whole number integer ([1-9]+)
Users should not specify both spec.domain.cpu and spec.domain.resources.[requests/limits].cpu, which we did.
We did hard-code spec.domain.cpu=2
Number of vCPUs is counted as sockets * cores * threads or if spec.domain.cpu is empty then it takes value from spec.domain.resources.requests.cpu or spec.domain.resources.limits.cpu. As we did set spec.domain.cpu the value we did put in spec.domain.resources.requests.cpu was useless.

I removed the hard-coded  spec.domain.cpu=2 to keep only the spec.domain.resources.requests.cpu (same value as spec.domain.resources.limits.cpu).
Which issue(s) this PR fixes (optional, in fixes #<issue number> format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Optional Release Note:

NONE

/retest

/retest

/retest

/cherry-pick release/v1.42

/cherry-pick release/v1.42

/cherry-pick release/v1.42

/cherry-pick release/v1.42


npm prettier

Project description
make a derivation to install the prettier, the "opinionated code formatter"
Metadata

homepage URL: https://github.com/prettier/prettier
source URL: https://github.com/prettier/prettier
license: mit
platforms: unix, linux, darwin

It’s already there: nix-env -iA nixpkgs.nodePackages.prettier.
(Although this is admittedly hard to discover with tools like nix search, since nodePackages does not have recurseIntoAttrs.)

Perhaps it could be added to aliases.nix?

Maybe the odd really important node package should be aliased but putting recurseIntoAttrs on nodePackages so it shows up in nix search might be best.
Would adding it increase the evaluation time or are there any other negatives?

aliases.nix is for backwards-compatibility issues.
If you want to put an alias for it, open a PR modifying all-packages.nix.

There appears to be no bin/prettier in the package, at least not for the node-16_x.nodePackages on nixpkgs-unstable:
$ ls -F /nix/store/lajmk0widqbg8g7q6xhxjv9pfjmxfkjj-prettier-2.4.1
lib/

λ ls $(nix path-info nixpkgs#nodePackages.prettier)/bin
 prettier

It's in the normal nodePackages but not nodePackages_latest
λ ls $(nix path-info nixpkgs#nodePackages.prettier)/bin
 prettier

λ ls $(nix path-info nixpkgs#nodePackages_latest.prettier)/bin
lsd: /nix/store/mah61fapmawvg62jxgd3z889gy9gryf3-prettier-2.4.1/bin: No such file or directory (os error 2).
λ ls $(nix path-info nixpkgs#nodePackages_latest.prettier)
 lib

Please open a separate issue

That’s #145432.


Mailgun API, form templates

Fixed #1257, also discovered & fixed some minor bug re. languages lookup, continued work on external forms.

Most likely this will get extended. Not merging yet.

More places to send autoresponds from will be added later, in a separate PRs.


[Idea] Event Based Trigger Mechanism

Concept
Tie function plugins into the future hook system and other phases of the proxy lifecycle. Currently functions only run on the first phase.
Why?
Core concept within serverless
Resources

https://docs.aws.amazon.com/lambda/latest/dg/invoking-lambda-function.html
https://docs.microsoft.com/en-us/azure/azure-functions/functions-triggers-bindings

I've just briefly thought about this.
Kong's current implementation only allows plugins to be executed on the different "phases" (access, body-filter, etc). In order to be able to provide what this feature request suggests, we would have to overhaul the current Plugin system so that any plugin can act on a hook. This is no simple feat - the current system is complex on its present form, and this would add more complexity to the mix. There's also performance concerns to take into account.
Once that big amount of work is done, implementing this feature on this particular plugin would be relatively easy; a thin layer on top of that.


Update version to 3.4.3

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.13%. Comparing base (9724d44) to head (d22b70d).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #274   +/-   ##
=======================================
  Coverage   70.13%   70.13%
=======================================
  Files         385      385
  Lines       31962    31962
  Branches     4490     4490
=======================================
  Hits        22418    22418
  Misses       8500     8500
  Partials     1044     1044

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


2.1.0

Release notes here https://github.com/Juicy/juicy-ace-editor/releases/tag/2.1.0

Does it change the old way to use it?

I see now. I think you need to update the release notes. This still doesn't use ES modules. I merged the PR to my branch to continue working on it. But never had time to continue working on it. See https://github.com/Juicy/juicy-ace-editor/pull/41#issuecomment-387429361

So, I remove the ES Modules mention and release.
Could you publish your PR, so it will be at least visible as "pending"?


fix: Simplify ECCVM prover constructor and add a TODO

Simplify ECCVM prover constructor and clarify the confusion around what we thought was the use of uninitialized polynomials via an issue and a TODO.
Note: I think there may have been a merge mistake or something in Cody's recent PR (which I approved) that left the ECCVM prover constructors in a very weird state. (Also there was a WORKTODO left in code). There is still some weirdness (captured in the TODO) but hopefully this clarifies the situation a bit.

Benchmark results
Metrics with a significant change:

note_successful_decrypting_time_in_ms (8): 202 (-17%)
note_trial_decrypting_time_in_ms (8): 102 (+133%)
note_trial_decrypting_time_in_ms (64): 103 (+243%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
676
676
676

l1_rollup_calldata_gas
6,424
6,400
6,412

l1_rollup_execution_gas
585,757
585,733
585,745

l2_block_processing_time_in_ms
1,301 (-2%)
4,753
9,338 (-2%)

note_successful_decrypting_time_in_ms
:warning: 202 (-17%)
529 (+5%)
886 (-3%)

note_trial_decrypting_time_in_ms
:warning: 102 (+133%)
39.6 (-65%)
:warning: 103 (+243%)

l2_block_building_time_in_ms
10,383
37,765 (+1%)
72,198 (-3%)

l2_block_rollup_simulation_time_in_ms
10,269
37,365 (+1%)
71,446 (-3%)

l2_block_public_tx_process_time_in_ms
4,897 (-1%)
19,129 (-1%)
38,014 (-2%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,369 (-1%)
26,755 (-3%)

note_history_successful_decrypting_time_in_ms
1,126 (+1%)
2,204 (-1%)

note_history_trial_decrypting_time_in_ms
41.9 (-4%)
75.7 (-3%)

node_database_size_in_bytes
18,608,208
35,049,552

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
143
44,281
26,164

private-kernel-inner
152
71,648
26,164

private-kernel-ordering
112
50,830
39,026

base-parity
3,944 (+2%)
128
311

root-parity
1,053 (-6%)
1,244
311

base-rollup
13,285 (-1%)
110,464
861

root-rollup
49.4 (-1%)
4,359
725

public-kernel-app-logic
79.2 (-1%)
47,599
40,661

public-kernel-tail
112 (-1%)
53,372
7,125

merge-rollup
10.4 (-3%)
2,568
861

public-kernel-teardown
75.7 (+3%)
47,599
40,661

public-kernel-setup
74.9 (-1%)
47,599
40,661

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.0
16.0 (-2%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.583
0.494 (-2%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
46.0 (-1%)
72.4
232
445
868
1,725 (-1%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.471 (-1%)
0.447
0.422
0.416
0.413
0.413
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
54.3 (-1%)
106 (-1%)
335 (-1%)
660
1,310
2,606 (-1%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
105
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.480 (-1%)
0.480
0.456
0.455
0.453
0.453 (-1%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
60.9

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.533

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
40,372
496,534

Transaction size based on fee payment method

Metric
native fee payment method
fpc_public fee payment method
fpc_private fee payment method

tx_with_fee_size_in_bytes
905
1,161
1,377

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
1,172 (+2%)
820
4,001 (+4%)

Metric
1 public data writes
2 public data writes
3 public data writes
4 public data writes
5 public data writes
8 public data writes

tx_sequencer_processing_time_ms
419 (-1%)
285 (+4%)
737 (+4%)
367 (+4%)
1,204 (-1%)
429 (+3%)


maybe some thing wrong in utilities3.py ? or another bug in the code?

Hi, i am trying to reproduce your exepriments result.
I just change the TRAIN_PATH and TEST_PATH to '/home/data/NavierStokes_V1e-5_N1200_T20.mat'
and change T from 40 to 10, and then run:
 python3.7 fourier_3d.py
The result that i got as follow, maybe some thing wrong,
Please take a look, thanks,
489 10.252128057181835 0.019634807519614696 0.08073463827371598 0.22321295380592346
490 8.146521881921217 0.019635397335514428 0.08073771142959595 0.2236676424741745
491 10.866600763052702 0.019629415897652505 0.08073480159044266 0.2232614892721176
492 8.50566947995685 0.019635313227772712 0.08075259721279145 0.22349709749221802
493 10.444014868000522 0.019620819902047514 0.08072293162345887 0.2237038666009903
494 12.948399811983109 0.019610148882493378 0.08068822592496872 0.2233934336900711
495 8.396875259000808 0.019591552671045065 0.08063761121034622 0.22345163762569428
496 8.427478078054264 0.019569343607872724 0.08057864302396775 0.22329660773277282
497 10.278667164035141 0.01957395510748029 0.08059006589651108 0.22355687558650972
498 8.890411228872836 0.01956247627735138 0.0805699879527092 0.22378367960453033
499 8.894694186048582 0.019547253418713807 0.0805325716137886 0.2240159559249878
Traceback (most recent call last):
  File "fourier_3d.py", line 303, in <module>
    out = y_normalizer.decode(out)
  File "/home/zongyi/fourier_neural_operator/utilities3.py", line 99, in decode
    x = (x * std) + mean
RuntimeError: The size of tensor a (10) must match the size of tensor b (64) at non-singleton dimension 3

Sorry, it seems to be a shape mismatch. Could you change line 302
out = model(x)
to
out = model(x).view(1, S, S, T)
Btw, I assume you are using a very small modes/weight/training data. It seems the model is quite overfitted. In general, for a smaller time step like T=10 it may be better to use the iterative method fourier_2d_time.py.

Thanks, that change works for this case.
I am using the dataset as follow:
NavierStokes_V1e-5_N1200_T20.zip
I will follow your advice to use fourier_2d_time.py.
Bug i also have a question is that why fourier_3d.py has such a high demand for CPU that scripts can barely run in a machine when it is not idle, and that each epoch takes a long time(some times takes 150s+ per epoch).

Hi, I'm an undergraduate trying to replicate the Darcyflow code from paper using the 'fourier_2d.py' file.
I'm using the dataset “Darcy_421/piececonst_r421_N1024_smooth1.mat” “Darcy_421/piececonst_r421_N1024_smooth2.mat” and encountering the following error.
How should I resolve it?
And I’m using ''2.0.1+cu118''torch version.
2376449
RuntimeError                              Traceback (most recent call last)
Cell In[14], line 27
24 loss = myloss(out.view(batch_size,-1), y.view(batch_size,-1))
25 loss.backward()
---> 27 optimizer.step()
28 scheduler.step()
29 train_l2 += loss.item()
File ~/.conda/envs/scx7009/lib/python3.8/site-packages/torch/optim/lr_scheduler.py:69, in LRScheduler.init..with_counter..wrapper(*args, **kwargs)
67 instance._step_count += 1
68 wrapped = func.get(instance, cls)
---> 69 return wrapped(*args, **kwargs)
File ~/.conda/envs/scx7009/lib/python3.8/site-packages/torch/optim/optimizer.py:280, in Optimizer.profile_hook_step..wrapper(*args, **kwargs)
276         else:
277             raise RuntimeError(f"{func} must return None or a tuple of (new_args, new_kwargs),"
278                                f"but got {result}.")
--> 280 out = func(*args, **kwargs)
281 self._optimizer_step_code()
283 # call optimizer step post hooks
File ~/.conda/envs/scx7009/lib/python3.8/site-packages/torch/optim/optimizer.py:33, in _use_grad_for_differentiable.._use_grad(self, *args, **kwargs)
31 try:
32     torch.set_grad_enabled(self.defaults['differentiable'])
---> 33     ret = func(self, *args, **kwargs)
34 finally:
35     torch.set_grad_enabled(prev_grad)
File ~/.conda/envs/scx7009/lib/python3.8/site-packages/torch/optim/adam.py:141, in Adam.step(self, closure)
130     beta1, beta2 = group['betas']
132     self._init_group(
133         group,
134         params_with_grad,
(...)
138         max_exp_avg_sqs,
139         state_steps)
--> 141     adam(
142         params_with_grad,
143         grads,
144         exp_avgs,
145         exp_avg_sqs,
146         max_exp_avg_sqs,
147         state_steps,
148         amsgrad=group['amsgrad'],
149         beta1=beta1,
150         beta2=beta2,
151         lr=group['lr'],
152         weight_decay=group['weight_decay'],
153         eps=group['eps'],
154         maximize=group['maximize'],
155         foreach=group['foreach'],
156         capturable=group['capturable'],
157         differentiable=group['differentiable'],
158         fused=group['fused'],
159         grad_scale=getattr(self, "grad_scale", None),
160         found_inf=getattr(self, "found_inf", None),
161     )
163 return loss
File ~/.conda/envs/scx7009/lib/python3.8/site-packages/torch/optim/adam.py:281, in adam(params, grads, exp_avgs, exp_avg_sqs, max_exp_avg_sqs, state_steps, foreach, capturable, differentiable, fused, grad_scale, found_inf, amsgrad, beta1, beta2, lr, weight_decay, eps, maximize)
278 else:
279     func = _single_tensor_adam
--> 281 func(params,
282      grads,
283      exp_avgs,
284      exp_avg_sqs,
285      max_exp_avg_sqs,
286      state_steps,
287      amsgrad=amsgrad,
288      beta1=beta1,
289      beta2=beta2,
290      lr=lr,
291      weight_decay=weight_decay,
292      eps=eps,
293      maximize=maximize,
294      capturable=capturable,
295      differentiable=differentiable,
296      grad_scale=grad_scale,
297      found_inf=found_inf)
File ~/.conda/envs/scx7009/lib/python3.8/site-packages/torch/optim/adam.py:442, in _multi_tensor_adam(params, grads, exp_avgs, exp_avg_sqs, max_exp_avg_sqs, state_steps, grad_scale, found_inf, amsgrad, beta1, beta2, lr, weight_decay, eps, maximize, capturable, differentiable)
439 torch.foreach_add(device_state_steps, 1)
441 if weight_decay != 0:
--> 442     device_grads = torch._foreach_add(device_grads, device_params, alpha=weight_decay)
444 # Decay the first and second moment running average coefficient
445 torch.foreach_mul(device_exp_avgs, beta1)
RuntimeError: The size of tensor a (2) must match the size of tensor b (12) at non-singleton dimension 4


Optimize the response of AI agent APIs

Description
When calling the APIs like api/assistant/summary, api/assistant/insight and other related APIs, if OpenSearch cluster returns 4xx error, we just return 500 error or 400 error directly, which is not elegant, we should optimize this by checking the status code, if it's 4xx error, we return the original response, and if it's 5xx error, return an general error message.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test.

[ ] New functionality has user manual doc added.
[ ] Commits are signed per the DCO using --signoff.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@ruanyl @SuZhou-Joe @Hailong-am @xluo-aws please help to review this PR, thanks!


CmiFree reference count was zero abort on XC30

Original issue: https://charm.cs.illinois.edu/redmine/issues/546

A ChaNGa run fails on "Piz Daint" with the following error:
Big step 1171 took 64.667949 seconds.
ReOrder particles ... took 2.669491 seconds.
Writing Tipsy file ... took 59.006577 seconds.
Writing arrays ... took 0.000007 seconds.
Starting big step 1172
Rung distribution: ( 36267769, 12448115, 10134489, 12951858, 4067274, 1334, 0, 0
, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,)
MaxRung: 5
Maximum memory: 2896 MB
Kick Open:
Rung 0: 4.28953e-05
Rung 1: 2.14477e-05
Rung 2: 1.07238e-05
Rung 3: 5.36192e-06
Rung 4: 2.68096e-06
Rung 5: 1.34048e-06
Drift: Rung 5 Delta 2.68096e-06
Step: 1171.031250 Time: 0.100970 Rungs 5 to 5. Gravity Active: 1334, Gas Active:
0
Domain decomposition ... ------------- Processor 2044 Exiting: Called CmiAbort -
Reason: CmiFree reference count was zero-- is this a duplicate free?
[2044] Stack Traceback:
[2044:0] [0x20295535]
[2044:1] [0x2029fee2]
[2044:2] [0x202965d6]
[2044:3] [0x2029d113]
[2044:4] [0x2029d29e]
The traceback points to:
0x202965d6 <PumpRemoteTransactions+182>
as the place where CmiFree() is called.
Steps to reproduce:
build charm version v6.6.0-rc4-0-g63de3ca with:
./build ChaNGa gni-crayxc hugepages smp -j4 -O2
build ChaNGa version v3.0-94-g3e38ac3 with:
./configure --enable-bigkeys; make
Submit the attached ".job" script (fails on both 128 and 256 cores) using the attached ".param" file, and the Eris_Core.001170 dataset (currently being transferred to BlueWaters in ~trq/scratch/eris; also available on Piz Daint at: /users/trq/scratch_daint).  The crash will happen in a few minutes.  From the output, it seems that the crash is happening during a phase involving a lot of particle movement as the code rearranges the particles from "output order" to "tree order".

Original date: 2014-08-14 20:26:37

For the sake of simplification, does this happen with charm built with ./build ChaNGa gni-crayxc hugepages -j4

Original date: 2014-08-14 21:23:36

With charm built with "./build ChaNGa gni-crayxc hugepages -j4 -O2" and changa
run with
aprun -n 128 -N 8 ./ChaNGa.nsmp -oi 1 -D 1 -p 65536 -wall 30 -killat 2000 T_S.param +balancer MultistepLB_notopo +pemap 0-7 +setcpuaffinity +LBCommOff +LBDebug 1 +noAnytimeMigration
It gets past that point and dies slightly further on with:
libhugetlbfs [nid04228:11639]: WARNING: New heap segment map at 0x100ac000000 failed: Cannot allocate memory
The memory use seems to indicate that this is expected (memory use in ChaNGa can be much higher without SMP).
I reran with 256 cores and it ran OK.
Bottom line (at this point) non-SMP seems to be OK.

Original author: Yanhua Sun
Original date: 2014-08-15 09:22:44

Is this problem specific on 'Piz Daint' or also happening on other machines? I still have not got account on Piz Daint.  that is one reason that I am asking.

Original date: 2014-08-15 17:02:57

It does not happen on Blue Waters.  I do not have an account on Edison, so I can't try it there.

Original date: 2014-08-18 22:25:29

I tried it on Edison and the crash was not reproducible. I ran it on 192 cores (8 nodes) in SMP mode, gnu compiler.

Original author: Gengbin Zheng
Original date: 2014-09-03 20:23:49

Tom,
can you try with REMOTE_EVENT turned off?

Original date: 2014-09-05 19:18:25

With
#define REMOTE_EVENT 0
in machine.c, the run does not crash.
As far as I can tell, there isn't a performance difference.

Original date: 2014-11-05 20:00:17

When I try disabling REMOTE_EVENT and enabling persistent messages I get the following error:
machine.c(3276): error #165: too few arguments in function call
status = send_smsg_message(queue, ptr->destNode, ptr->msg, ptr->size, ptr->tag, 1, ptr);
^
It appears that several calls to send_smsg_message() were not updated with the new "ischarm" final argument.

Original author: Yanhua Sun
Original date: 2014-11-05 20:12:42

Sorry that I forgot to mention that several minor changes are required to make it compile. I will submit a patch for Charm to let persistent work without remote event

Original author: Yanhua Sun
Original date: 2014-11-07 18:16:55

checked in  http://charm.cs.uiuc.edu/gerrit/425

Original date: 2014-11-11 18:58:15

Does there exist another issue which keep an open bug report, so that we can actually fix this bug in future, instead of disabling the remote events?


Replace whitelist for allow-list

Thanks so much for your first contribution!  You rock, @leonardobazico!


python3Packages.fints: 3.0.0 -> 3.0.1

Motivation for this change
Things done

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

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

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

Result of nixpkgs-review pr 116386 at 8134b55d run on x86_64-linux 1

2 packages built successfully:

python38Packages.fints
python39Packages.fints

1 suggestion:

warning: unclear-gpl
lgpl3 is a deprecated license, check if project uses lgpl3Plus or lgpl3Only and change meta.license accordingly.
Near pkgs/development/python-modules/fints/default.nix:29:5:
   |
29 |     license = licenses.lgpl3;
   |     ^


sequelize_1.Model[(staticMethodPrefix + "conformOptions")] is not a function

Creating a new model instance yields the following error:
sequelize_1.Model[(staticMethodPrefix + "conformOptions")] is not a function
    at Function.BaseModel.prepareInstantiationOptions (/Users/admin/src/synec-serverless/node_modules/sequelize-typescript/lib/models/BaseModel.js:117:63)
    at new Model (/Users/admin/src/synec-serverless/node_modules/sequelize-typescript/lib/models/v4/Model.js:8:45)
    at new User (/Users/admin/src/synec-serverless/node_modules/morganisations/dist/db/models/general/user.js:14:12)
    at Function.build (/Users/admin/src/synec-serverless/node_modules/sequelize/lib/model.js:2139:12)
    at Function._target.(anonymous function) [as build] (/Users/admin/src/synec-serverless/node_modules/sequelize-typescript/lib/models/BaseModel.js:42:33)
    at Function.create (/Users/admin/src/synec-serverless/node_modules/sequelize/lib/model.js:2192:17)
    at Function._target.(anonymous function) [as create] (/Users/admin/src/synec-serverless/node_modules/sequelize-typescript/lib/models/BaseModel.js:42:33)
    at Object.<anonymous> (/Users/admin/src/synec-serverless/node_modules/morganisations/dist/middlewares/cognito-auth.js:65:34)
    at Generator.next (<anonymous>)
    at fulfilled (/Users/admin/src/synec-serverless/node_modules/morganisations/dist/middlewares/cognito-auth.js:8:26)

The presented available object keys are (using Object.keys on sequelize_1.Model):
[ 'getAssociations',
  'getAssociationForAlias',
  'Mixin',
  'default',
  '_setupHooks',
  'runHooks',
  'addHook',
  'removeHook',
  'hasHook',
  'hasHooks',
  'beforeValidate',
  'afterValidate',
  'validationFailed',
  'beforeCreate',
  'afterCreate',
  'beforeDestroy',
  'afterDestroy',
  'beforeRestore',
  'afterRestore',
  'beforeUpdate',
  'afterUpdate',
  'beforeSave',
  'afterSave',
  'beforeUpsert',
  'afterUpsert',
  'beforeBulkCreate',
  'afterBulkCreate',
  'beforeBulkDestroy',
  'afterBulkDestroy',
  'beforeBulkRestore',
  'afterBulkRestore',
  'beforeBulkUpdate',
  'afterBulkUpdate',
  'beforeFind',
  'beforeFindAfterExpandIncludeAll',
  'beforeFindAfterOptions',
  'afterFind',
  'beforeCount',
  'beforeAssociate',
  'afterAssociate',
  'beforeSync',
  'afterSync',
  'beforeBulkSync',
  'afterBulkSync' ]

The calling code:
user = yield db_1.User.create({
  uuid: cognitoUser.id,
  email: cognitoUser.email
});

extra info
I have two projects. Project A and project B. Project A defines the models, gets compiled from typescript to javascript (tsc, es6). Project B then consumes Project A's build javascript files via npm.
Any ideas?

What version of sequelize are you using?
I just got this error in a brand new project and the cause of the problem was sequelize@5. I downgraded it to v4 and it worked.
sequelize-typescript supports sequelize v5 only in canary version

It works up thru 5.1.1.
A change was made in 5.2 that https://github.com/sequelize/sequelize/commit/ef18710 a method from this.constructor._conformOptions(options, this.constructor); to this.constructor._conformIncludes(options, this.constructor); which seems to be cause of this particular error. For now, I've just locked my Sequelize version to 5.1.1 and things are working well for now. Hopefully it's just a simple method name change that won't require a ton of changes to fix.

@tcatkins99 Thanks for pointing that out!

@tcatkins99 I'm just wondering: Are you using<EMAIL_ADDRESS>with sequelize@5?

@tcatkins99 I'm just wondering: Are you using<EMAIL_ADDRESS>with sequelize@5?

Yes:
    "sequelize": "5.1.1",
    "sequelize-typescript": "^0.6.8",

@tcatkins99 This is strange. I was sure, that the bundled typings of sequelize@5 conflicting with the typings depended by sequelize-typescript: @types/sequelize. Have you done something special in order to prevent collision?

@tcatkins99 This is strange. I was sure, that the bundled typings of sequelize@5 conflicting with the typings depended by sequelize-typescript: @types/sequelize. Have you done something special in order to prevent collision?

Nope, (at least not intentionally, LOL). Just standard npm install ...


TokenProvider with post request

Hey 👋
Is it possible to use POST request to url instead of GET request?
Example is graphql api which accepts only POST requests.
Thanks

The library automatically issues a GET requests when attempting authentication - it is not possible to change this behaviour.


Custom template

Great work with both code generator and gradle plugin!
Just wanted to check if you have any plans for supplying a custom (mustache) templates, e.g. for class generation, and preferably also support in the gradle plugin?
Thanks!

Thanks for the compliments :smile:
The ability to supply custom templates is already there:
    val directory = File("path/to/directory")
    val codeGenerator = CodeGenerator()
    codeGenerator.setTemplateDirectory(directory) // optional second parameter - extension (default ".mustache")

Generation of a class starts with a template named class. This is also customisable:
    codeGenerator.templateName = "custom_class"

I suspect you won't find it easy to create your own templates without a knowledge of the structures from which they derive their data - your best approach is probably to start by looking at the existing templates (at https://github.com/pwall567/json-kotlin-schema-codegen/tree/main/src/main/resources).
And I hadn't planned to add the specification of the template directory to the Gradle plugin - you're actually the first person to have asked about custom templates!
Good luck!


Items not drawn

Hello there !
Thanks for the work on this library, it rocks ;)
By the way, I have an issue when scrolling up with this LayoutManager. Items are nicely recycled and drawn when scrolling down, by nothing is drawn on scolling up. An empty space with the good items heigh stays on top of the recyclerview, instead of the recycled views.
Has someone already experienced a similar issue ?
Edit: just to know, the same adapter code works fine with a classic GridLayoutManager.

Hi @prouland I can confirm this issue exists too. Were you able to find a solution for it?

Hi @prouland I believe I've discovered where the bug was coming from, this method:
   protected fun fillBefore(position: Int, recycler: RecyclerView.Recycler, extraSpace: Int) {
        var position = position
        var limit = scroll - extraSpace

        val start = 0

        //~~~~~~This conditional assignment causes the error, limit is always zero
       //after this and the block in the while won't execute, making the views not to be
       // recycled correctly. I'm still investigating the effects of removing this block because
       //the issue goes away when I remove this block.~~~~~~/
        if (limit < start) {
            limit = start
        }

        while (canAddMoreViews(Direction.START, limit) && position >= 0) {
            makeAndAddView(position, Direction.START, recycler)
            position--
        }
    }

https://github.com/bendaniel10/SpannedGridLayoutManager/blob/81bf6edeb543d1b473f84554e04532643244c059/spannedgridlayoutmanager/src/main/java/com/arasthel/spannedgridlayoutmanager/SpannedGridLayoutManager.kt#L588

Hi @prouland  This has been solved here: https://github.com/Arasthel/SpannedGridLayoutManager/issues/15 Adding an item decorator seems to solve this.

Hi @prouland this has been solved here: https://github.com/Arasthel/SpannedGridLayoutManager/issues/15 adding an item decorator solves this issue.

Hi @bendaniel10
Thanks for sharing a workaround.
Since I was not able to find a fix, I finally got the Nick Butcher LayoutManager : https://github.com/nickbutcher/plaid/blob/grid_span/app/src/main/java/io/plaidapp/ui/recyclerview/SpannedGridLayoutManager.java

@prouland I was uploading a new version as you wrote the message 😅. Sorry for the long delay, I haven't had much free time to work on my libraries lately.

Also, thanks to @bendaniel10 for the tip, it probably saved me hours of debugging 👍


add w3guides.com to stackoverflow copycat + some cleanup

Thanks a lot!
I have separated this PR in two (add w3guides and another one with the cleanup)


"Remove arrow function for better browser compatibility"

Hi, arrow function still exists in anime.js and anime.min.js - hence I take it we are to transpile it to support older browsers?
In support section, it says IE10+, a bit confusing since it doesn't support IE10+ out of the box.

anime.min.js is compiled to ES5 syntax.

thanks,this is answer i is seeking.