Fix aggregation optimisation inconsistent behaviour

Purpose
$subject
Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Codecov Report

Merging #1533 into master will increase coverage by <.01%.
The diff coverage is 85.71%.

@@             Coverage Diff              @@
##             master    #1533      +/-   ##
============================================
+ Coverage     67.88%   67.89%   +<.01%
  Complexity       45       45
============================================
  Files           648      648
  Lines         27309    27313       +4
  Branches       4293     4296       +3
============================================
+ Hits          18540    18544       +4
  Misses         6972     6972
  Partials       1797     1797

Impacted Files
Coverage Δ
Complexity Δ

.../io/siddhi/core/util/parser/AggregationParser.java
75.27% <85.71%> (-0.01%)
0 <0> (ø)

...java/io/siddhi/core/stream/input/InputHandler.java
69.44% <0%> (-5.56%)
0% <0%> (ø)

...o/siddhi/core/aggregation/IncrementalExecutor.java
81.25% <0%> (+0.78%)
0% <0%> (ø)
:arrow_down:

...siddhi/core/aggregation/IncrementalDataPurger.java
84.86% <0%> (+1.31%)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


[Jira Search (Self-Hosted)] ...

Extension
https://www.raycast.com/emanguy/jira-search-self-hosted
Raycast Version
1.82.3
macOS Version
No response
Description
Extension always show error Check you network connection
Tested from local host with curl command
curl -H "Accept: application/json" \
     -H "Authorization: Bearer MDgwMTE4OTY2N.................." \
     "https://jira.example.net/rest/api/2/project"

Result shows all projects. So token is working and also there is no error with network.
Steps To Reproduce
1.defining jira server url and token
2. unsafe https is disabled ( tried also with enabled, same result)
Current Behaviour
error Check you network connection
Expected Behaviour
Working extension

I have the same problem.

Same issue, it's a matter of changing authorizationHeader from jiraCredentials.ts
Instead of Basic ${btoa(${email}:${token})};
Could be something like this:
Bearer ${token};
Or give the option between personal token and basic authentication. And depending on config use proper authorizationHeader
Kind regards,

Any plan on fixing this?


Losing Shopping Lists/Full reset of Addon

Recently Ive been having issues with Auctionator Shopping lists resetting. I log-on to roughly 58 different servers per day and have had this reset happen multiple times, enough to where I have counted how many log-ins I have after a reset, seems to be on the 50th or 51st log in where I receive a lua error with the error below. Moments ago this reset happened again but without the lua error. I implemented the /atr c auctions_history_length 3 fix mentioned in Discord on every single character running Auctionator with no luck.
1x FrameXML\Bindings.xml:1 FrameXML\Bindings.xml:1 constant table overflow
1x FrameXML\Bindings.xml:1 constant table overflow
1x constant table overflow

Do you use the price histories from Auctionator?

I don't believe so..is that an additional addon/plugin that is downloaded with Auctionator?

If you don't use the price histories (tooltip values or selling tab listing) you can turn them off with ```
/atr nopricedb
That will resolve the reset issue

Ahh I understand now, sorry I misunderstood what you were asking! I will run that right now, thank you. The only thing I use the Shopping lists for are PointBlankSniper and BoE snipes list.


Fix adding ( to autocompleted method signatures with type hints enabled

Fixes https://github.com/godotengine/godot-vscode-plugin/issues/323
Autocomplete adds ( when completing method calls but should not do it when completing method signatures.
To see if it was a method signature it was checking that it ended with ): which is true when type hints are disabled, but fails when type hints are enabled.
It seems checking only that it ends with : is enough. See https://github.com/godotengine/godot-vscode-plugin/issues/323#issuecomment-1050073504.
I tested this only in 3.x.

Closing since I just found #49391 which I believe is the correct fix.


can not compile on Linux

i copied the files of the package zig-linux-x86_64-0.10.1 next to your files and compiled like this :
$ ./zig build -Drelease-fast
/home/roelof/Compiled/Lambergar-0.3.1c/build.zig:6:21: error: root struct of file 'std' has no member named 'Build'
pub fn build(b: *std.Build) void {
                 ~~~^~~~~~
referenced by:
    runBuild: /home/roelof/Compiled/Lambergar-0.3.1c/lib/build_runner.zig:231:45
    usage__anon_5126: /home/roelof/Compiled/Lambergar-0.3.1c/lib/build_runner.zig:242:13
    remaining reference traces hidden; use '-freference-trace' to see all reference traces

i also tried just ./zig build but same error occurs.
what's wrong ?
[ i'm on Xubuntu 22.04 ]

What version of zig compiler are you using?

version number is in my filename : 0.10.1

I used version 0.11, there have been some breaking changes from compiler version 0.10 to 0.11. Currently I am using version 0.12.0-dev.1536+6b9f7e26c.
Problem is that code will also not compile with newest version of compiler.

OK, which version should i use to compile your code ?
i find several versions at https://ziglang.org/download/ ..

I think version zig-linux-x86_64-0.11.0 should work.

that works !
roelof@roelof-HP-Elite-x2-1012-G2:~/Compiled/Lambergar-0.3.1c$ ./zig build
roelof@roelof-HP-Elite-x2-1012-G2:~/Compiled/Lambergar-0.3.1c$ cd zig-out/bin/
roelof@roelof-HP-Elite-x2-1012-G2:~/Compiled/Lambergar-0.3.1c/zig-out/bin$ ./lambergar
uci
id name Lambergar v0.3
id author Janez Podobnik
option name Hash type spin default 128 min 1 max 4096
uciok

Great, nice to know which version of compiler is OK.

another error occurs when compiling v0.4.1 on Xubuntu 22.04 :
$ ./zig build
zig build-exe lambergar ReleaseFast native: error: the following command failed with 1 compilation errors:
/home/roelof/Compiled/Lambergar-0.4.1/zig build-exe /home/roelof/Compiled/Lambergar-0.4.1/src/main.zig -lc -OReleaseFast --cache-dir /home/roelof/Compiled/Lambergar-0.4.1/zig-cache --global-cache-dir /home/roelof/.cache/zig --name lambergar --listen=-
Build Summary: 0/3 steps succeeded; 1 failed (disable with --summary none)
install transitive failure
└─ install lambergar transitive failure
   └─ zig build-exe lambergar ReleaseFast native 1 errors
src/history.zig:18:57: error: invalid builtin function: '@abs'
    return _entry + bonus - @as(i32, @intCast(@divTrunc(@abs(_entry * bonus), history_divider)));

i use Zig 0.11.x


feat: adding unit tests for newsroom component

Description

Newsroom component , of the website , has following components within it , and each component represented a piece of Newsroom component . Following are the components that are included for this :

Featured BlogItem

Newsroom

-NewsroomArticle

NewsroomBlogPost

-NewsroomSection

-NewsroomYoutube

-YoutubeCard

-Swiper

Related issue(s)
fixes #1773

/rtm


Swift 3 loop nil will infinite loop

`swift
func testNUllArray() {
var count = 0
for (_, _): (String, JSON) in JSON.null {
count += 1
print(count)
}
XCTAssertEqual(count, 0)
}

`

Can you try out https://github.com/SwiftyJSON/SwiftyJSON/pull/653 and see if it fixes the issue?

#653 fixed the bug, and pass the test, thanks.


Update for gentx command

The amount in gentx is not a flag anymore:
wasmd gentx my-key-name 1000000stake --home=/path/to/home/dir ...

This is fixed


Using List<String>.getItems().add("...");

When using a List<String> Widget, using list.getItems().add("some string"); causes the item to display, but not be selectable.
    final List<String> list = new List<String>(game.skin);
    list.getItems().add("Abc");
    list.getItems().add("Def");
    list.getItems().add("GHI");
    list.getItems().add("JKL");
    list.getItems().add("MNO");
    list.addListener(new ChangeListener() {
        @Override
        public void changed(ChangeEvent event, Actor actor) {
            Gdx.app.log("Menu", list.getSelected());
        }
    });
    table.add(list);

Using setItems however, works as expected:
    final List<String> list = new List<String>(skin);
    list.setItems("Abc", "Def", "GHI", "JKL");
    list.addListener(new ChangeListener() {
        @Override
        public void changed(ChangeEvent event, Actor actor) {
            Gdx.app.log("Menu", list.getSelected());
        }
    });
    table.add(list);

Is this a bug, or is it not intended to be able to add items dynamically in this way? (If it is not, then why does it display them?)
I'm using libGDX 1.5.3

Selection needs to be validated upon changing items. See relevant code. You can try emulating that yourself or calling something like list.setItems(list.getItems()), which should do the job, but is not very robust. I don't think this is a bug, getItems() like methods rarely support modification of returned collection, unless explicitly stated.

Yep. Call setItems after modifying the items.

Ok, thanks. May I suggest improving the JavaDoc of public Array<T> getItems () to make this clearer? At the moment, there is no JavaDoc on getItems() at all.


Add Abstract override support for Script types.

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Godot 3.0
Issue description:
Scripts already have a virtual can_instance() function to be implemented, but the implementation in the derived Script types never seem to take into account information that users can define for the Script.
// modules/gdscript/gdscript.cpp
bool GDScript::can_instance() const {
    return valid || (!tool && !ScriptServer::is_scripting_enabled());
}

// modules/visual_script/visual_script.cpp
bool VisualScript::can_instance() const {
    return true; //ScriptServer::is_scripting_enabled();
}

I would've expected some sort of call("can_instance") statement to be included in these implementations so that users can define for themselves whether a Script is instanceable (or just a boolean flag that shows up as a CheckButton in the case of VisualScript).
It doesn't make sense for a method of this kind to be exposed to the scripting API, but for the method to effectively be an "is this script valid" function, which, if you are calling it validly through the script, is obviously going to be true. The only way for it to have any meaning would be to create implementations for all types of scripts that take into account some scriptably-overrideable method of evaluating the function.
Edit: By the way, I tested whether overriding the function in GDScript, static or non-static, true or false, would have any effect on my ability to call .new() on the Script...it had no effect whatsoever. So, that's probably a bug if nothing else.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Add Protocol / ProtocolExecution and examples

https://github.com/microbiomedata/berkeley-schema-fy24/pull/11

https://github.com/microbiomedata/berkeley-schema-fy24/pull/11


Conflicting PERFILE_DAX flag

Incompatible PERFILE_DAX flag. I only noticed about this project as Vivek Goyal pointed me to it on the fsdevel list and I checked used flags because of this patch https://marc.info/?l=linux-fsdevel&m=165002361802294&w=2
And then noticed that PERFILE_DAX flag is conflicting with FUSE_INIT_EXT
Looks like you are on a non-upstream kernel with patches?
linux master include/uapi/linux/fuse.h
...
#define FUSE_INIT_EXT		(1 << 30)
#define FUSE_INIT_RESERVED	(1 << 31)
/* bits 32..63 get shifted down 32 bits into the flags2 field */
#define FUSE_SECURITY_CTX	(1ULL << 32)
#define FUSE_HAS_INODE_DAX	(1ULL << 33)
Btw, any reason you are not using 1 << number for the flags? In my personal opinion so much easier to read...

Thanks for reminder:)
The current valued is used for early per-file DAX patch versions. The value has been changed just before merging the PR. And we are lacked behind to update fuse-backend-rs to the latest code:)
Also PR is welcomed here:)
https://lwn.net/Articles/838916/

Btw, any reason you are not using 1 << number for the flags? In my personal opinion so much easier to read...

That's a good suggestion. The current coding style is derived from original work from the crosvm project. And we are happy to improve code readability.

@bsbernd Hi, looks like PR #105 fixed this issue. Maybe you can upgrade the dependencies soon.


Include addtask command in developer guide

Currently, our developer guide does not include documentation for the addtask command. Let us update the developer guide to include documentation on the addtask command.
The following items are added to the developer guide:

Information and execution about the addtask command
Sequence diagram
Activity diagram

Codecov Report
Base: 72.19% // Head: 72.19% // No change to project coverage :thumbsup:

Coverage data is based on head (c0a2ae1) compared to base (e0a1459).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #93   +/-   ##
=========================================
  Coverage     72.19%   72.19%
  Complexity      496      496
=========================================
  Files            86       86
  Lines          1568     1568
  Branches        171      171
=========================================
  Hits           1132     1132
  Misses          384      384
  Partials         52       52

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Clean code duplication in HTTP request structure

For the moment in the Requests structure there is a lot of code duplication.
The better idea would be to have one function used by all the methods: post,get,patch,put that would set up the request object and make the request.

I think this issue was closed by the efforts applied here https://github.com/meilisearch/meilisearch-swift/pull/283
If I'm mistaken, feel free to reopen this issue @bidoubiwa


Dependency on Powershell

librain-puppet dependency issue when using new version of puppetlabs/powershell, can you please modify the metadata.json so that it does not depend on old version of powershell anymore?

Duplicate of https://github.com/voxpupuli/puppet-iis/issues/131

@riponbanik Have you tried puppet/iis with puppetlabs/powershell 2.x.  Did it work?

It worked. But librarian-puppet fail to install modules if there is any dependency issue where one module requires powershell > 2 and this module requires powershell < 2

@riponbanik Thanks for testing.  We should be able to adjust the metadata to fix this issue.  We have another windows module with the same dependency issue, but it turned out that module didn't work correctly with puppetlabs/powershell 2 without fixes.

Possibly related: https://github.com/puppetlabs/puppetlabs-powershell/pull/156

Hi. I'm going to close this Issue. We deprecated the module and the new version is available here: https://forge.puppet.com/puppetlabs/iis


checkov: 3.2.343 -> 3.2.344

Diff: https://github.com/bridgecrewio/checkov/compare/refs/tags/3.2.343...3.2.344
Changelog: https://github.com/bridgecrewio/checkov/releases/tag/3.2.344

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 367315

x86_64-linux

:white_check_mark: 2 packages built:

checkov
checkov.dist

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 367315

x86_64-linux

:white_check_mark: 2 packages built:

checkov
checkov.dist


Center image not rendering on NodeJs

When we provide Qr Code an image with a base64 encoded data uri, no image is rendered in the middle of the QR Code using nodejs.
I have personally tried to debug the problem, but it is very hard to diagnosticate the problem, since this fails silently. This could very well be a problem with the jsdom or canvas libraries. The issue arises when the QR Code tries to convert the qrSVG to base64 and loads it in the canvas. The  tag containing the center image does not render an image on nodejs.
I found multiple relevant issues, mainly #163.
This was first implemented 3 years ago in #43 and was recently fixed in #241. I'd be curious if this was working prior. From looking at the nodejs example, it originally tried to load the image with an xml http request, and the image was linked directly from a web http url, but that code was removed in #241.
Either way, trying to load an image, either locally, remotely, or directly with a data base64 encoded uri doesn't work anymore. I'd like to know if this is a regression caused by this package or by canvas.

We are experiencing the same issue. It does load after refreshing the browser tho.
@BuyMyBeard Could you try version 1.6.0? It seems to be working in that version.

We are experiencing the same issue. It does load after refreshing the browser tho.

You refresh your browser in NodeJS? how do you do that?

@BuyMyBeard Could you try version 1.6.0? It seems to be working in that version.

I have tried making a repro with<EMAIL_ADDRESS>but it couldn't compile because it wasn't compatible with<EMAIL_ADDRESS>and canvas @2.11.2.
How'd you get it to work in that version?


Settings migration should normalize input before saving

redirect_url_prefixes needs to be an array
enable_standalone needs to be boolean

Applies to v1.1
Requires port to v1.2
Requires running mix omg.config --migrate once more


Make master the primary branch again.

I’m leaving q v2 behind as an ambition from the past I don’t intend to further. I think it makes sense make v1 the master branch again. Any objections?

+1
I think it'd be great to do an npm release of v1 as well.

Yeah, published v1.5.0 a few hours ago.

So it is done. master is the primary branch.

:)


[MAINT]- Docs theme improvement

There are some accessibility fails in the current docs - as well as general bad practices with how URLs are displayed.
This PR aims to make the theme more cohesive and better aligned with WCAG accessibility standards.

:wheelchair: Enhance accessibility
:lipstick: Update css style

I cannot remember when / why we made that change.
But in general having one single (really long) CSS is not best practice.
I am adhering to a more "react-like" structure here where the components and general theming are distributed across scss files.
Also I am favouring using scss directly over CSS as the former is way more flexible and readable than the latter

I have no argument against breaking up the long css file (I think it was auto-generated from the docusaurus template). I also have no argument on scss vs css - I didn't even know scss was a thing...
My comment was only directed at the folder in which they were kept.

The current folder structure better aligns with js/React conventions

SO managed to fix the deploy - 😶🌫  forgot yarn complains with license files sometimes


Add support for LiteX and friend repositories

https://github.com/enjoy-digital/

The "big three" would be;

[ ] litex
[ ] liteeth
[ ] litedram

The smaller ones would be;

[ ] litepcie
[ ] litescope
[ ] litevideo
[ ] litesata
[ ] liteusb

Example migen receipe at https://github.com/nathanrossi/meta-hdl/commit/b8b45708ccbfc95bae4716592894b5f58a9b6042#diff-f98e6fb1f28d2f22cf691c93008fef91

Recipes for litex and lite* cores (excluding liteusb which is deprecated) are now available in master.


FileNotFoundError: [Errno 2] No such file or directory: 'models\resnet_stride16\group1-shard1of23.bin'

Hi,
First of all, thank you for sharing your technology, which has inspired me a lot. Therefore, I want to reproduce it and learn from it
However, I encountered the above problem, I ran download_models.py only got the JSON file, but the.bin file was missing

Hi, could you check if you have all the bin and json files in the folder ./models/stride_16/? I think this has to do with the folder naming and the download script works fine.

Hi,
I haven't made any changes to your open source (virtual-walk) (only the output folder path has been modified in "download_models.py")
But, I had a problem which is shown in the attached figure.
Thank you.
------------------ 原始邮件 ------------------
发件人:                                                                                "Moving-AI/virtual-walk"<EMAIL_ADDRESS>发送时间: 2020年10月10日(星期六) 晚上7:32
收件人<EMAIL_ADDRESS>抄送: "张雯欣"<912650845<EMAIL_ADDRESS>主题: Re: [Moving-AI/virtual-walk] FileNotFoundError: [Errno 2] No such file or directory: 'models\resnet_stride16\group1-shard1of23.bin' (#20)
Hi, could you check if you have all the bin and json files in the folder ./models/stride_16/? I think this has to do with the folder naming and the download script works fine.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I can't see any attached figure. I tried the code as it is and the only problem I could find was related to the folder naming (everything downloaded perfectly but in a different folder). I will solve that issue it in the following hours in an upcoming PR, but I don't know if you are referring to the same thing.
Thank you for your help!

Hi，
You can see the attached file (two images) by dragging down the current page of the email.
My problem is that  missing  （such as group1-shard1of23.bin） .bin file when I run download_models.py
My configuration environment is windows.

The red box shows what is generated by the Models after running the code（download_models.py）

2.  After I run this code (that is download_models.py ) and run execute.py, I got the error shown in the figure below
Thank you for reply.
I don't know if you understand my mistake.
------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2020年10月11日(星期天) 凌晨2:19
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [Moving-AI/virtual-walk] FileNotFoundError: [Errno 2] No such file or directory: 'models\resnet_stride16\group1-shard1of23.bin' (#20)
I can't see any attached figure. I tried the code as it is and the only problem I could find was related to the folder naming (everything downloaded perfectly but in a different folder). I will solve that issue it in the following hours in an upcoming PR, but I don't know if you are referring to the same thing.
Thank you for your help!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Could you upload the images using the github comments or any other public platform? I can't see them in either email or github.
Thanks!

Thank you for your letter. I will put the image on Github as soon as possible.
I will describe my problem as follows:
Firstly, My environment is Windows and others (requirement.txt )
Secondly, I will describe the problem.

After I run this code (that is download_models.py ) and run execute.py, I got the error shown in the figure below

The red box shows what is generated by the Models after running the code（download_models.py）

Thank you for your reply.
I don't know if you understand my mistake.


Update Podfile in template

Summary
Added comment on Podfile under template directory.
I'm working with monorepo and didn't realize it easily to specify the path for react_native_post_install, so I thought it would be better to add this comment.

Changelog
[IOS] [ADDED] - add comments for specifying the path to React Native

Test Plan

@jacdebug Thanks, updated 👍


Use of custom isotopics with thermal expansion turned on does not correctly preserve mass

#1745 added the ability to combine a custom isotopic density vector with the material properties of a material class. I reviewed that PR and thought that everything looked good, but now I've been trying to use it on a real case and I've found a problem with mass conservation.
It appears that the component properly thermally expands. But during the thermal expansion, the component's number density is not adjusted away from the value specified in custom isotopics definition. This means that the density and the volume of the component become out of sync with each other, and as the component expands it gains mass.
I've attached a simple example input that you can use to reproduce the problem. It has only one single real component which uses a custom isotopic as well as a material class. Follow these steps to see the problem:

Note that Tinput == Thot in the component
Run the case
Open iPython
import armi; armi.configure()
from armi.bookkeeping.db import databaseFactory
db = databaseFactory("anl-afci-177.h5", 'r')
with db:     r = db.load(0,0)
r.core[0][0].getMass()
Note the mass as 26.567467
Edit the blueprints with Thot: 250
Rerun the case
db = databaseFactory("anl-afci-177.h5", 'r')
with db:     r = db.load(0,0)
r.core[0][0].getMass()
Note the mass as 26.7636707

These masses should be exactly equal, but they are not. I have tried the same thing without using a custom isotopic in the component definition to confirm that this is not a problem with the axial expansion routine itself, and the masses were the same in that case.
@bsculac @onufer
example.zip

I replicated the problem, the output from the test case shows that the component with custom isotopics is initialized to the same mass, specifically the info message for the application of custom isotopics is the same:
[impt] A custom material density was specified in the custom isotopics for non-custom material <Material: MOX>. The component density has been altered to 26.567467529113692.
However, the BOL mass report indicates the difference:
25 C:

250 C:

So the error is occurring somewhere between component construction and the BOL mass report.
Outputs attached.
25.txt
250.txt

I am not stepping through the debugger, but the mass density of ~26 g/cc seems odd and is set here - https://github.com/terrapower/armi/blob/41ce854dd59a845b57786c0d3ed33afd28464ea0/armi/reactor/blueprints/componentBlueprint.py#L278

Don't focus on the value of ~26, it is just some contrived example that I built with placeholder values.

Don't focus on the value of ~26, it is just some contrived example that I built with placeholder values.

I understand now, thanks! Update - the material is MOX and not custom. Still debugging by I think I know where the issue is.

@keckler / @bsculac - This is not necessarily a good fix, but can you test this out - https://github.com/terrapower/armi/pull/1822?

@jakehader Even with #1822 the issue would persist, I think. The example blueprints have inputHeightsConsideredHot: False so something is being missed when the cold dimensions are expanded.

https://github.com/terrapower/armi/blob/main/armi/reactor/converters/axialExpansionChanger.py#L112 This line in the axial expansion changer is most likely what is increasing the density of the components. I talked with @albeanth , my understanding is that the axial expansion changer assumes the heights provided in the blueprints are for hot heights, so the cold density of the component is increased such that the expanded/hot density will match the input density. So, #1822 could fix this but I'm still confused as to why the axial expansion changer will always increase the density. The function linked above even states that it should only be called when the inputHeightsConsideredHot setting is used, though it doesn't say whether that setting should be true or false. https://github.com/terrapower/armi/blob/main/armi/reactor/converters/axialExpansionChanger.py#L268-L283

my understanding is that the axial expansion changer assumes the heights provided in the blueprints are for hot heights, so the cold density of the component is increased such that the expanded/hot density will match the input density.

The axial expansion changer doesn't assume anything about the blueprints height. That assumption of hot heights in the blueprints is in the building of the blueprints (i.e., armi/reactor/blueprints) and is a legacy assumption that hasn't been changed. So to correct for cold heights in the blueprints, the expandColdDimsToHot method gets called. This corrects the relationship between number density and volume (really the height).

I'm still confused as to why the axial expansion changer will always increase the density

It only increases the number density to account for the cold heights. I.e., when inputHeightsConsideredHot is false.

should only be called when the inputHeightsConsideredHot setting is used

This isn't the best description, I agree. Rather, it should say "when the setting inputHeightsConsideredHot is False" (since False is not the default).
The confusion stems from 1) a lack of documentation 😞 , and 2) hacking together solutions to fit the legacy way of having hot heights in the blueprints. It'd probably be clearer to just pull the plug and refactor the blueprints system to have inputted heights in the blueprints match Tinput. That's just not something we've prioritized.

Thanks for the clarification. Since blueprints assume hot densities, I'm more on board with the fix in #1822 until we do something like what is discussed at https://github.com/terrapower/armi/discussions/767#discussioncomment-9887790.
I tested the fix in #1822 and the block masses come out the same for the 25C and 250C cases. The BOL mass report shows a discrepancy still, but that may be from having frozen sodium (the 25C case has more mass). Chris is going to be testing on a non-toy problem using #1822.

Hi all - thanks for the discussion. I didn't intend #1822 to be the final solution but it peaked my curiosity. @bsculac - I know you originally made the PR that this ticket is about so you're welcome to take over the branch and we can think about a better way to resolve this without passing in a case setting down to the component construct method.
I got to a similar conclusion as you and @albeanth did but didn't get to write down all the nuances so I'm glad you spoke about it and wrote it down. I think that assuming hot dimensions for blocks is a modeling choice and we have historically had this because there was no axial expansion consideration for ARMI until it was implemented. Now that it is, there may be benchmark edge cases where this modeling choice is still convenient. It's hard to say we should remove it but doing so would definitely simplify the logic for expansion and make documenting the behavior or intended behavior much more straight forward.
Let me know if you'd like to drive that straw man PR / branch to closure or please provide me some feedback on how we can better account for this density scaling when Tinput != Thot. I could use some help crafting some better unit testing for this too. I think we should add testing around mass fractions and other areas of customIsotopics since it is useful but not stress tested all the time.

@jakehader see https://github.com/terrapower/armi/discussions/1824. Let's chat a little more about simplifying this beast 💪


[FLINK-30799][runtime] Make SinkFunction support speculative execution for batch jobs

What is the purpose of the change
This pull request introduces a way to enable speculative execution for SinkFunction
Brief change log

Speculative execution would be enabled if a SinkFunction inherits SupportsConcurrentExecutionAttempts

Verifying this change
This change added tests:

Add new test cases in StreamingJobGraphGeneratorTest
Add new test cases in SpeculativeSchedulerITCase

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (JavaDocs)

@zhuzhurk Please take a look when you have time.
By the way, the failed test case seems unstable. I think it's not caused by this PR.

@zhuzhurk Thanks for reviewing! I have addressed the comments.

Azure testing failed again, it's caused by https://issues.apache.org/jira/browse/FLINK-30727

Shall we re-trigger Azure testing or merge this pr directly? @zhuzhurk

Will merge it since the case is pretty unstable.


Read data in chunks

Hey there, obviously it's not possible to read the BIOPAC file in chunks. Or if yes, how to do it? The code is not really self explanatory. Thanks!

It is... what are you looking to do?
An example of incrementally reading these files is here:
https://github.com/uwmadison-chm/bioread/blob/master/bioread/runners/acq2hdf5.py#L160
Or:
from bioread import reader as br
r = br.Reader.read_headers(acq_file)
streamer = r.stream()
for chunk in streamer:
    for data_buffer in chunk:
        print(buf.channel_slice)
        # Data is in buf.buffer, buf.channel_slice indexes you into an output slice

Basically, you can create a reader object, call stream() on it, and then use the result as an iterator to run through your file a chunk at a time.
Note that if your Biopac file is compressed, you can't do the same thing, you currently need to read a whole channel in at once as I haven't implemented streaming gzip reading.

Hey Nate,
this is very helpful, I’ll try it. Thanks!
Best,
Dominik
Von: Nate Vack<EMAIL_ADDRESS>Gesendet: Dienstag, 14. April 2020 00:32
An: uwmadison-chm/bioread<EMAIL_ADDRESS>Cc: Weber, Dominik<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [uwmadison-chm/bioread] Read data in chunks (#28)
It is... what are you looking to do?
An example of incrementally reading these files is here:
https://github.com/uwmadison-chm/bioread/blob/master/bioread/runners/acq2hdf5.py#L160
Or:
from bioread import reader as br
r = br.Reader.read_headers(acq_file)
streamer = r.stream()
for chunk in streamer:
for data_buffer in chunk:

    print(buf.channel_slice)

    # Data is in buf.buffer, buf.channel_slice indexes you into an output slice

Basically, you can create a reader object, call stream() on it, and then use the result as an iterator to run through your file a chunk at a time.
Note that if your Biopac file is compressed, you can't do the same thing, you currently need to read a whole channel in at once as I haven't implemented streaming gzip reading.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/uwmadison-chm/bioread/issues/28#issuecomment-613129271, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APDY2KBREEZ4LHQZGHMNYA3RMOHGBANCNFSM4MEYGSFA.

I'm closing this for now, feel free to reopen it if you keep having trouble.


Add a seperate column for "Quantity"

Add a seperate column for Quantity to the tables, that defaults to 1, so that we do not have to do "@quantity[xdy]" (ie: @quantity[2d6] ) in the Results Details field.

this is done, closing


Video and microphone streams are not added after permissions accept

Describe the bug
When a room is already joined, modal video and microphone permissions accept does not add devices streams.
To Reproduce
Steps to reproduce the behavior:

Open main page
Permission modal will pop up, keep it open
Write name, room name and click join room
Wait until room will be joined
Accept video and microphone permissions on modal

Expected behavior
When room is joined, on video and microphone permissions accept it should add these streams.
Environment:

OS: MacOS Catalina 10.15
Browser: Chrome 81.0.4044.138 (Official Build) (64-bit)
App Version: 0.1.0
SDK Version: 2.4.0
Node.js version: 12.12.0

Additional context
Name and room name inputs has to be moved to the right to prevent covering by permissions modal

the issue still exists

Thanks for the issue @aevaldas!
I've discussed this issue with the team. We feel that the best solution here is to simply prevent the user from joining the room until the browser's permission dialog has been accepted or denied. We have a PR up here to solve the issue.


Analyzer nuget package misconfigured

The nuget page for the analyzer has this:
<PackageReference Include="Spectre.Console.Analyzer" Version="0.44.0" />

But it's an analyzer, so it should be:
<PackageReference Include="Spectre.Console.Analyzer" Version="0.44.0">
  <PrivateAssets>all</PrivateAssets>
  <IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
</PackageReference>

For example, see here.
I assume this has something to do with the way the nuget was packaged.

Ah, I think we would need to add <DevelopmentDependency>true</DevelopmentDependency> to the Spectre.Console.csproj.
If you're up for doing this, a PR is very welcome.


Build Optimization Error with ton Package in Angular Dapp

Description:
I am encountering an issue with the ton package when building my Angular Dapp for production with build optimization enabled. The package works perfectly in the development environment, but the following error occurs during the production build:
Cell.js:107 Uncaught TypeError: Cannot read properties of undefined (reading 'Ordinary')
    at new ie (Cell.js:107:40)
    at 9786 (Cell.js:207:14)
    at r (bootstrap:19:1)
    at 121 (Builder.js:12:16)
    at r (bootstrap:19:1)
    at 1665 (contractAddress.js:11:19)
    at r (bootstrap:19:1)
    at 5230 (index.js:33:25)
    at r (bootstrap:19:1)
    at 2971 (index.js:25:14)

Steps to Reproduce:
Integrate the ton package into an Angular project.
Ensure everything works correctly in the development environment.
Build the project for production with build optimization enabled using the Angular CLI command: ng build --prod --optimization.
Expected Behavior:
The project should build successfully without any errors and work as expected in the production environment.
Actual Behavior:
The build process fails with the above-mentioned error, indicating an issue related to undefined properties in Cell.js.
Environment:
Angular version: 14.2.12
ton package version: 13.11.1
Node.js version: 18
OS: MacOs
Additional Information:
Any other relevant information or configurations that might help in diagnosing the issue.
Request:
I would appreciate any guidance or fixes to resolve this issue so that the ton package works seamlessly with Angular's production build optimizations.
Thank you!

Hi team any update on solution.


Accesskey and secretkey should be generated by accepting uid and gid instead of accesskey and secretkey

Environment info
NooBaa Version: 5.15
Platform: Linux
Accesskey and secretkey should be generated by accepting uid and gid instead it is expecting accesskey and secretkey
Actual Behaviour:
Command :
[root@ramya-sc-worker noobaa-core]#node src/cmd/manage_nsfs account add --config_root /ibm/fs1/ces/s3-config --name noobaa --email<EMAIL_ADDRESS>--new_buckets_path /ibm/fs2 --access_key a-abcdefghijklmn123456 --secret_key s-abcdefghijklmn123456
Oct-25 22:38:34.010 [/2817991] [LOG] CONSOLE:: Resource created : a-abcdefghijklmn123456
Expected Behaviour:
Command:
[root@ramya-sc-worker noobaa-core]# node src/cmd/manage_nsfs account add --config_root /ibm/fs1/ces/s3-config --name noobaa1 --email<EMAIL_ADDRESS>--new_buckets_path /ibm/fs2 --uid 1023 --gid 1024
Oct-25 22:38:34.010 [/2817991] [LOG] CONSOLE:: Resource created : a-abcdefghijklmn123456
ACCESSKEY: a-abcdefghijklmn123456
SECRETKEY: s-abcdefghijklmn123456

@ramya-c3
we will not generate based on the UID and GID.
if the access keys are not provided we will generate the keys as we do in the operator CLI calls.

These secrets are generated by NooBaa following AWS complexity considerations. I don't think we should allow users to input their own. We should allow listing with secrets and then this option should not be neccesary


docker run with multiple networks

Please improve the docker run syntax so running a container connected to multiple networks would be possible from the beginnings (no need to run docker network connect x y seperately).

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Pls refer to https://github.com/docker/docker/pull/17796#issuecomment-162612624 and other comments in that issue & https://github.com/docker/docker/pull/18906 which resolves this use-case. Please let us know if https://github.com/docker/docker/pull/18906 satisfies your use-case of running a container in multiple networks before the container starts

Sorry for the dupe, I will join the referenced threads.


How to debug and see the parsing process trace of chumsky combinators.

I flipped through the documents and found that the current status is TODO.

https://github.com/zesterer/chumsky/blob/main/guide/debugging.md

Yes, unfortunately this is not implemented for 1.0 yet. If you have thoughts about features you'd like to see, feel free to throw them in #19!

@zesterer Thank you for your prompt reply.


Fix collect date in utils and drive

Some dates may have different formats. Check before inserting.
Change set_default_value_for collection_date. The parameter name is wrong

Closed this in #13


Create Linux MySQL Recipe

 Documentation: https://docs.newrelic.com/docs/integrations/host-integrations/host-integrations-list/mysql-monitoring-integration
Output to user when the recipe is executed:
_ Note: __ You need to have a MySQL user with replication permissions for enabling this integration. If you’d like to create a new user please use these commands as reference. _
sudo mysql -e "CREATE USER 'newrelic'@'localhost' IDENTIFIED BY \ __'YOUR_SELECTED_PASSWORD';_"_
sudo mysql -e "GRANT REPLICATION CLIENT ON *.* TO 'newrelic'@'localhost' WITH MAX_USER_CONNECTIONS 5;"
Necessary ConfigurationAllow the user to confirm the defaults line by line or change if necessary.

Username (No default, have user enter)
Password (No default have user enter)
Hostname (default is localhost, have user confirm or enter a new hostname)
Port (default is 3306, have user confirm or enter a new hostname)

For Logging: Ensure that common log file directories/patterns & process name for MySQL is populated in the beginning of the recipe.
This recipe needs to also be plugged into the testing framework and the according test file definitions must be written.
Aha! Link: https://newrelic.aha.io/features/VIRTUOSO-57

Hello, We've got a question regarding the tickets present on this board. We could see the links to Aha! but we do not seem to have an access to it. Is it something that would be provided to us in the near future or is the information provided in the ticket + documentation enough?

@sheilakrishnan and @rishimukhop1 I believe this work is completed. We should close this issue.

@sheilakrishnan and @rishimukhop1 I believe this work is completed. We should close this issue.


Query: Trim very long external labels and add cmd flag to optionally specify metric labels to collect

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
Issue: #5540

trimmed external label
added []string flag query.metric-label for optionally specifying labels to collect
added unit tests
Verification

Wrote unit test

Thanks for this PR. I am not sure whether we should even keep external labels in metrics. I would consider just dropping them completely and avoiding complications.

Thanks for this PR. I am not sure whether we should even keep external labels in metrics. I would consider just dropping them completely and avoiding complications.

Thanks for the feedback @fpetkovski . Dropping external labels was discussed. Checkout the discussion here.

Checks are failing.


Enhance SubscriptionStatus orphans cleanup by implementing increasing…

… the frequency of the cleanup with a cap of 1 hr interval
for https://issues.redhat.com/browse/ACM-2767
Signed-off-by: Mike Ng<EMAIL_ADDRESS>

/hold
for more testing.
@xiangjingli I couldn't use the backoff function you recommended because it will terminate (backoff) eventually. I think we need this function to run forever in the background but at the max 1 hr interval as discussed. I couldn't find other wait function that has the cap interval and the backoff unfortunately has the cap but it terminates.

/unhold
I can see from the logs the cleanup is taking place at the pace we want it.

/lgtm


docs: mentioned missing info regarding putting npm organization while…

… importing workspace
For coherence on the page, I mentioned the missing information of adding the npm organization or user space as well if it is there in the imported workspace's package.json file as part of it's name during import of a workspace in another workspace.
For example, if it is @mycompany/shared-utils instead of shared-utils.
This case is not mentioned in the 'Workspaces which depend on each other' section.
Currently the 'Workspaces which depend on each other' section only explains the case where npm organization or user space is not part of the imported workspace's name

Hey @mbsc26! Thank you for the PR.
We explicitly specify the name of the package in a previous step in this walkthrough. If the user chooses to add a namespace to the package name we assume that they're also capable of knowing that they will need to further adjust the following steps accordingly.
{
  "name": "shared-utils"
}

I'm going to close this as it is advanced usage beyond what we intend to cover in our walkthrough.


Are absolute paths working on windows?

Describe the bug
Using absolute path in createServer root option causes vite to return 404 error
Reproduction
let server = await createServer({

  // all these work fine
  root: undefined,
  root: "",
  root: "./",

  // but not this one
  root: __dirname,

  fsServe: {
    // nothing works here and this is actually the main problem for me...
    root: "../somedir/"
  }
})

System Info
  System:
    OS: Windows 10 10.0.19042
    CPU: (8) x64 Intel(R) Core(TM) i7-3770K CPU @ 3.50GHz
    Memory: 8.16 GB / 15.96 GB
  Binaries:
    Node: 15.13.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 7.7.6 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 91.0.4472.106
    Edge: Spartan (44.19041.1023.0), Chromium (91.0.864.48)
    Internet Explorer: 11.0.19041.1
  npmPackages:
    vite: ^2.3.7 => 2.3.7

Used package manager: pnpm

@ruifortes Is this still an issue in the latest Vite version?

Closing due to lack of response, and there doesn't seem to be anyone else having the same issue. Let me know if this still happens and we can re-open this.


bug: MontgomeryBackendPrimeField gives incorrect addition for small limb fields

While using MontgomeryBackendPrimeField<1> to fuzz against my implementation of the u64 Goldilocks prime field I encountered an error is MontgomerBackendPrimeField<1>:
Input:
left = U64Goldilocks, right = MontgomeryBackendPrimeField:
Output of `std::fmt::Debug`:

	(
<PHONE_NUMBER>15286439,
<PHONE_NUMBER>256140342,
	)

The assertion that fails:
    let add_u64 = &a + &b;
    let addition = &a_expected + &b_expected;
    assert_eq!(add_u64.representative(), addition.representative().limbs[0]);

Based on the output the Montgomery Backend is off by one:
assertion `left == right` failed
  left:<PHONE_NUMBER>171426781
 right:<PHONE_NUMBER>171426780

Reproducing this in sage it appears the discrepancy is in MontgomeryBackendPrimeField.
sage: f = GF(2^64 - 2^32 + 1)
sage: a = f(732582227915286439)
sage: b = f(3906369333256140342)
sage: a+b
4638951561171426781

Thanks @PatStiles !

Thanks!

Solved in #602


Segfaults on launch on openSUSE

Before you ask, yes I saw https://github.com/clangen/musikcube/issues/122 and https://github.com/clangen/musikcube/issues/108, but just to make things clear: what version of ncurses is required?
Thread 1 "musikcube" received signal SIGSEGV, Segmentation fault.
                                                                                0x00007ffff7113547 in _nc_printf_string_sp (sp=0x0, fmt=fmt@entry=0xa00778 "settings_first_run_dialog_title",
    ap=ap@entry=0x7fffffffcdb8) at ../ncurses/base/safe_sprintf.c:251
251             if (screen_lines(SP_PARM) > MyRows || screen_columns(SP_PARM) > MyCols) {

I've tested musikcube with ncurses5 (Debian) and ncurses6 (Arch) -- it seems to work fine with both. But you must make sure you link against the "wide-character" libs, otherwise things will likely crash.
I just got done installing an OpenSUSE Tumbleweed VM. After enabling the "packman" repo, I did the following:
sudo zypper install libfaad-devel libcurl-devel libmicrohttpd-devel boost_1_64-devel cmake ncurses5-devel libogg-devel libvorbis-devel flac-devel liblame3-devel pulseaudio libpulse-devel alsa-devel zlib-devel
Then I cloned the sources and built and ran without issue. It seems to be fine here.
What version of OpenSUSE are you running? And are you sure you are compiling the latest master branch? Those two tickets you referenced are fixed, but the fixes have not been released yet, they are still hanging out in the master branch.

I tested with master and it works.


Unexpected behaviour while initializing v2 models

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Hello there,
I ran into an issue while migrating our codebase to pydantic v2.
Previously in v1, we were using a logic similar to my example code. I just realized that this used to work as long as you added the underscore_attrs_are_private = True as a part of the configuration. Since this option is now removed and set to True for all v2 models, I thought I could migrate it as I displayed in the example.
Unfortunately, this fails with a 'MyClass' object has no attribute '__pydantic_private__' error, and the error message itself is also a bit confusing.
As a potential solution, I thought about moving the logic within the __init__ call to a model_validator with the parameter mode="before" but I was not sure if this was the most optimal solution here. Moreover, I want to report this in case it indicates a bug in the code.
Example Code
from pydantic import BaseModel

class MyClass(BaseModel):

    _hidden_param: int

    def __init__(self, hidden_param: int = None, **kwargs):
        # If hidden param is provided, overwrite the hidden value.
        if hidden_param:
            self._hidden_param = hidden_param

        super(MyClass, self).__init__(**kwargs)

    # ---- Only uncomment while testing it in v1 ----
    # class Config:
    #     underscore_attrs_are_private = True

a = MyClass()  # This works!

a._hidden_param = 4  # This works as well
print(f"The private value {a._hidden_param}")

print(f"Class vars: {MyClass.__class_vars__}")
print(f"Pydantic Private: {MyClass.__pydantic_private__}")
print(f"Private Attributes: {MyClass.__private_attributes__}")

MyClass(hidden_param=4)  # This fails in v2 with: 'MyClass' object has no attribute '__pydantic_private__'.

Python, Pydantic & OS Version
pydantic version: 2.6.4
pydantic-core version: 2.16.3
pydantic-core build: profile=release pgo=true
install path: /Users/bariscandurak/.pyenv/versions/3.10.11/envs/zenml/lib/python3.10/site-packages/pydantic
python version: 3.10.11 (main, Sep 28 2023, 17:25:26) [Clang 14.0.3 (clang-14<IP_ADDRESS>.1)]
platform: macOS-14.4.1-arm64-arm-64bit
related packages: fastapi-0.110.0 typing_extensions-4.11.0 mypy-1.7.1 pydantic-settings-2.2.1
commit: unknown

@bcdurak,
Thanks for reporting your issue! Could you please submit the original code snippet that you were using in V1 as well?

Hey @sydney-runkle,
Not sure if I got the question correctly. In any case, if you are asking for the v1 version of the example code that I submitted, it is basically the same code snippet with the config class uncommented, which looks something like this:
from pydantic import BaseModel

class MyClass(BaseModel):

    _hidden_param: int

    def __init__(self, hidden_param: int = None, **kwargs):
        # If hidden param is provided, overwrite the hidden value.
        if hidden_param:
            self._hidden_param = hidden_param

        super(MyClass, self).__init__(**kwargs)

    class Config:
        underscore_attrs_are_private = True   # It only works if this is set

a = MyClass()  # This works!
a._hidden_param = 4
print(f"The private value {a._hidden_param}")

b = MyClass(hidden_param=5)  # This works as well
print(f"The private value {b._hidden_param}")

However, if you are asking how we used it in our original code, you can check out the FileSyncModel in our repository.

Just realized that calling super(MyClass, self).__init__(**kwargs) before accessing the _hidden_param solves the issue in v2 as it helps to set up the pydantic built-in attributes.

Apologies for not following up earlier. Things were a bit hectic pre-Pycon! Settling into some solid development now 🚀 !


reexporting linked hashmap for convenience

hey there,
what do you think of this: it would be convenient to reexport the LinkedHashMap struct, otherwise users of that feature will have to add the crate themselves.
thanks :D

yaml::Hash is already exported. It will be the right type whether or not preserve_order is used. Exported here..

My bad, I have overlooked that. Thank you :D

Just out of curiosity, doesn't a BTreeMap already preserve order?

No:
let mut preserve_order = LinkedHashMap::new();
let mut sort_keys = BTreeMap::new();

for &k in &["b", "a", "c"] {
    preserve_order.insert(k, ());
    sort_keys.insert(k, ());
}

// b a c
for k in preserve_order.keys() {
    println!("preserve_order: {}", k);
}

// a b c
for k in sort_keys.keys() {
    println!("sort_keys: {}", k);
}


feat(csv): add first modified date in csv export

Download Artifacts :

win-ia32
win-x64
linux-x64


Store the Sync wrapper in a variable for clean removal.

This commit primarily fixes this issue:
https://github.com/koalyptus/TableFilter/issues/258
The fix here is to store a Sync call closed over "this" straight within
the object we're referencing. This having access to that object, we can
always retrieve the exact function reference/pointer/instance for removal
from handling events.

Thanks for your contribution @archisgore, I appreciate!
Can you please increment the package version to 0.2.43 in package.json:
https://github.com/koalyptus/TableFilter/blob/master/package.json#L3
Cheers

Thanks for your feedback and quick reply. I made the changes and updated the code.

Almost there :smile:
I advise you to npm install and run the linter with the  grunt eslint command to avoid CI build to fail.
Also please bump the package version.
Thanks in advance

Tx for your patience:
:+1:


Refactor PrimitiveBuilder Constructors

Which issue does this PR close?
Partially implements https://github.com/apache/arrow-rs/issues/2054
Rationale for this change
Refactor apis to make a consistent new and with_capacity functions
What changes are included in this PR?
Removes capacity from new and added with_capacity function instead.
Are there any user-facing changes?
Yes

Going to get this in as the potential for conflicts is high, thank you :+1:

The archery tests in CI pipeline failed . May have know what archery is and how to debug that failure ?


Cloudflare Tunnel warp_routing issues - plan need to be applied twice

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
registry.terraform.io/cloudflare/cloudflare v3.34.0
Affected resource(s)
cloudflare_tunnel_config
Terraform configuration files
resource "cloudflare_argo_tunnel" "tunnel" {
  account_id = var.cloudflare_account_id
  name = "test_tunnel"
  secret = "REDACTED"
}

resource "cloudflare_tunnel_config" "tunnel_test" {
  account_id = var.cloudflare_account_id
  tunnel_id = cloudflare_argo_tunnel.tunnel.id

  config {
    ingress_rule {
      service = "http://<IP_ADDRESS>:8080"
      hostname = "tunnel.${var.cloudflare_zone}"
    }
    warp_routing {
      enabled = false
    }
  }
}

resource "cloudflare_record" "tunnel_cname" {
  zone_id = var.cloudflare_zone_id
  name = "tunnel"
  value = "${cloudflare_argo_tunnel.tunnel.id}.cfargotunnel.com"
  type = "CNAME"
  proxied = true
}

Link to debug output
https://gist.github.com/calderonth/1eeed1c19592b9c2955bba78d661adf7
Panic output
No response
Expected output
The plan/apply should correctly set the warp_routing values on the first pass.
It sends an empty JSON structure on the first pass which results results in a broken configuration (no data routed throught the tunnel).
Running a second time the apply actually fills out the the warp_routing structure and the tunnel configuration is set accordingly.
Actual output
The plan shows it has applied successfully on the first apply when it hasn't.
Steps to reproduce

Adapt the configuration provided
Run terraform apply
Run terraform plan after and observe that the warp_routing setting still needs applying

Additional factoids
I have also tried to remove the warp_routing section as it is supposed to be optional but that also results in an unusable tunnel when created from TF.
References
No response

this issue is related to sending zero values in Terraform where it cannot differentiate the two in state (hence two runs fix it).
i don't have an ETA on this but we have a larger body of work that is tracking zero values across the provider and i'll note this one against that.

Since Cloudflare Enterprise Support redirect to this issue could we have any update please.
Thanks.

Do you happen to have any news on this one?

This is no longer an issue and can be closed. It has been addressed within the Cloudflare Tunnel's API service.


Redact log

What?
Let's convert token printed on the console as asterisks for the sake of security.
(redact means to remove words or information from a text before it is printed or made available to the public)
How?
Use winston library (deprecated)
Use pino library (MIT license, published 2 months ago)
https://www.npmjs.com/package/pino

To use Pino in TypeScript, you can follow these steps:

Install the Pino library and its corresponding types:

npm install pino @types/pino

Import the necessary modules in your TypeScript file:

import pino from 'pino';

Create a logger instance with the desired configuration:

const logger = pino({
  level: 'info', // Set the log level (e.g., info, error, debug)
  redact: ['token'], // Specify properties to redact (e.g., token)
  prettyPrint: true, // Enable pretty-printing of logs (optional)
});

In the above example, we're using the level option to set the log level. The redact option is used to specify properties to redact from log messages (e.g., "token"). You can customize these options based on your logging requirements.

Use the logger instance to log your token:

const token = 'mySecretToken123';

logger.info({ token }, 'Received a token');

In this example, we're using the logger.info method to log the token. The token is passed as a property in the log object ({ token }).
Pino provides various features and options to further customize your logging experience. You can explore Pino's documentation (https://github.com/pinojs/pino) to learn more about its capabilities and how to adapt it to your specific use case.

I tried to apply pino to vscode Logger, but it seems that our logger need to be streamified for that.
I simply added Regex replace step to redact.
https://github.com/Samsung/ONE-vscode/pull/1610


Add web page links

Describe the bug
The buttons are not redirecting to the correct webpage
To Reproduce
Steps to reproduce the behavior:

Add the correct webpage links

Expected behavior
On clicking the button we need to go to the playstation or xbox series purchase page
Video
https://github.com/TusharKesarwani/Front-End-Projects/assets/104245345/953ca992-de3c-42d1-aab1-d283281ca258
Additional context
Here is the link of the project
https://github.com/TusharKesarwani/Front-End-Projects/assets/104245345/7302209c-8fcd-4f2b-bff4-5b0f6b3e4fd1

@Gungun1643 Can you provide more information or details, please?

Not required


PATCH submissions/edit: add test cases for closing/closed forms

@matthew-white should this test pass to fulfil https://github.com/getodk/central-backend/issues/536?

I think that test is failing because you can't create a new submission to a closed form. That behavior is intended, but you should be able to edit (that is, create a new version of) an existing submission to a closed form. The tests in this PR send requests to the PATCH endpoint, but I think that endpoint is actually OK. It's the PUT endpoint that I think will need to change:
https://github.com/getodk/central-backend/blob/8377a0f1f62607098148af77a939f2cce3710994/lib/resources/submissions.js#L182


fix(plugins/plugin-kubectl): --kubeconfig path in commands are not always expanded

Fixes #7363

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[x] 🐛 Bug fix
[ ] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

Nice, this should cover all cases that pass --kubeconfig, no matter how they flow to the apiServer.
/lgtm


Sign and notarize macOS app (dmg)

To allow anyone to run Tippy.app smoothly.

[x] Create Certificate
[x] Sign Tippy.app
[ ] Sign Tippy.dmg
[ ] Notarize Tippy.dmg
[ ] Staple Tippy.dmg
[ ] Add necessary steps from above to tools/osx and GitHub actions

Experiment and CI setup: https://github.com/nervosnetwork/tippy/pull/38

Experiment and CI setup: https://github.com/nervosnetwork/tippy/pull/38


[experiment][cluster-test] perf bench for network packet loss

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

Gonna close this out as its over a month untouched.


formatter: changes in id.tpl

Fixes the id.tpl file, since Jinja does not accept
list comprehensions.

Signed-off-by: Ilias Koutsakis<EMAIL_ADDRESS>

formatter: fix list comprehension in id.tpl

...

@Lilykos thanks. Can you please rebase and re-push your branch?


Maximum residency wrong?

Does it make sense that in the above the maximum residency is lower than the current one?
ekg-<IP_ADDRESS>

old issue, but pretty certain this only updates on GC

old issue, but pretty certain this only updates on GC


insert a space between two word.

A minor bugfix, just insert a space between two word.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4566/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4567/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4568/
Test PASSed.

There's been no progress on this PR for a while. Please re-open if you would like to continue the work.


How do you patch ButtonsDxe?

Prerequisites

[Yes] Have you read the wiki?
[No] Is your device in the supported list?
[Yes] Does the device have a Snapdragon SOC which is supported in this repo?

Description
[Description of the bug or feature]
Expected behavior: [What you expected to happen]
Detailed instructions in the documentation.
Actual behavior: [What actually happened]
No documentation on how to patch ButtonsDxe.

https://github.com/edk2-porting/renegade-project.org/blob/master/en/sm8150pkg/tutorial/4_Preparing_files_workspace.md begin reading from here

https://github.com/edk2-porting/renegade-project.org/blob/master/en/sm8150pkg/tutorial/4_Preparing_files_workspace.md begin reading from here

Documentation should still be provided in a non-archived repository.


I would like to add "Binary Search" in Python

This task involves adding "Binary Search" and its corresponding solution to the repository. Please assign this task @YuweAEC

sure your can proceed with the code and pr
I'm assigning this issue to you

added pls check it


Where is matrix B which relates the optional control input to the state x?

Relate to the code Line 50. Not optional more?

Yeah, we need B


Assorted Linux fixes

Turn off fpo- stack walking depends on it on retail builds
Speed up debug builds by improving performance of IsAddressOnStack
Add support for debug and retail builds being in different directories

Changing binary drop location may require runtests.py update?
LGTM otherwise.

Speed up debug builds by improving performance of IsAddressOnStack

Curious, how did we do that?
'fno-omit-frame-pointer' does otherwise.

@obastemur the two are inrelated- -fno-omit-frame-pointer should not noticeably affect build time (it might slow down runtime on functions with high register pressure). The debug build speed up was done by improving the performance of IsAddressOnStack- it used to call GetThreadStackBounds very frequently and the Linux implementation of that function is slow- I optimized IsAddressOnStack by not having to call GetThreadStackBounds and can do a much cheaper check.
@jianchun looks like runtests.sh needs to be updated too- I'll update the PR

fno-omit-frame-pointer should not noticeably affect build time (it might slow down runtime on functions with high register pressure).

I'm more optimistic with runtime performance.

The debug build speed up was done by improving the performance of IsAddressOnStack- it used to call GetThreadStackBounds very frequently and the Linux implementation of that function is slow

@digitalinfinity I didn't understand how that would improve debug build time? Are we using this method on build ?
LGTM

I didn't understand how that would improve debug build time?

He didn't mean speed up "build time". He meant speed up debug build binary.


tree lstm

Hi,
I am checking the implementation, and I  couldnt find the parts related to the tree lstm.  are you planning to release that part too?
thanks

Hi,
I am sorry for the late response. I am plan to release it after ACL 2017 (July 5).
-Qian

Hi, I'm reading your paper for now, and really interested in your implementation, but I coudn't find the part of tree lstm. Could you share your coding of tree lstm of your paper?
thank you


What are conditions for spawning a new Auth Token (Connected Applications)?

Hi!
As I'm moving our codebase to your new library I became a bit confused about your authentication mechanisms
I was wondering if you could explain me a few things, one of which is why my team is constantly spawning tokens in Connected Apps.
Currently we use an old version of your library, the one that had "DocuSign.Integration.Client" namespace.
Here is what we currently do:

We prompt a user for credentials.
Post credentials to https://demo.docusign.net/restapi/v2/oauth2/token and receive an auth token.
Request looks something like like this:

"grant_type":"password"
 "client_id":{our integrator key}
 "username":{username}
 "password":{password}
 "scope":"api",
 "token_type":"bearer"

Store user's email as a username and an acquired token as an ApiPassword (sic!) together and use it with your old library. Note that we use a token, which is supposed to be an oAuth token, as an ApiPassword. I'm not sure that this was intended, but it works.

Now I am moving our codebase to your new library. As I understand you have 2 authentication mechanism. One is "X-DocuSign-Authentication", and another one is oAuth.
In first case one has to add a username, a password and an integrator key to every request. And in case of oAuth you simply have to add an access token to request , like `"bearer":"{access_token}".
It seemed to me to that moving us to a genuine oAuth flow would be a good thing, so I started to work in this direction. But soon I realised, that I am confused. My whole team tends to use one DocuSign account for everyone and we run out of tokens from time to time. And I assume that your old library uses  "X-DocuSign-Authentication" mechanism, since it requests both email and password.
Your documentation says about Connected Apps list that

"For example, if the client application were an iPad app and the user owned two iPads, then both might appear in the list"

So, my first question is:
Could you, please, shed light on what is hiding behind this "might appear"? What are conditions for spawning a new token in case of running a Web site?
My second question is:
Would using an oAuth authentication keep spawning new tokens?
Thank you!

Hi @tehnetaholic , I'm trying to confirm the info but for now here are my responses regarding your two questions:
#1:  The text you are reading refers to the older, previous OAuth paradigm we have place.  In the REST API Guide the older auth is in the section titled:  "DocuSign OAuth (Resource Owner Credentials or SAML)".  The new (standard) OAuth paradigm is listed in the section titled "DocuSign OAuth (Authorization Code Grant or Implicit Grant)".  Sorry for the confusion, we're in the process of cleaning that up.
#2:  Will confirm once I know for sure but I do not believe it spawn new tokens in the Connected Apps list of your account (and therefore go against your limit).  Haven't had time to test this yet though.

Hi @tehnetaholic, to close the loop regarding the Connected Apps question - I spoke with the team and learned that right now the new OAuth mechanism is not yet managed (hooked up to) the Connected Apps in your account, however eventually you will be able to manage through there.
The connected apps limit that you are currently observing will go away, and it would only take up 1 slot per integration, not 1 slot per access token. So an app could request many tokens but it would only show up once.
Hope this helps!
Regards,
--Ergin

@Ergin008 thank you for the clarification
That is good news

Closing this issue.


feat: upgrade webpack

Now that https://github.com/webpack/webpack/issues/8656 has been resolved in the latest webpack release, I believe the version range restriction can finally be removed.
Should the updated .lock file be commited aswell?

Better use ^4.0.0 as there might be other plugins in the ecosystem also has this version restriction, forcing the latest version may result in failing dependency deduping.
We do lockfile maintenance regularly so no need to commit it in PRs.

@sodatea done

Will merge this after fixing the failing tests.
For now, I've narrowed it down to this commit in webpack: https://github.com/webpack/webpack/pull/8917

😅Just found out that both webpack & preload-webpack-plugin are doing it right. It's html-webpack-plugin that adds the unused chunk-common chunk to the html file…
I'll fix the test cases.


Correct Usage instructions

Hyperlink to clone instructions/guidance
Point to src/eyelets directory

Also, confirm that the units of measurement are up to the user, but need to be consistent.


Intercept in Contrast Coding Schemes

Expected Behavior
The constant (all values 1) intercept column should not be added when applying contrast coding schemes (i.e. backward difference, sum, polynomial and helmert coding)
I don't think this intercept column is needed. If you fit a supervised learning model it is probably gonna help to remove the intercept column. I think it is there because when fitting linear models with statsmodels you have to add the intercept.
However I don't like that the output of an encoder would then depend on whether the intercept column is already there or not, e.g. if I first apply encoder A on column A and then encoder B on column B the intercept column of B overwrite A's intercept column hence not adding a new column. Also if I have (for some reason) a column called intercept that is not constant it would get overwritten.
Any opinion? Am I missing something? Is the intercept necessary?
Actual Behavior
A constant column with all values 1 is added
Steps to Reproduce the Problem
Run transform on any fitted contrast coding encoder, e.g.
        train = ['A', 'B', 'C']
        encoder = encoders.BackwardDifferenceEncoder(handle_unknown='value', handle_missing='value')
        encoder.fit_transform(train)

Can intercept be added as a class parameter?
If so, then this is the way to go, imo. Then these classes could be tested with different values of intercept to catch errors and bugs.

Yes we could I think, that should be rather straight forward as well. Would you set with_intercept=True as a default for backwards compatibility or not (which might be more correct)?

Yes we could I think, that should be rather straight forward as well. Would you set with_intercept=True as a default for backwards compatibility or not (which might be more correct)?

Yep, I would set it to True to keep the default behavior intact

Is this one still relevant?

I think it is, I'm slightly in favor of setting with_intercept to False as I don't think many people will need the intercept.
However I'd like to keep this issue open for a little while and hope for more voices from the community, maybe from actual users of the contrast coding schemes. I feel a little uneasy to take this decision by myself without further discussion

user's perspective: imo encoders should do what they name says they do - encode existing columns. If there's a need to add an intercept, it has to be done consciously by the user. Especially for someone using statsmodels,  where the default way is to use
sm.add_constant. If you're in favor of keeping this feature as a parameter I'd suggest to make it False by default, otherwise I'd remove it completely.

Ok this seems to be a consensus, let's remove it - at least by default

Hello, any news for that issue? When multiple category encoders are used, most models will fail because of multiple intercept column in the dataset.


[DOCFIX] Fix incorrect -D args in IntelliJ config

Followup to #15025.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.8 please


Added new HTTP status code 103

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?

Fixed issues
none

IETF introduced new HTTP status code 103:  Early Hints.
more info at https://datatracker.ietf.org/doc/draft-ietf-httpbis-early-hints/

IETF introduced new HTTP status code 103: Early Hints.

its in draft mode only:

Status of This Memo
[...]
Internet-Drafts are draft documents valid for a maximum of six months
and may be updated, replaced, or obsoleted by other documents at any
time.  It is inappropriate to use Internet-Drafts as reference
material or to cite them other than as "work in progress."

Closing since it's in the draft. Will re-consder when it will be finalized.


[QUESTION]Should the streamClosedHandler be invoked for both stopSync() and stop()

During the uninit, we use the stopSync and noticed that streamClosedHandler invoked. I believe the stopSync( ) is a blocking call. Is this a correct behavior to get an callback for this? If so, does the ack convey the stopSync completed or just the API return is fairly enough?
And the same callback is seen when the stop( ) is used.

** Any design considerations/constraints **
We are using 3.0.0 SDK.
** If you would not like to open an issue to discuss your solution in open-platform, please email your question to<EMAIL_ADDRESS>**

stopSync() is a blocking API indeed. However, that has no bearing on whether the callbacks will be called or not. The core of the SDK is live-less (aka - no threads of it's own). It uses any of the liveness (threads) that call public APIs for control or events to drive the state machinery that can result in callbacks being issued.
In this case, once the stream is properly closed, the stream closed callback will be called. The async stop() will also trigger the callback, after all, stop sync is calling stop and they awaits for this callback itself to unblock. If the stream has no data in the buffer and the last ACK has already been received, the stop() async API will promptly close the stream as there is no outstanding work to be done.

@Subburaj47 ,
Closing this assuming your question is answered. Feel free to open an issue if you have face any issues. Thanks!


Latest commit breaks get_results

https://github.com/golismero/openvas_lib/commit/e53f81bffb2a5ab072bf6b3be2d850b71f5d2af4#diff-6fe303a22197e4810951633f15c7bfc9
get_results expects self.__task_report_id to be set, but that doesnt happen unless you launch a scan (launch_scan)

did anyone resolved this issue?
@LavernaSec  i tried your forked version and did easy_install. I'm facing issue as below:
Traceback (most recent call last):
File "/Users/sabinranjit/PycharmProjects/openvas-stuffs/scanopenvas.py", line 45, in
generate_report(scan_id)
File "/Users/sabinranjit/PycharmProjects/openvas-stuffs/scanopenvas.py", line 37, in generate_report
openvas_results = scanner.get_results(scan_id)
File "build/bdist.macosx-10.6-intel/egg/openvas_lib/init.py", line 961, in get_results
File "build/bdist.macosx-10.6-intel/egg/openvas_lib/init.py", line 90, in report_parser_from_text
TypeError: Expected str, got '<type 'unicode'>' instead
my test code is a follow:
`def generate_report(scan_id):
try:
print(scan_id)
scanner = VulnscanManager(HOST, USER, PASSWORD)
openvas_results = scanner.get_results(scan_id)
print(openvas_results)
except VulnscanException as e:
    print("Error:")
    print(e)`

can you help me with any possible solution?


Feature: actively processing

A first draft in the direction of #147, let me know what you think.
For the record here are my flamegraphs for the granular bench example:
before

after

The code I use for the flamegraphs (I did not manage to make something simply work with a benches directory)
use std::fs::File;
use std::time::Instant;

use rand::Rng;

use web_audio_api::context::{BaseAudioContext, OfflineAudioContext};
use web_audio_api::node::AudioNode;
use web_audio_api::SampleRate;

fn main() {
    let duration = 120. / 16.;
    let sample_rate = SampleRate(48000);

    let length = (duration * sample_rate.0 as f64) as usize;
    let mut context = OfflineAudioContext::new(1, length, sample_rate);
    // let mut context = OfflineAudioContext::new(1, 256, sample_rate);

    let file = File::open("samples/think-mono-48000.wav").unwrap();
    let buffer = context.decode_audio_data_sync(file).unwrap();

    let mut offset = 0.;
    let mut rng = rand::thread_rng();

    // this 1500 sources...
    while offset < duration {
        let env = context.create_gain();
        env.connect(&context.destination());

        let src = context.create_buffer_source();
        src.connect(&env);
        src.set_buffer(buffer.clone());

        let rand_start = rng.gen_range(0..1000) as f64 / 1000. * 0.5;
        let rand_duration = rng.gen_range(0..1000) as f64 / 1000. * 0.999;
        let start = offset * rand_start;
        let end = start + 0.005 * rand_duration;
        let start_release = (offset + end - start).max(0.);

        env.gain().set_value_at_time(0., offset);
        env.gain().linear_ramp_to_value_at_time(0.5, offset + 0.005);
        env.gain().set_value_at_time(0.5, start_release);
        env.gain()
            .linear_ramp_to_value_at_time(0., start_release + 0.05);

        src.start_at_with_offset_and_duration(offset, start, end);

        offset += 0.005;
    }

    let start = Instant::now();
    let buffer = context.start_rendering_sync();
    let duration = start.elapsed();

    println!("> processing duration (ms): {:?}", duration.as_millis());
    println!("> buffer.duration (s): {:?}", buffer.duration());
    println!(
        "> speedup vs. realtime (ms): {:?}",
        (buffer.duration() * 1000.) / duration.as_millis() as f64
    );
}

No visible improvement in the bench numbers, but I guess the visit takes so much place this is difficult to see any improvement there.

Hey, digging into this, I found an issue with how connections are mixed in inputs (connection are mixed according to the producer config, not the receiver one which is not right according to [1] and produce unpredictable results).
I just added a failing test case in graph.rs to spot that
It shouldn't be very difficult to fix though, I will do that soon
[1] https://webaudio.github.io/web-audio-api/#channel-up-mixing-and-down-mixing

This is fixed, let me know what you think !
Storing incoming connection (as you did at some point) could simplify the code as we could do all the add and mixing just before the node.process in graph.render. I can go on with that too if you want

maybe this is best to merge that fix, and continue in actively processing nodes in a dedicated PR (I also realize it would be maybe best to finish #34 before)

Good catch in the output mixing! I forgot to refactor that apparently in def9304f8151353b
I have not really had the time to look at this PR, there's a lot going on. I will try to put some comments on it later this week.


Automation: Cronjob Run Now does generate resources in Jobs Detail page

Based on the work done in https://github.com/rancher/dashboard/issues/8683, let's write an automated test to confirm that running a cronjob generates the appropriate resource when ran.
To Reproduce:

Create a cronjob
Click Run Now in the UI

Expected Result:
Jobs can appear on the CronJob details page, and Pods can appear on the Job details page.

Test has to be done on the model
https://github.com/rancher/dashboard/pull/8734/files#diff-629ca51684689c01e229c34044b8d49c235192a18f593431d9b070ee5e2d815e


[Bug]  Using uninitialized "test" variable, client: x.x.x.x, server: my.doamins.com requests ...

Current Behavior
using lasted version in Docker in Unraid  + Authelia+Cloudflare .
After last update have a lot of warnings For Every Proxy hosts:
[warn] 464#464: *60 using uninitialized "test" variable, client: 162.158.x.x, server: sub.mysite.com, request: "GET /api/sync HTTP/2.0", host: "sub.mysite.com"
and also vaultwarden have this :
 an upstream response is buffered to a temporary file /var/tmp/nginx/proxy/2/35/0000001352 while reading upstream, client: x.x.x.x, server: sub.mysite.com, request: "GET /api/sync HTTP/2.0", upstream: "http://<IP_ADDRESS>:4743/api/sync", host: "sub.mysite.com"
Expected Behavior
No response
Steps To Reproduce
No response
Environment

OS:
OS version:
CPU:
Docker version:
Device model:
Browser/OS:

Container creation
Docker in Unraid
Container log
..

Container inspect
No response
Anything else?
No response

[warn] 464#464: *60 using uninitialized "test" variable, client: 162.158.x.x, server: sub.mysite.com, request: "GET /api/sync HTTP/2.0", host: "sub.mysite.com"

Ok I will push a fix for this.

an upstream response is buffered to a temporary file /var/tmp/nginx/proxy/2/35/0000001352 while reading upstream, client: x.x.x.x, server: sub.mysite.com, request: "GET /api/sync HTTP/2.0", upstream: "http://<IP_ADDRESS>:4743/api/sync", host: "sub.mysite.com"

I think this should not cause any issue.  Do you see any problem accessing your service ?

I think this should not cause any issue. Do you see any problem accessing your service ?

all hosts are working, there was problem with sync on vaultwarden, it was his issue, after exited from extensions it's working fine, and still have this warning.
Also can find 'test' variable in hosts advanced settings.

Thanks, working fine


*: Add initial scaffolding with basic synchronization client

This commit adds the basic scaffolding needed for the testground Rust
SDK. This includes:

Licenses
README
A CI workflow
Dependabot config
Cargo.toml

In addition this commit implements a basic synchronization client,
connecting to the testground synchronization service, enabling users to
send signals and await barriers.

I will merge here. Reviews are still welcome. I will do any changes as follow-up pull requests.


Gifs play faster than expected

Steps to Reproduce
Image.network('https://github.githubassets.com/images/spinners/octocat-spinner-128.gif')

Flutter version:
Flutter 1.3.9 • channel dev • https://github.com/flutter/flutter.git
Framework • revision f91df4abe1 (16 hours ago) • 2019-03-09 21:19:28 -0500
Engine • revision 4e54bc93ca
Tools • Dart 2.2.1 (build 2.2.1-dev.1.0 2fb6cd9f5f)

Possibly related to #24804

Code Sample
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Image.network(
            "https://image.xiniujiao.net/bf265029ea1183ba84a18f4b6da75bdd.gif"),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel dev, 1.19.0-1.0.pre, on Linux, locale en_US.UTF-8)
    • Flutter version 1.19.0-1.0.pre at /home/tesser/Code/flutter_dev
    • Framework revision 456d80b9dd (8 days ago), 2020-05-11 11:45:03 -0400
    • Engine revision d96f962ca2
    • Dart version 2.9.0 (build 2.9.0-7.0.dev 092ed38a87)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /home/tesser/Code/SDK
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /home/tesser/Code/SDK
    • Java binary at: /home/tesser/Code/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Android Studio (version 3.6)
    • Android Studio at /home/tesser/Code/android-studio
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.8052
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.45.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.10.2

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 10 (API
      29) (emulator)

• No issues found!

I want to know how to resolve it,I face the same problem.

From @darshankawar
It does seem to play gifs faster than normal speed. Verified on the latest stable. Used Image.network official example.
65549.mp4.zip

flutter doctor -v
[✓] Flutter (Channel stable, 1.20.3, on Mac OS X 10.15.2 19C57, locale en-US)
    • Flutter version 1.20.3 at
      /Users/deeptibelsare/Documents/Fluttersdk/flutter
    • Framework revision 216dee60c0 (10 days ago), 2020-09-01 12:24:47 -0700
    • Engine revision d1bc06f032
    • Dart version 2.9.2

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/deeptibelsare/Library/Android/sdk
    • Platform android-30, build-tools 29.0.3
    • ANDROID_HOME = /Users/deeptibelsare/Library/Android/sdk
    • ANDROID_SDK_ROOT = /Users/deeptibelsare/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
    ✗ Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit
      https://flutter.dev/docs/get-started/install/macos#android-setup for
      detailed instructions.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)

[✓] Connected device (1 available)
    • SM A260G (mobile) • 5200763ebcfa861f • android-arm • Android 8.1.0 (API
      27)

! Doctor found issues in 1 category.

Workaround for this issue is to use the webp format for gifs, which plays  normally.

In this case, can _nextFrame.duration be zero? Using print, it is 0 so _hasFrameDurationPassed always be true.
https://github.com/flutter/flutter/blob/5d37734ea1032f25dfc022f6130916e28b98e7f3/packages/flutter/lib/src/painting/image_stream.dart#L900
https://github.com/flutter/flutter/blob/5d37734ea1032f25dfc022f6130916e28b98e7f3/packages/flutter/lib/src/painting/image_stream.dart#L919-L921

If the delay time of a frame is less than 20ms, mainstream browsers will set 100ms by default.
https://nullsleep.tumblr.com/post/16524517190/animated-gif-minimum-frame-delay-browser
https://qiita.com/razokulover/items/34962844e314bb4bfd04

Anyone found a solution?

Still same problem with https://cdn.discordapp.com/attachments/414258067870449665/445736475158380544/discord.gif
@itssidhere Solution didn't work

Verified this issue using latest versions and observed that the behavior persists.
https://user-images.githubusercontent.com/67046386/207852464-8e554976-8527-425d-85fb-edcca6f7fab1.mov
In above video, the sample gif is directly run in chrome while other is in Flutter app which is faster.

 runnable code sample

import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Center(
          child: Image.network('https://github.githubassets.com/images/spinners/octocat-spinner-128.gif')
        ),
      ),
    );
  }
}

 stable, master flutter doctor -v
[✓] Flutter (Channel stable, 3.3.9, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.3.9 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b8f7f1f986 (12 days ago), 2022-11-23 06:43:51 +0900
    • Engine revision 8f2221fbef
    • Dart version 2.18.5
    • DevTools version 2.15.0

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.7.0-9.0.pre.11, on macOS 12.2.1 21D62 darwin-x64,
    locale en-GB)
    • Flutter version 3.7.0-9.0.pre.11 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision dbc9306380 (7 hours ago), 2022-12-14 13:53:20 -0800
    • Engine revision 0a6a4a58f4
    • Dart version 3.0.0 (build 3.0.0-21.0.dev)
    • DevTools version 2.20.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Also finding that animated WEBP's are faster than ones shown on the web, etc.

same issue,
I forked the cached_network_image repository to resolve this issue.

https://github.com/Baseflow/flutter_cached_network_image/pull/988

On the web (Chrome), when the playback speed per frame is less than 10ms, it assigns 100ms.
  cached_network_image:
    git:
      url: https://github.com/LOCKEDFILE/flutter_cached_network_image.git
      path: cached_network_image

https://github.com/user-attachments/assets/d7b0017f-171e-4776-8ac5-58c86798713d