Ivy-only Webpack Compiler Plugin

This change introduces a new Ivy-only Webpack compiler plugin.  The plugin is based on and leverages the Ivy ngtsc APIs  from within the @angular/compiler-cli package.  The plugin also simplifies and reduces the amount of code within the plugin by leveraging newer TypeScript features and capabilities.  The need for the virtual filesystem has also been removed.  The file replacements capability was the primary driver for the previous need for the virtual filesystem.  File replacements are now implemented using a two-pronged approach.  The first, for TypeScript, is to hook TypeScript module resolution and adjust the resolved modules based on the configured file replacements.  This is similar in behavior to TypeScript path mapping.  The second, for Webpack, is the use of the NormalModuleReplacementPlugin to facilitate bundling of the configured file replacements.  An advantage to this approach is that the build system (both TypeScript and Webpack) are now aware of the replacements and can operate without augmenting multiple aspects of system as was needed previously.
The plugin also introduces the use of TypeScript’s builder programs.  The current primary benefit is more accurate and simplified dependency discovery.  Further, they also provide for the introduction of incremental build support and incremental type checking.
NOTE: The deprecated string format for lazy routes is not supported by this plugin.  Dynamic imports are recommended for use with Ivy and are required when using the new plugin.
Performance and memory usage have also been improved with these changes.  Initial benchmarking indicates an average ~10% reduction in production build time and a similar ~10% reduction in peak memory usage.  Watch mode (including the development server) no longer requires the forked type checker which provides a ~30% improvement in memory usage for typical development scenarios.  The plugin is also not yet fully optimized.  The next phase of development will introduce performance tracing and support further improvements.

Is it available as a beta API though? Can I just replace AngularCompilerPlugin with NgTscPlugin?

Is it available as a beta API though? Can I just replace AngularCompilerPlugin with NgTscPlugin?

Plugin means two different things between these two classes.
AngularCompilerPlugin is a webpack plugin. NgTscPlugin is a plugin to tsc_wrapped from the Bazel build infrastructure. It might be wrappable in a webpack context (as ivy.AngularCompilerPlugin in this PR does), but that is left as an exercise for the reader currently.


output explain plan when compatibility verifier rejects change in query execution

Makes the compatibility verifier a bit easier to understand when it fails by printing the explain plan.

Codecov Report

Merging #10063 (2a40f96) into master (c4ae332) will decrease coverage by 4.50%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #10063      +/-   ##
============================================
- Coverage     68.75%   64.24%   -4.51%
+ Complexity     5685     5669      -16
============================================
  Files          1996     1941      -55
  Lines        107802   105335    -2467
  Branches      16388    16094     -294
============================================
- Hits          74115    67675    -6440
- Misses        28440    32757    +4317
+ Partials       5247     4903     -344

Flag
Coverage Δ

integration1
?

unittests1
67.90% <ø> (+<0.01%)
:arrow_up:

unittests2
13.60% <ø> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...va/org/apache/pinot/common/config/NettyConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/pinot/common/metrics/MinionMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/apache/pinot/common/metrics/ControllerMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/BrokerQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ache/pinot/server/access/AccessControlFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/messages/TableDeletionMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...pinot/minion/exception/TaskCancelledException.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...not/common/exception/HttpErrorStatusException.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...t/core/startree/plan/StarTreeDocIdSetPlanNode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../core/startree/plan/StarTreeTransformPlanNode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 372 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[FLINK-4436] Unclosed DataOutputBuffer in Utils#setTokensFor()

DataOutputBuffer dob = new DataOutputBuffer();
 credentials.writeTokenStorageToStream(dob);
dob should be closed upon returning from the method.
Also in YarnApplicationMasterRunner#createTaskManagerContext().

+1 to merge

merging


Exclude locales, manpages & docs

This script reduces the size of the resulting image significantly by excluding locales, docs and manpages during the bootstrapping process.
EDIT: 83.21 MB vs. 125.1 MB to be specific.

Please see the discussion over in #48.

I read it. I wasn't expecting you to merge it actually, just something to get the ball rolling, show the mechanism of how you can avoid having to delete anything post-build etc.

This repository is now deprecated in favor of https://github.com/debuerreotype/docker-debian-artifacts, which builds reproducible Debian rootfs tarballs using https://github.com/debuerreotype/debuerreotype. :+1:


Generic provisioning with Debian 8 creates invalid APT source file

When you try to create a docker machine using the generic driver for a remote Debian 8 machine, the following invalid apt source file is created:
deb c https://apt.dockerproject.org/repo debian-jessie main
That c character after deb should not be there. I tried to fix it manually, but as soon as I run the command again, the file is overwritten with the invalid content.
Version is 0.6.0, build e27fb87
As a sidenote, when creation fails, the machine is still added to the local machine list, so when you re-run the command you get an error saying the machine already exists. Shouldn't it wait until the creation succeeds until it is added?

After running it with the debug option it seems the issue is in the install script at get.docker.com. In the following line, it seems the brackets needs to be escaped
$sh_c "echo deb [arch=$(dpkg --print-architecture)] https://apt.dockerproject.org/repo ${lsb_dist}-${dist_version} ${repo} > /etc/apt/sources.list.d/docker.list"

Might want to file an issue on https://github.com/docker/docker/issues/new; thanks!

Thanks, submitted here instead: https://github.com/docker/docker/issues/21459


Auth instances cleanup

@samtstern While working on the anonymous auth PR, it was frustrating to have to pass in flow params each time I wanted to get an auth or user instance. This PR fixes that issue with a slightly hacky solution of storing the flow params in a static variable that's reset each time the helper activity is created.
This PR is built on top of #785 which is built on top of #784.

This change strikes me as a little strange.  We've avoided, until now, keeping FlowParams as static.  Instead each Activity and Fragment has kept the flow params as a member variable and passed it along through the flow.
If we do this change, then we'd have a mixed approach.  A more consistent thing to do would be to make this change and then add a static FlowParameters getFlowParams() method to AuthInstances but I don't really like that idea since static variables are generally code smell.

@samtstern Yeah, upon looking back at this PR it does seem kinda gross. I've updated it to convert the AuthInstances into an AuthHelper object. While I know you were trying to get rid of the helpers, I feel like keeping a simple and contained helper just for the auth stuff makes sense and reduces the bloat of having to pass in those FlowParams each time. What do you think?

@samtstern Well that was rough. Finally got the build to pass though! 😄

Sorry I forgot about this, I am fine with this change!

@samtstern awesome possum! 😀


Kafka lag exporter caching old consumer groups data

Even though I'm deleting consumer groups which are not active(from cmd in kafka-broker).. I am seeing some consumer groups data available in grafana..

Hi @saithejareddy.  When we no longer detect consumer groups we will unpublish metrics from prometheus.  Any metrics recorded for consumer groups from the past will remain in prometheus, you shouldn't see data for the groups later in the time series.

Got it, Thanks a lot, It helped me


Feature request: get_original_path_from_repo_root function

TL.DR.: I would like to have the function that works differently to get_path_from_repo_root
the same as get_original_terragrunt_dir works differently to get_terragrunt_dir.

Suppose there is a repo with file structure:
.
├── project.hcl
├── environment1
│   └── terragrunt.hcl
└── environment2
    └── terragrunt.hcl

# project.hcl
locals {
  default_aws_tags = {
    repo_path = get_path_from_repo_root()
    abs_root_path = get_terragrunt_dir()
    abs_env_path = get_original_terragrunt_dir()
  }
}

# terragrunt.hcl
locals {
  project_vars = read_terragrunt_config(find_in_parent_folders("project.hcl"))
}

inputs = merge(
  local.project_vars.locals,
  {# some other inputs}
)

get_path_from_repo_root() will return . in both environments, but I need some function to return the correct path from original calling directory, not a directory in which the function is used.
Using get_original_terragrunt_dir() will return absolute path of environment directory, unlike get_terragrunt_dir() will return absolute path of repo's root.
As a workaround I currenlty used a trick:
repo_path = replace(get_original_terragrunt_dir(), get_parent_terragrunt_dir(), get_path_from_repo_root())
that works not exactly the same in all cases.

Hi,
I was wondering if can be used repo_path = run_cmd("pwd") to get exact path to directory

It will return absolute path, but I want to have a related path from the repo root to the calling module


Start index parameter for searching pubs.

This PR adds an optional parameter to search_pubs, where you can define the starting index of the list of publications.
If parameter is not defined, the URL will remain unchanged.
If start > 0, appends &start=INDEX to query URL.
This allows for the query to start further in the list, if you already searched for the same terms and would like to skip the initial values that were already parsed previously, in another run.

Call the parameter start_index instead of start

Call the parameter start_index instead of start

Done and squashed.


Tweak many gizmos stress test

Objective
Make the stress test example many_gizmos a little bit more fun.
Solution
Draw random lines in a cube,  seeding by the elapsed seconds.

This was discussed in the gizmos PR. https://github.com/bevyengine/bevy/pull/6529#discussion_r1118050303
I agree with james in that it makes it difficult to use it as a benchmark.
Maybe instead of replacing the "simple" setup, you could add more setups? Replacing Config.fancy: bool by an enum that cycles between "simple black line" "disc", "cube" etc.

Remove variability if fancy is off, the state is constant now.


Ignore stat error when no hidden files in directory

Changes
This PR will fix the old error of stat complaining cannot stat '.*' by adding a filter for it - like the already filtered error cannot stat '*'.
Checklist

[x] have tested my change
[x] eslint is not complaining

Thanks @sebjulliand !


defaultValue doesn't work on update replace

default value doesn't work when an existing row is being replaced, with another row where a required field is null

Can you give me some more details in what you're trying to achieve? Reset a value to that you have defined in withDefault?

I have created a value in database, and now I want to replace that value with a new object (which was created using the fromJson method)

I am using update(xxx).replace(aaa)

Sorry for the delay, but I think I still need more info. UpdateStatement.replace should set fields that are null back to null in the database. If that's not happening it would be a bug in the library. If you need a different behavior, please explain what you'd expect to happen.

I would expect the replace function to work exactly as create function. Remove the old row, and create a new row in its place.

Sorry for the delay. This is pretty much the same problem as in #25. If a column is set to null we can't know if that's null as in "reset to the default value" or "override this column to NULL".

This is only a problem, when the attribute is nullable. Secondly, I have read your comment on #25 too, and I would simply disallow default values for nullable attributes if I were you.

This is now implemented on develop - absent values will be replaced with their default value.


Update it_it.json

Wait, that's a wrong branch 😅. Please make a PR targetting 1.19.3-Unified, I don't plan to update localization on old versions.


Add endpoint that returns the version of all services in an environment at a given commit

REV: DSN-S9VVRW

FYI I renamed the PR to say "commit" instead of "time"


save somewhere

like cloud or something
maybe local icloud thing

save:

raw: audio, image
processed:

OCR
speech to text
multimodal image
multimodal audio
embeddings? using ImageBind or such embedding that work with image+text+audio :D,
etc

https://github.com/facebookresearch/ImageBind


Config sync

Continuing the config cleanup parts from https://github.com/PX4/Firmware/pull/5962.

@dagar Giving up on this one. I see the value but I failed creating a quick-and-dirty comparison tool. I think we should come back to this with a tool to validate


NIFI-10472 Upgrade SnakeYAML from 1.29 to 1.31

Summary
NIFI-10472 Upgrades SnakeYAML references from 1.29 to 1.31 and replaces several module version overrides with a managed dependency declaration in the root Maven configuration.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

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

Pull Request Formatting

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

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

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

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

Licensing

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

Documentation

[ ] Documentation formatting appears as expected in rendered files

Does 1.31 actually fix it? I think when we upgraded 1.31 where I work, the enforcer plugin still reported it as vulnerable.


feat: support extmark_overlay

It allow highlight character different with current character on buffer
it working with hop and lightspeed.
lightspeed need  #869 to have same result like neovim

@andyljones @theol0403

Fantastic! I pulled #868 and #869 into the same branch and ran the extension, and it all seems to work! Happy to pay out soon as it's merged.

Great to see this implemented! Funny timing as I was just starting to have the time to give it a shot, but it seems like you've done a great job of the PR :). I'll have to test this more thoroughly, but I've noticed some bugs:

viewport height broken: I know you've mentioned this, but lightspeed scans much too far out out of the viewport, affecting performance, but also does not work when you scoll down beyond 200 lines. Hopefully the only solution is not to patch lightspeed, and instead provide a way for the native nvim api to know the vscode viewport.
"instant mode" cursor, which comes after sneaking or f/t-ing, the fake cursor is hidden. Solution is to do hi LightspeedCursor gui=reverse
eol labels: lightspeed supports doing s<CR> which then shows a label after the last character. These are currently hidden.
I'll have to figure out how to reproduce, but I've found that sometimes the 1-character highlighting produces some weird overlay text

about the viewport height issue. I think @asvetliakov  already try to sync visible range to vscode but it have problem.
I already try and the scrolling is not good. I am not sure about jumplist.
https://github.com/vscode-neovim/vscode-neovim/blob/master/src/cursor_manager.ts#L583
https://github.com/vscode-neovim/vscode-neovim/blob/master/vim/vscode-neovim.vim#L89
that why i create #869  and do a patch to lighspeed.
https://gist.github.com/windwp/5fd492d8ace55fbe12bb7be67594bb60
It is a hacking way but it working well and don't break anything.

I'll have to figure out how to reproduce, but I've found that sometimes the 1-character highlighting produces some weird
overlay text

if that character has the same character on buffer then it won't use overlay method. It highlight same as a normal character but with different color. It makes better performance with another plugin because some plugin highlight ext_mark for background text too so every character is overlay and it slow vscode.

I will fix eol issue and add test.

I think @asvetliakov already try to sync visible range to vscode but it have problem. I already try and the scrolling is not good. I am not sure about jumplist.
https://github.com/vscode-neovim/vscode-neovim/blob/master/src/cursor_manager.ts#L583 https://github.com/vscode-neovim/vscode-neovim/blob/master/vim/vscode-neovim.vim#L89

That's an old hack using winline(). Since then, Nvim RPC UI has added the win_viewport UI event:
https://github.com/neovim/neovim/blob/f876a0eeb5af6f41acb187eb9442062c3984b26d/runtime/doc/ui.txt#L603

That's an old hack using winline(). Since then, Nvim RPC UI has added the win_viewport UI event:
<EMAIL_ADDRESS>
This should also fix #149. @windwp it would be awesome if you could implement this!

I think vscode neovim already implement win_viewport.
https://github.com/vscode-neovim/vscode-neovim/blob/b9def15f267d0e212c989981c883741de4256993/src/highlight_manager.ts#L88-L99
https://github.com/vscode-neovim/vscode-neovim/blob/52ff62612d3bb9ae2a50ce00b9e068c5d175abda/src/cursor_manager.ts#L109
vscode know the viewport of neovim but that viewport is not same of current visible ranges on vscode.
neovim viewport height is 200 and cursor always on a center position . set scrolloff=100.
https://github.com/vscode-neovim/vscode-neovim/blob/master/vim/vscode-options.vim#L16
vscode manage all scroll function . Neovim doesn't have API to scroll and spam   is not good.
https://github.com/vscode-neovim/vscode-neovim/blob/master/vim/vscode-neovim.vim#L89

viewport height broken: I know you've mentioned this, but lightspeed scans much too far out out of the viewport, affecting performance, but also does not work when you scoll down beyond 200 lines. Hopefully the only solution is not to patch lightspeed, and instead provide a way for the native nvim api to know the vscode viewport.

Impressed that you spotted this - I was trying to get a minimal example of this behaviour earlier and was pretty puzzled.

Fantastic! I pulled https://github.com/vscode-neovim/vscode-neovim/pull/868 and https://github.com/vscode-neovim/vscode-neovim/pull/869 into the same branch and ran the extension, and it all seems to work!

I gave this a try with vim-sneak and label mode enabled. The overlay itself seems to work now. Great! I noticed, however, the cursor is no longer placed on the first occurrence (as it is still is with label mode disabled). I need to e.g. press <space> to do so (or press w which moves from the found word onwards). Also, the first occurrence is highlighted in a less visible way than following occurrences. I know this is very early in progress, but wanted to share my findings with you.

@trkoch  you should try to change highlight  color on vim-sneak label.

Rebase and tests should work now

@theol0403 Is there anything else blocking the merge?

@theol0403 Is there anything else blocking the merge?

Sorry for the delay. There are two things:

Avoidance of lightspeed patch. Ideally, this PR would include a solution that does not involve patching lightspeed. This is so that all plugins do not need to be patched like hop/sneak/leap. I was hoping @justinmk could provide a suggestion to https://github.com/vscode-neovim/vscode-neovim/pull/868#issuecomment-1111619159, which basically says that there exists an api for nvim scrolling->vscode scrolling but not vscode scrolling -> nvim set scrolling. It would be ideal if @windwp could contribute a solution that allows nvim and vscode scrolling to be synchronized, fixing jump plugins and also C-u/C-d scrolling.
Some bugs I haven't had time to report:
- highlighting tabs cause heart symbol to appear as overlay:

- also, searching in lightspeed in a buffer with tabs cause hearts
- vscode previous cursor does not disappear when lightspeed tries to hide it (minor)
- once, I experienced that the grey wash did not disappear after lightspeed was done, and I had to make a new search for the grey to go away (minor)

If the strange heart issue is resolved, I can merge this.

@theol0403  can you provide a step to get "heart char"

it is the solution for scrolling,
https://github.com/vscode-neovim/vscode-neovim/blob/1442396107c7314af024e6fdd8832cc06dc0792e/src/cursor_manager.ts#L583
https://github.com/vscode-neovim/vscode-neovim/blob/master/vim/vscode-neovim.vim#L89
maybe it have another issue so @asvetliakov comment it.
lightspeed doesn't need scrolling to work it only need to know the viewport of vscode because on the first key s it find a unique text  on screen, another jump target plugin doesn't have that feature.
so  i suggest to make a new option to allow scroll neovim by  or  . we can uncomment that code and see what happen.

@theol0403 can you provide a step to get "heart char"

Copy this C file with tabs:
static uint8_t notify_func(struct bt_conn *conn,
			   struct bt_gatt_subscribe_params *params)
{
	if (!data)
		printk("[UNSUBSCRIBED]\n");
	printk("[NOTIFICATION] data %p length %u\n", data, length);
	return BT_GATT_ITER_CONTINUE;
}

Line select all to get:

My theme: Monokai Pro

Thanks for the fix! I found one more bug:

eol on empty line: lightspeed shows labels when doing s<CR> on empty lines. However, label isn't shown on those empty lines in vscode.

I'll merge after this, we can look into the scrolling issues separately. I realized that I've been incorrectly assuming what the problem is, its less major than I thought. I had thought that lightspeed didn't work below line 200 at all, not that it just doesn't know about scrolling (but does know about position).
I'd say the other significant problem is that lightspeed tries to match all 200 lines in the buffer, which reduces performance and accuracy of lightspeed. The solution is to reduce the nvim buffer size to the vscode window size and solve issues related to that - PRs welcome!

Looks like extmark tests fail on latest commit. Will merge once fixed.

Thank you for the PR! And thanks @andyljones for the incentive :)
Just a reminder that for optimum lightspeed UX:

hi LightspeedCursor gui=reverse for fake cursor
https://github.com/vscode-neovim/vscode-neovim/pull/869 and https://gist.github.com/windwp/5fd492d8ace55fbe12bb7be67594bb60 for proper viewport height for labeling

@windwp your code is very well written and you have a great understanding of nvim and this plugin. Even after this PR, would you consider being a contributor? Thanks!

@theol0403  Thank i will try to make a sync viewport on neovim currently it have issue with fold.
Maybe we can rewrite some of vim code to lua.  Create _G.vscode and vscode.api.

Hey, great work here :D One question though. Is there any way to disable this feature / highlighting? thank you.

@theol0403  is this your email<EMAIL_ADDRESS>@unTanana it  only highlight when plugin call it

@theol0403 is this your email<EMAIL_ADDRESS>Yep! Thanks!

Because it seems to be:

Activated when using visual mode for selecting multiple lines (which is fine I guess)
Not going away after I'm done selecting
Not going away after I'm done jumping with an easymotion plugin either.

Actually, I've noticed that too, but I didn't realize it was from this PR. It appears as though tabs are highlighted extra. Not sure why they aren't going away though.

I can confirm that downgrading fixes this, and also my issue with easymotion excessively triggering the output tab, covering my screen each time I want to do a jump

Hi, thanks for adding support for the lightspeed plug-in.

There is a delay using the "f" and "F" keys.
With "s" and "S", the cursor won't jump for the first match (white highlighted match) until I press escape, which is kind of confusing sometimes as it is not sure whether we jumped to that place or not until press escape

Sorry for my laziness to open a new issue. I post the issue I am encountering, looks like has some bugs for some plugins using virttext with overlay.
Using https://github.com/kevinhwang91/nvim-fFHighlight,

Hi, thanks for adding support for the lightspeed plug-in.

There is a delay using the "f" and "F" keys.
With "s" and "S", the cursor won't jump for the first match (white highlighted match) until I press escape, which is kind of confusing sometimes as it is not sure whether we jumped to that place or not until press escape

https://github.com/vscode-neovim/vscode-neovim/pull/868#issuecomment-1131963354

Just in case this code is very old (from the first versions) and i'm not sure if it would work anymore.

@theol0403 Thank i will try to make a sync viewport on neovim currently it have issue with fold.
Maybe we can rewrite some of vim code to lua. Create _G.vscode and vscode.api.

Sorry, I missed this. Awesome, thanks a lot! And yes those sound like great places to start!


[EAGLE-749] Add unit test for model.

Add unit test for model which under alert-common moudle.
Fix equals() hashcode() inconsistent for PartitionedEvent,StreamEvent,Publishment.
Fix bug for StreamColumn BooleanType.

https://issues.apache.org/jira/browse/EAGLE-749

The module coverage will change to 56%.


Customer Experience Outcomes blog, initial draft.

Description
Initial PR for  Customer Experience Outcomes
Issues Resolved
#2579
Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

I'm away until 3/19, but I'm good with this once the noted changes have been made.

I've gone ahead and signed off and merged on every suggestion you've made. Moving to publish - please reach out if you find anything else and we need to make a post-publish edit.
Thanks Nate!


FAQ: 'back to normal Fedora defaults' method is outdated, as of 3.5.0

The FAQ describes restoring Fedora defaults like so:
sudo rm -f /etc/dconf/db/local.d/01-ublue

As of 3.5.0, the file has moved to /usr/share/glib-2.0/schemas/zz0-bluefin-modifications.gschema.override, as per the diff.
Unlike the previous method, the file cannot be outright removed at runtime as /usr/share is read-only.

Nice catch! @fiftydinar do you have any recommendations on how we can support people who want a vanilla GNOME experience?

Oh, forgot that you have this in documentation.
I think that we can have ujust script, which dumps all gschema XMLs + overrides + dconfs that are not Bluefin's, & dconf load's them.
For convenience, we can place those in /usr/share/ublue-os/dconfs/vanilla/, so we load them.
We can easily convert gschema-overrides on the fly with the script I made for Bazzite's PR (dconf-override-converter). For gschema XML's, I'll have to see.

On another note, I think that we can just utilize /usr/local/share/glib-2.0/schemas/ or ${HOME}/.local/share/glib-2.0/schemas/ location for dumping all Gschema XMLs & overrides that are not Bluefins. Than do glib-compile-schemas. Those will then override the default location.
For reverting to default, just remove all those files there & voila, you're back.
Will test all of this

Going through this, I can definitely conclude that pure dconf approach is simpler in this regard for local-users.
But, I think that with this approach, we can mitigate this:
In build-time:

Copy gschema-override to /usr/share/ublue-os/dconfs/
Convert it to dconf with dconf-override-converter
(manually fix key name of "sort-folders-first" with sed, as it's slightly different)
Mirror the default values from Fedora in that dconf (either manually or automatically, maybe some XML parsing can help us with that, but it can be too complex)
Rename the converted dconf to vanilla-fedora-settings

On boot:

In user-setup, on every boot, copy vanilla-fedora-settings from /usr/share/ublue-os/dconfs/ to /etc/dconf/db/local.d/, if vanilla-fedora-settings dconf is available in local.d folder. This will assure that dconf is updated over-time when just script is used (see below).

Have a just script which would do this:
Going to Vanilla settings:

Remove dconfs in /etc/dconf/
Copy dconf from /usr/share/ublue-os/dconfs/ to /etc/dconf/db/local.d/

Going back to Bluefin settings:

Copy dconfs from /usr/etc/dconf/ to /etc/dconf/
Remove dconf from /etc/dconf/db/local.d/

Would a vanilla session in GDM be easier?

I don't know how to achieve that, but sounds like another idea to explore

Think we can declare sessions in /usr/local/share/wayland-sessions

Researching about how sessions work, I think that it would be easier & more future-proof to solve this issue like I described in this comment: https://github.com/ublue-os/bluefin/issues/1743#issuecomment-2394975922
If someone has a different opinion regarding this, please tell.


C++: Multiple redefinition when including webview.h

What OS are you using (uname -a, or Windows version)?
Linux asus 5.4.0-29-generic #33-Ubuntu SMP Wed Apr 29 14:32:27 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

What programming language are you using (C/C++/Go/Rust)?

gcc (Ubuntu 9.3.0-10ubuntu2) 9.3.0
What did you expect to see and what you saw instead?

Due to #369 I had to try out C++ :).
Though I face a weird error with redefinition of webview.h when I include it more than 2 times in my project. I tried putting #pragma once but it didn't change anything.
I have a CMake Project, where I have main.cpp and Injector.cpp.
CMakeLists.txt:
cmake_minimum_required(VERSION 3.16)
project(program)

set(CMAKE_CXX_STANDARD 14)

find_package(PkgConfig REQUIRED)

# include webview
pkg_check_modules(WEBVIEW REQUIRED webkit2gtk-4.0)

include_directories(${WEBVIEW_INCLUDE_DIRS})
link_directories(${WEBVIEW_LIBRARY_DIRS})
add_definitions(${WEBVIEW_CFLAGS_OTHER})

#include gtk3
pkg_check_modules(GTK3 REQUIRED gtk+-3.0)

include_directories(${GTK3_INCLUDE_DIRS})
link_directories(${GTK3_LIBRARY_DIRS})
add_definitions(${GTK3_CFLAGS_OTHER})

add_executable(program main.cpp webview.h Injector.cpp Injector.h)
target_link_libraries(program ${WEBVIEW_LIBRARIES} ${GTK3_LIBRARIES})

main.cpp:
const int width = 920;
const int height = 600;

#include "webview.h"
#include "Injector.cpp"

int main() {
	webview::webview w(true, nullptr);
	w.set_title("Parrot Software Center");
	w.set_size(width, height, WEBVIEW_HINT_MIN);
	w.navigate("http://localhost:3000");
	Apt(w);
	w.run();
	return 0;
}

Injector.h:
#ifndef INJECTOR_H
#define INJECTOR_H

#include "webview.h"

int Apt(webview::webview w);

#endif //INJECTOR_H

Injector.cpp:
#include "Injector.h"
#include "webview.h"

int Apt(webview::webview w) {
	w.bind("aptInstall", [](std::string seq, std::string req, void*) {
	}, nullptr);
	return 0;
}

I'm getting next error stack trace:
/usr/bin/cmake --build /should/not/matter/cmake-build-debug --target program -- -j 4
Scanning dependencies of target program
[ 33%] Building CXX object CMakeFiles/program.dir/main.cpp.o
[ 66%] Building CXX object CMakeFiles/program.dir/Injector.cpp.o
[100%] Linking CXX executable program
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `std::_Rb_tree<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::pair<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const, std::pair<std::function<void (std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, void*)>*, void*>*>, std::_Select1st<std::pair<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const, std::pair<std::function<void (std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, void*)>*, void*>*> >, std::less<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > >, std::allocator<std::pair<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const, std::pair<std::function<void (std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, void*)>*, void*>*> > >::key_comp() const':
/should/not/matter/webview.h:1163: multiple definition of `webview_create'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1163: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `webview_destroy':
/should/not/matter/webview.h:1167: multiple definition of `webview_destroy'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1167: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `webview_run':
/should/not/matter/webview.h:1171: multiple definition of `webview_run'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1171: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `webview_terminate':
/should/not/matter/webview.h:1175: multiple definition of `webview_terminate'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1175: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `webview_dispatch':
/should/not/matter/webview.h:1180: multiple definition of `webview_dispatch'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1180: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `webview_get_window':
/should/not/matter/webview.h:1184: multiple definition of `webview_get_window'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1184: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `webview_set_title':
/should/not/matter/webview.h:1188: multiple definition of `webview_set_title'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1188: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `webview_set_size':
/should/not/matter/webview.h:1193: multiple definition of `webview_set_size'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1193: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `webview_navigate':
/should/not/matter/webview.h:1197: multiple definition of `webview_navigate'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1197: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `webview_init':
/should/not/matter/webview.h:1201: multiple definition of `webview_init'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1201: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `webview_eval':
/should/not/matter/webview.h:1205: multiple definition of `webview_eval'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1205: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `webview_bind':
/should/not/matter/webview.h:1212: multiple definition of `webview_bind'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1212: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `webview_return':
/should/not/matter/webview.h:1222: multiple definition of `webview_return'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/webview.h:1222: first defined here
/usr/bin/ld: CMakeFiles/program.dir/Injector.cpp.o: in function `Apt(webview::webview)':
/should/not/matter/Injector.cpp:23: multiple definition of `Apt(webview::webview)'; CMakeFiles/program.dir/main.cpp.o:/should/not/matter/Injector.cpp:23: first defined here
collect2: error: ld returned 1 exit status
make[3]: *** [CMakeFiles/program.dir/build.make:99: program] Error 1
make[2]: *** [CMakeFiles/Makefile2:76: CMakeFiles/program.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:83: CMakeFiles/program.dir/rule] Error 2
make: *** [Makefile:118: program] Error 2

You don't have to include the webview.h in your Injector.cpp, if you already included it into the Injector.h
And in your main.cpp you should include the Injector.h and not .cpp

You don't have to include the webview.h in your Injector.cpp, if you already included it into the Injector.h
And in your main.cpp you should include the Injector.h and not .cpp

Yeah, ofc, I actually tried out a bunch of things in attempts to make it work, but ofc I didn't include cpp initially.
However, the outcome is absolutely the same.
Looks like defined #ifdef ... #endif is somehow ignored.
Right now I have webview.h included in Injector.h and main.cpp, the error is still the same.

Looks like defined #ifdef ... #endif is somehow ignored.

It is not ignored, otherwise you would get errors before the linking.
The problem here is, that webview.h defines functions and these functions get included in multiple compilation units. During the linking process the linker has a problem to know now which function definition it should use. That means: It is not possible to use the webview functions in more than one compilation unit.
I think you have to reconfigure your CMakeLists to make it clear, that webview.h is a header-only library. Or as a workaround, build your own wrapper around webview.h and just use the functions of this wrapper in your code. That way only the wrapper compilation unit includes the webview functions.

Looks like defined #ifdef ... #endif is somehow ignored.

It is not ignored, otherwise you would get errors before the linking.
The problem here is, that webview.h defines functions and these functions get included in multiple compilation units. During the linking process the linker has a problem to know now which function definition it should use. That means: It is not possible to use the webview functions in more than one compilation unit - that is the case for any header-only library.
I think you have to reconfigure your CMakeLists to make it clear, that webview.h is a header-only library. Or as a workaround, build your own wrapper around webview.h and just use the functions of this wrapper in your code. That way only the wrapper compilation unit includes the webview functions.

Correct me if I'm wrong, but aren't header-only libraries supposed to have only inline functions?
Functions that raised errors appeared to have inline missing.
After putting inline to those functions everything works.

You can #define WEBVIEW_HEADER and have only the header stuff. Not the cleanest but it works.

Closing this due to inactivity. Feel free to open another issue if you still are experiencing problems.

make those inline

WEBVIEW_HEADER

I am getting the same multiple definitions error, now where do I define that?


[Website] change width of the additional case studies cards

change width of additional case-studies cards, so 5 cards fit in a row  instead of 4

before:

after:

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Verified, works as expected. Reviewed and tested by QA

@pabloem, @aijamalnk, could you, please, help with review of current PR?

LGTM, thank you!


Libzip?

I'm having trouble with Libzip, in particular I'm running into the following after installing it on ubuntu with brew install libzip. Any idea to where the LIBZIP_LIBRARY should be linked? I see that the package was installed in the Cellar correctly.
-- The C compiler identification is GNU 5.5.0
-- The CXX compiler identification is GNU 5.5.0
-- Check for working C compiler: /home/linuxbrew/.linuxbrew/bin/cc
-- Check for working C compiler: /home/linuxbrew/.linuxbrew/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /home/linuxbrew/.linuxbrew/bin/c++
-- Check for working CXX compiler: /home/linuxbrew/.linuxbrew/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Performing Test COMPILER_SUPPORTS_CXX11
-- Performing Test COMPILER_SUPPORTS_CXX11 - Success
-- Performing Test COMPILER_SUPPORTS_CXX0X
-- Performing Test COMPILER_SUPPORTS_CXX0X - Success
-- Found OpenCV: /usr (found version "3.2.0")
-- Found PkgConfig: /home/linuxbrew/.linuxbrew/bin/pkg-config (found version "0.29.2")
-- Could NOT find LIBZIP (missing: LIBZIP_LIBRARY)
-- Boost version: 1.65.1
-- Found the following Boost libraries:
--   python
-- Found PythonLibs: /usr/lib/x86_64-linux-gnu/libpython3.6m.so (found version "3.6.7")
CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
LIBZIP_LIBRARY
    linked by target "extract_gpu" in directory /.../dense_flow
    linked by target "extract_cpu" in directory /.../dense_flow
    linked by target "extract_warp_gpu" in directory /.../dense_flow
    linked by target "denseflow" in directory /.../dense_flow

could you try libzip-dev?

Hi @cinjon , @mynameismaxz is right. Please try apt-get install libzip-dev.


Renaming is cancelled if a new file is created in the workspace

VSCode Version: 1.52.1
OS Version: Windows 10

Steps to Reproduce:

Begin a long-running process which creates files within the workspace. In my example, it is a web scraping script downloading stock data.
Begin renaming a folder in the File Explorer sidebar
Every few seconds, as a new file is written, the rename is cancelled/completed

Note that this happens even in the case when the new files being created are not siblings or children of the folder being renamed -- they are hidden away in a different subfolder. But their creation causes the entire sidebar to refresh and cancels the rename operation/confirms the partially typed name.

Does this issue occur when all extensions are disabled?: Yes/No

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

Renaming a file twice causes unwanted rename abortion (#115549)

Can confirm that this is not a duplicate.
Nice try though, bot.

Looks like a duplicate of an issue fixed in the upcoming 1.53, expected this week.

Looks like a duplicate of an issue fixed in the upcoming 1.53, expected this week.

That's great news, thanks :)

/duplicate of #111652
If you still see the problem after 1.53 lands please reopen.


The success/warning messages should be shown properly

Bagisto Version: Master
Steps To reproduce--
1. log in as an admin
2. Try to add any item to the cart.
Expected result: The success/warning messages should be shown properly in velocity theme.
Actual result: The success/warning messages are not showing properly. Please find the screenshot Here

@prateek-webkul
Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot for future reference Here.


I get an error 'NoneType' object has no attribute 'shape' in line 514.

I have installed every requirement and I don't know what I've done wrong.
Here is my output:
File "C:\Users\user\source\repos\lleaguepybot\leaguepybot\leaguepybot.py", line 514, in template_match

width = int(template.shape[1] / ratio)
File "C:\Users\user\source\repos\leaguepybot\leaguepybot\leaguepybot.py", line 483, in lookup
x, y, *_ = template_match(sct_img, template)
File "C:\Users\wolf2\source\repos\leaguepybot\leaguepybot\leaguepybot.py", line 596, in screen_watcher (Current frame)
if lookup(CLIENT_LOGIN_BOX, "patterns/client/login.png") != (0, 0):

Is there any way to solve this?
Thank you

I wouldn't waste too much time with v1 if I were you.
I'm a bit busy now but will finish v2 soonish.
The stack trace is incomplete to understand the error you are getting.
In any case, v1 being based on screenshots only, the locale of your client has to match the screenshots so there's some manual work to do to prepare the files because the ones in the repo match the japanese client.

I wouldn't waste too much time with v1 if I were you.
I'm a bit busy now but will finish v2 soonish.
The stack trace is incomplete to understand the error you are getting.
In any case, v1 being based on screenshots only, the locale of your client has to match the screenshots so there's some manual work to do to prepare the files because the ones in the repo match the japanese client.


Failing pytest with ModuleNotFoundError when using hatch integration with versioningit-onbuild option

I am using v3.1.* with Hatch v1.12.0
I have this in pyproject.toml:
[tool.hatch.build.hooks.versioningit-onbuild]
source-file = "src/dagcellent/__init__.py"
build-file = "dagcellent/__init__.py"
require-match = true

When I run my tests, it fails:
ImportError while importing test module '/Users/timon/work/dagcellent/dagcellent/tests/test_dag_config.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/opt/homebrew/Cellar/python@3.12/3.12.3/Frameworks/Python.framework/Versions/3.12/lib/python3.12/importlib/__init__.py:90: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
tests/test_dag_config.py:7: in <module>
    from dagcellent.dynamic_dag import Config, parse_config_file
E   ModuleNotFoundError: No module named 'dagcellent.dynamic_dag'

When I remove the on-build part, it works again.
Any ideas if I am misconfiguring something or how to dig deeper in the problem? It seems like that my package is not in the test venv.
p.s. I am willing to submit a PR/provide more info.

It seems like that my package is not in the test venv.

Yes, it certainly seems that way.
Can you provide a link to your source code repository, ideally including the onbuild configuration?  If you can't share your code for one reason or another, I'd at the very least need to see an MVCE.

I created an MVCE repo: https://github.com/timonviola/hatch-test

I've figured out what's going wrong: the combination of Hatch, editable/development installs, and versioningit's onbuild hook do not play nice together.  Specifically, the onbuild hook runs for Hatch editable installs (which it's not supposed to, but I don't know how to stop it), and the behavior of the hook triggers a bug in Hatch's editable install support that results in only your dagcellent/__init__.py file being visible on sys.path.
Until someone from Hatch gets back to me on the linked issues, your options are:

Don't use onbuild
Don't use editable/development installs; ways to accomplish this include:

Set tool.hatch.envs.static.dev-mode = false (but then you might have to recreate your Hatch envs if your code changes; I'm not sure)
Use tox, nox, or similar instead of Hatch's envs

OK, I got a response, and I now have a PR (#90) that should fix the problem.  If I test out the PR on your MVCE by changing the versioningit build requirement to "versioningit @ git+https://github.com/jwodder/versioningit@gh-89", then hatch run test:test works.  Can you confirm it works in your original code?

Thanks for looking into it. I tried to update my projects, but it is still not working. I updated the example repo with the suggested change, but I still get module not found:
https://github.com/timonviola/hatch-test/commit/da95a3d7ea8d965b2a4047643e338cf49e130452
-    "versioningit == 3.1.*",
+    "versioningit @ git+https://github.com/jwodder/versioningit@gh-89",

Did I make the right config changes?

You need to change the versioningit dependency in build-system.requires, on this line.  The versioningit in extra-dependencies is only for being able to run the versioningit command via the version script; the command doesn't make use of the "onbuild" code, so it has no relation to the original error.

I can confirm it is working using the patch version. Thank for the help and the quick response!

The patch has now been released in versioningit v3.1.2.


[Epic] waku relay (node)

Problem
A near real time decentralized and secure communication protocol is needed to enable dapps, wallets and other apps (e.g. chat app) to use a decentralized infrastructure while maintaining an acceptable UX,.
Solution
Provide a JS library that allows developers to use waku v2 relay for near real time communication purposes.
Ensure the JS library is functional by including a suite of tests.
Demonstrate the usage of the API by building a CLI chat app that can communicate with the nim-waku example chat app.
For now, the solution can be focused on node support only. A separate issue will track browser support.
Acceptance Criteria
Tests
Demonstrate the following features as working in test:

[x] Publishing messages between js-waku instances #7
[x] Publishing messages between js-waku & nim-waku instances
[x] Encoding/Decoding of Waku messages
~NAT passthrough support #12~

Chat app

[x] Receive and send messages from the fleet #15

Issues
See https://github.com/status-im/js-waku/milestone/1

Done!


Error in WalletConnect Hedera Integration: "INVALID_SIGNATURE" on executeTransaction

Hello, I am a wallet developer and I am developing a Hedera WalletConnect for Android/iOS environments, referencing the guide below:
https://specs.walletconnect.com/2.0/blockchain-rpc/hedera-rpc#hedera_signtransaction
The issue arises during testing on https://wc.hgraph.app/ where the signature map created via signTransaction from my wallet results in an "INVALID_SIGNATURE" error when executeTransaction is performed.
To Reproduce Steps to reproduce the behavior:

Go to 'https://wc.hgraph.app/'
Input necessary field and process "Initialize WalletConnect"
Click "Open WalletConnect Modal" and connect with my wallet (based on hedera-sdk-java for Android, or hedera-protobufs-swift for iOS)
Input necessary field and process "6. hedera_signTransaction".

My code for signTransaction:
byte[] base64DecodedTxBodyBytes = Base64.decode(transactionBody, Base64.DEFAULT);
TransactionBody txBody = TransactionBody.parseFrom(base64DecodedTxBodyBytes);
byte[] signature = privateKey.sign(message); // privatekey for signer
String signatureHexString = Numeric.toHexStringNoPrefix(signature);
SignatureMap sigMap = SignatureMap.newBuilder()
        .addSigPair(SignaturePair.newBuilder()
            .setEd25519(ByteString.fromHex(signatureHexString))
            .setPubKeyPrefix(ByteString.fromHex(publicKey)).build()) / / publicKey : my public key string for signer
    .build();
String base64Sigmap = new String(Base64.encode(sigMap.toByteArray(), Base64.NO_WRAP), "UTF-8");
resultObject.put("signatureMap", base64Sigmap);

Now I get a result as below with success popup:
{
    "params": {
        "signerAccountId": "hedera:testnet:0.0.741192",
        "transactionBody": "ChoKDAjnp4mxBhCx/OalARIICAAQABjIni0YABIGCAAQABgDGIDC1y8iAgh4MgByJAoiCg8KCAgAEAAYyJ4tEM8PGAAKDwoICAAQABj6ni0Q0A8YAA=="
    },
    "signatureMap": "CnAKLDAqMAUGAytlcAMhAHh6CY/kiByrJG8EW2q98oShnIRj5u6wjLzwjK4Zg0CCGkBezWe0pyWIVOXEx8V374TtL6+2LT+TicnXQQolZtrQTOrVyP12qHWvXt4ipLsKudncP7twCMHy4L6HGETcnGsN"
}

Copy above data and paste them into fields on "2. hedera_executeTransaction"
Click "Submit to wallet" and proceed.

Now I get a tranactionlist as a parameter as below:transactionList:  Cs4BKssBClUKGgoMCOenibEGELH85qUBEggIABAAGMieLRgAEgYIABAAGAMYgMLXLyICCHgyAHIkCiIKDwoICAAQABjIni0Qzw8YAAoPCggIABAAGPqeLRDQDxgAEnIKcAosMCowBQYDK2VwAyEAeHoJj+SIHKskbwRbar3yhKGchGPm7rCMvPCMrhmDQIIaQF7NZ7SnJYhU5cTHxXfvhO0vr7YtP5OJyddBCiVm2tBM6tXI/Xaoda9e3iKkuwq52dw/u3AIwfLgvocYRNycaw0=

My code for executeTransaction:byte[] base64DecodedTxBodyBytes = Base64.decode(base64SignedTransaction, Base64.DEFAULT);
Transaction transaction = Transaction.fromBytes(base64DecodedTxBodyBytes);
TransactionResponse transactionResponse = (TransactionResponse)  transaction
    .execute(client);

See error

Submitted transaction fails with following error.

<EMAIL_ADDRESS>failed pre-check with the status INVALID_SIGNATURE
com.hedera.hashgraph.sdk.PrecheckStatusException: Hedera transaction<EMAIL_ADDRESS>failed pre-check with the status INVALID_SIGNATURE

I attempted these actions in the iOS environment based on hedera-protobufs-swift as well, but encountered the same error.

Expected behavior A clear and concise description of what you expected to happen.
The transaction should be submitted successfully.
Desktop (please complete the following information):

OS: [macOS Sonoma 14.2.1]
Browser [chrome]
hedera-sdk-java SDK Version [2.29.1]
grpc-okhttp SDK Version [1.46.0]

Smartphone (please complete the following information):

Device: [Galaxy S24 Ultra]
OS: [Android 14]

Additional context Add any other context about the problem here.
I'll refer to the wallet on https://wc.hgraph.app/ as "example wallet." When conducting the following test using the Dapp on https://wc.hgraph.app/, the issue occurred only when executing the signature map generated from my wallet in my wallet.

Executing the transaction singed by my wallet on the example wallet : SUCCESS  → means my wallet sings a transaction fine.
Executing the transaction signed by the example wallet on my wallet : SUCCESS → means my wallet executes a signed transaction fine.
Executing the transaction signed by the example wallet on the same one : SUCCESS → means the sample wallet works fine in signing and executing transactions.
Executing the transaction signed by my wallet on the same one : FAIL (INVALID_SIGNATURE) → No idea for now.

We have encountered a similar issue where the signature in the transaction is breaking due to makeTransactionBody. You can find more details about this in the next discussion:
https://github.com/hashgraph/hedera-sdk-js/issues/2218
I believe it would be better to move all of the transaction serialization to the transaction.toBytes() method.

@franfernandez20 Thanks for your replying and sharing similar issue. In my case,

I think the issue is that a Transaction object in the SDK contains multiple actual transactions (HAPI transactions) so that the SDK can deal with node connection failures without prompting for additional signatures.
The wallet connect library expects a single HAPI transaction, not a Transaction. So when you .toBytes() a transaction, you're sending a higher level object the wallet connect library, the signature is returns is on the higher level object, not the HAPI transactions within.
Let me look how this can be done with the Java SDK.
For clarity and example
Transaction in the sdk <- This is what you're asking wallet connect to sign
HAPI Transaction for node 0.0.3
HAPI Transaction for node 0.0.4 <- this is what you need wallet connect to sign
HAPI Transaction for node 0.0.5
HAPI Transaction for node 0.0.6
HAPI Transaction for node 0.0.7

This may work
//import com.hedera.hashgraph.sdk.AccountId;
//import com.hedera.hashgraph.sdk.PrecheckStatusException;
//import com.hedera.hashgraph.sdk.PrivateKey;
//import com.hedera.hashgraph.sdk.ReceiptStatusException;
//import com.hedera.hashgraph.sdk.TransactionResponse;
//import com.hedera.hashgraph.sdk.TransferTransaction;

import com.hedera.hashgraph.sdk.*;
import io.github.cdimascio.dotenv.Dotenv;

import java.util.Objects;
import java.util.concurrent.TimeoutException;
import java.util.function.Function;

public class CreateSingleTransaction {

  private CreateSingleTransaction() {
  }
  static AccountId OPERATOR_ID = AccountId.fromString(Objects.requireNonNull(Dotenv.load().get("OPERATOR_ID")));
  static PrivateKey OPERATOR_KEY = PrivateKey.fromString(Objects.requireNonNull(Dotenv.load().get("OPERATOR_KEY")));

  public static void main(String[] args) throws PrecheckStatusException, TimeoutException, ReceiptStatusException { //throws TimeoutException, PrecheckStatusException, ReceiptStatusException {
    Client client = Client.forTestnet();

    // Create a dummy key for the client
    PrivateKey dummyKey = PrivateKey.generateED25519();

    client.setOperator(OPERATOR_ID, dummyKey);
    client.setMaxNodesPerTransaction(1);
    Transaction transaction = new TransferTransaction()
            .addHbarTransfer(OPERATOR_ID, Hbar.from(1).negated())
            .addHbarTransfer(AccountId.fromString("0.0.3"), Hbar.from(1))
            .freezeWith(client);
    transaction.signWith(OPERATOR_KEY.getPublicKey(), signer());

    TransactionResponse response = (TransactionResponse) transaction.execute(client);
    response.getReceipt(client);

    System.out.println("done");
  }
  private static Function<byte[],byte[]> signer() {
    //  you'd invoke wallet connect here and get the signature from it
    return (
            t -> OPERATOR_KEY.sign(t)
    );
  }
}

replace
return (
            t -> OPERATOR_KEY.sign(t)
    );

in the signer() function with your implementation of the wallet connect integration

Note this requires that you know the public key of the wallet being invoked through wallet connect

I don't know about the Java sdk, but with the JavaScript sdk, that only works when the Transaction is internally holding only one actual transaction in the TransactionList, in other words the Client will have needed to have been created with only one node, using the custom forNetwork([{}]) form.

In the example above, I set  client.setMaxNodesPerTransaction(1); to ensure only one HAPI transaction exists within the Transaction, the SDK will still randomly select from its address book.
The signer is invoked for every inner (HAPI) transaction so the actual bytes of the transaction are signed, not the Transaction. If setMaxNodesPerTransaction was set to 10, the signer would be invoked 10 times.
The Javascript SDK also supports the Signer method, wonder if that would work with wallet connect (the downside is that you need to know the wallet's public key) with the .sign method.
A signer can also be setup at the client level, but again, I think the public key has to be known, the signer only returns a signature which isn't enough for a signature map.

maybe a signature map with no public key (or prefix) should be accepted if there's only one possible signature...

@gregscullard
Thanks for your guide in detail. I was able to understand what the issue could be.
In order to handle the hedera_signTransaction request, the Wallet does not create a signed Transaction, but rather creates a signed SignatureMap so I am not sure I can set MaxNodesPerTransaction during hedera_signTransaction process.
In my understanding, for 'hedera_signTransaction' method, it requires only SignatureMap type as a return value from signed TransactionBody object so I cannot set  MaxNodesPerTransaction during signing a given TransactionBody and converting to SignatureMap object for return.
For hedera_executeTransaction, the sample Dapp on https://wc.hgraph.app converts my providing signature map into TransactionList object.
Wouldn't it be possible that the sample Dapp might need to set the MaxNodesPerTransaction property when creating the TransactionList?

@gregscullard Hello,
We have resolved the issue. There were two factors causing the 'Invalid Signature' error.

During the creation of the signature map, I utilized a public key in DER format, which resulted in an error.

Prior to signing the transaction body, I inserted the bytes into a TransactionBody proto container using TransactionBody.parseFrom(base64DecodedTxBodyBytes); - I should have directly signed the base64DecodedTxBodyBytes instead."

It would be cool if the team could review the guide for any additions that could prevent encountering such issues.
If the information is already in the guide, I would appreciate it if you could let me know where I can find it.
Thank you.


feature(cli): Replace CLI code with ss-generator npm

This replaces the CLI code with the ss-generator module.

This seems to be failing on a test that is unrelated to the CLI:
  1) code formatter loading API #call should support alternate extensions:
      AssertionError: expected '// couldn\'t format ./abc/index.a issue=1431377849247' to be 'require.define("/abc/index",function(e,t,n,r,i){window.a="formatter index.a"})'
      + expected - actual
      +require.define("/abc/index",function(e,t,n,r,i){window.a="formatter index.a"})
      -// couldn't format ./abc/index.a issue=1431377849247

Is this fixed in master? If so, we can pull in those commits and try again.

next will need to catch up with master, that is part of the tradeoff. Of course the benefit is that we can take it easy with the 0.5 release and fix 0.4 which I think is quite important.
on your particular case, I don't recognise the problem, so I don't know if it is.

Hi Henrik, it seems to happen when running Travis CI. I can also sometimes get failing tests on the master branch when I run the tests ~ 4/5 times - 1 test run eventually fails.

Changes Unknown when pulling 6bd5ed7af7ad0c86c5126a6a10c5dd4f96ec2141 on feature/separate-cli-generator-v3 into ** on next**.


TEA.YAML

TEA.YAML BY DAOPTIMIST

AMAZING PROJECT

Hey! Just a heads up, this PR is probably spam — some discussion here and here


Training fails

Hi, I'm having this issue when I run training:
python3 train.py ./data/train.csv.zip ./training_config.json
CRITICAL:root:Accuracy on test set: 0.9971641706053186
Traceback (most recent call last):
File "train.py", line 161, in
train_cnn_rnn()
File "train.py", line 151, in train_cnn_rnn
os.rename(path, trained_dir + 'best_model.ckpt')
FileNotFoundError: [Errno 2] No such file or directory: './checkpoints_1486165230/model-2700' -> './trained_results_1486165230/best_model.ckpt'
I'll spend a bit of time tomorrow to see how t fix this problem.

Did you check the saved model directory? Looks like model-2700 doesn't exist.
os.rename(path, trained_dir + 'best_model.ckpt')
FileNotFoundError: [Errno 2] No such file or directory: './checkpoints_1486165230/model-2700' -> './trained_results_1486165230/best_model.ckpt'

@jiegzhan yes, model-2700 files do exist. but there is no model-2700 file as such nor it's a directory:
ls -lrt ./checkpoints_1486165230/
total 71404
-rw-r--r-- 1 root root     1433 Feb  3 23:41 model-1600.index
-rw-r--r-- 1 root root 13073080 Feb  3 23:41 model-1600.data-00000-of-00001
-rw-r--r-- 1 root root  1543734 Feb  3 23:41 model-1600.meta
-rw-r--r-- 1 root root     1433 Feb  3 23:41 model-1700.index
-rw-r--r-- 1 root root 13073080 Feb  3 23:41 model-1700.data-00000-of-00001
-rw-r--r-- 1 root root  1543734 Feb  3 23:41 model-1700.meta
-rw-r--r-- 1 root root     1433 Feb  3 23:42 model-2200.index
-rw-r--r-- 1 root root 13073080 Feb  3 23:42 model-2200.data-00000-of-00001
-rw-r--r-- 1 root root  1543734 Feb  3 23:42 model-2200.meta
-rw-r--r-- 1 root root     1433 Feb  3 23:42 model-2400.index
-rw-r--r-- 1 root root 13073080 Feb  3 23:42 model-2400.data-00000-of-00001
-rw-r--r-- 1 root root  1543734 Feb  3 23:42 model-2400.meta
-rw-r--r-- 1 root root     1433 Feb  3 23:42 model-2700.index
-rw-r--r-- 1 root root 13073080 Feb  3 23:42 model-2700.data-00000-of-00001
-rw-r--r-- 1 root root      241 Feb  3 23:42 checkpoint
-rw-r--r-- 1 root root  1543734 Feb  3 23:42 model-2700.meta
I'll have a look if train.py didn't write something correctly or if os.rename command is incorrect.

python3 -c 'import tensorflow as tf; print(tf.version)'
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcublas.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcudnn.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcufft.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcurand.so locally
0.12.1

My tensorflow version is 0.9

Do you get more files created in checkpoints directory? I see *.meta, *.index, .data- and checkpoint.

My tensorflow version is 0.9, it only produce two training files.

Found this:
https://github.com/tensorflow/tensorflow/blob/master/RELEASE.md
New checkpoint format becomes the default in tf.train.Saver. Old V1 checkpoints continue to be readable; controlled by the write_version argument, tf.train.Saver now by default writes out in the new V2 format. It significantly reduces the peak memory required and latency incurred during restore.

set up the write_version argument if you are in a hurry.
I will try to upgrade the tensorflow and make changes soon.
Thanks for  pointing this out.

Yep, testing it with V1 now.

Or if you have time, please fork this repository and make changes. Thank you.

Yep, works fine with :
saver = tf.train.Saver(tf.all_variables(), write_version=tf.train.SaverDef.V1)
This is the how the warning message looks like:
WARNING:tensorflow:*******************************************************
WARNING:tensorflow:*******************************************************
WARNING:tensorflow:TensorFlow's V1 checkpoint format has been deprecated.
WARNING:tensorflow:TensorFlow's V1 checkpoint format has been deprecated.
WARNING:tensorflow:Consider switching to the more efficient V2 format:
WARNING:tensorflow:Consider switching to the more efficient V2 format:
WARNING:tensorflow:   tf.train.Saver(write_version=tf.train.SaverDef.V2)
WARNING:tensorflow:   tf.train.Saver(write_version=tf.train.SaverDef.V2)
WARNING:tensorflow:now on by default.
WARNING:tensorflow:now on by default.
WARNING:tensorflow:*******************************************************
WARNING:tensorflow:*******************************************************
Thanks

Hai Guys
Any Solution for the above issue . If yes please reply.


New rule: Prefer in-place operators

I don't think any linter currently checks for this, although I did open a feature request in those I knew of where this feature might be in scope (https://github.com/MartinThoma/flake8-simplify/issues/188 and https://github.com/dosisod/refurb/issues/310). Whether they accept it, (and who knows when it would be implemented), this is still a check I'd like to see in Ruff as I don't directly use those other linters and autofixing should be possible. If you think this is a viable rule.
In-place operators lead to more concise code that is still readable. I'm not sure if there's any objective drawbacks (like a common pitfall for certain types). And performance-wise my understanding is that it should be faster (due to the in-place nature) or equivalent (thanks to Python duck-typing that will fallback to  __add__ if __iadd__ is not implemented).
Any of the following:
some_string = (
  some_string
  + "a veeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeery long end of string"
)
index = index - 1
a_list = a_list + ["to concat"]
some_set = some_set | {"to concat"}
to_multiply = to_multiply * 5
to_divide = to_divide / 5
timeDiffSeconds = timeDiffSeconds % 60
flags = flags & 0x1
# etc.

Could be re-written as:
some_string += "a veeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeery long end of string"
index -= 1
a_list += ["to concat"]
some_set |= {"to concat"}
to_multiply *= 5
to_divide /= 5
timeDiffSeconds %= 60
flags &= 0x1
# etc.

Makes sense to me, I'm surprised this does not exist already.

I just thought of a special case with string interpolation that idk if should be included:
# from
base_string = f"{base_string} end of string"`

# to
base_string += " end of string"`

We could support that conditional on the ability to determine that the type of base_value is str.

We could first implement the simple cases of var = var <operator> <expr>. I think this could go into the RUF category. If anyone's interested, I'd be happy to give a look at the implementation.

Maybe this should be a separate rule, but there might be an opportunity for some more micro optimizations here. For example, if it were faster, these:
a_list += ["to concat"]
a_list += ["to concat 1", "to concat 2"]
some_set |= {"to concat"}
some_set |= {"to concat 1", "to concat 2"}
# ect.

could potenchally be further rewritten as:
a_list.append("to concat")
a_list.extend(("to concat 1", "to concat 2"))
some_set.add("to concat")
some_set.update(("to concat 1", "to concat 2"))
# ect.

Hello, I would like to contribute to this issue if it hasn't been taken yet.

Hi, I've made an attempt implementing this rule. Please help to have a review and let me know your thoughts. Thanks.

Closed by https://github.com/astral-sh/ruff/pull/9932.


Added instructions to install, build, and compile as well as screenshots in the readme :)

Noticed you didn't have packaging instructions

Great Work! Thanks


Single dream #15

closes issue #15
Basic view set up - had trouble getting the sprite scene to render in a not-full-screen way. Will look into this.
Maybe we should just bypass a-frame react and go straight to react?

was about to merge PR but noticed your question about by-passing a-frame = let's talk more about this


Add bpi integration illustration

Illustration includes the steps defined in

5.5. BPI Integration > Example BPI Interoperability Workflow

The source file is included in the core/images and can be edited using draw.io.
Figure numbers are updated accordingly.

Merged in another PR.


Args object has no attribute 'cudas'

Hi,
I am trying to replicate your project, but when running your sample single-experiment command, I am getting the cudas attribute not found error. Could you please give me some help? Thank you!
Traceback (most recent call last):
File "scripts/main.py", line 38, in
main()
File "scripts/main.py", line 28, in main
run_experiment.run(args)
File "/home/VCML/scripts/run_experiment.py", line 164, in run
device = args.cudas[i * args.num_gpus // args.num_parallel]
AttributeError: 'Args' object has no attribute 'cudas'

/home/VCML/scripts/run_experiment.py(164)run()
162
163     for i in range(0, args.num_parallel):
--> 164         device = args.cudas[i * args.num_gpus // args.num_parallel]
165         p = ctx.Process(target=ready_go,
166                         args=(args, i, message[i], control[i],

ipdb>

I think I got the reason, it is because of torch.cuda.device_count() gives 0 while it needs to be greater than 0

You should use a server with GPU installed. Closing.


Investigate using OnPreview key events for NavigationViewItems

After the update to NavigationView where ListView was replaced with ItemsRepeater, the NavigationViewItems utilize both the KeyDown and KeyUp events (same as the settings item behaves). However, using theOnPreviewKeyDown or OnPreviewKeyUp  event can improve implementation as we'd only need to use one keydown event type (KeyDown or KeyUp), as brought up in this comment.

Is this issue still relevant given that we moved to only using KeyDown some time ago?


K8s API error

Unable to install on AWS EKS. It is working locally with minikube
Error: Chart requires kubernetesVersion: 1.11.0 - 1.18.0 which is incompatible with Kubernetes v1.17.12-eks-7684af

This PR will solve that: https://github.com/JahstreetOrg/spark-on-kubernetes-helm/pull/52 . ETA - this weekend. Please follow the updates.

Hi @KamalGalrani , is that issue fixed for you?


Get rid of do { ... } catch { ... } for expected errors

Motivation:
Get rid of do { ... } catch { ... } for expected errors
Modifications:
Replacing all uses of this pattern in Tests/**.
do {
    try someOperation()
XCTFail("should throw") // easy to forget
} catch error as SomethingError {
    XCTAssertEqual(.something, error as? SomethingError)
} catch {
    XCTFail("wrong error")
}

with
XCTAssertThrowsError(try someOperation) { error in
    XCTAssertEqual(.something, error as? SomethingError)
}

Result:
Fixes https://github.com/apple/swift-nio-http2/issues/190

This appears to have conflicts with the master branch: can you resolve them?

Thanks for review. I have resolved the conflict.

@swift-nio-bot test this please


User microservice

This one belongs to the Wallet epic because the wallet mcs depends on the user service in order to refer the wallet.rapyd_e_wallet record to the user.id record.

[ ] getUserByTelegramUserId returns the User model by passing a telegram_user_id
[ ] createUser creates a User record. telegram_user_id should be mandatory.

See the user mcs at aufaxbot for reference.


Is it possible to add support for using context in gpt-3.5?

Hey, thank you for the great work!
I am wondering if it is possible to add the function of using context in gpt-3.5 model? As that model is considerably cheaper, adding context is relatively affordable and may help to improve the translation quality.
Moreover, if the context length can be adjusted, e.g., 1 or 2 or 3 paragraphs, that will be awesome!
Thanks again for this great work!

Hey, I could probably add this in as the model works currently. When I first added the context feature, the 3.5 model's total token limit was considerably shorter than it is now. Also, great idea about the context length, I should add that in too!

@chaojiang06 @astromaddie
Hi I implented this feature.
https://github.com/yihong0618/bilingual_book_maker/pull/420
@yihong0618
pls check this PR


Thought: slaves

Do Coordinator and Datanode Slaves work correctly?

Still not tested in 0.2.0. If anyone has or would like to investigate this, please say! :)

I am looking at this now.

Thanks a lot! Again, it would be nice to hear anything interesting you discover when looking into this.

I have a working version of this as we talked about before for slaves of gtm, coordinators and datanodes. Will have a look at the helm chart now and see if there is a cleaner way which could work for kubernetes and docker swarm.
Basically all I'm currently doing is setting an env variable for the MASTER_HOST. If present it creates the config needed and runs as a slave. If the env variable is removed it removes this config which means its now a master (Used for promotion in the event of node corruption).
Biggest problem is setting up a replication user for this which I currently do via a docker secret  replicationUser, replicationUserPassword. I also have adminUser, adminUserPassword secrets which create/update admin users. I append a version at the end of the secrets for rotation like this https://anthonymineo.com/rotating-your-docker-secrets-can-be-easy-if-you-plan-for-it/ have a small app which generates the passwords and a cronjob that rotates them. However I think using hashicorp vault is a better answer to this.

I propose closing this, unless it's being actively worked on; it's been open for 2.5 years, without a clear goal (entirely my fault). :)


checkpoint_epoch_36_nus_mamba

I tried to use checkpoint_epoch_36_nus_mamba.pth to make predictions on the nuscenes point cloud data and found that the score for each box was very low (almost no accurate boxes)

@Damoooon
Hi, can you get right performance (72.1 NDS, 68.0 mAP) by evaluating LION on the nuScenes validation set?

Hi, The result of eval is normal, but the strange thing is that for the same frame of point cloud data, the result of pred_dicts obtained by running test.py with the same model is very different from the result of pred_dicts obtained by using demo.py. Have you tried to visualize 3D prediction frames using demo.py

@Damoooon Hi, can you get right performance (72.1 NDS, 68.0 mAP) by evaluating LION on the nuScenes validation set?

Hi, The result of eval is normal, but the strange thing is that for the same frame of point cloud data, the result of pred_dicts obtained by running test.py with the same model is very different from the result of pred_dicts obtained by using demo.py. Have you tried to visualize 3D prediction frames using demo.py

@Damoooon
Hi, you should use the data_processor of nuScenes dataset to process input data for inference. You can refer to the test.py for creating the dataloader and testing LION.

@Damoooon Hi, you should use the data_processor of nuScenes dataset to process input data for inference. You can refer to the test.py for creating the dataloader and testing LION.

ok, But if DemoDataset doesn't work then how do I inference to my own data?

@Damoooon
Hi, if you want to use your data, you should retrain LION with your data.

I have solved the problem, checkpoint_epoch_36_nus_mamba.pth uses multiframe for training, but the dataloader in demo.py is singleframe, which maybe caused the data domain difference between the training data and the demo data, if use the The singleframe trained model can solve the problem of poor prediction result of demo.py, for example, use argov2_mamba.pth, or change the dataloader in demo.py to multiframe


Error when loading custom dataset

Appreciate all of your fine work done :)
I have a tiny suggesion regarding implementation of custom dataset.
test 773
test shape: (773, 1, 96, 1) (773, 1, 96, 1)
test shape: (773, 96, 1) (773, 96, 1)
Traceback (most recent call last):
  File "run.py", line 149, in <module>
    exp.test(setting)
  File "/workspace/Time-Series-Library/exp/exp_long_term_forecasting.py", line 245, in test
    register_metric(mae, mse, self.args)
  File "/workspace/Time-Series-Library/utils/tools.py", line 147, in register_metric
    target_x = seq_len_index + dataset_name_dict[dataset_name] * 20
KeyError: 'KACO1.csv'

Error message above pops up when I try to implement custom dataset.
def register_metric(mae, mse, configs):     seq_len = configs.seq_len     pred_len = configs.pred_len     model = configs.model     dataset_name = configs.data_path     pred_len_dict = {         96: 0,         192: 1,         336: 2,         720: 3,     }     dataset_name_dict = {         "ETTh1.csv": 0,         "ETTh2.csv": 1,         "ETTm1.csv": 2,         "ETTm2.csv": 3,         "electricity.csv": 4,         "traffic.csv": 5,         "weather.csv": 6,         "exchange_rate.csv": 7,         "national_illness.csv": 8,     }
Apparently, register_metric function in Tools.py does not automatically include datasetname into the defined dictionary. Perhaps custom dataset name should be added to the dictionary automatically?
Thanks.

data_provider里的文件看看，tool.py和run.py 也看看 你少添加自己数据集信息往字典里了。仔细检查一下

Hello！
Thank you for your feedback. We will remove this unnecessary function in the next commit.


more migration to bddl

Closed in favor of #249. Cem suggested that we only let the CI to re-run and update the generated files.

Closed in favor of https://github.com/StanfordVL/bddl/pull/249. Cem suggested that we only let the CI to re-run and update the generated files.


reduce package size. Switch to redux dev tools extension

In order to reduce the file size of this package I moved us from using
webpack to simply using babel. Currently, there is no reason for use to
use webpack and the babel build can take better advantage of shared
code.
Even though the DevTools code was only being implemented in development
mode, the DevTools code was being included in the build. There has been
many requests to switch to using the Redux DevTools Chrome extension and
that saves us roughly 100kb in file size.

👍


nbd: 3.17 -> 3.18. Also, add which to buildInputs (needed by test pha…

…se).
Motivation for this change
To update nbd to the latest version. While building, I noticed the test scripts complaining about the lack of which, so I added that too. I have briefly tested the client and server with two real hosts.
Things done

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

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

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

Oof. Out of date. Ignore.


Discord button footer link

Added the discord button footer link, I named it Discord as it makes the most sense, if you'd like it to be something else let me know. #370

Dunno why that one always fails.

@AlphaSheep Whats happening with this?


Menu to add other defaultOpen dropdown components, triggers their setOpen event

Current behavior
This is like the old #2450 issue, but in a different way 🤔
The regression seems to come from the v0.2.16 onward, and the workaround from #2450 using autoFocusOnHide solve it partially as this time I cannot use element?.focus() to eventually refocus the Add filter menu button.
Steps to reproduce the bug

Open sandbox: https://codesandbox.io/p/sandbox/tos-dash-filtersbar-93dmhl
Try to add a filter through the "Add filter" menu
notice that it tries to add the filter, but immediately close it

Expected behavior
I expect to add a select defaultOpen to remain open without human interaction
Workaround
https://github.com/ariakit/ariakit/issues/2773#issuecomment-1710984357
Possible solutions
No response

Sorry! I've left uncommented the autoFocusOnHide thing, now the bug should be visible

Also, can you reproduce it on this example? https://ariakit.org/examples/select-menu-default-open

I've tried, but in the example is working properly, I don't know why yet, but I suspect that in the example you used the setValues and values props of useMenuStore (in my version I don't use them)

I'm struggling to distinguish between your example and the one on the Ariakit website. Could you possibly modify a fork to mirror your situation, ideally with as few changes as possible?
https://stackblitz.com/run?file=select-menu-default-open%2Fstyle.css,select-menu-default-open%2Ffilter-select.tsx,select-menu-default-open%2Ffilter-menu.tsx,select-menu-default-open%2Fall-filters.ts,select-menu-default-open%2Findex.tsx&theme=dark

Hi! @diegohaz, after several attempts, starting from the ariakit website example, the steps to reproduce the "bug" are:

move the SelectMenu in the same div of the filters array;
the SelectMenu uses SelectItem instead of checkboxes, and the onClick handler in the SelectItem instead of the onValuesChange prop;

Here the fork: https://stackblitz.com/edit/6jtpwr-sxcvpe?file=select-menu-default-open%2Findex.tsx
I suspect that the bug comes from the React render algorithm that behave differently because of points 1 and 2.
I'm curious of your considerations, cheers!

I correct myself, the bug seems to pop out based on SelectMenu order:

SelectMenu before filter list -> no bug
SelectMenu after filter list -> bug

Thanks! I can reproduce it now.
In the meantime, you can either change the order of the elements or delay the onAdd call a bit:
onClick={() => {
  requestAnimationFrame(() => {
    onAdd?.(field)
  });
}}

I'm curious, what was the issue?


delete obsolete cache records

When running dataset-config-names force-refresh for datasets with only one config for https://github.com/huggingface/datasets-server/issues/1550 , I found that for dataset triple-t/dummy, the previous config remains in the db triple-t/dummy.
All cache records related to  triple--t/dummy should have been removed.
See

We could do it after previous job has finished.

Done.
db.cachedResponsesBlue.estimatedDocumentCount()
5720377
db.cachedResponsesBlue.countDocuments({"config": /--/i})
805504
db.cachedResponsesBlue.deleteMany({"config": /--/i})
{ acknowledged: true, deletedCount: 805504 }
db.cachedResponsesBlue.estimatedDocumentCount()
4915107
db.cachedResponsesBlue.countDocuments({"config": /--/i})
0


chimer: use async for chime delay, generalise next-chime logic

@aaronrolls would you mind if I published an MR like this to tweak the busy-loop in chimer? I've also generalised the logic for deciding how long to wait for the next hourly/half-hourly/etc chime, if that looks ok to you?

That looks fine to me. Thanks for trimming down the code.

Thank you, and thanks for the chime idea in the first place, takes me back to a watch I had decades ago!

Thanks!


Start in (optional) Parameter

Hello,
I need to set the Start in (Optional) setting for Windows 7 Task.
Does anyone know how I can perform this task.
Thanks,
Originally posted: 2016-05-13T15:43:14

When creating an ExecAction, add a value to the WorkingDirectory property.
// In constructor, last param is WorkingDirectory
var action = new ExecAction("cmd.exe", "echo Working...", @"C:");
// In object, use property
action.WorkingDirectory = @"C:\Windows";
Originally posted: 2016-05-13T17:42:49


login for demo server not working

The following error message appears: "Invalid username or password!" when using demo:demo on http://demo.mycontroller.org/#/login

@MPParsley fixed. thanks for reportoing this.

You're welcome. Thank you for this very promising project!


The current node list is empty.

Hi,
I have been using this scrapper for a long time.
But since yesterday we are not able to scrap any values from the playstore.
We are getting error "The current node list is empty".
Can you please help us out with this issue.

I'm living same problem.

I am also getting same issue.
Please resolve it ASAP. It's urgent.....
Thanks

I used Goutte Client for scrapping for now.
``  $crawler = $playStoreScraper->request('GET', 'https://play.google.com/store/apps/details?id=hu.viala.newiapp&hl=en');
$result['title'] = $crawler->filter('[itemprop=name]')->text();
$result['image'] = $crawler->filter("[itemprop=image]")->attr('src');
``

I have exactly the same problem!!!
Did Google block this lib or changed they way they structure their data?

Hi, I have the same issue. I'd really appreciate a solution. Thanks a lot!

The google play site structure has changed and it seems they have obfuscated css class names.
Can you check if branch hotfix-15 works for you? At least for app detail scrapping.

Hi,
I am getting the same error.
The current node list is empty.

Hi Is there another hot fix? I tried the hotfix branch and comment out that line but still not work with that error message.

The same issue current node list is empty is occurring today again.
Is there any more fix required?

Same problem

Hi.. When the issue is going to be fixed? Can we have some workaround for this?
@am3do is the latest fix working for you. Any suggestions are welcome.We are  stuck due to this issue

Hi! Unfortunately there are different issues from time to time, so I am not using that code anymore. It’s a pity since the base is really great. It is google who causes the difficulties because of frequent changes in the site structure. So it is more than reasonable that Raul seems to be sick of fixing that again and again.

Hi,
The google play site markup changed again, making it even harder to parse. I've added a new commit to hotfix-15 branch that should work for app scraping.
Unfortunatelly list retrieving is totally broken. I don't have much time to spend in this, so any pull request is welcome.
@am3do, the code mainly uses Symfony's DomCrawler component, I think most of the code could be understood after reading its documentation:
https://symfony.com/doc/current/components/dom_crawler.html
https://api.symfony.com/master/Symfony/Component/DomCrawler/Crawler.html

I moved to node.js ,made a server for google play scrapper which can be consumed by php project over API call.

Hi, @raulr! Thanks you!

Hi,
The hotfix-15 works perfect for me, thank you very much @raulr !!


GH-43986: [C++][Acero] Some code cleanup to Grouper

Rationale for this change
See #43986.

What changes are included in this PR?
Mostly trivial changes, plus removing one Grouper implementation that's not wired.

Are these changes tested?
No new tests needed.

Are there any user-facing changes?
None.

GitHub Issue: #43986

Thanks for the improvements @zanmato1984 !


GEODE-9138: Modified hasSeenEvent to log the message if necessary and…

… increment a stat
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

We can add a bit of documentation  PR : https://github.com/apache/geode/pull/6395