we must return v8::Undefined, the way its now the object will be invalid on the place of the function invocation

Scope & Purpose
The current implementation will crash arangosh while trying to do error handling.

[x] :hankey: Bugfix (requires CHANGELOG entry)
[x] :book: CHANGELOG entry made

Backports:

[x] Backports required for: 3.8, 3.9

Testing & Verification

[x] This change is a trivial rework / code cleanup without any test coverage.
[x] The behavior in this PR was manually tested

* Test "shell_client_2"
    [FAILED]  tests/js/client/shell/shell-collection-counts-cluster.js

      "testWrongCountOnLeaderFullSync2MultipleFollowers_OldFormat" failed: Error: at assertion #14: assertEqual: (200) is not equal to (100)
(Error
    at assertEqual (/work/ArangoDB/js/common/modules/jsunity/jsunity.js:108:19)
    at Object.testWrongCountOnLeaderFullSync2MultipleFollowers (tests/js/client/shell/shell-collection-counts-cluster.js:497:7)

definitely unrelated, replication thing broken.


Allow more aggressive transform caching in multiproject monorepos

🐛 Bug Report
In multiproject configs, each project root gets its own jest transform cache. This leads to duplicate work transforming the same files, even if they use the same configs. If a file is used in n projects, it will be transformed n times.
Locally, this increases cache disk usage. In CI, where the cache will not be warm, it increases runtime as duplicate work is required.
Writing a custom transformer with a simpler cache key implementation does not solve this - each project gets a separate cache folder.
The relevant code that does this is in https://github.com/facebook/jest/blob/c98b22097cb6faa3ed3fabf197cbe4f466620b9f/packages/jest-transform/src/ScriptTransformer.ts#L132-L136 - forces a unique cache path per config.name
If unassigned, config.name is assigned to a hash based on the path and index.
I've tried adding a common name to all projects' jest configs. That fixes the transform problem, but breaks other things (manual mocks in an __mocks__ folder don't work consistently). On our large monorepo, this gave a ~30% improvement in total runtime, but __mocks__ becoming unpredictable
I appreciate that there are edge cases to handle here (potentially different jest configs could warrant a different cache), but I think it should be available for the jest transformer to decide whether this is important (e.g. if relevant, a transformer could include config.name in the cache key manually).
e.g. optionally allow transformers to provide their own implementations of getCacheFilePath, which overrides the use of HasteMap.getCacheFilePath(       this._config.cacheDirectory,       'jest-transform-cache-' + this._config.name,       VERSION,     )
If this sort of change would be accepted, I can probably provide a PR.
To Reproduce
Steps to reproduce the behavior:

Set up a multiproject config
Run tests with a cleared cache
Either observe the cache on disk, or tap into the transform to count the number of times a file is transformed

Expected behavior
If the transform config is the same, each file is only transformed once
Link to repl or repo (highly encouraged)
https://github.com/lexanth/jest-projects-repro
This is a monorepo with 3 packages (A, B and C). A and B consume C. the code in C currently gets transformed once per package, even with the transformer (in the jest-preset package) giving a super aggressive cache key implementation (yarn test:ci - could be used e.g. in CI, if we know the other relevant configs are constant).
Adding name: process.env.USE_SIMPLIFIED_CACHE ? '_' : undefined to each package's jest config makes them all use the same cache, but in my actual repo breaks other things, being a bit of a hack.
Everything is running in band because the tests are so fast that multiple workers all start transforming before another can populate the cache anyway.
envinfo
  System:
    OS: macOS 10.15.7
    CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
  Binaries:
    Node: 12.18.1 - ~/.nvm/versions/node/v12.18.1/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.5 - ~/.nvm/versions/node/v12.18.1/bin/npm
  npmPackages:
    jest: ^26.6.3 => 26.6.3

Allowing transformers to override caching directory seems fine to me. @cpojer @thymikee @jeysal thoughts?
https://github.com/facebook/jest/blob/a66eec74b55dc98ab072be330c13c54c35cba5f7/packages/jest-transform/src/ScriptTransformer.ts#L128-L132

I think we can make transformers smarter about when to bust transform cache, by picking only configuration that changes the output, instead of relying on one-size-fits-all solutions like stringified config or project name. If it's done well, overriding from transformers shouldn't be necessary.

Agreed with @thymikee, anything else is hacky.

@thymikee At the moment in a multi-project run, transformers can't be smart about it - jest will always split the cache into separate caches per project.
We could either remove this._config.name from the calculation of the cache path (and leave it all down to the transformer), or allow the transformer to override the cache folder here.
Happy to also look at a follow up change to the babel-jest transformer to make the cache key only depend on the relevant parts of the jest config. For comparison, ts-jest looks to be a bit smarter here about what bits of config to use - https://github.com/kulshekhar/ts-jest/blob/master/src/index.ts

I don't understand the "let's be smarter about busting in transformers" comments - even if the cache key is the same it's a cache miss since Jest will look in different directories for different projects when checking if the cached file exist. If you by "instead of relying on one-size-fits-all solutions like stringified config or project name" mean "remove project name from the algorithm" that has nothing to do with the transformers themselves. That code lives in @jest/transform. I'm down with just removing that part of it which should solve it as we'll be trusting the cache key from transformers.

We could either remove this._config.name from the calculation of the cache path (and leave it all down to the transformer),

That's what I'm after.

I'm down with just removing that part of it which should solve it as we'll be trusting the cache key from transformers.

Kinda related to my comment here: https://github.com/facebook/jest/pull/10834#discussion_r524115961 I'm all in for that 👍

even if the cache key is the same it's a cache miss since Jest will look in different directories for different projects when checking if the cached file exist

Let's make it so the cache directory is the same across the root project. We should be able to do that when getCacheKey is properly calculated based on dependencies of a transformer.

Works for me 👍 Introduces race conditions between projects (in theory alleviated through write-file-atomic) and makes getCacheKey more important, but I think that's fine. If a transformer does not define getCacheKey the default implementation takes the whole config string, which makes sense to me: https://github.com/facebook/jest/blob/a66eec74b55dc98ab072be330c13c54c35cba5f7/packages/jest-transform/src/ScriptTransformer.ts#L113-L119. We could also deprecate falling back to default and say "pls implement one", if nothing else just using the new @jest/create-cache-key-function package

Sounds like a plan. I'd be up for adding a deprecation warning in Jest 27 and keep it for a while until community adapts, which may take a while. Then remove. Btw, why are we not using @jest/create-cache-key-function as a default now? Sounds like a good case for some reuse :D

Btw, why are we not using @jest/create-cache-key-function as a default now? Seems like a good case for some reuse :D

Dunno. PR welcome? 😀

One further thought on this - will just removing the this._config.name will result in all jest projects (as in completely separate ones, rather than multiproject) sharing a cache directory? I presume that's where this came from originally.

Sounds like a plan. I'd be up for adding a deprecation warning in Jest 27 and keep it for a while until community adapts, which may take a while. Then remove. Btw, why are we not using @jest/create-cache-key-function as a default now? Seems like a good case for some reuse :D

One cent on @jest/create-cache-key-function, I like this package but turned out causing coverage issue that I had to revert to the cache key constructing way of babel-jest. Either I didn’t use it correctly or something wrong with the package.


Approve and merge successful dependabot PRs

Approve PRs from dependabot and merge if the change is a minor or patch semver bump

Codecov Report

Merging #367 (6782294) into main (ba3e311) will decrease coverage by 20.20%.
Report is 1 commits behind head on main.
 Consider uploading reports for the commit 4cc5da3 to get more accurate results

@@             Coverage Diff             @@
##             main     #367       +/-   ##
===========================================
- Coverage   85.87%   65.67%   -20.20%
===========================================
  Files          27       27
  Lines         906      906
===========================================
- Hits          778      595      -183
- Misses        128      311      +183

Flag
Coverage Δ

api
65.67% <ø> (ø)

unit
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 10 files with indirect coverage changes


Feature request. page:before-change on back button

Hi, sorry for bad English. I didn't find mailing list that's why my question here. Is there a plans to allow to prevent clicking back button as it were made with links (page:before-change)? It will be usefull in situations with plugins like "dirty form" or "are you sure" to prevent loosing partially filled forms during redirect.

Is it there any workaround for this? Really need to intercept the back button

The popstate event can't be canceled.  And it makes sense why browsers implement it that way.  The user has explicitly told the browser they want to go back to the previous page.  A website shouldn't be able to prevent that from happening.
I'm going to close this, for the reason I just described and also the fact that there's no apparent way to implement it.

My workaround was adding to the history stack with turbolinks set to false:
var url = 'fakeurl.com/url#step-1'
history.pushState({turbolinks: false, url: url}, '', url);
I then added a hash on the id link of my multi-step form where I navigated to
window.location.hash = 'step-2';
and set up a listener to fade the form in and out based on the hash change:
window.onhashchange = checkHashChange;
var checkHashChange = function() {
doSomething();
}


Dismiss on Scroll

Option to dismiss the cookie policy on Scroll

The project has undergone a significant rewrite, so this pull request will no longer work.
You may be happy to learn that CookieConsent V3 comes with a dismissOnScroll option by default, where you pass a scrollTop value that must be exceeded before consent is triggered


Xbox 360 not detected, linux

Hi, I'm using the latest Ubuntu, forge, and minecraft Java edition. I've just downloaded the latest controllable version.
In the settings nothing appears in the select a controller window. The controller was on when launching minecraft and it tests perfectly on game pad tester. I use these controllers to play on multiple emulators so I know they work. They are Xbox 360 wireless controllers with a wifi dongle.
Is there something I could have done wrong to get this result?

I'm using MultiMC to load Forge, the Controllable mod, and launch Minecraft on Linux Mint, also with an Xbox 360 controller and wireless USB dongle. Everything seems to work, except attacking with a sword (which is a known issue, I think addressed in the next update, 1.18.2). I had to grab MultiMC deb file from their website (also JDK 17 from Oracle). I'm still learning Mint, and I'm sure there's a way to get it from repositories--I just found the website d/l first.
Just providing another Linux perspective in case anything I did helps. (Where did you get Forge?)

Same issue here, Using Pop_OS! 22.04
Minecraft is installed via Flatpak from Pop_Shop, also tried the binary file from minecraft.net
Using an Xbox S Controller, which works in many games and is recognized by my system, but the list in the mod is empty

Fixed it!
Works after adding the binary launcher to steam, getting into Big Picture Mode, allowing the Controller for the game, then starting the game over big picture and it should show up as steam virtual gamepad

Fixed it!
Works after adding the binary launcher to steam, getting into Big Picture Mode, allowing the Controller for the game, then starting the game over big picture and it should show up as steam virtual gamepad

That's a workaround not a actual fix.. this issue still needs to be resolved for Linux, I have the same issue on Manjaro Gnome and even on Steam big picture mode it isn't working. It can see the Steam input but it doesn't work so this issue a workable work around.

I've written some guides on How to Connect a Gamepad. Check out the Linux section.
https://mrcrayfish.gitbook.io/controllable-documentation/guides/how-to-connect-a-gamepad

I've written some guides on How to Connect a Gamepad. Check out the Linux section. https://mrcrayfish.gitbook.io/controllable-documentation/guides/how-to-connect-a-gamepad

That's great but that doesn't resolve the issue

@coreybruce Did you follow the Steam Virtual Gamepad guide? Make sure the controller is enabled in Steam too. At what point in the guide did the steps not work? I'm asking because there is no "fix".

Yes I reported back in Oct 22, 2022, I can now see virtual controller the left joystick isn't working so I can't move around.

"Why does ti work on Windows properly but not on Linux?"
Because Microsoft natively supports Xbox controllers on Windows. It's pretty easy to understand.
"I shouldn't need Steam virtual input to work when any other game works fine"
Any game you play from Steam with a controller will use Steam Input.
Also it sounds to me that the mappings are out of date. The mappings available in each release Controllable are the ones that were available at the time of release. A 3 year old version of Controllable will have 3 year out of date mappings. This means gamepads release after that may not be recognized correctly. You can find a guide here to update them:
https://mrcrayfish.gitbook.io/controllable-documentation/troubleshooting/solving-incorrect-mappings
Otherwise I still suggest Steam Virtual Gamepad method if you're playing on Linux, it doesn't sound like you tried it since I only wrote the guide over a month ago, not last year.
https://mrcrayfish.gitbook.io/controllable-documentation/guides/how-to-connect-a-gamepad/method-4-steam-virtual-gamepad


lost functionality after aurelia update (Shadow DOM changes?)

Hi all.
Today I've updated the aurelia packages to the newest version. I've lost some of functionalities in a system which is written in Aurelia.
Currently I've found only one problem:
I have a templates creator where I can add some UI elements to create a questionnaire. The flow was simple got to creator and add some stuff. It handled also edition o exisiting templates. There is meny custom-elements inside ans some custom attributes. The logic and HTML is quite complicated. After update the functionality is not working at all. Nothing is displayed inside the creator - I have there a if.bind logic where I decide what to show.
I noticed that the code in HTML don't "see" a parent scope inside its scope any more. I used it to call methods on parent scope or to get properties. I know it was bad, but it was quick&dirty solution.
Is it something with Shadow DOM?

It's nothing to do with shadow dom. It has to do with Aurelia Components. Aurelia's components are encapsulated which means they cannot see the binding context from outside of the component. There was a regression in encapsulation that temporarily made it possible to see that outside scope. That bug has been fixed to property encapsulate components. To solve your problem, create bindable properties on your components and pass in the data from the outer context. This will make things more explicit in your architecture.


fractal: add gtk support to gstreamer

Closes #96618
Suggested-by: @jtojnar
Motivation for this change
When fractal crashes I can't chat.
Things done

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

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

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

CC maintainers: @dtzWill @worldofpeace

Thanks.


Dictionary.GetOrCreate extension method

Needed this for something in the main repository and thought it may be useful to other framework users as well. Allows you to pass a method to generate a fallback value in case the lookup key was not present in the dictionary before. The generated value will be added to the dictionary.

One thing to note is that this should not be used in any high performance scenarios due to delegate allocations.
Not sure I see the need for this, especially as an extension method (it’s almost impossible for a consumer to find that this exists). Am neutral on it being merged.

If you don't think it is necessary, feel free to close the PR. I'm using a variant of this method in many of my projects, but I agree that most users probably wouldn't be aware of its existence.

Please rename the method to ensuredirectory, because that name is acommenly used one for the functionality

Closing as unneeded


If pyproject.toml does not have a project attribute, an error will be reported

Traceback (most recent call last):
  File "/opt/hostedtoolcache/node/18.18.2/x64/lib/node_modules/semantic-release-pypi/dist/py/set_version.py", line 24, in <module>
    toml['project']['version'] = args.version
  File "/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/site-packages/tomlkit/container.py", line 613, in __getitem__
    raise NonExistentKey(key)
tomlkit.exceptions.NonExistentKey: 'Key "project" does not exist.'

Hey @leeqvip
semantic-release-pypi needs a project or a tool.poetry section to set the version.
If you have neither in your pyproject.toml file, this is the expected error.
Your PR #32 would get rid of the error, but the version would no longer be updated.
Please explain to me why you need this?

In my project, pyproject.toml is only used for lint checking, and the version is still configured in setup.cfg

I see. I can think of two solutions

add a new option, which allows the user to force set the build interface (pyproject.toml or setup.py)
use<EMAIL_ADDRESS>- this version will always set the version in setup.cfg

I'm happy to make a PR on pycasbin to fix the release workflow if you'd like.

I'm happy to make a PR on pycasbin to fix the release workflow if you'd like.

Thanks !


Add swift-server CI

Motivation
We want to use the swift-server CI going forward. For this we need to add a bunch of docker magic to work.
Changes

Add docker-compose files to run tests in swift-server ci

Open ends
Some of the CI jobs will fail. We should enable it and one by one make it green. First one is already posted: #52.

@fabianfett how do builds get triggered?

@swift-server-bot test this please

@Joannis CI should be triggered automatically once I set them up, or they can be force-triggered using "test this please" comment (e.g., https://github.com/swift-server/RediStack/pull/54#issuecomment-1594935859)

CI fails as expected. Will fix with follow up PR #52.


fix: use provided gas price in env

Motivation
Closes #1143

Solution
gas_price will be 0 by default, if it's manually set, then this will now also be set in the Env used when executing an transaction

fixed now, will need a separate review

Hello,
I just fetched the newest forge and it seems that the gas_price parameter in foundry.toml is ignored. Current mainnet gas price is used instead.
My foundry.toml:
[default] src = 'src' out = 'out' libs = ['lib'] remappings = ['ds-test/=lib/ds-test/src/', 'forge-std/=lib/forge-std/src/'] eth-rpc-url = 'RPC_URL' fork_block_number = 14411926 gas_price =<PHONE_NUMBER>0

@mattsse wanna take a look when you have time?^

yeah, we're probably not setting the limit when creating the env for the fork,
@onbjerg I think there was a related issue, but I can't remember


refactor: magic bytes to constants op codes

Pull Request Overview
This pull request changes the way we handle opcodes: from using magic bytes to using constants.
Testing Strategy
This pull request was tested locally.
TODO or Help Wanted
N/A
Formatting

[x] Ran cargo fmt
[x] Ran cargo check
[x] Ran cargo build
[ ] Ran nix fmt
[ ] Ran treefmt

Github Issue
N/A
Author
Signed-off-by: Popescu Adrian<EMAIL_ADDRESS>

Closed in favor of https://github.com/DLR-FT/wasm-interpreter/pull/23


Exception when creating new card

Anonymous user posted:

Hi Greg I'm getting the exception below when I try to create a new card. Here is the relevant part of my code: ... TrelloConfiguration.Log = new TrelloLog(); var serializer = new ManateeSerializer(); TrelloConfiguration.Serializer = serializer; TrelloConfiguration.Deserializer = serializer; TrelloConfiguration.JsonFactory = new ManateeFactory(); TrelloConfiguration.RestClientProvider = new WebApiClientProvider(); TrelloAuthorization.Default.AppKey = trelloKey; TrelloAuthorization.Default.UserToken = trelloToken; ... var list = new List(listId); var card = list.Cards.Add("Test Card"); (listId is a string containing the Id of the list where I want to add the card). This is the output: ... 2016-02-15 09:42:54.5589|Info|Queuing: Post cards 2016-02-15 09:42:54.5589|Info|Sending: Post cards 2016-02-15 09:42:55.0819|Error|Manatee.Trello.Exceptions.TrelloInteractionException: Trello has reported an error with the request. ---> System.AggregateException: One or more errors occurred. ---> System.Net.Http.UnsupportedMediaTypeException: No MediaTypeFormatter is available to read an object of type 'IJsonCard' from content with media type 'text/plain'. at System.Net.Http.HttpContentExtensions.ReadAsAsync[T](HttpContent content, Type type, IEnumerable1 formatters, IFormatterLogger formatterLogger, CancellationToken cancellationToken) at System.Net.Http.HttpContentExtensions.ReadAsAsync[T](HttpContent content, Type type, IEnumerable1 formatters, IFormatterLogger formatterLogger) at System.Net.Http.HttpContentExtensions.ReadAsAsync[T](HttpContent content, IEnumerable1 formatters) at Manatee.Trello.WebApi.WebApiClient.<ExecuteAsync>d__e1.MoveNext() in c:\Users\gdennis\OneDrive\Projects\Manatee.Trello\Manatee.Trello.WebApi\WebApiClient.cs:line 85 --- End of inner exception stack trace --- at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions) at System.Threading.Tasks.Task1.GetResultCore(Boolean waitCompletionNotification) at System.Threading.Tasks.Task1.get_Result() at Manatee.Trello.WebApi.WebApiClient.Execute[T](IRestRequest request) in c:\Users\gdennis\OneDrive\Projects\Manatee.Trello\Manatee.Trello.WebApi\WebApiClient.cs:line 25 at Manatee.Trello.Internal.RequestProcessing.QueuableRestRequest1.Execute(IRestClient client) in C:\Users\Greg\OneDrive\Projects\Manatee.Trello\Manatee.Trello\Internal\RequestProcessing\QueuableRestRequest.cs:line 68 at Manatee.Trello.Internal.RequestProcessing.RestRequestProcessor.Execute(IRestClient client, QueuableRestRequest queuableRequest) in C:\Users\Greg\OneDrive\Projects\Manatee.Trello\Manatee.Trello\Internal\RequestProcessing\RestRequestProcessor.cs:line 127 --- End of inner exception stack trace --- 2016-02-15 09:42:55.0819|Error|Manatee.Trello.Exceptions.TrelloInteractionException: Trello has reported an error with the request. ---> System.AggregateException: One or more errors occurred. ---> System.Net.Http.UnsupportedMediaTypeException: No MediaTypeFormatter is available to read an object of type 'IJsonCard' from content with media type 'text/plain'. at System.Net.Http.HttpContentExtensions.ReadAsAsync[T](HttpContent content, Type type, IEnumerable1 formatters, IFormatterLogger formatterLogger, CancellationToken cancellationToken) at System.Net.Http.HttpContentExtensions.ReadAsAsync[T](HttpContent content, Type type, IEnumerable1 formatters, IFormatterLogger formatterLogger) at System.Net.Http.HttpContentExtensions.ReadAsAsync[T](HttpContent content, IEnumerable1 formatters) at Manatee.Trello.WebApi.WebApiClient.d__e1.MoveNext() in c:\Users\gdennis\OneDrive\Projects\Manatee.Trello\Manatee.Trello.WebApi\WebApiClient.cs:line 85 --- End of inner exception stack trace --- at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions) at System.Threading.Tasks.Task1.GetResultCore(Boolean waitCompletionNotification) at System.Threading.Tasks.Task1.get_Result() at Manatee.Trello.WebApi.WebApiClient.Execute[T](IRestRequest request) in c:\Users\gdennis\OneDrive\Projects\Manatee.Trello\Manatee.Trello.WebApi\WebApiClient.cs:line 25 at Manatee.Trello.Internal.RequestProcessing.QueuableRestRequest1.Execute(IRestClient client) in C:\Users\Greg\OneDrive\Projects\Manatee.Trello\Manatee.Trello\Internal\RequestProcessing\QueuableRestRequest.cs:line 68 at Manatee.Trello.Internal.RequestProcessing.RestRequestProcessor.Execute(IRestClient client, QueuableRestRequest queuableRequest) in C:\Users\Greg\OneDrive\Projects\Manatee.Trello\Manatee.Trello\Internal\RequestProcessing\RestRequestProcessor.cs:line 127 --- End of inner exception stack trace --- Packages:        Am I missing anything? Thanks for your help. Regards, Romain (P.S.: My Trello user is rmeisterhans)

Romain Meisterhans (Meisty on Bitbucket) wrote:

Here is the bug reported formatted correctly in a text file

trello_issue.txt

I've seen this when trying to connect to the API without a valid token or some other API issue. Unfortunately, WebAPI is throwing the exception before I can log the contents of the actual message. Can you capture the call with Fiddler (or equivalent) and post the message content, please?
Regarding the token, make sure that it hasn't expired and try again.

Romain Meisterhans (Meisty on Bitbucket) wrote:

Hi Greg

Thanks for your response. The problem was with the token. I will need to write some code to get a new token each time.

Glad that worked for you. If your security model allows it, you can try getting a non-expiring token and persisting it. Otherwise, yes, you'll have to get a new token each time.


Flutter tool "crashed" instead of displaying error when iOS Simulator was closed

I had my iOS Simulator closed (maybe I closed it while flutter run was starting?)
And instead of a nice message I got a crash log:
Launching lib/main.dart on iPhone 6 in debug mode...
Running Xcode build...                               14.4s

Oops; flutter has exited unexpectedly: "Exit code 53 from: /usr/bin/xcrun simctl install 80FD76E9-4422-479B-B503-2EFE5C622125 /src/bug/build/ios/iphonesimulator/Runner.app:
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=53):
Error returned in reply: Connection interrupted
Software caused connection abort".
Sending crash report to Google.
Crash report sent (report ID: bc87b60e323035df)
Crash report written to /src/bug/flutter_01.log;
please let us know at https://github.com/flutter/flutter/issues.

command
flutter run
exception
String: Exit code 53 from: /usr/bin/xcrun simctl install 80FD76E9-4422-479B-B503-2EFE5C622125 /src/bug/build/ios/iphonesimulator/Runner.app:
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=53):
Error returned in reply: Connection interrupted
Software caused connection abort
#0      runCheckedAsync (package:flutter_tools/src/base/process.dart:242)
<asynchronous suspension>
#1      SimControl.install (package:flutter_tools/src/ios/simulators.dart:129)
#2      IOSSimulator._buildAndInstallApplicationBundle (package:flutter_tools/src/ios/simulators.dart:419)
<asynchronous suspension>
#3      IOSSimulator._setupUpdatedApplicationBundle (package:flutter_tools/src/ios/simulators.dart:401)
<asynchronous suspension>
#4      IOSSimulator.startApp (package:flutter_tools/src/ios/simulators.dart:323)
<asynchronous suspension>
#5      FlutterDevice.runHot (package:flutter_tools/src/resident_runner.dart:267)
<asynchronous suspension>
#6      HotRunner.run (package:flutter_tools/src/run_hot.dart:211)
<asynchronous suspension>
#7      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:348)
<asynchronous suspension>
#8      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:253)
<asynchronous suspension>
#9      FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:193)
<asynchronous suspension>
#10     CommandRunner.runCommand (package:args/command_runner.dart:194)
<asynchronous suspension>
#11     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:279)
<asynchronous suspension>
#12     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:109)
#13     new Future.sync (dart:async/future.dart:222)
#14     CommandRunner.run (package:args/command_runner.dart:109)
#15     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:162)
#16     run.<anonymous closure> (package:flutter_tools/executable.dart:145)
<asynchronous suspension>
#17     AppContext._run (package:flutter_tools/src/base/context.dart:76)
<asynchronous suspension>
#18     AppContext.runInZone.<anonymous closure> (package:flutter_tools/src/base/context.dart:66)
#19     _rootRun (dart:async/zone.dart:1120)
#20     _CustomZone.run (dart:async/zone.dart:1001)
#21     runZoned (dart:async/zone.dart:1467)
#22     AppContext.runInZone (package:flutter_tools/src/base/context.dart:65)
#23     run (package:flutter_tools/executable.dart:116)
<asynchronous suspension>
#24     main (package:flutter_tools/executable.dart:68)
<asynchronous suspension>
#25     main (file:///src/flutter/packages/flutter_tools/bin/flutter_tools.dart:16)
#26     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:263)
#27     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:151)

flutter doctor
[✓] Flutter (on Mac OS X 10.12.6 16G29, locale en-US, channel master)
    • Flutter at /src/flutter
    • Framework revision 154f2e5402 (3 hours ago), 2017-10-26 09:55:36 -0700
    • Engine revision ec19da1c80
    • Tools Dart version 1.25.0-dev.11.0

[✓] Android toolchain - develop for Android devices (Android SDK 26.0.2)
    • Android SDK at /Users/eseidel/Library/Android/sdk
    • Platform android-26, build-tools 26.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] iOS toolchain - develop for iOS devices (Xcode 8.3.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.3.3, Build version 8E3004b
    • ios-deploy 1.9.2
    • CocoaPods version 1.3.1

[✓] Android Studio (version 2.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Community Edition (version 2017.2.5)
    • Flutter plugin version 17.0
    • Dart plugin version 172.4155.35

[✓] Connected devices
    • iPhone 6 • 80FD76E9-4422-479B-B503-2EFE5C622125 • ios • iOS 10.3 (simulator)

This has since been fixed


Wendy/eng 2490 fix jiggling of detailspaneltimestamps updating loader

https://github.com/systeminit/si/assets/8495209/a5135fdc-9a54-4b8c-8b82-28d9085e2e6b

bors merge


Add support for ARM64 architecture

Add support for ARM64 architectures using Box64.
This change is based on the https://github.com/thijsvanloef/palworld-server-docker which supports ARM64 using BOX64.
The AMD64 build stays intact. The only general changes are:

Exposing the server port at the Docker Compose example file.
Add Depot Downloaded as an alternative to steamcmd for machines not compatible with x86. It can be configured at the environmental variables.
Force the download of the dedicated server to Linux and x64 architecture.

resolve #45

As ARM64_DEVICE is an environment variable it should not be required to set it in the docker file or? It should work if the variable is set via docker-compose or -e on startup of the container. Or did I miss some point?

I have tested by removing it at the image, so it uses generic and the set it to adlink as my systems is an Oracle one. Everything is working so far. I will remove the variable from there and write in the README that it needs to be set there.

Just tried to build the amd64 docker file on my Linux host an on Windows (WSL). Unfortunately, there are two errors. The arm64 build works on raspberry. (I only can build but not test on the raspberry, as raspberry 4 seems to be too weak)
On Linux:

Sending build context to Docker daemon  214.5kB
Step 1/27 : FROM cm2network/steamcmd:root AS base-amd64
 ---> 948fa5048611
Step 2/27 : FROM --platform=arm64 sonroyaalmerol/steamcmd-arm64:root-2024-11-24 AS base-arm64
 ---> bcf59c9d3c4a
Step 3/27 : ARG TARGETARCH
 ---> Using cache
 ---> 05c63c132527
Step 4/27 : FROM base-${TARGETARCH}
invalid reference format

I will need to check if this is cause by some host system problems. 🤔
On Windows (WSL):
 > [stage-2 4/8] RUN case "${TARGETARCH}" in         "amd64") DEPOT_DOWNLOADER_FILENAME=DepotDownloader-linux-x64.zip ;;         "arm64") DEPOT_DOWNLOADER_FILENAME=DepotDownloader-linux-arm64.zip ;;     esac     && wget --progress=dot:giga "https://github.com/SteamRE/DepotDownloader/releases/download/DepotDownloader_2.7.4/${DEPOT_DOWNLOADER_FILENAME}" -O DepotDownloader.zip     && unzip DepotDownloader.zip     && rm -rf DepotDownloader.xml     && chmod +x DepotDownloader     && mv DepotDownloader /usr/local/bin/DepotDownloader:
0.423 /bin/sh: 1: wget: not found

May wget is missing in the base image and must be installed.

I have added wget for the normal build as the image doesn't have it.
Also fix the error where TARGETARCH becomes empty after using the build image at line:
FROM base-${TARGETARCH}

I have tested it on an AMD64 machine and build it correctly.

I have added wget for the normal build as the image doesn't have it.
Also fix the error where TARGETARCH becomes empty after using the build image at line:
FROM base-${TARGETARCH}

I have tested it on an AMD64 machine and build it correctly.

Thank you.
The amd64 build works perfekt. Just tried build and run on my windows (WSL / Debian) host.
The arm64 build als works fine. (Im unable to run, but this is a problem of the raspberry 4)

I have added wget for the normal build as the image doesn't have it.
Also fix the error where TARGETARCH becomes empty after using the build image at line:
FROM base-${TARGETARCH}

I have tested it on an AMD64 machine and build it correctly.

Thank you.
The amd64 build works perfekt. Just tried build and run on my windows (WSL / Debian) host. The arm64 build als works fine. (Im unable to run, but this is a problem of the raspberry 4)
Edit: The build on my Debian host still throws invalid reference format I will try to set up a virtual machine later. But I think this is more a local problem.
Edit: I can verify the amd64 build and run on a fresh Debian installation (virtual machine)👍

The ARM64 image that is uses is only compatible with Raspberry Pi 5 as mention here: https://github.com/sonroyaalmerol/steamcmd-arm64?tab=readme-ov-file#box64-builds
You can create a Free forever Oracle ARM instance and use it there as I do.

Hello @feserr thank you for this PR. LGTM, are we good to go?
Thank you @Micke90s too, as always.

Hello @feserr thank you for this PR. LGTM, are we good to go?
Thank you @Micke90s too, as always.

Sure, it requires a reviewer approval. Who should do it?

The ARM64 image that is uses is only compatible with Raspberry Pi 5 as mention here: https://github.com/sonroyaalmerol/steamcmd-arm64?tab=readme-ov-file#box64-builds

@feserr Thank you. I missed that. Now I know why all my implementations failed 🤣 At least tests looked similar to your solution, so I wasn’t wrong completely. 😄

You can create a Free forever Oracle ARM instance and use it there as I do.
@feserr Thank you. Sounds very interesting. I will take a look on this.

Thank you @Micke90s too, as always.
@arguser You're welcome.

I wish you a nice weekend 😄


load graph into blazegraph endpoint

Load graph into a blazegraph endpoint similar to what is done for kg-covid-19:
Making blazegraph journal
Loading blazegraph journal
One idea would be to load eco-kg into the same blazegraph instance where kg-covid-19 is loaded.

In KG-construction meeting, we discussed possibly Anne using neo4j locally instead of blazegraph


Issue with UsernamePasswordCredential

The current sdk implementation seems to only take tenantId, clientId, userName, and userPwd. There is no way to pass client_secret. The underlying call to getToken also doesn't use or expect client_secret.
But I get the following error sdk. I tried the same from Postman and it works if I supply client_secret there.
AADSTS7000218: The request body must contain the following parameter: 'client_assertion' or 'client_secret'.
Trace ID: 5287fa68-f789-4399-b246-0c0d6d703000
Correlation ID: 48d9aa86-cf93-4988-b3e4-0e2139e5004d
Timestamp: 2021-08-21 19:51:53Z
at IdentityClient. (D:\My\projects\nodejs\azgettoken\node_modules@azure\identity\dist\index.js:402:31)
at Generator.next ()
at fulfilled (D:\My\projects\nodejs\azgettoken\node_modules\tslib\tslib.js:114:62)
at processTicksAndRejections (internal/process/task_queues.js:95:5) {
statusCode: 401,
errorResponse: {
error: 'invalid_client',
errorDescription: "AADSTS7000218: The request body must contain the following parameter: 'client_assertion' or 'client_secret'.\r\n" +
Version: "@azure/identity": "^1.5.1"

Hello, @kannanMSFT !
I’m seeing two ways to answer to your question. In order to provide two clear answers, based on what I’m understanding, I will rephrase your question in two different ways:

Why won’t the UsernamePasswordCredential accept a client secret?
Why am I getting the error The request body must contain the following parameter: 'client_assertion' or 'client_secret'?

My answers:
Why won’t the UsernamePasswordCredential accept a client secret?
I’m currently asking my team for feedback regarding your issue. We currently do not allow specifying a client secret in this credential. If you want to specify a client secret, you can use the ClientSecretCredential, and you won’t need a password in that case.
Why am I getting the error: The request body must contain the following parameter: 'client_assertion' or 'client_secret'?
This happens because of how the AAD application is configured. Your AAD app registration seems to be configured as a confidential application.
Let’s check something first:

Go to the Azure Portal.
Go to the “Manifest” page.
See if the property allowPublicClient is set to true.

My intuition tells me that, for your current issue, this setting must be set to false.
So, how does one allow public client authentication on your AAD?

Go to the Azure Portal.
Go to the “Authentication” page.
Scroll to the bottom of this page, and you will see something that says: Allow public client flows, and close to that, you will see a yes and no switch. Make sure this is set to yes.

After that, you should not need to specify a client secret to authenticate with this credential.

While I come back with more information, please try with the approaches above and, when you find some time, answer again with your feedback. Do these approaches work for you?
Your time is appreciated!

Thanks for detailed explanation @sadasant . Yes by turning on 'Allow public client flows' it works without the need for client secret. So does this mean UsernamePasswordCredential doesn't support confidential clients ?

That is correct. It currently does not support confidential clients! I’ll follow up as soon as possible regarding whether we are planning to support confidential clients on this credential or not.
Daniel,

From: kannanMSFT @.>
Sent: Monday, August 23, 2021 8:44:01 PM
To: Azure/azure-sdk-for-js @.>
Cc: Daniel Rodríguez @.>; Mention @.>
Subject: Re: [Azure/azure-sdk-for-js] Issue with UsernamePasswordCredential (#17021)
Thanks for detailed explanation @sadasanthttps://github.com/sadasant . Yes by turning on 'Allow public client flows' it works without the need for client secret. So does this mean UsernamePasswordCredential doesn't support confidential clients ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-sdk-for-js/issues/17021#issuecomment-904231737, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AADFZ6EHX4NTPPIG6OD6L6DT6LTNDANCNFSM5CSE4TDA.

@kannanMSFT hello hello. Please help me understand:

Is it ok for you to use the UsernamePasswordCredential with only public clients?
Would the ClientSecretCredential be a good alternative for you for your confidential client cases?

We want to understand what would be the reasons to use UsernamePasswordCredential for confidential clients, instead of ClientSecretCredential.

Is it ok for you to use the UsernamePasswordCredential with only public clients?

For my use case of running tests I guess its okay.

Would the ClientSecretCredential be a good alternative for you for your confidential client cases?

The only reason I was trying use UsernamePasswordCredential was because I was trying to get the token for a particular existing user.
Btw, feel free to close this case. I kept it open so anyone who later has these questions can learn about the intent / context.

@kannanMSFT thank you very much!


No matching distribution found for Tensorflow for pip-20.3

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution: Arch Linux
TensorFlow version: N/A (pip-20.3 can't install any version of Tensorflow)
Python version: 3.9.0
CUDA/cuDNN version: N/A (running on CPU)
GPU model and memory: N/A (running on CPU)

Describe the problem
When trying to install Tensorflow from pip-20.3, pip says:

ERROR: Could not find a version that satisfies the requirement tensorflow (from versions: none)
ERROR: No matching distribution found for tensorflow

Provide the exact sequence of commands / steps that you executed before running into the problem

pip install tensorflow
pip3 install tensorflow
pip3.9 install tensorflow
(All of these commands yield the same error message)

Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@BearNinja123,
Currently TensorFlow is comptible with upto Python v3.8. Please take a look at the tested build configurations for more information.

Version
Python version
Compiler
Build tools

tensorflow-2.3.0
3.5-3.8
GCC 7.3.1
Bazel 3.1.0

tensorflow-2.2.0
3.5-3.8
GCC 7.3.1
Bazel 2.0.0

tensorflow-2.1.0
2.7, 3.5-3.7
GCC 7.3.1
Bazel 0.27.1

The request for TensorFlow support on Python 3.9 is already being tracked in #44485, please follow that thread for more information. Thanks!


loading specs: [Errno 2] No such file or directory

After running dusty config set specs_repo github.com/gamechanger/dusty-specs-repo and dusty repos update I get:
Pulling latest updates for all active managed repos:
Updating managed copy of specs-repo before loading specs
ERROR: [Errno 2] No such file or directory

Completed preflight check successfully and ran setup with the defaults.
$ dusty repos list
+-----------------------------------------+------------------+----------------+
|                Full Name                |    Short Name    | Local Override |
+-----------------------------------------+------------------+----------------+
| github.com/gamechanger/dusty-specs-repo | dusty-specs-repo |                |
+-----------------------------------------+------------------+----------------+

I have fresh installs of nginx, boot2docker and docker-compose (didn't have them previously).  Did that, ran the install script and rebooted.
Looks like a great tool... if I can get it running

Full log from install:
Testing dusty daemon's preflight...
2015-07-07 23:51:18 INFO:root Starting preflight check
2015-07-07 23:51:18 INFO:root Checking for existence of nginx
2015-07-07 23:51:18 INFO:root Checking for existence of rsync
2015-07-07 23:51:18 INFO:root Checking for existence of VBoxManage
2015-07-07 23:51:18 WARNING:py.warnings /tmp/_MEIrzG3KE/dusty/preflight.py:42: UserWarning: Your virtualbox version (4.3.28) deviates from the supported version (4.3.26).

2015-07-07 23:51:18 INFO:root Checking for existence of boot2docker
2015-07-07 23:51:18 INFO:root Checking for existence of docker
2015-07-07 23:51:18 INFO:root Checking for existence of docker-compose
2015-07-07 23:51:18 INFO:root Adding github ssh key to roots ssh known_hosts file
2015-07-07 23:51:19 INFO:socket_logger # github.com SSH-2.0-libssh-0.7.0
2015-07-07 23:51:19 INFO:root SSH_AUTH_SOCK already set
2015-07-07 23:51:19 INFO:root Setting SSH_AUTH_SOCK to /private/tmp/com.apple.launchd.nz0aaDczPh/Listeners
2015-07-07 23:51:19 INFO:root Completed preflight check successfully
Loading dusty daemon

Also, I had VirtualBox version 4.3.20 before, and upgraded.  No difference between the two.

Hey @oehokie, I think this may be as simple as a typo, could you try running dusty config set specs_repo github.com/gamechanger/dusty-example-specs [1] (rather than dusty-specs-repo) and continuing from there? Sorry about the confusion. It sounds like we could make the error messaging around a non-existent repo more clear, perhaps we could make a separate issue for that.
[1] https://github.com/gamechanger/dusty-example-specs

Unfortunately, that doesn't work for me either

You're seeing the same errors after that?

Also, as an option, you can join us on our public Slack and we can help you out in realtime dusty-slackin.herokuapp.com

I think you can close this issue.  To sum up... I went on slack and discovered a few problems in my configuration (git was installed somewhere custom, my ssh key was named something unexpected, etc).  Ended up spawning #366, #367, #368
Probably save to close this issue


Disable animation

Hi, how to disable icon animation ?

No way! You can fork and make changes to the code


Add intersect method to bloom filter

Per https://en.wikipedia.org/wiki/Bloom_filter#Interesting_properties , it is valid to construct a new bloom filter from a set of constituent bloom filters by applying an 'AND' operation to the constituent bits sets.
"... The intersect operation satisfies a weaker property: the false positive probability in the resulting Bloom filter is at most the false-positive probability in one of the constituent Bloom filters, but may be larger than the false positive probability in the Bloom filter created from scratch using the intersection of the two sets."
This PR adds a method to support this operation, and a unit test to verify.

@alstaplesmoore thanks.  this looks good to me.  Can you add another test that tests a larger intersection to ensure it works with non-trivial amounts of data?

PR https://github.com/addthis/stream-lib/pull/134 may have more tests that we can use.


restoring settings partial in partner profile edit

Description
I found an issue on staging where the settings had disappeared from the partner profile edit (from the partner pov)., it appears from #4051.    This restores it.
Noting, though,  that we should also have a test that this partial is appearing -- since we didn't catch it disappearing through either our functional or technical reviews -- and most partners  won't notice that it's gone.
I'm also getting a lot of unrelated failures on my local (11).
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
manual check  -- went in as<EMAIL_ADDRESS>and checked that the fields appeared.

I've put a GFI in the backlog around this.


Add 1Password Beta Chrome ID

Failure to load 1Password Beta extension from Chrome
1Password Beta Version would be -
EXTENSION_NAME = "phicbbndgmmpogmijjkbmdhpioaieaha"

@SethCalkins the workaround should not be necessary anymore actually. Try reinstalling sudolikeaboss with the latest instructions on github.com/ravenac95/sudolikeaboss. If that doesn't work for you let me know. I want to make sure that sudolikeaboss still works for everyone. I use it everyday so I would understand the frustration if it were broken.


WIP: Build arm64 container and reduce cloud image size

Summary
Allows for building ARM64 images of the CLI and cloud containers. Also cleans up the structure of the cloud container to greatly reduce its size (from 7.05 GB to 3.32 GB after the initial restructuring).
Test Plan
Build the containers and run them to ensure they are still working
Checklist

[ ] If required, I have updated the Plural documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have added a meaningful title and summary to convey the impact of this PR to a user.
[ ] I have added relevant labels to this PR to help with categorization for release notes.

I can confirm that it works on arm64 (M1 Mac).

Tested on amd64 also and it works.

Can you squash this pr into 1 commit if possible?


Unable to create a mask with given length ( Ionic RC 1 )

I am trying to create a mask  'X-X-X-X', but character length is being ignored.
public codeMask: any[] = [/\d{1}/, '-', /\d{1}/, '-', /\d{1}/, '-', /\d{1}/] ;
public testMask: string = '';

<ion-item>
  <ion-input
      [(ngModel)]="testMask"
      [textMask]="{mask: codeMask}"
       type="text" placeholder="X-X-X-X" clearInput>
  </ion-input>
 </ion-item>

You can write double digits in any field with /\d{1}/, but only one in the input.
Mask is being treated like this:
Input: 1-2-3-4, Model: 1-2-3-4_

@nikoTM You don't have to specify {1} in the regex. Each element in the mask array can only accept one element anyway. I think the problem you're having is related to the model not being updated correctly, which is the same problem as issue #246.
What do you think @PerfectPixel?

I am surprised that it works at all. I guess I'll have to set up Ionic for a test run.

This maybe related to #294 so I'll go ahead and close it. Thanks!


feat/remove ingest code, use new dep for tests

Description
Alternative to https://github.com/Unstructured-IO/unstructured/pull/3572 but maintaining all ingest tests, running them by pulling in the latest version of unstructured-ingest.

Minus the small comments above, looks really close!
For posterity, noting the below tests are currently skipped in main and are still being skipped in this branch. No need to resolve as part of this PR.
2024-09-16T19:16:03.9406352Z notion.sh (skipped checking error code: 1)
2024-09-16T19:16:37.8436464Z hubspot.sh (skipped checking error code: 1)
2024-09-16T19:16:56.3926658Z local-embed-octoai.sh (skipped due to missing env var)
2024-09-16T19:17:04.9366930Z local-embed-voyageai.sh (skipped due to missing env var)
2024-09-16T19:17:06.2143546Z local-embed-mixedbreadai.sh (skipped checking error code: 1)
2024-09-16T19:19:25.0674346Z ######## SKIPPED TESTS: ########
2024-09-16T19:19:25.0683707Z notion.sh (skipped checking error code: 1)
2024-09-16T19:19:25.0685480Z hubspot.sh (skipped checking error code: 1)
2024-09-16T19:19:25.0687228Z local-embed-octoai.sh (skipped due to missing env var)
2024-09-16T19:19:25.0688386Z local-embed-voyageai.sh (skipped due to missing env var)
2024-09-16T19:19:25.0689563Z local-embed-mixedbreadai.sh (skipped checking error code: 1)


add a key and value to existing dict in kcl

General Question
How do i add a key and value to existing dict in kcl?
For example: I want to be able to do something like this in kcl
mydict = {
key1 = "1"
key2= "2"
}
mydict[key] = value

For example, you can use the attribute operator and union operation to do this.
mydict = {
    key1 = "1"
    key2= "2"
}  | {
    key = "value"
}

Or the unpack operator
mydict = {
    key1 = "1"
    key2= "2"
}
newDict = {
    **mydict
    key = "value"
}

See more documents here: https://kcl-lang.io/docs/user_docs/support/faq-kcl#4-how-to-add-elements-to-a-dict

Additionally, thank you very much for opening the issue for discussion. We will regularly synchronize user questions to the FAQ document.


Add a timeout for "RETRY" and "WAIT" pathBlockedStrategies

Currently the "RETRY" and "WAIT" pathBlockedStrategy will retry/wait forever. There should be a timeout value to make it configurable.

closed with 2.3.0


refactor tests

use common methods to create url params for example

use DI instead of global vars and extract common things to generics

https://github.com/gennadyterekhov/metrics-storage/pull/54
still messy but tests pass


Test fails with yorkie 0.4.5

Description:
test fails when using yorkie 0.4.5 in the test environment, and even with 0.4.4, test fails due to broken backward compatibility.
failed test samples:

https://github.com/yorkie-team/yorkie-js-sdk/actions/runs/5910049803/job/16031244683?pr=622
https://github.com/yorkie-team/yorkie-js-sdk/actions/runs/5901242545/job/16007047437?pr=618

Why:

I think this is because the CI workflow's docker based testing environment is not compatible with JS SDK right now.
Since the Yorkie main repository recently introduced changes that break backward compatibility that the Yorkie JS SDK has not yet introduced, the latest Yorkie server and the Yorkie JS SDK will not be compatible.

It appears to be caused by the issue below:
https://github.com/yorkie-team/yorkie-js-sdk/pull/617#issuecomment-1685424983

During the v0.x.x phase, we plan to make multiple changes that might break backward compatibility in both APIs and the database. However, once we reach v1.x.x, our commitment is to maintain strict backward compatibility for both APIs and the database.
For more details, you can refer to https://semver.org/#spec-item-4.


Kyma - Statement Update: Event triggers

Hello,
Scenario URL: https://developers.sap.com/tutorials/cp-kyma-redis-function.html#c1579029-cff3-43a6-b9ff-116af0477c97
6.4: In the Configuration scroll down and choose Event Triggers.
->> Statement should be Choose "Add event triggers" instead of just choose event triggers.
Please update the same.
Labels to be added:
Landscape label: Landscape label: trail1, trail2, trail3
Sprint Label: SCPTest-2101
Scenario label: SCPTest-Kyma
Prio: low-prio
Thanks,
Shubham Agarwal

Thanks for the feedback, fixed in qa and will be pushed in next update.

Thanks for the feedback, fixed in qa and will be pushed in next update.


Capitulate to Apple?

What's the community's opinion on adding mqs for the new iOS 9 half screens on iPads?
eg:
/*ipad landscape split-screen safari*/
		@media only screen
			and (width:507px)
			and (height:660px){
			.some-class{some-rule:stupidpx;}
		}
/*ipad landscape split-screen Chrome*/
		@media only screen
			and (width:507px)
			and (height:653px){
			.some-class{some-rule:stupidpx;}
		}

These overlap the abandoned skeleton queries but they at least let you target some elements.

I'd say it's out of scope.  Skeleton's goal is to be a starting point for projects, and this sort of decision making is too particular to individual project needs.

Right then.  Sounds ggod.


Test GA packages with TS 3.7 Beta

As 3.7 will be coming out roughly coincident with our GA release, we need to make sure we'll not be broken.

/cc @ramya-rao-a

I can probably write a script to update every package to use the new TS and open a PR. We can run CI and tests against that PR.

That sounds like a reasonable approach to me!


Added realization for dominant-baseline svg attribute

I don't see any changes to save the new attribute in the binary format.  Am I missing something?  It seems like bumping up the file version would be necessary, too.  Also, a couple of regression tests would be needed.  You can run the regression tests in a reproducible manner by running the appropriate Github action.

Okay, I understood. I will add bin format support and some tests


Non payable message in Ask! ?

In Ask! message is payable even if there's no modifiers.
For example,
@message({ mutates: true })
  flip(): void {
    this.data.flag = !this.data.flag;
    let event = new FlipEvent(this.data.flag);
    // @ts-ignore
    env().emitEvent(event);
  }

This will generate metadata.json payable true, and it is actually payable.
"messages": [
        {
          "mutates": true,
          "payable": true,
          "args": [],
          "docs": [],
          "label": "flip",
          "selector": "0x633aa551"
        }
]

Is there a way to specify message to be non-payable?

Some things need to be considered first when support non-payable.
The logic of checking non-payable need to make sure Balance type first in decorator.
So we need to make sure the abstract level of Balance type.

I missed TODO comment in the source code for some reason, I recognize it intentional and non-payable hasn't not supported yet.


Move template_app to a cookie cutter repo

Can use cookiecutter so new snakebids apps can be easily instantiated

Created a cookiecutter here: http://github.com/akhanf/snakebids-app

Created a cookiecutter here: http://github.com/akhanf/snakebids-app

added in #11

added in #11


Screwed Up Mem Sizes and Addr Widths in Chisel

I'm pretty sure this is the issue that is making compile times huge.  I'm comparing SingleLayerConv_RCIO in old and new spatial.    It takes seconds to compile.
In new spatial, there is one of these mems, banked by 9.  This means each bank should be 9216 and should take 14 bits to address.  For some reason, in new spatial only, I can put printlines in the MemPrimitives template and see that it is the correct  size Mem in Mem1D and it has the correct width for the address ports, everything from the SRAM template down to the Mem1D is sized correctly, yet somehow the verilog that is generated is only giving the Mem1Ds 4 bits for addressing...  Weirder, if I just make each bank double the size, it gives the address ports 1 bit width, and if I divide the bank sizes by 4, it gives 17 bits of width for the address ports.

Turned out to be because the MemPrimitive abstract class supported the interface for all memory templates, including shift regfiles that have a wire per element.
Separated the interfaces


Add shape implementation

Fixes this warning on master
In [1]: import numpy as np; import dask.array as da

In [2]: np.shape(da.arange(12))
/Users/taugspurger/sandbox/dask/dask/array/core.py:1272: FutureWarning: The `numpy.shape` function is not implemented by Dask array. You may want to use the da.map_blocks function or something similar to silence this warning. Your code may stop working in a future release.
  FutureWarning,
Out[2]: (12,)

Same for union1d.

Thanks for tracking that down @jrbourbeau.  I'm restarting the Appveyor error to see if it's a flaky test.

Hmm no I haven't seen that error before. I'll keep an eye out for the Appveyor rerun

Tests passed on the Appveyor re-run. I'm going to merge this (thanks for working on this @TomAugspurger) and we can investigate further if the failure pops up again.


Without make

Add bd.zip, which contains the result of make in am-cpu-tests and rv-tests, in case that some people are using Windows.

No need for change of repo, and I will try to get a release for that usage.


[PROPOSAL] More documentation on ec2 autoscaler

Description
The EC2 autoscaler is poorly documented in what it can and cannot do (Galaxy dependency!?). This ask is that better documentation be included.
Motivation
See https://github.com/apache/incubator-druid/pull/6953 for some details

more documentation would in deed be useful as the autoscaler is quite nice. Especially the different strategies are difficult for me to wrap my mind around.
By the way, I believe the off-the-shelf autoscaler is probably not for everybody because it is quite specific. I wrote an HTTP autoscaler extension if this is of any use. It basically delegates all autoscaling decisions to an external webservice.
For folks that are not 'brave' enough to fork a big code repo and put their own customization in, it might be a less daunting proposition to just implement a webservice in a technology of their own choosing.
We have currently only one webservice implementation that would spin up EMR clusters with middlemanagers deployed on the EMR master nodes. One could include a minimalistic autoscaler example within the Druid codebase - a single-file python script which would perhaps even do nothing else but to print out the scaling decisions relayed by Druid, might be very helpful for people to play around with and get a feeling for how the druid-autoscaler mechanism behaves. They might then feel empowered to extend this webservice to their own needs.
But I'm quite off topic now. Sorry. Just meant to mention it because the autoscaling feature in Druid is quite nice but I don't have the feeling that many people are using it which might be due to its specific nature (Galaxy and such)


Dynamic attribute optimizations

I was profiling some code that handles json_api_client results, and makes a lot of use of the dynamic attribute methods. I noticed that if I just implemented my own getter methods:
class Article
  def title
    attributes[:title]
  end
end

...then it got surprisingly faster.
I looked at the source, and spotted a few optimizations I could make. Here are 4, each with before/after benchmark times (the benchmark code is committed here as a unit test, which we can yank or move before this is merged).
The only commit without benchmark times is the change to use end_with?('=') instead of =~ /^(.*)=$/, because it didn't actually speed it up. I committed it here as a curiosity; because I think ends_with?('=') is clearer than the regex; and because I've seen people say that it's faster, so this heads off the conversation.
Probably the most controversial optimization is the last one: to auto-define getter methods for attributes from DynamicAttributes#method_missing. This is a significant speed-up: basically, we'll never have to run that method_missing logic again for that method name. Since the classes already respond to those methods, and handle those messages in an equivalent (but slower) way, this seems OK to me. Also, I don't think we have to worry about overwriting client code, because by definition, if #method_missing is running, then they didn't define this one.
Here's the overall benchmark difference, from the first commit (which only adds the benchmark unit test), to the last (with the 4 optimizations):
Before:
       user     system      total        real
read:   1.090000   0.010000   1.100000 (  1.091464)
write:  1.130000   0.000000   1.130000 (  1.136750)

After:
       user     system      total        real
read:   0.020000   0.000000   0.020000 (  0.016366)
write:  0.690000   0.000000   0.690000 (  0.687914)

@danbernier great work on this!  I've pulled this into a released fork of this gem jsonapi-consumer along with some other outstanding bug fixes.

@chingor13, thanks for merging this! Glad to see this project is still alive.
And @jsmestad thank you, too, glad you found it helpful!


Still have problems while loading websites

It is related to #92 issue. The related test passes:
public async Task ContextLoadAmazonWithCss()
        {
            if (Helper.IsNetworkAvailable())
            {
                var url = "http://www.amazon.com";
                var config = new Configuration().WithDefaultLoader().WithCss();
                var document = await BrowsingContext.New(config).OpenAsync(url);
                Assert.IsNotNull(document);
                Assert.AreNotEqual(0, document.Body.ChildElementCount);
            }
        }

but the test itself is not correct, it would pass when exceptions in asyncronious loading of images, css, etc. whould accur.
The correct test would probably be:
public async Task ContextLoadAmazonWithCss()
        {
            if (Helper.IsNetworkAvailable())
            {
                var url = "http://www.amazon.com";
                var config = new Configuration().WithDefaultLoader().WithCss();
                var document = await BrowsingContext.New(config).OpenAsync(url);

                await Task.WhenAll(document.Requests);

                Assert.IsNotNull(document);
                Assert.AreNotEqual(0, document.Body.ChildElementCount);
            }
        }

Now it fails :)

But that is not the purpose of this test. Otherwise you need to define when exceptions in these requests (e.g. when the resource is not available) are acceptable and when not.
What breaks / what is your specific problem?

I also added the WhenAll call - still works for me.

Today I have some time to dig deeper into this :)
After last code update the test passes, but still getting null ref exceptions when debugging the test:
at AngleSharp.Dom.Css.CssStyleEngine.Parse(IResponse response, StyleOptions options) in e:\Projects\AngleSharp1\AngleSharp\DOM\Css\CssStyleEngine.cs:line 77
at AngleSharp.Dom.Html.HtmlLinkElement.<>c__DisplayClassf.b__a(Task1 m) in e:\Projects\AngleSharp1\AngleSharp\DOM\Html\HtmlLinkElement.cs:line 249    at System.Threading.Tasks.ContinuationTaskFromResultTask1.InnerInvoke()
at System.Threading.Tasks.Task.Execute()
Since there is no resource loading in default loader, request result is always returned as null and than passed to parse:
(HtmlLinkElement.cs, 252)
_sheet = engine.Parse(response, options);

which results in null ref exception

I see. This will be fixed! Thanks for investigating :+1:


[PE-D][Tester B] Sort by grade level does not seem to work

Sort grades in ascending order seems to sort by the ascii value instead of the actual grade value
Suggestion: Could consider comparing just the numerical part of the grade instead of the whole string

Labels: severity.Medium type.FunctionalityBug
original: ignatiusoey09/ped#8

This is not counted as a bug. We should clarify this behaviour in the documentation.


Add security resources

Add beginner friendly security resources

Hey! Can you elaborate on what I hay to do?


import WIP to video template

Please import the WorkInProgress component into the VideoTemplate found inside components/ContentTemplates folder.
Then render the WorkInProgress component instead of the current div and placeholder sentence.
See the AlertsTemplate component if unsure how to do this and follow the same pattern.

Is this issue still open ? I will like to work on it please assign it to me.

Sure @uddeshya-amit


Add option to set a repair callback

This allows the user to abort the repair process and receive progress notifications
Fixes #639

@veryordinally / @raphjaph can you take a look at this API when you get a chance? Will merge it, if it looks like it covers your use case

Nice! This works for us. It looks like Self::verify_primary_checksums(mem) is called first, before the repair callback is invoked. Can this take a long time? If so, it would be nice to do a quick check if anything that takes a long time is going to happen, and call repair_callback first if so. Related, if repair_callback is called first, then having progress return 0 for the first call would avoid needing to check if we should skip printing the message, i.e., we could just do if handle.progress() == 0.0 { print_message() }.

I think that's already implemented. It's just in the caller of do_repair. See here: https://github.com/cberner/redb/pull/700/files#diff-7d7d3a6ce5c2a3e8b39ce4f49b78a71365c8e06c0d9ddb78bc2dca68bfeb4a5bR683-R688

Ah, okay awesome. Looks good to me!

I would recommend against if handle.progress() == 0.0 though. The API doesn't guarantee that the first call will be with a progress value of 0.0 and I might find some reason to change it in the future. Although that does seem unlikely


Work not properly under React 18 with StrictMode

Describe the Bug
I'm using react 18-RC.0, everything doesn't work.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
https://codesandbox.io/s/react-flow-node-shapes-forked-qo8ch?file=/src/index.js
Expected behavior
Should work
Screenshots or Videos
No response
Platform
React 18
Additional context
No response

ReactDOM.createRoot(document.getElementById("root")).render(
  <React.StrictMode>
    <App />
  </React.StrictMode>
);

The only workaround is to remove React.StrictMode

Reference: https://github.com/reactwg/react-18/discussions/19

React 18 is not yet supported. However I just checked it with the next version and it is working. The next version is already released as a beta. You can find the installation instructions and changed API in this PR: #1555. We will not do any changes for v9 in order to support React 18.

Thanks for the feedback. I've migrated my project to v10, and it works perfectly.


Minerva attack on ARM architecture

@tomato42 and I have tested OpenSSL in ARM architecture and we found that it may be vulnerable to a variant of the Minerva attack. We used statistical analysis to confirm the presence of side channels but we did not perform the Minerva attack against the implementation.
In the test scenario, we measure the time of signing of random messages using the EVP_DigestSign API (Init, Update, and Final) and then use the private key to extract the K value (nonce) from the signatures. Then based on the bit size of the extracted nonce we compare the signing time of full-sized nonces to signatures that used smaller nonces using statistical tests.
In our initial test, for ARM we tested P-256 in 3 microarchitectures (Ampere Altra Max CPU, Cavium's ThunderX2 CN99XX CPU with Vulcan microarchitecture, and Ampere's eMAG CPU with Skylark microarchitecture) and still found differences between them. Ampere Altra Max CPU doesn't show a signal while the other two microarchitectures do. In these results we can see a clear leak: there is a dependency between the bit size of K and the size of the side channel.
For Ampere Altra Max CPU:

Results for the non-deterministic path of the code. Skilling-Mack test p-value: 6.201480e-02. The sample tested has 291,513,535 observations.

Results for the deterministic path of the code. Skilling-Mack test p-value: 9.524591e-01. The sample tested has 161,951,709 observations.
For Cavium's ThunderX2 CN99XX CPU with Vulcan microarchitecture:

Results for the non-deterministic path of the code. Skilling-Mack test p-value: 3.958440e-20. The sample tested has 647,808,394 observations.

Results for the deterministic path of the code. Skilling-Mack test p-value: 1.215735e-12. The sample tested has 161,951,519 observations.
For Ampere's eMAG CPU with Skylark microarchitecture:

Results for the non-deterministic path of the code. Skilling-Mack test p-value: 2.028643e-12. The sample tested has 647,829,103 observations.

Results for the deterministic path of the code. Skilling-Mack test p-value: 6.354241e-59. The sample tested has 64,785,572 observations.

In the description:

OpenSSL in s390x architecture

Missing search and replace with Arm? :)

@GeorgePantelakis the signal is very small ~1ns and I would be very curious to see any real world attack scenario that could take an advantage of such small signal.

@t8m That might be true, at least for now, but as you can see for Skylark in the deterministic path the signal is ~3ns so that could be an issue.

@GeorgePantelakis the signal is very small ~1ns and I would be very curious to see any real world attack scenario that could take an advantage of such small signal.

Note that the leakage is most likely in scalar multiplication, which means ECDH is also most likely affected. So, anything that exposes a network API that uses static ECDH, for example, Json Web Encryption, can be attacked through this.
Confirming 1ns difference over the network is 100% possible, I've done that multiple times, it's just a question of persistence.

There is a reason why an ECDH secret is usually not used in more than one key agreement.

Depends on the use case. TLS doesn't because TLS uses signing keys for the long-lived credential, but there are plenty of protocols that designed differently. HPKE, other ECIES schemes, Signal, the original QUIC handshake, etc.
Constant time ECDH isn't meaningfully harder than constant time ECDSA signing. Arguably easier. ECDSA is a point multiplication and then some. (Although they're not quite the same multiplication. ECDSA is a base point multiplication and ECDH is an arbitrary point. But that doesn't change how hard constant time is.)

There is a reason why an ECDH secret is usually not used in more than one key agreement.

that's why I wrote "static ECDH", not "ECDHE".
Yes, for ECDHE, like in TLS, IPsec or SSH, when the key is used once, or just few hundred times, a remote network timing attack is basically impossible with leakage this small.

Hello, we have an update on the issue.
We have also tested P-384 and P-521 for non-deterministic, determinist, and already digest paths and we found that they are not vulnerable. We believe that there is some curve-specific assembly on ARM that introduces the vulnerability.
For P-384 curve:

Non-deterministic code path. Skilling-Mack test p-value: 2.591940e-01. Not vulnerable.

Deterministic code path. Skilling-Mack test p-value: 4.956091e-01. Not vulnerable.

Already digested code path. Skilling-Mack test p-value: 1.910694e-01. Not vulnerable.
For P-521 curve:

Non-deterministic code path. Skilling-Mack test p-value: 8.750479e-01. Not vulnerable.

Deterministic code path. Skilling-Mack test p-value: 2.051492e-01. Not vulnerable.

Already digested code path. Skilling-Mack test p-value: 3.801709e-01. Not vulnerable.

@t8m is there an easy way to compile openssl on ARM without the curve-specific assembly for P-256?

@t8m is there an easy way to compile openssl on ARM without the curve-specific assembly for P-256?

You can use no-asm, however that will of course disable all the assembler code. There is no specific option for the nistz assembler.

@t8m and how hard would be to modify the code so that it doesn't use curve-specific code/assembly with P-256 on arm?

Also, we have tested that P-224 is not vulnerable. That just strengthens our suspicions that the curve-specific code is leaky and should be fixed or removed.

You can try to configure with ./config -DECP_NISTZ256_REFERENCE_IMPLEMENTATION
to compare the results against the reference implementation without assembler support.

@bernd-edlinger I compiled OpenSSL the configuration but the OpenSSL speed suggests there is no change so I would say it did not work.
version: 3.4.0-dev
built on: Wed Aug 28 12:57:39 2024 UTC
options: bn(64,64)
compiler: gcc -fPIC -pthread -Wa,--noexecstack -Wall -O3 -DOPENSSL_USE_NODELETE -DOPENSSL_PIC -DOPENSSL_BUILDING_OPENSSL -DNDEBUG -DECP_NISTZ256_REFERENCE_IMPLEMENTATION
CPUINFO: OPENSSL_armcap=0xbd
sign    verify    sign/s verify/s
192 bits ecdsa (nistp192)   0.0003s   0.0003s   2945.0   3293.9
224 bits ecdsa (nistp224)   0.0001s   0.0002s  12029.0   5550.0
256 bits ecdsa (nistp256)   0.0000s   0.0002s  22335.0   6340.0
384 bits ecdsa (nistp384)   0.0004s   0.0009s   2500.0   1165.0
521 bits ecdsa (nistp521)   0.0008s   0.0016s   1234.0    641.0

Yes, we have this flag enabled by default.

This is actually no assembler code.
does the timing signal change when you add no-asm ?
or remove the ec_nistp_64_gcc_128 option ?

We have found out that P-256 for ARM is somehow special. It uses specific code that is leaky and no flag will disable only this and use the default. We can use the no-asm flag but this will decrease the speed of all the curves (even the ones we found non-leaky) by a factor of at least 5. The DECP_NISTZ256_REFERENCE_IMPLEMENTATION does not work and the code is still running. We need a flag that will be combined with enable-ec_nistp_64_gcc_128 and only disables the special code of P-256 in ARM.

Once again, did compiling with -DECP_NISTZ256_REFERENCE_IMPLEMENTATION have an effect on the timing signal?
If you compile with -g you can e.g. set a breakpoint on ecp_nistz256_point_double
which should be a C function in that case, not the assembler module.

can you explain (point to the code) why you think that setting -DECP_NISTZ256_REFERENCE_IMPLEMENTATION will have an effect on timing signal without having effect on the performance of the P-256 curve?

If that code I mentioned is involved, it is possible that the part of the assembler code that is disabled by that degine is buggy, or the other part, low-level functions that are independent of that.  The timing signal would tell us where to look for the bug.

In theory the following 3 assembler functions will be replaced by equivalent C functions which could have an effect:
https://github.com/openssl/openssl/blob/0b97a5505efa8833bb7b8cabae45894ad6d910a2/crypto/ec/ecp_nistz256.c#L248-L259

ah, ok, yes, there is a small difference for the code compiled with and without the -DECP_NISTZ256_REFERENCE_IMPLEMENTATION option:
with:
                              sign    verify    sign/s verify/s
 256 bits ecdsa (nistp256)   0.0000s   0.0002s  22574.7   6340.0
 384 bits ecdsa (nistp384)   0.0019s   0.0014s    539.0    702.0

without:
                              sign    verify    sign/s verify/s
 256 bits ecdsa (nistp256)   0.0000s   0.0002s  22651.0   6664.0
 384 bits ecdsa (nistp384)   0.0019s   0.0014s    539.0    698.0

(i.e. about 5% difference for verify)


Language selector options are not translated

Description of the bug
If you have added custom language files their translations are not shown. If the app is stateless it helps to refresh the page for some reason. It seemes to be related to this issue i think: https://github.com/digdir/designsystem/issues/180
Steps To Reproduce

Go to an app with translations
Open the language selector
the options are not translated

Additional Information

The textresources are probably fetched after the languageSelector has been rendered so the options will only use the key instead of the textresources value.

Huh, the component should rerender if the textresources change. Do you have an app with custom languages I can take a look at?

@bjosttveit if you want to see the behavior: https://udir.apps.tt02.altinn.no/udir/pbu-test-anonym and username: t-gangs-105 password: srxn9

Ahh, so it shows up when selected but not in the dropdown, weird!

I changed the version of react app frontend to 3.58.0 and that works:

Yeah that component was recently switched out to use digdir/designsystem

Weird, I managed to reproduce this in my own app. But only by omitting the language name in some resource files. It worked for me as long as each language name was defined in all of the resource files.

Interessting I can try that!

That doesn't seem to help for me... It seems wierd that it has to be defined in all the resourcefiles because it just fetches one language resource file at a time?
It could behave different in local development. Our demoapplication show it correct after a refresh: https://udir.apps.tt02.altinn.no/udir/pbu-test-demo?survey=survey-5-aarstrinn5

That link worked without refresh for me

This seemes to be fixed


buildah: 1.21.2 -> 1.21.3

https://github.com/containers/buildah/releases/tag/v1.21.3

Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 130405 at 175ead9a run on x86_64-linux 1

2 packages built successfully:

buildah
buildah-unwrapped

I am adding a backport tag because of:

chroot: fix environment value leakage to intermediate processes (CVE-2021-3602)


Request for Search behavior to clear cache

Hi.
In case that we add new information to the DB that can be searched by the module, it would be great if we could clear the cache so it can access that new information. Right now, the only way to do that is setting the module to cache: false or reloading the page, but that's not a nice solution...

Added

Is there any solution for clear cache in the remote dropdown?

This functionality is not documented in theDocumentation. I'm not sure where to put it so I'm hesitating to create a pull request.


Publish new ladybug-frontend jar on npmjs

This is about the ladybug-frontend jar that we build with Maven, no longer via external server webjars.


#68 Add Landing Page with Mockup Design

Now with simplistic landing page.
I use an orange placeholder box for the NFT-drop lists.
I'm not entirely sure, how André will want this to be. Let's just implement it and then ask.
This is how it looks:

The banner styles for small screens (below 900px) has been added and looks good for me at least.
I think, I'm done. If you approve this today and we merge it, then we could show it tomorrow morning.


Feature request: flag to view all files in the same view

Is your feature request related to a problem? Please describe.
Can't view multiple files at the same time
Describe the solution you'd like
A flag to view all passed files in the same view rather than skipping between them
Contribution
I could potentially contribute, but would appreciate some pointers on how to do so!

That is something we thought about but would definitely be not trivial. We could be done more easilly is some kind of "FolderView" where one only see thumbnails (static image) of each files in a grid and can switch between files easily, then actually open a specific file wheen needed.
Still a big task but seems doable, and in line with what other softwares do. It could look like this:

What do you think @Meakk ?

@nermolov Do you want to visualize all the files in the same view because they are related, or view every file in its own view in a multi-view window?

@Meakk I would like to visualize them in the same view because they're related, e.g. multiple components for the same part generated by a CAD system.

Could you share a sketch or screenshot of what you hope ti have ideally ?

It can be a very useful feature indeed. Maybe an option --append where all the meshes are appended to the view instead of replacing the previous one. For instance, a call f3d --append *.step would display all the step files in the current directory.
It's quite complex to implement considering the current architecture is designed for a single file (see bool F3DLoader::LoadFile(int load)) and we also need to consider different file types (full scene or geometry only).

I dived in F3D's code to add an append option. My goal is to submit a pull request 😊

@SamuelTallet
While a good idea, this issue is not a easy one and i would not recommend it as a first contribution, especially as @Meakk and I are working on #52 .
That being said, before knowing how it could be done, we need to check IF it could be done, and this should ne a much more manageable task.
For that, you would need to just not clean the renderer and try loading multiples files.

Finally, I won't submit a PR. Here's plan I had in mind:

Add an --append argument with a default value set to false.
Store geometry actors in a vtkAssembly instead of a vtkActor.

vtkAssembly is an object that groups vtkProp3Ds [...]

If append: don't remove the renderer, reuse the same renderer.
When we get geometry actor from importer, append this actor to geometry actors collection with vtkAssembly::AddPart().
In F3DStarter::Start(), if append: call this->LoadFile(f3d::loader::LoadFileEnum::LOAD_NEXT) many times to load all files.

related to #129

One way to approach this instead of "appending" is to consider multiples file to be a "serie of file" and to load them using a dedicated reader that will create a composite output, one file being one block of the output. Of course, the serie reader will internally use the right reader for each file.
This would be fully transparent to the libf3d, (one reader produce one dataset that will be shown) and most of the logic will be in F3D.
One question is the right config file to use, I suppose that we will use the one from the first file and be done with it.
Wdyt @Meakk ?


Strange state behavior when calling an actionCreator in a redux subscriber

Hi,
I came across a strange state behavior using redux 0.12 and I don't know if it's a bug or a misuse. The code above does not reflect my real code but is the best minimal example I could came up with that highlight the "bug".
// test.js
import { createRedux } from 'redux'

var stores = {
    connectStore(state = {}, action) {
      console.log('store.connectStore(), action received: state', state, 'action', action)
      switch (action.type) {
        case 'CONNECT':
          return {
            ...state,
            logged: true
          }
        default: return state
      }
    },
    profileStore (state = {}, action) {
      console.log('store.profileStore(), action received: state', state, 'action', action)
      switch (action.type) {
        case 'PROFILE':
          return {
            ...state,
            description: 'test desc'
          }
        default: return state
      }
    }
}

var actionCreators = {
    connect() {
      console.log("\n", 'actionCreators.connect()', "\n")
      return { type: 'CONNECT' }
    },
    getProfileData() {
      console.log("\n", 'actionCreators.getProfileData()', "\n")
      return { type: 'PROFILE' }
    },
    aLastAction() {
      console.log("\n", 'actionCreators.aLastAction()', "\n")
      return { type: 'ANOTHER_ACTION' }
    }
}

var redux = createRedux(stores)
var fetchedProfileData = false

redux.subscribe(() => {
  console.log('SIMPLE REDUX SUBSCRIBER - connectStore:', redux.getState().connectStore)
  if (redux.getState().connectStore.logged) {
    if (!fetchedProfileData) {
      redux.dispatch(actionCreators.getProfileData())
      fetchedProfileData = true
    }
  }
})

redux.dispatch(actionCreators.connect())

setTimeout(() => { redux.dispatch(actionCreators.aLastAction()) }, 500)

Running this script using babel-node --stage=0 test.js produce the following output:

store.connectStore(), action received: state {} action { type: '@@INIT' }
store.profileStore(), action received: state {} action { type: '@@INIT' }

 actionCreators.connect()

store.connectStore(), action received: state {} action { type: 'CONNECT' }
store.profileStore(), action received: state {} action { type: 'CONNECT' }
SIMPLE REDUX SUBSCRIBER - connectStore: { logged: true }

 actionCreators.getProfileData()

store.connectStore(), action received: state {} action { type: 'PROFILE' }
store.profileStore(), action received: state {} action { type: 'PROFILE' }
SIMPLE REDUX SUBSCRIBER - connectStore: {}

 actionCreators.aLastAction()

store.connectStore(), action received: state { logged: true } action { type: 'ANOTHER_ACTION' }
store.profileStore(), action received: state {} action { type: 'ANOTHER_ACTION' }
SIMPLE REDUX SUBSCRIBER - connectStore: { logged: true }

What I interpret as a bug are the connectStore value being {} just after calling actionCreators.getProfileData().

...
SIMPLE REDUX SUBSCRIBER - connectStore: { logged: true }

 actionCreators.getProfileData()

store.connectStore(), action received: state {} action { type: 'PROFILE' }
store.profileStore(), action received: state {} action { type: 'PROFILE' }
SIMPLE REDUX SUBSCRIBER - connectStore: {}

...

My understanding of redux was that after having filled the connectStore with {logged: true}, a new action triggered should be applied on this new store value. But unfortunately I get an empty connectStore when calling  actionCreators.getProfileData()...
I was expecting to have:

...
SIMPLE REDUX SUBSCRIBER - connectStore: { logged: true }

 actionCreators.getProfileData()

store.connectStore(), action received: state { logged: true } action { type: 'PROFILE' }
store.profileStore(), action received: state {} action { type: 'PROFILE' }
SIMPLE REDUX SUBSCRIBER - connectStore: {}

...

After some console.log in redux I found out that this behavior seems to be due to the state being kept in 2 different places:

in createDispatcher.js via a closure on dispatch
in Redux.js via this.state

so when calling dispatch(action) (in createDispatcher.js), if another action is triggered in a redux listener (in Redux.js), the second action does not benefit from the update that the first action triggered (because it was not written yet in closured state in createDispatcher.js). Here is some redux code above to explain what I mean.
    // createDispatcher.js
    var state = setState(store(initialState, INIT_ACTION)); // the closure state variable
    function dispatch(action) {
      // state = ... is only effective AFTER all listeners in Redux.setState have been called
      state = setState(store(state, action));
      return action;
    }

  // REDUX.js
  Redux.prototype.setState = function setState(nextState) {
    this.state = nextState; // the state property of Redux
    this.listeners.forEach(function (listener) {
      // if another action is called in 'listener', we'll end up with Redux.state != createDispatcher.state
      return listener();
    });
    return nextState;
  };

Maybe this is the normal behavior but I don't feel like calling an action from a listener is a wrong since it is pretty common when you realize that the listener may call a React component lifecycle method (thanks to @connect decorator).
Thanks!

This indeed is a bug. It was fixed in 1.0 RC.
1.0 will be released in a few days.
You can try 1.0 RC now: https://github.com/gaearon/redux/releases/tag/v1.0.0-rc
Related: #110, #119

Thanks @gaearon and sorry for the duplicate!


"Spacer" widget

I love the pock widgets. I'm actually happy with just the now playing, date/time, and control icons. (Use case is using everything except dock) Is it possible to add a flexible space between widgets? For example, I would like escape and music on left side and the "status bar" plus icons on the right side, without a dock in the middle
Is your feature request related to a problem? Please describe.
I'd like the pock widgets to not bunch up on the left side if they don't take up the entire dock width
Describe the solution you'd like
A flexible space widget that would take up all the space in the middle so my escape button and music would be on left with status bar and brightness/volume icons would be on the right. Additionally, if there were multiple spaces inserted into dock they'd be able to evenly space things out if the user desired. In my mind is css flexboxes spacing logic if no elements have a width style set.
Describe alternatives you've considered
n/a
Additional context

Would love to see this. This is the reason I uninstalled Pock. Currently better off with just the native Expanded Control Strip. The native one has a "Space" widget that dynamically fills the empty space.

This and correct spotify icons are the only two missing pieces for me.  That makes the touch bar useful again for me without the insanity that is better touch tool. So having a spacer would be the only real thing missing.

In my case I wanted to just have the "Now playing" and control icon widgets aligned to the right and the left side completely empty.
The workaround I found is adding "dock" as the leftmost element, checking "hide finder", "show only running apps", "hide persistent items" and specifying -1000 item spacing.
Very dirty but it got the result I wanted. 😄

100 times this. Muscle memory know where the buttons should be. I loathe the touchbar with all my heart, but at least with spacers I could place them where they normally are.


Not work in Chinese Language Environment

My Revit 2016 language is CHS. When I tried to use CurvesFromCADLayers, there is warning. But if I set the language setting to ENU, everything works fine.

Currently, multi-language support is not . However, it'll look into how easy it is to add support for your language.
Firstly, can you confirm if the CAD.CurveFromCADLayers works if you set the createModelCurves input to false (so it only outputs Dynamo and not Revit curves)?


--asar should be enabled by default

I don't see why you wouldn't want those and I think most do. We could instead have --no-asar and --no-prune flags. I much prefer us to have sane defaults, which means less config mess for users.

I support both of these as defaults

i had seen a lot of open issues involving asar on the electron repo since it's relatively new. but if we think its bug-free now then i'm good with it

I don't think it's bug free, but it's pretty stable now, and Atom is using it so any issue will be fixed fast.

aside: as someone who just had to leave their computer on overnight copying over a large ~/dev directory full of hundreds of thousands of files from node_modules directory trees on a faulty SSD, I kinda wish a certain software platform would move to archive-based dependencies, too :)

@malept, I don't think these are the same issues. The problem that I have is that electron-packager looks at all the modules that are in my devDependencies which take a lot of time given their size. Adding --prune flag doesn't help it. I was only able to mitigate that problem by adding the ignore flag mentoned above.

Only bad part of this is you'll have to re-install when you prune. Here's what I'm doing:
# make a regex of node_modules to ignore
ignores=`npm ls --dev --parseable`
# get the package name
ignores=$(echo $ignores | sed -e 's@.*/@@')
# skip the first line
ignores=`echo $ignores | awk '{if(NR > 1) print $0}'`
# join with a pipe
ignores=${ignores//$'\n'/'|'}

# build mac distibutable app
./node_modules/.bin/electron-packager . "Dev Tool" \
  --platform=darwin --arch=x64 --version=0.35.6 \
  --ignore=\"$ignores\" \
  --app-bundle-id "com.dev-too.appl" --app-version 0.1.0 \
  --out ./built

zsh: command too long :(

if you pass the --prune flag to electron-packager it prunes for you in its tmpdir copy, which doesnt affect your source folder

well then there we go!

I have no problem with making prune default to true.
The main problem I have with making asar default to true is that as soon as that gets released, there is going to be an influx of issues filed on electron-packager with "module not found" errors when people use native extensions. Or really, any exception listed in the Electron docs.

I mentioned this issue in Electron's Slack channel a few weeks ago. Paul Betts mentioned that the way to handle my concerns about making asar true by default is to make sure that all shared libraries (.dll, .so, .dylib, .node) are backlisted from being packed by default. If someone implements that plus writes the appropriate amount of documentation (we may have to start a docs/faq.md), then I think I will be :+1: with moving forward.

@malept @sindresorhus @maxogden sorry to bother you all...
Was anything ever done re this one?
I have a proper ball ache of an issue where my packaged electron app fails to find react in any of my installed dependencies (react-player, react-countup, prismic-reactjs etc).
could anyone offer any advice?

Was anything ever done re this one?

This issue is flagged "help wanted" and is still open.

could anyone offer any advice?

If you need assistance, please use one of the community forums.

asar packaging produces broken icon in about window on linux platform: https://github.com/electron/electron/issues/23850


material-symbols: unstable-2023-11-17 -> unstable-2024-05-16

Description of changes
Closes  #312230

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

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

> nix build .#material-symbols
> tree result/
 result
└──  share
   └──  fonts
      ├──  TTF
      │  ├──  MaterialSymbolsOutlined.ttf
      │  ├──  MaterialSymbolsRounded.ttf
      │  └──  MaterialSymbolsSharp.ttf
      └──  woff2
         ├──  MaterialSymbolsOutlined.woff2
         ├──  MaterialSymbolsRounded.woff2
         └──  MaterialSymbolsSharp.woff2

Apart from the extra formatting of the argset, LGTM.

It was done by running nixfmt-rfc

Okay then, no objections.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1644

Add the unstableGitUpdater update script in passthru.updateScript like in f4c6449
passthru.updateScript = unstableGitUpdater { };

Done!

@JohnRTitor Should be good to go

@OPNA2608 Should be resolved now

aarch64 build failed, looked like a temporary issue?
remote: Compressing objects: 100% (43/43), done.
error: RPC failed; curl 92 HTTP/2 stream 5 was not closed cleanly: CANCEL (err 8)
error: 4962 bytes of body are still expected
fetch-pack: unexpected disconnect while reading sideband packet
fatal: early EOF
fatal: fetch-pack: invalid index-pack output
Unable to checkout ace1af08508a6562ef05055a606cf44ea583ce3b from https://github.com/google/material-design-icons.git.

@ofborg build material-symbols material-symbols.passthru.tests