Need packet merging

see https://github.com/roblabla/ProtoDef/issues/24

going to do https://github.com/PrismarineJS/node-minecraft-protocol/blob/7df40739dce6ac346a17878d65ba0ca3ac9eb111/lib/client.js#L29 until we get something cleaner inside protodef to handle these stuff

done


[FIX] add hitSlop to button

The TouchableWithoutFeedback prop hitSlop was missing from button. This adds that.
https://facebook.github.io/react-native/docs/touchablewithoutfeedback.html#props

Is there a way to user ... instead to avoid similar issues in the future ?

This does not work on android.
Error while updating property 'hitSlop' of a view managed by: RTC View

Okey on android you need to supply all 4 fields ! top, bottom, right, left or this crashes.

Thanks @note89 !


[Hobby Project] NNTrainer in MLPerf

This is a hobby project that does not require specific deadlines or project milestones.
Anyone may do this in their "hobby time"

See if nntrainer may participate in the performance benchmarks in mlperf.
Submit results to mlperf (w/ common hardware platform so that we can initiate competition in the on-device training community. e.g., RPI4)

:octocat: cibot: Thank you for posting issue #2027. The person in charge will reply soon.


GPUParticles3D: Emitting with oneshot + explosiveness within editor doesn't work if triggered too soon after 1st time.

Godot version
v4.1.1.stable.mono.official [bd6af8e0e]
System information
Godot v4.1.1.stable.mono - Windows 10.0.19044 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3070 Laptop GPU (NVIDIA; <IP_ADDRESS>98) - AMD Ryzen 9 5900HX with Radeon Graphics (16 Threads)
Issue description
Situation
If One Shot checkbox is enabled, Emitting will uncheck itself after its lifetime. Normal procedure so far.
Problem
As soon as Emitting boolean is automatically unchecked, hitting it again will not work (it unchecks itself again, but the particles do not show). It seems you have to wait an additional amount of time on top of its Lifetime to preview its emission properly.
It wouldn't be a problem if the particle's Lifetime was less than a second. However, it's time-consuming with long-lasting particle emissions (eg. Lifetime >= 1.5s ). I can tell this extra delay is roughly proportional to the Lifetime value.
Example:

Set Lifetime to 1.0s
Set Emitting to true (particles spawned successfully)
Emitting automatically sets to false after 1.0s
(Wait twice its Lifetime since turning Emitting ON to expectedly emit again.)
Set Emitting to true (particles spawned successfully again)
Emitting automatically sets to false after 1.0s
Set Emitting to true (it fails: close to zero or no particles emitted)

The 4th step has been mandatory ever since I started with Godot 3, regardless of hardware. Note that I'm running an arguable high-end computer and this still happens. At this point, I assume this is an engine issue.
Steps to reproduce
Case A: Without explosiveness

New project
Create GPUParticles3D with basic setup

Amount = 4
Time -> One Shot = ON
Time -> Lifetime = 1 (default)
Process Material -> New ParticleProcessMaterial
Draw Passes -> Pass 1 -> New QuadMesh

Emitting = true
Do it again, as soon as Emitting changes to false.

When activating Emitting a second time, immediately after it turns false, one of two happens:
A. Spawns one or two particles, Emitting turns false immediately.
B. No particle spawning at all, Emitting turns false abruptly.
Case B: Explosiveness ON

Same steps above, with an additional in GPUParticles3D

Time -> Explosiveness = 1 (or higher values than 0.5)

When activating Emitting a second time, while done as soon as it was turned false automatically, no particle spawns at all and Emitting turns false with approx. same delay as Lifetime value. If you don't wait at least twice as long as its Lifetime before turning Emitting ON again, it will still not emit.
Minimal reproduction project
Godot411-GPUParticles3D-oneshot.zip

This is currently the expected behaviour for one_shot particles. I agree that the behaviour is confusing. This issue comes up a lot with new users especially.
For clarity, the extra delay is exactly the lifetime of the particle system when explosiveness is 0 and it decreases as explosiveness approaches 1. The delay is essentially the amount of time that a particle may survive after having been emitted. When explosiveness is 0, a particle may be emitted right when the lifetime is reached, and it may survive for a lifetime amount of time.
For 4.2, we merged https://github.com/godotengine/godot/pull/76859 which allows you to connect a signal to particles that will fire when the particle has finished (and thus you can enable emitting to restart the system).
That being said, for some users it seems that they expect that toggling emitting off and then on when using one_shot particles will restart the particle system. But currently, the way to restart a particle system is to call restart(). restart() will also kill any active particles, so it may not be the ideal solution.
We have been discussing recently whether we should change the behaviour of the emitting property to match user expectations better. The discussion is here: https://github.com/godotengine/godot-proposals/issues/7322
Since this isn't technically a bug, I encourage you to add your thoughts to  https://github.com/godotengine/godot-proposals/issues/7322 so we can track the conversation in one place. Accordingly, I will be closing this bug report

For clarity, the extra delay is exactly the lifetime of the particle system when explosiveness is 0 and it decreases as explosiveness approaches 1.

Emitting turns to false automatically when no particle is alive, therefore it should emit as soon as Emitting is true again. But it's not emitting, and this is the issue here.
It doesn't make sense to wait twice the Lifetime of the particles to work if Emitting already switched to false on its own.

That being said, for some users it seems that they expect that toggling emitting off and then on when using one_shot particles will restart the particle system.

In my steps I have never toggled off Emitting. While it would be ideal to have for long-lasting particles, that's not the issue I'm addressing here. On the off-topic: it's good to know how far we can go with the current restart() route.

Since this isn't technically a bug, I encourage you to add your thoughts to https://github.com/godotengine/godot-proposals/issues/7322 so we can track the conversation in one place. Accordingly, I will be closing this bug report.

Let me rephrase the situation and problem:

Emitting is changing to false on its own, implying that it's ready to emit again.
However, switching it back to true immediately doesn't emit particles.

This behavior is technically a bug here. There is no explanation in the docs stating that you have to wait an invisible amount of time (despite of the Emitting property being switched to false on its own), there is no reason to not call it a bug here.
While this way of emitting doesn't seem to change anytime soon (Godot 4.2 plans seem to be set in "virtual stone"), I suggest to, at least, try resolving this matter for the time being.


Scoped pool

Thanks for this neat threadpool implementation!
I believe an integration of crossbeam::scope and futures executors is currently not readily available, and find it personally valuable.  The utiliity is to create threadpools that can run non 'static futures.
We went ahead and tweaked the data-structures available here to create a "scoped pool".  It simple records the lifetime as PhantomData in the threadpool, and accepts futures alive more than that lifetime.  Unfortunately, the code simply uses an unsafe transmute to use the rest of the threadpool code here.  I felt crossbeam's guarantee suffices for safety here, but I may be missing some / all corner cases.  Please find the gist here.
It would be great if you are willing to add this use-case.  The main reason is its absence in current executors (futures-rs / tokio-rs).  Otherwise, it would be great if you could make the (invaluable) data-structures public so we can build on them.

My understanding is that you'd like to have something like:
let v: String;
juliex::scope(|s| {
    s.spawn(async {
        println!("{}", v);
    }); // Here we automatically block on all tasks spawned inside this scope.
});

This seems reasonably straight forward to implement using crossbeam. But it would be even better if it didn't block. This is harder to achieve, and requires some effort in the implementation to get right.
Overall this seems quite useful, and probably worth investigating further!

@yoshuawuyts  What you are suggesting is interesting;  I did not think about it earlier.  However, here is a (possibly naive) question.  If I do not want the juliex::scope call to block, it also means we have to move all references into the block inside the juliex::scope call.  Otherwise, the outside stack-frame might collapse anytime independent of the references in the Future.
In this case, the Future might as well be 'static which is already handled isn't it?  Somewhere we need to guarantee that we're finishing up the thread-pool before a particular lifetime expires.  In my use-case, I wanted to use the values of the computation in the thread-pool, and it was natural to wait for it to finish.

Created a PR so we can try out different approaches as suggested above.
The PR still only does the naive integration with crossbeam but might help with other approaches.
I tried to sketch APIs for what is suggested above, but not sure I understand how to prevent the stack frame containing v (in the example two posts above) from collapsing without blocking the calling thread.


Tautomers with endocyclic double bonds should be preferred over exocyclic ones

Currently the tautomer scoring function does not seem to discriminate between tautomers with endocyclic double bonds vs exocyclic ones. In fact, both are aromatic according to the RDKit aromaticity model, but I still would prefer the endocyclic double bonds to score higher. After discussion with Greg we agreed this should be corrected.
This gist shows an example of the above:
https://gist.github.com/ptosco/20b06985cd8830d5e549165f6b9fc969

Yeah, we just hit two days ago as well. If  we can fix this and the missing stereochemistry, I'll be happy.
We're ending up with structures like this which are suboptimal :)

@bp-kelley I love "suboptimal", that's going to be my euphemism of choice : )
By missing stereochemistry, are you referring to the issue addressed by the new tautomerRemoveSp3Stereo CleanupParameter or something different?


CDI extension for Oracle Streaming Service (OSS)

Environment Details

Helidon Version: 1.1.1
Helidon MP

Enhancement Description
Add a CDI extension for Oracle Streaming Service. I think the client is the OCI SDK, so this should be similar to supporting the object store.

+1


Error when trying to place a bid on an online only lot in staging

You want to make sure this is in force too, it's their bug 👍

dupe of https://github.com/artsy/force/issues/5175


[Databox] SDK refresh to support new API version (2019-09-01)

Azure Spec PR : https://github.com/Azure/azure-rest-api-specs/pull/7317

Can one of the admins verify this patch?

@isra-fel  : I closed PR https://github.com/Azure/azure-sdk-for-net/pull/8176 since it was showing others commit as well somehow. so,  I have created a new PR and addressed your code review comments in it. Please take a look and let me know If you have any concerns
Thanks


Custom structural print

This PR aims to provide the host with a way to anonymize expressions with more deterministic mask values.

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.


Load all Network Images on Custom Splash Screen or Loading Screen.

i am using flutter web and  i want to load all network images to load on startup and till then show Loading Screen.
Please provide the code snippet :)

This seems more appropriate for stackoverflow or with a simple google search. You shouldn't create issues for things like these.

This seems more appropriate for stackoverflow or with a simple google search. You shouldn't create issues for things like these.

the provided methods include using cacheNetworking packages but still the thing is I want them to load while showing Loading Screen and use them as some kind of Image variable.

The Github issue tracker is for bugs and features requests for the Flutter SDK. Help and support questions like these are better suited for Q&A sites like StackOverflow or on one of the flutter community sites at https://flutter.dev/community .

The discussion is moved to StackOverflow here
===Closed====


Requirements

Hello,
I'm trying to evaluation but i'm getting some errors which is related to tensorflow, keras versions. Can you provide requirements with their versions if possible.
Thanks

I have the same problem. Aiuda.

I changed the imports for tensorflow.keras and i did another thing because it is not enough. You hace to change the name of one variable in this line https://github.com/kurapan/CRNN/blob/90df957f5fd8e19ec933f3e9c9d3c46741c89a4f/STN/spatial_transformer.py#L36
i just put this instead that:
self.trainable_weights1 = self.locnet.trainable_weights
and that's all!

i did an api for with this code, idk how works the legal licenses with this but you can see it in here.
if you know if i have to put something in my github for the license of the code please tell me.

if you know if i have to put something in my github for the license of the code please tell me.

I put MIT license to this repo so that it can be pretty much freely reused. You could do the same if you want. It should grant free reuse of the code without any liability.

I changed the imports for tensorflow.keras and i did another thing because it is not enough. You hace to change the name of one variable in this line
Do you use Keras API included in TensorFlow? I wrote this code with a standalone Keras installation and TF backend so making it work for tensorflow.keras certainly requires some changes.


Fixes #76

Added a filter on the string converter of NumberField to check if the input is a number
Added a WarningAlert class that displays a warning dialog with a custom message

Some thoughts:
I'd rather you not add to Tools. It's not something I'm particularly proud of, just a dumping ground I've yet to get rid of.
The setOnEditCommit argument probably warrants its own class. That'll make it easier to test as well. Maybe now's the time to start with Mockito.
I don't think that WarningDialog should be a thing. If we have a new class for the edit commit logic, we can push that code to a method there.


Object.freeze issue on IE11

Hi,
In this commit https://github.com/stonecircle/ember-body-class/commit/1df34bb5d6aaa7df7fc34ddfe12c5bfceea00254 you defined bodyClasses with Object.freeze.
This cause issue in our app for IE11 like so:

And IE11 refuse to boot the app.
If I remove the Object.freeze constraint, the app boot normally.
Why this constraint has been introduced in the first place?
Will you accept a PR for removing it?

@ctjhoa thanks for the issue, yes that's a great catch and I would be very open to a PR fixing this 👍

@mansona Any time to check my PR?


codehilite and fenced_code don't work together

I'm trying to get syntax highlighting to work with fenced code, but it's not cooperating.
Here's the contents of the input file, codetest.md:
~~~~{.r}
# r code
c(1,2,3)
~~~~

And here's what happens when I try to run it. Apparently there's some sort of problem with pygments? This is running on Ubuntu 11.10, python 2.7.2.
Traceback (most recent call last):
  File "/home/winston/.local/bin/markdown_py", line 5, in <module>
    pkg_resources.run_script('Markdown==2.1.0', 'markdown_py')
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 467, in run_script
    self.require(requires)[0].run_script(script_name, ns)
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 1200, in run_script
    execfile(script_filename, namespace, namespace)
  File "/home/winston/.local/lib/python2.7/site-packages/Markdown-2.1.0-py2.7.egg/EGG-INFO/scripts/markdown_py", line 34, in <module>
    run()
  File "/home/winston/.local/lib/python2.7/site-packages/Markdown-2.1.0-py2.7.egg/markdown/__main__.py", line 81, in run
    markdown.markdownFromFile(**options)
  File "/home/winston/.local/lib/python2.7/site-packages/Markdown-2.1.0-py2.7.egg/markdown/__init__.py", line 416, in markdownFromFile
    kwargs.get('encoding', None))
  File "/home/winston/.local/lib/python2.7/site-packages/Markdown-2.1.0-py2.7.egg/markdown/__init__.py", line 346, in convertFile
    html = self.convert(text)
  File "/home/winston/.local/lib/python2.7/site-packages/Markdown-2.1.0-py2.7.egg/markdown/__init__.py", line 280, in convert
    self.lines = prep.run(self.lines)
  File "/home/winston/.local/lib/python2.7/site-packages/Markdown-2.1.0-py2.7.egg/markdown/extensions/fenced_code.py", line 128, in run
    code = highliter.hilite()
  File "/home/winston/.local/lib/python2.7/site-packages/Markdown-2.1.0-py2.7.egg/markdown/extensions/codehilite.py", line 99, in hilite
    noclasses=self.noclasses)
  File "/usr/lib/python2.7/dist-packages/pygments/formatters/html.py", line 347, in __init__
    self.noclasses = get_bool_opt(options, 'noclasses', False)
  File "/usr/lib/python2.7/dist-packages/pygments/util.py", line 58, in get_bool_opt
    string, optname))
pygments.util.OptionError: Invalid type [False, 'Use inline styles instead of CSS classes - Default false'] for option noclasses; use 1/0, yes/no, true/false, on/off

It works fine if I use just codehilite (with indented code), or if I use fenced_code without codehilite.

Oh hey, I was having this same problem. I fixed it on my fork, with this commit.
I'll submit a pull request once Waylan either closes or accepts my current pull request.


docs: contributing.md typos and grammar

CONTRIBUTING.md for issue #48
Spelling and grammatical changes in docs/CONTRIBUTING.md.
I also changed:

instances of "participants/contributors" to just "contributors" and "maintainers/mentors" to "maintainers" for simplicity.
the word "Issue" to lowercase "issue" to match Github's docs.

@dejmedus If you are interested, please do contribute in more doc pages :)


Add additional context for "Authentic Signing Time"

          In this context it is not clear why we need two. We should add additional information for the **Authentic Signing Time** and how it differs from the **Signing Time**.  I understand that this is clarified in the linked section but the previous bullet has additional context making it clearer.

Originally posted by @toddysm in https://github.com/notaryproject/notaryproject/pull/256#discussion_r1199497862

Close this issue as "Authentic Signing Time" is explained in https://github.com/notaryproject/specifications/blob/main/specs/signature-specification.md#signing-time


Draw a rocket on CLI initialization

Draw a rocket 🚀 with characters on CLI init
Example:

Hey, I have just tried something. Please see #32.

Fixed in #32
Thank you @yigitcukuren !


Update Projects Page

OSPool only jobs
Add Jobs Column
Sort by Jobs default
Update all sorting methods
Round to integer
Underline text to make clickable rows obvious

https://osg-htc.org/web-preview/preview-project-v2


Keep build failed when try run on IOS simulator

I am using mac m1 machine to run this project however keep getting error when running on simulator, did anyone success running this project on mac m1 machine?
error 1:
** BUILD FAILED **

The following build commands failed:
        SwiftVerifyEmittedModuleInterface normal x86_64 Verifying\ emitted\ module\ interface\ CatCrypto.private.swiftinterface /Users/liho/Library/Developer/Xcode/DerivedData/AriesBifold-ebjmvcwivyukondoygpmxoqkkvib/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/CatCrypto.build/Objects-normal/x86_64/CatCrypto.private.swiftinterface (in target 'CatCrypto' from project 'Pods')
        SwiftVerifyEmittedModuleInterface normal x86_64 Verifying\ emitted\ module\ interface\ CatCrypto.swiftinterface /Users/liho/Library/Developer/Xcode/DerivedData/AriesBifold-ebjmvcwivyukondoygpmxoqkkvib/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/CatCrypto.build/Objects-normal/x86_64/CatCrypto.swiftinterface (in target 'CatCrypto' from project 'Pods')
        SwiftVerifyEmittedModuleInterface normal arm64 Verifying\ emitted\ module\ interface\ CatCrypto.private.swiftinterface /Users/liho/Library/Developer/Xcode/DerivedData/AriesBifold-ebjmvcwivyukondoygpmxoqkkvib/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/CatCrypto.build/Objects-normal/arm64/CatCrypto.private.swiftinterface (in target 'CatCrypto' from project 'Pods')
        SwiftVerifyEmittedModuleInterface normal arm64 Verifying\ emitted\ module\ interface\ CatCrypto.swiftinterface /Users/liho/Library/Developer/Xcode/DerivedData/AriesBifold-ebjmvcwivyukondoygpmxoqkkvib/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/CatCrypto.build/Objects-normal/arm64/CatCrypto.swiftinterface (in target 'CatCrypto' from project 'Pods')
(4 failures)

error 2:
The following build commands failed:
        CompileC /Users/liho/Library/Developer/Xcode/DerivedData/AriesBifold-ebjmvcwivyukondoygpmxoqkkvib/Build/Intermediates.noindex/AriesBifold.build/Debug-iphonesimulator/AriesBifold.build/Objects-normal/x86_64/BCWallet_vers.o /Users/liho/Library/Developer/Xcode/DerivedData/AriesBifold-ebjmvcwivyukondoygpmxoqkkvib/Build/Intermediates.noindex/AriesBifold.build/Debug-iphonesimulator/AriesBifold.build/DerivedSources/BCWallet_vers.c normal x86_64 c com.apple.compilers.llvm.clang.1_0.compiler (in target 'AriesBifold' from project 'AriesBifold')
        CompileC /Users/liho/Library/Developer/Xcode/DerivedData/AriesBifold-ebjmvcwivyukondoygpmxoqkkvib/Build/Intermediates.noindex/AriesBifold.build/Debug-iphonesimulator/AriesBifold.build/Objects-normal/arm64/BCWallet_vers.o /Users/liho/Library/Developer/Xcode/DerivedData/AriesBifold-ebjmvcwivyukondoygpmxoqkkvib/Build/Intermediates.noindex/AriesBifold.build/Debug-iphonesimulator/AriesBifold.build/DerivedSources/BCWallet_vers.c normal arm64 c com.apple.compilers.llvm.clang.1_0.compiler (in target 'AriesBifold' from project 'AriesBifold')
(2 failures)

The underline libraries are not supported to run on iOS simulators. That work is currently in progress by the upstream projects Aries Bifold, AFJ, etc.

The underline libraries are not supported to run on iOS simulators. That work is currently in progress by the upstream projects Aries Bifold, AFJ, etc.

You mean I able to run on IOS physical device?
But still mac m1 machine arm64 when build the project got failed...

@liho00 , there has been a number of changes. Please re-open and provide more details if this issue still persists


Update server url in readme example

Hi. The echo.websocket.org service is no longer available, so the example in the readme doesn't work.
I updated it to websocket-echo.com, which is also what ws uses in its readme.

LGTM


Rust Scrypt deprecation

This PR aims to replace rust-scrypt crate with the pure rust scrypt crate implementation for all platforms.
@r8d8 is there a reason to maintain rust-scrypt and not do this transition on unix?

Ran the banchmarks, encoding takes forever with both crates somehow.
Performance
scrypt
decode: 2,088,595 ns/iter (+/- 6,930)
encode: TimeOut
ryst-scrypt
decode:  2,085,978 ns/iter (+/- 206,936)
encode: Time Out

rust built-in benchmark was giving unreal results.
The most accurate real-life bench I had, is measurements of generation time for account through emerald-cli interface.

@r8d8 I've made custom bench for that, built-in benches couldn't chew through encoding.
Performance
scrypt
decrypt: 2090547 ns/iter
encrypt: 1279643 us/iter
rust-scrypt
decrypt:  2150338 ns/iter
encrypt:  1326455 us/iter
Though the difference is not very noticeable, scrypt is a bit faster then rust-scrypt
Do not merge this until #291 is merged, this PR depends on #291


Detected blocking call to open inside the event loop

2024-06-08 18:45:19.218 WARNING (MainThread) [homeassistant.util.loop] Detected blocking call to open inside the event loop by custom integration 'rinnai' at custom_components/rinnai/init.py, line 49: hass.data[DOMAIN][entry.entry_id][CLIENT] = client = await async_get_api( (offender: /usr/local/lib/python3.12/site-packages/botocore/loaders.py, line 172: with open(full_path, 'rb') as fp:), please report it to the author of the 'rinnai' custom integration
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in _run_code
File "/usr/src/homeassistant/homeassistant/main.py", line 223, in
sys.exit(main())
File "/usr/src/homeassistant/homeassistant/main.py", line 209, in main
exit_code = runner.run(runtime_conf)
File "/usr/src/homeassistant/homeassistant/runner.py", line 190, in run
return loop.run_until_complete(setup_and_run_hass(runtime_config))
File "/usr/local/lib/python3.12/asyncio/base_events.py", line 672, in run_until_complete
self.run_forever()
File "/usr/local/lib/python3.12/asyncio/base_events.py", line 639, in run_forever
self._run_once()
File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1988, in _run_once
handle._run()
File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run
self._context.run(self._callback, *self._args)
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 742, in async_setup_locked
await self.async_setup(hass, integration=integration)
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/rinnai/init.py", line 49, in async_setup_entry
hass.data[DOMAIN][entry.entry_id][CLIENT] = client = await async_get_api(

@sambeetm I just wanted to check and see if you are still seeing this. I haven't got it in my instance yet.

Yes, this issue still present in latest release of HA. Its filling-up the logfile.
2024-06-28 22:17:05.794 WARNING (MainThread) [homeassistant.util.loop] Detected blocking call to open inside the event loop by custom integration 'rinnai' at custom_components/rinnai/init.py, line 49: hass.data[DOMAIN][entry.entry_id][CLIENT] = client = await async_get_api( (offender: /usr/local/lib/python3.12/site-packages/botocore/loaders.py, line 172: with open(full_path, 'rb') as fp:), please report it to the author of the 'rinnai' custom integration
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in _run_code
File "/usr/src/homeassistant/homeassistant/main.py", line 223, in
sys.exit(main())
File "/usr/src/homeassistant/homeassistant/main.py", line 209, in main
exit_code = runner.run(runtime_conf)
File "/usr/src/homeassistant/homeassistant/runner.py", line 190, in run
return loop.run_until_complete(setup_and_run_hass(runtime_config))
File "/usr/local/lib/python3.12/asyncio/base_events.py", line 672, in run_until_complete
self.run_forever()
File "/usr/local/lib/python3.12/asyncio/base_events.py", line 639, in run_forever
self._run_once()
File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1988, in _run_once
handle._run()
File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run
self._context.run(self._callback, *self._args)
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 742, in async_setup_locked
await self.async_setup(hass, integration=integration)
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/rinnai/init.py", line 49, in async_setup_entry
hass.data[DOMAIN][entry.entry_id][CLIENT] = client = await async_get_api(

I may have a solution to this. I am going to run it a little longer in my instance and then if all goes well I will publish a release.

if it's any help to you, I'm happy to help test/debug

I dropped the ball on this one. I have tested and confirmed the issue is fixed, but I forgot I didn't publish a new release yet. I will get this out today.


How to get best video and audio with subs and then mux subs in a MKV file?

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.02.11. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[X] I've verified and I assure that I'm running youtube-dl 2018.02.11

Before submitting an issue make sure you have:

[X] At least skimmed through the README, most notably the FAQ and BUGS sections
[X] Searched the bugtracker for similar issues including closed ones
[X] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[X] Question
[ ] Other

Description of your issue, suggested solution and other information
I can't download video and subtitles and then mux into a MKV file. I've tried many combinations but they didn't work.
My command line:
youtube-dl -i --prefer-ffmpeg --no-overwrites --download-archive motivacionais.txt --no-post-overwrites -f best --merge-output-format mkv --all-subs --convert-subtitles srt --embed-subs --config-location D:/AUTO/gabyoutubevideos.conf "https://www.youtube.com/channel/UC8PICQUP0a_HsrA9S4IIgWw/"

All I get is a MP4 file. Youtube-dl downloads the subs and then remove them.

Thanks! It works!


Implement New Album Cover

Need to replace album cover with higher resolution photo

Here's the album cover. If it's not high-res enough for you, the Illustrator file is in the Goggle Drive folder.

This is resolved, please verify.


Simplify date question

This all came out of the work on the new HMRC Smart Answer (calculate-your-part-year-profits). I found the implementation of Question::Date to be somewhat more complicated than it needed to be. This pull request aims to simplify the class.
In summary, I've:

Removed the unused Question::Date#default method.

Simplified the implementations of Question::Date#from, #to, #default_day, #default_month, #default_year so that they can only be set by providing a block rather than by providing a block or a value argument.

Removed Question::Date#defaulted_day?, #defaulted_month? and #defaulted_year? and replaced their use with Date#default_day, #default_month and #default_year.

DRYed up calls to the blocks stored in @from_func and @to_func.

Restricted the type of exceptions we rescue in Question::Date#parse_input and #to_response. We were previously rescuing everything that inherits from StandardError, which isn't ideal as it means we might end up swallowing genuine exceptions.

Simplified the implementation of Question::Date#parse_input when it's parsing a date supplied as a hash.

Other than my minor comments/suggestions, this LGTM :+1:

I've just rebased this on master and force pushed to this remote branch.

I've force pushed again now that I've fixed the failures in date_question_test.rb.


使用pip isntall itchar 安装时报错:

使用pip isntall itchar 安装时报错:
could not find a version taht satisfies the requirement itchar (from versions:)No matching distribution found for itchar
怎么解决呢？
python版本python 3.7
Originally posted by @Liushuo12138 in https://github.com/littlecodersh/ItChat/issues/1#issuecomment-485743473

你项目名称写错了

@taobaogd You have to use pip3 install itchat, note that is itchat but not itchar


Add sandbox app

This PR creates a Rails 5 sandbox application that can be used to test changes, reproduce issues, general debugging etc.

This does not get added to the gem package itself since it all lives under the spec directory https://github.com/smridge/swagcov/blob/23e1d900ecc8316b171fcd06e8a151eedb446e70/swagcov.gemspec#L19

LGTM! I wonder what will be the effort to maintain dependencies on this thing. I guess we'll have to wait and see.

That's a great thought! Definitely a wait and see 🤞 🤷♀


Support for Sensu 2 (Go)

The latest version of Uchiwa for Docker does not appear to work nicely with Sensu 2.0.0-beta.3.
Expected Behavior
Sensu dashboard to show up in Uchiwa when configured like so:
{
  "sensu": [
    {
      "name": "Testing",
      "host": "<omitted>",
      "port": 8080
    }
  ],
  "uchiwa": {
    "host": "<IP_ADDRESS>",
    "port": 3000
  }
}

Current Behavior

No dashboards.
Possible Solution
I believe this is possibly because it is still configured for the previous version where Redis was being used instead of etcd ?
Steps to Reproduce (for bugs)

Start Sensu

docker run -d --name sensu-backend \
-p 2380:2380 -p 8080:8080 -p 8081:8081 \
sensu/sensu:2.0.0-beta.3 sensu-backend start

Start Uchiwa

docker run -d -p 3000:3000 -v ~/uchiwa.json:/config/config.json uchiwa/uchiwa

Context
N/A
Your Environment

Uchiwa version used: 1.3.1-1
Sensu version used: 2.0.0-beta.3
Operating System and version (e.g. Ubuntu 14.04):

  Operating System: Arch Linux
  Kernel: Linux 4.18.4-arch1-1-ARCH
  Architecture: x86-64

Hi @MrSaints, thanks for your request. Although it's not well-documented at this time, Sensu 2.0 provides a built-in web UI. As a result, there are no plans to implement support for Sensu 2.0 in Uchiwa.
Like you've come to expect from Uchiwa, Sensu 2.0 built-in web UI listens on port 3000 by default. I believe you can simply add -p 3000:3000 to your existing docker run  command for sensu-backend and you'll be able to access the new web UI in your browser.


js inproper syntax highlighting

VSCode Version:
Version 1.14.0
Commit b8cd5b9556d8b70ea560d35b903422363f6c5c40
Date 2017-07-10T18:36:31.751Z
Shell 1.6.6
Renderer 56.0.2924.87
Node 7.4.0

OS Version:
Microsoft Windows 7 Professional

Steps to Reproduce:

Open .js file with content:
var x = { '01': '<' }; console.log('c', x);
Code after < char have wrong syntax highlightingting.

Correct:

Reproduces without extensions: Yes
Thanks!

Thanks. I've opened https://github.com/Microsoft/TypeScript-TmLanguage/issues/487 to track this upstream


Move to Github Actions

I noticed that the travis build is no longer working so I thought it might be a good opportunity to switch to Github Actions.
If you don't want to switch feel free to just close this PR. Otherwise leave a comment and I'll continue working on it.
https://github.com/somleng/adhearsion/runs/5282043628?check_suite_focus=true
Todo:

[x] Configure (or remove) code climate (No changes needed)
[x] Fix cucumber
[x] Remove .travis.yml
[x] Test against latest Ruby 2.x and 3.x (depends on https://github.com/adhearsion/adhearsion/pull/654)

After a bit of research into this, it appears that Travis isn't a good fit for Adhearsion right now. Thanks for making a start on the migration to Github Actions

I think this is done. Github Actions is configured to run against the latest ruby 2.x and 3.x versions. Here's a link to the build on my fork.


Switch to RV32E toolchain

We'd like to use the RV32E toolchain (which uses only 16 registers) for a couple of reasons. One is to save registers for internal use, which may make the wrapper/harness code that's used to run Athena programs in ZK simpler. Another is for possible performance gains when running on hardware with 16 registers such as amd64 (although preliminary results on this are mixed).
Note that LLVM mainlined support for RV32E but this support is still considered experimental:
https://github.com/llvm/llvm-project/blob/ae1596a31a6fac2f4daafe1e256d4a5cf3742617/llvm/docs/RISCVUsage.rst?plain=1#L52-L57
https://github.com/llvm/llvm-project/blob/ae1596a31a6fac2f4daafe1e256d4a5cf3742617/llvm/docs/RISCVUsage.rst?plain=1#L220-L221
Another issue here is lack of stdlib support in Rust. Right now we're using https://github.com/succinctlabs/rust which, just like risczero, includes partial stdlib support. This uses RV32IM (the riscv32im-succinct-zkvm-elf Rust target, to be exact).

Making good progress on this. Was able to compile a toolchain, see https://github.com/athenavm/rustc-rv32e-toolchain. Still getting some errors about missing atomics (and updating the target settings doesn't seem to help), but found a temporary workaround. To do:

finish recompiling tests and examples using the new target
finish updating cargo athena build to use the new toolchain/target
figure out why the elf files are so much larger and now always contain debug info unless stripped (adding some optimizations in Cargo.toml seems to help); check if we want to strip
investigate why the succinct toolchain doesn't run into the missing atomic operation errors

I put the RV32E stuff behind a feature flag. The VM should work fine with both. But having full support for both would mean two sets of compiled examples, two sets of compiled tests, support for building and downloading two different toolchains, etc. - I didn't go so far as to enable all of that. Can revisit in future.


GeoPySpark 0.3.0

Tested this using
cd terraform-nodocker/
terraform init
terraform apply

... and the cluster has artifacts of correct version.
To provide OAuth credentials I created GitHub app to authenticate against.

@echeipesh It looks like you forgot to update the jar version in the travis script: https://travis-ci.org/geodocker/geodocker-jupyter-geopyspark/builds/314484366#L643-L645

Superseded by https://github.com/geodocker/geodocker-jupyter-geopyspark/pull/73

oops ... I didn't see this.


investigate .gitattributes file usages

During research I noticed that a lot of repo's contain a .gitattributes file.
This has something to do with line endings of files. Do we need this in our project?
https://help.github.com/articles/dealing-with-line-endings/

Does anyone have a usecase?

No use case provided, no longer relevant. Closing.


Panoptic segmentation task

Hi, @nuneslu
Thanks for your great work. Would you please share the code for Panoptic segmentation task?

Hi, we took a while to release the panoptic segmentation code but here it is now: https://github.com/PRBonn/MinkowskiPanoptic

Good job. Thank you very much.


Save current work

Just to save my current work

Codecov Report

Merging #41 into master will decrease coverage by 0.22%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master     #41      +/-   ##
=========================================
- Coverage    8.19%   7.96%   -0.23%
=========================================
  Files           9       9
  Lines        1074     979      -95
=========================================
- Hits           88      78      -10
+ Misses        973     888      -85
  Partials       13      13

Impacted Files
Coverage Δ

firewall.go
0.00% <0.00%> (ø)

machine.go
9.96% <0.00%> (-0.61%)
:arrow_down:

metal.go
57.14% <ø> (-0.86%)
:arrow_down:

network.go
0.00% <0.00%> (ø)

size.go
0.00% <0.00%> (ø)

image.go
0.00% <0.00%> (ø)

switch.go
0.00% <0.00%> (ø)

... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a19565f...53f732f. Read the comment docs.

replaced by #45


list without json format

Comparing with lv2ls, lv2ls gives just a list of LV2 plugins/urls, while clap-info -l dumps it to stdout in json format. I think the lv2ls is useful on Unix systems, while it's easier to pipe with Unix tools like grep and such.
Clap-info could provide both ways probably.

Why not just use jq?
paul ~/dev/music/clap-projects/clap-info (upgrade-118) % ./cmake-build-debug/clap-info -l | jq -r '.result[]'
/Users/paul/Library/Audio/Plug-Ins/CLAP/Stochas.clap
/Users/paul/Library/Audio/Plug-Ins/CLAP/Dexed.clap
/Users/paul/Library/Audio/Plug-Ins/CLAP/Diopser.clap
/Users/paul/Library/Audio/Plug-Ins/CLAP/Surge XT.clap
/Users/paul/Library/Audio/Plug-Ins/CLAP/Surge XT Effects.clap
/Users/paul/Library/Audio/Plug-Ins/CLAP/Odin2.clap
/Users/paul/Library/Audio/Plug-Ins/CLAP/PaulXStretch.clap
/Users/paul/Library/Audio/Plug-Ins/CLAP/tuning-workbench-synth.clap
/Users/paul/Library/Audio/Plug-Ins/CLAP/Shortcircuit XT.clap
/Users/paul/Library/Audio/Plug-Ins/CLAP/tuning-note-claps.clap
/Users/paul/Library/Audio/Plug-Ins/CLAP/LauridsenSchodderStereo.clap

That's a option, but do you think that's a convenient option? I think clap-info should provide it.

well. yes. the idea we had with clap-info was to produce a variety of structured data you could read with tools!
If you want to add a --list-raw option I guess i would merge it, but really as you know the unix philosophy is to pipe together programs to do transforms, and jq is a great example of that.
./cmake-build-debug/clap-info -s | jq '.result[].plugins[].id'

for instance extracts all the ids from a scan
I don't think we want to add 100 options to take all the permutations of data when we can just output the data and use a widely available tool to parse it.

oooh neat like here's another fun one
paul ~/dev/music/clap-projects/clap-info (upgrade-118) % ./cmake-build-debug/clap-info -s | jq '.result[]."clap-version"'  | sort | uniq
"0.21.0"
"1.0.0"
"1.0.2"
"1.1.1"
"1.1.8"

Who says 100 options? This is a request for one default/ reasonable option.

yeah so i get what your request is. For one and only one case don't output json so you don't have to pipe and if anyone asks for any other case to do the same, say no, and tell them to use jq.
Like I said if you want to add --list-raw i'd merge it I guess. but the tool is really meant to be used in conjunction with a pipeline which can read json.

Anyway if you want to do that then
https://github.com/free-audio/clap-info/blob/271a6f3e4c863656fbc3de611ad5327b50ef89a9/src/clap-info/main.cpp#L148
that's where we make the list json. Add another option above for list raw, copy that branch to instead of appending to doc, dump to stdout, and then suppress the output of the document on return using the mechanism in the error paths elsewhere

oh finally, clap-info is written so the scanner is a separate library from the executable itself, so you can use the C++ software to write other scan and inspect tools of your choosing!


closing dropdown which is out of the viewport causes viewport to scroll

Version: Select2 4.0
Example: https://jsfiddle.net/z7L1px15/
To reproduce:

In the above fiddle click the select2 dropdown to open it.
Scroll the viewport down to the 2nd input and click it to focus on it

What is expected:
The 2nd input will have the focus and be visible.
What happens:
The 2nd input receives the focus, but the viewport scrolls up to the select2 dropdown so the focused input is no longer visible.
This seems like somewhat of an edge case but we see people hit it with some regularity on a longish form where the submit button is at the bottom of the page.  They have to scroll back down to the bottom and click the submit button again for the form to be submitted.
The line responsible is https://github.com/select2/select2/blob/master/src/js/select2/selection/base.js#L84
That line returns the focus to the element after the dropdown closes. I think what we really want is for this focus event to occur only when a value is selected, not every time the dropdown is closed.

Yup, was able to confirm this with the jsfiddle.

This can also be observed in the following case:

Click select2 dropdown to open it
Scroll so that the dropdown is out of view

The viewport is scrolled back to the dropdown

If I remove line you mentioned that causing this bug, focus will not be set and window will not be scrolled up, but if have submit button instead of 2nd input, submit event will not work.
Does anybody have solution / workaround for this bug?

There is some discussion in https://github.com/select2/select2/issues/4983#issuecomment-331560917.  @roman4e @zoka123 @jpic I have closed #4983 as a duplicate, so we will keep this open as the canonical issue.

Any update on this? Having the same issue with a long page that has multiple select2 boxes. Not desirable to have the page scroll to the previous select2 if no selection is made and a click event occurs elsewhere.

I just submitted an pull-request, my change will not trigger focus() on element if there was an outside click

The continuation of this discussion will be inside PR -> https://github.com/select2/select2/pull/5353

Please reopen issue #3663.

What about using preventScroll parameter for focus event? https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/focus


Upgrade travis build

Moving travis build to xenial instead of trusty, as this last one won't make our pypy tests working. This includes some linting fixes & minor test fixes.

Coverage decreased (-0.03%) to 81.9% when pulling 9c2a48e13e257cba43bd6480e78e04c8d45df176 on mycroft:upgrade_travis into bc100512116ba515c7e271436be59b78ef6fb970 on criteo:master.

Coverage decreased (-0.03%) to 81.9% when pulling 9c2a48e13e257cba43bd6480e78e04c8d45df176 on mycroft:upgrade_travis into bc100512116ba515c7e271436be59b78ef6fb970 on criteo:master.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_96d12b6f-6f6c-4ae0-a3f2-1c63d63cdc70/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_96d12b6f-6f6c-4ae0-a3f2-1c63d63cdc70.git

Docs Build status updates of commit 4964a5f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-07-14-03-35-34 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[Event Request] Table Segment Line procedure CreateInteractionFromInteractLogEntry

Hello,
Is it possible to add a new event publisher in the function procedure CreateInteractionFromInteractLogEntry:
 procedure CreateInteractionFromInteractLogEntry(var InteractionLogEntry: Record "Interaction Log Entry")
    var
        Cont: Record Contact;
        Salesperson: Record "Salesperson/Purchaser";
        Campaign: Record Campaign;
        Task: Record "To-do";
        Opportunity: Record Opportunity;
    begin
        OnBeforeCreateInteractionFromInteractLogEntry(Rec, Salesperson); // <<DevOps104207>>

        if Task.Get(InteractionLogEntry.GetFilter("To-do No.")) then begin
....
  [IntegrationEvent(false, false)]
    local procedure OnBeforeCreateInteractionFromInteractLogEntry(var SegmentLine: Record "Segment Line"; var Salesperson: Record "Salesperson/Purchaser")
    begin
    end;

Thank you!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 43300.


routing with bboxes seems to ignore the bboxes

import gdsfactory as gf

c = gf.Component()
columns = 2
ptop = c << gf.components.pad_array(columns=columns, port_orientation=270)
pbot = c << gf.components.pad_array(port_orientation=270, columns=columns)
ptop.dmovex(300)
ptop.dmovey(300)

obstacle = c << gf.c.rectangle(size=(300, 100), layer="M3")
obstacle.dymin = pbot.dymax - 10
obstacle.dxmin = pbot.dxmax - 10

routes = gf.routing.route_bundle_electrical(
    c,
    pbot.ports,
    ptop.ports,
    start_straight_length=100,
    separation=20,
    bboxes=[
        obstacle.bbox(),
        pbot.bbox(),
        ptop.bbox(),
    ],  # will be able avoid obstacles soon
    sort_ports=True,
)

c.show()

Partially fixed by https://github.com/gdsfactory/kfactory/pull/388
At least for starting/end bboxes. It is howerver not implemented for general obstacles like this one (yet) or waypoints.

Also in this special example, there is no way to avoid that obstacle unless I am missing something. You can't just avoid it with 20um separation and still have space to route


[Suggestion] Allow /role commands to be executed via command block.

I don't really know if this is possible, but it would be really nice if /role commands could be executed via command block.
For instance, perhaps as a measure to stop spam, when a new player spawns into the world, they are assigned a muted role to mute them. Then the player needs to leave the spawn area or push a button to be removed from the muted role and gain the ability to speak.
Just an example, but I'm sure there are other instances where this might be handy in making minigames as well.

This already works- you'd just need to give command blocks level 4 permissions. For example, the same applies to the /stop command which can only be run by command blocks if they are given level 4 permissions.

This already works- you'd just need to give command blocks level 4 permissions. For example, the same applies to the /stop command which can only be run by command blocks if they are given level 4 permissions.

Perhaps I'm ignorant, but how does one elevate the permission level of command blocks? I apologize. I know this isn't a forum for this kind of help, but I'm just not familiar with how to give permission levels to command blocks and Google did not have the answer.

Perhaps I'm ignorant, but how does one elevate the permission level of command blocks? I apologize. I know this isn't a forum for this kind of help, but I'm just not familiar with how to give permission levels to command blocks and Google did not have the answer.

Oh- I am mistaken! Sorry! I was thinking of the permission level for functions in the server.properties, but command blocks are hardcoded to permission level 2.
On the same reasoning that /stop is not accessible from command blocks, I think /role from command blocks is generally not a good idea.
What might be worth doing in the mod, though, is potentially having the ability to assign a specific role to command blocks / functions.
For now, the solution would be to wrap this command in a function with the function permission level set to 4 in the server.properties.

Oh- I am mistaken! Sorry! I was thinking of the permission level for functions in the server.properties, but command blocks are hardcoded to permission level 2.
On the same reasoning that /stop is not accessible from command blocks, I think /role from command blocks is generally not a good idea.
What might be worth doing in the mod, though, is potentially having the ability to assign a specific role to command blocks / functions.
For now, the solution would be to wrap this command in a function with the function permission level set to 4 in the server.properties.


It takes about 2 hours to compile the file, and the token is only valid for 1 hour

How can I solve this problem?
Is there any alternative to this token？
  release:
    name: Create Release
    needs: build
    runs-on: ubuntu-latest
    steps:
      - name: Download OpenWrt_firmware
        uses: actions/download-artifact@v1
        with:
          name: OpenWrt_firmware
      - name: Create Release
        id: create_release
        uses: actions/create-release@v1
        env:
          RELEASES_TOKEN: ${{ secrets.RELEASES_TOKEN }}
        with:
          tag_name: ${{ github.ref }}
          release_name: Release ${{ github.ref }}
          body: ${{ github.ref }}
          draft: false
          prerelease: false
      - name: Upload Release Asset
        id: upload-release-asset
        uses: actions/upload-release-asset@v1
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        with:
          upload_url: ${{ steps.create_release.outputs.upload_url }}
          asset_path: ./OpenWrt_firmware
          asset_name: ${{ env.FILE_NAME }}-${{ steps.create_release.outputs.tag }}.zip
          asset_content_type: application/zip

Solved with svenstaro/upload-release-action


Package Management not Working

Currently, I am unable to install the .yaml due to ResolvePackage errors, likely due to a difference in operating systems. Downloading JJ's compressed environment also did not work, likely due to similar reasons.
--freeze-installed might be an option to stop updating preexisting packages.
Or we could try something called package pinning.

Now have a system that works, via gihub actions


As a data developer, I want to easily update the data pickles

Description
If we added a simple global constant like UPDATE_PICKLES = False to the data/data-pipeline/data_pipeline/etl/score/tests/test_score_post.py file, then when you need to update the pickles, you just set that parameter to True.
And within each test, it would be something like:
if UPDATE_PICKLES:
  data_path = Path.cwd()
  score_transformed_actual.to_pickle(data_path / "data_pipeline" / "etl" / "score" / "tests" / "snapshots" / "score_transformed_expected.pkl", protocol=4)
  pytest.fail("Test fails because `UPDATE_PICKLES` is True.")
else:
   pdt.assert_frame_equal(
        score_transformed_actual, score_transformed_expected, check_dtype=False
    )

We would never set UPDATE_PICKLES = True in the actual code in the app, it would just be used to temporarily update these things.

Potentially copy or expand upon framework developed by https://github.com/usds/justice40-tool/pull/1249


Weird tsan flake

While testing an updated version of Absei (not a release, just using Abseil's HEAD) in draft PR https://github.com/googleapis/google-cloud-cpp/pull/5130, our tsan build failed with a strange stack trace. After a re-run everything passed. The stack trace doesn't really include much Abseil code, so it doesn't look like an Abseil bug, but I'm not sure.
Failed log: https://source.cloud.google.com/results/invocations/91181182-5c03-4ed0-b81d-9709b2af3526/targets/cloud-cpp%2Fgithub%2Fgoogle-cloud-cpp%2Fmaster%2Fdocker%2Ftsan-presubmit/log
SUMMARY: ThreadSanitizer: data race /proc/self/cwd/google/cloud/pubsub/samples/samples.cc:1180:13 in (anonymous namespace)::SubscriberRetrySettings(std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > const&)::$_2::operator()(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >) const::'lambda'(google::cloud::pubsub::v1::Message const&, google::cloud::pubsub::v1::AckHandler)::operator()(google::cloud::pubsub::v1::Message const&, google::cloud::pubsub::v1::AckHandler) const

I tried about 20x to reproduce this on my local machine. I succeeded in reproducing a tsan error on my local  machine one time, and the stack trace had nothing to do w/ pubsub (as the one in the log above does). Here's the tsan error from my local machine.
local-tsan-error.txt
SUMMARY: ThreadSanitizer: data race /usr/bin/../lib/gcc/x86_64-redhat-linux/9/../../../../include/c++/9/bits/unique_ptr.h:291:6 in std::unique_ptr<grpc_impl::Alarm, std::default_delete<grpc_impl::Alarm> >::~unique_ptr()

I now seem to be unable to reproduce the problem again.

Note: both of the above tsan logs contain stack frames like
    #8 google::cloud::v1::internal::DefaultCompletionQueueImpl::Run() /proc/self/cwd/google/cloud/internal/default_completion_queue_impl.cc:134:3 (libgoogle_Scloud_Slibgoogle_Ucloud_Ucpp_Ugrpc_Uutils.so+0x34d87)
    #9 google::cloud::v1::CompletionQueue::Run() /proc/self/cwd/./google/cloud/completion_queue.h:46:23 (libgoogle_Scloud_Slibgoogle_Ucloud_Ucpp_Ugrpc_Uutils.so+0x3468d)
    #10 google::cloud::v1::internal::AutomaticallyCreatedBackgroundThreads::AutomaticallyCreatedBackgroundThreads(unsigned long, google::cloud::v1::internal::AutomaticallyCreatedBackgroundThreads::NormalizeTag)::$_0::operator()()

/cc @coryan and @devbww who have experience with some of this code and may have some ideas

The stack trace at https://github.com/googleapis/google-cloud-cpp/issues/5141#issuecomment-697996806  is basically a thread blocked on the completion queue event loop.
I cannot figure out where exactly the problem is for the build: there are no symbols in those stack traces... the LLVMSymbolizer says it cannot read some file, but does not say which, sigh...
Your local tsan error is in spanner, while the build error is in pubsub... the local tsan error is a problem with a grpc::Alarm looks like it is canceled by one thread and deleted by another (maybe it is expiring as it is being deleted?).  That might be easier to figure out.

I now seem to be unable to reproduce the [original] problem again.

We probably shouldn't be surprised by that if it is a rare data race.
The question, it seems, is whether after
  auto session = subscriber.Subscribe(..., handler, ...);

that
  session.cancel();
  auto status = session.get();

implies that we are now after every execution of handler.
I recall discussing this during a review of similar code, but don't remember the details of the argument.  TSan, at least, seems to think that a write to (I presume) count in the handler occurred after the (otherwise unsynchronized) read after the cancel/get in the main thread.
We don't see any of the output from SubscriberRetrySettings(), even though reaching line 1180 or line 1193 implies there should be some, and that makes it harder to discern what might be going on.  Not that I understand the expected execution well in the first place.  :-(  I'm not sure what kExpectedMessageCount signifies, for example, given how many messages the previous cases actually received.

I now seem to be unable to reproduce the [original] problem again.

We probably shouldn't be surprised by that if it is a rare data race.

Specially since I ran these tests thousands of times with TSAN until I did not get more races.  That is, whatever races are left (if any) are even more rare than usual.

The question, it seems, is whether after
  auto session = subscriber.Subscribe(..., handler, ...);

that
  session.cancel();
  auto status = session.get();

implies that we are now after every execution of handler.

That is the intent.  The session counts how many requests are outstanding, and .get() should only return when that is zero.  The definition of "outstanding" is material, a handler is considered "no longer outstanding" when the AckHandler::ack() or AckHandler::nack() request is made... I will go into why in a second, but the code says:
https://github.com/googleapis/google-cloud-cpp/blob/aa2c541e58ec53ea0dbb1cbf108b262aed0e8f49/google/cloud/pubsub/samples/samples.cc#L1194-L1202
you will note that the mutex and the counter and a bunch of other stuff is accessed after the std::move(h).ack() call.  As to why we do not treat the return as "no longer outstanding": this is to support asynchronous handlers that delegate their work to other thread.
Maybe we should consider a handler outstanding until both the handler return and the AckHandler is used... let me file a bug.  But I think fixing this sample would not be hard.

I recall discussing this during a review of similar code, but don't remember the details of the argument. TSan, at least, seems to think that a write to (I presume) count in the handler occurred after the (otherwise unsynchronized) read after the cancel/get in the main thread.

See above. Just trying to explain what was going on made the bug kind of obvious, maybe.

Thanks.  It seems, then, that this could be more easily reproduced (and hence any fix tested) by temporarily sleeping after the ack().

Maybe we should consider a handler outstanding until both the handler return and the AckHandler is used

Sounds reasonable, but perhaps could be seen as too pessimistic.  Maybe we should just make it clear(er) that any state that needs to be visible after the get() has to be set before the ack()/nack().

There were errors reported in spanner too, we have not discussed those.

There were errors reported in spanner too, we have not discussed those.

Consider the body of the DefaultCompletionQueueImpl::Run() loop ...
    auto op = FindOperation(tag);
    if (op->Notify(ok)) {
      ForgetOperation(tag);
    }

and what happens if op is an AsyncTimerFuture that is cancelled (asynchronously) somewhere during that sequence.
AsyncTimerFuture::Cancel() will std::move(alarm_), writing to it.
The loop's op shared ptr will hold the last reference (as AsyncTimerFuture::Notify() always succeeds), so we'll run ~AsyncTimerFuture(), which will read alarm_`.
But there is no synchronization between the write and the read.

Thanks, I think I can fix the CompletionQueue bug too.  Let me know if I am missing something, but I think we can mark this bug as fixed once I merge those fixes.


Code examples for each usage case

The readme file is quite thorough and detailed, but it can be too much.
It would be nice to add some examples, containing the actual metaflow code and requirements file being used, for each to the main use cases.

That's a good suggestion. I will add a few examples of some of the environments I use to test things out.

The documentation has some examples of external requirements.txt files taht can work. Are you also looking for stuff inside @conda or @pip in a Metaflow flow?

Yes i would love too to have some short examples, to illustrate the use of main variables
Thank you for this package

I'll try to add some more examples.


Permissions

Currently I am not able to assing the issues to the project board, nor assign people, nor add labels to issues.
This means issues not opened by you will not show on the project board unless you add them @IgNoRaNt23.
This can be fixed by giving us the permissions in the seetings tab of the organization. More here

Switched general permissions to 'Write' for everyone. Does this solve your issue?

Yea this does thank you


Improvement - Fix button story not showing icon in basic example

Describe pull-request
Describe what the pull-request is about
In the button story in storybook, when enabling icons in the basic example, the icon won't show. I have added the import for web-font icons.
How to test
Add description of how to test if possible

Go to button story basic example
Try adding an icon with the control
Icon is now added

Additional context
Add any other context about the pull-request here.
There are some improvement that needs to be done with the icon inside the button.

Looks good to me. It fixes the issue.
@alexlioliopoulos - If we do not have, please make a ticket for an issue related to text and icon not being in the horizontal center. It is not related to Mena's change - it was there from before.

Good point!
Also now for you @alexlioliopoulos , you should be able to see the button with icons now

Nice Mena! Good point Johnny, I will add this to the backlog (if not already there)!


Undefined reference to roundeven functions

I did a little poking around and didn't find the source of this issue.  It seems like it should be using the gcc builtins for this, but instead are looking to musl which doesn't implement those at the moment.  I did at least verify that this gcc version has those builtins via __has_builtin.
Compiler: GCC 14.2.0
Libc: Musl 1.2.5
test command:  meson test --no-rebuild --print-errorlogs -C output
ninja: job failed: gcc  -o test/arm/neon/rndn-native-c test/arm/neon/rndn-native-c.p/rndn.c.o -Wl,--as-needed -Wl,--no-undefined -pie -Wl,--as-needed,-O1,--sort-common -Wl,-z,pack-relative-relocs -Os -fstack-clash-protection -Wformat -Werror=format-security -fno-plt -lm
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-native-c.p/rndn.c.o: in function `test_simde_vrndnq_f64':
rndn.c:(.text+0x619): undefined reference to `roundeven'
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-native-c.p/rndn.c.o: in function `test_simde_vrndn_f32':
rndn.c:(.text+0x6cb): undefined reference to `roundevenf'
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-native-c.p/rndn.c.o: in function `test_simde_vrndnh_f16':
rndn.c:(.text+0x7d6): undefined reference to `roundevenf'
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-native-c.p/rndn.c.o: in function `test_simde_vrndn_f64':
rndn.c:(.text+0x940): undefined reference to `roundeven'
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-native-c.p/rndn.c.o: in function `test_simde_vrndnq_f32':
rndn.c:(.text+0x9db): undefined reference to `roundevenf'
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-native-c.p/rndn.c.o: in function `test_simde_vrndnq_f16':
rndn.c:(.text+0xafc): undefined reference to `roundevenf'
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-native-c.p/rndn.c.o: in function `test_simde_vrndn_f16':
rndn.c:(.text+0xc36): undefined reference to `roundevenf'
collect2: error: ld returned 1 exit status
ninja: job failed: gcc  -o test/arm/neon/rndn-emul-c test/arm/neon/rndn-emul-c.p/rndn.c.o -Wl,--as-needed -Wl,--no-undefined -pie -Wl,--as-needed,-O1,--sort-common -Wl,-z,pack-relative-relocs -Os -fstack-clash-protection -Wformat -Werror=format-security -fno-plt -lm
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-emul-c.p/rndn.c.o: in function `test_simde_vrndnq_f64':
rndn.c:(.text+0x619): undefined reference to `roundeven'
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-emul-c.p/rndn.c.o: in function `test_simde_vrndn_f32':
rndn.c:(.text+0x6cb): undefined reference to `roundevenf'
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-emul-c.p/rndn.c.o: in function `test_simde_vrndnh_f16':
rndn.c:(.text+0x7d6): undefined reference to `roundevenf'
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-emul-c.p/rndn.c.o: in function `test_simde_vrndn_f64':
rndn.c:(.text+0x940): undefined reference to `roundeven'
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-emul-c.p/rndn.c.o: in function `test_simde_vrndnq_f32':
rndn.c:(.text+0x9db): undefined reference to `roundevenf'
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-emul-c.p/rndn.c.o: in function `test_simde_vrndnq_f16':
rndn.c:(.text+0xafc): undefined reference to `roundevenf'
/usr/lib/gcc/x86_64-alpine-linux-musl/14.2.0/../../../../x86_64-alpine-linux-musl/bin/ld: test/arm/neon/rndn-emul-c.p/rndn.c.o: in function `test_simde_vrndn_f16':
rndn.c:(.text+0xc36): undefined reference to `roundevenf'
collect2: error: ld returned 1 exit status
ninja: subcommands failed

Yes, this sounds correct. emscripten uses musl and ran into this as well:
https://github.com/simd-everywhere/simde/blob/bd0532032487df1ea501ae8dc3f8e32abd5309b1/simde/simde-math.h#L1269
One could add a SIMDE_MUSL (or similar) to also skip the use of __builtin_roundeven for other musl users, unless there is a way for a header-only library to detect the use of musl at compile time

ah, maybe there is a way to detect musl!
https://stackoverflow.com/a/78062291/1585509


Project cards -> case study

link project cards to case study using remark
https://www.gatsbyjs.com/plugins/gatsby-transformer-remark/

lol stale


Error with "kiri-ai/t5-base-qa-summary-emotion" model

Hey,
If I understand correctly, then this library should work with all t5 models including finetuned ones right? When I try to generate onnx representation of model "kiri-ai/t5-base-qa-summary-emotion", I get the following error:
File "C:\Users\Oren\Anaconda3\envs\emotion-qa\lib\site-packages\transformers\models\t5\modeling_t5.py", line 499, in forward     scores += position_bias RuntimeError: output with shape [5, 12, 1, 2] doesn't match the broadcast shape [5, 12, 2, 2]

the issue has been addressed before here, I've posted a fix for it there.

Thanks - you might want to keep that issue open / pin it until the PR is merged so that people will be able to see it :)


cellRenderer (checkbox) in column does not re-render when using setState

I can use jquery to keep the state in the browser and that works just fine.  However, I was looking for a react way and it seems I ran into a re-render issue in the fixed-data-table.
Is my thinking wrong?
See the original question here:  http://stackoverflow.com/questions/29067247/check-all-boxes-in-a-fixed-data-table-by-using/29090399
'use strict';

var React = require('react/addons');
var Table = require('fixed-data-table').Table;
var Column = require('fixed-data-table').Column;

// Fixed data tables (Facebook) requirement:http://facebook.github.io/fixed-data-table/
require('../../styles/fixed-data-table.css');
var TableTester = React.createClass({
  getInitialState: function(){
    return ({
      selected:[],
      rows: [
        ['a1', 'b1', 'c1'],
        ['a2', 'b2', 'c2'],
        ['a3', 'b3', 'c3'],
      ]
    })
  },
  render: function () {
    var data = this.state.rows;

    return (

        Check All <input type="checkbox" onClick={this._checkall} />

          <Column label="Col 1" width={500} dataKey={0} />
          <Column label="Col 2" width={500} dataKey={1} cellRenderer={this._renderCheckbox} align="center" />

    );
  },

  _checkall:function(e){
    var newSet = [];
    if (e.target.checked) {
      this.state.rows.forEach(function(columns){
        newSet.push( columns[1] );
      });
    }
    this.setState({
      selected: newSet
    });

  },

  _onChange: function(e) {
    var {checked, value} = e.target;
    var newSet = this.state.selected;
    if ( checked && newSet.indexOf(value) < 0 ) { // does not exist in list
      newSet.push(value);
    } else {
      newSet.splice(newSet.indexOf(value), 1);
    }
    this.setState({
      selected: newSet
    });

  },

  _renderCheckbox: function(data){
    var checked = this.state.selected.indexOf(data) >= 0; // value is in list
    return <input type="checkbox" value={data} onChange={this._onChange} checked={checked} />;
  },

  _rowGetter: function(rowIndex){
      return this.state.rows[rowIndex];
  }

});

module.exports = TableTester;

Hey @dannymk, the fixed data table applies a very aggressive shouldComponentUpdate function, which stops the cellRenderer from being called unless some data associated with that renderer has changed. The way we figure our what data is needed to render a row is via the rowGetter, so the issue above is that the state you are expecting to update your cellRenderer is not returned from the rowGetter so the diff does know that something has changed.
Here is an example of how it could work by embedding the data within the row object:
http://jsbin.com/temufa/18/edit
The other way to provide this behavior is for the cellRenderer to just return a single react container which is responsible for subscribing to store data changes and updating the cell via setState. We do this for a lot of our applications but this only makes sense if for larger tables with a lot of different possible data types.
We are thinking about removing this aggressive shouldComponentUpdate functionality to make the table render more predictability but it will have a large performance cost if all cells rerender with each set update, so we are still trying to find the right balance here :)

Thank you for your answer.   It helps a lot.  I look forward to to future version of this great component.

@pieterv The container approach sounds pretty neat, it might work for us.
We are building a grid based on FixedDataTable that has edit mode  but initial performance is really bad on update in cells because it seem to re-render every cell for some reason, and it takes almost a second(
I've noticed some components in FixedDataTable are PureRender and some are not, including Table.
Do you have any examples out there for editable table? We are currently trying to subscribe every cell to store but it may be too many listeners too. Additionally it breaks the convention to subscribe to store change only in higher-level 'controller' components...
Any hint would be appreciated, thanks!

http://jsbin.com/temufa/18/edit

Doesn't work anymore.

Steve, I just removed the "/** @jsx React.DOM */" at the top of your edit and it worked just fine.  What issues are you having?

My edit? I didn't realise this was globally editable.
I landed here because I have the same or similar issue. I'm trying to implement spreadsheet-style editing in a FixedDataTable and came across this issue of the cell renderer not being invoked. I have read @pieterv's explanation and I'm working through a solution. What I'm trying to do is track the coordinates of the cell that is currently being edited and, when that changes, flip between a div and an input. The approach I'm going to take now is to create a non-singleton store to which the container components subscribe. The container components will be returned by the cell renderer.

Sorry, can't help you there but I am glad you got it working.  I am swamped here and behind in quite a few projects.

Yeah, thanks, I don't really need help now that I understand the container concept and how it can be used in my situation. Good luck with your projects. Sounds like you'll need it.

I don't see how this strategy applies when you are receiving the rows as props. Can you update your example to show how this might work?
Currently I am trying to get the toggling of a checkbox in a row to render, but can't figure it out. When I toggle, I can update state and see my component where the Table is receive nextProps, but the cell doesn't re-render properly. If I browse away and cause a re-render of the table though my checkboxes render their state accurately.
Here is some of the code
        <Column
          align='center'
          width={60}
          dataKey={"isSelected"}
          headerRenderer={this.selectAllCheckbox}
          cellRenderer={this.renderCheckbox}
        />

  renderCheckbox(cellData, cellDataKey, rowData, rowIndex) {
    return (
      <input type="checkbox"
        onChange={this.toggleItem.bind(this, rowData)}
        checked={cellData}/>
    )
  }

  toggleItem(data) {
    if (data.isSelected) {
      this.props.deselectApp(data.id)
    } else {
      this.props.selectApp(data.id)
    }
  }

At this point, I'd take a hack or a workaround, been banging my head against the wall on this one for most of the day...

Interestingly, when I set a breakpoint in the toggleItem function, I can see the checkbox stays checked until some point in the event dispatching lifecycle. Is there a way to override that?

I found a workaround (since I didn't want to use stores or something else for single table), I had to add possibility to edit some row cells. So on button click I just changed that row data into new object with same values:

  onEditClick(id, rowIndex) {

        //mock data change so row rerenders itself
        let filteredData = this.state.filteredData;
        filteredData[rowIndex] = {...filteredData[rowIndex]};

        this.setState({
            editingId: id,
            filteredData,
        });
    },


Invite Customization

Is it possible to change the time or the number of uses for the invites?

@kingpin24
I have made a pr regarding your comment. Hope it would be merged.


Printing inconsistent width characters and spacing

What version are you using?
21.4.1
What did you do?

What did you expect to see?
Consistent spacing around emojis.
What did you see instead?
Inconsistent spacing around emojis.
cc @fnando @elizabethengelman

I remember this when I added emojis to the snapshot create command, there were some emojis that were two code points wide and the terminals I tested in that affected the spacing against following characters. In that command I compensated by adding in an additional space after some emojis to have them align.
We need to check how this renders in a variety of terminals.
We may wish to limit our emojis to single codepoint emojis if we can't find common consistency.

Look for different icons

This is an issue that affects both the vscode terminal and the default macOS terminal (iterm works fine).


Implement a ConcurrentIdentityHashMap

Implementation of a ConcurrentIdentityHashMap to avoid using
Netty dependencies

<placeholder for description of changes to netty dependencies>
Closes #9449

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4796/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4803/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4822/
Test FAILed.

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4829/
Test PASSed.

A note to hold off on merging this as I still want to remove the org.jboss netty dependency from the POM files in this PR as well.

@ZacBlanco If you'll replace the dependency in this PR it would be good to have proper test validation prior to merge for verifying performance is not impacted.

@ZacBlanco the description has <placeholder for description of changes to netty dependencies>. Did you want to update that?

@gpang I still haven't made the changes necessary to update the description for that placeholder

is this pr ready to merge? @ZacBlanco

@apc999 no I still want to push one more commit

@bf8086 Any advise on how to put DataMessageMarshaller's buffer map under stress ?

@ggezer I removed the netty dependency in the core-common module POM in the most recent commit. I'll leave the rest of the work removing the dependency throughout the codebase to you in https://github.com/Alluxio/alluxio/pull/9455
Still looking into stress testing. I might also try a few synthetic tests with the new HashMap vs the old one just to get a relative idea about performance

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4899/
Test PASSed.

If we are good with the final change to the POM in this PR is can now be merged

Here is the code I used to run some synthetic benchmarks:

  @Test
  public void testConcurrentHashMap() throws Exception {
    List<Map<Long, Long>> maps = new ArrayList<>();
    maps.add(new ConcurrentIdentityHashMap<>());
    maps.add(new alluxio.collections.ConcurrentIdentityHashMap<>());
    long seed = new Random().nextLong();
    Integer[] i = { 10_000, 100_000, 500_000 };
    Integer[] p = { 4, 8, 16, 32 };
    List<Integer> itemTest = Arrays.asList(i);
    List<Integer> parallelismVals = Arrays.asList(p);

    int trials = 50;
    for (int numItems : itemTest) {
      for (int parallelism : parallelismVals) {
        System.out.println(String.format("Trials for %s map items with %s parallelism", numItems,
         parallelism));
        ExecutorService svc = Executors.newFixedThreadPool(parallelism);
        maps.forEach(map -> serialMapPerfTest(map, seed, trials, numItems));
        maps.forEach(map -> parallelMapPerfTest(map, seed, trials, numItems, parallelism, svc));
        maps.forEach(m -> m.clear());
        svc.shutdown();
        System.gc();
        // System.gc() is more like a suggestion to the JVM to gc so sleep this thread
        // a few seconds to let the JVM gc if it decides to
        Thread.sleep(2000);
      }
    }
  }

  public void serialMapPerfTest(Map<Long, Long> map, long seed, long trials, int numItems) {
    Random r = new Random(seed);
    List<Long> putTimes = new ArrayList<>();
    List<Long> getTimes = new ArrayList<>();

    for (int i = 0; i < trials; i++) {
      map.clear();
      long start = System.currentTimeMillis();
      for (int j = 0; j < numItems; j++) {
        Long l = r.nextLong();
        map.put(l, l);
      }
      long end = System.currentTimeMillis();
      putTimes.add(end-start);
      start = System.currentTimeMillis();
      for (Long k : map.values()) {
        map.get(k);
      }
      end = System.currentTimeMillis();
      getTimes.add(end-start);
    }
    Double putTime = ((double) putTimes.stream().reduce((l1, l2) -> l1+l2).get()) / trials;
    System.out.println(String.format("Serial mean put time for %s: %s",
        map.getClass().getCanonicalName(),
        putTime));

    Double getTime = ((double) getTimes.stream().reduce((l1, l2) -> l1+l2).get()) / trials;
    System.out.println(String.format("Serial mean get time for %s: %s",
        map.getClass().getCanonicalName(),
        getTime));
  }

  public void parallelMapPerfTest(Map<Long, Long> map, long seed,
      long trials, int numItems, int parallelism, ExecutorService service) {
    map.clear();
    Random r = new Random(seed);
    AtomicInteger counter = new AtomicInteger(0);

    List<Long> putTimes = new ArrayList<>();
    List<Long> getTimes = new ArrayList<>();

    // Run for X trials
    for (int x = 0; x < trials; x++) {
      map.clear();
      counter.set(0);
      // Fill map with items and time it
      long start = System.currentTimeMillis();
      runNThreads(() -> {
        while (counter.getAndIncrement() < numItems) {
          Long l = r.nextLong();
          map.put(l, l);
        }
      }, parallelism, service);
      long end = System.currentTimeMillis();
      putTimes.add(end-start);
      // Get the list of all keys in the map and shuffle them into a queue
      ArrayList<Long> l = new ArrayList<>(map.keySet().stream().collect(Collectors.toList()));
      Collections.shuffle(l);
      ConcurrentLinkedQueue<Long> q = new ConcurrentLinkedQueue<>(l);
      start = System.currentTimeMillis();
      // Time how long it takes flush the queue while doing a containsKey() across all items in the
      // map
      runNThreads(() -> {
        while (!q.isEmpty()) {
          map.containsKey(q.poll());
        }
      }, parallelism, service);
      end = System.currentTimeMillis();
      getTimes.add(end-start);
    }

    Double putTime = ((double) putTimes.stream().reduce((l1, l2) -> l1+l2).get()) / trials;
    System.out.println(String.format("Parallel mean put time for %s: %s",
        map.getClass().getCanonicalName(),
        putTime));

    Double getTime = ((double) getTimes.stream().reduce((l1, l2) -> l1+l2).get()) / trials;
    System.out.println(String.format("Parallel mean get time for %s: %s",
        map.getClass().getCanonicalName(),
        getTime));
  }

  /**
   * run N tasks across an executor service until all have finished successfully.
   */
  private void runNThreads(Runnable r, int parallelism, ExecutorService svc) {
    List<Future<?>> futures = new ArrayList<>();
    for (int i = 0; i < parallelism; i++) {
      futures.add(svc.submit(r));
    }
    waitForFutures(futures);
  }

  /**
   * Blocks until all futures in a list
   */
  private void waitForFutures(List<Future<?>> futures) {
    for (Future<?> f : futures) {
      try {
        f.get();
      } catch (Exception e) {
        throw new RuntimeException(String.format("Got exception while running: %s", e));
      }
    }
  }

Results for parallel benchmark, on 2015 MBP w/ an Intel Core i7 (4C/8T), 16GB DDR3  1660MHz RAM. Java build 1.8.0_191-b12
Graphs:

Table:

LGTM

alluxio-bot, merge this please.


AMQ-9192 - Fix flaky AdvisoryTests

Properly shutdown broker for each test and speed up tests by sending less messages

I created a PR here mostly to let the tests run to see if it fixes things and then it can be merged and I will back port it to 5.17.x as well (along with any tweaks)

There might still be something to tweak but it looks like it fixed the vast majority of the Advisory test issues so I'm going to merge this and let it run against main and then fix whatever else.


Release 1.2.1

Added

Added the --engine flags that allows to specify the container engine to use
between docker and podman. The default behavior is not changed, if the flag is
not specified fyne-cross will auto detect the engine.

Fixed

Windows builds no longer pass "-H windowsgui" #97
Multiple tags cannot be specified using the -tags flag #96

Let's release 1.2.1 since fixes some regressions :)


TS cannot find type (type is unknown) for extension of Map class

Suggestion
Love TS! Here is a bit where type inferences seems to be lacking.
Here is the raw code:
export class DESMap<K,V> extends Map<K,V> {

  constructor() {
    super();
  }

  toJSON(){
    return Array.from(Object.fromEntries(this)).map((v) => {
      return [v[0], v.value];
    });
  }

}

Here is the lack of inference:

Seems like just an improvement could be made here.

if it's not clear, the type of v is unknown

This code doesn't actually work:
const map = new DESMap<string, string>();
map.set("foo", "bar");
console.log(Object.fromEntries(map));  // prints { "foo": "bar" }
console.log(map.toJSON());  // prints []


FetchTessGSOut does not consider Stream output var decoration when changing XFB buffer

Description
When changing transform feedback decorations for GS output - Stream decoration is not taken into account when changing XFB buffer.
SPIRV spec says:

All Stream Decorations must be the same for all objects decorated with the same XfbBuffer XFB Buffer Number.

GS output before modification:
layout(location = 1, xfb_buffer = 1, xfb_stride = 16, xfb_offset = 0, stream = 1) out vec4 _33;
layout(location = 0, xfb_buffer = 0, xfb_stride = 16, xfb_offset = 0, stream = 0) out vec4 _38;

(different stream with different buffers)
After FetchTessGSOut:
layout(location = 1, xfb_buffer = 0, xfb_stride = 32, xfb_offset = 16, stream = 1) out vec4 _33;
layout(location = 0, xfb_buffer = 0, xfb_stride = 32, xfb_offset = 0, stream = 0) out vec4 _38;

(different stream with the SAME buffer)
Steps to reproduce

Capture dEQP-VK.transform_feedback.simple.multiquery_omit_write_1 test
Click on a draw call
Driver may crash (at least debug builds of RADV and Turnip)

Environment

RenderDoc version: 9254eff4904f31bbe529d5803234d25780df02e8
Operating System: Linux
Graphics API: Vulkan
Driver: debug builds of RADV and Turnip

Should be fixed with that commit.


[WIP] Refactor Github Actions workflow to re-use a workflow

What
We have a handful of groups of tests that we want to run against the
repository, and they all share the setup steps for the environment. In the
past, Github Actions didn't support any kind of task re-use, so we made do with
a matrix to run the groups of tests as different configurations.
GHA now supports task re-use, so we should refactor the workflow to properly
represent each set of tests as a different job with a shared task.
How to review
Describe the steps required to test the changes.

🚨⚠ Please do not merge this pull request via the GitHub UI ⚠🚨

I don't think there's enough value in this to chase it up, given how hard it is to test.


User Story: System CPU Usage Monitoring

As a system administrator,

I want to monitor the CPU usage of the system,
So that I can identify periods of high usage and optimize performance.

Ticket:

#3
#4
#5

Developer: Chia Hans.
I have assigned myself to feature user story: system CPU Usage Monitoring!
Research and document methods to retrieve CPU usage data using python
Implement a python script to monitor and record CPU usage
Write unit tests for the CPU monitoring script @stephane-segning


add download component and activate route + add drag and drop js module

Downlaod was added using ng generate component download/download --flat

Download for now use node js server to upload file to the server. (xls, csv)


Initial code move

If your build fails looking for the old shakenfist_go module, try cleaning your go/pkg directory...

LGTM.


Proivde .tar.gz linux releases

I would like to make an Archlinux package for this application! Can you create a tar.gz package? or .zip ? you can do it easily using electron-builder; but not sure what are you using right now :)
Thanks!

@bil-elmoussaoui Thanks for the input! The build system uses lectron-builder behind the scenes.
Does Arch require pacman or tar.gz?

just provide .tar.gz would be more than enough

@bil-elmoussaoui I'd prefer pacman, as using tar.gz build would require adding the platform type to all packages (because it's a platform-agnostic format). Just added a .pacman build to 0a47efdbd5cae7c9a260b1c6c5b09bc1c082c521 - can you conduct a test installation?

Well i couldn't install it. Normally the pacman build can be installed using sudo pacman -U myfile.pacman
Doing so shows this here
warning: cannot resolve "gconf-service", a dependency of "pb-for-desktop"
warning: cannot resolve "gconf2", a dependency of "pb-for-desktop"
warning: cannot resolve "libappindicator1", a dependency of "pb-for-desktop"
warning: cannot resolve "libnotify4", a dependency of "pb-for-desktop"
warning: cannot resolve "libnss3", a dependency of "pb-for-desktop"
warning: cannot resolve "libxtst6", a dependency of "pb-for-desktop"

The package names are different from Ubuntu which causes this issue

That was fast :-) Please try that again, those package dependency constraints were just removed for pacman.

It works just fine ;) I will create an AUR package so i can include all the dependencies within it ;)
Anyway, i've noticed that the notifications sent from your application does not use any icon (but uses the information dialog by default) which is quite disturbing as i didn't knew that it was sent from PB at all :(

Here you go ! the package is done https://aur.archlinux.org/packages/pb-for-desktop/
Users can install it using yaourt -S pb-for-desktop
Btw, i've checked the desktop file and you seem to point to /opt/Pb for Desktop/pb-for-desktop I think the best way to do this, is to create a symlink from the /opt/Pb for Desktop/pb-for-desktop to /usr/bin/ and then use just pb-for-desktop In the Exec(in the desktop file)

@bil-elmoussaoui Thank you for your contribution, that's awesome. We'll test the symlinking soon-ish, need an Arch VM setup for that. Cheers.


HSM: add firmware to Base image

fixes https://github.com/shiftdevices/bitbox-base-internal/issues/369
Currently, we update the HSM firmware together with the Base image. For the Middleware to update the HSM, the specific version of the HSM firmware needs to be included in the disk image.
This issue extends the build script to provide a specific release version in the disk image. The version is specified like HSM_VERSION='4.3.0', that is then used to create the valid download URI like this:
https://github.com/digitalbitbox/bitbox02-firmware/releases/download/firmware-bitboxbase%2Fv4.3.0/firmware-bitboxbase.v4.3.0.signed.bin

It is important, that the tag of the release always follows this syntax: firmware-bitboxbase/v4.3.0
and the file is named like firmware-bitboxbase.v4.3.0.signed.bin.
The release binary file is then stored in the disk image:
/opt/shift/hsm/firmware-bitboxbase.signed.bin

The version is deliberately omited in the filename, as the Middleware reads the internal version number (of the monotonic counter) directly out of the file, and compares it with the currently installed firmware on the HSM.
The semantic version is stored in Redis as hsm:firmware:version, but not actively used at the moment.
This commit:

extends the build script
adds the new Redis key hsm:firmware:version

Otherwise it looks good to me, including the constant local filename (good thinking!).


Add typing annotations (PEP 484)

For those of us using MyPy in our projects it'd be really helpful to add typing annotations. Now that we only support Python 3.6+ we can do this directly in the code instead of having to use comments. I'd be willing to take this on if owners are interested.
If we don't want to do this in the codebase here I could do it on typeshed but that's less than ideal.

Although I've never made much use of annotations myself (as you can see!), I would welcome their addition, so long as they don't overly complicate the code too much. To apply them in s3fs, you would likely need to work on fsspec too.

To apply them in s3fs, you would likely need to work on fsspec too.

I can do that too if you guys are on board

If no one objects within a reasonable time, I'd say go for it.


Fix issue where the registry lookup was a hardcoded name and didn't a…

…ccount for the connector path
The previous commit to ManagementContext to prevent rebinds on the RMI registry used a hardcoded lookup path, as opposed to considering the connectorPath attribute. This commit should fix that.

@jgallimore do you mind to squash the two commits in a single one ? Thanks !

Sure, see #471 .


KOGITO-3554 Bump KIE7 to 7.44.0.Final

Upgrades Drools used to latest available 7.44.0.Final.
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Pull Request title is properly formatted: KOGITO-XYZ Subject
[x] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains link to any dependent or related Pull Request
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket

A rebuild did actually succeed:
https://rhba-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/KIE/job/kogito/job/ghprb-webhooks/job/kogito-runtimes/1518/


Trouble to get Patrol working with Android

Steps to reproduce

Installing Patrol like described on the website
running the example_test.dart on Android Pixel 7 Pro API 35 emulator with the following command:
patrol test -t integration/example_test.dart -d emulator-5554 --verbose

Actual results
I got the following response from patrol process:
✗ Failed to execute tests of apk with entrypoint test_bundle.dart on emulator-5554 (Gradle test execution failed with code 1) (8.7s)
Exception: Gradle test execution failed with code 1

I thought the reason might be the Java version, so I installed openjdk@17 through homebrew. Afterwards I used the following command:
flutter config --jdk-dir<EMAIL_ADDRESS>to ensure that this version is used. But it made no difference. Are there any additional tweaks or entries I should look at?
Thanks for your help. On iOS it works fine, and it is really awesome. Thanks for that tool! Hope to get it running on Android as well.
Kind regards,
Jonas
Logs

Logs
logs.txt

Patrol version
patrol: ^3.13.0
patrol_cli v3.4.0
Patrol Doctor output

Patrol Doctor output
patrol doctor --verbose
Verbose mode enabled. More logs will be printed.
Patrol doctor:
Patrol CLI version: 3.4.0
Flutter command: flutter
  Flutter 3.24.4 • channel stable
Android:
• Program adb found in /Users/jonasrosskamp/Library/Android/sdk/platform-tools/adb
• Env var $ANDROID_HOME set to /Users/jonasrosskamp/Library/Android/sdk
iOS / macOS:
• Program xcodebuild found in /usr/bin/xcodebuild
• Program ideviceinstaller found in /opt/homebrew/bin/ideviceinstaller

Flutter Doctor output

Flutter Doctor output
flutter doctor --verbose
[✓] Flutter (Channel stable, 3.24.4, on macOS 15.1.1 24B91 darwin-arm64, locale en-DE)
    • Flutter version 3.24.4 on channel stable at /Users/jonasrosskamp/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 603104015d (5 weeks ago), 2024-10-24 08:01:25 -0700
    • Engine revision db49896cf2
    • Dart version 3.5.4
    • DevTools version 2.37.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/jonasrosskamp/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/jonasrosskamp/Library/Android/sdk
    • Java binary at<EMAIL_ADDRESS>    • Java version OpenJDK Runtime Environment Homebrew (build 17.0.13+0)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.16.2

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

[✓] Android Studio (version 2024.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 21.0.3+-79915917-b509.11)

[✓] IntelliJ IDEA Community Edition (version 2024.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 82.2.4
    • Dart plugin version 243.22562.3

[✓] Connected device (6 available)
    • sdk gphone64 arm64 (mobile)     • emulator-5554                        • android-arm64  • Android 15 (API 35) (emulator)
    • iPhone von Jonas (mobile)       • XXXX           • ios            • iOS 18.1.1 22B91
    • iPhone 15 Pro (mobile)          • XXXX • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-5 (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 15.1.1 24B91 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 15.1.1 24B91 darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 131.0.6778.86

[✓] Network resources
    • All expected network resources are available.

• No issues found!

I just read a comment in an other issue that it could run with API 34 and it does! But why does it not run with API 35?

hi @Friesenjung
Thank you for report. There is a problem with Patrol and API 35. We are currently investigating it.