30 gigs of ram usage?

map_Mechanyon_rebuild.zip
Tried running the voxelizer at 1 scale on this OBJ, task manager using 30 gigs of my ram (no end in sight?)
(obj attached)

The coordinates on the file seem excessively large.
You should be able to generate a model at 1/100 scale.

Thank u! I think it's actually blenders fault, it seems to scale any fbx export up by 100x
Thanks for making this software ~
On Aug 18, 2021, at 11:40 PM, "Jakub Łabor"<EMAIL_ADDRESS>wrote:
The coordinates on the file seem excessively large.
You should be able to generate a model at 1/100 scale.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/CheezBarger/textured-voxelizer/issues/1#issuecomment-901651119, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFAXZGGG5HW5EEOO7UMYURLT5SRMHANCNFSM5CG3FUJQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.


bug: error fetching price for player buys anyway

Prerequisites

[X] I have searched for existing issues that already report this problem, without success.

Script Version
Latest
Platform
Android
Current Behavior
Autobuyer just bought Thiago Silva for 9.2k when he was worth 6.2 on futbin
Expected Behavior
If there is an error fetching price from futbin, skip the player altogether
Steps to Reproduce
Not sure
Additional Information
No response

Seems like you are using the outdated version of the script, uninstall and install the latest script from the releases tab.


Price information

Show price information of tokens to indicate how much money is at stake.

Best thing is probably using something like coingecko's price API and putting it in an edge runtime function for efficient edge caching.

We're using DEXes as price sources. Right now it is making requests on the frontend, but it may be a good idea to move more things to edge functions for better caching / performance in the future.


New feature suggestions

MXML is great!
About 10 years ago, I ported MXML V2.7 on VxWorks 5.5, which was perfectly suited to handling XML files on embedded systems.
One of the major porting changes was the realloc call in mxml_set_attr. Because realloc adds 1 _mxml_attr_t each time, the parsing process will get stuck here when the XML file is large. Changed to 4 or 8 _mxml_attr_t at a time and parsing works fine.
Recently I ported the latest MXML V4 version and I have a few requirements:
1, about the realloc tuning parameter settings, such as in the mxml_set_attr and mxmlIndexNew function calls. The parameters can be macro definitions or global variables.
2, add mxmlNodeIsBlank function, used to determine the blank node, in order to later clean up the document processing.
3, add the document to save the formatting settings, it is recommended that the Boolean switch parameters, as well as an indentation parameter.
4, in some functions, the stack allocation variable is very large, such as 8192 or 16384. this is a higher requirement for the task stack of the embedded system. It is suggested that it can be changed to a tuning parameter or other better ways.
Thank you very much.

@zlifes So if I can summarize what you are asking for:

In mxml_set_attr, expand the attributes array by more than 1 element at a time to make the reallocation more efficient.
Update all functions with large local character array variables to allocate them instead.

I have questions about the rest:

WRT mxmlIndexNew, the current code allocates node pointers 64 at a time for efficiency. Are you asking for a different multiple or a way to configure the allocation increment at compile or run-time?
WRT mxmlNodeIsBlank, how do you envision this working? Currently the only way to have a blank MXML_TYPE_TEXT node is to have whitespace between elements, otherwise the node is not blank.. And I'm not sure having a function just for this test is useful or necessary...
WRT document save settings, what settings are you referring to? There are no "Boolean switch parameters", and indentation is a function of the whitespace callback.

I apologise for the lack of clarity in the previous description.

In the case of mxml_set_attr and mxmlIndexNew, or other cases where there is realloc-like allocation granularity, provide macro definitions to specify the number of increments per increment at compile time. Of course, it would be nice to provide runtime configuration parameters.

In MXML_TYPE_TEXT mode, the text will be divided into multiple nodes according to the blank characters. In MXML_TYPE_OPAQUE mode, the text content is one node, which is easier to handle, but at the same time there are many blank nodes. I actually want to remove these blank nodes, which was the original purpose of mxmlNodeIsBlank. After loading the XML file, I can call mxmlNodeIsBlank to determine and remove blank nodes as needed.

When saving the XML file, I need to write a callback function for formatting. I hope there can be a parameter to configure whether to format or not. At the same time there is another parameter to specify the number of indented characters formatting, such as mxmlSaveFd(mxml_node_t *node, mxml_options_t *options, int fd, bool isfmt, int indent).

In some embedded systems such as VxWorks, the task stack is generally allocated smaller, such as 2 KB, so like 8192 or 16384 BYTE stack array will lead to task stack out of bounds. A possible way is to allocate memory on the heap, such as malloc.

I apologise for the lack of clarity in the previous description.

In the case of mxml_set_attr and mxmlIndexNew, or other cases where there is realloc-like allocation granularity, provide macro definitions to specify the number of increments per increment at compile time. Of course, it would be nice to provide runtime configuration parameters.

I can certainly add a #define in mxml-private.h so you can tweak things. A run-time option is, IMHO, overkill. Tracking with issue #318.

In MXML_TYPE_TEXT mode, the text will be divided into multiple nodes according to the blank characters. In MXML_TYPE_OPAQUE mode, the text content is one node, which is easier to handle, but at the same time there are many blank nodes. I actually want to remove these blank nodes, which was the original purpose of mxmlNodeIsBlank. After loading the XML file, I can call mxmlNodeIsBlank to determine and remove blank nodes as needed.

So the type callback can return MXML_TYPE_IGNORE to ignore the whitespace for specific child nodes, but I could look at adding a load option that eliminates blank values.  Tracking with issue #317.

When saving the XML file, I need to write a callback function for formatting. I hope there can be a parameter to configure whether to format or not. At the same time there is another parameter to specify the number of indented characters formatting, such as mxmlSaveFd(mxml_node_t *node, mxml_options_t *options, int fd, bool isfmt, int indent).

The whitespace callback returns a string to control indentation and is what you want.

In some embedded systems such as VxWorks, the task stack is generally allocated smaller, such as 2 KB, so like 8192 or 16384 BYTE stack array will lead to task stack out of bounds. A possible way is to allocate memory on the heap, such as malloc.

Tracking with #319.

Thanks again.
@.***
From: Michael R Sweet
Date: 2024-03-30 02:11
To: michaelrsweet/mxml
CC: zlifes; Mention
Subject: Re: [michaelrsweet/mxml] New feature suggestions (Issue #316)
I apologise for the lack of clarity in the previous description.
In the case of mxml_set_attr and mxmlIndexNew, or other cases where there is realloc-like allocation granularity, provide macro definitions to specify the number of increments per increment at compile time. Of course, it would be nice to provide runtime configuration parameters.
I can certainly add a #define in mxml-private.h so you can tweak things. A run-time option is, IMHO, overkill. Tracking with issue #318.
In MXML_TYPE_TEXT mode, the text will be divided into multiple nodes according to the blank characters. In MXML_TYPE_OPAQUE mode, the text content is one node, which is easier to handle, but at the same time there are many blank nodes. I actually want to remove these blank nodes, which was the original purpose of mxmlNodeIsBlank. After loading the XML file, I can call mxmlNodeIsBlank to determine and remove blank nodes as needed.
So the type callback can return MXML_TYPE_IGNORE to ignore the whitespace for specific child nodes, but I could look at adding a load option that eliminates blank values. Tracking with issue #317.
When saving the XML file, I need to write a callback function for formatting. I hope there can be a parameter to configure whether to format or not. At the same time there is another parameter to specify the number of indented characters formatting, such as mxmlSaveFd(mxml_node_t node, mxml_options_t options, int fd, bool isfmt, int indent).
The whitespace callback returns a string to control indentation and is what you want.
In some embedded systems such as VxWorks, the task stack is generally allocated smaller, such as 2 KB, so like 8192 or 16384 BYTE stack array will lead to task stack out of bounds. A possible way is to allocate memory on the heap, such as malloc.
Tracking with #319.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.*>


Allow React to be detected in NPM Workspaces

Details
React is detected by looking for a react devDependency or dependency. NPM Workspaces (mono repos) do not define these in the project package.json. The mono repo will fail if they are defined. The mono repo root has the dependencies listed.
When looking for React in this type of environment, the peerDependencies should be inspected instead.
Code of Conduct

[ x ] I agree to follow this project's Code of Conduct
[ x ] I agree to license this contribution under the MIT LICENSE
[ x ] I checked the current PR for duplication.

Contacts

Discord ID: deavial

If your PR is accepted, we will award you with the Contributor role on Discord server.
To join the server, visit: https://www.plasmo.com/s/d

@louisgv any updates on this pr? currently i have to manually create a patch on minified code for each release.

@deavial merging and releasing soon - sorry for the delay. I was trying to fix up some test and the resolver before putting this in.


Feature request: blur method

@furqanZafar Hey there, thanks again for putting the onEnter event in, it has really helped us a lot. I wondered if it is possible to expose a blur method, so that we can close the drop onEnter when submitting search?
thanks again.

onBlur is there already https://github.com/furqanZafar/react-selectize/blob/develop/API.md

@furqanZafar yes the event exists. However to forceably close the dropdown. A blur method for the simple select?

Added blur method in release 0.5.0, also pressing return/enter key on an empty dropdown will close it (by default)
The release has some breaking CSS changes so do check the changelog


Apostrophe characters are displaying differently in platforms

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

Expected Result:
Single quotes/apostrophes should display consistently across platforms, and in particular
In particular:

if there’s text directly before do the curl left version
if there’s no text directly before but there is text directly after, do the curl right version
if there’s no text directly on either side, use the straight version

Actual Result:
Apostrophes are not displaying the same across platforms
Action Performed:

Send a message with an apostrophe on iOS
Send a message with an apostrophe on Desktop
Notice that the apostrophe character is different

Workaround:
Visual issue
Platform:
Where is this issue occurring?

[ ] Web

[x] iOS

[ ] Android

[x] Desktop App

[ ] Mobile Web

Version Number:  1.0.22-1

Notes/Photos/Videos:

first is desktop, second is iOS:

view this job on Upwork
View all open jobs on Upwork

Triggered auto assignment to @chiragsalian (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@puneetlath Just thinking aloud if this is s bug. https://developer.apple.com/forums/thread/89706

Hm, yeah seems like it might just be. Weird that Apple doesn't do smart punctuation consistently across platforms.
@shawnborton what do you think? Is this worth doing something about on our end?

I wonder if it's worth doing once we have a better compose box that uses contenteditable=true. Right now we're using a standard textarea for the compose box which doesn't allow us to dynamically change things as you compose before you actually send the message. We've talked about potentially upgrading this for things like emojis, etc. So perhaps when that day comes, we'd be able to detect (on all platforms) if the apostrophe should be changed contextually.

@shawnborton would you recommend closing this issue out?

I'm thinking this is better addressed when we have a different kind of text input, so yeah, I think I would recommend just closing this one out.

@shawnborton ty!
closing for now


Confusing error when adding optional dependency to [features]

Anybody who does not know that optional dependencies implicitly define a feature of the same name might end up with a  Cargo.toml file like the following:
[package]
name = "foo"
version = "0.1.0"
edition = "2021"

[dependencies]
tokio = { version = "1", optional = true }

[features]
tokio = ["tokio"]

For this entirely reasonable human error Cargo however yields a completely confusing error message:
error: failed to parse manifest at `/tmp/foo/Cargo.toml`

Caused by:
  optional dependency `tokio` is not included in any feature
  Make sure that `dep:tokio` is included in one of features in the [features] table.

Reproduced with:
% cargo --version
cargo 1.56.0 (4ed5d137b 2021-10-04)
% cargo +nightly --version
cargo 1.58.0-nightly (ad50d0d26 2021-11-17)

In the long run namespaced-features will hopefully make this mess less common.
In the short run I would love to see error messages improve, although I have no idea how difficult that would be in this case.


Fix Nemo Launcher cmd generation, add tests

Summary

Fixed cmd generation for Nemo Launcher.
Add acceptance tests.

Test Plan
CI
Additional Notes
—

https://redmine.mellanox.com/issues/4158680


Can't set a filter when stepping in v.75

button unresponsive, right click->step filters also unresponsive

Fixed in >7.5-beta.14


WIP(universe): add ansible module

This PR aims to implement an universe module in order to run ansible playbooks with dagger.
There are two steps in order to run a playbook:

Call the #Ansible struct which installs ansible in the required version and load the specified config file
Call the #Playbookstruct which runs the playbook with the specified options

Questions

I didn't find another way to load a default config variable than storing it in a long string variable. Maybe storing it in a gist and curl it might be lighter?

There might some Ansible use cases which are not handled  by the module. Let me know if you see some options which should be added.

I'm not sure at all about the tests: I've added a @test at the end of the universe.bats but idk if there are other things to add in order to properly run tests on the module.

Moreover, up to now the test just check that the expected output of the Ansible playbook. Should it also test that the right config and inventory are loaded?

Hi @PaulBarrie , we just added an experimental namespace to universe.dagger.io, to allow merging contributions such as yours sooner, without rushing it to an official status. See #1991 and #1962 for what we've done for the Pulumi package contributed by @rawkode.
If you're still interested, we'd love to get your contribution merged (in your own experimental namespace) so that other developers can try it right away, and iterate from there, in the hope of getting to an official ansible package eventually.
Let me know and we can give you next steps (it's easy, mostly renaming your files to a new location).
Thanks and sorry for the slow response the first time around!

Hi @shykes,
It's ok for me, I will have time in a week to handle this. I will take a look at the Pulumi package and try to do the same for Ansible :)


Some issues we are currently facing with using dora-rs.

Dora does not seem to support multi-virtual environment control in conda

When running dora start in WSL, everything runs in the base environment. If you are using conda environments, you must develop in the base environment as it does not support multiple environments.

Unable to debug when running dora-start directly

If you run dora start directly, you cannot debug the Python code (as it starts in cmd mode).

Does not seem to support manual input in the workflow？
If there is an input command in the Python code, it will cause an error during execution.

Can one dataflow node call/embed another dataflow?

Does Dataflow support conditional judgment and branching?

Does Input support variable input？ Can Input get content input from the terminal window?

Dora does not seem to support multi-virtual environment control in conda
Yes, it's a bit of a stretch because conda is a bit of a mess but you can use absolute python path:

  - id: eval
    custom:
      source: /home/antoine/Pollen/lerobot/lerobot/scripts/eval.py
      args: -p cadene/2024_06_06_act_reachy2_mobile_base_3_060000 eval.n_episodes=1 eval.batch_size=1 env.episode_length=20000 policy.n_action_steps=100

Use conda run:
  - id: custom-node_1
    custom:
      source: conda
      args: run -n my-python-env node.py

Note that conda run can sometimes be slow for unknown reasons.
But I agree that it is not super explicit.

Unable to debug when running dora-start directly

I'm working on making possible to #517  run nodes dynamically by just calling:
python your_node.py

And also make logs more explicit with #542

Does not seem to support manual input in the workflow？

This would be fixed by #517

Can one dataflow node call/embed another dataflow?

It doesn't and might not in the short term

Does Dataflow support conditional judgment and branching?

We don't and probably won't

Does Input support variable input？ Can Input get content input from the terminal window?

By variable, do you mean different types? If so, yes, we use arrow types and you can match the types as input.
If you mean variable input source, we don't do this for the moment.

Can Input get content input from the terminal window?

So we're going to work on this after #517  probably but not in the short term.
Sorry, if we're not were you would expect us to be, but be sure that we're working really hard to be there. But, thanks for your interest!

Can I close this or do you guys still need help on specific point that dynamic_node does not fix?

dynamic_node

Dora does not seem to support multi-virtual environment control in conda

Yes, it's a bit of a stretch because conda is a bit of a mess but you can use absolute python path:
  - id: eval
    custom:
      source: /home/antoine/Pollen/lerobot/lerobot/scripts/eval.py
      args: -p cadene/2024_06_06_act_reachy2_mobile_base_3_060000 eval.n_episodes=1 eval.batch_size=1 env.episode_length=20000 policy.n_action_steps=100

Use conda run:
  - id: custom-node_1
    custom:
      source: conda
      args: run -n my-python-env node.py

Note that conda run can sometimes be slow for unknown reasons.
But I agree that it is not super explicit.

Unable to debug when running dora-start directly

I'm working on making possible to #517 run nodes dynamically by just calling:
python your_node.py

And also make logs more explicit with #542

Does not seem to support manual input in the workflow？

This would be fixed by #517

Can one dataflow node call/embed another dataflow?

It doesn't and might not in the short term

Does Dataflow support conditional judgment and branching?

We don't and probably won't

Does Input support variable input？ Can Input get content input from the terminal window?

By variable, do you mean different types? If so, yes, we use arrow types and you can match the types as input.
If you mean variable input source, we don't do this for the moment.

Can Input get content input from the terminal window?

So we're going to work on this after #517 probably but not in the short term.
Sorry, if we're not were you would expect us to be, but be sure that we're working really hard to be there. But, thanks for your interest!

Sure, here is the translation: "Of course it can be turned off. Thank you for your answer."

Can I close this or do you guys still need help on specific point that dynamic_node does not fix?
 Thank you for your answer.


Inconsistent error on malformed select

Sometimes the query is syntactically wrong but the system reports a json decoding error. We should find a way of capturing the Java error somehow and provide a meaningful feedback to users.
Example:

result = engine.select(query="select sometimes * where { service <x-sparql-anything:> { ?s ?p ?o } }")
print(result)

The first thing to try would be wrap this line in a try/except block and see if any exception is raised when passing a query with a syntax error.
https://github.com/SPARQL-Anything/PySPARQL-Anything/blob/f1edb42666836335a1c74d26b3ff6045188398ce/src/pysparql_anything/sparql_anything_reflection.py#L82


[Discussion] How many examples do I need for training new models

I only have a basic understanding of machine learning but just interested to know the minimum number of examples I need to train a new model?
Also, suppose I have already used spleeter to produce a 1000 acapellas, and I already have the corresponding studio acapellas, can I then go ahead and train it on these so that I could try and remove the imperfections on any other random diy acapellas?

It seems like the musDB dataset only contains around 150 songs, so if you have a thousand songs, you have roughly 7 times more.

Oh okay I see, cool.

@aidv Is there any significant benefit on using more songs for training?

@Scylla2020 Yes. If your dataset is too small or much of it is the same, there's a high risk of overfitting.
Overfitting basically means that the AI learns how to do replicate that specific dataset really well.
Ironically that's a good way of compressing data using AI, but not necessarily the correct way.
Anyways, I'm actually surprized that the MusDB dataset only contains 150 songs.
I want to believe that I'm wrong aobut that to be honest, it just seems unreasonable.
But hey, AI does what AI does. It's amazing.

Does anybody know if the pre-trained models provided via Github release have been trained using the MusDB dataset and its 150 songs, or if it was trained on a way larger dataset ? Could not find any info about the training setup of these pre-trained models.

It was trained using 20000+ songs
Pl see pdf of paper linked on read me
With AI or ml data becomes program
That is the various situations present in the data become program conditions and solutions
so model trained on 20,000 songs can be several orders of magnitude accurate than the model trained on 150 songs
Sent from my iPhone

On Nov 25, 2019, at 2:49 PM, anunge<EMAIL_ADDRESS>wrote:
Does anybody know if the pre-trained models provided via Github release have been trained using the MusDB dataset and its 150 songs, or if it was trained on a way larger dataset ? Could not find any info about the training setup of these pre-trained models.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I knew it. It didn't make sense for only 150 songs to produce results like that.

Having said that 150 songs contain hundreds of thousands of learning situations since short snippets are spectrogrammed and learned
So systems trained on 150 songs in diverse genres can learn quite a lot as seen in open unmix benchmark
But systems learning from
Millions of situations get way better provided architecture allows that learning
Ravi
Sent from my iPhone

On Nov 25, 2019, at 8:12 PM, aidv<EMAIL_ADDRESS>wrote:
I knew it. It didn't make sense for only 150 songs to produce results like that.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

@ravi-annaswamy Oh I see. So a small dataset with varying data can give similar if not equal results as a larger dataset, BUT, a large dataset will nonetheless give better results no matter what.

I would not put it that way.
A large dataset of 10000 jazz standards may not be as effective as 150 songs selected across pop rock jazz and world music. Because ability to discriminate voice or stem instruments against varying backgrounds is not in the jazz only training set.
10000 songs selected across pop and rock vs
150 songs across pop and rock will definitely be better is all I said.
Sent from my iPhone

On Nov 25, 2019, at 8:19 PM, aidv<EMAIL_ADDRESS>wrote:
@ravi-annaswamy Oh I see. So a small dataset with varying data can give similar if not equal results as a larger dataset, BUT, a large dataset will nonetheless give better results no matter what.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@ravi-annaswamy Will I be correct to say that currently, after any sort of training with thousands of examples the results can never be completely perfect even if you throw in a song that was used as part of the training?,due to how the neural networks work,

The number of songs doesn't correspond to the number training set m. You can use as training set just a part of the song with all its informations. Of course you will have for the same songs different informations that can be considered a training set, for example the intro, the verse, the chorus and so on. You can also apply some sort of distortion to the data so that you automatically create new feature without collecting more songs / parts of the songs.
What @aidv said about overfitting is not correct. Overfitting is a problem "caused" by the number of the feature and not by the number of the training set (in this case the sound taken from a song). Instead increasing the number of the training set, if you suffer of high variance (overfitting), is likely to help a lot your algorithm.
In order to verify that you can plot the learning curve and see what happens.
Incrementing the number of the feature didn't help only if you suffer of high bias (underfitting).

Thank you for giving a more in depth explanation to all of this.
So what I'm taking away from this is, and please correct me if I'm wrong: We could technically have 1000 songs, process them through 5 different effects, and each effect having 4 different settings, giving us 20000 different samples to use in our dataset.

That's what distortion is about. You obviously want to apply some distortion that could then be present in the songs you want to split.

Discordance mentioned the BEAN dataset, that contains:
24,097 songs
79 hours of audio stems
majority of pop/rock songs

Is it being used by the latest Spleeter version or is there a way to download &use it for the training files if not?
Thanks, Roger

Sorry I am late getting to this but could you please answer this ---
Discordance mentioned the BEAN dataset, that contains:24,097 songs
79 hours of audio stems
majority of pop/rock songs
Is it being used by the latest Spleeter version or is there a way to download &use it for the training files if not?
Thanks, Roger
On Monday, January 13, 2020, 7:13:21 AM CST, Moussallam<EMAIL_ADDRESS>wrote:

Closed #118.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Remove mentions from the digest

Shouldn't these be changed to @name -> @name from the digest posts? Otherwise these uses will get spammed from the digest posts.

I think we can change the references' '@' symbol to use fullwidth commercial at ('＠') which will not trigger references.
@damithc
＠damithc

I prefer something that's not deceptive (and easy to debug), such as using backticks.

I feel that it is much safer if we replace items with the fullwidth at sign as is more safer.
Since fetched comments are not generated, it's possible to escape referencing if we just wrap backticks like so:
@damithc => @damithc
a sample text @damithc text => a sample text @damithc text
A foolproof solution will require identifying backtick wrappers which might lead to more complications.

A foolproof solution will require identifying backtick wrappers which will lead to more complications as github supports wrapping with multiple backticks

I see your point. Let's use the fullwidth at sign.


Setup ownership-based Security Strategy

This strategy is required for Fine-grain permission management in Project folders.
Concept:

Each project has one primary and several secondary owners
The owners are being defined during project registration in LibreCores Web (GitHub accounts)
Owners automatically get permissions on CI using backend scripts or CLI

See https://wiki.jenkins-ci.org/display/JENKINS/Ownership-Based+security

It appears that Multi-Branch Pipeline is not supported in 0.9.0. https://issues.jenkins-ci.org/browse/JENKINS-38353 . It is required for the planned environment

Ownership with macros is deployed now


HA 0.88 brings a 'new' / 'better'? custom component layout...

Integrations need to be in their own folder. Change climate/evohome_cc.py to evohome_cc/climate.py. This will stop working soon.

I will fix this (and a few other issues) this weekend.

Thanks :)
FYI, it also spits out errors like:
Error loading custom_components.evohome_cc.climate. Make sure all dependencies are installed
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/homeassistant/loader.py", line 147, in _load_file
    module = importlib.import_module(path)
  File "/usr/local/lib/python3.7/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 962, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'custom_components.evohome_cc.climate'; 'custom_components.evohome_cc' is not a package

feedback to me after the weekend if it is still there...

I believe this has been 'fixed'. However, it doesn't work with HA v0.88b1 due to stuff missing from homeassistant/components/climate/const.py.
It does work with HA v0.89-dev (as at 16/02/2019), and may well work with HA v0.88 when it is released.
Don't bother attempting to use this code, unless you're running HA v0.88b2 or later.

I'm using 0.88B2 sadly and get the error about missing dependencies, guess I need to wait now for HA to sort the issue...

Same problem after update to 0.88.0. Missing dependencies.

I've had confirmation that the necessary change ruby be there until 0.89.
Until then, just use an older version of the component.

@Swiftnesses your first and second posts are due to different errors.
The first is survivable (i.e. the component will run ok), the second is not.
The second is because you're using master, and HA <0.89

I'm using homeassistant v0.88.0 and have it working by changing to the right repo using:
root@ubuntu-server:/etc/homeassistant/custom_components/evohome_cc# git checkout integrated-folders
Branch 'integrated-folders' set up to track remote branch 'integrated-folders' from 'origin'.
Switched to a new branch 'integrated-folders'
root@ubuntu-server:/etc/homeassistant/custom_components/evohome_cc# git pull
Already up to date.

Changed the custom_components.json file so the contents are using the new layout:
root@ubuntu-server:/etc/homeassistant/custom_components/evohome_cc# cat custom_components.json
{
    "evohome_cc": {
        "version":         "0.9.5",
        "local_location":  "custom_components/evohome_cc/__init__.py",
        "remote_location": "https://raw.githubusercontent.com/zxdavb/evohome/integrated-folders/__init__.py",
        "changelog":       "https://raw.githubusercontent.com/zxdavb/evohome/integrated-folders/CHANGELOG.md",
        "visit_repo":      "https://github.com/zxdavb/evohome"
    },
    "climate.evohome_cc": {
        "version":         "0.9.5",
        "local_location":  "custom_components/evohome_cc/climate.py",
        "remote_location": "https://raw.githubusercontent.com/zxdavb/evohome/integrated-folders/climate.py",
        "changelog":       "https://raw.githubusercontent.com/zxdavb/evohome/integrated-folders/CHANGELOG.md",
        "visit_repo":      "https://github.com/zxdavb/evohome"
    },
    "water_heater.evohome_cc": {
        "version":         "0.9.5",
        "local_location":  "custom_components/evohome_cc/water_heater.py",
        "remote_location": "https://raw.githubusercontent.com/zxdavb/evohome/integrated-folders/water_heater.py",
        "changelog":       "https://raw.githubusercontent.com/zxdavb/evohome/integrated-folders/CHANGELOG.md",
        "visit_repo":      "https://github.com/zxdavb/evohome"
    }
}

In configuration.yaml
evohome_cc:
    username: !secret evohome_username
    password: !secret evohome_password
    scan_interval: 360
    high_precision: true
    use_heuristics: true

Regards,
Donald.

I'm using homeassistant v0.88.0 and have it working by changing to the right repo using:
root@ubuntu-server:/etc/homeassistant/custom_components/evohome_cc# git checkout integrated-folders
Branch 'integrated-folders' set up to track remote branch 'integrated-folders' from 'origin'.
Switched to a new branch 'integrated-folders'
root@ubuntu-server:/etc/homeassistant/custom_components/evohome_cc# git pull
Already up to date.

Changed the custom_components.json file so the contents are using the new layout:
root@ubuntu-server:/etc/homeassistant/custom_components/evohome_cc# cat custom_components.json
{
    "evohome_cc": {
        "version":         "0.9.5",
        "local_location":  "custom_components/evohome_cc/__init__.py",
        "remote_location": "https://raw.githubusercontent.com/zxdavb/evohome/integrated-folders/__init__.py",
        "changelog":       "https://raw.githubusercontent.com/zxdavb/evohome/integrated-folders/CHANGELOG.md",
        "visit_repo":      "https://github.com/zxdavb/evohome"
    },
    "climate.evohome_cc": {
        "version":         "0.9.5",
        "local_location":  "custom_components/evohome_cc/climate.py",
        "remote_location": "https://raw.githubusercontent.com/zxdavb/evohome/integrated-folders/climate.py",
        "changelog":       "https://raw.githubusercontent.com/zxdavb/evohome/integrated-folders/CHANGELOG.md",
        "visit_repo":      "https://github.com/zxdavb/evohome"
    },
    "water_heater.evohome_cc": {
        "version":         "0.9.5",
        "local_location":  "custom_components/evohome_cc/water_heater.py",
        "remote_location": "https://raw.githubusercontent.com/zxdavb/evohome/integrated-folders/water_heater.py",
        "changelog":       "https://raw.githubusercontent.com/zxdavb/evohome/integrated-folders/CHANGELOG.md",
        "visit_repo":      "https://github.com/zxdavb/evohome"
    }
}

In configuration.yaml
evohome_cc:
    username: !secret evohome_username
    password: !secret evohome_password
    scan_interval: 360
    high_precision: true
    use_heuristics: true

custom_updater:
  component_urls:
    - https://raw.githubusercontent.com/zxdavb/evohome/evohome_cc/custom_components.json

Regards,
Donald.

With this "config" it doesn't track the evohome component in custom_updater.
Did it show's the evohome custom component at your tracker card?

This has been addressed with release tagged at v0.9.9 (on the master branch).


Drawer animation super slow Android & iOS

Hi All,
New to react to excuse me if i'm doing something wrong, coming from the Ionic / Angular Material / Ionic2 and obviously Cordova world that I'm pretty well versed in. One of my big reasons for making the jump is obviously performance and especially the performance of drawer animations... Which is why I am so surprised its as bad as it is.
To reproduce just fire up a vanilla ignite project and click on the drawer, iOS simulator is bad, Nexus 6P is pretty terrible, this is after I removed all the content on the presentation screen as well. I looked at the perfmonitor and the JS framerate is all that drops to around ~30 FPS, the device one stays at 59.6. I wouldn't have though 30FPS wound't be that bad but it look and feels like its about 5.
Anyway I'm not here to have a winge, I was just wondering if someone has resolved the issue already, its being worked on, or how I could implement this (and is it likely to work):
https://github.com/root-two/react-native-drawer/issues/163
If I can solve it I am happy to do a pull request.
There are a few threads of react-native-drawer's github page around performance but I figured its probably more of a configuration issue with ignite since every post isn't about it.

Heya @rknell !   Welcome!
So, yes the dev experience feeeeeeeels like it's 5 frames per second, but in my experience the problem is usually being in development mode.
By default, we have Redux logger, yellow box, reactotron, and packager remote.  All performance inhibitors.  When you turn all these off and ship to a device with the JS bundled neatly inside, the performance usually mirrors native speeds.
I know that doesn't sound good for fishing out performance smells on local dev, but having all those bells and whistles really helps app creation.   I suggest putting your app on a device once a week to make sure you're actually hitting native speeds.
If you notice any issues on device, please let us know!  If not, we're interested in how we could communicate these development costs in the future.  Perhaps in the readme, or a blog post, or perhaps indicators on the actual screen?

Development on an android device is super  slow, especially the navigator animations and the drawer animatons.

Can confirm this, once you build the APK it all runs surprisingly smoothly @rknell

Great news.  Ya, console.log is amazing just how bad it beats up RN android.  Reactotron was partly born out of this pain.


Disable CGO for Centos 5 to assist #952

Thanks, but now I realize it didn't affect the dockerfiles, since they're on docker hub :) Hopefully https://github.com/andyneff/git-lfs_dockers/pull/1 should work.

This fix won't actually work apparently. rhel isn't defined in RHEL 5 ;) I'll fix it in an upcoming PR


Make "External Links" a translatable String

Description
Made "External Links" a translatable String and added the Polish Translation.
"Icon Request Form" remains as the last String which I couldn't get to work.

Type of change

:x: Bug fix (non-breaking change which fixes an issue)
✅ : General change (non-breaking change that doesn't fit the above categories, such as copyediting)

Great, it would be "Liens externes" in french :)

@service-paradis thanks! Added it :)

Just in case, I will draw your attention to the idea that it is better to keep all translations in one file, because it will be easier to add, edit or delete lines.


Weird beheivour with Audio clips in BlendSpace2D while Sync is enabled

Tested versions

Reproducible in v4.2.1.stable [b09f793f5]

System information
Godot v4.2.1.stable.mono - Windows 10.0.22631 - Vulkan (Mobile) - dedicated NVIDIA GeForce RTX 3070 Ti (NVIDIA; <IP_ADDRESS>29) - 12th Gen Intel(R) Core(TM) i5-12600KF (16 Threads)
Issue description
Audio clips seem to have weird behavior when used in a blendspace2D with sync enabled. Sometimes the audio refuses to have its volume changed and has weird timing. Other times the audio does blend properly and causes audio to either pop and crackle because it reaches the max_polyphony or plays the sound completely with no volume change ,when it shouldn't be either muted or at not playing at all. After a while it also seems to kill the audio server if there are a lot of different sound effects in the animations (likely because it overflows??)
Example video of the max_polyphony being reached causing the audio to pop and crackle while technically the selected animation in the Blendspace2D has no audio
https://github.com/godotengine/godot/assets/29980512/367cdf31-f6ae-466f-bb6c-a9cc93927032
What should be happening in the video is that no audio should be playing because the animation on that point has no audio clips only the neighboring points should have their audio playing at -80DB while the other points only keep track of the current position in their respective animations.
Steps to reproduce

Create a AudioStreamPlayer3D, AnimationPlayer, AnimationTree
In the AnimationPlayer create 2 animations 1 with an audio clip track and 1 without
In the AnimationTree create a BlendTree with a BlendSpace2D connected to the output
Place 4 points of the animation with sound at the top, bottom, left, right and place the animation without sound in the middle
4.Enable sync

Minimal reproduction project (MRP)
AudioClip issue in blendspace2D.zip

@TokageItLab any ideas why the blending of audio has problems?

I assume it is simply too much audio being played at the same time.
AnimationTree does not support playback from the middle of audio, so currently audio must be played even at zero volume in order to take into account fading-in.
I think there is some room for optimization, but we would need to add methods like AudioPlayback and AnimationPlayback that allow keys with "length" to be played from the middle of the key, separate from getting them in Descrete. See also https://github.com/godotengine/godot/pull/86661.

I assume it is simply too much audio being played at the same time.

From the little testing I did I'd say thats about right. Audio crackling and poping only happens in my bigger project and not in the MRP, mainly because in example video there are about 70 different sounds at minimum split across 5 audio clip tracks technically playing at the same time.
Although there does seem to be an issue in the MRP where upon the initial start of the tree it will ignore the blending completely while sync is on and after turning the some clips will ignore the volume almost like those sounds are leaking through and then it starts kind of working again after finessing it.
Example of it upon initial start not working and then half working after turning sync off and on
https://github.com/godotengine/godot/assets/29980512/c68a5882-f2ae-4646-a35c-79cd9a0152e9
Example of the sound "leaking"
https://github.com/godotengine/godot/assets/29980512/1e68544e-3460-41bd-af1c-e39646c82928


Lets talk about online and offline presence updates

Hi to all!
Scenario 1
User1 connected at 13pm
User2 connected 5 minutes later
User1 will get status of user2.
But User2 wont get status of User1 until User1 refresh page.
But in that case they changed places, one of them not see online status
Scenario 2
User1 travel from page to page. User2 will see every time offline, online, offline, online......
Need to set timeout to 5 seconds of offline status by default i think.
About scenario 1 any thoughts?

It would be great to have a list of connected users. When user connect to chatkit he get list of all online users.

The presence state of all users can be found buried deep inside the CurrentUser object, but getting hold of it isn't ideal, and it isn't populated immediately. This is something that should be fixed in an upcoming release of the SDK too. Would an interface something like the following make sense?
const onlineUsers = currentUser.users.filter(user => user.presence.state === 'online')

Regarding scenario 2: we're planning a bit of an overhaul of how presence works at some point in the future, so this is certainly something we will consider then.

I think when user connect to chat he get status online in list of all users and that status goes to all online users. When he disconnect status changed to offline and others get that update of status.

@alexey13 That's the idea yeah, it's just quite hard to extract the initial status at the moment. We're working on making that easier, by having the presence data always be available directly on the user object.
This functionality should be available in the next version.

As @hot-leaf-juice said this will be fixed in the soon-to-be-released new version of the SDK.


Use case details

We should describe in detail (on about page or in docs) which use cases are best suited for plantit. Currently we just say image-based HPC/HTC phenotyping and point to CoGe/etc for genomics. Examples would be nice, and maybe a decision chart?

Could use this for visualizations in markdown, seems easier than making diagrams somewhere else and exporting/embedding images

This is probably sufficient


Add Option to permit Insecure Certs in ChromeHeadless

This expands on the ChromeDriver work in https://bugs.chromium.org/p/chromium/issues/detail?id=721739 to support Chrome Headless running in an environment with e.g. self-signed certs.

I can't quite figure out why Codacy is failing to analyze this - @undera if you have any guidance - I'll try and fix whatever it complains about.

The code looks fine, I guess Codacy just has technical problems


Update 07-repeated-field.rs

env is mandatory field

@dtolnay I got error in 07 testcase because env is not set. Anything I missed?

If the user calls env n times, they need to get a vector of n environment variables. So for example they can call 2 times to get 2 elements in the vector (this is tested by assert_eq!(command.args, vec!["build", "--release"])). Or they call 0 times to get 0 elements in the vector.

If the user calls env n times, they need to get a vector of n environment variables. So for example they can call 2 times to get 2 elements in the vector (this is tested by assert_eq!(command.args, vec!["build", "--release"])). Or they call 0 times to get 0 elements in the vector.

thanks, I see your explanation
I interpret mandatory field is that user at least call 1 time, because env is not a Option field, so we should report error if no env is set by user


Lint: ignore reportArgumentType and reportInvalidTypeForm errors

Ignoring pyright reportArgumentType and reportInvalidTypeForm errors until a better solution can be found.

@ansasaki @sergio-correia I believe this temporarily fixes the problem, as we previously discussed. In a new major release, when we have the opportunity to increase the minimum version of python, we can go back and remove these pragmas.


bioconda version of VEP

Do you think this will work in the automatic installer?
https://bioconda.github.io/recipes/variant-effect-predictor/README.html

Here are a few more additions:
Plink(2)

https://anaconda.org/bioconda/plink2
https://anaconda.org/biobuilds/plink

VCFTools

https://anaconda.org/bioconda/vcftools

Great!
Plink should work outof the box and probably vcfTools to. However, I am unsure about VEP. I do not know if the bioconda only installs the basic script and no cache or plugins. Will check it out though.

Here they have more information: https://github.com/bioconda/bioconda-recipes/blob/master/recipes/variant-effect-predictor/post-link.sh

I also guess some of the package depend on Perl ^^
https://anaconda.org/bioconda/perl

Judging from the post it just pulls down the VEP zip and unzips it.  Also I have had a lot of trouble with the conda perl and cpanm and running on the nodes. What we have works fine. I will leave VEP via bioconda for now and use the working SHELL installation we have.

I'll do a quick test on Allan but will revert to the official installer if I run into problems


Ignore files which no longer exist

This is especially important when adding a folder. If a file gets deleted during the zip process in the folder, the task will always continue

Thanks for the PR. Merging it.

I will also add a test or two after merging it.


Press "S" key close any window

Tested versions
From version 4.0.4-rc1 to Godot 4.3 rc1 (the last one you can download).
System information
Godot v4.3.rc1 - Windows 10.0.19045 - Vulkan (Forward+) - dedicated AMD Radeon(TM) RX 6500 XT - AMD Ryzen 5 3600 6-Core Processor (12 Threads)
Issue description
In any window, since the program is running, every time I press the “S” key on my keyboard, the current window closes. As if I press the command ctrl+Q. Occurs in the project selection window as well.
Steps to reproduce
Open it, press "S" key...
Minimal reproduction project (MRP)
N/A

Can't replicate based on these instructions

Maybe you have S assigned in ui_cancel.

I think you assigned S to a shortcut that quits the software.


Add RDFs calculation based on species

Currently, the RDFs can be calculated for floating species, requiring sites data or transition information. Could you please additionally implement a version that calculates RDFs between specified species without the need for transition data. I have provided the code we use for this calculation, which includes parallelization.

Hi @AstyLavrinenko thanks for this, I refactored the code a bit. I think I'm going to add it as a plot function to Trajectory:
trajectory.plot_rdf_between_species(...)

Would that work for you?
Todo

[ ] Add method on trajectory
[ ] Move hist code to plots/matplotlib
[ ] Add plotly version
[ ] Add integration test

Let me know if you are happy with this!

Hi @stefsmeets thanks a lot!
The only concern is that raw data is not easily accessible now, and it's nice to have such opportunity as sometimes we need to visualize several RDFs at one plot. Can we maybe move _get_radial_distribution_between_species function as a Trajectory function?

Yeah, makes sense. I came up with a better way to organize the code, I will make a PR soon.


add asm instrument

#2016

@jaehong-kim
I tried to read the code while it was really many! I just want to know can I instrument all sub-class of a specified base class?
I contributed the hystrix plugin. In hystrix, user inherit from HystrixCommand and implement the abstract method run(). This run() method should be intercepted but pinpoint previous release could only inspect a class you know its class name. All I want is to intercept run() method of all sub-class of HystrixCommand.
Can this PR support what I want? If not, will you consider support it in the future?

Not yet supported.
Performance is still difficult to check for inheritance between classes.
But someday we will support it.

@jaehong-kim
So you use ASM to replace Javassist just to improve performance? I have seen another APM using ASM and support what I want. While search in inheritance really will slow the boot time a lot. But you know, most apps boot time exceed a half minutes, it is not acceptable to double the boot time. ^-^


Fixed rage gauge calculation and added 5-GCD IR Usage

Fixed the longstanding bug in the rage calculation
Added the 5GCD IR Viewer (OGCDs not excluded)

'at'll do fer now


Added IsUrl

Added IsUrl function in Validator.cs and respective unit tests in ValidatorTest.cs

I don't know why its still showing previous commits in this PR. I have followed the exact steps mentioned in the tutorial.

Thank you once more, @PrasadHonrao. Awesome work as always :bear:!
I am not at all certain what is happening here. I think deleting your fork and local clone, and then re-forking and re-cloning will solve our problems going forward.


doc(api): loaders doc add function remove prompt

loaders doc add function remove prompt
https://github.com/webpack/webpack/issues/9919

@EugeneHlushko How can we transfer this to v4 docs? I think it should also be there.

@EugeneHlushko How can we transfer this to v4 docs? I think it should also be there.

yes it should be in v4


refactor(actions): use development GitHub environment

Motivation
[WIP]
This PR is meant to validate the use of GitHub Environments, which should give us extra visibility when we're deploying to Mainnet, Testnet, to our development environment and later on when we're production ready and have to handle all these different environment differently.
Specifications
https://docs.github.com/en/actions/deployment/targeting-different-environments/using-environments-for-deployment#environment-protection-rules
Complex Code or Requirements
Not yet
Solution

Review
Not ready to review
Reviewer Checklist

[ ] Will the PR name make sense to users?

[ ] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?

[ ] Does it change concurrent code, unsafe code, or consensus rules?

[ ] How do you know it works? Does it have tests?

Follow Up Work

Later today or tomorrow I'll be opening a PR to generate testnet checkpoints in CI in:
https://github.com/ZcashFoundation/zebra/blob/main/.github/workflows/continous-integration-docker.yml
It could have some merge conflicts with this PR, because I'll be launching zebrad on mainnet and testnet in:
https://github.com/ZcashFoundation/zebra/blob/main/.github/workflows/deploy-gcp-tests.yml
I'm also thinking of splitting this job into a reusable workflow, so I can call it for mainnet and testnet:
https://github.com/ZcashFoundation/zebra/blob/ef556ea06b1072670b9215015682a45bf1362326/.github/workflows/continous-integration-docker.yml#L80-L82
How did you want to manage those merge conflicts?
Can you avoid making any testnet-related changes in this PR?

@teor2345 I don't think I will make those Testnet changes here. You can continue, I can handle the conflicts.

@teor2345 I don't think I will make those Testnet changes here. You can continue, I can handle the conflicts.

Sounds good!
Because testnet is unreliable, we probably won't want to run tests on it, unless we ignore the results, or the tests are only run on demand.

This was tested and it's no longer needed as we won't implement GitHub environments this way


feat: add C implementation for stats/base/dists/triangular/skewness

type: pre_commit_static_analysis_report
description: Results of running static analysis checks when committing changes. report:

task: lint_filenames status: passed
task: lint_editorconfig status: passed
task: lint_markdown status: passed
task: lint_package_json status: passed
task: lint_repl_help status: na
task: lint_javascript_src status: passed
task: lint_javascript_cli status: na
task: lint_javascript_examples status: na
task: lint_javascript_tests status: passed
task: lint_javascript_benchmarks status: passed
task: lint_python status: na
task: lint_r status: na
task: lint_c_src status: passed
task: lint_c_examples status: passed
task: lint_c_benchmarks status: passed
task: lint_c_tests_fixtures status: na
task: lint_shell status: na
task: lint_typescript_declarations status: na
task: lint_typescript_tests status: na
task: lint_license_headers status: passed ---

type: pre_push_report
description: Results of running various checks prior to pushing changes. report:

task: run_javascript_examples status: na
task: run_c_examples status: na
task: run_cpp_examples status: na
task: run_javascript_readme_examples status: na
task: run_c_benchmarks status: na
task: run_cpp_benchmarks status: na
task: run_fortran_benchmarks status: na
task: run_javascript_benchmarks status: na
task: run_julia_benchmarks status: na
task: run_python_benchmarks status: na
task: run_r_benchmarks status: na
task: run_javascript_tests status: na ---

Resolves #3822
Description

What is the purpose of this pull request?

This pull request:

adds C implementation for @stdlib/stats/base/dists/triangular/skewness along with relevant tests, benchmarks and examples.

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #3822

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

@Planeshifter  Sorry for that mistake. I will make sure nothing like this happen again from my side.


Input styling

Changed input-styling and made it from using absolute to using flex box and also changed color scheme in dark mode but in dark mode the shadow of input is having white color this should be changed to #1d243d but i am unable to do so, so please look into it :)

Thank you so much for this! The website looks a lot cleaner and the code is now much more maintainable.
Just one question - what do you mean by the shadow of the input in dark mode?

The white part that looks odd is the one I'm takling about. I have made a #form-contenu-dark i think you have to change the js for it to function and then just change background-image property to a more suitable color thank you.

@UdhbhavSinghRana I've fixed enabling dark mode for the form background. I have no idea what colours to use, do you have any ideas?

@TheTechRobo you can try using #0e1323

Thank you!


fix: add priority field to message and sorting by priority within local store

Fixes issue: https://github.com/customerio/issues/issues/11663

Thank you for the comments, I applied @Shahroz16's recommendation as it handles nil cases better 🙏


GitHub Actions Plugin

https://github.com/amplication/amplication/issues/5802

This one is taken care of in: #347.


removing legend from clin till we figure out how we want to display it

Remove legend from clin until we decide how and where we want to display legend in when launched from clin

quarantest build successful. Demo link at
http://b2225eac.quarantest.iobio.io

Currently, the legend is shown in clin, with no way to hide or remove it.  This PR disables the legend showing when launched from clin, and is a temporary fix until we discuss how we want to handle the legend inside clin


Json compare

New Feature Description:
JSON compare, Since the ballerina, contain the JSON as a part of it native datatype. Comparing 2 JSON variable should be a native function in a ballerina. If the same elements and values are part of variables. It should return true regardless of the order of elements

Introduced with https://github.com/ballerina-platform/ballerina-lang/pull/10904.


The data cannot be shown because it is not valid

Hello Team,
In D365 Business Central, When we add Notes through Code system add the notes with Text - "The data cannot be shown because it is not valid"
Please see the attache Code.
Can anyone has idea about this ?
Thank you

Hello Team,
Can anyone have idea about above issue ?
Thank you.

Let's get the issue to a team who will fix and backport it, as this repo is only for issues related to the latest developer preview environment for Dynamics 365 Business Central. We suggest that you open a support case or file a bug in Collaborate, to ensure that all Business Central users benefit from your catch as soon as possible.
To open a support case, you can:

Open a support request to CSS through the PartnerSource portal
Contact your Service Account Manager (SAM) in your local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also direct you "step by step" how to open a support request or how to get credentials, if this is the first time for you or your company

If you file the bug in Collaborate, remember to include steps to reproduce the issue, and the Business Central build number and country version you're using.

Had the same issue after an upgrade from NAV2017CU4 (local serverinstallation) to NAV2017CU18 (on Azure).
Seems to be occuring only if the note is 20 or 21 characters long.


Enemy will not show up

For some reason after a random number of game restarts, the enemies that spawn on wave maybe 1 or 3 will not show up on screen. I need to check if they even spawn at all and are just not pursuing or what.

Possible fixed in 22369ec but I need to test some more


Do not save already destroyed pg_search_document, take 3

Old pull-requests:

https://github.com/Casecommons/pg_search/pull/201
https://github.com/Casecommons/pg_search/pull/210

This can happen in case of complex associations and callbacks, when
destroying associated objects results in model.save being called from
after_destroy hook.
Fix and spec for it are provided.

Merged as 30d8ad247d2e430d059402a3b647c4ca23d0b8e6. Thanks for you contribution!


Pie chart and Scatter chart won't display

Hi,
First of all, I'm fairly new to angular so I could be doing some dumb error in my controller ...
Anyway, i am trying to create a pie chart or a  scatter chart. I have a controller providing me with formatted data and a html template calling some angularjs-nvd3 directives on the data. The problem is that both charts don't display in my page even though i don't have any error printed in my browser's console (I am using chrome btw).
Here is my template:
https://gist.github.com/tahanajar/6da2a05ee3640ed29dad
And my controller :
https://gist.github.com/tahanajar/b3ca537ccccbef49b784
FYI, i am using the recommended version of nvd3 which is 1.1.15-beta.
Does anyone has any idea about this issue ?
Thanks a lot for any kind of help you might provide me.

I ran into the same issue of. Check out this other thread https://github.com/angularjs-nvd3-directives/angularjs-nvd3-directives/issues/338
I had success following the Plunkr written by @kylebradshaw (http://plnkr.co/edit/aZK8OZ9eq8jJpkGRILrM?p=preview)


[Plausible Analytics] ...

Extension
https://www.raycast.com/thebrokenfinger/plausible-analytics
Description
Error:
TypeError: Invalid URL

URL:url:775:36
Xe:index.js:37:16178
index.js:37:19422
    at new Promise (<anonymous>)

Steps To Reproduce
Can’t add our website (an.jetzt) through the extension, potentially to the newish TLD .jetzt failing validation?
Current Behaviour
No response
Expected Behaviour
No response

I can reproduce this. Doesn't work with any TLD, not even .com.


As a clear-minded SoC professor, I filter the student’s FYP projects based on what specialisation they are doing

... (e.g. : network research, AI)

This is a duplicate issue of #28.


c: Implement ifunc based dispatcher

ifunc based dispatcher is a linux/glibc/ELF feature which allows to
run detection code at early elf startup, and bind a particular
implementation for a function. After that, the chosen implementation
will be executed when refer to by the the common name.
Added an option to use the ifunc dispatcher for any Makefile target,
simply specify BLAKE3_DISPATCH=ifunc, like so:
make test_asm CFLAGS='-DBLAKE3_TESTING=1' BLAKE3_DISPATCH=ifunc
make test CFLAGS='-DBLAKE3_TESTING=1' BLAKE3_DISPATCH=ifunc

@sneves I'll defer to you on this one. @xnox have you been able to measure a performance improvement from using ifuncs? I've always assumed that our dispatching overhead was in the noise, but I don't have much data on it. (Compiling with RUSTFLAGS="-C target-cpu=native" on my AVX2 laptop does show a few cycles of improvement per compression, but I don't know how many other effects it has besides optimizing out the dispatch.)

@sneves I'll defer to you on this one. @xnox have you been able to measure a performance improvement from using ifuncs? I've always assumed that our dispatching overhead was in the noise, but I don't have much data on it. (Compiling with RUSTFLAGS="-C target-cpu=native" on my AVX2 laptop does show a few cycles of improvement per compression, but I don't know how many other effects it has besides optimizing out the dispatch.)

In the ifunc dispatcher I used builtins for CPU features detection. I wonder if I can bring that over to the regular dispatcher, instead of using hand coded cpuid calls. Or at least use cpuid.h on Linux there.
I will try to instrument measurements for this. In theory, this should only help very little for long running processes as dispatcher code is only executed once per application lifetime. Instead of on every invocation.

as dispatcher code is only executed once per application lifetime. Instead of on every invocation.

Might be worth clarifying, our dispatcher code today only executes once per process. Subsequent calls use the cached result: https://github.com/BLAKE3-team/BLAKE3/blob/master/c/blake3_dispatch.c#L85-L87

In the ifunc dispatcher I used builtins for CPU features detection. I wonder if I can bring that over to the regular dispatcher, instead of using hand coded cpuid calls. Or at least use cpuid.h on Linux there.

We don't target GCC/Linux exclusively. Using builtins just means we'd have to maintain the builtin version and the other version.
I considered at the time using function pointers instead of regular branches for dispatching, as I had done before on BLAKE2. One, perhaps irrelevant, scenario where it matters is if you enable Spectre retpoline mitigations (e.g. -mretpoline, -mindirect-branch=thunk); then your function pointer overhead spikes by ~70 cycles, whereas the current dispatch method remains more or less constant. But this doesn't matter when using ifuncs, since they're implemented as relocations.
Anyway, back to overhead. I'm not sure there's a measurable benefit. I wrote a specific benchmark to measure the overhead of a single blake3_compress_in_place call, which would be the most affected by this. The results, on a Skylake chip and the median of 2^24 measurements, are thus:
Current dispatching:
blake3_compress_in_place: 240 cycles/compression
blake3_compress_in_place_sse41: 234 cycles/compression

ifunc dispatching:
blake3_compress_in_place: 240 cycles/compression
blake3_compress_in_place_sse41: 234 cycles/compression

So performance-wise, it doesn't seem to make much of a difference. There's around 6 cycles of overhead relative to calling the function directly.
However, if we enable LTO, the detection code of blake3_compress_in_place can get inlined into the loop and the function call overhead is reduced:
blake3_compress_in_place: 238 cycles/compression

This is not the case for ifunc, which always remains at 240.
In any case, since this is mostly a very contained feature, I don't have much of a problem with including the ifunc dispatcher and letting users choose.

In the ifunc dispatcher I used builtins for CPU features detection. I wonder if I can bring that over to the regular dispatcher, instead of using hand coded cpuid calls. Or at least use cpuid.h on Linux there.

We don't target GCC/Linux exclusively. Using builtins just means we'd have to maintain the builtin version and the other version.

There will always be platform specific code. __builtin_cpu_supports is supported by GCC and CLANG, which gives coverage across MacOS/FreeBSD/Android/Linux, and Windows when those compilers are used. And a different code path is already used for Windows. What I am proposing is portable and is simplier to use and understand. And can be extended to more optimisations later.

I considered at the time using function pointers instead of regular branches for dispatching, as I had done before on BLAKE2. One, perhaps irrelevant, scenario where it matters is if you enable Spectre retpoline mitigations (e.g. -mretpoline, -mindirect-branch=thunk); then your function pointer overhead spikes by ~70 cycles, whereas the current dispatch method remains more or less constant. But this doesn't matter when using ifuncs, since they're implemented as relocations.
Anyway, back to overhead. I'm not sure there's a measurable benefit. I wrote a specific benchmark to measure the overhead of a single blake3_compress_in_place call, which would be the most affected by this. The results, on a Skylake chip and the median of 2^24 measurements, are thus:
Current dispatching:
blake3_compress_in_place: 240 cycles/compression
blake3_compress_in_place_sse41: 234 cycles/compression

ifunc dispatching:
blake3_compress_in_place: 240 cycles/compression
blake3_compress_in_place_sse41: 234 cycles/compression

So performance-wise, it doesn't seem to make much of a difference. There's around 6 cycles of overhead relative to calling the function directly.
However, if we enable LTO, the detection code of blake3_compress_in_place can get inlined into the loop and the function call overhead is reduced:
blake3_compress_in_place: 238 cycles/compression

This is not the case for ifunc, which always remains at 240.

Can you share the benchmark you did to detect this? This is very interesting analysis which might be helpful in other code too. Especially the effect of lto on dispatcher.

In any case, since this is mostly a very contained feature, I don't have much of a problem with including the ifunc dispatcher and letting users choose.

Thanks. But I'd want to reduce the number of compile options eventually, to publish a performant shared C library, on x86, arm, power, and z.

#include <cstddef>
#include <cstdint>
#include <cstdlib>
#include <cstdio>

#include <vector>
#include <algorithm>

#include <immintrin.h>

extern "C" {
	void blake3_compress_in_place(uint32_t cv[8], const uint8_t block[64], uint8_t block_len, uint64_t counter, uint8_t flags);
	void blake3_compress_in_place_sse41(uint32_t cv[8], const uint8_t block[64], uint8_t block_len, uint64_t counter, uint8_t flags);
}

namespace {

	static inline uint64_t rdtsc() {
	  _mm_lfence();
	#if defined(__clang__)
	  return __builtin_readcyclecounter();
	#else
	  uint32_t hi, lo = 0;
	  __asm__ __volatile__("rdtscp" : "=a"(lo), "=d"(hi)::"rcx");
	  return lo | (((uint64_t)hi) << 32);
	#endif
	}

	template<typename Function>
	static inline uint64_t bench(Function F, size_t const iters) {
		const uint8_t block[64] = {};
		uint32_t cv[8] = {};
		std::vector<uint64_t> cycles(iters + 1);
		for(auto& c : cycles) {
			c = rdtsc();
			F(cv, block, 64, 123, 0x99);
			 __asm__ __volatile__("" : : "m"(cv), "m"(block) : "memory");
		}
		for (size_t i = 0; i < iters; ++i) {
	    cycles[i] = cycles[i + 1] - cycles[i];
	  }
	  cycles.pop_back();
	  std::sort(begin(cycles), end(cycles));
	  return cycles[iters / 2];
	}

}

int main() {
	const size_t kIters = 1UL << 24;
	printf("blake3_compress_in_place: %lu\n", bench([](auto...args) { blake3_compress_in_place(args...); }, kIters));
	printf("blake3_compress_in_place_sse41: %lu\n", bench([](auto...args) { blake3_compress_in_place_sse41(args...); }, kIters));
	return 0;
}


extract_boundaries() - API error

error connecting to API when running extract_boundaries()

Fixed and merge to main. Try re-installing.
glamr::si_path("path_vector") |>
  gisr::get_vcpolygons() |>
  gisr::extract_boundaries(country = "South Africa", level = 4) |>
  gisr::gview()


Allow ghc-8.6

My travis is green https://travis-ci.org/phadej/hashable-time, so I went ahead and made a revision http://hackage.haskell.org/package/hashable-time-<IP_ADDRESS>/revisions/


disambiguate non-letter'd high yds grades

Possible tentative fix for: https://github.com/OpenBeta/sandbag/issues/137

I suggest two changes here:

round the top and bottom of the range in opposite directions
add some tests

@musoke updated according to your suggested implementation. Thank you so much for the guidance!
After updating, some tests (integration tests?) in readme.ts were failing due to hardcoded non-integer return values. This caused me to notice that French scale implementation was also suffering from the same defect, and could potentially return non-integer score values for slash grades. So, I updated with the same logic, assuming slash grades to be exclusive of adjacent grades in the french scale. Is that a correct?
Also, definitely LMK if there are any other issues with my implementation and I'll try to fix.

After updating, some tests (integration tests?) in readme.ts were failing due to hardcoded non-integer return values. This caused me to notice that French scale implementation was also suffering from the same defect, and could potentially return non-integer score values for slash grades. So, I updated with the same logic, assuming slash grades to be exclusive of adjacent grades in the french scale. Is that a correct?

Yes, now that you point it out, this this is going to be an issue with all the scales. Maybe you could define a function to do this check for each scales.  Or I can add it to other scales in a followup.

After updating, some tests (integration tests?) in readme.ts were failing due to hardcoded non-integer return values. This caused me to notice that French scale implementation was also suffering from the same defect, and could potentially return non-integer score values for slash grades. So, I updated with the same logic, assuming slash grades to be exclusive of adjacent grades in the french scale. Is that a correct?

Yes, now that you point it out, this this is going to be an issue with all the scales. Maybe you could define a function to do this check for each scales. Or I can add it to other scales in a followup.

Updated accordingly with a getRoundedScoreTuple utility func, and applied to other grade scales which could return a tuple with high/low based on adjacent scores. Is that what you had in mind? @musoke
It doesn't feel super DRY, but then again that seems to be the running style in this codebase? The enclosing getScore function is mostly the same across grade scales, so I also tried to follow roughly the same level of abstraction. Let me know if stylistically there is anything I should know 😄

That looks good to me.
Yes, is a  lot of repetition due to the adhoc evolution of sandbag.  The special cases in methods like getScore do make this a little tricky.  I would welcome refactors to improve this.


feat: Lazy memory initialization in assembly based implementation

This saves an unnecessary memory zeroing process, in the offline test,
this only brings ~ 0.3ms speedup, but in a multi-threaded environment
where multiple VM instances are running in parallel, this might make a difference

Closing this since #56 brings bigger performance boost, and deploying this after #56 will actually decrease performance.


Column searching, sorting Questions

Has anyone been able to disable sorting for specific columns? As far as I've seen its true or false in the  and applies to all columns.
Along with the sorting is it possible to apply searches to specific columns only?
Has anyone worked on implementing exact match searches? I have a task coming up and would be able to contribute some hours to implementing that feature if someone might have any pointers for places to start?
Really like this datatable.

exactSearch enforces exact match searches, however the rest needs to be implemented.
Probably add sortable and searchable overrides to columns so that a column can be specified to be not sortable and/or searchable sot that it overrides the global datatable settings.


Is it possible to create a generic base class of all the message class generated for C++

If all the message class generated for C++ can have a generic base class then it will make life very easy.

I'm closing this as it is too vague and not clear what the benefit is.

I'm closing this as it is too vague and not clear what the benefit is.

I am creating a factory of SBE messages and would like to return SBE object based on the basis of sbe message received but unable to return since actual SBE type is not known. If all SBE messages classes were derived from common SBE message class then i could have defined the factory method() returning the message base class that would have made my life easy.

We are happy to consider PRs and we also offer commercial support.


Support for TRIM, POSITION, CONCAT pipe operator, RAISE_ERROR in spark.

Support for Position function

Support for Trim function

Support for Concat Pipe operator

Latest Datetime patterns

Reverted back addition of date +- interval DAY(before it was data_add, date_sub), date +- interval month(add_months) to binary operators as QA wants it  that way.

Sorry for the late addition and delay in replying to review comments. Added but we were also getting big_query ERROR function in canonical form. Mapped to spark's RAISE_ERROR function. https://docs.databricks.com/spark/latest/spark-sql/language-manual/functions/raise_error.html

Please add a test case for RAISE_ERROR

added test case for BigQuery ERROR to Spark RAISE_ERROR function by adding the operator in test case itself as the operator is not part of calcite.


Suggestion: Grappling Hook enchantment

I would suggest 2 things:

Make the grapple able to hook onto flat walls.
Apply slow falling or damage reduction when jumping down ledges.

Thanks for your suggestion. We internally have plans to replace the grappling hook enchantment with an actual item, which will make things like grappling walls possible. I'm not sure whether slow falling would fit this item, but I'll keep this in mind for future discussions.


flutter pub run build_runner build loses interactive prompts

I have a project that uses built_value with build_runner.  If I do:

flutter clean
flutter pub get
flutter pub run build_runner build

I get an error:

[INFO] Checking for unexpected pre-existing outputs....
[INFO] Found 2 declared outputs which already exist on disk. This is likely because the.dart_tool/build folder was deleted, or you are submitting generated files to your source repository.
[SEVERE] Conflicting outputs were detected and the build is unable to prompt for permission to remove them. These outputs must be removed manually or the build can be run with --delete-conflicting-outputs. The outputs are: foo.g.dart
bar.g.dart
pub finished with exit code 78

If I run pub run builder_runner build without going through flutter, I instead get a prompt:

[INFO] Checking for unexpected pre-existing outputs....
[INFO] Found 2 declared outputs which already exist on disk. This is likely because the.dart_tool/build folder was deleted, or you are submitting generated files to your source repository.
Delete these files?
1 - Delete
2 - Cancel build
3 - List conflicts

It would be nice if running pub through flutter showed the same prompt.

Hi @jamesderlin
I just tried latest stable with latest package versions, no issue with flutter pub run build_runner build

logs
PS C:\Users\Taha\StudioProjects\testapp> fs pub run build_runner build
[INFO] Generating build script...
[INFO] Generating build script completed, took 269ms

[INFO] Precompiling build script......
[INFO] Precompiling build script... completed, took 4.9s

[SEVERE] Nothing can be built, yet a build was requested.
[INFO] Initializing inputs
[INFO] Building new asset graph...
[INFO] Building new asset graph completed, took 922ms

[INFO] Checking for unexpected pre-existing outputs....
[INFO] Checking for unexpected pre-existing outputs. completed, took 1ms

[INFO] Running build...
[INFO] Running build completed, took 4ms

[INFO] Caching finalized dependency graph...
[INFO] Caching finalized dependency graph completed, took 34ms

[INFO] Succeeded after 50ms with 0 outputs (0 actions)

PS C:\Users\Taha\StudioProjects\testapp>

flutter doctor -v
[✓] Flutter (Channel stable, 2.5.1, on Microsoft Windows [Version 10.0.22000.194], locale en-US)
    • Flutter version 2.5.1 at C:\Users\Taha\Code\flutter_stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ffb2ecea52 (5 days ago), 2021-09-17 15:26:33 -0400
    • Engine revision b3af521a05
    • Dart version 2.14.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\Taha\Code\android-sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_SDK_ROOT = C:\Users\Taha\Code\android-sdk
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.3)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.11.31702.278
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 2020.3)
    • Android Studio at C:\Users\Taha\Code\android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.60.1)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.26.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22000.194]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 93.0.4577.82
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 93.0.961.47

• No issues found!

@TahaTesser It does not appear that you did anything to trigger an interactive prompt from pub run, so I don't see any evidence that this is fixed.

I am still observing this behavior.  You can reproduce this by cloning the package:built_value and testing against its example:

git clone https://github.com/google/built_value.dart.git
cd built_value.dart/example
flutter pub get
flutter pub run build_runner build

Result:

Fails with no interactive prompt to correct the problem.

Compare against doing dart run build_runner build, which does prompt.
I tested with Flutter version 2.6.0-6.0.pre.172 buitl from a921c1cf3d.

Hi @jamesderlin
Thanks for the details, I can confirm the issue

logs
tahatesser@Tahas-MacBook-Pro example % fs pub run build_runner build
[INFO] Generating build script...
[INFO] Generating build script completed, took 396ms

[WARNING] Deleted previous snapshot due to missing asset graph.
[INFO] Creating build script snapshot......
[INFO] Creating build script snapshot... completed, took 11.8s

[INFO] There was output on stdout while compiling the build script snapshot, run with `--verbose` to see it (you will need to run a `clean` first to re-snapshot).

[INFO] Initializing inputs
[INFO] Building new asset graph...
[INFO] Building new asset graph completed, took 860ms

[INFO] Checking for unexpected pre-existing outputs....
[INFO] Found 7 declared outputs which already exist on disk. This is likely because the`.dart_tool/build` folder was deleted, or you are submitting generated files to your source repository.
[SEVERE] Conflicting outputs were detected and the build is unable to prompt for permission to remove them. These outputs must be removed manually or the build can be run with `--delete-conflicting-outputs`. The outputs are: lib/serializers.g.dart
lib/values.g.dart
lib/generics.g.dart
lib/enums.g.dart
lib/interfaces.g.dart
lib/collections.g.dart
lib/polymorphism.g.dart
pub finished with exit code 78
tahatesser@Tahas-MacBook-Pro example % dart run build_runner build
[INFO] Generating build script completed, took 380ms
[WARNING] Deleted previous snapshot due to missing asset graph.
[INFO] Creating build script snapshot... completed, took 13.1s
[INFO] There was output on stdout while compiling the build script snapshot, run with `--verbose` to see it (you will need to run a `clean` first to re-snapshot).
[INFO] Building new asset graph completed, took 671ms
[INFO] Found 7 declared outputs which already exist on disk. This is likely because the`.dart_tool/build` folder was deleted, or you are submitting generated files to your source repository.
Delete these files?
1 - Delete
2 - Cancel build
3 - List conflict

stable
master

❗
❗

Check flutter doctor -v outputs for each channel below

flutter doctor -v
[✓] Flutter (Channel stable, 2.5.1, on macOS 12.0 21A5522h darwin-x64, locale en-GB)
    • Flutter version 2.5.1 at /Users/tahatesser/Code/flutter_stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ffb2ecea52 (6 days ago), 2021-09-17 15:26:33 -0400
    • Engine revision b3af521a05
    • Dart version 2.14.2

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-31, build-tools 30.0.3
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.0, Build version 13A233
    • CocoaPods version 1.10.2

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

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.0 21A5522h darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 94.0.4606.54

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 2.6.0-6.0.pre.184, on macOS 12.0 21A5522h darwin-x64, locale en-GB)
    • Flutter version 2.6.0-6.0.pre.184 at /Users/tahatesser/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 64161b6caa (11 hours ago), 2021-09-22 18:38:04 -0400
    • Engine revision dd28b6f1c8
    • Dart version 2.15.0 (build 2.15.0-137.0.dev)

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-31, build-tools 30.0.3
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

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

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.0 21A5522h darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 94.0.4606.54

! Doctor found issues in 1 category.

tahatesser@Tahas-MacBook-Pro ~ % dart --version
Dart SDK version: 2.15.0-137.0.dev (dev) (Wed Sep 22 04:32:49 2021 -0700) on "macos_x64"
tahatesser@Tahas-MacBook-Pro ~ %

✅: No Issue ❗: Issue reproduced
Thanks for filing the issue


Cannot Filter on Nested Data using LocalPlugin

Griddle version
1.13.1
Expected Behavior
Entering a query in the filter box which matches a nested field should return the row with that data.
Actual Behavior
Entering a query in the filter box which matches a nested field returns no results.
Steps to reproduce
Run the storybook and browse to the 'with nested column data' story. Enter 'hawaii' in the filter, observe there are no results.
Pull request with failing test or storybook story with issue
Can be reproduced from the storybook.
Edit: it appears this last worked in v1.11.2

So having delved into this a little, it appears the reason for this is that rather than allow filtering on all base (non-nested) columns except those with fllterable set to false, it now ignores any column, without ColumnDefinition set. ie if a column isn't defined in the ColumnDefinitions it's not filtered on at all.
A workaround is to define your nested property as a non-visible column, so it's included in the list of columns to filter. This performs toString() on the nested Map or List object, exposing all the properties.
<ColumnDefinition id="location" visible={false} />

It doesn't feel like starting with the base properties and then filtering those based on the columnProperties is the right way round, especially considering this will mean all nested properties will be ignored.


[backport] Add order option in cupy.testing.shaped_random

Backport of #4091

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 9c2dbd3b0b42aa59e7e1e99f6799cf98ea9b7485, target branch v8) succeeded!

LGTM.


fix(ui-watt): prefer watt-button (WIP)

Description
This PR aims to use <watt-button> in the timepicker, datepicker and modal component. To accomplish this, a few changes had to be made:

Set the width of the icon button variant to be 44px. This aligns with the design in Figma, so could be considered a bug fix.
Use :focus-visible selector instead of :focus for all buttons. This means focus state is only visible to keyboard users.

These changes are potentially breaking (although only in a styling-sort-of-way), so it has to be tested and cleared with the other teams. There might also be accessibility concerns regarding :focus-visible that I'm not aware of, so that should also be looked into.

praise: looks nice. Hopefully enough to start the process of adjusting the icon button to look better in such contexts.
issue: EO bump 😒


RFC feat: publish --do-not-build

This adds a --do-not-build flag to wrangler publish. We've had a bunch of people asking ot be able to upload a worker directly, without any modifications. While there are tradeoffs to this approach (any linked modules etc won't work), we understand that people who need this functionality are aware of it (and the usecases that have presented themselves all seem to match this).

Hi,
I like this functionality. I have some workers that are just a simple javascript file and I like having them uploaded as-is.
My question though is, would it be possible for this to be a wrangler.toml option? With wrangler 1 I'm currently using type = javascript which makes it do a similar thing and understandably was removed for wrangler 2. What I'm envisioning is something like nobuild = true and then main file is used directly and never built or modified.
One of the struggles I came across as well as the file being built unnecessarily before upload was using wrangler dev since it spat out line numbers in errors that didn't mean anything relative to my source. I understand there are probably source map setups that can restore this but my feeling when it's a single javascript file is that it should just be left alone.
And this proposal seemingly wouldn't solve that since it looks like its only for publish?
Thanks!

This is exactly the kind of feedback I was looking for, thank you very much! We could probably do it for dev as well, as well as being able to specify it in wrangler.toml.

Sounds good!

I've renamed it to --no-build, and added it for wrangler dev as well. I'm going to hold back on adding it to wrangler.toml so we can try this out for a bit before doing so, and then I'll put it in a future PR.
Opened this up for review.

"no build" is actually a bad name for this flag, since "build" refers to custom builds, and this would imply that it wouldn't run custom builds. Will change this to a different name before releasing today. Open to suggestions. no-bundle? raw-publish?

custom-only? I think no-bundle sounds best.

Aight, let's go with --no-bundle. I'll put up a PR asap

https://github.com/cloudflare/wrangler2/pull/1399

I had been following his, hoping for an actual --no-build, and it's definitely clearer now that it's called --no-bundle.
However, what if I really wanted --no-build? How can I just upload the main file and skip the build command?
My reasons basically mirror what's in https://github.com/cloudflare/wrangler/issues/2219. I have CI building and testing from an existing webpack build, and for efficiency and consistency, I want to treat my built JavaScript as an "immutable" artifact that I can deploy as is. This is especially important as I redeploy the same script across multiple environments and because build inputs like environment variables could change the contents of our built script.

@wuservices we're separately working on "versioning" which should (theoretically) let you do reverts to a previously deployed version. I think that would be a proper solution here, instead of a "no-build" solution. Alternately, as we're building a first class api to manipulate wrangler, you may be able to wire this up yourself on the future.

Of course, another option is to not use the [build] field, and just manually run your build step yourself before running publish/dev.

@threepointone the ability to revert could certainly be helpful, but it wouldn't fit into our current TeamCity CI workflow as naturally as being able to retrigger a previous deployment that completed successfully. That's workable though. However, if you had multiple separate environments (or zones or accounts), versioning wouldn't allow you to deploy the exact same code everywhere.
However, it sounds like getting creative to avoid the [build] field could definitely work, even if this falls a bit outside the expected usage! Thanks for the idea.


Mmd/gh action build workflows

What does this PR change?
Introduces GitHub Actions workflows on PRs to run two builds on each PR: open source and with closed source. The workflows are based on the existing KCM CI build step.
The open source build supplants the old CircleCI workflow.
The closed source build is intended to give advanced warning for situations like https://github.com/kubecost/kubecost-cost-model/pull/697, where changes to CM can break a KCM build. The new closed source build should not block a CM PR from merging, I've tried to convey that in the workflow title.
Does this PR rely on any other PRs?

N/A

How does this PR impact users? (This is the kind of thing that goes in release notes!)
N/A
Links to Issues or ZD tickets this PR addresses or fixes
N/A
How was this PR tested?
Pretty sure both new workflows will run on this PR.
Have you made an update to documentation?
N/A

Once merged, we may have to manually go into CircleCI to remove the old workflow.


fetchpatch: add "decoder" argument for gerrit

Motivation for this change
(and gitiles)
This allows fetching a patch from servers that return them
base64-encoded, like this:
fetchpatch {
  name = "gcc.patch";
  url = "https://chromium.googlesource.com/aosp/platform/external/libchrome/+/f37ae3b1a873d74182a2ac31d96742ead9c1f523^!?format=TEXT";
  decode = "base64 -d";
  sha256 = "11j1bqz2p8xrfzgfrylgdvmqs45489c4ckl7l0ra1dpfgbqy94a8";
}

This will mean fewer patches have to be directly included in Nixpkgs, permanently bloating the repository size.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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.

needs doc in https://nixos.org/manual/nixpkgs/stable/#sec-patches doc/contributing/coding-conventions.chapter.md

Done.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/normalized-patch-is-empty/26122/15

Updated to avoid dumping potentially binary files to the terminal if the decoder fails, based on @samueldr's feedback on Matrix, and also made the error text be displayed if the decoder returns non-zero.