[Bug] A lot of irrelevant spans get together in one trace record

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
Python Agent (apache/skywalking-python)
What happened
I use the apache-skywalking 8.7.0  with the es storage and skywalking python agent 0.7.0
When I use it , There is a problem that some irrelevant spans  get together in one trace record, and the number of spans will be thousands.

What you expected to happen
Just don't know
How to reproduce
I use the apache-skywalking 8.7.0  with the es storage and skywalking python agent 0.7.0 . It's ok when test it. But when I use it in production environment, the wrong happens
Anything else
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

As you are also using Lua agent, can you confirm that it's not Lua agent's problem? For example, did that bug also happen when you invoke a Java API thorough the same APISIX?

From reading the graph, the parent span is a flask plugin for large numbers of children. It may relate to a multiple threads issue?

As you are also using Lua agent, can you confirm that it's not Lua agent's problem? For example, did that bug also happen when you invoke a Java API thorough the same APISIX?

When I use only python-agent, it also happens. So it's not lua agent's problem.

@zqm233 You should learn, typically, it would be very hard to diagnose thread-relative issue. It is better you could debug and find out the reason and scenario.

As you are also using Lua agent, can you confirm that it's not Lua agent's problem? For example, did that bug also happen when you invoke a Java API thorough the same APISIX?

When I use only python-agent, it also happens. So it's not lua agent's problem.

Do you have a reproducible demo project that can help to diagnose the bug?

@zqm233 You should learn, typically, it would be very hard to diagnose thread-relative issue. It is better you could debug and find out the reason and scenario.

tks. I'll try it.

As you are also using Lua agent, can you confirm that it's not Lua agent's problem? For example, did that bug also happen when you invoke a Java API thorough the same APISIX?

When I use only python-agent, it also happens. So it's not lua agent's problem.

Do you have a reproducible demo project that can help to diagnose the bug?

The bug only shows in production environment even the same project. I guess it related to high concurrency or something. And I use uwsgi and http config to run my flask app.  I don't know if it has something to do with it

As you are also using Lua agent, can you confirm that it's not Lua agent's problem? For example, did that bug also happen when you invoke a Java API thorough the same APISIX?

When I use only python-agent, it also happens. So it's not lua agent's problem.

Do you have a reproducible demo project that can help to diagnose the bug?

The bug only shows in production environment even the same project. I guess it related to high concurrency or something. And I use uwsgi and http config to run my flask app. I don't know if it has something to do with it
@zqm233
Hello, did you have any extra findings? It will be helpful if you can provide some more details, such as general specifications of your deployment, platform, versions and method of integration (i.e. non-intrusive or via import).

No update in months.


Rename Ballerina network logs to have word "HTTP"

Description:
At the moment, it looks like below.

Better to rename it to say "Ballerina HTTP trace logs" because only works for HTTP.

fixed with https://github.com/ballerina-platform/ballerina-lang/pull/18268


Spanner: BatchCreateSessions

The Spanner client library should use the new BatchCreateSessions API to create and maintain the session pool. In addition to using the new API, the following changes will be applied to the session pool to improve performance:

Change the order of the session pool from FIFO to LIFO. LIFO has been determined to show better and more stable query performance.
Use BatchCreateSessions to create and maintain the session pool.
Change the default session pool options to MinSessions > 0.


feat: update to latest flagd core release

This PR
Update flagd Go provider with the latest flagd core [1]
[1] - https://github.com/open-feature/flagd/releases/tag/core%2Fv0.9.0

Depends on - https://github.com/open-feature/flagd-testbed/pull/118


Add return_index_map argument in ssim()

Closes #53115

Here are the internal errors, @CohenAriel can you please verify ?  Thank you!
Traceback (most recent call last):
File "/tensorflow/python/ops/image_ops_test.py", line 5801, in testWithIndexMap
self.assertAllEqual(ssim_global, math_ops.reduce_mean(ssim_locals, axes))
File "/tensorflow/python/framework/test_util.py", line 1571, in decorated
return f(*args, **kwds)
File "/tensorflow/python/framework/test_util.py", line 3292, in assertAllEqual
np.testing.assert_array_equal(a, b, err_msg="\n".join(msgs))
File "/py/numpy/testing/_private/utils.py", line 934, in assert_array_equal
assert_array_compare(operator.eq, x, y, err_msg=err_msg,
File "/py/numpy/testing/_private/utils.py", line 844, in assert_array_compare
raise AssertionError(msg)
AssertionError:
Arrays are not equal
not equal where = (array([0]),)
not equal lhs = array([0.06525472], dtype=float32)
not equal rhs = array([0.06525473], dtype=float32)
Mismatched elements: 1 / 1 (100%)
Max absolute difference: 7.450581e-09
Max relative difference: 1.1417688e-07
x: array([0.065255], dtype=float32)
y: array([0.065255], dtype=float32)

Weird. Am I running the wrong command?
bazel test -k tensorflow/python:image_ops_test --test_filter=*SSIMTest.testWithIndexMap*

resulted in
INFO: Options provided by the client:
  Inherited 'common' options: --isatty=1 --terminal_columns=191
INFO: Reading rc options for 'test' from /home/ariel/Projects/tensorflow/.bazelrc:
  Inherited 'common' options: --experimental_repo_remote_exec
INFO: Reading rc options for 'test' from /home/ariel/Projects/tensorflow/.bazelrc:
  Inherited 'build' options: --define framework_shared_object=true --define=use_fast_cpp_protos=true --define=allow_oversize_protos=true --spawn_strategy=standalone -c opt --announce_rc --define=grpc_no_ares=true --noincompatible_remove_legacy_whole_archive --enable_platform_specific_config --define=with_xla_support=true --config=short_logs --config=v2 --define=no_aws_support=true --define=no_hdfs_support=true --experimental_cc_shared_library --experimental_link_static_libraries_once=false --deleted_packages=tensorflow/compiler/mlir/tfrt,tensorflow/compiler/mlir/tfrt/benchmarks,tensorflow/compiler/mlir/tfrt/jit/python_binding,tensorflow/compiler/mlir/tfrt/jit/transforms,tensorflow/compiler/mlir/tfrt/python_tests,tensorflow/compiler/mlir/tfrt/tests,tensorflow/compiler/mlir/tfrt/tests/ir,tensorflow/compiler/mlir/tfrt/tests/analysis,tensorflow/compiler/mlir/tfrt/tests/jit,tensorflow/compiler/mlir/tfrt/tests/lhlo_to_tfrt,tensorflow/compiler/mlir/tfrt/tests/lhlo_to_jitrt,tensorflow/compiler/mlir/tfrt/tests/tf_to_corert,tensorflow/compiler/mlir/tfrt/tests/tf_to_tfrt_data,tensorflow/compiler/mlir/tfrt/tests/saved_model,tensorflow/compiler/mlir/tfrt/transforms/lhlo_gpu_to_tfrt_gpu,tensorflow/core/runtime_fallback,tensorflow/core/runtime_fallback/conversion,tensorflow/core/runtime_fallback/kernel,tensorflow/core/runtime_fallback/opdefs,tensorflow/core/runtime_fallback/runtime,tensorflow/core/runtime_fallback/util,tensorflow/core/tfrt/common,tensorflow/core/tfrt/eager,tensorflow/core/tfrt/eager/backends/cpu,tensorflow/core/tfrt/eager/backends/gpu,tensorflow/core/tfrt/eager/core_runtime,tensorflow/core/tfrt/eager/cpp_tests/core_runtime,tensorflow/core/tfrt/gpu,tensorflow/core/tfrt/run_handler_thread_pool,tensorflow/core/tfrt/runtime,tensorflow/core/tfrt/saved_model,tensorflow/core/tfrt/graph_executor,tensorflow/core/tfrt/saved_model/tests,tensorflow/core/tfrt/tpu,tensorflow/core/tfrt/utils
INFO: Found applicable config definition build:short_logs in file /home/ariel/Projects/tensorflow/.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:v2 in file /home/ariel/Projects/tensorflow/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Found applicable config definition build:linux in file /home/ariel/Projects/tensorflow/.bazelrc: --copt=-w --host_copt=-w --define=PREFIX=/usr --define=LIBDIR=$(PREFIX)/lib --define=INCLUDEDIR=$(PREFIX)/include --define=PROTOBUF_INCLUDE_PATH=$(PREFIX)/include --cxxopt=-std=c++17 --host_cxxopt=-std=c++17 --config=dynamic_kernels --distinct_host_configuration=false --experimental_guard_against_concurrent_changes
INFO: Found applicable config definition build:dynamic_kernels in file /home/ariel/Projects/tensorflow/.bazelrc: --define=dynamic_loaded_kernels=true --copt=-DAUTOLOAD_DYNAMIC_KERNELS
INFO: Analyzed 2 targets (0 packages loaded, 0 targets configured).
INFO: Found 2 test targets...
INFO: Elapsed time: 8.992s, Critical Path: 1.66s
INFO: 33 processes: 1 internal, 32 local.
INFO: Build completed successfully, 33 total actions
//tensorflow/python:image_ops_test_cpu                                   PASSED in 1.7s
  Stats over 16 runs: max = 1.7s, min = 1.2s, avg = 1.5s, dev = 0.1s
//tensorflow/python:image_ops_test_gpu                                   PASSED in 1.7s
  Stats over 16 runs: max = 1.7s, min = 1.3s, avg = 1.5s, dev = 0.1s

Executed 2 out of 2 tests: 2 tests pass.
INFO: Build completed successfully, 33 total actions

Hi @SeeForTwo Can you please assist on above comments from @CohenAriel. Thank you!

When @gbaned ran the test, there appeared to be a trivial floating point rounding difference:

not equal lhs = array([0.06525472], dtype=float32)
not equal rhs = array([0.06525473], dtype=float32)
sMax absolute difference: 7.450581e-09
Max relative difference: 1.1417688e-07

If you agree that this is just a rounding issue (how rounding occurs on different computers), I would suggest changing the test to use self.assertAllClose(ssim_global, math_ops.reduce_mean(ssim_locals, axes)) instead of self.assertAllEqual(ssim_global, math_ops.reduce_mean(ssim_locals, axes)). The default tolerances for "close" are 1e-6. (In general, I favor using assertAllClose over assertAllEqual when comparing floats.)

Makes sense. I'll try that.

@tensorflow/api-owners

@CohenAriel, because adding this optional argument changes the public API, "the goldens" need to be updated.
Can you please try running:
bazel run tensorflow/tools/api/tests:api_compatibility_test  -- --update_goldens True

See "Py+XPP Test Suite - Ubuntu CPU, Python 3.9.." Details --> Details and search for "If this test fails, it means a change has been made to the public API" in the log.

After running that command, commit the changes that it (hopefully) creates.

For feedback/copybara — Google internal checks FAILED (i.e. since this is a public API change) could you please add a short summary of this change to RELEASE.md. My suggestion is in # Bug Fixes and Other Changes to add something like the following:
*   `tf.image`
    *   Added an optional parameter `return_index_map` to `tf.image.ssim` which
        causes the returned value to be the local SSIM map instead of the
        global mean.

Thank you for your patience!

No problem. I hope it's fine now.


wsk activation poll needs more robust name matching

@cfjedimaster commented on Mon May 22 2017
I filed a bug report on "wsk activation poll -h" not making it clear that it can match by name as well, but I want to expand on the feature in general. It feels like the name matching isn't very robust. It's also possible I'm not doing this right, but this is where better docs would help.
I've got an action called makeresult that lives in the serverless_superman package. The only way I can see to poll for it is to poll JUST for 'makeresult'. If I try /serverless_superman/makeresult, it will not match. If I've got 10 packages with makeresult in it, then I'll get a lot of noise back.
If I need to also provide a namespace, than I'd consider that a bug as well. It would work, but the CLI should just default to my namespace anyway. (And again, it should be documented.)
Also, there seems to be no way to get everything called within one package. (Again, unless I'm running it wrong.)

@rabbah commented on Mon May 22 2017
confirmed bug - you can name an action in the default package but not an action in a named package.
it may be possible to support polling on all actions in a package as part of the same fix.

@dubeejw commented on Thu Jun 22 2017
I had a PR open to update the activation poll usage, but was informed that activation IDs may not apply to rules and triggers in the future. I can update the usage to just mention actions though. See: https://github.com/apache/incubator-openwhisk/pull/2398.

@dubeejw commented on Thu Jun 22 2017
Linked the PR to the proper issue.

@dubeejw @mdeuser I think recent changes in views allows to filter by package, can you double check if this is correct and whats needs to be done (update cli help?)


nightly is now Python 3.7 and fails

Using tox-travis with nightly now shows
$ pip install tox-travis
$ tox
ERROR: unknown environment 'py37'

fwiw, using python: 3.6 breaks in travis.  I am pretty sure this has been raised in the public travis issues repo.
Using worker: worker-linux-docker-91207869.prod.travis-ci.org:travis-linux-3

...
3.6 is not installed; attempting download
bzip2: (stdin) is not a bzip2 file.
tar: Child returned status 2
tar: Error is not recoverable: exiting now

The command "sudo tar xjf python-3.6.tar.bz2 --directory /" failed and exited with 2 during .

Your build has been stopped.

I think the py37 thing is because that's not yet known to tox. That's a bug in tox, in my opinion, and I'm sure it'll be fixed in short order.
I'm not sure of the other is a bug, otay need to be declared as the beta version, since the stable version isn't out yet.

Correct on all counts ;-)
https://github.com/tox-dev/tox/pull/382


The Alt modifier to change the selection mode clashes with move window on Ubuntu

Hello,
I run into a bit of a problem, and I can't figure out how to remap the key correctly as the documentation is very unhelpful as it doesn't give a list of 'command ids', and I can't find one online. Anyway, I'm using Ubuntu 15.04 and when I hold down the Alt key to highlight in the 'box mode' (like the windows command line does), it drags the window instead. I want to switch it to AltGr instead, but I can't find the appropriate entry for my keymap.json.
Could someone help me please?
Thanks,
Starbeamrainbowlabs

You can use brackets-display-shortcuts to see all the shortcuts (extensions included). The built in ones can be found from the Commands.js file as stated in wiki/User-Key-Bindings.

@petetnt Thanks for the help! I can't seem to find the option in Commands.js, even though it's a built in feature. I'm probably just missing it though.

It might be a https://github.com/codemirror/codemirror keybinding too, might be worth checking into. I couldn't find the command with a quick look myself either.

Workaround for others with this issue: http://askubuntu.com/a/118179/139735


0.4.0 daemon, unable to create server & starting servers stuck in 'starting'

This is from an upgrade from 0.5 to 0.6, the panel is working fine. But having issues with the daemon.
I am unable to create servers, daemon wont create the docker containers. With the starting issue, the servers will start up, but will stay in the starting phase. The servers boot and you can join them. But the panel/daemon reports them still starting. Only using/tested with vanilla Minecraft servers.
Daemon logs did pop up an error:
TypeError: Cannot read property 'done' of undefined
    at Async.parallel (/srv/daemon/src/services/index.js:177:57)
    at /srv/daemon/node_modules/async/dist/async.js:3830:24
    at eachOfArrayLike (/srv/daemon/node_modules/async/dist/async.js:1003:9)
    at eachOf (/srv/daemon/node_modules/async/dist/async.js:1051:5)
    at _parallel (/srv/daemon/node_modules/async/dist/async.js:3829:5)
    at Object.parallelLimit [as parallel] (/srv/daemon/node_modules/async/dist/async.js:3912:5)
    at Service.onConsole (/srv/daemon/src/services/index.js:147:15)
    at Service.onConsole (/srv/daemon/src/services/minecraft/index.js:34:22)
    at Server.output (/srv/daemon/src/controllers/server.js:347:22)
    at Carrier.carry.data (/srv/daemon/src/controllers/docker.js:225:29)

uname -a:
Linux ns543214.ip-144-217-11.net 4.4.0-77-generic #98-Ubuntu SMP Wed Apr 26 08:34:02 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

php -v:
PHP 7.1.5-1+deb.sury.org~xenial+1 (cli) (built: May 11 2017 14:07:52) ( NTS )
Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.1.0, Copyright (c) 1998-2017 Zend Technologies
    with Zend OPcache v7.1.5-1+deb.sury.org~xenial+1, Copyright (c) 1999-2017, by Zend Technologies

docker info:
Containers: 6
 Running: 2
 Paused: 0
 Stopped: 4
Images: 6
Server Version: 17.05.0-ce
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 65
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 9048e5e50717ea4497b757314bad98ea3763c145
runc version: 9c2d8d184e5da67c95d601382adf14862e4f2228
init version: 949e6fa
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.4.0-77-generic
Operating System: Ubuntu 16.04.2 LTS
OSType: linux
Architecture: x86_64
CPUs: 8
Total Memory: 31.07GiB
Name: ns543214.ip-144-217-11.net
ID: 35B2:P2H2:HART:XPGS:VHF3:RVPN:ALKN:ARJH:MBKE:NEOA:GBAK:X4SQ
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

This appears to be a duplicate of #443, however it can read startup, but not the done property.
Is this a custom service?


RDI-55-whoas-transform

Helpful background context
As I noted in the mapping spreadsheet, the dc.description.sponsorship value appears to always be a descriptive statement and the Funder.funder_name attribute is the best available option but not a great fit. Not sure if this justifies creating a description attribute for the Funder class but just noting it here as well.
How can a reviewer manually see the effects of these changes?
Pending discussion of how we want to do this
What are the relevant tickets?

https://mitlibraries.atlassian.net/browse/RDI-55

Developer

[x] All new ENV is documented in README
[x] Stakeholder approval has been confirmed (or is not needed)

Code Reviewer

[ ] The commit message is clear and follows our guidelines
(not just this pull request message)
[ ] There are appropriate tests covering any new functionality
[ ] The documentation has been updated or is unnecessary
[ ] The changes have been verified
[ ] New dependencies are appropriate or there were no changes

Includes new or updated dependencies?
NO

I added code and tests to account for blanks fields, it's not anything I've encountered with this collection so far but you're right that it's good to be prepared for that


Support custom metadata objects.

Custom metadata as described in the TUF spec can be specified when adding a new bundle to a repository, for example:
...
repo.add_bundle(
    new_version='2.3.4',
    new_bundle_dir='dist/myapp',
    custom_metadata_for_archive=None,  # optional
    custom_metadata_for_patch=dict(foo='bar', ...),  # optional
)
...

The custom metadata ends up in the targets.json file as follows:
{
  "signatures": [
    "..."
  ],
  "signed": {
    "...": "...",
    "targets": {
      "...": {},
      "myapp-2.3.4.patch": {
        "custom": {
          "foo": "bar"
        },
        "hashes": {
          "sha256": "..."
        },
        "length": 18709
      },
      "...": {}
    }
  }
}

On the client side, this metadata is made available via the TargetMeta class. For example,
...
new_target_meta = client.check_for_updates()
if new_target_meta:
    ...
    if new_target_meta.custom:
        ...

fixes #99

Note the CLI does not support custom metadata yet.
We could introduce an extra CLI option that allows the user to specify a summary message, for example.
Although possible, I think anything more complex, like writing json on the command line, would become cumbersome and error prone.


Md-dialog config backdropClass has either a bug or awkward functionality which removes .cdk-overlay-dark-backdrop class and access is only on sibling div

Bug, feature request, or proposal:
Bug? proposal?
What is the expected behavior?
When using the configuration backdropClass and setting a custom class the original overlay css should remain.  I believe the implication here is that a person would want their own backdrop class to color the overlay.
Also, one consequence of this is it is nice to have a class that is custom per the dialog box however, one can only reach the sibling div by adding a + or ~ operator to the css class.
What is the current behavior?
this class is removed
.cdk-overlay-dark-backdrop {
background: #000;
}
What are the steps to reproduce?
any plnkr will show this
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: https://goo.gl/DlHd6U
What is the use-case or motivation for changing an existing behavior?
Adding a class should be the requisite for removing a class... I mean a simple override would have the same affect.  The ability to add a class allows for full customization per dialog box which is better functionality.  If the purpose is to remove the class and then add ones own class that should be a 2 step configuration process or at least let the api give that particular twofer option
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
ang 4.2.0.beta1 Material 2.0.0-beta.5
Is there anything else we should know?
nope.

I'm confused about what exact issue you are having.
You said "I believe the implication here is that a person would want their own backdrop class to color the overlay." As you saw, the cdk-overlay-dark-backdrop class has one property, which is to set the background color. Another default class also exists cdk-overlay-transparent-backdrop which is used for menus and other things. These are set using the OverlayState.backdropClass property. So when you supply your own custom backdropClass you are replacing the default one that is applied to the dialog backdrop. This class, by default, is only used to apply a background color. The class cdk-overlay-backdrop will still be applied to the same element and still includes all of the necessary positioning and sizing properties for the backdrop. By supplying your own class, you can create different dialogs with different backdrops, maybe some transparent or some funky colors, or even do something completely unrelated to just background color.

@YeomansIII What you say create different dialogs (I get the backdrop point) that would imply there is a class on a parent div affecting the dialog modal differently from modal to the next.  This is not the case... The class gets added to the overlay field leaving only the sibling div which house the modal being accessed only by a + (sibling) or ~ css class.

I think what you want is a separate feature from backdropClass. It seems like you are trying to apply custom CSS to the overlay-pane, correct? And you want to use this backdropClass field to do that, which isn't possible (without the sibling selectors you mentioned) and was not the intended purpose of this feature. This backdrop class is unique per dialog, as overlays overlap each other.
Am I correct in understanding that you are trying to apply CSS to the dialog's overlay-pane or the md-dialog-container itself?

I'm curious about what your use-case is, what are you trying to style outside of the domain of the actual component itself?
Also, the class cdk-overlay-backdrop is always applied to the backdrop, regardless of what class you apply with the backdropClass property. Couldn't you target this class instead of cdk-overlay-dark-backdrop?

@YeomansIII the use case is that we have different modals for different purposes.  For example sometimes we take snippets of our web pages and use them inside of modals to quick access to information.
Other modals might be totally different which are simple error messages or informational type modals.  The use cases are many

@YeomansIII any update on this?

@xtianus79 Still not sure what the issue is. What you described are differences that can take place in the dialog component's HTML and CSS. Why do you need the backdrop class to style the inside of a component? The backdrop class should only be used to apply a background color to the overlay, just as cdk-overlay-dark-backdrop is doing.

@xtianus79 not totally certain what functionality you're looking for, but does https://github.com/angular/material2/pull/4722 help? It allows you to pass additional classes the cdk-overlay-pane. You can then use that to target you md-dialog-container since it is a child.
Combined with @YeomansIII's point about the cdk-overlay-dark-backdrop only affecting the single background style of the cdk-overlay-backdrop, you should be able to manipulate everything without the use of sibling selectors.

@willshowell and @YeomansIII

you should be able to manipulate everything without the use of sibling selectors.

Perfect!!! When will this be released?  In beta.7?

@xtianus79 yeah it will be in the next release. Alternatively you can use now via material2-builds. And to answer the question you made on the PR itself, it will not show up in the API until a release is made.

@willshowell cool... thank you sir

Closing this since I think it's resolved by the aforementioned PR


Add the suite starter to an ETOS cluster

Applicable Issues

fixes: #288
Description of the Change

This will enable us to run both versions of ETOS using the same ETOS cluster.
Alternate Designs

None really.
Possible Drawbacks

Lots of code that needs to be removed later, but the ability to use the same cluster to run both versions of ETOS is a significant improvement.
Sign-off

Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
Signed-off-by: Tobias Persson<EMAIL_ADDRESS>

The routing key tag change is problematic when running in the controller environment. Must fix that before merge

Regarding this:

This will enable us to run both versions of ETOS using the same ETOS cluster.

Please describe in a README (or similar) how this separation is designed to work. Not very clear just by reading the code. It would help understand potential problems, some of which we already seen (e. g. routing tagging).

Regarding this:

This will enable us to run both versions of ETOS using the same ETOS cluster.

Please describe in a README (or similar) how this separation is designed to work. Not very clear just by reading the code. It would help understand potential problems, some of which we already seen (e. g. routing tagging).

There is no real separation in this code, we just make sure to deploy the suite starter so that the ETOS API can send a TERCC that's being picked up and not ignored.


az webapp --help

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp
Errors:
No module named 'decorator'
Traceback (most recent call last):
python3/dist-packages/fabric/connection.py, ln 5, in <module>
    from invoke.vendor.six import StringIO
ModuleNotFoundError: No module named 'invoke.vendor.six'

...

python3/dist-packages/fabric/connection.py, ln 10, in <module>
    from decorator import decorator
ModuleNotFoundError: No module named 'decorator'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp --help

Expected Behavior
Environment Summary
Linux-4.4.0-19041-Microsoft-x86_64-with-glibc2.29
Python 3.8.10
Shell: bash

azure-cli 2.0.81

Extensions:
azure-devops 0.17.0

Additional Context

route to CXP team to help with

@notywala433 Looks like you are using a pretty old version of az. Could you try upgrading to a newer version and try again?

@notywala433 please upgrade your CLI version & re-try. The error indicates something missing on the cli install package.


Coverage check fails with "Cannot read property 'find' of undefined"

The coverage check for #485 failed with:
Run cargo-tarpaulin
Run<EMAIL_ADDRESS>  with:
    version: 0.12.3
    timeout: 300
    out-type: Xml
##[error]Cannot read property 'find' of undefined

A few hours ago, the coverage checks worked for my previous PR #478.

Tarpaulin has occasionally been flakey, a subsequent coverage run succeeded so I think it's independent of your changes.

I'm hesitant to ignore flakeyness in our automation, even from the coverage testing stuff. I don't think we should necessarily prioritize this but I don't think we should close this bug on the basis that it passed in a subsequent run.

When I've worked with unreliable CI, it's been helpful to keep a tracking issue open, so we can see how often it fails. That helps us prioritise further investigation.
(We could also leave this issue closed until we see the next failure. Not sure if GitHub is good at recommending similar closed tickets though.)

I've seen repos where github recommends similar closed/open issues but it doesn't happen all the time, I wonder if thats a setting...

Let's see if we can reproduce and/or find an upstream tarpaulin issue to link to and see if we can ever resolve this with the upstream dep or at the very least try to remove non-deterministic fails.

This has been resolved now I think.


Remove default location from usage test and let cobra handle that

Description
This PR fixes #1994
Notes for Reviewers
Signed commits

[.] Yes, I signed my commits.

@kushthedude @leecalcote Sorry for keeping this stale. gofmt'ed the file and pushed. Do review and merge :)


More runner and workflow metrics

Is your feature request related to a problem? Please describe.
As an ARC operator, I want to be able to monitor and alert on the status of the runners I manage. It would be convenient if these metrics were bundled as a part of the ARC deployment instead of using another 3rd part adapter
A few things that would be interesting & actionable:
runner status & availability

runner startup times
runner status (idle, available, offline)

workflow status

queued workflows
in-progress workflows
run times
pass/fail stats

Describe the solution you'd like
ARC is well-positioned to aggregate and expose the data for these metrics.
Keeping up-to-date state for running Workflows would probably require a webhook, since the status can change faster than polling can keep up.
eg.
# HELP github_workflow_runtime Duration all workflow runs
# TYPE github_workflow_runtime histogram
github_workflow_runtime_bucket{conclusion="success", workflow_id="12345678",le="0.1"} 5
github_workflow_runtime_bucket{conclusion="success", workflow_id="12345678",le="1"} 5
github_workflow_runtime_bucket{conclusion="success", workflow_id="12345678",le="10"} 5
...
github_workflow_runtime_bucket{conclusion="success", workflow_id="12345678",le="+Inf"} 5
github_workflow_runtime_sum{conclusion="success", workflow_id="12345678"} 1000
github_workflow_runtime_count{conclusion="success", workflow_id="12345678"} 5

# HELP github_workflow_conclusion Conclusions of all github workflow
# TYPE github_workflow_conclusion counter
github_workflow_conclusion{conclusion="success", workflow_id="12345678"} 5

# HELP github_workflow_status Live status information
# TYPE github_workflow_status gauge
github_workflow_status{status="in_progress", workflow_id="12345678"} 5
github_workflow_status{status="queued", workflow_id="12345678"} 5

# HELP github_runner_status Live runner status information
# TYPE github_runner_status gauge
github_runner_status{status="idle", runner_id="4321"} 5
github_runner_status{status="active", runner_id="4321"} 5
github_runner_status{status="offline", runner_id="4321"} 5

# HELP github_runner_startup Duration all runner startups
# TYPE github_runner_startup histogram
github_runner_startup_bucket{runner_id="4321",le="0.1"} 5
github_runner_startup_bucket{runner_id="4321",le="1"} 5
github_runner_startup_bucket{runner_id="4321",le="10"} 5
...
github_runner_startup_bucket{runner_id="4321",le="+Inf"} 5
github_runner_startup_sum{runner_id="4321"} 100
github_runner_startup_count{runner_id="4321"} 5

Describe alternatives you've considered
https://github.com/Spendesk/github-actions-exporter is an available solution for getting some of these metrics, but it has a few problems:

It is not able to deal with ephemeral runners, and leads to invalid metrics
The polling model for workflows data can miss states that don't last long (eg. queuing, short-lived workflows)
The workflow metrics creates an infinitely growing list of metrics series

Is this within the scope of ARC? If so I am willing to work on the feature

@ColinHeathman Hey! Thanks for bringing this up.

Is this within the scope of ARC? If so I am willing to work on the feature

Yes, if and only if ARC is going to be the only viable solution to the problem.
ARC doesn't currently track workflow runs so I wonder how ARC can provide github_workflow_runtime_bucket better than github-actions-exporter.
github_workflow_status seems relatively fine, because it looks like almost like a snapshot of workflow run statuses that ARC might probably know about when the pull-based autoscaling metric is TotalNumberOfInProgressAndQueuedWorkflowRuns. But the same information can be observed by github-actions-exporter too. The same can be said for github_runner_status.
github_runner_startup_bucket is interesting. How are you going to measure the startup duration though? 🤔 AFAIK, we have no reliable way to detect when the runner has finished starting up. Did actions/runner had a hook that can be used to notify ARC about when it finished starting, or shall we follow logs from the runner container so that we can pattern-match against a message actions/runner writes once it finished starting?

I am also interested in what alerting people use to make sure their self hosted github actions cluster is healthy.

I think github_workflow_status would be unreliable for any pull-based autoscaling metric. I think that it would be necessary to put the logic into the github-webhook-server and make it a prerequisite for getting workflow metrics. github-actions-exporter would have to be rewritten to support webhooks, which is why I think it might make sense to be a part of ARC.
github_runner_startup_bucket is more difficult. It might belong in a separate discussion. I would think that the best way to measure it would be to track the time between a pod creation, and when Github first sees the runner as either "Active" or "Idle" for the first time.

Not stale

@cep21 What's the criteria for alerting folks of unhealthy runners? This feature request still seems to lack the exact use-case it should support?

My use case:
I work in infrastructure.  My team is responsible for the health and availability of the self hosted runners.  I would like to proactively know if something is broken, rather than wait for teams to tell me the system is broken.  Devops and infrastructure teams often use metrics for this kind of workflow, where they create alerts on these metrics.  Here are a few metrics that will help me

How long does it take for a job to find a self hosted runner to run on

This is the most important one for me.  It represents slowness in the self hosted runner setup.

Current (and maximum) number of auto scaled runners: so I can alert when that number equals the maximum size
Can self hosted runners log out the action job that they are running currently?
An alert or log or metric on how long it takes the controller to give me a pod for a runner
A metric on the number of successful (and failed) workflow runs
The time delta between the system asking for a self hosted runner, and a self hosted runner being up and available.

For infrastructure teams, it's often not enough to just deploy software: a critical part of the workflow is to setup alerting and metrics so we can know when things are broken or misbehaving.

@cep21 I hear you, but it looks like all of those metrics can be implemented outside of ARC by leveraging GitHub Actions API and workflow_job webhook events emitted by GitHub 🤔 Have you already surveyed other solutions like https://github.com/Spendesk/github-actions-exporter?
I can also recommend reading https://www.cbui.dev/github-actions-self-hosted-runner-observability-and-monitoring/

Is anyone working on this? If not, we should better close this as stale and encourage other folks to start a new project as a place for collaborations around metrics and monitoring. ARC should provide metrics about its own resources, not metrics across all the Actions related resources owned by GitHub.

I can give an example of one possible metric and a use case:
Metric:
The state of the runners created by ARC

Number of active, idle, and offline runners by label

And I think this is something that ARC already is checking when it is using the metric PercentageRunnersBusy  in the Pull Driven Scaling and could make sense to have  ARC , because ARC is creating the .runners
The use cases:

Tracking usage of the runners, if I have a HorizontalRunnerAutoscaler  that the tenx is ten and the desired value is already 10, I can think that my users already don't have enough runners, and I need to create more runners. But in reality, frotenthat 10 runners, I already have only 6 or 7 are in the active state
Tracking the offline runners that are leftovers from ARC

If you approve of this feature, I don't mind working on that. As a workaround, I built a service that tracked a lot of data related to GitHub Actions and Runners.

I'm planning on making a PR to enable a new prometheus server on the githubwebhookserver runtime, and expose these metrics:
github_workflow_runtime (histogram)
github_workflow_queue (histogram)
github_workflow_conclusion (counter)
github_workflow_status (gauge)

Currently we are most interested in monitoring queue times. We have an elastic cluster, and custom startup logic that takes time so queue time is something we have had problems with in the past, and we need a solution to monitor this.

@ColinHeathman Hey! Thanks for your help. Nit but I think the prefix should be github_workflow_job_ rather than just github_workflow_ as actually "workflow jobs" get scheduled onto and run by self-hosted runners, rather than "workflows".
The ideas of four metrics sound good!
Just my two cents- Implementing three histogram metrics can be a bit tricky, as you might need to track all the statuses of ongoing workflow jobs that are deduced from received workflow job events.


Update profile image name and image link in AboutUs

Codecov Report

Merging #84 (26d48a4) into master (a78187a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #84   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 a78187a...26d48a4. Read the comment docs.


An error after running hook:pre-commit hook

I just tried to run hook:pre-commit, but I keep getting this below error:
failed PHP Error:Class "\SebastianFeldmann\Git\Operator\Index" not found in phar:///home/iman/Sites/test/vendor/bin/captainhook/vendor/sebastianfeldmann/git/src/Repository.php line 209
How to reproduce:
mkdit test && cd test
composer init
git init
composer require --dev captainhook/captainhook-phar
vendor/bin/captainhook hook:pre-commit

Should be fixed in 5.18.1
Please let me know if it works for you as well.

@sebastianfeldmann Yes, It's fixed, thanks

Thanks for letting me know :)


compileTemplate function needs new id argument

Using this package with compiler-sfc 3.0.4 shows the following message:
[@vue/compiler-sfc] compileTemplate now requires the `id` option.`.

Should be an easy fix, just I thought I'd point it out. The plugin still seems to be working, I thiiink...? Not completely sure.

Thanks for reporting :) will be released as v2.0.0-1


Clean up extra spaces

Description of what you did:
Clean up extra spaces, although these errors are not important, they affect the code specification.

My PR is a:

[ ] 💥 Breaking change
[ ] 🐛 Bug fix
[x] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[ ] Multiple commits are squashed into one commit.
[ ] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[ ] All npm dependencies are pinned.

thanks!
/approve
/lgtm


Return  value of Push.create is undefined for the first invocation

The first time a notification is created using Push.create, the return value is undefined. This is reproducible in the example.html provided.
Because of this, calling close() on the notification fails.

You say for the first invocation... upon calling it in future requests, is it not undefined?

That is correct.

/* Request permission if it isn't granted */
            if (!self.Permission.has()) {
                self.Permission.request(function () {
                    return create_callback(title, options);
                });
            } else {
                return create_callback(title, options);
            }

When the flow has to request the permission, the return value is undefined to the caller.

I guess it would better to promisify and return a promise ...

@venkatramachandran Oh I see what you mean... guess that was an oversight. I can just change the scope of that return and it should work just fine, unless @sarbbottam, is there any definitive advantage to using a promise? I'm not too familiar with them. Could you elaborate?

if (!self.Permission.has()) {
    // self.Permission.request() is a async operation
    // whenever self.Permission.request() is invoked
    // self.create will return undefined
    // for sure you don't want to make self.Permission.request() synchronous
    // so, if you like to return a handle to the user
    // I guess you have to use promise
    self.Permission.request(function () {
        return create_callback(title, options);
    });
} else {
    return create_callback(title, options);
}

It would be something like.
// self.create is a Promise
return new Promise(function(resolve) {
    ...
    if (!self.Permission.has()) {
        // self.Permission.request is a promise
        retrurn self.Permission.request()
      .then(function() {
            // create_callback is also a promise
            return create_callback(title, options);
        })
    } else {
        // create_callback is a promise
        return create_callback(title, options);
    }
    ...
}

@sarbbottam I see what you're saying... in fact, when I posted earlier I overlooked the fact that that call is in-fact asynchronous and so doing some research I think promises would be appropriate. However, what is the browser compatibility like for this? According to caniuse.com, IE doesn't support promises at all. Is that a kind of shim framework that could be used for older browsers?

According to caniuse.com, IE doesn't support promises at all.

IE doesn't support notifications either, I guess, it should be OK, for this use case. But its supported in Edge

Is that a kind of shim framework that could be used for older browsers?

babel-polyfill could be used, but as stated I guess, it is not required.

Wound up using the tiny 4kb library promise-polyfill to fix this, as seen in PR #32. Closing issue.


Configure transparency percentage?

Hello,
I really liked this extension, and I think it would be even greater if we could configure the transparency percentage.
If you're too busy now to implement this, I can make a PR.
Regards

Sorry for responding late. Currently I have no time for this package, please make a PR, all PR are welcome and appreciated. Thank you.

Hello,
If you actually want to change the background opacity, there is a way that you can did it by yourself.
Add this to your stylesheet:
.all-times-you-know {
  opacity: 0.5; // what you want between 0.0 and 1.0
}


It's even legal?

This software is designed to harm the sites of the Russian Federation, but at the beginning of the file README.md it is said that "This software is created for educational purposes and you should use it only on your own website."

Tell me one international law that prevents anyone from running this software. There isn`t any.
If your country prevents you from doing so, be a good law-abiding citizen and don't use it. Simple as that.

Creating software is 100% legal. Its on yours responsibility how you use it and what are you doing with it.
We are providing software "as is". If its not legal in your county, you should not use it. Same with other popular software like MHDDoS, LOIC, .... - they also can be used "... to harm the sites of the Russian Federation"


build_deb_image.sh:add debug option for testing

Following https://github.com/scylladb/scylla-pkg/pull/2393, All image
builds (official or debug) have the same image name prefix.
Let's make sure when we (releng) building an image (ami,gce or azure) we
create the name of the image with our prefix releng-testing
Once this will be merge, we need also to change hydra.sh script in SCT
repo to fetch the latests AMI images and ignore debug releng-testing
prefix (https://github.com/scylladb/scylla-cluster-tests/issues/3950)

Testing
Gce: https://jenkins.scylladb.com/view/master/job/scylla-master/job/releng-testing/job/gce-image/448/console OK
Azure: https://jenkins.scylladb.com/view/master/job/scylla-master/job/releng-testing/job/azure-image/114/ - OK
Aws: https://jenkins.scylladb.com/view/master/job/scylla-master/job/releng-testing/job/ami/762/ - OK


feat: Add cli for listing all module accounts

cref: https://github.com/cosmos/cosmos-sdk/pull/9812
Description
This PR adds a CLI for querying all module accounts, module-accounts, in the auth module.
Using this command would display all the account information, including human readable name of the module, module account address, account number, permissions, etc.
This command would be especially useful for developers using cosmos-sdk as there are lots of instances where a developer would have to look into the module account, but only to do so by getting the module account address after getting the module name, hashing it and bech32fying it in the status quo. By using this command, users would be able to get a quick look on the list of all module accounts' information.

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] followed the guidelines for building modules
[ ] included the necessary unit and integration tests
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[x] confirmed the correct type prefix in the PR title
[x] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[x] reviewed tests and test coverage
[x] manually tested (if applicable)

Yeah this is a cherry pick from the upstream

reverting the commit, merging it identical to upstream


[Bug] CVR Animator Driver component does not like the Override Controller in the Animator Controller field.

The Animator Driver will bug out if you use an Override Controller in your Animator slot.
This tool will automatically place the Override Controller in your Animator Controller slot on compile to make debugging in Play Mode easier, but due to this issue, there should be a toggle to disable the functionality...

https://feedback.abinteractive.net/p/animator-driver-gui-will-break-if-override-controller-is-used-in-animator
I cannot do anything to fix this.

This was fixed in CCK3.5.


Make StyledObject able to output on stderr as well.

Task Description
Imagine an application: it lets you explore the directories, and then, on user action, it prints the desired path to stdout so that the cd "$(application)" command can cd into that path.
It doesn't work because nothing is displayed if I try to draw the directory tree in stdout: it's redirected to the command. This scenario is common solved by having the application write to stderr until the user choose a directory and print the chose path to the stdout.
Task Todo
Figure out how to replace hardcoded stdout with a type that implements Write.; both stdout, sterr, implement the Write trait.

[ ] stdout
reference 1
[ ] stdout reference 2

This is fixed in #175, no flushes are done, and the Display writes to the given formatter.


Mysql syntax error during migration. code: 1059

Hi,
I got an mysql error during migration process.
Syntax error or access violation: 1059
Cause by name of unique key being too long.
reference source
A key name 'translator_translations_locale_namespace_group_item_unique' will be used automatically, whose length is 58.
And if database prefix is being used, it may  violate the restriction of maximum length(64 characters).
Is it possible to just give it a specific short key name?
like
$table->unique(['locale', 'namespace', 'group', 'item'], 'short_name');

Same here


Dragging inside modal and release outside of modal closes the modal

Summary:
Dragging inside modal and release outside of modal closes the modal
Steps to reproduce:

Drag something inside of a modal
Release the mouse outside the modal
Modal closes due to detecting a "click" outside the modal

Expected behavior:
Modal should not respond to mouse up that didn't start inside of the overlay
Link to example of issue:
https://codesandbox.io/s/mgwy6V6E
Additional notes:
I noticed this while using a draggable interface component inside of the modal. It's natural to sometimes overshoot the modal and drag onto the overlay. This results in an unexpected close of the modal.

Hi @jameskraus
Yeah, that's pretty annoying.
I stumbled upon the same issue recently after replacing a regular textarea with a Draft.JS component in my modal dialogs. I wonder if you know how to solve it. I was thinking of catching overlay clicks, but apparently react-modal doesn't currently provide this callback.

This is probably related #383

It seems, it worked well before. Example from official documentation uses version 1.6.2.
Here is the same example, but with version 2.2.2:
https://codesandbox.io/s/9rkADGWz
As you can see, if you select text in the window and mouseUp out, the window will be closed

Guys, if someone is annoyed by this issue, you can rollback to v1.6.2. That was a solution for me

Hi @tsmirnov
Indeed, 1.6.5 works!
What's interesting is that #217 says this:

If the user perform a mouse down on the content, then a mouse up on the overlay: do nothing

Sounds like the original idea was not to close the modal, but this is exactly what's happening.

@tsmirnov Thank you for tracking down the issue.
@tsmirnov @denisftw There is a test for this behavior, kinda weird we have a regression on this? Any extra information about this (browser version...)?

@diasbruno I am on Mac.
Chrome Version 60.0.3112.78
Safari Version 10.0.3 (12602.4.8)
But I don't think this is related to browser version. All people in my office experience that issue.

@tsmirnov ok.

@diasbruno I created pull request with the fix. When it can be added to release package?

@tsmirnov Awesome. It looks good. We have some PRs to merge too, so later I'll release a new version.

@jameskraus A fix is available on v2.2.4.

@diasbruno Awesome! I've just updated the package and can testify that it works perfectly :+1:
Thanks @tsmirnov and @diasbruno !


how to set buffercapacity in sbe without prediction

how to set buffercapacity in sbe without prediction. i want  to set the buffer capacity without prediction or hardcoded. what is  the best solution plz. guide me.

Can you provide a bit more detail on what you would like to do? Maybe with a worked example.

when we are encoding  any data which is variable data.we can't predictable how much size(nuber of bytes) it has .without predicting the size how can we assign the size  for ByteBuffer

ByteBuffer. allocateDirect (number of bytes)

If you don't know the size of your data within a reasonable bound then SBE might not be the best option. I prefer to chunk data rather than have unbounded message sizes.
A possible option is to create an implementation of MutableDirectBuffer that will resize an underlying buffer to accomidate unbounded messages.

final ByteBuffer byteBuffer = ByteBuffer.allocateDirect(Integer.valueOf(ConstantsUtility.BUFFER_SIZE));
final UnsafeBuffer directBuffer = new UnsafeBuffer(byteBuffer);
this is we are right now hardcoding buffersize.but we need to pass dynamic data size.

A schema will have an explicit max length based on field sizes for numInGroups and for length in VarDataEncoding. Also, the system probably should have an implied max length based on practical sizes for those fields or based on convention or configuration.
Depending on the complexity of the schema, determining the size of a message based on var data lengths as well as number of groups is relatively straight forward. In fact, there is an enhancement for this #291
An implementation of MutableDirectBuffer that can resize is also quite useful on its own.
Another option is to pick a size that seems reasonable. Use the normal UnsafeBuffer. And keep bounds checking on. When a bounds check is hit, then resize to double previous value and re-encode.

@satyasarmawalkingtree I've pushed an ExpandableArrayBuffer to Agrona that will expand to the required capacity of your messages. Please give it a try and provide feedback.
https://github.com/real-logic/Agrona/commit/a481ead567d2b6cf16a34aac70ab1ad86d88af26


Some HTML entities in JSX are removed (e.g  )

I noticed that some of our code includes   in JSX, but these seem to be removed when we moved over to SWC.
e.g with&nbsp
Config
  jsc: {
        target: 'es2018',
        parser: {
             syntax: 'typescript',
             tsx: true
        }
}

Expected behavior
These should be retained
The version of @swc/core: 1.2.47

I tested it and I found that swc emits
const Span = React.createElement("span", null, "with\xa0");

for
const Span = with ;

and
const Span = React.createElement("span", null, "with&nbsp");

for
const Span = with

@kdy1 I think there's still an issue here when entities are on their own
import React from 'react';

export const Example = () => {
  return (
    <>
      Hello something long to not trigger line break

    </>
  );
};

gets transformed to
import React from 'react';
export var Example = function() {
    return( /*#__PURE__*/ React.createElement(React.Fragment, null, /*#__PURE__*/ React.createElement("span", null, "Hello something long to not trigger line break")));
};

Whereas babel is producing
import React from 'react';
export const Example = () => {
  return /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement("span", null, "Hello something long to not trigger line break"), "\xA0");
};

i.e. including the nbsp as a separate child of the fragment.


3D coordinate calibration

Hi Daniil,
me again ;) I encountered some problems with 3D calibration. I used ROS camera calibration node based on http://wiki.ros.org/camera_calibration/Tutorials/MonocularCalibration. Then I imported R and T matrices into --extrinsics parameters.
I obtained 3D poses of left shoulder and right shoulder at keypoints 3 and 9.
left shoulder: [-72.70494   27.558527  -7.819101]
right shoulder: [-56.104603  18.041153 -10.176827]
Since the line of two shoulders is parallel to one axis,  which means the numbers of one coordinate should be very similar,  but the number of three axises look all quite different. Do you have any idea what might be the reason?
Considering of calibration, is there also somewhere in the code for the distorted parameters?
One more question is about the axis direction. In the line of 107 in the demo. py,  why do we need to change the direction and sequence of the axis? What do the axises look like after such transition?
Thanks a lot for your help!
Best,  Fan

As the first step try to compare 3D values with default extrinsics and new one. I believe they should have similar module, this can check if extrinsics formats correspond to each other. We need to swap axes due to different coordinate systems used in annotation and for visualization. You can select the actual axis you need by passing coordinates, e.g (0, 0, 1), into drawing code and see where point with such coordinates will be drawn.

Hi Daniil, thank you for your reply! I found the axes' directions. ;)
I still have some problem with the extrinsic. I guess it might due to the different image size when I do the calibration.
I noticed there is only height_size and fx in the demo.py, which means we assume the image is square. What if the height_size differs weight_size? For example, I have height with 480 and weight with 640, can I still use the pre-trained model human-pose-estimation-3d.pth, or do I need to retrain my own model?
Thanks for your help!

Hi! Extrinsics are used only for visualization, network accepts images captured under different angles. Usually fx ~ fy, so for simplicity one value is used. Image aspect ratio can be any. Default extrinsics correspond to HD camera with 0 id from CMU Panoptic Dataset, may be it can help somehow.

Hi Daniil, I wanted to examine how accurate the human pose prediction is. So I used an Intel Realsense camera and could get the real depth of the joint. However, I noticed the depth from pose estimation is ~10 cm less than the depth from the realsense camera.
Do you have any idea why that comes? Is the origin of the camera frame the camera itself or inside 10 cm? Thanks a lot!

The code which finds the translation is here. Discussion is drag on little bit regarding the first two questions, may be it is better to have a talk (if so, just mail me at gmail, will schedule a meeting)?

Hi Daniil,  thank you. This is my email<EMAIL_ADDRESS>Looking
forward to hearing from you.
Regards,
Fan
Daniil-Osokin<EMAIL_ADDRESS>schrieb am Di., 17. Nov. 2020,
20:50:

The code which finds the translation is here
https://github.com/Daniil-Osokin/lightweight-human-pose-estimation-3d-demo.pytorch/blob/master/modules/parse_poses.py#L124-L135.
Discussion is drag on little bit regarding the first two questions, may be
it is better to have a talk (if so, just mail me at gmail, will schedule a
meeting)?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Daniil-Osokin/lightweight-human-pose-estimation-3d-demo.pytorch/issues/45#issuecomment-729162754,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKUYVCGWAZODSZLMWNQJY7TSQLHXPANCNFSM4TEEIZQA
.


Clean up prior introduction of symbolic engine

This PR factorizes all functions that inspect and manipulate sympy expressions in a new module called symbolics.py. These functions are also renamed by using the prefix dse_ (devito symbolic engine).
I'm planning to add more functionalities to the symbolic engine in the following weeks

Now travis fixed
As for the imports: I can do that, but that'd mean importing 5 modules, which is > 80% of symbolics. I can do this tomorrow, when I'll also incorporate feedback from @navjotk and @mloubout

OK, I fixed the import thing, Travis is happy, and there's not much more to say about this, so merging...


Fix missing date parameter

Checklist:

[x ] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[x ] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

The missing date parameter is not part of the test or the learning objective. It tends to cause a 400 error while setting up the exercises for the log test.

@OsakaStarbux , thank you, for helping contribute to the freeCodeCamp community. It is taking a while to get through QAing PRs. So, thank you, for the ping.
I might be forgetting something, but I am not sure there is any reason to add a date parameter to the request, as the project description does specify:

I can add an exercise to any user by posting form data userId(_id), description, duration, and optionally date to /api/exercise/add

So, the camper's /add logic should account for creating the date object if one is not provided. So, I believe this test was purposefully written to take advantage of this.

You're right. Although, because the test is not actually using the form,
that parameter is not just empty, it's missing. It means that I can pass
the test for the /add route and then silently fail the /add route during
the log test. Also, it would be better if that were a separate test so that
campers know what they are failing.
On Mon, Jul 27, 2020 at 7:38 AM Shaun Hamilton<EMAIL_ADDRESS>wrote:

@OsakaStarbux https://github.com/OsakaStarbux , thank you, for helping
contribute to the freeCodeCamp community. It is taking a while to get
through QAing PRs. So, thank you, for the ping.
I might be forgetting something, but I am not sure there is any reason to
add a date parameter to the request, as the project description does
specify:
I can add an exercise to any user by posting form data userId(_id),
description, duration, and optionally date to /api/exercise/add
So, the camper's /add logic should account for creating the date object
if one is not provided. So, I believe this test was purposefully written to
take advantage of this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/pull/39266#issuecomment-664049134,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3ESTX7QBUG4WJC3OJWVGDR5SV4RANCNFSM4O3LSSUQ
.

Hi @OsakaStarbux, thanks for your contribution.
Like @Sky020 mentioned, I'm not sure if adding a date parameter there is necessary. I tried the example project at https://exercise-tracker--freecodecamp.repl.co/ both with the form on the page and with Postman, and leaving the date out works as expected in both cases.
The actual assert statement for that test is quite forgiving as well, and really just checks that an array with a length of 1 is returned.
Could you give us some examples of when the API could silently fail in the /add route you modified?

@scissorsneedfoodtoo Thanks for trying it out. I don't know if you passed an empty string or no parameter at all but I'm glad it works as you expect.

@OsakaStarbux, I tried it with both an empty string and no parameter and got the current date in the response. That's not to say that all projects work that way, though, just the example.
I do agree that the test text for this and the other backend projects could use some clarification, though. And I do like the idea about splitting some of the tests up to test for specific functionality like you mentioned. Dates and times are notoriously tricky to test for, but it would be good to have more granular tests.


fix(prover): speed up LWG and NWG

What ❔

Loading of proofs for each recursive circuit is now done asynchronously
Each recursive circuit is now processed in a blocking thread, which speeds up serialization and other CPU-sensitive processing.

Locally, with additional artificial network delays (500 ms), a x5 speed up is observed
Checklist

[ ] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.
[ ] Code has been formatted via zk fmt and zk lint.

Did you run tests both ways (sync/async)?

Yes, the difference is noticeable. In different cases, the strategy for using threads by tokio may be different depending on blocking/non-blocking context.

The spawned task may execute on the current thread, or it may be sent to a different thread to be executed. The specifics depend on the current Runtime configuration.

https://docs.rs/tokio/latest/tokio/task/fn.spawn.html

tldr: afaik, async is single-threaded, but blocking tasks aren’t

tokio::spawn will run your task on one of the runtime threads (tokio uses the same threadpool to schedule and run stuff). tokio::spawn_blocking will run your task on a dedicated blocking threadpool, which is much larger.
handle.block_on(..) will block the thread you run on, but it won't block the runtime itself, e.g. runtime will only be utilized for async stuff.


Update state system to use StateName

This PR updates the state(::SiteType,...) system to use a new type StateName"Name"
to allow the system to be user-extensible and offer automatic error handling, etc.
It falls back to the older system for backwards compatibility.
This PR also updates the has_fermion_string system used by AutoMPO to use OpName
for similar reasons.

Redesign of state system to use StateName type
Update physics site types to use StateName overloads
Update has_fermion_string system to use OpName

Looks good! Nice to get rid of all of those if-statements, the code looks much cleaner now.

Thanks for the helpful review


Test installation instructions in clean conda environment

Check that pip installation functions as expected. This could be accomplished in https://github.com/kostaleonard/mlops-image-generation-example

Installation appears to work as expected in #28


Question: 256 block high nether?

I'm looking for a mod that will generate a full 256 block high nether a la the Tall Nether Spigot plugin. Is this something you have on the roadmap?

I had planned included this in the mod at one point, but it was really buggy and it lagged really badly. At this time, I do not plan on adding it back in.


Adding default value for ClaimsEndpointAccessTokenName

Adding default value for B2C Technical Profile OAuth / ClaimsEndpointAccessTokenName metadata.

@piotrgradzinski : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@yoelhor, fyi - see also related PR https://github.com/MicrosoftDocs/azure-docs/issues/66364
#sign-off

@yoelhor, fyi - see also related PR https://github.com/MicrosoftDocs/azure-docs/issues/66364
#sign-off


Add the labels on the mobile menu

Describe the feature in detail (code, mocks, or screenshots encouraged)
I want to add the label next to the links in the mobile menu like on the bigger screen (aside section). It should look like this on mobile

What type of pull request would this be?
Docs
Provide relevant links or additional information.
No response

Go for it!


Fix #23944 - UI crash in cluster discovery

Signed-off-by: James Talton<EMAIL_ADDRESS>https://github.com/stolostron/backlog/issues/23944

/lgtm

/lgtm


[release-1.10] Fix cronjob timestamp trimming.

Manual cherry-pick of https://github.com/istio/istio/pull/35655 fix https://github.com/istio/istio/issues/36146

@bianpengyuan: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio_release-1.10
8a35ebfcb3513b495e898bfa1d54f8cc8ac26c4f
link
true
/test unit-tests_istio_release-1.10

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan looks like legitimate unit test failure with:
== RUN   TestCronJobMetadata
=== RUN   TestCronJobMetadata/cron-job-name-sec
    util_test.go:212: Object metadata got &ObjectMeta{Name:sec,GenerateName:sec-1234567890-pod,Namespace:,SelfLink:,UID:,ResourceVersion:,Generation:0,CreationTimestamp:0001-01-01 00:00:00 +0000 UTC,DeletionTimestamp:<nil>,DeletionGracePeriodSeconds:nil,Labels:map[string]string{},Annotations:map[string]string{},OwnerReferences:[]OwnerReference{OwnerReference{Kind:Job,Name:sec-1234567890,UID:,APIVersion:v1,Controller:*true,BlockOwnerDeletion:nil,},},Finalizers:[],ClusterName:,ManagedFields:[]ManagedFieldsEntry{},} want {Name:sec GenerateName:sec-1234567890-pod Namespace: SelfLink: UID: ResourceVersion: Generation:0 CreationTimestamp:0001-01-01 00:00:00 +0000 UTC DeletionTimestamp:<nil> DeletionGracePeriodSeconds:<nil> Labels:map[] Annotations:map[] OwnerReferences:[] Finalizers:[] ClusterName: ManagedFields:[]}
    util_test.go:215: Type metadata got &TypeMeta{Kind:CronJob,APIVersion:batch/v1beta1,} want {Kind:CronJob APIVersion:batch/v1beta1}
=== RUN   TestCronJobMetadata/cron-job-name-min
    util_test.go:212: Object metadata got &ObjectMeta{Name:min,GenerateName:min-12345678-pod,Namespace:,SelfLink:,UID:,ResourceVersion:,Generation:0,CreationTimestamp:0001-01-01 00:00:00 +0000 UTC,DeletionTimestamp:<nil>,DeletionGracePeriodSeconds:nil,Labels:map[string]string{},Annotations:map[string]string{},OwnerReferences:[]OwnerReference{OwnerReference{Kind:Job,Name:min-12345678,UID:,APIVersion:v1,Controller:*true,BlockOwnerDeletion:nil,},},Finalizers:[],ClusterName:,ManagedFields:[]ManagedFieldsEntry{},} want {Name:min GenerateName:min-12345678-pod Namespace: SelfLink: UID: ResourceVersion: Generation:0 CreationTimestamp:0001-01-01 00:00:00 +0000 UTC DeletionTimestamp:<nil> DeletionGracePeriodSeconds:<nil> Labels:map[] Annotations:map[] OwnerReferences:[] Finalizers:[] ClusterName: ManagedFields:[]}
    util_test.go:215: Type metadata got &TypeMeta{Kind:CronJob,APIVersion:batch/v1beta1,} want {Kind:CronJob APIVersion:batch/v1beta1}
=== RUN   TestCronJobMetadata/non-cron-job-name
    util_test.go:212: Object metadata got &ObjectMeta{Name:job-123,GenerateName:job-123-pod,Namespace:,SelfLink:,UID:,ResourceVersion:,Generation:0,CreationTimestamp:0001-01-01 00:00:00 +0000 UTC,DeletionTimestamp:<nil>,DeletionGracePeriodSeconds:nil,Labels:map[string]string{},Annotations:map[string]string{},OwnerReferences:[]OwnerReference{OwnerReference{Kind:Job,Name:job-123,UID:,APIVersion:v1,Controller:*true,BlockOwnerDeletion:nil,},},Finalizers:[],ClusterName:,ManagedFields:[]ManagedFieldsEntry{},} want {Name:job-123 GenerateName:job-123-pod Namespace: SelfLink: UID: ResourceVersion: Generation:0 CreationTimestamp:0001-01-01 00:00:00 +0000 UTC DeletionTimestamp:<nil> DeletionGracePeriodSeconds:<nil> Labels:map[] Annotations:map[] OwnerReferences:[] Finalizers:[] ClusterName: ManagedFields:[]}
    util_test.go:215: Type metadata got &TypeMeta{Kind:Job,APIVersion:v1,} want {Kind:Job APIVersion:v1}
--- FAIL: TestCronJobMetadata (0.00s)
    --- FAIL: TestCronJobMetadata/cron-job-name-sec (0.00s)
    --- FAIL: TestCronJobMetadata/cron-job-name-min (0.00s)
    --- FAIL: TestCronJobMetadata/non-cron-job-name (0.00s)

@Monkeyanator fixed. thanks!

@bianpengyuan: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio_release-1.10
8a35ebfcb3513b495e898bfa1d54f8cc8ac26c4f
link
true
/test unit-tests_istio_release-1.10

integ-security-k8s-tests_istio_release-1.10
c08ebbc05af3b72c5e83049855fcf307d06685b6
link
true
/test integ-security-k8s-tests_istio_release-1.10

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@bianpengyuan: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-k8s-tests_istio_release-1.10
c08ebbc05af3b72c5e83049855fcf307d06685b6
link
true
/test integ-security-k8s-tests_istio_release-1.10

integ-pilot-k8s-tests_istio_release-1.10
c08ebbc05af3b72c5e83049855fcf307d06685b6
link
true
/test integ-pilot-k8s-tests_istio_release-1.10

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Customize labels for Grafana Pod

Currently I don't see how can I set the labels for the Grafana Pod which is managed by the operator. It would be great if I could set custom labels which are then applied to the Grafana Pod, or even better if the Pod would inherit the labels of the Grafana CRD itself.

You're right, this is currently not configurable, the operator will add a label app: grafana to the deployment which is then passed down to the pods. Do you have a use case for custom labels on the pod?
In general the idea is that the operator takes care of setting up and managing all resources including the service and route / ingress for which this label is used.

Well, in my case I'm considering how to migrate our own Grafana to the grafana-operator. For the transition period, we'll have two grafanas, and I want to have two separate services in front of them. I have to differentiate the two Grafana Pods, that's why I need to configure the labels on both Pods.
This is a use case for anyone who for any reason wants to run two Grafanas in the same namespace.
About the operator setting up the ingress/route for me, I'm not running in OpenShift, we are running a self-managed Kubernetes cluster so this feature is not really useful for us.

I see. I'll try to add a way to pass extra labels to the deployment and pod (and possibly other managed resources?).
As a workaround for now, one label is currently added (app: grafana), maybe you can use that to identify the instances?

The problem is that every Grafana installation will have app=grafana added as a label. What I need is additional labels to the label selectors can be more specific, about which grafana Pod we target.

@zendai #39 gives you the option to customize the pod label with an operator flag. I've pushed an image with this change (quay.io/integreatly/grafana-operator:customize-pod-label). You can add customize the pod label by adding an additional flag to the operator args in operator.yaml:
          args:
            - '--grafana-image=quay.io/openshift/origin-grafana'
            - '--grafana-image-tag=4.2'
            - '--pod-label-value=<custom label value>'

Please let me know if this works for you.

The change has been merged and is now also available in the latest tag.

Available in https://github.com/integr8ly/grafana-operator/releases/tag/v1.4.0


CTRL-C to exit out of Governor when paused waiting for a vacuum doesn't exit

When paused in the Governor waiting for a vacuum the interrupt is caught but processing isn't correctly stopped and remain in the Governor loop waiting for PostgreSQL vacuum.

See: https://github.com/Senzing/g2-python/issues/153

See #71


Maintenance

React Boilerplate

[x] You have followed our contributing guidelines
[x] Double-check your branch is based on dev and targets dev
[x] Pull request has tests (we are going for 100% coverage!)
[x] Code is well-commented, linted and follows project conventions
[ ] Documentation is updated (if necessary)
[x] Internal code generators and templates are updated (if necessary)
[ ] Description explains the issue/use-case resolved and auto-closes related issues

@BenLorantfy I saw you disabled some ESLint rules in the new Airbnb config to keep in line with the react-boilerplate code. Here I went with disabling only the props-no-spreading rule but kept the one about static properties of a class and updated the injectors accordingly. You don't have to follow the same rule. This is just what I went for here.

Coverage remained the same at 100.0% when pulling 82faa24959c5fc0108480a42ee10d2daef524e27 on maintenance into 5a13b22017c28a01994a40aabb8985d3ce1ca048 on dev.


passing paste event

passing paste event (which is trusted) allows to use document.execCommand() to further manipulate the clipboard (for example overwriting image data with url to uploaded image)

Coverage remained the same at 100.0% when pulling 0c6c46cf59f6501323e4e6886304fd5ba3015c3c on dreszczyk:master into 672c0143edcc5d9c84ad352adc327dea232986a0 on redgeoff:master.

Looks good, but unit tests are failing


For multiple listening instances, only the last started instance can receive messages

Question desc:
For multiple listening instances, only the last started instance can receive messages

ok, I'll take a look for this problem


Comment-Based Help ".LINK" Section Duplicates Lines In Some Scenarios

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
In some scenarios on PowerShell 7.3, when using Get-Help [FunctionName], links are duplicated in the related links section. I've noticed that this only seems to happen if there is more than one line in the .LINK section.
This will reproduce it:
function ExampleFunctionDuplicateLink {
    <#
        .LINK
        http://google.com/
        Line 2
    #>

    # No Code
}
Get-Help ExampleFunctionDuplicateLink

Output:
RELATED LINKS
    http://google.com/
    Line 2
    http://google.com/
    Line 2

Removing the single forward slash from the first line (after google.com) will have the output go back to normal:
function ExampleFunctionDuplicateLink {
    <#
        .LINK
        http://google.com
        Line 2
    #>

    # No Code
}
Get-Help ExampleFunctionDuplicateLink

Output:
RELATED LINKS
    http://google.com
    Line 2

Expected behavior
RELATED LINKS
    http://google.com
    Line 2

Actual behavior
RELATED LINKS
    http://google.com/
    Line 2
    http://google.com/
    Line 2

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.3.0
PSEdition                      Core
GitCommitId                    7.3.0
OS                             Microsoft Windows 10.0.19045
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

@Sekers - thank you for reporting this. Let me add @sdwheeler and we will investigate.

Adding @adityapatwardhan too


Cosign binary fixes

I keep having issues with the pre commit hook so I skipped over it for this commit. Apologies


Get-AuthenticodeSignature: Add embedded cert opt

PR Summary
Added the ability to retrieve an embedded authenticode signture of a file that ignores any certificates inside a .cat file. This is done through a new switch parameter -EmbeddedSignature on the Get-AuthenticodeSignature cmdlet.
This is a WIP until I find some time to add some tests. Edit: see https://github.com/PowerShell/PowerShell/pull/23821#issuecomment-2121733574 as to why I cannot add tests
PR Context
Fixes: https://github.com/PowerShell/PowerShell/issues/23820
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[x] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively - https://github.com/PowerShell/PowerShell/pull/23821#issuecomment-2121733574
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

I've tried adding tests but I cannot get Windows to see the newly installed .cat file generated in the test as valid. I followed this guide on adding the .cat file to the Windows caroot store and that was able to add the file successfully. I was able to verify that both the embedded signature of my test file and the .cat signature were valid and trusted according to Windows Explorer. Unfortunately it seems like the call to WinVerifyTrust that the catalog extension method calls fails with CERT_E_UNTRUSTEDROOT. Adding a trace for this call I can see it is called with the following data:
- Function: Wintrust.dll!WinVerifyTrust
  Time: 2024-05-21T13:18:30.6104744+10:00
  ThreadId: 14220
  Arguments:
    Hwdn: 0xFFFFFFFFFFFFFFFF - HWND
    ActionId:
      Raw: 0xAB341EF5C0 - PGUID
      Value: f750e6c3-38ee-11d1-85e5-00c04fc295ee
      ID: DRIVER_ACTION_VERIFY
    Data:
      Raw: 0xAB341EF0C0 - PWINTRUST_DATA
      CBStruct: 88
      PolicyCallbackData: 0xAB341EF170 - Pointer
      SIPClientData: 0x00000000 - Pointer
      UIChoice: 0x00000002 - WTD_UI_NONE
      RevocationChecks: 0x00000000 - WTD_REVOKE_NONE
      UnionChoice: 0x00000002 - WTD_CHOICE_CATALOG
      UnionData:
        Raw: 0xAB341EF120 - PWINTRUST_CATALOG_INFO
        CBStruct: 72
        CatalogVersion: 0
        CatalogFilePath:
          Raw: 0xAB341EF5D4 - Pointer
          Value: C:\Windows\system32\CatRoot\{F750E6C3-38EE-11D1-85E5-00C04FC295EE}\PowerShell-AuthenticodeTest-8472624b-d66e-43ef-bfe4-e81ce9095fd4.cat
        MemberTag:
          Raw: 0xAB341EFA70 - Pointer
          Value: '\\?\C:\Users\vagrant-domain\AppData\Local\Temp\script-with-cat.ps1'
        MemberFilePath:
          Raw: 0x00000000 - Pointer
          Value: null
        MemberFile: 0x00000000 - HANDLE
        CalculatedFileHash:
          Size: 20
          Raw: 0x296B7D41730 - Pointer
          Value: B24D16D1E0B673B610A8C641E8AAB74F9E357E86
        CatalogContext: 0x00000000 - PCCTL_CONTEXT
        CatAdmin: 0x00000000 - HCATADMIN
      StateAction: 0x00000001 - WTD_STATEACTION_VERIFY
      WVTStateData: 0x00000000 - HANDLE
      URLReference:
        Raw: 0x00000000 - Pointer
        Value: null
      ProvFlags: 0x00001080 - WTD_REVOCATION_CHECK_CHAIN_EXCLUDE_ROOT, WTD_CACHE_ONLY_URL_RETRIEVAL
      UIContext: 0x00000000 - WTD_UICONTEXT_EXECUTE
      SignatureSettings: 0x00000000 - PWINTRUST_SIGNATURE_SETTINGS
- Function: Wintrust.dll!WinVerifyTrust
  Time: 2024-05-21T13:18:30.6241246+10:00
  ThreadId: 14220
  Result: -2146762487
  Info:
    ErrorCode: CERT_E_UNTRUSTEDROOT

The cat filepath, member path, and hash are all correct, it just seems like I'm missing something where it's considering the root is untrusted. My guess is that only WHQL or EV signed files are trusted in the context of WINTRUST_CATALOG_INFO but that is just a guess.
Unfortunately unless someone knows more info as to why this fails and whether there is a workaround I'm not sure there is a good way to test this new functionality. Relying on builtin files will probably be brittle as they are updated/removed in the future so I don't think that's a good idea to add as a test here. I probably shouldn't spend much more time on this but happy to have another look if anyone has any recommendations.

We should add tests

Unless someone from the pwsh team can provide an embedded and catalog that we can add to the root as a test then I'm not sure if that's going to be possible. Trying to do this against a known executable that Windows provides would mean the test breaks anytime Windows changes either the embedded signature or resigns it with something else.

ntdll.dll in system32 should always be catalog signed.  That should provide a positive test for catalog signing and a negative test for embedded signatures.
You can always self-sign something for the positive test for embedded signatures.

ntdll.dll in system32 should always be catalog signed.  That should provide a positive test for catalog signing and a negative test for embedded signatures.   At minimum, we should have the negative test for embedded signatures for that scenario.

At least on my preview server 2025 build it has both a catalog and embedded signature

I can use this as a test but I'm not sure you would want CI to be reliant on this staying the same and whether that's just a side effect of using a preview build.
I can use cmd.exe which doesn't have an embedded signature but also who knows if this could change in the future.


server: fix race between container create and cadvisor asking for info

Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>
- What I did
same as #1616 but for master
- How I did it
- How to verify it
- Description for the changelog

/test all

LGTM

/test kata-containers

/test kata-containers

/test kata-containers

/test kata-containers

@rhatdan kata is broken at the moment, we were notified on IRC

Ignoring Kata Errors, until they fix it.  Merging anyways.


Use two separate timers for local and remote initiated procedures.
Signed-off-by: Thomas Ebert Hansen<EMAIL_ADDRESS>

Rebased


Mismatching branch name between picture and caption

What is the current behavior?
https://docs.github.com/en/github/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/committing-changes-to-a-pull-request-branch-created-from-a-fork (edited by maintainer to update the link)
The actual branch name in the picture is main, not test-branch.
What changes are you suggesting?

Should the picture or test be updated?

👋 @Un3qual Thanks for opening an issue! I think the text test-branch can be updated to match the image. You (or anyone else!) are welcome to open a PR to make that fix if you'd like 🎉

@janiceilene That was my original thought, but I feel like having the from repo have a different branch name than the to repo might help make it as clear as possible and having a separate feature branch for a PR seems like a best practice.

Ooh, that's a good callout @Un3qual I think you're right that it'd be better to replace the image! We'll need to search and check if the image is being used anywhere else before updating it.

Not sure how this issue slipped through the net for a content design plan - sorry @Un3qual 😞
I've checked and this image is used only in this article, so updating it will be fine. I'll update the issue summary with an image and a content design plan.

There was an issue while pushing from local. So, I made changes from website which, may, the reason of creating multiple PRs.


merged tooltip functionality removed?

Describe the bug
In version 2.7.8 of this component, I was able to create a merged tooltip of two values like this:
              <vue-slider
                v-model="pinLength"
                :min="1"
                :max="12"
                :piecewiseLabel="true"
                tooltip="always"
              >

                  Min-Max
                  Min
                  Max

              </vue-slider>

But in the latest version 3.0.7, it doesn't look like that functionality exists anymore. Is there another way to accomplish this?

Never mind.
I had to change it to this:
              <vue-slider
                v-model="pinLength"
                :min="1"
                :max="12"
                :marks="true"
                tooltip="always"
              >

                  Min-Max
                  Min
                  Max

              </vue-slider>

Yes 😿 , because the new version of tooltip is attached to the slider(dot) component, it's not easy to implement merge tooltip.
I'll consider adding it to the examples in the future.


Exception: SNlM0e value not found.

Following up on a post from nikk-16 on the same topic:

There is no issue int this API.
But after excessive usage, the Google knows that cookie is having unauthorized access so the 1PSID is revert in some form.
The reason we get : Exception: SNlM0e value not found. Double-check __Secure-1PSID value or pass it as token='xxxxx'
How can you get rid of it:
Clear cookies of bard.google.com and again

login into the account
F12 > Applications > Cookies > bard > __Secure-1PSID
Copy cookie and paste into your code.

Re run and you are good to go.

Originally posted by @nikk-16 in https://github.com/dsdanielpark/Bard-API/issues/99#issuecomment-1633060128
My problem is that the cookies seem to expire in 10 minutes and then I have to manually reset the browser (incognito on FireFox) while I am working on a notebook in Chrome. I am more than happy to change my coding environment if this helps, but closing/cleaning/restarting the browser is not very efficient - all suggestions are welcome.
Happy coding,
Gijs

@dsdanielpark
Looks like Google added captcha into Bard, If you don't send the cookie which solved by captcha manually, Google will response with status 429

@dsdanielpark - I know you are busy with another project, but is there a workaround for this problem?

@dsdanielpark , thank you for the explanation; I will try to keep Bard active on Edge and make it work. Unfortunately, I can't use the official PaLM2 API because it's the active internet link I would like to use for my topic, and the official PaLM2 API is static.


Small fixes for dynamic select

There was two small issues with select, dynamic overlaying was not specifying which objects it wants to be selecting on and secondly Datasets weren't validating selections applied to them.

Looks good and the tests are passing. Happy to merge if you say it is ready.

Ready to merge.


custom status(active color) name for toggle

🚀 Feature Proposal
A clear and concise description of what the feature is.
Motivation
Please outline the motivation for the proposal.
Example
Please provide an example for how this feature would be used.

There is no specific property to do that, but you may follow Branding guide to achieve this in a more appropriate way.


[pattgen,sival] Port pattgen_ios to SiVal

This port required a number of changes to the C code: the DV test uses random parameters and only runs once. In SiVal, it's more effective to run it a loop so the code was changed to be able to run several several times. Also more synchronization was necessary between the host and device to achieve proper signal acquisition. I also had to change the way interrupts to handle OTTF flow control.

Failure is due to the FPGA having a difference peripheral clock than the silicon. I updated the test so that the device communicate the frequency to the host.

After investigation, the CI failure is due to a glitch on the pda1_tx line that only seems to occur with the FPGA. Weirdly, the glitch is completely deterministic so I am investigating with a logic analyzer to confirm that this is an actual glitch on the line.

I have updated the code. I marked the FPGA test as broken due to the output glitch fixed by #23206. Since the fix is only for the master branch, I will remove the broken tag when backporting.

This will need to manually ported to master with some changes after #23198 lands