Bluetooth: ISO: Fixes and improvements

A couples more fixes for properly handling CIS request and enable bt connections to list ISO connections.

@Thalley @joerchan I wonder if you guys can review it as addressed your comments.

@Vudentz There's something in regards to the ref counter that doesn't quite work with this PR.
A test where the peripheral disconnect, not only causes the CIS conn not to reach ref counter 0, but makes the ACL conn underflow.
See this following log from a disconnect:
[00:00:01.230,000] <dbg> bt_conn.bt_conn_unref: handle 2 ref 2 -> 1
[00:00:01.230,000] <dbg> bt_iso.bt_iso_chan_send: chan 0x5659da10 len 1
[00:00:01.230,000] <dbg> bt_conn.bt_conn_send_cb: conn handle 2 buf len 5 cb (nil) user_data (nil)
[00:00:01.230,000] <dbg> bt_conn.bt_conn_process_tx: conn 0x5659d740
[00:00:01.230,000] <dbg> bt_conn.send_buf: conn 0x5659d740 buf 0x5659dc9c len 5
[00:00:01.230,000] <dbg> bt_conn.send_frag: conn 0x5659d740 buf 0x5659dc9c len 5 flags 0x02
[00:00:01.230,000] <dbg> bt_conn.bt_conn_prepare_events:
[00:00:01.230,000] <dbg> bt_conn.conn_prepare_events: Adding conn 0x5659d460 to poll list
[00:00:01.230,000] <dbg> bt_conn.conn_prepare_events: Adding conn 0x5659d740 to poll list
[00:00:01.290,000] <dbg> bt_conn.bt_conn_ref: handle 0 ref 3 -> 4
[00:00:01.290,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 4 -> 3
[00:00:02.240,000] <dbg> bt_iso.bt_iso_chan_send: chan 0x5659da10 len 2
[00:00:02.240,000] <dbg> bt_conn.bt_conn_send_cb: conn handle 2 buf len 6 cb (nil) user_data (nil)
[00:00:02.240,000] <dbg> bt_conn.bt_conn_process_tx: conn 0x5659d740
[00:00:02.240,000] <dbg> bt_conn.send_buf: conn 0x5659d740 buf 0x5659dc9c len 6
[00:00:02.240,000] <dbg> bt_conn.send_frag: conn 0x5659d740 buf 0x5659dc9c len 6 flags 0x02
[00:00:05.910,000] <dbg> bt_conn.bt_conn_ref: handle 0 ref 3 -> 4
[00:00:05.910,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 4 -> 3
[00:00:49.890,000] <dbg> bt_conn.bt_conn_ref: handle 0 ref 3 -> 4
[00:00:49.890,000] <dbg> bt_conn.bt_conn_set_state: connected -> disconnect-complete
[00:00:49.890,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 4 -> 3
[00:00:49.890,000] <dbg> bt_conn.bt_conn_ref: handle 0 ref 3 -> 4
[00:00:49.890,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 4 -> 3
[00:00:49.890,000] <dbg> bt_conn.bt_conn_ref: handle 2 ref 1 -> 2
[00:00:49.890,000] <dbg> bt_conn.bt_conn_set_state: connected -> disconnect-complete
[00:00:49.890,000] <dbg> bt_conn.bt_conn_unref: handle 2 ref 2 -> 1
[00:00:49.890,000] <dbg> bt_conn.bt_conn_prepare_events:
[00:00:49.890,000] <dbg> bt_conn.bt_conn_prepare_events:
[00:00:49.890,000] <dbg> bt_conn.bt_conn_ref: handle 0 ref 3 -> 4
[00:00:49.890,000] <dbg> bt_conn.bt_conn_set_state: disconnect-complete -> disconnected
[00:00:49.890,000] <dbg> bt_conn.tx_notify: conn 0x5659d460
[00:00:49.890,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 4 -> 3
[00:00:49.890,000] <dbg> bt_conn.bt_conn_prepare_events:
[00:00:49.890,000] <dbg> bt_conn.bt_conn_ref: handle 0 ref 3 -> 4
[00:00:49.890,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 4 -> 3
**[00:00:49.890,000] <dbg> bt_conn.bt_conn_ref: handle 2 ref 1 -> 2**
[00:00:49.890,000] <dbg> bt_conn.bt_conn_set_state: disconnect-complete -> disconnected
[00:00:49.890,000] <dbg> bt_conn.tx_notify: conn 0x5659d740
[00:00:49.890,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 2 -> 1
[00:00:49.890,000] <dbg> bt_conn.bt_conn_prepare_events:
[00:00:49.890,000] <dbg> bt_iso.bt_iso_chan_send: chan 0x5659da10 len 3
[00:00:49.890,000] <dbg> bt_conn.bt_conn_send_cb: conn handle 0 buf len 7 cb (nil) user_data (nil)
[00:00:49.890,000] <err> bt_conn: not connected!
Failed to send ISO data (-107)
[00:00:49.890,000] <dbg> bt_conn.deferred_work: conn 0x5659d460
[00:00:49.890,000] <dbg> bt_conn.bt_conn_ref: handle 0 ref 1 -> 2
[00:00:49.890,000] <dbg> bt_iso.bt_iso_disconnected: 0x5659d740
[00:00:49.890,000] <dbg> bt_iso.bt_iso_remove_data_path: 0x5659d740
[00:00:49.920,000] <dbg> bt_conn.bt_conn_prepare_events:
[00:00:49.950,000] <wrn> bt_hci_core: opcode 0x206f status 0x02
[00:00:49.950,000] <dbg> bt_conn.bt_conn_prepare_events:
[00:00:49.980,000] <wrn> bt_hci_core: opcode 0x206f status 0x02
[00:00:49.980,000] <dbg> bt_iso.bt_iso_chan_disconnected: 0x5659da10, reason 0x08
[00:00:49.980,000] <dbg> bt_iso.bt_iso_chan_set_state_debug: chan 0x5659da10 conn 0x5659d740 connected -> bound
[00:00:49.980,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 2 -> 1
[00:00:49.980,000] <dbg> bt_iso.bt_iso_chan_set_state_debug: chan 0x5659da10 conn (nil) bound -> disconnected
ISO Channel 0x5659da10 disconnected (reason 0x08)
[00:00:49.980,000] <dbg> bt_iso.bt_iso_cleanup: 0x5659d740
[00:00:49.980,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 3 -> 2
[00:00:49.980,000] <dbg> bt_conn.bt_conn_prepare_events:
[00:00:50.010,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 1 -> 0
Disconnected: FA:4C:4B:DB:D3:89 (public) (reason 0x08)
[00:00:50.010,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 2 -> 1
[00:00:50.010,000] <dbg> bt_conn.bt_conn_prepare_events:
[00:00:50.040,000] <dbg> bt_conn.bt_conn_prepare_events:
[00:00:50.070,000] <dbg> bt_conn.bt_conn_prepare_events:
Scanning successfully started
[00:00:50.100,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 1 -> 0
[00:00:50.100,000] <dbg> bt_conn.deferred_work: conn 0x5659d740
**[00:00:50.100,000] <dbg> bt_conn.bt_conn_unref: handle 0 ref 0 -><PHONE_NUMBER>**

I've tried to figure out where the issue is, but I can't pinpoint it.
The issue can easily be reproduced by running the peripheral_iso and central_iso samples, and simply disconnect the peripheral in some way.

See #33889 (comment)

Patch has been updated.


Roll Flutter Engine from 3aa112c75ccb to cb1ef6f08a28 (15 revisions)

https://github.com/flutter/engine/compare/3aa112c75ccb...cb1ef6f08a28
2023-02-22<EMAIL_ADDRESS>Roll Skia from 78bad646fe78 to 335cabcf8b99 (2 revisions) (flutter/engine#39795)
2023-02-22<EMAIL_ADDRESS>[Impeller] ensure glyph type contributes to FontGlyphPair hash/eq (flutter/engine#39794)
2023-02-22<EMAIL_ADDRESS>[Impeller] Add root dockspace to playgrounds (flutter/engine#39789)
2023-02-22<EMAIL_ADDRESS>Roll Skia from 22b594afb4d3 to 78bad646fe78 (24 revisions) (flutter/engine#39793)
2023-02-22<EMAIL_ADDRESS>Add more log when receiving semantics node not part of update (flutter/engine#39777)
2023-02-22<EMAIL_ADDRESS>[common] Use FML macro to prevent copy/assignment (flutter/engine#39786)
2023-02-22<EMAIL_ADDRESS>[web] Add dart:js_interop to BUILD.gn. (flutter/engine#39792)
2023-02-22<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from dj-2XkYHvZUsqlZTc... to cGa3bk2BmrHL_waB8... (flutter/engine#39791)
2023-02-22<EMAIL_ADDRESS>Roll Dart SDK from 5003b952affb to 6f5df63655ad (1 revision) (flutter/engine#39790)
2023-02-22<EMAIL_ADDRESS>Roll Dart SDK from c9a4fd7fd7fa to 5003b952affb (1 revision) (flutter/engine#39787)
2023-02-22<EMAIL_ADDRESS>Roll Dart SDK from 0db7573c6b70 to c9a4fd7fd7fa (1 revision) (flutter/engine#39783)
2023-02-22<EMAIL_ADDRESS>Updated the wide-gamut constant and added a unit test for it. (flutter/engine#39776)
2023-02-21<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from 25mb67G0L_sZ_OQCC... to dj-2XkYHvZUsqlZTc... (flutter/engine#39782)
2023-02-21<EMAIL_ADDRESS>[platform_view] fix overlapping platform view not touchable (flutter/engine#39527)
2023-02-21<EMAIL_ADDRESS>Started using bgr10_xr for opaque surfaces. (flutter/engine#39727)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from 25mb67G0L_sZ to cGa3bk2BmrHL
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


flyctl: 0.2.51 -> 0.2.52

Description of changes
https://github.com/superfly/flyctl/compare/v0.2.51...v0.2.52
This is a version bump to 0.2.52.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 310803 run on aarch64-darwin 1

1 package built:

flyctl


use CONFIG_DEBUG_FEATURES to cover param check

Summary
use CONFIG_DEBUG_FEATURES to cover param check
for the speed improve
Impact
sem
mqueue
signal
Testing
VELA

@GUIDINGLI
Value checking is not just a debug activity.
Why do you deem these to be unnecessary?

@davids5 @pkarashchenko
please review again

@GUIDINGLI Same concern perimeter checking is not a debug activity, it is API calling validation. Why remove it?

@davids5 @pkarashchenko
Others always do like this, why this PR can't pass ?
like:
https://github.com/apache/incubator-nuttx/blob/8be4bca3ebdd7f2b3e6f35bbdae649c2baef1bee/sched/semaphore/sem_clockwait.c#L106
https://github.com/apache/incubator-nuttx/blob/8be4bca3ebdd7f2b3e6f35bbdae649c2baef1bee/sched/mqueue/mq_sndinternal.c#L75
https://github.com/apache/incubator-nuttx/blob/8be4bca3ebdd7f2b3e6f35bbdae649c2baef1bee/sched/mqueue/mq_rcvinternal.c#L71
This also for the speed up.

We always need user open CONFIG_DEBUG_FEATURES, if he cares the returns.
Or it can speed up !

@pkarashchenko @davids5 If we want to run fast than threadx, the argument check in the key functions(e.g. sem_xxx) need to remove at least in the production code. It's also a practice in the current code to guard the argument check by CONFIG_DEBUG_FEATURES. What do you think?

@xiaoxiang781216 @GUIDINGLI I think we need to define what has more priority, standard compliance or speed. Till now I was (and I am) assuming that standard compliance if the top. If some speed optimizations are needed, then those should be selected via a separate config option as @davids5 says. I do not see any other option here

@davids5 @pkarashchenko
I used another better way to improve the speed, please review, thanks

@davids5 @pkarashchenko do you have.more concern with the new approach?


Fix automatic gas abstraction re-applying

This is a fix to the automatic infinite gas abstraction (--auto-abstract-gas). It was causing infinite loops where the abstracted node would again be abstracted to a new node, which would then be abstracted, and so on.
Thanks to @lucasmt for finding this.

@nwatson22 can we add a test of this? Something where we run a test with this option and check the output?

Ok, so does the gas abstraction require the same amount of communication back-and-forth with the backend? Or does it require more? If it requires more, is it possible that it might counteract the performance gains of not carrying a large expression in the gas cell?

@nwatson22 can we add a test of this? Something where we run a test with this option and check the output?

I added a test in test_foundry_prove.py that uses the option on GasTest.testInfiniteGas.

I guess the expected outpput needs to be updated.


Command Line Code Generator

Extracting the current reflection-based code generator to a working command line tool will be the preparation for a new roslyn based code generator. This serves as a proof of concept to be able to easily implement and exchange new code generators based on different technologies.

I think I'll give up on that, since I can't find a way to load the dll. Doesn't seem to work
https://github.com/sschmid/Entitas-CSharp/blob/feature/%23158-command-line-code-generator/CodeGenerator/Program.cs#L22-L25

2nd take...
also add commands
entitas init // create a Entitas.properties file
entitas diff // list of code generators not used in Entitas.properties
entitas generate // generates source files based on Entitas.properties

Reached the first milestone! Entitas command line code generator paves the way for simple customization and roslyn integration

You're ready to generate.

Will you include the Entitas CodeGen CLI in future releases with the source in addons or remove them altogether and just include this in the release (possible unity menu to call it perhaps)

The sources for the Addons and all of Entitas will stay open source, but I'm working towards deploying dll in the zip files instead of sources.

So will it look something like this:
Addons/CodeGenerator/.dlls and exe's
Addons/CodeGenerator/Source/.zips
Or will it be source first with zipped exe/dlls?

aiming for dll-only


Add Audit export API (GEA-13176)

Adding the polling to the integration test and updated the description for request_id.


Update web-ui-dashboard.md: Adopted port-forward from official helm chart

Description
The suggested link kubectl proxy does not work anymore. Also the new helm chart's output of helm upgrade ... suggests the usage to kubectl port-forward.
Issue
Closes: #46475

/sig ui
/assign @floreks @maciaszczykm @shu-mutou

/remove-lifecycle stale


[DOC] Building from source code link is missing

Report incorrect documentation
The link provided in the readme directs to this address, which yields 404 not found.
https://github.com/rapidsai/cuvs/blob/branch-24.08/docs/source/build.md
Location of incorrect documentation
Below https://github.com/rapidsai/cuvs?tab=readme-ov-file#nightlies on "Please see the Build and Install Guide"
Describe the problems or issues found in the documentation
This should direct to the proper page on how to build from source.

Thank you for bringing this to our attention @T-Lind we will get this fixed.

There's also a few other links that are incorrect on the README. I'd go through each one and double-check.


Update to cynic 2.0

I'm the author of cynic, the graphql client library this project is using.  I just released v2 of cynic, so thought I'd raise a PR that updates triagebot to use it.
I've put some effort into improving compile times for large schemas in this release.  In my (not very scientific) tests the github-graphql crate in this repo compiles in about half the time after these changes.  Frankly it's still not as fast as I'd like but the github schema is a real challenge.
I also simplified a few of your queries & the associated handling code, hope that's ok.  Happy to revert that stuff if not though.

hey @obmarg great to see you around :) I love cynic, thanks for this patch!
At a cursory look it looks great. I'd leave it sink for a a bit so I can try it this week for the T-compiler meeting.
(but tbh I'm also fine to straight merge this if the triagebot maintainers say so)

@obmarg Hi! Thanks for this patch, this is great. It looks like we've edited the graphql we use a little since you made it (sorry!), but I'd be happy to merge a rebased version of it.

No worries @Mark-Simulacrum - have rebased & updated the new query


feat: find the directory where the executables are actually installed using package manager-independent methods

Might be better to use chokidar instead of fs.watch() function


[TE9][TC-SC-5.1] Lock-app is failing on step 2 (./chip-tool groupkeymanagement key-set-write)

In [TC-SC-5.1] Step 2,
./chip-tool groupkeymanagement key-set-write '{"groupKeySetID": 42,
"groupKeySecurityPolicy": 0, "epochKey0":
"d0d1d2d3d4d5d6d7d8d9dadbdcdddedf", "epochStartTime0": 2220000,"epochKey1":
"d1d1d2d3d4d5d6d7d8d9dadbdcdddedf", "epochStartTime1": 2220001,"epochKey2":
"d2d1d2d3d4d5d6d7d8d9dadbdcdddedf", "epochStartTime2": 2220002 }' 1 0
The result shows UNSUPPORTED_CLUSTER for lock-app.
Checking examples/lock-app/lock-common/lock-app.zap, there is no "Group Key Management" cluster as other applications.
If the test is mandatory for all type devices, lock-app should support the cluster.

@epeng100 is there a reason why you need to run this TC on lock app?

No, we tested it because the case is in TEDS list for lock app.

This cluster is mandatory for all devices.
That said, devices are allowed to support only one keyset per fabric (for the IPK) if they don't implement the Groups cluster.  And locks are absolutely allowed (and in fact strongly encouraged not to) implement the Groups cluster.  So doing that keyset write is absolutely allowed to fail for a lock, just not with UNSUPPORTED_CLUSTER.

Spec Issue Review: Sankar to confirm that this test case is indeed running and passing.

@woody-apple  This test is not passing in QA. This test has been linked to the testplan issue  1453
Test plan requires an update as well.

does that mean we should enable the groups-cluster in the lock-app and configure it properly?

Group key management, not groups.

Yep, sorry, I meant exactly that, thanks


Using container images in Charts, that are available on DockerHub

The current configuration of the Charts still points to container images build in the catenax-ng organization instead of DockerHub
See image.name for example
It also seems like neither frontend nor backend images are properly tagged on DockerHub
See:

https://hub.docker.com/r/tractusx/vas-country-risk/tags
https://hub.docker.com/r/tractusx/vas-country-risk-backend/tags

only :latest and :main tag present
this is blocking #46

Fixed also on PR https://github.com/eclipse-tractusx/vas-country-risk/pull/53
for version I suggest since I didn't do any release or version recently to docker hub , once the PR are closed we can do a release v.1.2.1 for the repos. charts are all ready according with that version , only changelog does not have the release date setup

I already did the release for backend. Frontend will follow, as soon as the fixes are all done.


How can I retry the scenario or step definition if the test was failed

There is the test retries configuration on cypress.json. But in some special situation, I don't want to retry all the tests, only some tests need to be retried.
Can we add the retry on a specific scenario?

Upvoted, I also need a way to add configurations per test scenario

Cypress does provide native retry: https://docs.cypress.io/guides/guides/test-retries#Custom-Configurations
And this lovely plugin does accept a test config block...
Given(   "the dashboard is sorted by column {string} in {string} order",    {     retries: {       runMode: 3,       openMode: 3,     },   },   (column, order) => {   ...   ...
But it's not working.  Tracing the generated test (as seen in the test runner):
`
(0, _steps.Given)("the dashboard is sorted by column {string} in {string} order", {
retries: {
runMode: 3,
openMode: 3
}
}, function (column, order) {
`
Looks OK but no dice.  If I do the same in a spec directly (no cucumber) the retries override does work as expected.
Have not been able to figure out why it's not working under this plugin.

As I dive into the source code, the generated code looks like this
  `describe(${filename}, ()=> {...})`

which will fall to the no-config case of cypress.
Actually I can help a pull request for this, but not sure where to put the configs in the feature file

😢
Is there a way to do this from within Cypress steps, as in configure this ad-hoc outside of the actual definition?

Retries doesn't work correctly. If I use for ex. retries:2 in cypress.json, and two test scenarios, when the first scenario fails it retries only the second scenario 4 times.

Did you get the solution to this problem?

Have you tried: https://docs.cypress.io/guides/guides/test-retries#Configure-retry-attempts-for-all-modes ?

Due to personal reasons, the previous maintainers of this package are stepping down and handing the reigns over to me, a long-time contributor to the project and a user of it myself. This is a responsibility I'm very excited about. Furthermore, I'd like to thank @lgandecki ++ for all the work that they've done so far.
Read more about the transfer of ownership here.
The repository has however moved and all outstanding issues are being closed. This is not a reflection of the perceived importance of your reported issue. However, if after upgrading to the new version, you still find there to be an issue, feel free to open up another ticket or comment below. Please make sure to read CONTRIBUTING.md before doing so.

Any updates on this?

Cypress offer no mechanism to retry a subset of a test, which means retrying a step definition isn't possible. The preprocessor is after all constrained / limited to do what Cypress can offer.
As for retrying specific scenarios, I'm open to suggestions as to how the interface could look like. How would one specify a single scenario to be retried N times?

Retries doesn't work correctly. If I use for ex. retries:2 in cypress.json, and two test scenarios, when the first scenario fails it retries only the second scenario 4 times.

Before the update this was working for me. I was able to set the retries in the cypress.json file and each scenario would retry accordingly if any step failed. This is currently not working and I am getting the same behavior mentioned above the step will actually just continue if the first one fails and I believe the last one is being retried.
As for retrying specific scenarios can we utilize the Smart Tagging feature and have a @retries:N or better yet an options  tag that takes a key=value list something like @options(retries=3, focus=true, tag=mobile) giving a clean way to provide a set of key=value smart tagging options as well as the tag key for filtering tests.

@ngocthuan1208 @phi0411141 You can probably try cytorus to retry scenario.

Before the update this was working for me. I was able to set the retries in the cypress.json file and each scenario would retry accordingly if any step failed. This is currently not working and I am getting the same behavior mentioned above. The step will actually just continue if the first one fails and I believe the last one is being retried N times.

I recommend opening a separate issue about this and follow the issue template. I can tell you immediately that I need to have a reproducible example that I can clone.

I've create an issue specifically related to the proposed update here and am closing this.


invoking code inside bash WSL results an extra cli.js

VSCode Version: 1.7.1
OS Version: Windows 10 14965

Steps to Reproduce:
Im Running code command inside bash without trouble...
But it always open extra cli.js everytime i open it On Windows 10 Bash.
I dont exprience anything Funky when i run code command in CMD terminal...
may i know how to fix this?

Duplicate https://github.com/Microsoft/vscode/issues/13138


vietnamese

Big thanks for creating Nutshell. I really ❤ love your projects!
Please check out my fan message: https://youtu.be/LLFhKaqnWwk

Sorry for the very late pull, thank you for translating! I'll check out your fan messa--
ಠ_ಠ
(but actually lol. blast from the past!)


tweak browse test descriptions

Actually, the tests were about the browse functionality, and they are currently just showing the datatable is loaded.
When the datatable is loaded, it contains data, not profiles.

Yep, my mistake. Thanks!


test: throw on deprecation

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #8   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          43     43
  Branches       12     12
=====================================
  Hits           43     43

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 4c70524...ef72a30. Read the comment docs.


Use a logger interface instead of the log package

Fixes #1250, also cleans up a bit of the back-off logic.

This broke some of the tests, see this for example:
https://travis-ci.org/google/certificate-transparency/jobs/172483328

BTW, there's a re-organization of the client code, so probably best to wait until that's in.  Also, it might be nice to separate distinct chunks of function (add logger, change back-off) into separate commits or pull requests.

I've backed out the back-off specific changes as they ended up being somewhat larger than I expected.

BTW, there's a re-organization of the client code, so probably best to wait until that's in. Also, it might be nice to separate distinct chunks of function (add logger, change back-off) into separate commits or pull requests.

Do you have a ETA for landing these changes? This PR is part of a set of changes the Let's Encrypt dev team would like to implement based on findings from our investigation of the Aviator log outage the week before last. If you expect this will be done relatively soon I'm happy to put this PR and the few other changes we have waiting on ice until the refactor is done.

The client re-jigging changes are now in; shout if they cause problems.  And apologies for forcing you to fix conflicts!

@daviddrysdale thanks for the heads-up! I'm 99% sure I resolved the conflicts cleanly...

@phad looks like you were looking at a older commit, as of 5928fc6 all back-off related changes were removed. I've gone through and fixed up the remaining issues except for the New change. Since there are already two different constructors, New and NewWithoutVerification, I'd rather not add another as it'd really require adding two more constructors to properly cover all cases.
How would you feel about adding a SetLogger(l Logger) method to jsonclient.JSONClient and client.LogClient instead and by default using the basicLogger wrapper for the log package?

@rolandshoemaker you're right, I looked at the first commit in the series and not the others.  Sorry!
@daviddrysdale and I were discussing the constructor situation, and agree that we wouldn't want another.
Another approach would be to have a single constructor that takes an options struct, into which the things required for verification and for logging are supplied.
Would you be willing to do that?  thanks!

Added the Options object solution which I agree is much cleaner.

One last comment: given that we're making an API change, is it worth folding the *http.Client into the Options too, so all of the optional parameters are together?

@daviddrysdale personally, I wouldn't do that.  I don't consider the *http.Client to be optional in the same way as those other items.  But the JSONClient code does create an http.Client if nil supplied...
Since we're in the realm of API breaking changes I'd sooner make the hc *http.Client param required.
Happy to hear other views though.

OK sounds like this change is good to go as-is, and we can separately consider making the http.Client non-optional

Yeah I think that's a good approach.


Doesnt work with fitbit Sense

I followed All the steps you described, But it still doesnt work in OBS.
the Connection from the Watch to the all Seems Fine and i added the right Browser source to obs.
But it still doesnt work.

When you open the /ui/index.html directly in a web browser, does the heart animation show and is the heart rate something greater than 0?

Yeah That World fine.

That works*

Could you share a screenshot of how you have the browser source configured in OBS?


Sunstone to allow folder upload

Author Name: Tino Vázquez (Tino Vázquez)
Original Redmine Issue: 1725, https://dev.opennebula.org/issues/1725
Original Date: 2013-01-11

Useful to register VMDK disks.

Original Redmine Comment
Author Name: Tino Vázquez (Tino Vázquez)
Original Date: 2013-04-11T11:08:58Z

Apparently, you can upload zip files. Documentation needs updating


Add configuration of local path

Would you please add a configuration of local path ? since I can only use the root path of work space as the local path, which can't meet my needs. If i open the sub folder as my work space, then i can't use the git features.
It is a very useful ftp-sync extension in vs code (compare to any other ftp extensions), nice work!
Expected Behavior
i can config the local path, not only the root path of my work space.  (sub folder of my project).
Actual Behavior
i can't config local path that sync to remote, it just uses the root path of my work space.
Steps to Reproduce the Problem

config SFTP
fill in address, port, remote path and so on...

Requisites (important to diagnose the problem!)
Extension Logs
Specifications

Version: 0.8.11
Platform: Mac OS 10.13.1 (VS code 1.19)

Working on it! Now you can use multi-root workspace as a workaround.

The sftp 0.9.0 now supports it.


Prompt user to provide a new port if 4000 is already in use

Issue #22
Done like in create-react-app

I can merge this after lint issues being fixed 😅

Done!


BufferManager: nvim_win_set_cursor: Cursor position outside buffer

Using the latest 0.0.73 I got this error when I came back to an unsaved file in normal mode. The cursor was frozen, then after moving around some other files and returning then it worked again.
2021-01-01T20:38:46.607Z BufferManager: nvim_win_set_cursor: Cursor position outside buffer

Not sure how to reproduce, but I thought I would post it as an issue in case anyone else got it.

I also get this happening, but I haven't been able to reduce this to a reliably reproducible situation, as it seems kind of random.
It seems that most of the time this happens when changing focus (with a mouse click) from an open editor to another one or the explorer, etc. is what triggers this, but it's inconsistent. Sometimes I can go 4 hours without it ever happening, sometimes it happens every 20 minutes. I've also had it happen when opening vscode and trying to immediately type in one of the restored/reopened editors from the last session.
I'm currently using version 0.0.82 and have experienced it on both Windows 10 and Ubuntu 20.10 and 21.04.

The first time it happened was when I did View -> Editor layout -> Split in Group and then it followed every split/unsplit action until I restarted VSCode and then it has gone. Can't reproduce.
VSCode Neovim <IP_ADDRESS>
------------------------
NVIM v0.6.1
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by brew@BigSur

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/Cellar/neovim/0.6.1/share/nvim"
-------------------------
MacOS 11.6

I am having the same issue all the time when I move between tabs.

I am having the same issue all the time when I move between tabs.

I have the issue in this same situation too.

I have same issue
NVIM v0.5.0
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: /usr/bin/gcc-11 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNVIM_TS_HAS_SET_MATCH_LIMIT -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/home/runner/work/neovim/neovim/build/config -I/home/runner/work/neovim/neovim/src -I/home/runner/work/neovim/neovim/.deps/usr/include -I/usr/include -I/home/runner/work/neovim/neovim/build/src/nvim/auto -I/home/runner/work/neovim/neovim/build/include
Compiled by runner@fv-az242-526

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/home/runner/work/neovim/neovim/build/nvim.AppDir/usr/share/nvim"

Run :checkhealth for more info

I have the same issue, it happens frequently, and I haven't noticed any usage pattern that triggers it:

When working with Jupyter in VSCode, and when I scroll around a notebook, this error occurs.

When working with Jupyter in VSCode, and when I scroll around a notebook, this error occurs. EDIT: not stably reproducing, either.

Same here

When I remove my $HOME/.local/share/nvim folder, it solved.
I think runtime dir make crash. I suggest replace to custom runtime path using rtp.

I get this sometimes too, but I don't know how to fix without reproduction steps.

For reproduction, I got this pretty consistently initially because I had a keybinding collision using ctrl+n/p for navigation:
  {
    "command": "-vscode-neovim.send",
    "key": "ctrl+p",
    // "when": "neovim.mode != cmdline_normal"
  },
  {
    "key": "ctrl+p",
    "command": "workbench.action.previousEditorInGroup",
    // "when": "neovim.mode != cmdline_normal"
  },
  /* solution for my workflow was adding the comments above and this block.
  {
    "command": "-vscode-neovim.send",
    "key": "ctrl+n",
    "when": "neovim.mode != cmdline_normal"
  },
  {
    "key": "ctrl+n",
    "command": "workbench.action.nextEditorInGroup",
    "when": "neovim.mode != cmdline_normal"
  },
 */

I'm getting the same error when opening Jupyter Notebooks using the Jupyter Extension

I get this error a lot when editing settings.json with "workbench.settings.useSplitJSON": true,

Having lots of these warnings with the Jupyter Notebooks in VsCode :(

Can reproduce when the output panel is activated, but it is uncertain if there are any differences in the discussions under this issue.

Hopefully this will be fixed in the latest release.

@theol0403 I still encounter this issue since latest update. Here's how I reproduce it.

Make a global mark in current file. Press mA in normal mode.
Switch to another file and go back to mark A by pressing 'A.
Then you can catch the issue.

I don't know whether it's the same issue as #1223 ?

obviously not

@theol0403 I still encounter this issue since latest update. Here's how I reproduce it.

Make a global mark in current file. Press mA in normal mode.
Switch to another file and go back to mark A by pressing 'A.
Then you can catch the issue.

I don't know whether it's the same issue as #1223 ?

The error message is the same, but the reason is different.

@xiyaowong I got it. Since mark is really useful, I hope this bug could be fixed either. Thanks a lot. 😀

I had this error until I assigned an 'affinity' to VSCode Neovim. I did this by clicking 'yes' in a warning message.
On v0.6.1 on macOS 13.6, after I reload the window, a new temporary file (eg. Untitled-3) pops up which has the contents of whichever file I was last on. In the temporary file I can use neo-vim shortcuts. Once I close the temporary file and go back to the original file I get 3 error messages and the warning about the 'affinity':
1. `BufferManager nvim_set_current_win: Invalid window id: 1012`

2. `CursorManager: nvim_win_set_cursor: Invalid window id: 1012`

3. `CursorManager: nvim_win_set_cursor: Invalid window id: 1012`

4. `Do you want to assign an Affinity to VSCode Neovim?`  -> Click yes on this message, may have to list all messages to see it.

Same error when i switch tab using

Any updates on this? This is very annoying and forces the user to quit VSC and open again.

Any updates on this?

Any logs?

This is very annoying and forces the user to quit VSC and open again.

You can use vscode-neovim.restart

@xiyaowong Sure, what kind of logs do you need? I can share them here.

Same issue, had to disable plugin.

Same issue here. Very annoying. For a brief moment I don't realize it and keep writing and it does all kinds of unexpected things that for some reason u can't undo.

@pbassut You can open a new issue and provide reproduction steps or a screen recording.

same issue, the best way to recover it is switch to another tab and back current tab


Some odd-looking symbols in script editor

Godot 3.0 master
Minor in general,  noticed a few characters look a bit odd with the new Hack font / icons:

My main concern is indentation marks look out of place. I liked to code with them as they were fine, discrete enough and improved readability for me and made indents visible (they matter :p), but now I find them too noticeable (alternating shape, size and different thickness). I preferred the old ones, is it coming from the font though?
Minor: already noted earlier, but the zeroes look weird. I understand it's to differentiate with O and I like this, but filling the whole oval isn't common.
Minor: more personal note, but I got used to wider angled brackets. Hack ones are more tight compared to Consolas, for example.

New:

Old:

(other than that the font looks fine so far)
Edit: Github shows no images anywhere on my end, I hope you can see them...?
Edit 2: At the moment I'll most likely try another font myself and stick with it. I'm trying to test a TTF but I see no way of setting it up easily in Godot...

Indentation marks are icons.
https://github.com/godotengine/godot/blob/master/scene/gui/text_edit.cpp#L1078

I also noted folling issues with them beeing icons:

Scaling text does jot scale them (meta + scroll)
color adaption for white theme does not work
color cant be set when usojg coustom thext color theme. So they can look even more out of place

Isnt there a really good font icons that could be used?

@Zylann i think the two minor ones are very personal.
I like the braces. But since there is still the option for a coustom font im fine with any default. (but i think it is not worth to start another discussion about default code font. There was one recently,if im not mistaking, so the overall opinion seems to what this as the default.

@toger5 , using a character might be a really good idea, i think it should be this one ⇥, it never cossed my mind, also yeah its clear that the icon wont scale and thats a bad thing.

I really like these | indentation marks used in Notepad++ or VSCode.

Problem here is that using a 'line'-character could lead to little spaces in between:
|
|
|

@toger5 I managed to get my favourite font setup so I don't really mind anymore about the two minor points :)
So yeah... I think we could find more discrete and regular indentation marks, like the old Godot ones, or the verticals from Sublime (but in ST verticals are procedural, they aren't icons or even characters)

I like both proposals:
The character @djrm posted (i will tri it fith the current code font in godot)
And the line (@pwab)
Maybe it even would be worth to implement the line drawing? Not sure how to do it... So i cant really tell how much work it is.

Not a fan of the ⇥, because it stands out too much for something that is shown all the time (unless the font happens to have a discrete indicator by pure luck...). I know it really is supposed to represent a tab, but I never see it used as indentation guide, but more likely shown to stand out in diff tools or "show non-printed characters" modes.
In ST it is shown only on selection (with spaces too):

just another idea. Could we adda custom character to the current default font so that it looks like it does right not but it is not an icon?

Best solution by far is the vertical line (imo)
but that is also most work.

Vertical lines for indentation and arrows for all other tabs (to highlight excessive tabs at the end of line).

What's wrong about the icon though? You could also draw it with a scaling rectangle based on line height, or draw a line instead (or have a line in the icon heheh), which isn't much work, just different drawing.

Or just draw an appropritaly scaled line instead of the icon...
Maybe the line approach is easier to inpmement than i thought.

There is:
⎸U+23B8 LEFT VERTICAL BOX LINE
│U+2502 BOX DRAWINGS LIGHT VERTICAL

@bojidar-bg character-based lines leave spaces in between rows


EventListener is limiting

I've noticed when working on a project that EventListener can be limiting in regard to the fact that we don't expose the raw *js.Object event OR similiar functionality.
For example, in one scenario I wanted to call preventDefault on the event object within the event handler, i.e. dependent upon a condition within my event handler. Particularly, if a specific key was pressed I would call preventDefault but if it wasn't a specific key then I would retain the default behavior by not calling preventDefault. Right now, however, it is only possible to call PreventDefault as a blanket-statement for the entire event handler -- not conditionally -- because we don't expose the raw event nor such a method to do it conditionally (it can only be done on the EventHandler object, which the callback has no access to).

Seconded... Exposing the raw event object would make things more flexible... Exposing these as methods on the handler is neat, but it always struck me as a bit weird. In fact, the first time I needed to use preventDefault, I spent quite a while scratching my head and referring to examples before I worked it out. It's a lot more familiar to call it within the event handler, and not being able to call it conditionally is an important omission.
p.s. We recently fixed this to expose stopPropagation in the same way as preventDefault.

The initial idea for that preventDefault implementation was to be able to automatically call the event listener in a new goroutine to allow blocking code by adding a go statement here: https://github.com/gopherjs/vecty/blob/master/dom.go#L97. This may not be the best solution.
Keep in mind that React still sees a reason for adding an abstraction on the native event: https://facebook.github.io/react/docs/events.html This might be especially necessary when we add event pooling for performance.

From @davelondon

In order to do a drag+drop file upload, you need access to dataTransfer from the js event. The js native event object isn't currently presented to the vecty event handler. I've added this in my fork as an anonymous struct member, and it seems to feel good in use:
https://github.com/davelondon/uploader/blob/master/editor.go#L133
I seem to remember this was discussed before?

From @shurcooL

need access to dataTransfer from the js event. The js native event object isn't currently presented to the vecty event handler.

Just asking, but have you considered simply filling that gap and exposing dataTransfer natively?

From @slimsag

I think this is a duplicate or related to https://github.com/gopherjs/vecty/issues/55

I agree it is limiting, and the best choice for now is probably to expose the native event object. Although in #55 neelance does make some good points about event pooling in the future.. I'm not sure yet.

@shurcooL there are so many DOM event properties I think it would be a bit crazy to add them all :P

From React's event documentation it seems like the main reason they use synthetic events is to standardize how events act across browsers. That actually feels a bit strange to do IMO, but I do not know all the details I suppose.
They too, also expose the native event:

If you find that you need the underlying browser event for some reason, simply use the nativeEvent attribute to get it.

And event pooling appears to just be for purposes of not creating the same JS object over and over again (something that could be achieved by simply using the event object the browser gives us).
I think they might do more behind the scenes to improve event performance, but I think for now it is OK for us to expose native event objects.


RLException: [marm.launch] is neither a launch file in package [mobot_urdf] nor is [mobot_urdf] a launch file name

执行：
$ roslaunch mobot_urdf marm.launch

有如下报错：
RLException: [marm.launch] is neither a launch file in package [mobot_urdf] nor is [mobot_urdf] a launch file name
The traceback for the exception was written to the log file

发生此错误的原因可能是在创建ROS工作空间时没有将devel/setup.bash写入~/.bashrc，导致后期找不到marm.launch文件，解决办法如下：
$ cd ~/catkin_ws/
$ source devel/setup.bash

类似问题大多与环境变量有关，请参考第一期文档中“配置PX4&ROS环境”部分。


[Helm] Prometheus scrape annotations on Service

Describe the bug
When Prometheus scraping is enabled from the values, the Prometheus scrape annotations are applied on the service. This would mean that the metrics are scraped from only one pod at any given time, and the request round-robins every time the /metrics endpoint is scraped.
Steps To Reproduce
Enable prometheus and scrape the endpoint
Expected behavior
Every pod is scraped for metrics

Fixed in https://github.com/helm/charts/pull/11651 which is being merged to master - this issue can be closed.

Usually prometheus is setup to find the endpoints behind the service and scrape all of them.
I have a "kubernetes-service-endpoints" job which does this.

See https://github.com/helm/charts/compare/master...davidkarlsen:fixKuberHealthyPorts?expand=1 which fixes this.
The two containers now show up with healthy state in my /prometheus/targets now.

This issue can be closed

Thanks @davidkarlsen!  @logicfox your change has been merged into the helm chart, going to mark this issue as closed, thanks for the contribution!


XML Column result click formatting

Azure Data Studio Version: 1.0.0

Steps to Reproduce:

Select data from a table that contains an xml column
Click on the data in the results grid for the xml column
The xml for the column is displayed in a new tab  but is no longer formatted.

In previous versions of SQ Operrations studio the xml was formatted.

In addition to the "unformatted" XML, it is also "unformattable" - pressing the Shift+Alt+F key combination in the new tab displays the message:

Since there is no method do this - I heartily concur with this. I had to switch back to SSMS in order to do this, uber sigh!

I'll take a look at why this regressed.  It happened the last time we merged VS Code source code.

The next insiders build will contain a fix for this issue.


No doc back when eating besybuy url

get nothing back from eat when eating this url  http://www.bestbuy.com/site/Intel+-+SSDSA2CW120G3+120+GB+Internal+Solid+State+Drive+-+1+x+Retail+Pack/3768583.p?id=1218431566927&skuId=3768583&st=solid state drive&cp=1&lp=7

just had to send user-agent
fixed in eat 0.1.5
https://github.com/seamusabshere/eat/commit/0bef33386aad72f00d19bdc0e60273bffc86bce4

did a bundle install and upped the version but getting this error when starting rails
rails s
/Users/admin/.rvm/gems/ruby-1.9.2-p290/gems/eat-0.1.5/lib/eat.rb:11:in `module:ObjectExtensions': uninitialized constant Eat::VERSION (NameError)
from /Users/admin/.rvm/gems/ruby-1.9.2-p2

From: Seamus Abshere<EMAIL_ADDRESS>To: snovell<EMAIL_ADDRESS>Sent: Tuesday, December 6, 2011 5:56 PM
Subject: Re: [eat] No doc back when eating besybuy url (#3)
just had to send user-agent
fixed in eat 0.1.5
https://github.com/seamusabshere/eat/commit/0bef33386aad72f00d19bdc0e60273bffc86bce4

Reply to this email directly or view it on GitHub:
https://github.com/seamusabshere/eat/issues/3#issuecomment-3042125

my bad, fixed in eat 0.1.6


FancyZones: add approved apps list and unify window filtering

Summary of the Pull Request
Add a "approved apps" settings, similar to the "excluded apps" one. If an app is on this list, more of the apps window will be zonable.
Unifies the code that decides if a window is zonable. Before this, WinKey + Arrows used different code than dragging the window.
PR Checklist

[x] Applies to #894 (SUMo and XYplorer) #1104
[x] CLA signed.

Validation Steps Performed
Manual.

The "unify window filtering" part as been merged in a separate PR https://github.com/microsoft/PowerToys/pull/1184
The "approved apps list" part still needs some work before been ready to be merged.
Closing this PR for now.


Don't even spawn a job if the parent has not completed

Right now even though a job always waits for its parent to complete before it runs, the job is nevertheless spawned and scheduled; it just blocks until its parent completes.  This wastes cluster resources.  This PR makes it so that a job is not even spawned if its parent has not completed.

To fix CI just merge in master.

Actually I'll do it for you :P so CI can start running immediately.


Forgot brazil region : ^)

After looking at my production logs I've noticed that i forgot to add brazil region to the thirds preset.

smh poor brazil

wait did you lint this

yes : ^)


Workaround for upgrading from 3.2 to 3.4

+1, made small grammar/punctation changes

+1, assuming the info came from dev and doesn't need another dev review? @jasonblais please resolve the conflicts then we can merge

And yes, the update came from a developer and the steps resolved at least one 3.2 -> 3.4 upgrade: https://forum.mattermost.org/t/cant-login-after-upgrade-from-3-2-0-to-3-4-0/2240/2


SpringRestPactRunner doesn't appear to call destroyMethod on Spring beans

I have a Pact test using SpringRestPactRunner that passes when I run it standalone.
Elsewhere in my app I have a @Bean annotation with a destroyMethod parameter to control my HazelcastInstance.
    @Bean(destroyMethod= "shutdown")
    public HazelcastInstance hazelcastInstance() {
        LOGGER.info("Starting Hazelcast");
        return Hazelcast.newHazelcastInstance(config());
    }

In my other functional tests I can see in the logs that Hazelcast starts and stops correctly.
However, in my pact test Hazelcast is never shut down - this causes my Maven build to fail as the next test tries to start Hazelcast and the application fails with an exception at that point as it's already running.
My thinking is that when I use the SpringRestPactRunner it's not calling the destroyMethod from the @Bean annotation.

Hi.
That test already has an @After method.

Hi, is there anything else I can try to get this resolved please?

By default the application context is destroyed when the test JVM exits, but you can now use the @DirtiesContext Spring test annotation to change that to be after the test class or after each test method.

3.5.19 has been released with this fix

Thanks, this has fixed the issue.


IPv6 cluster

I cannot setup a IPv6 cluster, because the _cluster_setup action add_node doesn't accept IPv6.
curl -H 'Content-type: application/json' "admin:password@host1.vlan0:5984/_cluster_setup" -d '{"action": "add_node", "host":"host2.vlan0", "port": 5984, "username": "admin", "password":"password"}' | jq
{
  "error": "invalid_ejson",
  "reason": "{conn_failed,{error,nxdomain}}",
  "ref": 878456848
}

host1.vlan0 resolves (AAAA) to an address similar to abcd:1234:abcd:1234:abcd:123:abcd:1
host2.vlan0 resolves (AAAA) to an address similar to abcd:1234:abcd:1234:abcd:123:abcd:2

When trying to set the "host" to the IPv6 address, I get:
{
  "error": "invalid_ejson",
  "reason": "{url_parsing_failed,{error,invalid_uri}}",
  "ref": 878456848
}

I also try to include the IPv6 between [ ] (thinking maybe couchdb tries to setup a URL like http://{ host }:{ port }) ... but it fails with the same error as if I used an hostname (error, nxdomain). (I wonder if there's a regex figuring out if the thing is an IP that only matches IPv4?)
Note: another hiccup was that apache.jfrog.io does not resolve in IPv6 (solved by downloading and copying the deb file manually... meh). But that's a different issue.

Tried the same yesterday and got "invalid URL" it's a pity 🥺

The latter might be related to #5223, it looks like there's some URL string manipulation bugs.


any way to ignore  the downloads.db when downloading...

I'm not concerned with downloading duplicates... and actually, expecting it occasionally... just curious if its possible to ignore the downloads
i.e. "putio files download -force" or something...

Working on something now...


Training Data Missing

Will there be data provided to run the code?

Yes, sorry for the inconvenience. I will write a detail README file as soon as possible. The dataset used in our paper can be downloaded from https://github.com/fbadine/LSTNet and https://github.com/liyaguang/DCRNN. Thanks for your interest.


Fix release version checker

After the big breaking change from 0.4.x to 0.5.x we should have to change the release script as well. Since I didn't change the latest release failed to be released.

bors merge


Google Maps API key support

This PR contains a fix of the settings merge problem that disallows to provide Google Maps API key.

Fixed in 1c75d6c23bda86f0dac24a6044769864aa0b39e8


Add documentation for Cypress

Describe the task

[ ] Add cypress to pre-requisites, as a concept/tool to have an understanding about. Maybe we should have a readme dedicated to all our CI tools
[ ] Add some Cypress related tasks to our resources repo and have them linked here in the relevant readmes. This would be helpful for someone new getting started in the Portal.

This is looking great. Thanks very much @am-Anmol


Prep for Travis push releases

Prep for Travis push releases

Other than the secure line being wrong, this lgtm.

Other than the secure line being wrong, this lgtm.

Fixed


Disabling inline variable values (maybe not nvim-dap-ui related)

How do I disable inline variable values like = array([], dtype='<U50')?

If it's not nvim-dap-ui related, this issue can be closed without comment. Extra Karma points for pointing to the right resource or helping me out anyway!

Finally, found the solution after too much time searching the internet.


Wonder if you know how to easily add operationId to the latest version of Plumber today.

I'm working on connecting the plumber api to the openAI service, but they ask about operationID, I read the plumber documentation and don't get anything there. however in this repo you use it, but that was 6 years ago. Do you know an easier way now?

Phew 6 years ago ;) I'm afraid I can't remember- it looks like it's simply an id to name the endpoint ? But I'm sure plumber repo issues you will get a more up to date answer

yes, got the solution now. Big Thanks! Just in case someone need it. I post solution here. it works for plumber <IP_ADDRESS>00
pr() %>%
  pr_set_api_spec(function(spec) {
    spec$info$title <- "Nama API"
    spec$info$description <- "Deskripsi API"
    spec$info$termsOfService <- "https://contoh.com/tos"

    # Set response schema for /hello
    spec$paths$`/hello`$get$responses$`200`$content$`application/json`$schema <- list(type = "string")

    # Set response schema for /add
    spec$paths$`/add`$post$responses$`200`$content$`application/json`$schema <- list(type = "number")

    spec$paths$`/hello`$get$operationId <- "hello_world"
    spec$paths$`/add`$post$operationId <- "add_numbers"
    spec
  }) %>%
  pr_get("/hello", function(req) {
    "Hello World!"
  }) %>%
  pr_post("/add", function(req) {
    as.numeric(req$POST$body$a) + as.numeric(req$POST$body$b)
  }) %>%
  pr_run(port = 8000)


adds deep coverage zone routed percentage to TP's routing widget

What does this PR (Pull Request) do?

[x] This PR fixes #3752

Note: the TP dashboard is not currently under test.
Which Traffic Control components are affected by this PR?

Traffic Portal

What is the best way to verify this PR?

Run the UI tests and ensure they pass
Navigate to the TP dashboard and view the routing methods widget and ensure there is an entry for "Deep Coverage Zone" with a percentage that lines up with GET /api/1.4/cdns/routing

The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4770/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4771/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4773/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4775/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4776/


test(vllm): fix FinishReason, standardize and enable vLLM E2E tests

Description
Creates and enables vLLM E2E tests that touch each SDK gRPC endpoint at least once with a "happy path" test. Enables and uses the GPU runner for the E2E tests. Standardizes the Completions test across to llama-cpp-python. Fixes the FinishReason enum typing issues across Completions and ChatCompletions.
BREAKING CHANGES

FinishReason is now an enum in both Completions and ChatCompletions protobufs

The API gRPC, typing, and helpers use a Enum class to define and transform the stub responses

CHANGES

adds and enables simple E2E vLLM testing in local and CI environments
adds Completion test to llama-cpp-python, to standardize with vLLM E2E suite

Related Issue
Fixes #1037
Relates to #854
Checklist before merging

[x] Tests, documentation, ADR added or updated as needed
[x] Followed the Contributor Guide Steps

There seems to be an issue with our GPU runner configurations that is blocking the vLLM E2E tests from running: https://github.com/defenseunicorns/leapfrogai/actions/runs/10894219888/job/30230952202
The upstream UDS Common action is unable to install UDS CLI into the bin directory. It seems like it needs sudo or higher permissions, even though that is not usually required in our CPU (large or regular) runners. As a side note, the vLLM E2E tests run locally.

Blocked until #1085 is merged into main.


Campfire occasionally doesn't UpdateGameTime and age out

Noticed when I left the perk tree sitting in real time for a long time; reproducibility unknown.

Cannot reproduce.


On update_value check if value is string

This PR

[x] Adds a check if value is_string

Context
In some cases when copying flexible layouts, value would default to white, because instead of actual $value which is a slug, $color object would be sent.
With this case if value is not empty and it's not string we can be sure that the value is color object and we can pull the slug which will result in correct result.

Thanks for the PR.
I assume you have tested this? You mentioned "in some cases" so I'm not sure how to reproduce the issue on my end to confirm the fix.


Uploading file in disconnected mode

Hi,
After I disconnected from the server and save a file it gets uploaded to the server. Is it right behavior?

Yes, you can disable the auto upload setting in the package preferences


Linking with C

I am obviously missing something very obvious, but can enkiTS be linked with C?
Everything compiles fine with C compiler, but I can only link with C++ compiler.
What am I missing?

To answer my own question:
The library must be compiled and linked as a shared library and not as a static library.


How to get port reference objects ?

Description of the feature
if one has two reference objects,how can we get all of them ? Then I will choose one of them to set the real reference .Thanks!

Steps for implementing the feature
No response
Useful links and references
No response

@kclme84  I've implemented a new class containing all ports info for Hfss3dLayout. this is returned automatically when you create a port is created with pyaedt or read in h3d.boundaries list of objects when you read it from an existing project. the new class has also getters and setters so you can for instance change the reference directly from it.
example
h3d = Hfss3dLayout()
h3d.boundaries[0]["Reference"] = "Bottom:poly_756"


Alefvanoon's instances

Alefvanoon's instances should be removed since he doesn't seem to be hosting any of them anymore.

Nevermind, its the Libredirect extension that its still including his instances, sorry.


Add PCDs for selective feature enablement

Signed-off-by: Michał Kopeć<EMAIL_ADDRESS>

uh, i was under the impression that i need to create variables to be able to access these from VFR. will remove

@miczyg1 please check again

uh, i was under the impression that i need to create variables to be able to access these from VFR. will remove

In the VFR you just declare the structure name containing the variable store, but this store is nothing else than a buffer holding current form state. So the constructor is responsible for initializing the form state before it is displayed for the first time. It is up to you what you will do with this variable store, you can save whole varstore as a single variable, or split its fields into separate variables, or just save only some parts of it. VFR doesn't care about it. It is the C source responsibility to handle the varstore changes and how they should be handled.

@mkopec does it still work?

@miczyg1 Yes, tested and working, i currently have Networking and USB enabled on a VP2.


Fix title at the top in right column of stream settings overlay.

First commit adds space between # and stream name. (kept this commit separate in case we want to do minor CSS changes in the second commit).
Second commit fixes the icon to be according to type of stream.

GIFs or screenshots:

Merged, thanks @sahil839!
Though one thing to check -- does component live-update properly if the stream name or privacy permissions are changed?


wip: root and model-related endpoints documented in a swagger spec

For the curious, this WIP is a partial Serrano API specification written according to the OpenAPI (Swagger) specification. I'll continue to whack away at the rest of the endpoints.

This is great! I did a quick by pasting the YAML in the online editor: http://editor.swagger.io/#/ I will take a closer look likely tomorrow.

Ah, I uncommented line 9 and 16 in the editor and I could send requests.. and it worked!

I tried /categories and it failed because the server wants a trailing slash. This is just a limitation of the editor for not handling the redirect, but it would be best for the trailing slash to be included in the YAML file.

Ah, I uncommented line 9 and 16 in the editor and I could send requests.. and it worked!

Yes, that is a cool feature. Now we just need an up-to-date OpenMRS demo. There are definitely differences between the Serrano API that the demo uses and the current one. That would probably be worth my looking at. In the future we should probably version the API and its implementation and put those versions in the service info ....

I just noticed that I was working with an older version of Serrano and added the _links properties; will remove that.

OpenAPI/Swagger 2.0 can't express one aspect of the Serrano API -- the use of mixed types in result row arrays. There is a vaguely amusing thread in which Swagger developers imply that APIs that do such things are defective. However, perhaps as a result of getting heat for this position, the next version of the spec will support the JSON Schema AnyOf keyword.
I guess for now I'll pretend that results are all strings.

So is the HTML file the generated output of the new RAML file? Since you are not using Swagger can you remove all references to it so it is clear which parts are RAML specific? Also since the docs directory has not been used, can you remove all of Sphinx files so just your new API docs remain?

Yes, the HTML is generated from the RAML. OK. And I need to complete this. I've done a bit more uncommitted work cleaning up the review endpoint and adding the export endpoints, in the process learning RAML's approach to code reuse (resource types and traits).


format group: Replace .log with .txt

This commit is trivial. It is to synchronize the existing file extensions
consistently. Let's replace .log with .txt to read log files normally in
most of the web browsers.
*Changes

Added comment
Replaced .log with .txt

Signed-off-by: Geunsik Lim<EMAIL_ADDRESS>

:memo: Version: 1.20190225. Thank you for submitting PR #508. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.


Interval is not working for axis Type "Time".

Version
5.4.3
Link to Minimal Reproduction
https://echarts.apache.org/examples/en/editor.html?c=candlestick-simple&code=PYBwLglsB2AEC8sDeAoWsAeBBDEDOAXMmurGAJ4gCmRA5JALZW0A0J6AhrngDIcBGVADYFUpUhGhgqAJwBuHIbAIAmNuIC-JDetjkc-Ikh0kAJhzAc8VMEQDa7YuPR5gAVxkBjGrAfPSYv6kHEQqAAy6QbD8RADMACyRQZ5EAIwRjv6mcQAcSVkWPrThqQDsALTp5SrxtJmkJlGBUSGw8RlR6DGwsQCs-c4pPR2d2bC9I1Hm0nQlFVUqvXVRjUHNQa2xqQPi3bF59eJDsbE7pGPxiYfnhbNhZZVh1QBsy0Gr_uv-mwed0Wn9a7oIaAv4XNRA2DTIpzR7VUpvfxafwAXUcyNgKN01hkECohF8ji-ZEoRU8HGgpiE-MgngA1qx6lRoJ5gKYfMT0Bg6NDGVFyPZaBxWLBaPwRbRPBLTLQUbAAPTy2AAC1kVGisHwsAA5hA5MyyAB3CDeWAsCAAM1ghvVuv1sE8mrAeGtwBkdMk2tg9Qx6AxaI0AG4gA
Steps to Reproduce

plot a candle series
xAxis Type should be time axis
give interval value to the chart in axislabels

xAxis: {
type: 'time',
axisLabel:{
interval :2,
}
}
Current Behavior
Interval is not working for axis Type "Time".
option = {
xAxis: {
type: 'time',//  type:'category' working
axisLabel:{
interval :2,
}
},
yAxis: {},
dataset: [
{
source: [
{
a: 20,
b: 34,
c: 10,
d: 38,
date: '2017-10-24'
},
{
a: 40,
b: 35,
c: 30,
d: 50,
date: '2017-10-25'
},
{
a: 31,
b: 38,
c: 33,
d: 44,
date: '2017-10-26'
},
{
a: 38,
b: 15,
c: 5,
d: 42,
date: '2017-10-27'
}
]
}
],
series: [
{
type: 'candlestick',
encode: {
x: 'date',
y: ['a', 'b', 'c', 'd'] // here b is given twice ,if we give c its working
}
}
]
};
Expected Behavior
interval should also work for time axis
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

Its not supposed to work for axis type time. From the documentation:

Interval of Axis label, which is available in category axis.


Update to OmniSharp 1.27.0

I believe that CI is failing due to the issue fixed with https://github.com/OmniSharp/omnisharp-roslyn/pull/1017. Once that issue and a couple of others are merged, I'll build OmniSharp 1.27.1 and update this PR with that.

@rchande eventually, it will be great if automation can create this class of PR for us.


Improvement for k8s.io/docs/concepts/services-networking/ingress-controllers/

You need to put more images in all concept-related pages!

/retitle Add more diagrams into concepts section
Is that fair @sahsz123?

Similar to (closed) issue https://github.com/kubernetes/website/issues/84

Duplicates issue #28085
/triage duplicate


feat: ellipsis on table cell

Sometimes we have a large information to show on a column, that makes a cell too large on both aspects, width and height:

We need to be able to, when we set a width on a column, we can also set if that column could be partially hidden using ellipsis properties. We also need that, when we decide to use ellipsis on a column, the table automatically shows a tooltip with the complete information. Something like this:

Qual o sentido de ter uma tabela que esconde o que eu estrou mostrando nela?
Esse comportamento me parece mais um acordion do que uma tabela.

Qual o sentido de ter uma tabela que esconde o que eu estou mostrando nela?
Esse comportamento me parece mais um acordion do que uma tabela.

The default behavior is to show all information, this feat will only allow the possibility to hide part of the information that can be hidden. Every project has it's own specifications.

What do you guys think of these solution? Any other idea to solve this problem?
@iurynogueira @andersonnagelo @Jeter-Megaron @qrayannes (I can't find Edney's user)

If the user is not finding the information that him needs, the column is not representing their necessity.

@mizek1 @luccskywalker  Sometimes the solution may not be a table, the solution could be related to another component... As I don't understand the context of the demand itself, I can't say so much if in this case it would be a table or not , but regarding the density of information, it also interferes with the user's cognitive process and loses the meaning of the component that serves to systematize the information, which must be presented in a clear and precise way, to facilitate the reading and understanding of the data. If I have a table that brings me such a high density of data and I can't deal logically with it, I have to rethink whether I'm using the right component for the solution.

@edneyql what if the table is just a way to extract through actions?

I believe we can create a discussion at github.com/Brisanet/ion/discussions instead of discussing it here in the issue.

The solution has already been aligned with the SIGA team, we are going to suppress the column information in the table, the user will be able to see all the data related to the selected profile with 1 additional click on the detailing column of the desired line.

Guys, can follow with this implementation because can have other cases to use this abbreviation too. We need give other options to the dev and this improve will be more one.


Add scaladoc to codec annotations

Issue https://github.com/zio/zio-schema/issues/203 has been implemented but it looks like there are missing scaladoc for the annotations:

... add and document these annotations inside zio.schema.annotation ...

/bounty $100

I would like to try this one

@jdegoes @vigoo actually, for Scala 3 we can extract Scala doc at macro execution. But this ticket is now already rewarded and has a merged pr. Should we open a new pr?

If we can capture it it would definitely be a nice feature


rust mode breaks  (goto-address-at-point) when there is trailing ')'

For example (http://www.google.com)  .  (goto-address-at-point) will grab http://www.google.com) as url.

I tried this in text mode and C mode and it also fails there.  So I suspect maybe it's an underlying problem, not specific to rust-mode.  However, is there a mode where it does work for you?  I've been testing it without visiting the URL using (browse-url-url-at-point), which is what goto-address-at-point eventually calls.

yeah, I found that too. the only mode that works for me is text-mode

hmm, () is valid url characters.      https://en.wikipedia.org/wiki/Dunnet_(video_game)
It seems text-mode is broken :)


Remove Rails dependency?

Currently it requires Rails to get the path of the .env.sample file, check if we can make it more generic

I think that would be a great idea. It also depends on activesupport because it is using blank?

@arielj I think I've taken care of this issue by removing the reference to Rails.root:
https://github.com/fastruby/dotenv_checker/blob/main/lib/dotenv_checker.rb#L92-L94
And then also using fast_blank instead of activesupport:
https://github.com/fastruby/dotenv_checker/blob/main/dotenv-checker.gemspec#L15-L16
I think that should be enough, but we should add more tests to confirm that fast_blank is a proper replacement for that call to blank?

great! yes, I think that removing Rails.root and .blank? is enough.
Maybe we don't need to use fast_blank? value is currently either nil or a string, we could check if value ~=/\A\s*/z/ to match any blank or empty string, I can't think of any edge case.

This is no longer an issue. 👍


Validate and cleanup annotations in SBML files

Examples

Publications with invalid PubMed ids
Invalid identifiers.org namespaces

Did some digging on the final 7, and figured out six of them.  Four were obvious (in retrospect) errors where 'ncit' was used instead of 'ido' (the latter had several examples throughout the biomodels; some within the same model, even).  The last ncit one just needed a 'C' to point to the correct element, and I discovered an appropriate CHEBI term with the same number for the sixth.
The seventh was the BSU one, and I also couldn't find anything, so I just removed the reference.
So, with that, this issue can be closed!  https://github.com/sys-bio/temp-biomodels/commit/d542b1bcf12c29e239483677b90d2038134d684a


Avoid an android crash introduced in #5003 (git: 92464860).

On android, sys.stdout.encoding does not exist (sys.stdout is redirected - of type LogFile - which has no encoding attribute).

There are pep8 errors that need to be fixed before it can be merged.

After scouring the python2 and python3 documentation, I'm pretty sure this is the only "right" way to query for sys.stdout.encoding. (Whether doing this query is the "right thing" is a different topic). This patch will at least fix the crash-on-android issue that many have encountered.
For more, see my longer comment on the  related p4a issue (1018)


"fetch" not defined when using Jest tests

Hi There,
I'm using a client (non-react/non-hook) query to run tests in jest and got the following error:
Invariant Violation:
    "fetch" has not been found globally and no fetcher has been configured.

I saw that Apollo in javascript has an argument to directly pass the fetch function:

https://stackoverflow.com/questions/50688998/using-apolloclient-with-node-js-fetch-is-not-found-globally-and-no-fetcher-pas
https://github.com/apollographql/apollo-client/issues/4892

The same argument doesn't seem to exist as a make() argument for ApolloClient.
let instance = {
  open ApolloClient
  make(
    ~cache=Cache.InMemoryCache.make(),
    ~connectToDevTools=false,
    ~uri=_ => graphql_api_address ++ "/graphql"
    // should fetch argument be supported here?
    (),
  )
}

Is there a workaround to be able to use Apollo in jest tests?

It's actually passed to the httpLink. Here's an example :
@module "node-fetch" @val
external nodeFetch: ApolloClient.Link.HttpLink.HttpOptions.Js_.t_fetch =
  "default"

let httpLink = () => {
  ApolloClient.Link.HttpLink.make(
    ~credentials="include",
    ~fetch=nodeFetch,
    ~uri=_ => graphqlEndpoint,
    (),
  )
}

let make = () => {
  ApolloClient.make(
      ~link=httpLink(),
      (),
    )
  )
}

@jeddeloh aah, that makes so much sense! Thank you!


Function "JsonPath.getList()" returns null when key contains a hyphen "-"

Hello,
I am pulling in com.jayway.restassured:json-path:2.6.0 as a dependency of com.jayway.restassured:rest-assured:2.6.0 (which is fantastic by the way!). When attempting to validate some properties of a JSON API, I ran into the following case, which looks like a bug.
Given a JSON document like {"key-name":["value1", "value2"]}, I can call JsonPath.getString("\"key-name\"[0]") to get the first list value value1.
But, if instead I ask for the entire list with JsonPath.getList("\"key-name\""), I just get null instead.
I have validated my syntax using other JSONPath parsers (like https://jsonpath.curiousconcept.com/) and everything looks correct. I have also included sample code below that shows how my syntax works for key with underscores, but fails for keys with hyphens.
import com.google.common.collect.ImmutableList;
import com.jayway.restassured.path.json.JsonPath;
import org.junit.Assert;
import org.junit.Test;

public final class JsonTest {

    @Test
    public void testParse() {
        String body;
        JsonPath json;

        // Original JSON, notice that the key contains an underscore
        body = "{\"key_name\":[\"value1\", \"value2\"]}";
        json = new JsonPath(body);

        // Assert #1 passes
        Assert.assertEquals(
            "value1",
            json.getString("key_name[0]")
        );

        // Assert #2 passes
        Assert.assertEquals(
            "value1",
            json.getString("\"key_name\"[0]")
        );

        // Assert #3 passes
        Assert.assertEquals(
            ImmutableList.of("value1", "value2"),
            json.getList("\"key_name\"")
        );

        // Same as original JSON, notice that the key contains a hyphen instead of an underscore
        body = "{\"key-name\":[\"value1\", \"value2\"]}";
        json = new JsonPath(body);

        // Assert #4 passes
        Assert.assertEquals(
            "value1",
            json.getString("\"key-name\"[0]")
        );

        // Assert #5 fails
        //java.lang.AssertionError: expected:<[value1, value2]> but was:<null>
        Assert.assertEquals(
            ImmutableList.of("value1", "value2"),
            json.getList("\"key-name\"")
        );
    }

}

If there is a workaround that allows me to extract the desired values, that would also be great, as it's completely possible that I'm not escaping something properly.
Thank you!

Wrong project.
https://github.com/jayway/rest-assured

Bug has been migrated to the correct repo (jayway/rest-assured/issues/601)
I now see that the class manifests don't match up (com.jayway.restassured:json-path versus com.jayway.jsonpath:json-path) that being said, can you please expound on the differences between these two (similarly named) projects? (for new-user eduction and historical purposes)

The projects, JsonPath and RestAssured, where started at about the same time. At the time this JsonPath implementation was very rudimental and far from feature complete. The Rest-assured project decided to add a groovy  dependency and use JsonSlurper to query JSON documents. It was a bad choice by the RestAssured project to call it JsonPath. First and foremost because it's not a JsonPath implementation and secondly it collided with an other Jayway project (this one). Unfortunately these decisions are hard to revert.


Groovy consumer: match array of strings

Similar to #256, but for the groovy consumer. Can we match a list of strings?
Something like:
willRespondWith(...)
withBody {
    permissions eachLike(3) { ->
        'GRANT'
     }
}

which would generate json like
"permissions": ['GRANT', 'GRANT', 'GRANT']

and matchers like
".permissions": {"min": 0, "match": "type"},
".permissions[*]": {"match": "type"}

I've implemented it without the need for the closure (they map to objects). Also, the second matcher in your example is not necessary as they cascade.
So, for
withBody {
        permissions eachLike(3, 'GRANT')
}

will generate the following JSON
{
    "permissions": ["GRANT", "GRANT", "GRANT"]
}

and matchers
{
    "$.body.permissions": {"match": "type"}
}

and now you can even get more fancy
withBody {
        permissions eachLike(3, regexp(~/\w+/))
        permissions2 minLike(2, 3, integer())
        permissions3 maxLike(4, 3, ~/\d+/)
}

Looks good! Thanks @uglyog


SOLID notifications

Feature description
The app should be able to manage the notification system, it should provide SOLID users a way of being notified when a route is shared with them or when a comment has been made on their own route.
Acceptance criteria

[x] Way of posting a notification of type 'routeShared'.

[ ] Way of posting a notification of type 'commentMade'.

[x] Way of retrieving the url of the route of a notification type 'routeShared'.

[ ] Way of retrieving the url of the route of a notification type 'commentMade'.

[ ] Way of erasing a notification.

A class NotificationManager.js was created with the following functions:

createNotificationContent: method that allows to create the content of the notification itself.

postNotification: being the method in charge to send the notification to the inbox of the friend.

getNotificationDocuments: it retrieves the documents of the notifications placed in the inbox folder

processSharedRoutes: it allows to retrieve the 'summary' of the notification, where the url of the route shared is contained.

This approach using tripledoc was taken when the hook useNotification did not work properly.

With the postNotification method there was a problem, in order to fix it, I turned to the solid community in order to fix this problem, and so, it was described and solved in this post


Explicit license in source files

I am working on having a python-pathlib2 package in Fedora.
Review Request: python-pathlib2 - Object-oriented filesystem paths
One issue is about having only a LICENSE.rst file.
It should be enough to have a confirmation that all files are
under MIT license.
Otherwise, something like 3 lines header should be enough,
as there is already the LICENSE.rst file. For example:
# Copyright (c) 2014 Matthias C. M. Troffaes
# Copyright (c) 2012-2014 Antoine Pitrou and contributors
# Distributed under the terms of the MIT License.

Thanks for packaging pathlib2 for Fedora!
Yes, all files are under the MIT license. I'll add a license header to every source file, indeed that's good practice, and thanks for suggesting it.


Better account for db version in path to grype dir, fix and add some tests.

Fixes a bug @zburstein found in the grype wrapper where the new file path with the grype db version in it isn't always resolved correctly. Adds and updates some tests around the same, and also fixes a few cosmetic test var and function name fixes I noticed.

Note that this overlaps slightly in a few places with the change I'm making in https://github.com/anchore/anchore-engine/pull/1062/files as I needed one of the improvements from there to do this. This is a smaller change that fixes a current bug so I want to get this in first - I will rebase 1062 after this is merged.


Add nat_gateway_eip_enabled variable

What

Add nat_gateway_eip_enabled variable to control the creation of Elastic IPs and NAT Gateways for private subnets

Why

In some environments like Elastic Beanstalk with a simple app deployed behind a public load balancer, we don't need to create NAT gateways and Elastic IPs for private subnets

Updated and fixed README.md.
Changed the default value for nat_gateway_eip_enabled variable to true.


NoEmptyLinesOpeningClosingBraces removes empty lines from body if functions start/end with comment

The following, when NoEmptyLinesOpeningClosingBraces is true,
func foo() {
  let x = 1

  // let y = 2
  // let z = 3
}

results in
func foo() {
  let x = 1
  // let y = 2
  // let z = 3
}

It also happens the other way around.
func foo() {
  // let w = 0

  let x = 1
}

becomes
func foo() {
  // let w = 0
  let x = 1
}

IMO both cases should be left unchanged, because these empty lines will likely be intentional. Also, in the stricter sense, they are not located after/before the opening/closing braces.

Synced to Apple’s issue tracker as rdar://136130930

@mateusrodriguesxyz Would you be able to take a look at this?


Better deployment channel for oasis-build

Honestly, keeping it up-to-date without rustup integration is all sorts of painful.

https://github.com/oasislabs/toolstate


Merge pull request #1 from apache/master

update merge

update merge

update merge

update merge

There're no file changes in this PR.


hoverpause doesn't work when hovering of prev/next buttons

This for me is a bug, so I decided to report it :)
I just noticed that hoverpause: true  doesn't work when hovering of prev/next buttons.
Not sure if that's intentional or not.

That is good hint. Will be in next release.

Fixed with 6526ca4

Awesome!
Can I just drop the updated file into my /js folder and it works? Or do I need to set something up first?
Thanks.

Only update glide js file.

Awesome!
FWIW, here's one of the pages I'm using it right now: http://ricardozea.design/project-pearlmax.php
Thanks for adding this functionality so fast.


Duplicate user in project

Prevent duplicate user in projects.
A userId cannot be in the users list of a project multiple times.
When a user is admin of a project, this user cannot appear in the users list of the project.

In the same sense, the application stops working when a user is part of a project and this user then gets deleted.

Prevented duplicate users in projects in the frontend, still needs to be done in the backend
Prevent users from adding non - existent  users to projects

Will have to prevent adding duplicte users in the backend.
Also if a user deletes its account, the project breaks for all users which in turn breaks it for everyone else.

Deleting a useraccount which is registered in a project no longer breaks the project and the project gets updated to no longer include the deleted user.


Random crashes when opening the app

Describe the bug
app randomly crashes and does not open when trying to open it to reopen the app i had to go to android app settings to clear the cache and storage for the app to work like normal
To Reproduce
Steps to reproduce the behavior:

Go to 'just open the app with 2 apps in the background and it would crash and the app would not open'

Expected behavior
i expected the app to open normally and not crash
Screenshots
none
Desktop (please complete the following information):

OS: Windows
Browser Brave
Version 1.2.8

Smartphone (please complete the following information):

Device: Vivo V40
OS: Android 14
Browser  Chrome
Version 1.2.8

It is not possible to reproduce the problem, everything is working stably. Can you record a video?


pipeline created

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

comment followed


burpsuite: 2023.5.3 -> 2023.6.2

Description of changes

Converted the hash with nix hash to-sri --type 256 edfad3f2fe2aba6ea94eea2800a7352035debb7be574129cf1a3ab3176943b05
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 243494 run on x86_64-linux 1

1 package built:

burpsuite

@exploitoverload I checked it today and saw there is a new version. I've just updated the PR, in case you want to test it again.


Icons: resizing and color

I tried out the new icons which is nice, but it doesn't allow you to change it's attributes. It would be nice to be apply have attrs exposed to be able to resize them and change colors.

Oh, totally forgot to add the attrs parameter 🤦🏻♂

haha, whenever you have the chance to add it, it would be a appreciated.


this commit adds a test that shows how its possible to break the

transport on backpreasure when sending 5000 messages at once
once adding onBackpressureBuffer on listen method it does not fails

https://github.com/scalecube/scalecube/issues/93


What is the best practice to upgrade operator nightly builds using OLM?

Type of question
community best practices
Question
I'm working on an project to build operators, and trying to use OLM to manage the lifecycle of the operators. I followed the documents to create images of bundles and bundle index with opm, and created a catalogsource based on the image to deploy the operator. It works well for the installation of the operators, and I would like to use OLM to upgrade operators in our test clusters automatically every day with nightly builds.
However, I haven't figured out how to do it in the most efficient way with minimal operational efforts. Do I need to maintain a list of all the nightly builds in the index, and always create a new index image containing the bundle of the snapshot version based on the previous one? Is it possible to always create a new index image with the snapshot bundle from scratch to make the nightly build "stateless", and upgrade operators automatically?
Thanks!
Environment

operator-lifecycle-manager version:

0.14.1

Kubernetes version information:

1.15.7

Kubernetes cluster kind:

GCP

Great question, @fantapsody - we're trying to improve our docs over at the olm book and this would be a great case study to include.
I think the specifics of how we get you to "minimal operational overhead" may depend on how you are building / etc. Any details you can share would help us help you 😄
In 1.14.1 we added CatalogSource polling for image types. See the docs here.
One way you could do what you're trying to do is by maintaining a moving tag for your index image (i.e. quay.io/my/catalog:nightly). When you have a new nightly build available, you can pull the existing nightly tag, run opm index add to add the nightly bundle to your index, and update the nightly tag to point to index you just pushed. (I would also tag these by git sha or time so that you have a record).

Thanks! @ecordell
I have tried to do as you suggested, and in this case I have to get the head version of the channel, and set spec.replaces in CSV correctly. And in order to do so, I have to run opm export to save the manifests to local disk and extract the head version in the package file. It is workable, while I also would like to know if there is a simpler way to do that.
On the other hand, there may be tens of or hundreds of versions in the catalog source image if we release snapshot builds frequently, would that be a problem? And should we use a separate index image or a separate channel?
CatalogSource polling seems to be promising and I also have tried it. One thing that bothers is that if I set the interval to 1 minutes, it may take up to 15 minutes to do another check, is this by design?


So trigger @death is not fired and neither do the fix for death mounts.

But this only happens when mount is damaged by something. If u just set hits 0, u will get dismounted as soon the mount dies and pass death trigger. So fix works but only if CChar::death() is not prevented by something.

I removed the damage line of the .kill function. And it works, raider gets dismounted when mount dies.
I dont find the problem. ontakedamage() should be doing something that prevent death() to happen. Or any weirdness fix.

well for some reason I dont understand if u kill a ridden pet by damage, it will not reach CChar::death, Any so periodictick is prevented.
But if u just set hits to 0, it dies and dismount u. :/

ok, I got it. now poison timers work and death() happens.


An option to see "my reservations" for both tutor and tutee

A user should be able to go into the both anytime and see his list of upcoming reservations for peer tutoring.

Added commit today-need to test it once userid is coming in properly and not from the test data.

Added commit on branch reservation.


Contact form solution

Pull Request: Enhanced Contact Form Implementation
Live demo solution Netlify solution
What does this PR do?
This pull request introduces an enhanced version of the contact form, focusing on improving accessibility, responsiveness, and user experience. The solution aligns closely with the provided design specifications and incorporates several new features aimed at making the form more user-friendly.
Fixes #147
Motivation and Context
The goal of this implementation is to create a contact form that not only looks good but also provides a seamless experience for all users, including those with disabilities. By improving accessibility and user experience, we aim to ensure that everyone can interact with the form without difficulty.
Dependencies
No additional dependencies are required for this change.
Screenshots

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How should this be tested?
To verify the changes, please follow these steps:

Check the accessibility features using a screen reader.
Fill out the form to test real-time validation for all input fields.
Submit the form and ensure the success message appears and disappears after 5 seconds.
Refresh the page to check if the saved draft is pre-filled in the form.

Test Cases

[ ] Test the accessibility of the form with various screen readers.
[ ] Validate required fields and ensure dynamic error messages display correctly.
[ ] Check the responsiveness of the form on different screen sizes.

Request for Merge
@beRajeevKumar  I kindly request a review of this pull request and look forward to your feedback. If everything looks good, I would appreciate merging it into the main branch. Thank you for your time and support!

@Titus210, I am unable to see the project on the main page. Can you please link the project so that it will visible on the main page.

@AkshitGarg24 I'm not sure what you mean by linking the project. Would you kindly elaborate more clearly.
Should it be merged first for it to appear in the project window? Its currently running on a branch. contact-form-solution under the directory 25-contact-form.

@AkshitGarg24 I'm not sure what you mean by linking the project. Would you kindly elaborate more clearly. Should it be merged first for it to appear in the project window? Its currently running on a branch. contact-form-solution under the directory 25-contact-form.

The live demo link that i have included on the project description is from my own Netlify account.

This is the link to the website that will be shown after merging your PR : Link.
After merging your PR, I should see some changes in the website, right? But I am unable to find any difference in the website. I guess you forgot to link the changes so as it should be visible on the main web-page.

@AkshitGarg24 I have understood what you mean. Try merging the PR first then we see it updating on the main website, from my end this pull request isn't merged yet.

The link provided shows how the website will appear after merging the PR. But for your convenience, I will merge the PR and then check the changes in the website.

You can check the website now @Titus210.

@AkshitGarg24 Thank you. However, i cannot also see the update on the website. Just for clarification...is there anything that I have missed out, I would appreciate if you took me through what is needed so that the website can be among other listed projects on the website?

Have a look at the PR #163. I guess you will get the idea what you are missing out.


Fix BlendFunction.ONE_MINUS_CONSTANT_COLOR

Fixes https://github.com/AnalyticalGraphicsInc/cesium/issues/7622

Thanks for the pull request @lilleyse!

:heavy_check_mark: Signed CLA found.
:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.


Update date.js

@Ganevru, I am experiencing the same described here: https://github.com/Chronoblog/gatsby-theme-chronoblog/issues/25
This tiny PR isn't the most elegant one, but it does prevent the issue from happening. (Since this is my first PR here, I decided to go as simple as possible.)
A solution to consider in the future (depending on your plans) is to add a timeZone option in siteMetadata (gatsby-config.js) and then parse it appropriately inside date.js.

The root cause of the issue is that in our frontmatters (as per the starters examples) we input dates in the following format:
date: 2020-08-13

When you insert that string into the Date constructor, Date objects are (implicitly) created with 00:00:00 time and UTC as their time zone:
// https://github.com/Chronoblog/gatsby-theme-chronoblog/blob/master/packages/gatsby-theme-chronoblog/src/components/date.js#L14
const newDate = new Date(date);

Then, when you convert the newDate using toLocaleString(language, options) to our local time zones, things can get weird.
Because @reallymello (Florida) and I live in time zones that are UTC-3 (or more) we noticed the issue. All our posts/notes/links are showed in the feed with the frontmatter day minus 1.
I assume you haven't noticed anything on your side because it seems you live in Moscow (which is UTC+3, give or take). Your days are never shifted!

Well, that's a very long explanation for such a tiny PR. 😅
In any case, thanks again for creating this great theme!

Thanks for the detailed explanation, this is a good practice even for small code changes.


can I use two motion controls and one navigation?

I wanted to know if you have to use two movement controls and one navigation just to be able to move around in games and if you want to know if there is also how to connect the navigation device on the same bluetooth as the movement ones, I heard about bthPS3 which is a driver  bluetooth for PS3 accessories but I didn’t know how to configure it if someone can help me I appreciate it

Go on youtube, you have some great tuto to setup your PSmove controller and your Cam with this plugin here.