Add min-length support for GeneratorSchema

Adds min_length to the GeneratorSchema.
Closes #392

Codecov Report

Merging #394 (82fabe6) into main (7dcf502) will decrease coverage by 0.08%.
The diff coverage is 100.00%.

:exclamation: Current head 82fabe6 differs from pull request most recent head 9eee5a0. Consider uploading reports for the commit 9eee5a0 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #394      +/-   ##
==========================================
- Coverage   95.31%   95.24%   -0.08%
==========================================
  Files          90       90
  Lines       10511    10534      +23
  Branches        8        8
==========================================
+ Hits        10019    10033      +14
- Misses        492      499       +7
- Partials        0        2       +2

Impacted Files
Coverage Δ

pydantic_core/core_schema.py
98.09% <100.00%> (-0.70%)
:arrow_down:

src/validators/generator.rs
91.86% <100.00%> (+1.25%)
:arrow_up:

src/input/input_abstract.rs
87.23% <0.00%> (-3.20%)
:arrow_down:

src/validators/recursive.rs
98.24% <0.00%> (-1.76%)
:arrow_down:

src/validators/function.rs
98.85% <0.00%> (-0.58%)
: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 7dcf502...9eee5a0. Read the comment docs.

Awesome, thank you.


Should we keep the z property in Node2D? (naming)

Godot 3.0
In Node2D, there is a property named z. When used, this property overrides the drawing order of the node. So far, so good.
However, in Godot 3.0 properties became first-class reserved names in scripts, which means any script inheriting Node2D & cie will be unable to use z for any variable, be it member, parameter or even locals.
I know good practice about naming variable names, but that one's name is either way too good, or just awful (depends on view point considering good practice xD), but I think it should be renamed, especially when you consider inheritance, and it's not used that much in scripts, in general, to the point of being a one-character variable.
Could it be renamed z_order? (now is the best time, 3.0 is still beta)

What about depth? I don't like z_order because to me an "order" is a global object, defining the ordering relation between different elements.

It's a good idea to rename this, can be boring to don't allow to use z as a variable.
To visualize a bit :

In Libgdx (which is for me a clean code and good named) it's ZIndex. In Phaser it's renderOrderID. In OpenGL we are rahter talking about GL_DEPTH_TEST...
For me, both z_order, z_depth or z_index can be used.

I like z_index or z_depth

I like z_index, or we could be more explicit and actually call it draw_order on Node2D

If it's ok for you, I have a ready commit for this :

Modification is ready to be checked and merged.
Do you know if an issue listing all breaking changes from 2.1 to 3.0 exist ? I must add this one !

reserved names in scripts, which means any script inheriting Node2D & cie will be unable to use z

this is untrue. unless that script is using the word z (and not a public member variable).
why are we continuing to make silly syntactical changes over use-cases that don't exist?

@girng it used to be true, name lookup was changed, which is why this issue is now closed

i see. maybe i misread issue, my bad.


Update deploying a devfile documentation

What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind documentation
[skip ci]
What does does this PR do / why we need it:

Updates the documentation regarding deploying a devfile
Changes the wording to include Kubernetes first and then OpenShift.
Moves the tutorial / prerequisites around so Kubernetes is first
Removes the part regarding CRC and adds it to the OpenShift
prerequisites and the same for Kubernetes with Minishift.

Which issue(s) this PR fixes:
https://github.com/openshift/odo/issues/4113
PR acceptance criteria:

[X] Documentation

[X] I have read the test guidelines

How to test changes / Special notes to the reviewer:
N/A
Signed-off-by: Charlie Drage<EMAIL_ADDRESS>

/approve

/lgtm


Proactive connection management refactoring

This PR has the following objectives:

Introduces a class ContainerDirectConnectionMetadata which will encapsulate all container-specific connection-related configuration.
Removes an extra checkArgument call to verify the correctness of the CosmosContainerProactiveInitConfig built.
Wires the JVM configuration - COSMOS.MIN_CONNECTION_POOL_SIZE_PER_ENDPOINT to DirectConnectionConfig. The objective here is to default to the JVM configuration and also make way for a public API to configure it through DirectConnectionConfig possibly in the future.
Breaking change: Modifies the JVM configuration - COSMOS.DEFENSIVE_WARMUP_CONCURRENCY to COSMOS.OPEN_CONNECTIONS_CONCURRENCY

CI failures due to known issue with CI pipeline

/azp run java - cosmos - tests


Viewport display modes not working in GLES 2

Godot version:

3.1
OS/device including version:

Pop OS 18.10/ NVIDIA GeForce 920MX
Issue description:

when you try to change the viewport render mode to something like wireframe with the gles 2 renderer nothing happens

when i tried it in gles 3 it worked fine

Steps to reproduce:
create a new project
switch to gles 2
add a mesh instance
change the viewport display mode to wireframe, overdraw, unshaded.

Yes. This is a known bug. It was reported here. https://github.com/godotengine/godot/issues/26406. Although you explain it much better. :)

I'm working on a project that relies on gles2.0 would be great to have it fixed.
Is wireframe not possible because of some limitations on this version of the graphic library?

@Toshiwoz It just hasn't been done yet! It is a lot of work.

I can confirm only Normal works.
Note that Orthogonal doesn't work either in GLES2 editor viewport.  It just displays everything as clear color.

Still valid in 3.2.4 beta3
No longer relevant for 4.0, as GLES2 for 3D is not planned.

No longer relevant for 4.0, as GLES2 for 3D is not planned.

To clarify, we don't know for certain whether GLES2 will support 3D in 4.0. See https://github.com/godotengine/godot-proposals/issues/877.


check for termination when finite values are exhausted

We don't want an infinite loop when generating a small number of values in testHarness

I really think  the right thing to do here might just be to abandon the idea of skipping repeat test cases.  For one thing, it means we've got this constantly growing trie that makes each new test case just that much slower, and more importantly adds memory usage over time, because it must compare against previously seen similar keys.  And for another thing, I'm increasingly convinced that the best thing to do is to demote the key, which is actually exactly what will happen if we just run the test and observe the 0% additional coverage anyway.
Removing this logic will remove some fairly complex code (all of the hand-coded trie in PartialKeySet) and do closer to what we want with fewer weird infinite loops, so.... win-win?


Make a list of contributors.md

As contributors keep increasing, we can make a contributors.md to list their names and recognize their efforts :smile:

I can do this

Sure, but we need this at the end(or the middle of the coding duration) when there are a significant number of the contributors. I am however, assigning this issue to you

It will always be there in insights-contibutors , right ? What is the use of a separate file ?https://github.com/the-ethan-hunt/B.E.N.J.I./graphs/contributors


Implement proposal from #552 when consensus reached

Should this be transferred to openssl/project?

It certainly could be, but I'm not sure I see a significant value in doing so, unless there just some book keeping value there

It does not belong in this repo IMO and should be transferred


Missing warnings and more

After migrating from portmaster to synth I ran into some problems:
synth install databases/postgresql11-server
installed server and client.
With make -C /usr/ports/mail/spamassassin config I enabled PGSQL so databases/p5-DBD-Pg became a dependency.
synth install mail/spamassassin installed several dependencies but probably due to PGSQL_DEFAULT?=9.5 in /usr/ports/Mk/bsd.default-versions.mk synth built and installed databases/postgresql95-client and deleted postgresql11-server without chance to confirm deletion. That is really bad and I have no idea how to handle such issues. Installing postgresql11-server afterwards caused deletion of spamassassin et al.
Setting DEFAULT_VERSIONS+=pgsql=11 in /etc/make.conf did not fix this issue with synth.
make -C /usr/ports/mail/spamassassin install or portmaster mail/spamassassin worked as expected and used a previous installed postgresql-client without expecting the default version, any previously installed version was not uninstalled.
IMHO synth should

delete installed packages only after explicit confirmation
check better which packages already fulfill requirements

Did I miss something? It's not clear whether synth uses parameters set in make.conf.
However: Thank you for development of this helpful tool.

make.conf options should be put into /usr/local/etc/synth/LiveSystem-make.conf. There's a section on this on the man-page in the FILES section.


chore: updates tutorials to make use of zksync-cli

Changes introduced in this PR:

Updates "Account abstraction multisig" tutorial project setup to make use of zksync-cli
Updates "Daily spending limit account" tutorial project setup to make use of zksync-cli
Updates "Building a custom paymaster" tutorial project setup to make use of zksync-cli

@dutterbutter maybe add a step to delete the Greeter.sol contract and the deploy-greeter.ts and use-greeter.ts files from the deploy folder in the tutorial that do not use them

added it to the quickstart as well

and also adds it to the governance tutorial


error

-- WebDriver Auto-Installer --
[  INFO  ] Google Chrome version: 123.0.6312.106
[  INFO  ] Google Chrome webdriver version: 123.0.6312.105
[   OK   ] The driver has already been updated to the browser version!
[  INFO  ] Initializing chrome-webdriver for Windows
DevTools listening on ws://<IP_ADDRESS>:12758/devtools/browser/371e5c21-cc26-4023-9052-4a4035739fa3
[  INFO  ] [1secmail] Mail registration...
[ FAILED ] Traceback (most recent call last):
File "urllib3\connection.py", line 198, in _new_conn
File "urllib3\util\connection.py", line 85, in create_connection
File "urllib3\util\connection.py", line 73, in create_connection
TimeoutError: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "urllib3\connectionpool.py", line 793, in urlopen
File "urllib3\connectionpool.py", line 491, in _make_request
File "urllib3\connectionpool.py", line 467, in _make_request
File "urllib3\connectionpool.py", line 1099, in _validate_conn
File "urllib3\connection.py", line 616, in connect
File "urllib3\connection.py", line 213, in _new_conn
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x0000013D2BF04070>: Failed to establish a new connection: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "requests\adapters.py", line 486, in send
File "urllib3\connectionpool.py", line 847, in urlopen
File "urllib3\util\retry.py", line 515, in increment
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='www.1secmail.com', port=443): Max retries exceeded with url: /api/v1/?action=genRandomMailbox&count=1 (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x0000013D2BF04070>: Failed to establish a new connection: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "main.py", line 111, in register
File "requests\api.py", line 73, in get
File "requests\api.py", line 59, in request
File "requests\sessions.py", line 589, in request
File "requests\sessions.py", line 703, in send
File "requests\adapters.py", line 519, in send
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='www.1secmail.com', port=443): Max retries exceeded with url: /api/v1/?action=genRandomMailbox&count=1 (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x0000013D2BF04070>: Failed to establish a new connection: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "main.py", line 930, in
File "main.py", line 113, in register
RuntimeError: SecEmailAPI: API access error!

Check your internet connection

If stable try use vpn

I am getting an error:
[  INFO  ] License uploads...
[ FAILED ] Traceback (most recent call last):
File "main.py", line 959, in
File "main.py", line 714, in getLicenseData
File "main.py", line 282, in untilConditionExecute
RuntimeError: untilConditionExecute: the code did not return the desired value! TRY VPN!
The same, if using internet or VPN

File "main.py", line 959, in

File "main.py", line 959, in module

-- WebDriver Auto-Installer --
[ INFO ] Google Chrome version: 123.0.6312.106 [ INFO ] Google Chrome webdriver version: 123.0.6312.105 [ OK ] The driver has already been updated to the browser version!
[ INFO ] Initializing chrome-webdriver for Windows
DevTools listening on ws://<IP_ADDRESS>:12758/devtools/browser/371e5c21-cc26-4023-9052-4a4035739fa3
[ INFO ] [1secmail] Mail registration... [ FAILED ] Traceback (most recent call last): File "urllib3\connection.py", line 198, in _new_conn File "urllib3\util\connection.py", line 85, in create_connection File "urllib3\util\connection.py", line 73, in create_connection TimeoutError: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond
The above exception was the direct cause of the following exception:
Traceback (most recent call last): File "urllib3\connectionpool.py", line 793, in urlopen File "urllib3\connectionpool.py", line 491, in _make_request File "urllib3\connectionpool.py", line 467, in _make_request File "urllib3\connectionpool.py", line 1099, in _validate_conn File "urllib3\connection.py", line 616, in connect File "urllib3\connection.py", line 213, in _new_conn urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x0000013D2BF04070>: Failed to establish a new connection: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond
The above exception was the direct cause of the following exception:
Traceback (most recent call last): File "requests\adapters.py", line 486, in send File "urllib3\connectionpool.py", line 847, in urlopen File "urllib3\util\retry.py", line 515, in increment urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='www.1secmail.com', port=443): Max retries exceeded with url: /api/v1/?action=genRandomMailbox&count=1 (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x0000013D2BF04070>: Failed to establish a new connection: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last): File "main.py", line 111, in register File "requests\api.py", line 73, in get File "requests\api.py", line 59, in request File "requests\sessions.py", line 589, in request File "requests\sessions.py", line 703, in send File "requests\adapters.py", line 519, in send requests.exceptions.ConnectionError: HTTPSConnectionPool(host='www.1secmail.com', port=443): Max retries exceeded with url: /api/v1/?action=genRandomMailbox&count=1 (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x0000013D2BF04070>: Failed to establish a new connection: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last): File "main.py", line 930, in File "main.py", line 113, in register RuntimeError: SecEmailAPI: API access error!

Sometimes temporarily 1secmail can go away on tech. Maintenance. Just try again later.

And use latest version


graph deployment to studio

Description :

Make sure that subgraph.yaml got updated with the correct chain name and contract addresses (we could automate this)
~ https://github.com/scaffold-eth/create-eth/pull/25#discussion_r1599915498

This PR tries to automate this nicely but they still need to do some minor tweaks(like startBlock etc).
Maybe we can merge #25 to main and later test this PR nicely and merge this later.
Updates :

Updated abi_copy.ts to be more robust, so that it takes into consideration all networks (previously in https://github.com/scaffold-eth/create-eth/pull/25#discussion_r1590001379 it was only searching for 31337)..... present in both deployedContracts.ts & externalContracts.ts

the reason we are relying on contract abi's from frontend deplyedContracts.ts and externalContracts.ts is this makes its solidityframe work agnoistic and works with both foundry and hardhat.  deplyedContracts.ts will be always up to date with deployments.

Updated "Shipping to Subgraph Studio" section of README with comprehensive instructions (lol maybe went too much comprehensive so feel free to suggest changes)

Updated @graphprotocol/graph-cli from .55.0 => 0.71.2 this allows deployment to new networks like base-sepolia etc

To test follow the same instructions as mentioned https://github.com/scaffold-eth/create-eth/pull/25#issue-2278067294, just make sure you are on graph-prod-deploy branch and once project is scaffolded follow the generated README
Here is the test link : https://final-graph.vercel.app/

Closing this here will create be creating this pr against https://github.com/scaffold-eth/create-eth-extensions/tree/subgraph


Sum kernel

Hello,
I'd like to implement a sum-type kernel in GPy, but with a twist. Assume that I am observing data in the form of ${A_i;F(A_i)},{A_j;F(A_j)},\ldots $ pairs, where $A_i = {a^i_k}{k=1}^{N_i}$ etc. My model is $F(A_i) = \sum_k^{N_k} f(a^i_k)$, so the kernel elements are $K(A_i,A_j) = \sum{k,l} k(a^i_k,a^j_l)$. The problem is that input_dim could be different for two data points and I am not quite sure how to work around that.
From the documentation, it seems I could do $k(a,b) = k_1(a,b)+k_2(a,b)$ but this is not quite what I want. Is there a simple way to define such a kernel?
Thanks in advance.

Seems like the initial sum sum_k^{N_k} f(a^i_k) could be a preprocessing step, if I read that correctly? It's just tough to read the pre-rendered tex - maybe render and post as an image?


Mention that Google Maps API key is now required

Coverage remained the same at 98.251% when pulling bf0cf0446841aa14a8dc541bd4d9a94506135c43 on mb-gmaps-api-key into 91fea170b2788f1de015097a2b1ce291400b641b on master.


DRAFT - Splitting test suite to find flaky tests 2/3

What this PR does / why we need it:
So far we have been adding tests to quarantine one by one and we always find new failing tests, which brought us to think that it is possible that tests with side effects are causing following tests to fail.
This bunch of PRs divides the suite in order to:

Identify portions of the test suite that have 100% pass rate
Identify portions of the test suite that might interfere with each other

This PR is one of multiple copies. They are used to run the CI in parallel.
Release note:
None

/test all

/test all

/test all

/test all

/hold

/retest

/retest

/retest


Widget/Text/overlay additions/toggles

Would it be possible to add an option to “alt” or “ctrl” toggle for ground items to appear/disappear. Also adding more text options that are less bulky for widgets and overlays such as clue scrolls,mouse over text etc.
being able to fill in the background of these overlays and widgets would along with the text changes would be a nice customization option.

Maybe. It would help to close the issues as duplicates if you made 1 issue per request.


Add error handling for not being able to obtain a lock

If we cannot place the message on the queue for whatever reason, eg. upstream services slow to respond, then we should handle that error, and return an appropriate response to the client, instead of the service crashing. We'll still log the issue so that it can be investigated

It seems like the application doesn't actually crash on unhandled errors, closing


Option for auto inline based on filesize

Hi @borodean, another useful feature would be to auto inline assets based on file size if specified.
Like url-loader's limit option.

@tsm91 there was such a feature in 1.x, however we decided to remove it – https://github.com/assetsjs/postcss-assets/issues/1

In the 1.x README you can see how it worked: https://github.com/assetsjs/postcss-assets/blob/aa5f4a45612b89ce57139e5477467428fa2eb2e0/README.md
Assets were converting every url(...) into either resolved or inlined url depending on the file size.

Oh, my bad. It wasn't 1.x, it was 0.x: https://github.com/assetsjs/postcss-assets/blob/941045f12958bbc7213ff326f5b09e267ad7383c/README.md

The issue tells everything. If there would be an overwrite argument to resolve it would do the trick.
like resolve('sample.jpeg', 2, false) <-- false would mean, even if the file size meets the requirements, do not inline this

Leaving this unresolved by now.


Remove version: OpenJS.NodeJS.LTS version 14.19.2

Replaced by another Pull Request
Microsoft Reviewers: Open in CodeFlow

https://www.github.com/microsoft/winget-pkgs/pull/143386#issuecomment-1986949152 [Policy] Needs-Author-Feedback

Close with reason: See discussion in above linked issue;


Issue #97 - Allow HTML in field labels

#97 - Don't escape field labels in field templates

Merged it, works like a charm. Thanks!


gnutls: fixup tpm support

This fixup the build of gnutls to support tpm
Things done

Built on platform(s)

[x] 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
[x] 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

[x] Fits CONTRIBUTING.md.

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

I'm not sure what tpmSupport = true is supposed to do on non-Linux, but I guess noone cares really.

That is actually used for provisioning an Endorsement Key with swtpm. I don’t use it myself, I have another provisioning procedure. But I see it used on macOS for testing/ci purposes.

I don't see it referenced in the current nix expression for gnutls, i.e. usage of tpmSupport is always conditioned by .isLinux or !.sDarwin in there.


Rotation of a Tile by a later Player.

The THB Home tile after laid by Company A, was then able to be rotated by Company B. When Company B chooses to lay a tile, and selects the home tile, it unlocks the rotation for the tile.
Need to Replicate.
Test over a Network

Initial tests on Single-Player cannot replicate issue.

This may be an issue that on Network Game, a Tile is placed, and Rotated, the client is in Tile Mode, and then Exits. The actual "Lock Tile Orientation" step is done on the Client, but no Action/Effect is generated to set that flag on remote clients.

Attempts to replicate over a network game failed.
Re-examination of the code base unlocks the tile rotation at the start of applying a Tile Rotation, and re-locks it at the end of the Tile Rotation.

Finally tracked down this issue -- When Tile Placement was completed (Exit Mode button clicked when in Place Tile Mode) the Tile Rotation was not locked... this allowed on the Client Player MapFrame the tile to be rotated on a future tile lay. The tile is now locked. The LockTileOrientation and unlockTileOrientation is now properly called.


emptyu black box in middle of screen

i get am empty black box in the middle of the screen (this is on current release, with me using the most up to date version of rimworld). i prevents any interaction with the ui. i have to alt-f4 out of the game.

I get the same thing.

No additional information provided or clarification whether using the mod on an unsupported build (one not listed as being compatible with), closing as a result.


Getting Local models/wasm working with Create React App

Question
I realize there's been a lot of talk about this in other issues, but I'm trying to gather if getting local-only model and wasm files will work with Create React App. I'm using WhisperForConditionalGeneration from @huggingface/transformers version 3.0.0-alpha.9.
My setup:
env.allowRemoteModels = false;
env.allowLocalModels = true;
env.backends.onnx.wasm.wasmPaths = process.env.PUBLIC_URL + "/dictation/";
env.localModelPath = process.env.PUBLIC_URL + "/dictation/models/";

... and in my {packagename}/public/models folder I've got:
ort-wasm-simd-threaded.jsep.wasm
models/config.json
models/generation_config.json
models/preprocessor_config.json
models/tokenizer_config.json
models/tokenizer.json
models/onnx/decoder_model_merged_q4.onnx
models/onnx/encoder_model.onnx

This returns the SyntaxError: Unexpected token '<', "<!DOCTYPE "... is not valid JSON error that has been discussed in other issues. If I set env.allowRemoteModels = true; and
env.allowLocalModels = false;, and clear my application cache, this works fine. My questions on that:

How can I get the wasm file to load locally only? It caches fine and calls locally (
http://localhost:3000/dictation/ort-wasm-simd-threaded.jsep.wasm) after the initial CDN call, but I don't want to rely on an external CDN.
How can I get the model files to only call locally? (we will need to further train our own models). I have yet to get this working, but I assume the above error is to blame.
The main question: is this a limitation with CRA? I noticed that if I load the wasm file from the CDN first, it caches fine locally. It's just that initial call to the wasm local file (if not cached from the CDN) that fails, which people have said may be a CRA issue.

Thanks! Sorry for the long-winded question. Happy to provide any more code if needed.

Found the solve for this! Including the fix in case anyone else runs into it. Needed to match the directory structure to the model_id value:
this.model_id = "onnx-community/whisper-base";

models/onnx-community/whisper-base/config.json
models/onnx-community/whisper-base/generation_config.json
models/onnx-community/whisper-base/preprocessor_config.json
models/onnx-community/whisper-base/tokenizer_config.json
models/onnx-community/whisper-base/tokenizer.json
models/onnx-community/whisper-base/onnx/decoder_model_merged_q4.onnx
models/onnx-community/whisper-base/onnx/encoder_model.onnx

Found the solve for this! Including the fix in case anyone else runs into it. Needed to match the directory structure to the model_id value:
this.model_id = "onnx-community/whisper-base";

models/onnx-community/whisper-base/config.json
models/onnx-community/whisper-base/generation_config.json
models/onnx-community/whisper-base/preprocessor_config.json
models/onnx-community/whisper-base/tokenizer_config.json
models/onnx-community/whisper-base/tokenizer.json
models/onnx-community/whisper-base/onnx/decoder_model_merged_q4.onnx
models/onnx-community/whisper-base/onnx/encoder_model.onnx


Add flip function

I wasn't sure what heading to put this under as Control didn't seem right so I created a new section Function.

Let's move the Function section to immediately after the Composition section (in both files).

I'm sorry for neglecting this pull request, Stefano. If you rebase it (to ensure the tests pass) I'll merge it. :)

Oops, I don't know what I did but I didn't mean to close this and i seem to have removed the commits from the PR!

I've totally messed this up, I think I'll just create a new branch from master with the flip code tomorrow.

If you reopen this pull request then push a commit to svozza:flip-function, we should be able to revive the pull request. :)

So tried to fix it but just getting tied up in knots. I've to be at a wedding in a few hours so I haven't time to untangle this mess. Haha. I'll just open a new PR.


DetectPosesInImage() takes forever to do inference

First of all, thanks for making this package opensource. I'm using your package on a Raspberry Pi 4B with EdgeTpu to do pose estimation.
The instructions in the readme worked perfectly for me, and I was able to get the keypoints positions and confidence scores. However, I am facing a problem when I'm trying to use your package in my own project. In my project I have a method which uses DetectPosesInImage() to get the pose estimate from an image. My method is stuck at DetectPosesInImage(), and taking forever to do inferencing. I m unable to find where the problem is
Here is the code where it is getting stuck
        # All initial Image Processing Starts here
        count +=1
        ok, frame = self.video.read()
        if not ok:
            break
        img = frame

        hsl_img = cv2.cvtColor(img,cv2.COLOR_BGR2HLS)
        hsl_img[:,:,1] = cv2.equalizeHist(hsl_img[:,:,1])
        img = cv2.cvtColor(hsl_img,cv2.COLOR_HLS2BGR_FULL)

        # All initial Image Processing Ends here

        frame_ = cv2.resize(img, (641, 481))
        poses_, inference_time_ = self.pose_estimator.engine.DetectPosesInImage(frame_)

Additional information:
frame size = (480X640)  # (widthxheight)
model = posenet_mobilenet_v1_075_481_641_quant_decoder_edgetpu.tflite
This method which I implemented uses only 1 core out of 4 cores available on Raspberry Pi 4. I'm doing this using multiprocessing package in python.
Any help is much appreciated. Do let me know if additional information is required. Thanks in advance.

Hmm, from this its hard for me too know where the issue is. If the provided example works and this doesn't, you should be able to compare the exact inputs into the inference call in DetectPosesInImage and figure out what the difference is.
Does the call hang or just proceed very slowly ?
Note that because there is only one TPU you can't have multiple threads trying to schedule work on the TPU simultaneously. You mention multiprocessing - if several callers are trying simultaneously to use the TPU it wont work and threads may deadlock, maybe that's the problem ?

I compared exact inputs and tried using  DetectPosesInImage() with a sample toy script. It worked. But it doesn't work in my project.
I don't know if it's hung/ doing slowly. Is there any way to check if the inferencing is in progress?
I'm not calling TPU from several threads. What I'm referring is, 'I'm only calling TPU from single process'. For example, I have a 4 cores for my RasPi, 1 use one core (1 process) to perform tracking, 1 core to turn the motors, 1 core to run a PID controller. So, basically I'm doing TPU inferencing inside the tracking process (which uses only one core). Here's where I'm experiencing an issue. I suspect the problem could be here, Does your inferencing method uses more that 1 core? can a single core be able to handle it?

Hi @mtyka . I found out the problem was with the multiprocessing. The inference doesn't work (or) takes forever if I do the inference within a Process (multiprocessing.Process).
I couldn't fix the issue, but I was able to figure out an alternative solution for my application without using multiprocessing.Process

Closing due to inactivity. I think, its not possible to use the TPU with multiprocessing.


Create meta-packages to bring built in NuGet dependencies

We need to discuss the granularity, but we should provide meta-packages (tracking the runtime version) to bring the appropriate dependencies available as shared assemblies, which will make the local development process simpler.

This impacts the dev experience of bringing a .dll instead of a .csx for using things like TraceWriter/etc.


HEADROOM-3: Upgrade packages

Is there a command to install flow types without stubs? If not I think it's easier to have them in the repo, so flow-typed install doesn't do anything if there are no new types.

There is the --skip parameter: https://github.com/flow-typed/flow-typed/wiki/CLI-Commands-and-Flags#flags-1
I think we should use this to avoid these bloated change sets.

@maxammann Thanks, I removed the stubs.


net/http: invalid http request methods accepted in NewRequest and NewRequestWithContext

What version of Go are you using (go version)?

$ go version
go version go1.15.7 darwin/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN="/Users//Documents/go/bin"
GOCACHE="/Users//Library/Caches/go-build"
GOENV="/Users//Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/markus.mahlberg/Documents/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users//Documents/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/opt/local/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/opt/local/lib/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="/usr/bin/clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users//go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/dr/k5sk_14s2t1dzqzqxfbq5j5c0000gn/T/go-build193393271=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
	req, err := http.NewRequest("foobar", "http://www.example.com", nil)
	if req!= nil && err == nil {
		fmt.Printf("Method %s was accepted by 'http.NewRequest'", req.Method)
	}

https://play.golang.org/p/CEpOCUm_RmA
What did you expect to see?
I expected reqto be nil and err.Error() to be "net/http: invalid method foobar" as per https://github.com/golang/go/blob/fa6752a5370735b8c2404d6de5191f2eea67130f/src/net/http/request.go#L845-L847
What did you see instead?
A request with an invalid method.

"foobar" is a valid HTTP method: https://github.com/golang/go/blob/fa6752a5370735b8c2404d6de5191f2eea67130f/src/net/http/request.go#L794

@ericlagergren True. Forgot the extensions. Works as intended.


Allow local non-root certificate generation

Use Case
Create SSL certificates locally. In a second step (another role) the certificates could then be distributed to the remote hosts.
Current situation
Non-root user without sudo rights cannot create certificates with the role locally. E.g. running
- name: Local preparations
  hosts: localhost
  roles:
    - role: ome.ssl_certificate

fails, because the role requires sudo rights.
Improvement
Remove the become: true from the tasks in tasks/main.yml. This allows normal users to generate certificates locally. E.g.
- name: Local preparations
  hosts: localhost
  roles:
    - role: ome.ssl_certificate
  vars:
    ssl_certificate_selfsigned_create: True
    ssl_certificate_selfsigned_subject: "/CN=localhost"
    ssl_certificate_selfsigned_days: 365
    ssl_certificate_public_path: etc/ssl/localcerts/global.crt.pem
    ssl_certificate_key_path: etc/ssl/localcerts/global.key.pem
    ssl_certificate_bundled_path: etc/ssl/localcerts/global.bundled.crt
    ssl_certificate_combined_path: etc/ssl/localcerts/global.combined.pem

The old use-case, remote certificates with root rights can easily be reproduced with
- name: Local preparations
  hosts: localhost
  become: true
  roles:
    - role: ome.ssl_certificate

Thanks for the feedback @vinjana. There are a few codepaths but overall the current logic of of the role is:

either copy certificates e.g. from an Ansible vault or a password store to a remote location
or generate self-signed certificates on the remote server via openssl
optionally generate the combined/bundled certificates

One of my main concerns with the solution proposed above is its breaking impact as every single consuming playbook will need to explicitly become: true. Coming back to your use case and specifically  In a second step (another role) the certificates could then be distributed to the remote hosts.,  this is precisely the primary intent of this role i.e. managing SSL certificates on production environments.
I concur that the SSL generation step does not need to be executed with privilege escalation in a local-only context (although it might still be needed to ensure openssl is installed). Is there a way to isolate these tasks so that the default behavior can still invoke them with a become: true wrapper but one can also generate local certificate?

It seems not to be possible to override a static become: true in the tasks of the role. So the only alternative solution I see in the moment is to introduce a variable ssl_become and set become: "{{ ssl_become }} in the tasks. This seems ugly to me, but it  works. The default could then be set to true for backwards compatibility.
Furthermore, for my specific use case, it would be good if the installation task could be turned off. Ideally this would be implemented with a when: "{{ ssl_install_openssl }}" in the package installation task. I am not sure, whether this is possible with tags.
Still, to me the best solution seems be to remove the become from the role's tasks, and make the different functions of the role executable separately -- probably by variables and when. Then it's possible to freely combine become and each function (install, create cert, distribute cert) of the role (locally or remote). But these are just ideas. I can just make a fork and extend the role however I want -- thanks to your nice work!


Move RL experiments out of main directory. Treat Maddux as module/dependency

Moved into experiments and put predefined environments into maddux. Everything should work but something might have broken.


fix(docs): update oxidizer website url

Motivation
The project is no longer associated with a custom domain.
Have you read the Contributing Guidelines on pull requests?
Yes.
Related PRs
@PatelN123 kindly informed us about the website appearing down at https://github.com/oxidizer-rs/website/issues/163, which is related to https://github.com/facebook/docusaurus/pull/6231.

Awesome! Glad I could be of use

Awesome! Glad I could be of use


Disk snapshots fail on GCP regional volumes

What steps did you take and what happened:
When using regional volumes on GCP, backups fail with the error below. Here is a basic configuration for creating a PVC backed by a regional volume.
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: regional-magnetic
provisioner: kubernetes.io/gce-pd
parameters:
  type: pd-standard
  replication-type: regional-pd
  zones: us-central1-b, us-central1-c
reclaimPolicy: Retain
---
apiVersion: v1
kind: PersistentVolume
metadata:
  name: regionalVol
  labels:
    app: foo
spec:
  storageClassName: regional-magnetic
  capacity:
    storage: 10Gi
  accessModes:
    - ReadWriteOnce
  gcePersistentDisk:
    fsType: ext4
    pdName: regionalVol
---
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: regionalPVC
spec:
  accessModes:
    - ReadWriteOnce
  storageClassName: regional-magnetic
  selector:
    matchLabels:
      app: foo
  resources:
    requests:
      storage: 10Gi

What did you expect to happen:
I expected the snapshots to be created similarly as they are for non-regional volumes.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
time="2018-08-14T05:56:14Z" level=error msg="backup failed" error="[error creating snapshot: rpc error: code = Unknown desc = googleapi: Error 400: Invalid value 'us-central1-f__us-central1-c'. Values must match the following regular expression: '[a-z](?:[-a-z0-9]{0,61}[a-z0-9])?', invalidParameter, error creating snapshot: rpc error: code = Unknown desc = googleapi: Error 400: Invalid value 'us-central1-b__us-central1-c'. Values must match the following regular expression: '[a-z](?:[-a-z0-9]{0,61}[a-z0-9])?', invalidParameter, error creating snapshot: rpc error: code = Unknown desc = googleapi: Error 400: Invalid value 'us-central1-f__us-central1-c'. Values must match the following regular expression: '[a-z](?:[-a-z0-9]{0,61}[a-z0-9])?', invalidParameter, error creating snapshot: rpc error: code = Unknown desc = googleapi: Error 400: Invalid value 'us-central1-b__us-central1-c'. Values must match the following regular expression: '[a-z](?:[-a-z0-9]{0,61}[a-z0-9])?', invalidParameter]" key=heptio-ark/my-backup logSource="pkg/controller/backup_controller.go:280"

Environment:

Ark version (use ark version): 0.9.3

Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.1", GitCommit:"b1b29978270dc22fecc592ac55d903350454310a", GitTreeState:"clean", BuildDate:"2018-07-18T11:37:06Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10+", GitVersion:"v1.10.5-gke.4", GitCommit:"6265b9797fc8680c8395abeab12c1e3bad14069a", GitTreeState:"clean", BuildDate:"2018-08-04T03:47:40Z", GoVersion:"go1.9.3b4", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes installer & version: gke
Cloud provider or hardware configuration: gcp

Thanks for reporting this @robbyt, we'll look into it and let you know when we can get it fixed

@skriss I put this against the backup replication Epic

@rosskukulinski given that this appears to be a bug, I'm not sure putting in under replication is appropriate?

@wwitzel3 PTAL

The problem is that in GKE, when using a regional volume we get two labels:
failure-domain.beta.kubernetes.io/region: us-central1
failure-domain.beta.kubernetes.io/zone: us-central1-b__us-central1-c

Ark looks for failure-domain.beta.kubernetes.io/zone and passes that along to the disks.createSnapshot API call passed in as the zone. Which results in the failure, since us-central1-b__us-central1-c is not a valid zone.
I came up with a solution, but after testing it and doing a bit more research, my fix of parsing the combined zones out of the failure-domain.beta.kubernetes.io/zone and then passing them to our existing CreateSnapshot call was a bit naive.
I think we need to look in to checking for the failure-domain.beta.kubernetes.io/region label and if we find it, using the regionDisks API to perform our operations.

So I'm thinking then, we could leave the plugin interface as it, and just special case this in the gcp plugin, checking the incoming volumeAZ for __. 👍

Most definitely - this is quite GCP specific

Also worth mentioning that regional disks in GCP are beta at this time

Should we add a volumeRegion (along side volumeAZ) to the BlockStore interface methods? As of now, GCP would be the only provider to use that field, so it seems like the wrong way to go.
I keep writing stuff and then just deleting. I can just pick an approach, put up the PR, and let people digest it and see if we want to change it.

How is the status on this? It is still a problem in my version of ark

@bartimar we have a PR in flight for this -- look for it in an upcoming patch release (we do these every 2 weeks on thursdays)


#244: fix extra symbol T in f.since.utc.iso8601[0..10]

@Yegorov thanks!

@Yegorov Thanks for your contribution of 10 hits-of-code! You've earned +5 points: +25 base, -15 for no code review, -10 for no reviewer comments, -10 for fewer than 20 hits-of-code, +15 to meet the minimum. Remember, more hits-of-code and encouraging code reviews can boost your points significantly. Keep up the good work and aim for those higher bonuses! Your running balance is +161.


Change JSX sample

Parsing labels to atoms is a bad practise as it can lead to attacks overflowing the atom table and filling the memory with garbage. I switched the example to a different parser option that is safe to use.

Right, that was just the first thing that came to my mind.
I agree we should not promote bad/dangerous practices, thanks!


Unexpected crash trying to build types for @google-cloud/...

Command line:
% dts-gen -m @google-cloud/nodejs-repo-tools
Result:
Unexpected crash! Please log a bug with the commandline you specified.
/usr/local/lib/node_modules/dts-gen/bin/lib/run.js:125
throw e;
^
Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>at Object.openSync (fs.js:447:3)
at Object.writeFileSync (fs.js:1206:35)
at Object. (/usr/local/lib/node_modules/dts-gen/bin/lib/run.js:107:12)
at Module._compile (internal/modules/cjs/loader.js:774:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:785:10)
at Module.load (internal/modules/cjs/loader.js:641:32)
at Function.Module._load (internal/modules/cjs/loader.js:556:12)
at Function.Module.runMain (internal/modules/cjs/loader.js:837:10)
at internal/main/run_main_module.js:17:11 {
errno: -2,
syscall: 'open',
code: 'ENOENT',
path<EMAIL_ADDRESS>}
When I explicitly state the output file:
% dts-gen -m @google-cloud/nodejs-repo-tools -f nodejs-repo-tools.d.ts
it works fine. I suspect the issue is the @ in the path when writing the output. I tried escaping it but same result. Note, I get the same issue with the -d option

You need to specify the output file name since by default it would use the input package name which does not make for a valid file name.

@dcramb give it a shot creating the @google folder first
mkdir @google
dts-gen -m @google-cloud/nodejs-repo-tools

I had the same issue with some other namespaced module and creating the namespace dir first works around the issue.
However, I think dts-gen should figure it out :-)


Agent: make AgentWorkspaceDocuments more robust

The PR https://github.com/sourcegraph/cody/pull/4277 was a lesson that we have very brittle code around dealing with document synchronization. This PR is an attempt to improve code health by

Adding tests!
Being more rigorous with null handling
Making the code a bit easier to follow (hopefully)

Test plan
See new test suite.

The integration tests caught a very critical regression (nice!). While fixing this, I saw that our implementation of vscode.TextEditor is not using a singleton instance for selection, which I'm fixing now.

I will do a second round of review tomorrow since the additional diff ended up being quite large after your approval @abeatrix .

Just to be 100% sure, I ran VS Code with the local diff
--- a/vscode/src/main.ts
+++ b/vscode/src/main.ts
@@ -753,6 +753,11 @@ const register = async (
         platform.extensionClient.provide({ enterpriseContextFactory }),
     ])
     disposables.push(extensionClientDispose)
+    for (const editor of vscode.window.visibleTextEditors) {
+        setInterval(() => {
+            console.log({ uri: editor.document.uri.toString(), text: editor.document.getText() })
+        }, 1000)
+    }

and confirmed that the text contents reflect the latest version as I make changes to the document even if I reuse the vscode.TextEditor instance.


AWS S3 cache miss should not issue warning

This fixes https://github.com/microsoft/vcpkg/issues/31932
Previously, when using the AWS S3 binary cache, every cache miss triggered the following message:
warning: error: aws failed with exit code: (1).

This was both noisy and misleading (because it was not an actual error).
Now AWS cache misses are treated silently, the same way as cache misses from the local cache are.

[!NOTE]
I don't have access to either Google Cloud Storage or Tencent Cloud Object Storage sources, so I don't know whether the same kind of issue happens there too; in any case, fixing those should be easy using the newly introduced interface. (Their stat() and download_file() methods need to be adapted to return CacheResult::miss in the relevant cases.)

@microsoft-github-policy-service agree company="Formlabs"

Thanks for the fix!

Thanks for your detailed review! I'll address your comments in a couple days. I added some questions in replies where something was unclear to me.

@BillyONeal
What's your preference on changes addressing review comments? Should I force-push the branch to rewrite history with the "correct" version incorporating the changes, or should I simply add my changes on top of the current ones in a separate commit?

@BillyONeal What's your preference on changes addressing review comments? Should I force-push the branch to rewrite history with the "correct" version incorporating the changes, or should I simply add my changes on top of the current ones in a separate commit?

There isn't widespread surgery here so I think you can do whatever you want :)

@BillyONeal: I've updated my branch. I split the changes into two commits:

The first one does not change the behaviour at all, only introduces the ability to return CacheAvailability/RestoreResult, but all providers always either return available or error, just like before.
The second one makes AWS's stat() return unavailable instead of an error in case the standard output is empty, and the exit code is either 1 (current behaviour of aws-cli) or 0 (possible future behaviour if they decide the address this issue).

If you have a better name for flatten_generic(), I'm open to suggestions.

I learned today that you can do this


Chrk/2 charts

resolves #2

resolves https://github.com/opensource-construction/model-checker/issues/7 too


chore: release 0.1.52

Missing version bump in https://github.com/blockfrost/openapi/blob/master/src/definitions.yaml

Missing version bump in https://github.com/blockfrost/openapi/blob/master/src/definitions.yaml

good catch! it would be great to mention it in the readme.


[SPARK-41595][SQL] Support generator function explode/explode_outer in the FROM clause

What changes were proposed in this pull request?
This PR supports using table-valued generator functions in the FROM clause of a query. A generator function can be registered in the table function registry and resolved as a table function during analysis.
Note this PR only adds support for two built-in generator functions: explode and explode_outer with literal input values. We will support more generator functions and LATERAL references in separate PRs.
Why are the changes needed?
To make table-valued generator functions more user-friendly and consistent with Spark's built-in table function Range.
Does this PR introduce any user-facing change?
Yes. Before this PR, the built-in generator function explode/explode_outer cannot be used in the FROM clause:
select * from explode(array(1, 2))

AnalysisException: could not resolve `explode` to a table-valued function;

After this PR, we can support this usage:
select * from explode(array(1, 2))

+---+
|col|
+---+
|  1|
|  2|
+---+

How was this patch tested?

New SQL query tests.

cc @cloud-fan

The failed python ml tests are definitely unrelated, merging to master, thanks!


msq: update known issue about GROUPING SETS and COUNT DISTINCT

This PR has:

[x] been self-reviewed.

thanks!


Update to Bit

:warning: Validation status: warnings

File
Status
Preview URL
Details

advocates/index.html.yml
:warning:Warning
View
Details

advocates/index.yml
:warning:Warning
View
Details

advocates/media/BitMicrosoft.png
:white_check_mark:Succeeded

advocates/media/bit.png
:white_check_mark:Succeeded
View

advocates/media/bitda.png
:white_check_mark:Succeeded
View

advocates/index.html.yml

[Warning] the value ProfileList is undefined for metadata pagetype

advocates/index.yml

[Warning] the value ProfileList is undefined for metadata pagetype

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:warning: Validation status: warnings

File
Status
Preview URL
Details

advocates/index.html.yml
:warning:Warning
View
Details

advocates/index.yml
:warning:Warning
View
Details

advocates/media/BitMicrosoft.png
:white_check_mark:Succeeded

advocates/media/bitda.png
:white_check_mark:Succeeded
View

advocates/index.html.yml

[Warning] the value ProfileList is undefined for metadata pagetype

advocates/index.yml

[Warning] the value ProfileList is undefined for metadata pagetype

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Prepare non-interop network to test full deposit contract flow

We need to have non-interop network running on GCP cluster like dev and staging. Because of this there are requirements to:

Prepare genesis.ssz flow for non-interop.
Prepare namespace configuration for non-interop network.
Verify chain-config.yaml.
Deploy network changes to the default cluster namespace.

Branch is https://github.com/lukso-network/l16-multinet/tree/epic/41/deposit-contract-flow

Except the 4 we have reached stable network. @atif-konasl and @meta-bot provided working solution based on gcloud. We should consider if point no 4 is needed at all in k8s

Except the 4 we have reached stable network. @atif-konasl and @meta-bot provided working solution based on gcloud. We should consider if point no 4 is needed at all in k8s

You are right, it's not required at all.
@atif-konasl and @meta-bot have prepared working l15 non-interop test network in l15-testnet project:

Ethstats:

In connection with Prepare genesis.ssz flow for non-interop. we are able to generate ssz without mainnet flag by:
bazel build //tools/genesis-state-gen --//proto:network=minimal from prysm commit id: e4388c7fce6bcc27155bae2860a6ab4f550fb6af and then just generate by:
./genesis-state-gen --output-ssz=./vanguard_private_testnet_genesis.ssz \
--output-json=./vanguard_private_testnet_genesis.json \
--deposit-json-file=./deposit_data.json \
--genesis-time="$GENESIS_START"


g.dev/AustinRay-4031630597026839
content_copy

https://g.dev/AustinRay-4031630597026839

@Ag2279122,
We see that the issue template has not been filled, could you please do so as it helps us analyze the issue [SDK version, steps followed before you ran into this error or stand alone code/colab gist to reproduce the issue faced and reopen this issue. Thanks!


slstk3400a : Update doc and periph

Updated doc.txt and perip_conf.h.
Ref : https://hackmd.io/njFHwQ33SNS3sQKAkLkNtQ

I'll close this issue now that we have a PR for RIOT-OS.


How to drag a item between two ReactGridLayout

[ ] Bug
[ ] Feature Request
[X] Question

<ReactGridLayout   layout={this.state.layout1}>         {this.generateDOM()} </ReactGridLayout> <ReactGridLayout layout={this.state.layout2} >         {this.generateDOM()} </ReactGridLayout>
anyone resolve this question?

If it can be done I'd love to know as well... it may be possible since this is based on react-draggable and that library allows dragging outside of the containing control.

@fairps  thankyou, but react-beautiful-dnd is only for list , the ReadMe.md say

Not for everyone
There are a lot of libraries out there that allow for drag and drop interactions within React. Most notable of these is the amazing react-dnd. It does an incredible job at providing a great set of drag and drop primitives which work especially well with the wildly inconsistent html5 drag and drop feature. react-beautiful-dnd is a higher level abstraction specifically built for vertical and horizontal lists. Within that subset of functionality react-beautiful-dnd offers a powerful, natural and beautiful drag and drop experience. However, it does not provide the breadth of functionality offered by react-dnd. So this library might not be for you depending on what your use case is.
so I will try to use react-dnd to ceate a wheel  project just like GridLayout.

I've looked at this much further after my last comment... maybe PR #682 that is waiting to be merged might allow react-grid-layout to act as a drop target for incoming objects.  Don't underestimate the amount of work that has been done in react-grid-layout and react-beautiful-dnd beyond what react-dnd is offering out of the box... both are using react-motion to provide a drag experience that feels natural.  For me, the only difference right now between react-grid-layout and react-beautiful-dnd with a series of side by side lists is the inability to resize elements and have them extend across those columns (lists).  If PR #682 will allow the drop target, then I will use react-beautiful-dnd to create a toolbox of sorts and drag items from it into my primary react-grid-layout... or at least that's how I am leaning at the moment.

@fairps  Yeah, I had used react-dnd successfully to complete card grid layout and move card to other list, see this in my drag gif.

@SunXinFei got a sample for that? About to do that myself!

@martinwebrant    my code, you can view this code, but sorry  I did not have a sample for that, maybe complete the project，free time will commit a sample templete.
I have been optimizing drag and drop render performance these days，

@SunXinFei thanks for posting that link.  I Really appreciate it.

@fairps ，recently I have been optimizing and modifying this code

@SunXinFei  Hey, I've run into some trouble starting your code. Does it still work?

@SunXinFei Hey, I've run into some trouble starting your code. Does it still work?

oh!   I create the project react-grid-layout-between. you can run it, maybe it can help you  . when i'm not busy , i will create the component.

@SunXinFei  Thank you! Works perfectly well :)

@SunXinFei Thank you! Works perfectly well :)

@blizzerand  yeah, just a moment ago,  I  added the logic about switch Compaction Type: horizontal || vertical.

@SunXinFei Have you completed the awesome functionality?

@fairps Yeah, I had used react-dnd successfully to complete card grid layout and move card to other list, see this in my drag gif.

@SunXinFei Have you completed the awesome functionality?

@fairps Yeah, I had used react-dnd successfully to complete card grid layout and move card to other list, see this in my drag gif.

you can view this project https://github.com/SunXinFei/react-grid-layout-between

Not stale just ignored

Hi Good morning all, please I want to achieve this http://mmoinuddin.com/assets/img/grid-layout-dnd.gif, the goal is to extend the algorithm, so a module can move within the layout container. The module can't move outside of the top, right, and left edges of the layout container. The bottom edge behaves differently. When a module is moved downwards beyond the bottom edge, the bottom edge should also move downwards to extend the height of the layout container. Also, Module objects cannot overlap each other.
So how can I achieve that using React-typescript

I solved it by creating two grids with isDraggable to false and followed the "Drag from outside" example.

@spedy please can I see the code of how you implemented it .


Create sponsoring and contribution layers for the Node-RED project

Current Behavior
I cannot sponsor @knolleary if I wanted to on GitHub (GitHub Sponsor) and the project has no Open Collective (https://opencollective.com/).
Expected Behavior
I would like myself and others to be able to reciprocate to this project.
Steps To Reproduce
Stop it.
Example flow
💰!= 🤗  ... but it is still needed
Environment

Node-RED version: NA
Node.js version: NA
npm version: NA
Platform/OS: NA
Browser: NA

Thanks for the interest in supporting the project. As you note, we don't have an OpenCollective, and it's up to individuals whether they use GH Sponsors or not - something I have chosen not to use.
I appreciate the intent of the issue, but I'm going to close it off.

NodeRED had been an instrumental tool in my teaching for the last 3 years. It had created plenty of Aha moments for my students. What gets you going and how can I express my gratitude to the project you started.
I can show up at your door with flowers 🌹 But there's no guarantee it would be something you'd appreciate.


Better docs for Faker::Food.description

Insert docs for Food Class

@jujulisan could you also rebase this branch with master and fix the Rubocop violations? If you need instructions, check the item 7 out.


Change from Lodash to Immutable

Inspired by the wave of recent changes from @regiskuckaertz I thought I'd try and clean up some of the remaining dependencies on Lodash and also constant declarations that we make that are not explicitly Immutable.

Awesome :+1: I have a rewrite of UndoManager with Immutable, let me know if it can help

Have spoken to @rrees, we will alias Immutable in a separate PR. :+1:


Add mkdir as part of .NET core install instructions

From issue: https://github.com/dotnet/cli/issues/3988
Else the rest will fail.
Also add in that you have to have Homebrew installed.

hmmm.  Didn't need to do.
Do these instructions also need updating?
https://www.microsoft.com/net/core#macos

@bryanmacfarlane I would recommend adding it in. Essentially if you don't have a /usr/local/lib/ then it will fail to link. This causes issues with the vsts-agent.
On a fresh machine it does not have it.
I have now tested on 3 machines and 2 of the 3 times it has failed. Others have also ran into this issue.  As you can see here: https://github.com/dotnet/cli/issues/3988

OK - I had it on my fresh machine.  hmmm
Put commend above
ensure directory exists


redirecting logged in user to home. also contains test

Closes #3316

Automated message from Dropbox CLA bot
@avisrivastava254084, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

  Check out the Zulip commit message guidelines for more details: https://zulip.readthedocs.io/en/latest/version-control.html#commit-messages

@timabbott
Will this be good?
"Redirecting to home: Redirects the logged in users, landing on the login page, to the home page.
This fixes the issue of users experiencing the login page despite they being already logged in. It redirects them to the home page, which is the normal and expected behavior. Along with the redirection view, test is written to test the mentioned behavior.

I think that's too long, something under 80 characters would be nice.
Something like auth: Redirect users to home from /login if logged in

You should squash both commits into one with the commit message of the second commit.

@avisrivastava254084 can you rebase?  It appears you've gotten some recently merged changes from other developers squashed into your branch.

@timabbott I have rebased. I think I haven't been successful though.

Check out our GitHub guide and commit guidelines for details on how to rebase properly.
I went ahead and rebased this for you and merged this as 762b84710e124438634a5287e96a813187e4faec, but it's probably worth your doing a bit of practice with Git.  Thanks @avisrivastava254084!.

@timabbott Thanks fir rebasing it. I have now completed the git and github guide. Will abide by the guidelines from now on.


🐛 [unknown/unknown] Value for flash cannot be cast from Boolean to String

What were you trying to do?
Was trying to take a photo in android  11
Reproduceable Code
<Camera
						style={styles.cameraContainer}
						ref={camera}
						device={device}
						isActive={true}
						photo={true}
						torch={'off'}
					/>

What happened instead?
but was greeted with this error in metro :
🐛 [unknown/unknown] Value for flash cannot be cast from Boolean to String
Relevant log output
Possible Unhandled Promise Rejection (id: 0):
unknown/unknown: [unknown/unknown] Value for flash cannot be cast from Boolean to String
construct@[native code]
Wrapper@http://localhost:8081/index.bundle?platform=android&dev=true&minify=false&app=com.basementsports&modulesOnly=false&runModule=true:7135:61
construct@[native code]

Device
android 11 realme 8
VisionCamera Version
"^2.12.0"
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

You are passing a wrong value to the flash parameter. Also you didn't share your code, only the camera view.

Take out that torch={'off'} from camera component & then you can either add it in the camera.current.takePhoto code or omit it so that default is taken


Create main.yml single file init

test of nodeJS build

Checklist

[x] npm install && npm test passes
[x] tests are included
[ ] documentation is changed or added
[ ] commit message follows commit guidelines

Description of change

test z


Async signal handler causes error "Attempted to convert an unmarshallable managed type to Variant. Name: 'Task`1'"

Godot version
3.4.2 stable
System information
macOS Monterey 12.1 on MacBook Air M1 (8 cores, 16GB RAM)
Issue description
Error:
E 0:00:08.059   mono_object_to_variant_impl: Attempted to convert an unmarshallable managed type to Variant. Name: 'Task`1' Encoding: 21.
  <C++ Error>   Method failed. Returning: Variant()
  <C++ Source>  modules/mono/mono_gd/gd_mono_marshal.cpp:927 @ mono_object_to_variant_impl()
  <Stack Trace> :0 @ void Godot.NativeCalls.godot_icall_2_635(IntPtr , IntPtr , System.String , System.Object[] )()
                Object.cs:364 @ void Godot.Object.EmitSignal(System.String , System.Object[] )()
                Player.cs:96 @ void Player._PhysicsProcess(Single )()

The flow is essentially:

Player emits signal EmitSignal(nameof(ArrivedAtDestination), Position);
In _Ready(), game connects to signal: Player.Connect(nameof(Player.ArrivedAtDestination), this, nameof(HandlePlayerArrivedAtDestination));

Making the signal handler async causes the error above:
async Task HandlePlayerArrivedAtDestination(Vector2 localPos)
{
  await SomeAsyncMethod();
}

Changing the handler to void fixes the issue:
void HandlePlayerArrivedAtDestination(Vector2 localPos)
{
  SomeAsyncMethod();
}

Problem: async calls cannot be awaited if the handler is void.
Steps to reproduce
Emit a signal and make the connected handler async Task.
Interestingly, even with the error showing everything seems to work just fine.
Minimal reproduction project
No response

Something similar was discussed in https://github.com/godotengine/godot-proposals/issues/3527#issuecomment-965577154
The summary is: Godot doesn't support awaiting asynchronous methods but your method can still be declared async and use await.
To use your code as an example:
async void HandlePlayerArrivedAtDestination(Vector2 localPos)
{
    await SomeAsyncMethod();
    SomeOtherMethod();
}

The caller won't be able to wait for HandlePlayerArrivedAtDestination to finish because it returns void but the SomeOtherMethod is called after SomeAsyncMethod finishes as expected. Since the caller in your example is Godot (because you are using Connect) waiting would mean freezing the game which is not desired.
If you need to know when the method has finished you can use signals, emit a signal at the end of the method and you can execute whatever behavior needs to be executed after the method is done.

@raulsntos thanks for explaining. I understand that it is unsupported scenario and could potentially lead to a deadlock.
My question/suggestion is:
If it's not supported to have signal handlers declared async Task (which is making total sense), then fail properly and state that it's unsupported instead of leaving the user with a mysterious error message.

Yeah, that's definitely a reasonable request.


Document custom query timeouts.

@Gouthamve @fabxc

I think what we have here is a good solution addressing a good chunk of the
problem.
We can discuss other rules for a second iteration later.
On Sun, Mar 12, 2017 at 4:36 PM Brian Brazil<EMAIL_ADDRESS>wrote:

@brian-brazil commented on this pull request.
In content/docs/querying/api.md
https://github.com/prometheus/docs/pull/678#discussion_r105560479:

@@ -71,6 +71,7 @@ URL query parameters:

query=<string>: Prometheus expression query string.
time=<rfc3339 | unix_timestamp>: Evaluation timestamp. Optional.
+- timeout=<duration>: Evaluation timeout. Optional. Defaults to value of -query.timeout flag.

If they know the result is of no use to them if it's not fast.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/docs/pull/678#discussion_r105560479, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEuA8n-irQbs9e3A0E90u9ONUcYvWQaGks5rlBDygaJpZM4Maj3y
.

Ok, I updated the description.
I really like the ability to set a custom timeout, I'm just wondering if someone really has a use case for setting it lower, not higher, than the default.


Tag import error codes

Depending if there is a user expected error or a platform unexpected one.
In order to do that I've tagged every error with 'user' or 'cartodb'.
Please CR @rafatower
Also, @javisantana @saleiva please review the set tags per code

Frontend tests were OK :+1: (details)

:+1:

Deploying it. We can switch tags for a error code if we find necessary at any moment.


Slow upgrade / any action and high CPU utilization

Rancher Versions:
Server: 1.3.4
Cattle: 0.175.10
healthcheck: 0.2.0
ipsec: 0.0.2
network-services: 0.0.8
scheduler: 0.3.0
kubernetes (if applicable):
Docker Version: 1.13.0
OS and where are the hosts located? (cloud, bare metal, etc): ~50 hosts Ubuntu16, Bare Metal ~250 containers in 5 services.
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB) External DB r3.large, 2 Rancher hosts c4.xlarge (4 cores)
Environment Type: (Cattle/Kubernetes/Swarm/Mesos) Cattle v0.175.10
Steps to Reproduce:
Run rolling upgrade/finish/rollback on any service or perform almost ANY action on containers
Results:
It takes hours to rotate ~50 containers
both Rancher "masters" have CPU Utilization 100%
When running basic java prfiler:
./jvmtop.sh 9 --profile -n 50
we got:
 JvmTop 0.8.0 alpha - 06:15:02,  amd64,  4 cpus, Linux 4.4.0-59-, load avg 12.90
 http://code.google.com/p/jvmtop
 Profiling PID 9:         io.cattle.platform.launcher.Main
  23.02% (     2.13s) org.yaml.snakeyaml.resolver.Resolver.resolve()
  15.21% (     1.63s) org.yaml.snakeyaml.emitter.Emitter.writePlain()
  14.23% (     1.52s) ....yaml.snakeyaml.representer.SafeRepresenter$Represent()
   5.87% (     0.63s) ....yaml.snakeyaml.representer.Representer.representJava()
   5.78% (     0.62s) org.mariadb.jdbc.internal.util.buffer.ReadUtil.readFully()
   4.10% (     0.44s) org.yaml.snakeyaml.emitter.Emitter.writeIndent()
   3.97% (     0.43s) org.yaml.snakeyaml.emitter.Emitter.writeIndicator()
   3.41% (     0.37s) org.yaml.snakeyaml.emitter.Emitter.analyzeScalar()
...

It spends more 70% of CPU time encoding and decoding YAMLs!!!!!!

Any idea regarding this issue, please?

@tholcman was this an upgraded setup? What do you see under the admin -> processes tab?

@aemneina:
@tholcman is my co-worker, rancher is not upgraded, all there is in Admin -> processes is timed out volumestoragepoolmap.remove processes; clicking on them to view on API returns a 404:
{
"id": "c60f8c1f-06fc-4ddd-9ec2-fc73435c51e0",
"type": "error",
"links": { },
"actions": { },
"status": 404,
"code": "Not Found",
"message": "Not Found",
"detail": null,
"baseType": "error"
}

When there is no running service upgrade / scaling / whatever there are just about 25 of these processes and processors are quiet (2-2,5 CPU utilization).
...
1pi210943	volumestoragepoolmap.remove	volumeStoragePoolMap:1vspm7306	TIMEOUT	3 days ago		267257 sec
...

But if you run upgrade / start new services / scale +-10 there is service.upgrade and some instance.stop instance.create- I think pretty normal behavior.
But during upgrade the CPU utilization jumps to almost 100% even if we have really powerful configuration (2 instances, each with 16 CPU cores)!!! With smaller instances there were some Timeouts of instance.stop and instance.start After a while message (Need to restart service reconcile) appears next to the service and upgrade stops for a while and whole Rancher acts weird. But I think this is just consequence not a cause.
As I have written in my first comment almost whole CPU is consumed by YAML encoding and decoding!!! When rancher needs to decode and encode YAMLs?

I habe 10 Stacks running, when I stop 3 Stack my CPU drops to normal values.
My Audit Logs produced over the weekend 3863820 entries...pretty hight isn't it?

I had to optimize the mysql DB, it's now better but the question is still the same..why does Rancher write so many Audit Logs?
I can now run about 20 Containers before the mysql process and java process goes really up again.

@aemneina  Any progress?
I think this is not kind/question but serious bug which prevents using Rancher 1.2+ in larger deployments! Probably it is causing other bugs (Those with Timeout getting IP which breaks rolling upgrade until you restart metadata.network-services).
As I said CPU is eaten by encoding/decoding YAMLs, it is probably caused by requests like
$IP - E36520F4F2CE263C051F [14/Feb/2017:04:13:25 +0000] "GET /v1/configcontent/metadata-answers HTTP/1.1" 200 313636 "-" "Go-http-client/1.1"
$IP - ADB832433F71475DDFC4 [14/Feb/2017:04:13:25 +0000] "GET /v1/configcontent/metadata-answers HTTP/1.1" 200 314714 "-" "Go-http-client/1.1"
$IP - D75BBF22200CB87A0768 [14/Feb/2017:04:13:25 +0000] "PUT /v1/configcontent/metadata-answers?version=27126-8b84aebc80dd7a28cb0d0e082d379dd7 HTTP/1.1" 200 0 "-" "Go-http-client/1.1"

Can I help you to identify problem somehow? Are the Rancher/Cattle going to get rid of those /v1/ calls and if so when?

I believe those calls to metadata are expected in an environment that has a lot of updates/churn. Do you only experience that when making changes to services? Are your services in a healthy state?

Are all those hosts/containers within the same environment, or are they segmented across a few environments?

Currently we have ~100 hosts, in 3 environments. When there is no upgrade or any change running, the CPUs of Rancher are quiet.
In Environment1 we have 50 hosts, 5 services each with 50 containers. When I run upgrade on one service (50 containers) one-by-one it takes 30 minutes. And it utilize 32 CPU cores of Rancher almost to 100%.
If Rancher is on weaker instances then upgrade stuck trying to start one container  again and again. The messege Need to restart service reconcile appear on service and Error Timeout getting IP address on container. And it stays in this state until restart of metadata.network-services on host where container gets Timeout getting IP address.
When there is some disaster, no disaster, just some stumble it might kill metadata on almost all hosts then Rancher is useless, unable to do ANYTHING!!! Even with 32 CPU cores. And I am not talking about slowness. On Rancher 1.1 same upgrade took 4-5 minutes on 8 CPU cores!
I can understand need of metadata service to know about changes in environment. But the bad thing is that encoding and decoding of YAMLs it cannot take so much CPU!!!

I am running 1.4.1 with similar setup, roughly 100 nodes and 5 environments. I am also getting these 100% CPU and slowness, all in all it has around 1700 containers in all 5 environments.

@aemneina could we maybe have this issue reclassified from 'question' to 'bug'? Definitely seems like a software defect to me, and a pretty severe one, too.

@underyx done. Looks like some improvements could happen here.

@aemneina how would you feel about us adding 50 instances to a try.rancher.com environment to let you experience the problem firsthand? Let me know if that would be helpful for you.

@underyx The purpose of try is really for us to experience these pains first hand.  But don't worry I already am aware of these issues and most will be address in 1.5 with more improvements coming in 1.6.  1.5 will be released in a week.
My current use case I'm testing is 1 environment with 50 hosts and 10000 services of scale 1.

@underyx if you want to give me a reproducable test case I can run to ensure it works fine that would be great.

Has this been addressed in newer versions of Rancher?

in my case it was a problem with the external dns rfc2136.
I could fix it with this here: https://github.com/rancher/external-dns/pull/54

@Toonix Does that PR affect the performance of the metadata service? Or were you experiencing high CPU usage for different reasons?

High CPU around 80%, it also filled my database like crazy. I had dns update continuesly witch got all logged in audit too. Instead of one update all 60 seconds or so..the service looped and got worse when I had more then 10 Stacks with public ports...the mysql Database where after some day over 5GB big.

Good to know! I think this is a different issue than the one that @tholcman and I are having though

I wonder if replacing the existing YAML parser Rancher uses SnakeYAML w/ a faster parser would help. Given @tholcman's comment ("It spends more 70% of CPU time encoding and decoding YAMLs!!!!!!"), this seems quite possible.
This parser claims to be faster: https://github.com/reprezen/kaizen-openapi-parser

@ibuildthecloud can you confirm that this should be improved in the released 1.6?

Just a note here, upgrading the network-services stack and setting cpu share (on metadata service) to <= 300 will mitigate large CPU spikes. Test this out if you plan on rolling this out to your production system. This as a temporary workaround might be appealing. Note this will slow deployment as metadata wont have unlimited CPU cycles to consume or process its changes.

@aemneina That's a really good idea! We'll try it out and let you know how it goes

I believe that the log above is consisting a couple issues, not a single one.
I am also experiencing a meaningless long delay on container start stage. And my setup is rather small, really small. 8 hosts, 4 stacks. Upgrading only a single container service, which is waiting at the stage "Starting".
I fell like this has begun after we started using rancher-nfs, but this particular service does not have a volume.
Also just in case it helps; Delayed Processes list is filled with volumestoragepoolmap.remove and has around total ~150 processes delayed. But there were never that much volume operations done on this environment.
I hope this input helps figuring out at least one of the issues here.

Any news on that? Running v1.5.10 with just one host and one environment, 5 stacks with <60 services and containers. That issue really causes problems at the production machines when it happens.

Closing as stale, if you encounter issues with performance, please file a new issue with details so we can investigate.


Add namespace to ChartConfig metrics

This is required for preventing the guest cluster metrics to be dropped.

Changes LGTM but I know why e2e is borked :(
We added chart-operator to the indexBlob but I had to remove because chart-operator isn't deployed in host clusters yet. I removed from installations but I didn't sort here.
On it. I already created a vendor PR earlier but I didn't get a chance to investigate the e2e failures.

Ok, the changes for getting versions from the indexblob were reverted on #115, I also took the opportunity to fix e2e tests there. CI green here after merging master, merging.


Cleanup Istio section

The bash sections include the command output. Despite the content being an advanced lab, this may be confusing for some. Adding this issue to make sure we don't forget to clean it up.

Ref #17


Add timestamps for indexing endpoints.

What is needed?
Extend the v1/about/indexing/ json response with the blocks timestamps.
Response example
{
  "currentBlockNumber": 20871516,
  "currentBlockTimestamp": "2024-05-17T13:31:23Z",
  "erc20BlockNumber": 20871515,
  "erc20BlockTimestamp":"2024-05-17T13:31:23Z",
  "erc20Synced": true,
  "masterCopiesBlockNumber": 20871511,
  "masterCopiesBlockBlockTimestamp":"2024-05-17T13:31:23Z",
  "masterCopiesSynced": true,
  "synced": true
}

looking good to me, cheers!


Change the Color of Text challenge won't run test

https://www.freecodecamp.org/challenges/change-the-color-of-text
I searched for the issue but couldn't find this exact problem - please forgive me if it's there... there's lots to search through. :)
I believe the code is correct but the "Run tests" button won't run the test after completing the challenge. I've tried in Firefox, Chrome and Safari. I've tried resetting the code, clearing cache, restarting browser.
In Chrome I can at least get the "Run test" button to change color, this does not happen in Safari and Firefox when I click it.

Browser Name, Version: Firefox 58.0.2, Chrome 64.0.3282.167, Safari 11.0.2
Operating System: OS X 10.13
Mobile, Desktop, or Tablet: Desktop

Just realized my code was wrong and that was the issue! I guess I thought there would be more of an indicator that it was wrong. Sorry! :)

I guess I thought there would be more of an indicator that it was wrong.

Do you mean the test messages were not shown with your code?

If that is the case, please share your code here.

Hi @amberferrerolaub! It looks like this was indeed caused by your code. But as @raisedadead mentioned, if the message did not show up at all, that would be an issue. If that's the case, please let us know and we'll reopen the issue.
Happy coding!


Use standard 10s cache for my_enterprise

Purpose or Intent
When we are performing performance_capacity_timer, refresh, and others, the individual classes return their own parent. Many of these return MiqEnterprise.my_enterprise as the root node.
So in the code, many root enterprise records are looked up at the same time.
The current implementation blows away the relations for the enterprise.
MiqServer, MiqRegion, and others use a simpler cache, one that has the record valid for 10 minutes.
Can we just move MiqEnterprise.my_enterprise over to the same mechanism as my_server and my_region?
Performance numbers:
For 100vms this calls ~145 fewer queries to MiqEnterprise.my_enterprise.tags and 145 fewer queries to MiqEnterprise.my_enterprise.taggings.
And unfortunately, I am unable to pass this value around.
/cc @Fryguy @jrafanie @gtanzillo While this is just following the current common mechanism, this may generate a lot of chatter.

@kbrock I think this makes sense.  Where I'm unsure about it is if we're building a huge tree of objects with the enterprise at the root of the tree, this change would cause us to retain the root of a possibly very large tree of objects for 10 minutes... this could cause a lot of memory usage since none of the objects in the tree will be GC'd.  So, can you speak of how we use my_enterprise, do we create large trees of objects with this at the root?

I'm 👍 on this particular change - caching it (and its relationships) for 10 seconds. But have the same concern as @jrafanie if we decide to cache it for 10 min.

@jrafanie I can go from 10 minutes to 1. I just saw that number elsewhere.
Most of the relations are virtual except for mostly metrics based ones: tags, metrics, metric_rollups, and vim_performance_states.
I'm familiar with ASSIGNMENT_PARENT_ASSOCIATIONS and miq_region#perf_rollup_parents. They use tags, and I'm not aware of any metrics.
Tree builder does not do much and I can't imagine MiqPolicy#parent_enterprise uses metrics. miq_capacity_controller stores in @record so who knows what happens there.
Chargeback uses to optimize what I'm optimizing. So I imagine we can probably just tear that out.
Metrics can be quite large. Looking at this closer, the old implementation keeps the object always in memory. Clears it out upon next access. I guess it is hoping that the last one to call my_enterprise does not access metrics.

Is it 10 minutes or 10 seconds?  PR title says seconds?

Also, the "standard" value is 5.minutes for cache_with_timeout, which you would get by passing no value.  Not sure if you were going for seconds or minutes, which is why I'm asking, because maybe 5.minutes is sufficient?  If are not using the default value, can you then pass literally 10.minutes or 10.seconds instead of just 10 for clarity?

this change would cause us to retain the root of a possibly very large tree of objects for 10 minutes

To clarify...cache_with_timeout doesn't clear the cache after the time expires.  It simply let's the next call refetch.  If the code never makes the next call, the code is held in memory forever.

Sorry for the deluge of comments :)  One way to avoid holding the memory forever is that at the end of blocks of code that use this cache, it can be cleared directly with my_enterprise_clear_cache.  This way it's held during the run, and then cleared immediately after.

@Fryguy thanks for clarification.
I removed 10 and just using standard caching that we use for my_server and my_region.
I would prefer to use the existing mechanism as it doesn't mess up the relationships of an object that is returned. (forcing cap&u to keep looking up the same exact tags)

👍


TODO: Isolate test fixtures

Tests like: https://github.com/feross/buffer/blob/master/test/to-string.js
Could simply be 1 function iterating over an array of:
{
  'source': 'Ձאab',
  'source-encoding': 'utf8',
  'target': '1YHXkGFi',
  'target-encoding': 'base64'
}

Even if it is in the same file...  makes test extraction/addition/refactoring sooo much easier.

Meh, not a fan.
When a test fails, this would make it harder to tell exactly which case the test failed on.
I prefer verbose tests, and I use 'Find and Replace' regexes when I need to do massive refactoring (which is not that often).
Matter of taste, I guess.

@feross it does make using external test fixtures quite a bit more difficult?
My motivation for this was that I was intending to bring in some more UTF-8 test fixtures as well as use the existing ones in an external project.
Remembering tests can be isolated using their description,  which could be a combination of the test data that makes them unique and easily isolated,  negating:

When a test fails, this would make it harder to tell exactly which case the test failed on.

@dcousens Can you send a PR to demonstrate what such a change might look like?


NIFI-5901 Enable writing to JSON database fields from PutDatabaseRecord

NIFI-5901 Simplified the integration test simplepool objects. NIFI-5901 Simplifed main int test.

Summary
NIFI-00000
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[ ] Apache NiFi Jira issue created

Pull Request Tracking

[ ] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[ ] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[ ] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 8
[ ] JDK 11
[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Fair enough. I'll run some tests with PutDatabaseRecord on MySQL and Postgres to see how it well it can correctly insert into a JSON field.

Your strategy worked.


more strange behavior for bag.fold

When you specific an initial value during a fold you get some unexpected behavior.
>>> import dask.bag as db
>>> def countme(total, record):
...     return total + record + 1
...
>>> seq = [1, 2] * 10
>>> print db.from_sequence(seq, npartitions=5).fold(countme,initial=0).compute()
54
>>> print reduce(countme,seq, 0)
50
>>> print db.from_sequence(seq, npartitions=5).fold(countme).compute()
49
>>> print reduce(countme,seq)
49

fold uses both a reducer and a combiner.  The combiner combines results from two reduce operations.  By default, if combine= is not provided, the same binop function used in reduce is used.  In your example, you probably didn't intend to use countme as the combiner.  It is a really weird combiner.
In [1]: def countme(total, record):
   ...:     return total + record + 1
   ...:·

In [2]: seq = [1, 2] * 10

In [3]: npartitions = 5

In [4]: width = len(seq) / npartitions

In [5]: # by default, the function to combine results from partitions is
In [6]: # the same as the binop used in reduce.
In [7]: combine = countme

In [8]: # equivalent to dask.bag when not specifying a default
In [9]: reduce(combine, [reduce(countme, seq[i:i+width])
   ...:                  for i in range(0, len(seq), width)])
Out[9]: 49

In [10]: # equivalent when specifying a default
In [11]: reduce(combine, [reduce(countme, seq[i:i+width], 0)
   ....:                  for i in range(0, len(seq), width)])
Out[11]: 54

In [12]: # perhaps `countme` is the wrong function to combine results.  Lets try add.
In [13]: combine = lambda x, y: x+y

In [14]: reduce(combine, [reduce(countme, seq[i:i+width], 0)
   ....:                  for i in range(0, len(seq), width)])
Out[14]: 50

In [15]: # which gives the same result as...
In [16]: reduce(countme, seq, 0)
Out[16]: 50

@anorfleet, did this answer your question? If there's a way we can improve the docstring to make this behavior more clear, let us know.

This appears to be solved. If you still have questions, feel free to reopen. Closing.


Auto-update linted/formatted files

We are trying to provide new capabilities on MegaLinter ( thread )
Despite our researchs, it seems that it is not possible to :

call Invoke-ScriptAnalyzer with a list of files and auto-update them with fixes
call Invoke-Formatter with a list of files and auto-update them with formatting

Do you confirm that it is not possible, or did we miss a magic argument that would allow that ?
Note: we saw the ways to have fixes in stdout but it would require to apply ourselves the fixes, and that does not follow the patterns of 99% of linters embedded within MegaLinter

Invoke-ScriptAnalyzer has a -Fix switch, which achieves what you want. We have seen that sometimes the file encoding doesn't get preserved though.
If you search in issues here, you will find one asking for similar functionality for Invoke-Formatter but you can do it yourself with Get-Content and Set-Content


Minor fixes: Fixed header level

Hi @BeauNouvelle ,
this is the very minor modification.
I fixed the header level like below:
- ### Filter by String Keyword
+ #### Filter by String Keyword

Thank you!


Issues adding and editing images

The add image to page seems to behave erratically. For example, I have these images in the Images "library" section (Apos admin bar > Images):

I then try to add one of these images to a page:

but the "Choose Images" modal shows no images (no results for current filters with no filter criteria).

Why can't I select an image from the list of images already added to the image "library"?
Also, if I try directly adding the image to the page using "New Image", the image won't save and there is no error thrown in console. I add the image, title it, select published=yes and click Save Image button, but the "Choose Images" modal is empty, however, the page shows the "Edit Image" button, even though the image isn't displayed on the page or in the "Choose Image" modal.

(clicking the "Edit Image" button shows the empty "Choose Image" modal)

Looks like the widget had a minSize attribute, which was the source of the issue. Removing it resolved the issue.
{{ apos.area(data.piece, 'main', {
widgets: {
'apostrophe-rich-text': {
toolbar: [ 'Styles', 'Bold', 'Italic', 'Blockquote', 'Link', 'Anchor', 'Unlink', 'Table', 'BulletedList', 'NumberedList' ],
styles: [
{ name: 'Title', element: 'h3' },
{ value: 'h5', label: 'Subtitle' },
{ name: 'Paragraph', element: 'p' }
]
},
'apostrophe-images': {
minSize: [ 700, 350 ],
aspectRatio: [ 2, 1 ],
size: 'full'
},
'apostrophe-video': {}
}
}) }}

Were the images large enough to meet that requirement? If so, there could
still be a bug here.
On Tue, Nov 8, 2016 at 4:43 PM, Lance P<EMAIL_ADDRESS>wrote:

Looks like the widget had a minSize attribute, which was the source of the
issue. Removing it resolved the issue.
{{ apos.area(data.piece, 'main', {
widgets: {
'apostrophe-rich-text': {
toolbar: [ 'Styles', 'Bold', 'Italic', 'Blockquote', 'Link', 'Anchor',
'Unlink', 'Table', 'BulletedList', 'NumberedList' ],
styles: [
{ name: 'Title', element: 'h3' },
{ value: 'h5', label: 'Subtitle' },
{ name: 'Paragraph', element: 'p' }
]
},
'apostrophe-images': {
minSize: [ 700, 350 ],
aspectRatio: [ 2, 1 ],
size: 'full'
},
'apostrophe-video': {}
}
}) }}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/punkave/apostrophe-sandbox/issues/60#issuecomment-259268963,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB9fa9wyjCVby6QcTngzZis-Uw4drMnks5q8O0IgaJpZM4KrkoE
.

--
*THOMAS BOUTELL, *SUPPORT LEAD
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

It looks like it is working to spec. If I find any issue, I'll reply to this thread. Thanks.


[Feature] Like-Dislike (scriptable?), does it exist, if not, can I bribe you with kitten pics?

Hey there,
first of all, beautiful piece of software that you created there. It works very well on both Mac El Capitan and Windows 10, that's not an easy thing to achieve. Keep up the good work.
However, since I'm a perfectionist, I have an overly specific requirement that I hope you can help me with. I'd like to be able to like/dislike songs per key bind, optimally globally.
Could you perhaps add menu commands "Like current song" and "dislike current song"? That way, I could write myself an apple script that I could toggle via Keyboard Maestro from everywhere.
If the feature already exists, feel free to pour ash on my head.

Insert ash pouring here

brushes off ash, coughs
Are the hotkeys global? Otherwise I suppose I would have to write an AppleScript that sends key presses to the music player

They are indeed global 🎉

Oh! Awesome!
The page doesn't say clearly. Thank you very much.

Uh - it doesn't let me set any hotkeys. I can click into the field, and it says "Not set", and that's it. What am I doing wrong?

Click in the field and perform the hotkey, their is a limitation at the moment that the hotkey has to use Control (not Command on a mac), sorry 'bout that but haven't found a workaround yet

Oh - all right. You might want to add a descriptor to the key bind for the mac version?
Is there something I can do to help you come up with a workaround? Key codes? Anything?