Example of custom async task with single block_on

Objective

Compare performance between using single block_on and multiple block_on(s) when dealing with massive custom async tasks
The result shows (on windows) that using single block_on has better performance (by 20%~50%), is it feasible to perform poll_once inside ecs framework then, that would save a lot of boilerplate code?
By avoiding poll_once and block_on in user code, it gets the best performance.

Solution

Add an async-bench example
Run with release build cargo run --release --example async_bench
It pints result like

[handle_tasks_no_poll_once]n_frames executed: 206, avg fps: 34.8(target:120), duration: 5.920s
[handle_tasks]        n_frames executed: 72, avg fps: 12.4(target:120), duration: 5.784s
[handle_tasks_par]    n_frames executed: 123, avg fps: 21.6(target:120), duration: 5.697s
[handle_tasks_par_2]  n_frames executed: 88, avg fps: 15.3(target:120), duration: 5.751s

@NiklasEi Thanks! Sure I can add another micro-benchmark to the benches, in the meanwhile, this one is more like a small toy program that exercises the main stacks of bevy ecs, and show numbers in an E2E scenario.  I think micro-benchmark and small benchmark program serves different purposes and should co-exsist.
BTW, It does show a huge perf gap between using single and multiple block_on(s), regarding my question below, where is the best place for discussion? GH discussion or discord?

is it feasible to perform poll_once inside ecs framework then, that would save a lot of boilerplate code?

The difference between handle_tasks_par and handle_tasks_par_2 are due to the .collect::<Vec<_>>(). Also, you can squeeze a little more speed by changing your n_tasks to a bool, as you don't actually care about the number of task performed.
On the difference between multiple block_on vs a single bigger one, this is more a question for futures-lite but it makes sense as blocking the thread is expensive, so doing it only once is better. Discussion on GH discussion is best as answers will be easily searchable, don't hesitate to post your discussion link on discord if you don't get answers 👍

is it feasible to perform poll_once inside ecs framework then, that would save a lot of boilerplate code?

Are you suggesting something like
struct TaskResult<T>(T);

pub fn handle_tasks<T: Component>(
    mut commands: Commands,
    mut tasks: Query<(Entity, &mut Task<T>)>,
) {
    let done = tasks.iter_mut().map(|(entity, mut task)| async move {
        if let Some(r) = poll_once(&mut *task).await {
            Some((entity, r))
        } else {
            None
        }
    });
    block_on(async {
        for f in done {
            if let Some((entity, result)) = f.await {
                commands
                    .entity(entity)
                    .remove::<Task<T>>()
                    .insert(TaskResult(result));
            }
        }
    });
}

It's possible to add in Bevy, but as it needs to be generic over the task type, you would still need some kind of registration mechanic for your tasks

@mockersf Thanks! I'm thinking of sth like this, users still register Task but can query PollOnce<Task<T>> directly
I guess this requires implementing SysParams for PollOnce<Task<T>>
pub fn handle_tasks<T: Component>(
    mut commands: Commands,
    mut tasks: Query<(Entity, PollOnce<Task<T>>)>,
) {
    block_on(async {
        for entity, task in tasks{
            if let Some(result) = task.await {
                commands
                    .entity(entity)
                    .remove::<Task<T>>()
                    .insert(TaskResult(result));
            }
        }
    });
}

@mockersf After discussion, I found it's better to just totally remove block_on and poll_once from user system code, and here's my proposed API change: https://github.com/bevyengine/bevy/pull/2691

This PR adds a new example. Adding module and item level doc comments, as described in:

#3951 and
#4438,

would be really useful to those who will browse examples.

I think it's sufficient to move the description on the top, as a module level doc comment.
I also think this should not be advertized as a benchmark, as examples are built without optimizations by default.

Adding the Adopt-Me tag; I think this is valuable but the issues never got fixed.


Contribution guide updates

console to bash for scripts
composer requirements to run tests / CS checks

Thanks, @webimpress


Compact support for deferred loading

I found this code in the dart2js output:
  if (typeof($dart_deferred_initializers) === "undefined") $dart_deferred_initializers = Object.create(null);
I have two observations:

I'm not using deferred loading, so don't think this is needed.
The code could be more compact:

  $dart_deferred_initializers = $dart_deferred_initializers || Object.create(null);

I believe the suggested optimization was implemented. I think we also avoid outputting the deferred load hooks unless they are used.


Bootstrapping

Tasks:

[x] Add docker compatible .gitignore
[x] Aggregate all my docker implementations from other projects
[x] Ensure that no security/credentials information gets imported

commit: e9d24efdf6788f3375c6d8237fd0154acd13686f


Allows reordering of colours of palettes

Completed feature allowing users to easily reorder colours of a palette @kamalkishor1991

@Walia8416 thanks for the PR.
we have to ideally figure out if there is a way to do it using drag and drop.
I found few resources so looks like it is possible in react native.
https://www.npmjs.com/package/react-native-draggable
https://blog.reactnativecoach.com/creating-draggable-component-with-react-native-132d30c27cb0
Please check.
In case drag and drop is harder to implement we have to at least fix the animation so that the color moves downwards instead of animation happening from the side.

After researching drag and drop is uncharted territory and trying to implement has resulted in various bugs. However I will fix the animation issue.@kamalkishor1991

It is now implemented here - #171


Replace GoogleAnalyticsTracker with Snowplow-backed MeltanoTracker

Closes #5941
Closes #5959

There are 23 sites in the repository where a GoogleAnalyticsTracker method is called, and as best I can tell they will all be replaced by MeltanoTracker.track_command_event.
I'm thinking Project should have a tracker member. This will be the base tracker that only has the environment context. It comes with a context manager MeltanoTracker.with_contexts that can be used to track events with additional context. For instance:
project.tracker.track_xyz_event({'name': 'qwerty', 'magnitude': 789})

plugin_context = SelfDescribingJson(
    'iglu:com.meltano/command_event/jsonschema/1-0-0',
    {...}
)

with project.tracker.with_context(plugin_context) as tracker:
    # `tracker` is only provided by the context manager for convenience - the same instance is used
    assert project.tracker is tracker
    tracker.track_xyz_event({...}) # Tracks an xyz event as before, but this time with additional context

I'm thinking Project should have a tracker member. This will be the base tracker that only has the environment context.

An alternative would be to decouple the project from the environment context (i.e. split the environment context into an environment context, and a project context), or to start the tracker off with no contexts and make it so that the caller needs to explicitly provide the environment context.
I'm leaning towards the first option, since we'll always want the environment context, and burdening the caller with obtaining it and passing it through every time seems entirely unnecessary.

@WillDaSilva - The meeting today was super helpful, I think. Ping me or send me a zoom invite if you need anything further. Thanks!

GitLab CI for this branch is currently failing hard because the Poetry lock file is out of sync with pyproject.toml.
It's out of sync because I added a couple of dependencies (backports.cached_property and tzlocal), but I haven't been able to get Poetry to update the lock file. Poetry just keeps taking more and more memory before ultimately getting killed by the OOM killer.
I've verified that no versions of those dependencies were installed by Meltano previously, and I've manually checked the transitive dependencies of the 2 dependencies I added. I don't see any conflicts, so I'm not sure what's going wrong here.

Not sure why but I think its hanging on backports.cached-property:
$ poetry lock -vvv
<-- spam for infinity --->
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
   1: selecting backports.cached-property (1.0.1)
^C   1: Version solving took 12.258 seconds.
   1: Tried 1 solutions.

@pandemicsyn I'll switch to https://pypi.org/project/cached-property/ and see if Poetry likes that better.

@pandemicsyn Switching to the other cached property package seems to have resolve the issue. I've used both in the past without much trouble, and we're only going to be using the basic features of it, and it's temporary until we drop Python 3.7 support, so I think we're fine with taking the fancier one for now that is more than just a backport.

so I think we're fine with taking the fancier one for now that is more than just a backport.

@WillDaSilva 👍 sounds good.
fwiw - its weird - has to be a conflict caused by another dep because this minimal pyproject.toml in a fresh project generates a lock file fine 😕
[tool.poetry]
name = "test"
version = "0.1.0"
description = ""
authors = ["pandemicsyn<EMAIL_ADDRESS>
[tool.poetry.dependencies]
python = "3.7,<3.10"
"backports.cached-property" = "^1.0.1"
meltano = "^1.105.0"

[tool.poetry.dev-dependencies]

[build-system]
requires = ["poetry-core>=1.0.0"]
build-backend = "poetry.core.masonry.api"

Going back to our discussion about having a plugin event versus a plugins context, I'm thinking we should have a plugin context that is for a single plugin. That context is entered when the plugin in loaded. We'll then have a slim plugin_loaded event that is emitted within that context.
Though I have not used Snowplow before, I get the impression that it's ideal for us to rely primarily on contexts for gathering information, along with slim events whose main purpose is to provide info on what contexts were activate when the event was emitted.
Maybe I'm totally off-base with how Snowplow ought to be used. What do you all think? @aaronsteers @pnadolny13 @tayloramurphy @pandemicsyn

Maybe I'm totally off-base with how Snowplow ought to be used. What do you all think? @aaronsteers @pnadolny13 @tayloramurphy @pandemicsyn

@WillDaSilva yeah that makes sense to me. I personally havent used Snowplow until now and its super flexible on how we can send data so its really up to us.
The only thing I'd add is that as part of a modeling discussion with Joao from Snowcat Cloud he was encouraging us to model the data in a way that encapsulates all our data in a few flexible and generally useful events/context (i.e. Command vs Meltano ELT or Meltano Run), each context and event has its own schema so making them generic reduces the need to constantly be updating the schemas or adding new ones. I think thats normal modeling/DRY principles anyways but still worth sharing.

I'm not sure we wanna fire an event for every plugin seen/loaded

I agree with @pandemicsyn. There some inefficiencies in plugin loading a.t.m. (e.g. extra roundtrips) so events would be overcounted there.

Ok @aaronsteers @pnadolny13 and I just a had a group PR review over zoom to go over the schema's. Logged all the issues inline above. Next steps are:

resolve the circular import
address the PR feedback that came on the of the call
add cli tracking events to meltano invoke
merge this pr
profit!

This is gonna probably have a lot of linter & doc convention failures (both for the new additions and because theres adjecent updates in some files that haven't been touched during other linter fixups).

It lives!

Done.

@pandemicsyn - I went through all the files where my prior comments were lost. You should have all my feedback now.
Also - in case you haven't already seen, there are some merge conflicts noted that are pending resolution:

Ok, with the latest change set the legacy tracking is working as well. Verified that the events are at least accepted as "good" by snowplow micro (can't speak to whether they look like they old ones 😁 ).


Extract govuk::lvm to own module

In a bid to break up the monolithic govuk module.

:+1: merge when ready


Store Videos In Memory

PyTube takes around 1 minute for every 30 minutes of a 4K Youtube video. Since we aren't shipping an API, we simply need to store these in memory to pump out our MVP as fast as possible. Future functionality will use Playwright for video scraping.

Currently relying on cached clips that we scraped previously.


No events response is being logged with error level

What happened:
failed to get the latest event  {"error": "rpc error: code = NotFound desc = no events found"}
github.com/pipe-cd/pipe/pkg/app/piped/apistore/eventstore.(*store).GetLatest
        pkg/app/piped/apistore/eventstore/store.go:158
github.com/pipe-cd/pipe/pkg/app/piped/eventwatcher.(*watcher).modifyFiles
        pkg/app/piped/eventwatcher/eventwatcher.go:208
github.com/pipe-cd/pipe/pkg/app/piped/eventwatcher.(*watcher).updateValues
        pkg/app/piped/eventwatcher/eventwatcher.go:183
github.com/pipe-cd/pipe/pkg/app/piped/eventwatcher.(*watcher).run
        pkg/app/piped/eventwatcher/eventwatcher.go:158
failed to check outdated value  {"error": "failed to get the latest Event with the name \"helm-release\""}
github.com/pipe-cd/pipe/pkg/app/piped/eventwatcher.(*watcher).updateValues
        pkg/app/piped/eventwatcher/eventwatcher.go:185
github.com/pipe-cd/pipe/pkg/app/piped/eventwatcher.(*watcher).run
        pkg/app/piped/eventwatcher/eventwatcher.go:158

What you expected to happen:
How to reproduce it:
Environment:

piped version:
control-plane version:
Others:

/assign @nakabonne

/assign @nakabonne


Identifiers validation failed for dataset Colección de Sonidos Ambientales del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-CSA)

Identifier validation failed for the dataset Colección de Sonidos Ambientales del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-CSA):

Crawler attempt: 334
Publishing organization: Instituto de Investigación de Recursos Biológicos Alexander von Humboldt
Publishing country: COLOMBIA
Installation: IPT IAvH
Number of occurrences indexed for this dataset at the time of the issue: 22127
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 25893 total records; 25893 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3526378/step

New IDs sample:

Old IDs sample:

IAvH-CSA-18017
IAvH-CSA-18018
IAvH-CSA-18019
IAvH-CSA-18020
IAvH-CSA-18021
IAvH-CSA-18022
IAvH-CSA-18023
IAvH-CSA-18024
IAvH-CSA-18025
IAvH-CSA-18026

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Colección de Sonidos Ambientales del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-CSA) : https://doi.org/10.15472/scqpmf.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

@EstebanMH-SiB

Identifier validation still  failing for the dataset Colección de Sonidos Ambientales del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-CSA):

Crawler attempt: 359
Publishing organization: Instituto de Investigación de Recursos Biológicos Alexander von Humboldt
Publishing country: COLOMBIA
Installation: IPT IAvH
Number of occurrences indexed for this dataset at the time of the issue: 22127
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 25893 total records; 25893 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3997552/step

New IDs sample:

Old IDs sample:

IAvH-CSA-18017
IAvH-CSA-18018
IAvH-CSA-18019
IAvH-CSA-18020
IAvH-CSA-18021
IAvH-CSA-18022
IAvH-CSA-18023
IAvH-CSA-18024
IAvH-CSA-18025
IAvH-CSA-18026

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Colección de Sonidos Ambientales del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-CSA) : https://doi.org/10.15472/scqpmf.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Colección de Sonidos Ambientales del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-CSA):

Crawler attempt: 362
Publishing organization: Instituto de Investigación de Recursos Biológicos Alexander von Humboldt
Publishing country: COLOMBIA
Installation: IPT IAvH
Number of occurrences indexed for this dataset at the time of the issue: 22127
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 25893 total records; 25893 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/4052616/step

New IDs sample:

Old IDs sample:

IAvH-CSA-18017
IAvH-CSA-18018
IAvH-CSA-18019
IAvH-CSA-18020
IAvH-CSA-18021
IAvH-CSA-18022
IAvH-CSA-18023
IAvH-CSA-18024
IAvH-CSA-18025
IAvH-CSA-18026

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Colección de Sonidos Ambientales del Instituto de Investigación de Recursos Biológicos Alexander von Humboldt (IAvH-CSA) : https://doi.org/10.15472/scqpmf.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.


[NF] Notches + Hallbach

This is the first proposal on Notch implementation
Therefore there are some short comings and limitations for now:

notches on symmetrie lines are not possible, eg. on x-axis
only the first notch (distribution) is used, ie. lamimation.notch[0]
intersections eg. between different notch shapes are not checked

Possible future features

notches on the yoke

Hello Sebastian,
Thank you for your work !
Regarding your comment in create_FEMM_material: When there is no symmetry there are two surfaces that are need to draw the Lamination "Lamination_Stator_Bore" and "Lamination_Stator_Yoke". We only assign the material for the "Bore" line since it is the one we keep for symmetries. By the way, it will create some issue when splitting the surfaces. I thinking about introducing a new surface object defined by two surfaces.
Great feature to add the possibility to split an Arc in FEMM.
For the intersection, I think that the key will be to use the method comp_angle_opening to know between which angles the notches are located.
Best regards,
Pierre


Align/Redo Dropdown component in Figma and XD

Finding from Review: https://app.clickup.com/t/8669fy7tq
Guideline revision Style Guide: https://github.com/trimble-oss/website-modus.trimble.com/issues/412

CLosing this as its a duplicate


Fix static methods in dropdowns

Pull Request Description
Fixes #7824
The issue was caused by me who didn't think that we can use static methods in dropdowns.
https://github.com/enso-org/enso/assets/6566674/5045c5ce-e33b-48ff-9488-4228c016b563
Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

Merging without QA as the change is trivial


naming conventions: change 3-letter suffix for Data Lake Store from 'dtl' to 'dls'

Dear all, could you please consider if it make sense to change 3-letter suffix for Data Lake Store from 'dtl' to 'dls'. The justification is that other tools already use the 'dls' abbreviation (such as Azure CLI ). Also 'dls' seem to be appropriate for the resoure type 'Microsoft.DataLakeStore'. Ref. 5fe63d8. Thanks.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/best-practices/naming-conventions.md
:white_check_mark:Succeeded
View

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

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/mspnp/architecture-center, working directory: W:\zbyn-s, target branch: master, target branch commit: a8453c4bc7c870fa1a12bb3c02e3b310db87530c, source branch: 9653bdf9d208d55e422f182404b6fa7a/naming-conventions-datalakestore-suffix, source branch commit: .

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

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/mspnp/architecture-center, working directory: W:\xz24-s, target branch: master, target branch commit: a8453c4bc7c870fa1a12bb3c02e3b310db87530c, source branch: 9653bdf9d208d55e422f182404b6fa7a/naming-conventions-datalakestore-suffix, source branch commit: .

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


Malformed forecast URL causes 403 response from API

integration is making requests using comma separated lat & lon, which is causing Pirate Weather API to return a 403 Not Authorized. I confirmed that if the values are URL encoded correctly, the call returns a 200. Example of the requested URL:
https://api.pirateweather.net/forecast/MY_API_KEY/33.333333,-111.111111?units=us
My location is not specified in configuration, it is updated via an automation using a service calls (location changes daily as this is in an RV).
Environment:
HACS | 1.22.0
Pirate Weather | latest as of 2/6/2022
Version | core-2022.2.1
Installation Type | Home Assistant Container
Development | false
Supervisor | false
Docker | true
User | root
Virtual Environment | false
Python Version | 3.9.7
Operating System Family | Linux
Operating System Version | 5.10.63-v7l+
CPU Architecture | armv7l

Thanks using this integration and posting your environment details!
To clarify here: is the example URL you posted there working for you, or is that the issue? The location should be comma separated (with the date as an optional third argument), so that syntax looks correct. Or are you URL encoding the lat/lon/API, and that's returning the 403?

The example URL I included is what the pirate weather integration for HA is making a request for, and getting the 403 response. I am able to consistently reproduce the error using Postman/CURL and the full URL (with my API key). If I use the swagger front-end on pirateweather.com, browse to the forecast endpoint, enter my API key, location as a comma separated string, it generates a URL with the comma escaped (%2C), and gets a 200 response code from the API. Screenshot of this is below.

Ok, that's interesting (and as a side note, happy that the front end is being used).
The "correct" way to call the API is using a comma, so that's why the HA integration is using it. I'm not totally sure why the front end replaces it with the escaped character, but I think I've fixed that now- you can try it using the dev.pirateweather.net back end.
That doesn't solve the initial question about the 403 error though. Troubleshooting step 1, can you double check that you're subscribed to the API in the console (it's the button next to the logo)?

I was not subscribed; after hitting that and restarting HA, I still get the same 403 response. I confirmed that the button next to the pirate weather logo changed to "unsubscribe".  Is there a propagation time, or another step that I missed?

Ahh, that issue! I really wish there was some way to get rid of that extra step, but haven't found one yet.
It takes about 20 minutes to propagate, so let me know if it started working here

It's working great now, thank you.


Switch to slicing-by-8 CRC32 algorithm

Hi all,
This PR replaces the single look-up-table CRC32 implementation with the slicing-by-8 algorithm. The new algorithm is approximately 5 times faster than single table lookup CRC.
This improves Ogg FLAC encoding speed by about 5% and decoding by 10%. Other codecs benefit too. Opus decoding is roughly 1%, Vorbis decoding about 2% faster. Pure muxing of already encoded material will probably benefit the most, but I did not test this.
Switching to a slicing algorithm was proposed before by Rodney Brown, but the discussion came to nothing after raising questions about the patent situation. I did a patent search and found the last of the relevant patents on this algorithm having expired in December 2017.
See here for my original article on this and here for the discussion on HydrogenAudio.
It would be great to see this added to the official Ogg source.
Robert

LGTM, thanks! I think we can live with the extra 7.75KiB table space, so no need to make it configurable. I'll take a bit to review the docs you linked before merging.

Great! Thanks for having a look at it!
The _ogg_crc_init in the #if 0 block is not meant as an option to configure dynamic initialization. It's just to illustrate how the table is generated, just like _ogg_crc_entry before. I think it would be good to keep it to take some magic away from the numbers.

@tdaede Any news on this one?


[SwipeableTextMobileStepper] Remove unused classname

[X] I have followed (at least) the PR section of the contributing guide.

Mmh will open a new one.

Now in #14597


my application cannot find the key

I have a problem.
I have already done all the procedures with my api key, but my map does not redisplay the map. How do I fix this?

Here is my code

import React, {Component} from 'react';
import {View, Text} from 'react-native';
import mapStyle  from './SRC/Styles/MapStyle'
import MapView, {PROVIDER_GOOGLE} from 'react-native-maps'
export default class App extends Component {
render(){
return (

<MapView style={mapStyle.map}
loadingEnabled={true}
region={{
latitude: 37.78825,
longitude: -122.4324,
latitudeDelta: 0.015,
longitudeDelta: 0.0121,
}}>

  </View>
)

}
}

I have the same problem!!

Hi!! @josileudo, I had the same problem.
I solved this problem by following the recommendations in issue #3352 and following the section "Generate the API Key to Integration of Google map"  in this article: https://aboutreact.com/react-native-map-example/
I hope It will resolve you problem. :+1:

I am facing the same issue.


[Sema] Correctly reject throwing apply expressions in lazy property initializers [4.2]

Cherry-pick of https://github.com/apple/swift/pull/17022 by @hamishknight to swift-4.2-branch.
Fixes rdar://problem/40773549.
Thanks Hamish!

@swift-ci Please test


Fix/more validation fixes

Change Type (required)
Indicate the type of change your pull request is:

[ ] patch
[ ] minor
[ ] major

📦 Published PR as canary version: 0.4.2--canary.65.1484

Try this version out locally by upgrading relevant packages to 0.4.2--canary.65.1484

/canary

/canary


Bridge request for Docker Hub

Bridge request

General information
A bridge that let you pass the image name and receive updates on new releases.

Host URI for the bridge (i.e. https://github.com):
https://hub.docker.com/search?q=&type=image

Which information would you like to see?

image name (ability to filtering by this, example: there's redis image but probably I want only for image type: redis:alpine)
tags (versions)
architecture/OS
digest (optional)
update date

How should the information be displayed/formatted?

Docker Image - Image Name - version
Last update date
Link to Docker Hub

Which of the following parameters do you expect?

[X] Title
[X] URI (link to the original article)
[ ] Author
[X] Timestamp
[X] Content (the content of the article)
[ ] Enclosures (pictures, videos, etc...)
[ ] Categories (categories, tags, etc...)

Options

[ ] Limit number of returned items

Default limit: 5

[ ] Load full articles

Cache articles (articles are stored in a local cache on first request): yes
Cache timeout (max = 24 hours): 24 hours

[X] Balance requests (RSS-Bridge uses cached versions to reduce bandwith usage)

Timeout (default = 5 minutes, max = 24 hours): 5 minutes

Here's a project that does that job: https://github.com/TheConnMan/docker-hub-rss

It seems there is some work in progress for this https://github.com/RSS-Bridge/rss-bridge/pull/1996
@VerifiedJoseph if u want me to test it let me know.


Add ERC: Release tokens on predefined periods

Tokens can be linearly released based on the predefined periods,enable lock token balances and gradually release tokens.
When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md
We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

The PR edits only existing draft PRs.
The build passes.
Your GitHub username or email address is listed in the 'author' header of all affected PRs, inside .
If matching on email address, the email address is the one publicly listed on your GitHub profile.

Not sure if you're still working on this or if it's ready for re-review, but you will need to address the comments in the Rationale section before this can be merged.


Photo gallery

Added PhotoGallery activity, array adapter, and associated xml.

Looks good! However, it has all the bugs carried over from Map-Markers, I wonder if we want to instead merge this with Map-Markers so that all the issues will be fixed together?


openapi-types shouldn't be a runtime dependency

It isn't used at runtime.

it's not a runtime dependency but it is a dependency for the types provided by this package, thus can't be a devDependency.

I wonder what a common solution is. Now this package is deployed in production, which is wrong, too.

What do you mean by deployed in production? Perhaps you should use a bundler. :)


update version 2.0.0-dev.19 to 2.0.0-dev.28, this error has occurred when I run my rust unit test：flutter_rust_bridge::frb_generated_boilerplate!();  missing tokens in macro arguments

Describe the bug
update version 2.0.0-dev.19 to 2.0.0-dev.28, run command flutter_rust_bridge_codegen generate
this error has occurred when I run my rust unit test：
error: unexpected end of macro invocation
  --> src/frb_generated.rs:27:1
   |
27 | flutter_rust_bridge::frb_generated_boilerplate!();
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ missing tokens in macro arguments
   |
note: while trying to match `default_stream_sink_codec`
  --> /Users/y/.cargo/registry/src/rsproxy.cn-0dccff568467c15b/flutter_rust_bridge-2.0.0-dev.28/src/for_generated/boilerplate.rs:7:9
   |
7  |         default_stream_sink_codec = $default_stream_sink_codec:ident,
   |         ^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0425]: cannot find value `FLUTTER_RUST_BRIDGE_CODEGEN_VERSION` in this scope
  --> src/frb_generated.rs:31:1
   |
31 | flutter_rust_bridge::frb_generated_default_handler!();
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ not found in this scope
   |
   = note: this error originates in the macro `flutter_rust_bridge::frb_generated_default_handler` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0405]: cannot find trait `CstDecode` in this scope
  --> src/frb_generated.rs:37:16
   |
37 |     path: impl CstDecode<String> + core::panic::UnwindSafe,
   |                ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `CstDecode` in this scope
  --> src/frb_generated.rs:57:18
   |
57 |     number: impl CstDecode<i32> + core::panic::UnwindSafe,
   |                  ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `CstDecode` in this scope
  --> src/frb_generated.rs:77:16
   |
77 |     name: impl CstDecode<String> + core::panic::UnwindSafe,
   |                ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `CstDecode` in this scope
  --> src/frb_generated.rs:98:6
   |
98 | impl CstDecode<bool> for bool {
   |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `CstDecode` in this scope
   --> src/frb_generated.rs:103:6
    |
103 | impl CstDecode<i32> for i32 {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `CstDecode` in this scope
   --> src/frb_generated.rs:108:6
    |
108 | impl CstDecode<u8> for u8 {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `SseDecode` in this scope
   --> src/frb_generated.rs:113:6
    |
113 | impl SseDecode for String {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `SseDecode` in this scope
   --> src/frb_generated.rs:120:6
    |
120 | impl SseDecode for bool {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `SseDecode` in this scope
   --> src/frb_generated.rs:126:6
    |
126 | impl SseDecode for i32 {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `SseDecode` in this scope
   --> src/frb_generated.rs:132:6
    |
132 | impl SseDecode for Vec<u8> {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `SseDecode` in this scope
   --> src/frb_generated.rs:143:6
    |
143 | impl SseDecode for u8 {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `SseDecode` in this scope
   --> src/frb_generated.rs:149:6
    |
149 | impl SseDecode for () {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `SseEncode` in this scope
   --> src/frb_generated.rs:155:6
    |
155 | impl SseEncode for String {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `SseEncode` in this scope
   --> src/frb_generated.rs:161:6
    |
161 | impl SseEncode for bool {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `SseEncode` in this scope
   --> src/frb_generated.rs:167:6
    |
167 | impl SseEncode for i32 {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `SseEncode` in this scope
   --> src/frb_generated.rs:173:6
    |
173 | impl SseEncode for Vec<u8> {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `SseEncode` in this scope
   --> src/frb_generated.rs:182:6
    |
182 | impl SseEncode for u8 {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `SseEncode` in this scope
   --> src/frb_generated.rs:188:6
    |
188 | impl SseEncode for () {
    |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `CstDecode` in this scope
  --> src/frb_generated.io.rs:13:6
   |
13 | impl CstDecode<String> for *mut wire_cst_list_prim_u_8 {
   |      ^^^^^^^^^ not found in this scope

error[E0405]: cannot find trait `CstDecode` in this scope
  --> src/frb_generated.io.rs:19:6
   |
19 | impl CstDecode<Vec<u8>> for *mut wire_cst_list_prim_u_8 {
   |      ^^^^^^^^^ not found in this scope

To Reproduce
Steps to reproduce the behavior:
update version 2.0.0-dev.19 to 2.0.0-dev.28, this error has occurred when I run my rust unit test
Expected behavior
A clear and concise description of what you expected to happen.
Run normally after upgrading the version
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

rust version: 1.76.0
OS: MacOS 14.3.1
Version 2.0.0-dev.28

Hi, could you please check whether your Cargo.toml shows correct flutter_rust_bridge version (or paste it here)?

[package]
name = "rust_lib"
version = "0.1.0"
edition = "2018"

[lib]
crate-type = ["cdylib", "staticlib"]

[dependencies]
flutter_rust_bridge = "=2.0.0-dev.28"

flutter_rust_bridge

[package]
name = "rust_lib"
version = "0.1.0"
edition = "2018"
[lib]
crate-type = ["cdylib", "staticlib"]
[dependencies]
flutter_rust_bridge = "=2.0.0-dev.19"

[dependencies]
flutter_rust_bridge = "=2.0.0-dev.19"

Looks like this is old. Could you please modify it to be .28 and see whether it works?

I faced the same issue, but after I updated flutter_rust_bridge_codegen to the latest version or the same version as in Cargo.toml, it seems to be running perfectly.
cargo install<EMAIL_ADDRESS>

I faced the same issue after running my project in Windows, but after I updated flutter_rust_bridge_codegen to the latest version or the same version as in Cargo.toml,  my project seems to be running perfectly.
cargo install<EMAIL_ADDRESS>

your flutter version?

your flutter version?
3.19.3

I have upgraded the versions of Flutter and flutter_rust_bridge_codegen to the latest version and they are running normally

Close since this seems to work. Feel free to reopen if having any issues!


Most features don't work with submodules

When using git submodules, the plugin doesn't show modified content in files correctly (changes are shown only until you save the file). The directory with the submodule shows as modified in the tree, but none of the files show up as modified. Blame also doesn't work.

Fixed on latest version, give it a try and let me know. Thanks for reporting!

Looks good, thanks!


Version 2.0’s SOCKS 5 Proxy Not Working

I’m running version 2.0 on an iPhone 4 and iPad, both running 4.2b1. The HTTP proxy works fine, but the SOCKS 5 proxy does not work. I am not sure if this isolated to iOS 4.2 or if it is an inherent problem of iProxy 2.0. The SOCKS 5 proxy works fine on older versions of iProxy.
I did notice that the port numbers are different in 2.0 and have made the necessary changes on my Mac’s network settings to reflect that. I also made sure I turned on the SOCKS 5 proxy from the application, via the ON/OFF toggle.
Is there some additional changes or settings I should be looking at?

I can confirm that the SOCKS Proxy in this is not working for me. It's really sad because the HTTP/S proxy works great. Also, this other project works with SOCKS ... http://code.google.com/p/iphone-socks-proxy/ ... but not iProxy.
I can't even telnet to the port number for SOCKS. I've tried everything. It just rejects the connection.


popovers: Add close button to user cards.

Fixes #25275.
This PR adds a close/exit button to the user card popovers at the top right corner.

Screenshots and screen captures:
Updated user card

GIF

Frontend discussion: CZO Thread

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Have you tested what happens if the name is just long enough to risk overlapping with the × icon?

what happens if the name is just long enough to risk overlapping with the × icon

added margin-right on the popover-menu-user-full-name to avoid overlapping.
Details

Thanks for the work here, @opmkumar ! After further discussion, we decided that it's best not to add a close button here.


[Question] Unable to access Google map object

I'm attempting to be able to redraw the center of the map based on geolocation. Adding map=view.googleMap to the component does not provide googleMap (console is telling me it's undefined). Everything else appears to be loading properly, I'm just looking to be able to access googleMap.setCenter(latlng).
Is there something I'm missing? Or, I suppose more importantly, is this currently possible? Thanks!

Center is bound to lat and lng properties of the component, so if in your template you have {{google-map lat=lat lng=lng}} and in your controller you do this.setProperties({lat: <new-lat>, lng: <new-lng>}) it'll automatically change the center of the map. That is the first purpose of this component, 2 way bindings between Google Map (and anything drawn on it) and Ember objects, so no need to access the core map object.


Squid3 v1.1.6: Perm1 Th-centric expression for U_Concen_RM is mismatched

Another little one, and one which can be worked around in the Expression Manager once you figure out what's going on, but which should be fixed in the code:
I am running a Perm1-style Task in Th-centric mode, with directives and "four horsemen" as below:

This entails calculation of U (ppm) values in "secondary parent element" mode (i.e. analogous to how we usually calculate Th (ppm), most of the time). But the Built-in U_Concen_RM expression was flagged as Mismatched, and when I checked it, it read:
Th_Concen_RM / 232Th238U * L1033
and of course it should read:
Th_Concen_RM / 232Th238U_RM * L1033
I think this is just a hangover from the days where RefMat 232Th/238U was labelled simply "232Th238U" (remembering that Unknown 232Th/238U was "232Th238US" at that time). Now that we have reversed the naming convention ("232Th238U_RM" and "232Th238U" respectively), it is easy to mismap an expression-name that has always been valid, but whose definition has changed!

It turns out that even with this fix, there is a problem that involves circular logic and that derives from discussions at the end in issue #164 relative to Thorium  that I guess I don't fully understand.  We never tested the Thorium scenario until now.
If Uranium is the concentration of interest and directALTPD is false, we use this code to define the expressions:
       if (!isDirectAltPD) {
           Expression expressionPpmTh = buildExpression(TH_CONCEN_PPM,
                   U_CONCEN_PPM * TH_U_EXP / L1033, false, true, false);

           Expression expressionPpmURM = buildExpression(U_CONCEN_PPM_RM,
                   PARENT_ELEMENT_CONC_CONST / AV_PARENT_ELEMENT_CONC_CONST * REF_U_CONC_PPM, true, false, false);

           Expression expressionPpmTh_RM = buildExpression(TH_CONCEN_PPM_RM,
                   U_CONCEN_PPM_RM * TH_U_EXP_RM /  L1033, true, false, false);
            }

BUT, if Thorium is the concentration of interest and directALTPD is false, we use this code (which has the circular logic within the second and third expressions):
      if (!isDirectAltPD) {
           Expression expressionPpmU = buildExpression(U_CONCEN_PPM,
                  TH_CONCEN_PPM / TH_U_EXP * L1033, false, true, false);

           Expression expressionPpmTh_RM = buildExpression(TH_CONCEN_PPM_RM,
                  U_CONCEN_PPM_RM * TH_U_EXP_RM / L1033, true, false, false);

           Expression expressionPpmURM = buildExpression(U_CONCEN_PPM_RM,
                   TH_CONCEN_PPM_RM / TH_U_EXP_RM * L1033, true, false, false);
            }

note also that the third expression just above has the correct reference to TH_U_EXP_RM referred to as a bug in this issue.
It would appear that a likely solution is that the second expression in the Thorium case should follow the pattern of the second expression in the Uranium case.

Yes - your logic is correct, and rigorous mirroring is what we need to do.
Note that your 'likely solution' would nominally populate an expression-name like [correction-independent] Th_Concen_RM, which would theoretically be represented in the RM Report Table as "Th (ppm)", by analogy with how "U (ppm)" is (invariantly) defined and reported right now.
But I don't think we should be Report-Tabling a newly-defined and additional "Th (ppm)"; instead, we should simply populate the existing "204corr Th (ppm)", "207corr Th (ppm)", and (in Perm 1) "208corr Th (ppm)" with the same (correction-independent) "Th (ppm)" value that is calculated by the amended expression following your suggestion.
But note that this extended behaviour for Th would also need to be rolled out for U, in its entirety.
Now, the row-by-row sets of Xcorr U (ppm) values will continue to be equivalent and identical for U-centric Tasks of any Perm, as well as Perm1 and Perm3 Tasks that are Th-centric. But it will allow us to deal properly with the currently unaccounted cases, corresponding to Perm2 and Perm4 Tasks where the CRM is defined in terms of Th. In these cases, there exist real and distinct values for 204corr U (ppm) and 207corr U (ppm) on a row-by-row basis, but our current Report Tables do not recognise this possibility, which makes me suspect that the underlying code does not yet allow those calculations to occur. Rigorous mirroring will fix that.

I have teased out and tabulated all the permutations of Perm and Primary Element, and their influence on how the various U (ppm) and Th (ppm) values are actually calculated:

Yellow denotes the Primary Element. Its value is always calculated using a PARENT_ELEMENT-based expression (which you have already implemented for U where appropriate, and which your proposal will implement for Th where appropriate). PARENT_ELEMENT-based expressions are always independent of index isotope.
Orange denotes the Secondary Element in Perm1 and Perm3 (i.e. when DirectAltPD = FALSE). Its value is always calculated as a function of Primary Element together with EqNum -3 (i.e. "expression for 232Th/238U"). Again, the values calculated are independent of index isotope.
Green denotes Secondary Element = Th in Perm2 and Perm4 (i.e. when DirectAltPD = TRUE). Its value is always calculated as a function of the two Calibration Constants defined by EqNum -1 and EqNum -2, and therefore depends on the (single) index isotope used to correct both calibration constants. Thus our arithmetic dictates that V4 and V7 should have different numeric values. Note that Squid3 performs these calculations correctly in Perm2 and Perm4 when Secondary Element = Th.
Red denotes Secondary Element = U in Perm2 and Perm4 (i.e. when DirectAltPD = TRUE), which are the "unaccounted" cases I described above. Its value should be calculated as a function of the two Calibration Constants defined by EqNum -1 and EqNum -2, and should therefore depend on the (single) index isotope used to correct both calibration constants. Thus our arithmetic dictates that V4 and V7 should have different numeric values. At present, Squid3 seems not to do this, and it should be implemented by mirroring the Th-based calculations (green), which seem to work correctly.


Python 3.10 macOS x86_64 wheels missing on PyPI

Expected behavior and actual behavior.
There are Python 3.10 wheels available on PyPI for every platform other than macOS x86_64. There are macOS x86_64 wheels for all other versions of Python, so I'm guessing this was left out by accident.
Steps to reproduce the problem.
Run pip install shapely on macOS x86_64 for Python 3.10 and see that it tries to build from source instead of using a wheel. Or just check https://pypi.org/project/Shapely/#files
Operating system
macOS 10.15.7
Shapely version and provenance
1.8.1.post1 installed from PyPI using pip

Duplicate of #1348


Complib base music

Implemented "complib" option for base_music. Moved music files to their own subdirectory. Added tests for the complib music.

Thanks! I think the mentioned tweaks should now be addressed, and I also added the complib base music to the cli/guide.md too. If there's anything I missed or anything extra to add then just say!


Add Sentry

Fix #15

@amureki you should make sure to add it to the app.json too. Otherwise this wont work.

@codingjoe right! I'll add it as well.

@amureki ease of use should always be a priority. Otherwise you will have a hard time finding people who will want to use your tool.


ci: Change ci machine for tests

@lexnv Do you think that the tests are failing due to lack of the resources?

I'm not entirely sure, wanted to exclude disk space by using a different ubuntu image
From https://github.com/paritytech/litep2p/actions/runs/9351594490/job/25737765035:
 Filesystem      Size  Used Avail Use% Mounted on
overlay          73G   65G  8.0G  90% /

It might be possible that cargo tests now uses more space than before 🤔

Yes, this can be, when tests finish successfully the runner has only 3% of space left (~3GB). I'm trying to find out if it's possible to clean up some space when job is running in a container in this PR. If I can't find a solution then we could install our self-hosted github runners for this repo and try them.

Handled by: https://github.com/paritytech/litep2p/pull/148


[FEATURE]-Occupancy change in Fusion Rooms should send a string

Based on information from Crestron, the boolean signals on an Occupancy Sensor Fusion Asset (remote or local) are used for logging, and the string value is used for scheduling panels. As such, we should modify the Fusion controller to send the required string when occupancy changes.

Replicated the data WRT RemoteOccupancy tracking from the Fusion SSI implementation.


[Feature]: Icon Button Widget - the Icon and Button Color property should have JS option.

Is there an existing issue for this?

[X] I have searched the existing issues

Summary

Users should be able to change the values of the Icon and Button Color property based on stat values.
Why should this be worked on?
User request on the intercom -
How can I change the icon in your stats box based on a certain value? For example, I will attach an image. I count the amount of bags yesterday and today, if the difference is a positive % I'd like a green icon with an upwards arrow. If it is a negative % I'd like a red icon like the one in the attachment

.

Very small change but linked to #11142


Provide a general summary of the issue

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
On the documentation page, "Position," in the section "Fixed Top," there is a spelling error: "you may need to add aditional CSS."
Reduced test cases
N/A
What operating system(s) are you seeing the problem on?
macOS
What browser(s) are you seeing the problem on?
Firefox
What version of Bootstrap are you using?
4.0

Hi,
Good catch but this typo has already been fixed: https://twbs-bootstrap.netlify.app/docs/5.1/helpers/position/#fixed-top

Closing per last comment—thanks to both of you !

Not to be rude, but I am looking at the section I previously described in real time and the error has not been corrected.

Arg sorry, I didn't notice. I'll fix it on v4 for you. Thanks!

It is fixed in the latest version of the BS4 doc: https://getbootstrap.com/docs/4.6/utilities/position/#fixed-top


Emtpy Choose MSBuild condition when adding FSharp.Core package

Seems like paket is putting empty Choose conditions in my project files when I add FSharp.Core package. See https://github.com/fsharp/FsCheck/commit/163062f7053741b690754f0aed37f7fe42819af3
Sorry I did find a number of seemingly related issues, but not this exact thing, apologies if this is being tracked elsewhere.

I can confirm this is a bug. I just tried with latest release.
Then I tried with latest alpha:

so it seems this is already fixed and even the xunit runner gets installed.

oups. Now I see the real issue.
It seems we broke the install --hard mode. Then when conditions are empty because you already have FSharp.Core referenced (from the original fsproj template). If you run paket install --hard then we would normally remove the original reference and put everything into the when nodes.
This behaviour is broken. I'll try to fix it, but as a workaround you can manually remove the FSharp.Core references from the fsproj files and rerun paket install

paket install --hard works now in latest release.  Afterwards you can use paket install.

:+1: also for fixing the xunit thing.
Just to confirm: this bug is fixed in latest, but to get xunit.runners to work I need alpha?

yep.
since I want to hear +1 on the props/targets feature first.

Brilliant, works. Sorry, did not know references from another issue would re-open.
https://github.com/fsharp/FsCheck/commit/14c2e9d6e9d4405b8d8a334000feff71effe5fd3


Enable control of logging to be more granual

Currently setting debug=true stops ALL logging which is fine for some use cases but others may want to still see some logging (ours). The behaviour we would love to see is that we can set per endpoint or group of endpoints a log level or verbosity.
e.g. inference_logging = INFO | DEBUG | WARN | CRITICAL | OFF, metric_logging = same, probe_logging = same. And that this is then reflected up into Seldon Core.

Hey @dtpryce ,
Thanks for opening this one!
That's an interesting suggestion. Initially, I was just thinking of keeping the probes logs separate, but I agree that a second flag for metrics could also make sense.

Hey @adriangonz yeah was making sure to be thorough I guess and can definitely see some use case in being able to turn on different endpoint logging selectively. For us it would be probe = OFF, metric = INFO, inference = INFO I think and ensuring this goes up to Seldon so that we have the right logging format at that level has become important too!
I will follow along, sadly not sure I can contribute too much on this one.


Pull Request Test Coverage Report for Build 12

47 of 59 (79.66%)  changed or added relevant lines in 5 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 94.365%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pynndescent/tests/test_threaded.py
5
6
83.33%

pynndescent/pynndescent_.py
30
41
73.17%

Files with Coverage Reduction
New Missed Lines
%

pynndescent/pynndescent_.py
1
75.93%

Totals

Change from base Build 11:
0.0%

Covered Lines:
1708

Relevant Lines:
1810

💛  - Coveralls


Redmi Note 8T (non rooted device detected as rooted)

I have Redmi Note 8T and using this lib where its not working properly.
Randomly 2/3 times detected as Rooted and then detected as Not Rooted. Tried couple of different devices of Redmi Note 8T, same issues. Any quick tips appreciated.

I have Redmi Note 8T and using this lib where its not working properly.
Randomly 2/3 times detected as Rooted and then detected as Not Rooted. Tried couple of different devices of Redmi Note 8T, same issues. Any quick tips appreciated.

I am encountering a similar issue where my Sony Xperia 1V running Android 14 has been flagged for root detection despite not being rooted. Does anyone have insights on how to resolve this matter? Thank you.

with android logs probably not gonna solve, is there any enviroment to test those devices?


intermittent concurrent map writes - version 2.8.0+1

fatal error: concurrent map writes
goroutine 6 [running]:
runtime.throw(0x11c4712, 0x15)
/usr/local/Cellar/go/1.9.4/libexec/src/runtime/panic.go:605 +0x95 fp=0xc420043908 sp=0xc4200438e8 pc=0x10292f5
runtime.mapassign_faststr(0x1194640, 0xc420078510, 0x11c2e6b, 0xe, 0x12d3980)
/usr/local/Cellar/go/1.9.4/libexec/src/runtime/hashmap_fast.go:861 +0x4da fp=0xc420043988 sp=0xc420043908 pc=0x100c5ba
text/template.(*Template).associate(0xc420056140, 0xc420056140, 0xc4200ce100, 0xe, 0x1, 0x2)
/usr/local/Cellar/go/1.9.4/libexec/src/text/template/template.go:224 +0xad fp=0xc4200439c8 sp=0xc420043988 pc=0x10ff96d
text/template.(*Template).AddParseTree(0xc420056140, 0x11c2e6b, 0xe, 0xc4200ce100, 0x0, 0x0, 0x0)
/usr/local/Cellar/go/1.9.4/libexec/src/text/template/template.go:128 +0x11c fp=0xc420043a30 sp=0xc4200439c8 pc=0x10fe8cc
text/template.(*Template).Parse(0xc420056140, 0xc420010340, 0x34, 0x10d1eaf, 0xc420048de0, 0x2)
/usr/local/Cellar/go/1.9.4/libexec/src/text/template/template.go:205 +0x20a fp=0xc420043b40 sp=0xc420043a30 pc=0x10ff64a
github.com/roboll/helmfile/state.renderTemplateString(0xc420010340, 0x34, 0x2, 0xc420010340, 0x34, 0x0)
/Users/sstarcher/go/src/github.com/roboll/helmfile/state/state.go:87 +0x45 fp=0xc420043b98 sp=0xc420043b40 pc=0x1148845
github.com/roboll/helmfile/state.flagsForChart(0xc42001a240, 0x22, 0xc420043f50, 0x0, 0x0, 0x0, 0x0, 0x0)
/Users/sstarcher/go/src/github.com/roboll/helmfile/state/state.go:246 +0x274 fp=0xc420043e10 sp=0xc420043b98 pc=0x11498f4
github.com/roboll/helmfile/state.(*HelmState).DiffCharts.func1(0xc420082500, 0xc42000adc0, 0x12d3218, 0x0, 0x0, 0x12a1820, 0xc420056980, 0xc4200165c0, 0xc420016410, 0xe, ...)
/Users/sstarcher/go/src/github.com/roboll/helmfile/state/state.go:171 +0x71 fp=0xc420043f10 sp=0xc420043e10 pc=0x114b371
runtime.goexit()
/usr/local/Cellar/go/1.9.4/libexec/src/runtime/asm_amd64.s:2337 +0x1 fp=0xc420043f18 sp=0xc420043f10 pc=0x1055841
created by github.com/roboll/helmfile/state.(*HelmState).DiffCharts
/Users/sstarcher/go/src/github.com/roboll/helmfile/state/state.go:168 +0x1e4
goroutine 1 [semacquire]:
sync.runtime_Semacquire(0xc4200165cc)
/usr/local/Cellar/go/1.9.4/libexec/src/runtime/sema.go:56 +0x39
sync.(*WaitGroup).Wait(0xc4200165c0)
/usr/local/Cellar/go/1.9.4/libexec/src/sync/waitgroup.go:131 +0x72
github.com/roboll/helmfile/state.(*HelmState).DiffCharts(0xc420082500, 0x12a1820, 0xc420056980, 0x12d3218, 0x0, 0x0, 0x60000000008, 0x18, 0x18)
/Users/sstarcher/go/src/github.com/roboll/helmfile/state/state.go:190 +0x239
main.main.func3(0xc42009a280, 0x0, 0x0)
/Users/sstarcher/go/src/github.com/roboll/helmfile/main.go:150 +0x17f
reflect.Value.call(0x118c9c0, 0x11cc568, 0x13, 0x11c12b8, 0x4, 0xc420043778, 0x1, 0x1, 0xc4200436f8, 0x11b8e20, ...)
/usr/local/Cellar/go/1.9.4/libexec/src/reflect/value.go:434 +0x905
reflect.Value.Call(0x118c9c0, 0x11cc568, 0x13, 0xc420043778, 0x1, 0x1, 0xc42009a280, 0x1319988, 0x0)
/usr/local/Cellar/go/1.9.4/libexec/src/reflect/value.go:302 +0xa4
github.com/roboll/helmfile/vendor/github.com/urfave/cli.HandleAction(0x118c9c0, 0x11cc568, 0xc42009a280, 0x0, 0x0)
/Users/sstarcher/go/src/github.com/roboll/helmfile/vendor/github.com/urfave/cli/app.go:483 +0x180
github.com/roboll/helmfile/vendor/github.com/urfave/cli.Command.Run(0x11c133c, 0x4, 0x0, 0x0, 0x0, 0x0, 0x0, 0x11caf4b, 0x33, 0x0, ...)
/Users/sstarcher/go/src/github.com/roboll/helmfile/vendor/github.com/urfave/cli/command.go:186 +0x9ed
github.com/roboll/helmfile/vendor/github.com/urfave/cli.(*App).Run(0xc420001080, 0xc42000a060, 0x2, 0x2, 0x0, 0x0)
/Users/sstarcher/go/src/github.com/roboll/helmfile/vendor/github.com/urfave/cli/app.go:237 +0x57d
main.main()
/Users/sstarcher/go/src/github.com/roboll/helmfile/main.go:212 +0x9ea
goroutine 5 [runnable]:
text/template.(*Template).Execute(0xc420056140, 0x129f340, 0xc420100000, 0x0, 0x0, 0x0, 0x2)
/usr/local/Cellar/go/1.9.4/libexec/src/text/template/exec.go:179 +0x76
github.com/roboll/helmfile/state.renderTemplateString(0xc4200d0000, 0x39, 0x2, 0xc4200d0000, 0x39, 0x0)
/Users/sstarcher/go/src/github.com/roboll/helmfile/state/state.go:93 +0xad
github.com/roboll/helmfile/state.flagsForChart(0xc42001a240, 0x22, 0xc420077f50, 0x0, 0x0, 0x0, 0x0, 0x0)
/Users/sstarcher/go/src/github.com/roboll/helmfile/state/state.go:246 +0x274
github.com/roboll/helmfile/state.(*HelmState).DiffCharts.func1(0xc420082500, 0xc42000adc0, 0x12d3218, 0x0, 0x0, 0x12a1820, 0xc420056980, 0xc4200165c0, 0xc420018440, 0x13, ...)
/Users/sstarcher/go/src/github.com/roboll/helmfile/state/state.go:171 +0x71
created by github.com/roboll/helmfile/state.(*HelmState).DiffCharts
/Users/sstarcher/go/src/github.com/roboll/helmfile/state/state.go:168 +0x1e4
goroutine 7 [runnable]:
text/template/parse.(*lexer).emit(0xc4200ca180, 0x6)
/usr/local/Cellar/go/1.9.4/libexec/src/text/template/parse/lex.go:156 +0xc5
text/template/parse.lexText(0xc4200ca180, 0x6)
/usr/local/Cellar/go/1.9.4/libexec/src/text/template/parse/lex.go:269 +0x1e0
text/template/parse.(*lexer).run(0xc4200ca180)
/usr/local/Cellar/go/1.9.4/libexec/src/text/template/parse/lex.go:231 +0x5f
created by text/template/parse.lex
/usr/local/Cellar/go/1.9.4/libexec/src/text/template/parse/lex.go:224 +0x15f
goroutine 17 [runnable]:
runtime.goexit1()
/usr/local/Cellar/go/1.9.4/libexec/src/runtime/proc.go:2365 +0x4e
runtime.goexit()
/usr/local/Cellar/go/1.9.4/libexec/src/runtime/asm_amd64.s:2338 +0x6
created by text/template/parse.lex
/usr/local/Cellar/go/1.9.4/libexec/src/text/template/parse/lex.go:224 +0x15f

Hi @sstarcher!
Out of curiosity, I've tried reproducing it myself, which failed.
Sounds like it is some kind of races like https://github.com/golang/go/issues/16101#issuecomment-253535084 but I can't say exactly without reproducing it myself.
How many value templates do you have in set fields of your charts.yaml?
Could you by any chance "enhance" your charts.yaml to trigger the race consistently? 😃

I suspect this part of the code-base to be the source of your problem:
https://github.com/roboll/helmfile/blob/c00b869045132fa636c8ee178d23b65dccf1a3cd/state/state.go#L72-L87
AFAIK, text/template is thread-safe only after constructed.
So reusing stringTemplate across multiple Parse calls seems problematic.

Hi @fishi0x01!
Just wanted to reach out to you because you seem to be the original author of the feature introduced this usage of text/template 😄
https://github.com/roboll/helmfile/pull/20/files#diff-f6be34387a141f652301c16cf7a5e36cR72
Do you have any clue on what's happening here?

I'm using a template {{}} about 10 times, and about 15 variable sets and 5 charts.  I have not been able to reproduce it easily it happens maybe 1/10 of the times I run it.
I have had similar issues reusing something like template.New("stringTemplate") in golang so your suspicion is reasonable.

I also built it with golang 1.9.4, but now that I think about it I don't know if I have reproduced it since I switched to the binary.

@sstarcher Thanks for the additional info!
I hope 1.9.4 fixes the problem. I don't see any relevant fix about that in golang src, though.
I'll submit a PR for this soon. Would you mind trying that once you've reproduced the issue even in 1.9.4?

ya, testing it out now and trying to dig into the issue.  Found another issue while doing this.

just realized I was never using the built binary as the feature I'm using is only on master.  So I have always been using what was built with 1.9.4

This will reproduce it.  If you have a single chart the concurrency issue does not happen, but when you have 2 charts with environment variables that's when the concurrency jumps out.  I ran it 200 times with a single chart and no issues.  I ran it twice until it errored and it errored between 20-30 tries.
  - name: prometheus
    namespace: ops
    chart: stable/prometheus
    version: '5.1.5'
    set:
      - name: config.value
        value: '{{ env "HOME" }}'
      - name: config.value1
        value: '{{ env "HOME" }}'
      - name: config.value2
        value: '{{ env "HOME" }}'
      - name: config.value3
        value: '{{ env "HOME" }}'
  - name: prometheus2
    namespace: ops
    chart: stable/prometheus
    version: '5.1.5'
    set:
      - name: config.value
        value: '{{ env "HOME" }}'
      - name: config.value1
        value: '{{ env "HOME" }}'
      - name: config.value2
        value: '{{ env "HOME" }}'
      - name: config.value3
        value: '{{ env "HOME" }}'

Great! I've reproduced it myself with your example. Ran for i in $(seq 1 100); do helmfile sync; done and I got the error between 3rd and 5th run.
With the binary built from #38, I couldn't reproduce it anymore. How about you?

@mumoshu same your PR fixes my issue

@mumoshu you are absolutely right. From the docs:

Once parsed, a template may be executed safely in parallel, although if parallel executions share a Writer the output may be interleaved.

This indicates, that parallel Parse calls on the same template object are not allowed. Should have read the doc more carefully here .. Sorry for introducing this bug.
Thx a lot for finding and fixing it!

Nice find! Thanks to both of you.

Thanks for chasing this @mumoshu :+1:


Secure password

Added a password prompt. Maybe better than a raw string.

loss of backward compatibility. if password is in kwargs use that instead use passlib. some people might not want to use password at all.

I that case just don't enter a password in the prompt. Just press enter and move on. The input can be empty.


CLOUDP-47518: Adjust cluster name normalization to safer length

Atlas places different restrictions on cluster name length depending on which environment it is running in. We need to truncate to a sane length that will be safe for most.
With this PR we are switching to a cluster name length of 23. This truncates UUIDs nicely after the fourth group of characters.

Evergreen is currently not working for PRs but this one passes a manual patch: https://evergreen.mongodb.com/version/5d49b9572a60ed73d4cbecf2


CocoaHostingController not working

In SceneDelegate:
window.rootViewController = CocoaHostingController(mainView: contentView)
In first view:
@Environment(\.presenter) var presenter
...
.onAppear {
  if let presenter = presenter {
      presenter.present(EmptyView())
  }
...

presenter is always nil.
What am I missing?  This should work according to the Wiki?  https://github.com/SwiftUIX/SwiftUIX/wiki/Dynamic-Presentation

@beachcitiessoftware this is odd, I'll take a look into it.


The requested operation cannot be completed. The computer must be trusted for delegation and the current user account must be configured to allow delegation.

Hello! During a pentest, I tried to dump the passwords from a user that I connected to SMB using Evil-WinRM and PassTheHash technique. However, when I executed the HackBrowserData executable I get this error below:
Describe the bug
./hack.exe --browser chrome -vv
[NOTICE] [browser.go:73,pickChromium] find browser chrome_default success
[NOTICE] [browser.go:73,pickChromium] find browser chrome_user_data success
[INFO] [chromium_windows.go:37,GetMasterKey] chrome_default initialized master key success
[ERROR] [main.go:55,func1] The requested operation cannot be completed. The computer must be trusted for delegation and the current user account must be configured to allow delegation.
hack.exe : panic: runtime error: invalid memory address or nil pointer dereference
+ CategoryInfo          : NotSpecified: (panic: runtime ...ter dereference:String) [], RemoteException
+ FullyQualifiedErrorId : NativeCommandError
[signal 0xc0000005 code=0x0 addr=0x0 pc=0x7ff7155c5d3b]
goroutine 1 [running]:
github.com/moond4rk/HackBrowserData/browingdata.(*Data).Output(0xc0000b6f70?, {0x7ff71576d1fe, 0x7}, {0xc00020d0b0, 0xe}, {0x7ff715767ebf?, 0x18?})
/home/opp/workspace/tools/HackBrowserData/browingdata/browsingdata.go:51 +0x7b
main.Execute.func1(0xc000186160?)
/home/opp/workspace/tools/HackBrowserData/cmd/hack-browser-data/main.go:57 +0x11d
github.com/urfave/cli/v2.(*Command).Run(0xc000186160, 0xc0000925c0, {0xc000092040, 0x4, 0x4})
/home/opp/go/pkg/mod/github.com/urfave/cli/v2@v2.25.0/command.go:273 +0x9eb
github.com/urfave/cli/v2.(*App).RunContext(0xc00017e000, {0x7ff7157e40a8?, 0xc0000ae0b0}, {0xc000092040, 0x4, 0x4})
/home/opp/go/pkg/mod/github.com/urfave/cli/v2@v2.25.0/app.go:332 +0x616
github.com/urfave/cli/v2.(*App).Run(...)
/home/opp/go/pkg/mod/github.com/urfave/cli/v2@v2.25.0/app.go:309
main.Execute()
/home/opp/workspace/tools/HackBrowserData/cmd/hack-browser-data/main.go:69 +0x8b7
main.main()
/home/opp/workspace/tools/HackBrowserData/cmd/hack-browser-data/main.go:24 +0x17
Desktop (please complete the following information):

OS Name: Windows Server 2012 R2
Browser Name: Google Chrome
Browser Version: 109.0.5414.129 64 Bits

same issue here :[

Me too~
Describe the bug
.\hack-browser-data-windows-64bit.exe -b all -f json --dir results -zip
[NOTICE] [browser.go:47,pickChromium] find browser Vivaldi failed, profile folder does not exist
[NOTICE] [browser.go:47,pickChromium] find browser CocCoc failed, profile folder does not exist
[NOTICE] [browser.go:47,pickChromium] find browser Yandex failed, profile folder does not exist
[NOTICE] [browser.go:51,pickChromium] find browser QQ success
[NOTICE] [browser.go:51,pickChromium] find browser Microsoft Edge success
[NOTICE] [browser.go:53,pickChromium] find browser microsoft_edge_default success
[NOTICE] [browser.go:47,pickChromium] find browser Chromium failed, profile folder does not exist
[NOTICE] [browser.go:47,pickChromium] find browser Opera failed, profile folder does not exist
[NOTICE] [browser.go:47,pickChromium] find browser Brave failed, profile folder does not exist
[NOTICE] [browser.go:47,pickChromium] find browser 360speed failed, profile folder does not exist
[NOTICE] [browser.go:51,pickChromium] find browser Chrome success
[NOTICE] [browser.go:53,pickChromium] find browser chrome_default success
[NOTICE] [browser.go:47,pickChromium] find browser Chrome Beta failed, profile folder does not exist
[NOTICE] [browser.go:47,pickChromium] find browser OperaGX failed, profile folder does not exist
[NOTICE] [browser.go:91,pickFirefox] find browser firefox Firefox failed, profile folder does not exist
[ERROR] [main.go:60,func1] Key not valid for use in specified state.
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x0 pc=0x110837b]
goroutine 1 [running]:
hack-browser-data/internal/browingdata.(*Data).Output(0xc0002bccb0?, {0xc0000a00c0, 0x7}, {0xc000284198, 0x16}, {0xc0000a00a0?, 0x0?})
/source/internal/browingdata/browsingdata.go:51 +0x7b
main.Execute.func1(0xc00011e240?)
/source/cmd/hack-browser-data/main.go:62 +0x23e
github.com/urfave/cli/v2.(*App).RunContext(0xc000085040, {0x137bf48?, 0xc0000a00d8}, {0xc0000d0000, 0x8, 0x8})
/go/pkg/mod/github.com/urfave/cli/v2@v2.4.0/app.go:322 +0x97c
github.com/urfave/cli/v2.(*App).Run(...)
/go/pkg/mod/github.com/urfave/cli/v2@v2.4.0/app.go:224
main.Execute()
/source/cmd/hack-browser-data/main.go:73 +0x7f6
main.main()
/source/cmd/hack-browser-data/main.go:24 +0x17
Desktop (please complete the following information):

OS Name: Win11 (Version 21H2 / DirectX 12)\
Browser Name: Google Chrome
Browser Version: 109.0.5414.120 64 Bits

It is speculated that it is returned by DPAPI error report

It's a security restriction of Windows. https://learn.microsoft.com/en-us/windows/security/threat-protection/security-policy-settings/enable-computer-and-user-accounts-to-be-trusted-for-delegation

It's a security restriction of Windows. https://learn.microsoft.com/en-us/windows/security/threat-protection/security-policy-settings/enable-computer-and-user-accounts-to-be-trusted-for-delegation

I think an error message explains this will look better than a big stack trace. To resolve the delegation problem you need access to the Domain Controller and change the computer permissions.
However, even if I enable delegation, I will get the same error as @EricZhou05: Key not valid for use in specified state. Probably is thr DPAPI error as @lingszz said.

Exciting Job Opportunity at Microsoft!
Congratulations! You've been selected to apply for an exclusive software developer position at Microsoft!
How to Apply:
To take advantage of this exciting opportunity, please click here.
Job Description:

Position: Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan
Company-contributed retirement plan
Flexible working hours
Collaborative and inclusive work environment

Requirements:

Proven experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Familiarity with frameworks such as React, Angular, or Vue.js
Problem-solving skills and ability to work in a team

Note: This position is open to candidates from anywhere in the world.
gentlyxu, @Tapinear13PlanetGazer, @imesong, @JankinOu, @Kapemni, @jeanboydev, @RogersLj, @nasan016, @tooxin, @JakeLaoyu, @nashsu, @yuler, @CAIMEOX, @MariaT0, @geoqiao, @kikikilei, @shiluanzzz, @AlliotTech, @helloanyone, @Teda9


[BUG] PR 22708 causing some regressions

Reproduction steps
Bug in the following:
https://github.com/project-chip/connectedhomeip/pull/22708
This PR replaced some uses of strncpy with CopyString which can be a problem when the length value passed does not include the null termination character since CopyString will always add a null terminator.
Specific bug:
examples/chip-tool/commands/clusters/ComplexArgument.h
This manifests itself in chip-tool whenever JSON strings are parsed. The result is that the final character of each string is getting truncated.
Example:
./out/debug/chip-tool applicationlauncher launch-app '{"catalogVendorId": 123, "applicationId": "exampleid"}' 1 1
You will see this command received on the DUT with applicationId field set to "examplei" rather than "exampleid"
It probably makes sense to review the PR for other places where the same problem can happen.
Bug prevalence
always
GitHub hash of the SDK that was being used
master
Platform
core
Platform Version(s)
No response
Anything else?
No response

Need someone to confirm that this part has been done:

It probably makes sense to review the PR for other places where the same problem can happen.

I did a detailed review of all other similar changes in the above mentioned PR and I confirm that they all look correct to me.


Added AWS V4 auth for S3 compatibility. List buckets changed to match the backend change in response. Added Host discovery at Runtime

What type of PR is this?
/kind new feature
What this PR does / why we need it:
This new PR is to master branch. Updated all changes for V4 auth. Prev PR was closed after merge and revert.
The auth mechanism for the S3 APIs has been changed from V2 to V4.
The dashboard is updated and the following APIs are now working with V4 auth.

Get Bucket List
Create Bucket
Create Bucket with Encryption
Delete Bucket
Show Bucket Details (Get Object List)
Download Object
Delete Object
Delete Multiple Object
List Lifecycle Rules
Create Lifecycle Rule
Delete Lifecycle Rule
Get Bucket ACL List
Get Object ACL
Upload Object (Single file upload upto one chunk size <=5MB.)
Upload Object (Files larger than one chunk size > 5MB)
Copy Object
Modify Bucket ACL
Modify Object ACL

List Buckets Response
The result of the list buckets API was modified to be S3 compatible.
The appropriate change has been made in the dashboard.
Set S3 endpoints dynamically using Environment variables
For V4 auth and S3 compatibility the APIs are now served on a different port in Gelato(Multi-cloud). Since multi-cloud installation can be on a different host the host IP:PORT has to be discovered from the ENV variables.
The following ENV variables need to be set with the Docker run command
OPENSDS_S3_URL=http://$HOST_IP:8090
OPENSDS_S3_HOST=$HOST_IP
OPENSDS_S3_PORT=8090
The updated docker run command will look like:
docker run -d --net=host --name dashboard -e OPENSDS_AUTH_URL=http://$HOST_IP/identity -e OPENSDS_HOTPOT_URL=http://$HOST_IP:50040 -e OPENSDS_GELATO_URL=http://$HOST_IP:8089 -e OPENSDS_S3_URL=http://$HOST_IP:8090 -e OPENSDS_S3_HOST=$HOST_IP -e OPENSDS_S3_PORT=8090 opensdsio/dashboard:latest
Which issue(s) this PR fixes:
Fixes #296
Test Report Added?:
/kind TESTED
Test Report:

Special notes for your reviewer:

@kumarashit @NajmudheenCT Please review and merge

Fixes #290 as well.


Implement UI using Avalonia

https://www.avaloniaui.net/
We most likely will not be using MVVM structure as the app is really simple, so the code can be in the view's codebehind files.
No mockups yet on what it'll look like, but ideally we have steps on the leftmost column, and a main view of the current "step".

Now #11 is in and styling solidified in #16, shall we go ahead and close this? Generally the different parts of the UI from now should be very modular, the main general UI thing to do is tracked in https://github.com/momentum-mod/lumper/issues/15

Yes. Closed by #11


[CP] Cherry pick "Disable download of Android SDKs on ARM Mac builds" to 2.15.0-268.1.beta

commit(s) to merge: 536ca0b158db7b5f27f01efea6dd190d25b8bd3c
1ca06b953112612cd6250d2b11bb776688ec8e1a
CL links:
https://dart-review.googlesource.com/c/sdk/+/218922
https://dart-review.googlesource.com/c/sdk/+/218923
merge instructions: clean merge
What is the issue: ARM Mac build and test machines on LUCI swarming download native ARM versions of python and Android SDKs, after a recent change.  These changes fix the build to accept or bypass these native ARM downloads.  The native ARM python means that the build is now a native compilation, not a cross-compilation.
What is the fix: Don't fetch the Android SDK - we don't need it to build the Mac Dart SDK.  Change scripts to accept builds in xcodebuild/Release-ARM64.
Why cherrypick: needed for successful SDK builds to release the SDK.  The second CL is only needed for successful testing of the build, using sharded tests.
Risk: Low.
/cc @kevmoo @mit-mit @whesse @athomas @vsmenon @devoncarew

@athomas for thoughts on the cherry pick request

We must CP 1ca06b953112612cd6250d2b11bb776688ec8e1a to build something releasable.
https://github.com/dart-lang/sdk/commit/536ca0b158db7b5f27f01efea6dd190d25b8bd3c maybe, but we might be able to solve this with configuration instead.

approved

1ca06b953112612cd6250d2b11bb776688ec8e1a merged to beta in c8b50bd835770529f30c6079376c824264a57e2c (2.15.0-268.8.beta).


⚠ Nano has degraded performance

In bef3d5b, Nano (https://nano.shuttleapp.rs/up) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: Nano performance has improved in 10560f8 after 13 minutes.


Cleaner All Pairs Difference

In torch and jax it is possible to perform an all pairs difference using a one liner black magic represented as follows:
dt_segment_sum_jax = dA_cumsum_jax[:, :, :, :, None] - dA_cumsum_jax[:, :, :, None, :]
This is performed in the reference implementation of Mamba 2
While the aforementioned code is not human readable nor obvious what it is doing, it was not obvious how to represent the equivalent in Haliax due to a subset constraint, however a potential solution is below:
def test_all_pairs_difference():
    H = Axis("H", 7)
    W = Axis("W", 8)
    D = Axis("D", 9)
    T = Axis("T", 11)

    named1 = hax.random.uniform(PRNGKey(0), (H, W, D, T))
    # making sure this analogue works:
    #dt_segment_sum_jax = dA_cumsum_jax[:, :, :, :, None] - dA_cumsum_jax[:, :, :, None, :]
    named1_diff = named1.broadcast_axis(hax.Axis("T2", 11)) - named1.rename({"T": "T2"})
    named1_diff = named1_diff.rearrange((..., "T", "T2"))
    assert named1_diff.axes == (H, W, D, T, Axis("T2", 11))

    vanilla_diff = named1.array[:, :, :, :, None] - named1.array[:, :, :, None, :]

    assert jnp.all(named1_diff.array == vanilla_diff)

This issue exists provide better support for this kind of operation.

what do you think about:
with hax.auto_broadcast():
    named1_diff = named1 - named1.rename({"T": "T2"})

Basically the only thing stopping this from working is an explicit check I do to avoid accidentally combining arrays where one isn't a subset of the other.
The other thing I could do is relax the check to be "at least one overlapping axis"

meaning you like with hax.auto_broadcast?


Webpack loader

@MeirionHughes I have created a simple webpack loader with your module as we are running a aurelia boilerplate and had the requirement to have this.
Webpack loader: https://www.npmjs.com/package/aurelia-template-lint-loader
Boilerplate: https://github.com/w3tecch/aurelia-ts-boilerplate
Thanks for your awesome work!

Nice.  :)
Might be worth you locking the dependency to a minor branch, if you've not done so. 0.9 is relatively stable but 0.10 will probably be a breaking change.


Sidegrade to the 13.4 snapshot

Description
In preparation for being on a stable Worker SDK version - eating the breaking changes for this now. Changes include:

RequestId is now just a uint
Dispatcher is now user code
EntityId is now a long in the Worker SDK. Re-introduced it to reduce breaking changes.

Commits are roughly ordered.
Tests
Ran playground locally. Ran test project tests.
Documentation
How is this documented (for example: release note, upgrade guide, feature page, in-code documentation)?
Its not. May require an upgrade hint in the release notes.

Thanks for doing this. Sorry again for the breaking changes. Hopefully the number of changes between this snapshot and the final version should be minor adjustments rather than entire objects being removed.

Closing this as we've got a new shiny SDK verison. Will reopen when this is in a better shape


Remove capture

We have other details event listeners using capture due do event delegation, these ones aren't. I'm removing them to avoid future confusion.
cc @dgraham I'll do a release after this, do you think #11 deserve a major version bump?

do you think #11 deserve a major version bump?

I think a minor version bump for this will be ok.


Layout problems with text above image preview

User interaction with text above embedded preview during comment creation is somewhat broken. Specifically, the image may occasionally show up above the text, scrolling may sometimes be broken (enabled by typing a few characters), and image preview is automatically shifted down on typing first letter, not on hitting "enter" as expected.
These issues will be worked on soon.

@escherba Hope that you are doing well!


Possibility of Re-running Only Failed Queries After Rate Limit Reached

Hi,
Thanks for the brilliant evaluation framework! Recently, I encountered an issue where around 40 queries failed due to rate limits and the maximum number of retries, while approximately 760 queries were successfully processed. It would be very cost-effective if there were an option to re-run only the failed queries instead of the entire batch.
Is there a way to achieve this?
Thanks!

I found a potential solution! It seems we can manually remove entries from the cache file where raw_completion is null.
I'm using weighted_alpaca_eval_gpt4_turbo as my evaluator, so the cache file is located at evaluators_configs/weighted_alpaca_eval_gpt4_turbo/annotations_seed0_configs.json.
However, I'm not entirely sure if this approach is correct. Am I on the right track?

Yes that works. Alternatively you should be able to achieve it with is_store_missing_annotations=False see here


Issue 493 Change permission from tmp_dir so less privileged user can …

…run goss
Include a chmod 644 to set read permission from any user that runs the goss tests.
Let me know if there is something else that I need to do.

Thinking the chmod makes more sense around line 33. For everything below the folder to not conflict with line 29.

Fixes #493

I did a commit to change the line but it won't work
Wait a moment please, I am working on it

Yeah, Ideally goss binary should be 755, all other files 644.
Maybe chmod -R a+r $tmp_dir?

Looks good on a quick eyeball check. I'll test it later today and merge.

Ok @aelsabbahy thanks !


Slow BatchNormalization layers (CPU tested)

Hi,
I added BatchNormalization layers to my model and it suddenly took much more time to train.
It takes 561 seconds for one epoch with them:
Epoch 1/1
4096/4096 [==============================] - 561s - loss: 0.0946 - acc: 0.9006
This is if I comment out the BatchNorm layers (186 seconds):
Epoch 1/1
4096/4096 [==============================] - 186s - loss: 4.5043 - acc: 0.5933
I wrote on the user group and someone informed me that adding Dropout slows it down further, but even without Dropout this is much slower (above 500 seconds too).
Is it a CPU-related issue? Or is such a slowdown expected? Below is my model.
    model = Sequential()
    model.add(Convolution2D(32, 11, 11, subsample=(4,4),input_shape=(3,227,227)))
    model.add(PReLU())
    model.add(BatchNormalization())

    model.add(Convolution2D(64, 5, 5, subsample=(2,2)))
    model.add(PReLU())
    model.add(BatchNormalization())

    model.add(Convolution2D(64, 3, 3))
    model.add(PReLU())
    #model.add(BatchNormalization())
    model.add(MaxPooling2D((2,2), strides=(2,2)))

    model.add(Flatten())
    model.add(BatchNormalization())
    model.add(Dense(400))
    model.add(PReLU())
    model.add(BatchNormalization())
    model.add(Dropout(0.25))

    model.add(Dense(400))
    model.add(PReLU())
    model.add(BatchNormalization())
    model.add(Dropout(0.25))

    model.add(Dense(3, activation='softmax'))
    optimizer = keras.optimizers.RMSprop(lr=0.0005, rho=0.9, epsilon=1e-6)

    model.compile(loss='categorical_crossentropy',optimizer=optimizer)

Not sure if it's relevant but should BN be applied before activations?

Hi,
Any update on this ?
The BatchNorm is Keras is till tremendously slow. Is there any way to alleviate the problem ?
And it increases linearly with batch size. So even if my GPU is under utilized, I cannot increase my batch size, since that would make it even more slower.

+1 I observe same thing

In Theano, we juste merged a wrapper of cuDNN batch norm that must be used
manually:
http://deeplearning.net/software/theano_versions/dev/library/sandbox/cuda/dnn.html#batch-normalization
It speed up the compilation and execution time.
So if you or someone else modify keras to use it, it would be great.
On Thu, Jul 14, 2016 at 12:57 PM, iaroslav-ai<EMAIL_ADDRESS>wrote:

+1 I observe same thing
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/1309#issuecomment-232726054,
or mute the thread
https://github.com/notifications/unsubscribe/AALC-3IwuQ6xSPdXSJRiNkyk6zBBgWoCks5qVmqBgaJpZM4G4bYz
.

Nice @nouiz !!!
@fchollet @farizrahman4u tensorflow also has batch norm by default, should we benchmark this and create a batch_norm backend?

If its faster, then we defenitely should.

I just observed almost a 10x slowdown when using batch normalization using GPU. I am using the Theano backedn, CUDA 8 RC and CuDNN.

Has anyone made any progress on this? I'm working on something for myself but I'm hesitant to try to merge it because of style issues and the like. Has anyone started this on their own?

Has anyone attempted this? I would like to use it in my implementation (though it looks like the theano cudnn BN doesnt support 5 dimensional data yet)

Have encountered the same problem with Theano backend + GPU (980 Ti). At least two times slower than models without BatchNormalization.

Batch norm have a computation cost. Be sure to use cudnn 5.1. In particular
for Pascal GPU.
I don't know if Keras finished to wrap Theano cudnn batch norm or not. Make
sure that it use it.
On Sat, Sep 24, 2016 at 8:18 PM, dk1013<EMAIL_ADDRESS>wrote:

Have encountered the same problem with Theano backend + GPU (980 Ti). At
least two times slower than models without BatchNormalization.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/1309#issuecomment-249394900,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALC-wsycDfppe9hsNyOF4w7L_C3C7k6ks5qtb3sgaJpZM4G4bYz
.

Question : Is batchnorm actually worth all this trouble?

@farizrahman4u I really wish it was not... but at least for GANs, it is mandatory.

Yes Keras wraps cuDNN batch norm when available on the Theano side.
You could replace BN with gradient normalization or activation
normalization.
On Sep 27, 2016 12:44 PM, "Eder Santana"<EMAIL_ADDRESS>wrote:

@farizrahman4u https://github.com/farizrahman4u I really wish it was
not... but at least for GANs, it is mandatory.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/1309#issuecomment-249976357,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArWb3IDz_SiTGYyIgmCvL6F5wo4i6lvks5quXI5gaJpZM4G4bYz
.

@nouiz Have updated my cudnn from v5005 to v5103, but observed no performance gain.
Is cudnn important for BatchNormalization?
I also tested the same model with tensorflow backend (after adjusting image_dim_ordering conventions etc) and found tensorflow runs way faster than theano.

I wonder if there is any progress on this issue?

Yes. Update Theano to the dev version and a "recent" enought keras will use
the new Theano interface that should have this fast from Theano side. So it
work on the CPU and the GPU with and without CuDNN. If CuDNN is there, it
will be faster.
I'm not sure which recent enough mean for keras. For Theano the master of
Theano 0.9beta1 is recent enough.
On Tue, Feb 14, 2017 at 8:58 AM jiqiujia<EMAIL_ADDRESS>wrote:

I wonder if there is any progress on this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/1309#issuecomment-279713703,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALC-_ydEJPYtJuylD-xk3R1SD52b2yoks5rcbMXgaJpZM4G4bYz
.


Update to API 20

I followed instructions from updating-sdk.md
Changes

Update to API 20 (commit 0161d49d)
Bump lib version to 0.8.0
Update README
Support MacOS with M1 for generate-bindings.rs

Additional notes

To make running generate-bindings.rs work I had to copy libs from ${FIRMWARE_ROOT}/lib to ${FIRMWARE_ROOT}/build/f7-firmware-D/sdk/lib

Hi @burnoo,
Thanks for the PR! I regenerated the bindings based on the 0.80.1 release (still SDK 20), since 0161d49d is not yet in the release branch.
I try to keep the bindings matching a particular release to ensure that it's easy to update to the matching firmware.

OK. It seems that GitHub is a little confused, but your changes have been merged to master (d5a3d149e2713ebdeb289b937b908e9f389ded53). Thanks for the PR! I'll push the new versions shortly.
Closing this PR, since there doesn't seem to be any way to convince GitHub that it's been merged.

Released as flipperzero-0.8.0. Thanks again!


Fix React key warning in Next.js / React 18.3

Closes https://github.com/dohomi/react-hook-form-mui/issues/247
There has been a change in React 18.3 where keys should not be spread.
This PR explicitly sets the key on SelectElement child options

Thank you, @cjsewell!


add new QA HTML Test tool for comparing a font family.

Hi there,
this PullRequest is related to a new test tool to compare the character set of a given font family. Please see the short demo video:
https://user-images.githubusercontent.com/21055547/165339403-7f28806f-ce55-4382-822d-6d4674892a5e.mp4

@m4rc1e What do you think about this idea? Would be great to get your thoughts on that, because you have been responsible for the HTML test tools mainly, right?
I use the kind of comparison almost everyday and thought it would fit perfectly into the google html test tools.
Would be great to extend it with unencoded glyphs and meta-data information as well, but for the first version I assume it's ok like it is for getting your feedback.
Thanks, Olli

Thanks Olli, this looks great!
I'll take a proper look at this next week.

I've been toying with the idea of animating through a VF's axes using css. I feel this may be a better approach than relying on just fvar instances since we have many families with multiple axes arriving.

Excuse me @m4rc1e , but I am not sure if I understand you right. Can you please explain it in different words?

Something like https://codepen.io/michellebarker/pen/bPEWGK, but for every character individually.

Absolutely :) I have done something similar already in a different context. I was also thinking about having some interactive functions included to de/activate opentype feature, increase font size, etc. This pull request is just the first draft and it would be great if we could extend it in future.
Have you seen this page: https://www.fontspecimen.com/ (I have done it during my time at Monotype). A lot of the ideas could be useful in the family.html as well.

@m4rc1e Any idea, why the checks have restarted today and why they fail? A couple days ago, when I first made a pull request all checks passed.

Hey @m4rc1e , is there anything I can help you with to finish this first version?


Documentation

Documentation complete
Documentation for all existing modules complete
Documentation has been done using google docstring format
More info can be found at https://google.github.io/styleguide/pyguide.html
For doc generation , sphinx has been used.
One can find documentation at docs/source/index.html after running the follwing
$ cd docs/
$ make generate
$ make html

CHANGELOG.md has been added to track version release and changes

Added Codacy for code quality tests as well