artifacts did not get published for 0.0.9 release

👋  Looks like the 0.0.9 still does not get marked as the latest release yet, as result, there are artifacts for it. Thanks!

Hi, thanks for reporting this! @sylvestre was this because of the stdbuf issue? Or did something go wrong with the publish script?

Not sure what happened. Push a tag was enough in the past.
I did it by hand
https://github.com/uutils/coreutils/releases/tag/0.0.9

The release currently only has the source code artifacts, not the pre-built executables. Can we fix that somehow?
It looks like it failed to build in the last commit on the 0.0.9 tag, because of stdbuf. Maybe that's why it didn't release? Would it make sense to backport the stdbuf fix to 0.0.9 (with the previous MSRV) and release a 0.0.10? Because that wouldn't require workarounds in all the build scripts like the one this issue was mentioned in (https://github.com/Homebrew/homebrew-core/pull/93303).

In the past, they were automatically generated and integrated in the release.

Nothing is generated/published unless all the tests pass.

I'll try to look into an easier, tool-based, publishing method (probably using cargo-workspaces) this weekend.
Hopefully, I can design something that gives feedback and blocks/warns if it can't fully complete correctly.
@sylvestre , would you mind adding "uutils/Maintainers" as co-owner to your solo-owned uutil crates ('findutils', uu_basenc, uu_chcon, uu_csplit, uu_dd, uu_pr, and uu_runcon) so that I can push to them? (It's done with "Add owner" under the "Settings" tab for the crate.)

It looks like it failed to build in the last commit on the 0.0.9 tag, because of stdbuf. Maybe that's why it didn't release? Would it make sense to backport the stdbuf fix to 0.0.9 (with the previous MSRV) and release a 0.0.10? Because that wouldn't require workarounds in all the build scripts like the one this issue was mentioned in (Homebrew/homebrew-core#93303).

Instead of back-porting, it's probably easiest to just call "0.0.8" the pre-MSRV-change release and just push another working release. If we sync all the crate/sub-crate versions, (ie, for coreutils, uu_*, and uucore* crates), the next patch version would be 0.0.12.
Is there any reason not to sync them?

Instead of back-porting, it's probably easiest to just call "0.0.8" the pre-MSRV-change release and just push another working release.

That works too! I also think syncing them is a good idea.

@sylvestre , I've got a 0.0.12 release (all crates synchronized) cooking in CI.
If you have no objections, then once it passes, I'll push it to 'main'. That will fix the "release".
Then, you (or I) can publish it from that commit.

Is there any reason not to sync them?

Legacy :)

would you mind adding "uutils/Maintainers" as co-owner to your solo-owned uutil crates on crates.io ('findutils', uu_basenc, uu_chcon, uu_csplit, uu_dd, uu_pr, and uu_runcon) so that I can push to them?

Ah, bizarre why you aren't listed yet. Anyway, invites sent!
All of that sounds great, thanks :)

0.0.12 is released and available (including all binary artifacts).
@sylvestre , if you'll publish it to crates.io, we'll be caught up.

I wrote some release notes for this release based on GitHub's "generate release notes" feature here: https://github.com/uutils/coreutils/releases/tag/0.0.12. I can remove them if you don't like them.

Looks great. 👍
On Wed, Jan 19, 2022, 09:48 Terts Diepraam @.***> wrote:

I wrote some release notes for this release based on GitHub's "generate
release notes" feature here:
https://github.com/uutils/coreutils/releases/tag/0.0.12. I can remove
them if you don't like them.
—
Reply to this email directly, view it on GitHub
https://github.com/uutils/coreutils/issues/2892#issuecomment-1016600774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAATSBFVF3CNYPXRVRXKA5DUW3MLHANCNFSM5MFWTDVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

Terrific, thanks :)
Message ID: @.***>

done:
https://crates.io/crates/coreutils

Great team work ;)

I'll try to look into an easier, tool-based, publishing method (probably using cargo-workspaces) this weekend. Hopefully, I can design something that gives feedback and blocks/warns if it can't fully complete correctly.

@sylvestre , @tertsdiepraam , after looking into this, I think I can design some scripts and instructions around cargo-workspaces, gh (GitHub CLI client), and jq to more simply automate releases. I'll start looking into a full design and PR when I get some time, hopefully over the next couple of weeks...


Setup - TurboJPEG Install

Install TurboJPEG in MIVisionX-setup.py to get RALI by default.

#263 merge fixes this issue


fix: Adds fallback dictionary when serialising device info.

Goal
Previously the device tab was serialised using the dictionary supplied from the report, which could be null if the keypath did not exist. This could result in the device tab being completely null. This fix uses an empty dictionary as a fallback so that at least some information (freeDisk, locale, simulator) will always be returned.
Changeset
Changed
Initialise empty dictionary and defensively add items from report to it.
Tests
Added a failing unit test for the encountered scenario then passed it.
Review

For the submitter, initial self-review:

[x] Commented on code changes inline explain the reasoning behind the approach
[x] Reviewed the test cases added for completeness and possible points for discussion
[x] A changelog entry was added for the goal of this pull request
[x] Check the scope of the changeset - is everything in the diff required for the pull request?
This pull request is ready for:

[x] Initial review of the intended approach, not yet feature complete
[x] Structural review of the classes, functions, and properties modified
[x] Final review

For the pull request reviewer(s), this changeset has been reviewed for:

[x] Consistency across platforms for structures or concepts added or modified
[x] Consistency between the changeset and the goal stated above
[x] Internal consistency with the rest of the library - is there any overlap between existing interfaces and any which have been added?
[x] Usage friction - is the proposed change in usage cumbersome or complicated?
[x] Performance and complexity - are there any cases of unexpected O(n^3) when iterating, recursing, flat mapping, etc?
[x] Concurrency concerns - if components are accessed asynchronously, what issues will arise
[x] Thoroughness of added tests and any missing edge cases
[x] Idiomatic use of the language

👍

@martin308 I've added a test for a nil dictionary by using #pragma to loosen the restrictions.

Thanks for the review comments - I've addressed the pragma + validation comments.


Setup options.txt to be default settings

Allows for user configuration rather than the current personal settings.
Mod configs retain the same values.

Complete as of 1.0.3


[Memo] Update shell configuration files after install

This issue stores some thoughts and experiments while implementing this feature.

pyenv-init --detect-shell would currently output non-bash result for shells not having rc files. This is caused from the last PR I proposed.
The next PR will address this issue by not assigning placeholder values to rc files.
https://github.com/ianchen-tw/pyenv/tree/fix-export-shell

I found out that even pyenv-init could not detect user's shell very well.
Need to do some experiments that how this script output under different situation.
Here are some cases.

user shell
method to call pyenv-autoinstaller
result

bash
curl ... with bash
wip

bash
./auto-installer
wip

bash
bash ./auto-installer
wip

zsh
curl ... with bash
wip

zsh
./auto-installer
wip

zsh
bash ./auto-installer
wip

fish
curl ... with bash
wip

fish
./auto-installer
wip

fish
bash ./auto-installer
wip


widgets: Increase node tests coverage to 100%.

After this, I'll update node tests for voting_widget.

The reason I haven't enforced 100% coverage here is that this file is still in beta. So do we need to enforce 100%?
(also fixing failure)

LGTM, ready to merge when build passes

merged, thanks @shubhamdhama!


#1761 Document Islandora Advanced Search

Purpose / why
Create a new page documenting how to configure new Advanced Search. Resolves #1761
What changes were made?
New .md, new screenshots in the assets folder, and new entry in mkdocs.yml to add the page to the tree.
Verification
Review! This is mostly lifted from the great docs @nigelgbanks already made in the readme: https://github.com/nigelgbanks/islandora/blob/8.x-1.x-advanced-search/modules/islandora_advanced_search/README.md, with some changes to make it a docs page instead of a readme and to fit the style guide.
Interested Parties
@Islandora/documentation and @nigelgbanks

Checklist

Pull-request reviewer should ensure the following

[ ] Does this PR link to related issues?
[ ] Does the proposed documentation align with the Islandora Documentation Style Guide?
[ ] Are the changes accurate, useful, free of typos, etc?
[ ] Does this PR update the last updated on date on the documentation page?

Person merging should ensure the following

[ ] Does mkdocs still build successfully? (This is indicated by TravisCI passing. To test locally, and see warnings, see How To Build Documentation.)
[ ] If pages are renamed or removed, have all internal links to those pages been fixed?
[ ] If pages are added, have they been linked to or placed in the menu?
[ ] Did the PR receive at least one approval from a committer, and all issues raised have been addressed?

I tested locally. It can't find the images, I think you might need ../../assets?

Gave that a try! They are now all broken in my repo, but if they built in the real docs, then 👍

I think that's one .. too many, you need to go one level up and then one down into the assets folder:
../assets/

That does look better in the browser view

Action gifs! Nice!


Subs in type pages

exit() is being included in the Any page, which doesn't make sense for 2 reasons: it doesn't have much to do with it, and it's not a method.
https://docs.perl6.org/type/Any#sub_exit
item() is right below it, which might indicate it's a generalized issue.

Other interesting (similar) issues:

https://github.com/perl6/doc/issues/1045
https://github.com/perl6/doc/issues/1162

Also #1747

Already moved to the new page.

Now that a page for independent routines has been created, it's better to reopen this.

Probably a list of movable routines should be inserted above so that we can close this when finished.

The original item was moved; Any other subs/methods not in the correct place will eventually be caught by xt/check-signatures.rakutest


Clarify whether CRDs are cluster scoped in multitenancy page

Description
This PR updates the Kubernetes documentation to clarify the behavior of Custom Resource Definitions (CRDs) in the context of namespace isolation. The current wording incorrectly implies that namespace isolation does not apply to CRDs, without specifying that while CRDs are cluster-scoped, the resources they define can indeed be namespaced.
The revised sentence now reads:
"However, it can be difficult to configure, and it doesn't apply to resources that are cluster-scoped and not namespaced, such as Custom Resource Definitions (CRDs) themselves (though the custom resources they define can be namespaced), Storage Classes, and Webhooks."
This clarification will help users better understand the relationship between CRDs and namespace isolation.

Issue

Closes: #48112

/retitle Clarify whether CRDs are cluster scoped in multitenancy page

/lgtm

@mengjiao-liu @mickeyboxell @tengqm @sftim Just wanted to follow up on the PR—everything is looking good on your end, and no further changes are needed. Could you please review it when you have a moment? If all looks good to you, feel free to go ahead and merge.

/hold
Hold until the PR addresses the review comment https://github.com/kubernetes/website/pull/48156#discussion_r1800247602 by removing the changes from the unrelated file for this PR.
(OK to unhold once that's corrected)

This is a quick reminder about this PR. If everything looks good, could you please go ahead and merge it? Let me know if there’s anything else you need from me.


[5.7-04252022] Update no results found text

Rationale: Update no results found text
Risk: Low
Risk Detail: it only changes a text string
Reward: Low
Reward Details: Users will get a new text message when no results on navigator were found
Original PR: https://github.com/apple/swift-docc-render/pull/232
Issue: rdar://92612891
Code Reviewed By: @hqhhuang
Testing Details: use the navigator's filter input with random text, assert that feedback text when no results is updated

@swift-ci test


implement log.js and add convertCelciusToFahrenheit.js and convertCel…

…ciusToFahrenheit.test.js

Addresses Issue: #theRelatedIssueNumber

[] Running yarn lint does not trigger any linter errors
[] The test of the method you have fixed is passing
[] You have written a new skeleton method for someone else to work on!
[] You have written tests to accompany your skeleton method

But you seem to already be aware of that 😂  ! thanks :)


Seeder doesn't handle the resource removal

For instance, when a role assignment is changed, the old roles are still in place.

swift containers update. if container is removed, seeder doesn't recreate it as well as custom ACL.


QuickJump: Don't search for every keypress.

When the search box gets updated, wait 150ms before searching.

I see you that haddock-api/resources/html/package-lock.json was removed and haddock-api/resources/html/yarn.lock added. What is the reason for this?
Otherwise, this seems reasonable. Ideally, the UI would not be blocked while it searches, but that's a whole other problem.

I switched from npm package manager to yarn package manager (which is built on top of npm). Yarn has several advantages over plain npm so the js community as a whole is moving towards yarn.
They consume the same input files and work in a similar way but yarn is easier to use.
They use different lock files and it's dangerous to keep both since they can have locked different package versions.

Right, QuickJump could stand to have nicer API 😄. But this PR is a definite improvement, so let's get it merged!

I switched from npm package manager to yarn package manager (which is built on top of npm). Yarn has several advantages over plain npm so the js community as a whole is moving towards yarn.
They consume the same input files and work in a similar way but yarn is easier to use.
They use different lock files and it's dangerous to keep both since they can have locked different package versions.

Seems reasonable. Shouldn't the documentation here be updated to talk about yarn though?

Oh, sure, I'll update the instructions. I've got a lot going on now, but I'll get to it shorly I hope.

@mergifyio rebase

@kolmodin Do you still want to work on this?

No, sorry. Can someone take over, if this PR is still relevant?

I have no idea if it's still relevant, you opened it before I took office :)

quickjump usually lags out the JS engine instantly if I apply it to esqueleto or persistent e.g., so I think this PR is still relevant, although I am wondering if it is doing something exceedingly inefficient and which could be improved instead?

Hi, thank you for this PR, but Haddock now lives full-time in the GHC repository!
Read more at https://discourse.haskell.org/t/haddock-now-lives-in-the-ghc-repository/9576.
Let me know if you feel it is still needed, and I'll migrate it. :)


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 2.0.30 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Settings help

Hello, I got everything installed but cannot change the default music quality or directory. What is the command I need for this to change it to hires please

Hi there, thanks for having made an issue. In order to use some other directory as the parent directory for downloads, you pass it as the second argument: e.g.
$ tidal-wave https://listen.tidal.com/album/123456789 ~/Desktop/music/TIDAL/ --loglevel debug

Okay, thank you. So there is no way to change the default setting you have to override the download location evertime

Okay, thank you. So there is no way to change the default setting you have to override the download location evertime

No, not at the moment. You could create a Bash function or similar, but I’m trying to resist implementing more toggles than is necessary. KISS and all that


Fetching sources for libraries in IntelliJ throws CC errors for gradle/gradle

When hitting the "Download sources" button in IntelliJ for gradle/gradle I get the following errors:
78 problems were found storing the configuration cache, 7 of which seem unique.
- Initialization script '/private/var/folders/mt/w8xt125171b77qvh3q10jhd00000gn/T/ijmiscinit2.gradle': execution of task ':smoke-test:DownloadSources' caused invocation of 'Task.convention' in other task at execution time which is unsupported.
  See https://docs.gradle.org/8.3-20230606222914+0000/userguide/configuration_cache.html#config_cache:requirements:use_project_during_execution
- Initialization script '/private/var/folders/mt/w8xt125171b77qvh3q10jhd00000gn/T/ijmiscinit2.gradle': execution of task ':smoke-test:DownloadSources' caused invocation of 'Task.project' in other task at execution time which is unsupported.
  See https://docs.gradle.org/8.3-20230606222914+0000/userguide/configuration_cache.html#config_cache:requirements:use_project_during_execution
- Initialization script '/private/var/folders/mt/w8xt125171b77qvh3q10jhd00000gn/T/ijmiscinit2.gradle': line 17: execution of task ':smoke-test:DownloadSources' caused invocation of 'Task.convention' in other task at execution time which is unsupported.
  See https://docs.gradle.org/8.3-20230606222914+0000/userguide/configuration_cache.html#config_cache:requirements:use_project_during_execution
- Initialization script '/private/var/folders/mt/w8xt125171b77qvh3q10jhd00000gn/T/ijmiscinit2.gradle': line 17: execution of task ':smoke-test:DownloadSources' caused invocation of 'Task.project' in other task at execution time which is unsupported.
  See https://docs.gradle.org/8.3-20230606222914+0000/userguide/configuration_cache.html#config_cache:requirements:use_project_during_execution
- Plugin class 'org.gradle.api.plugins.GroovyBasePlugin': execution of task ':smoke-test:DownloadSources' caused invocation of 'Task.project' in other task at execution time which is unsupported.
  See https://docs.gradle.org/8.3-20230606222914+0000/userguide/configuration_cache.html#config_cache:requirements:use_project_during_execution
- Plugin class 'org.gradle.api.plugins.JavaBasePlugin': execution of task ':smoke-test:DownloadSources' caused invocation of 'Task.project' in other task at execution time which is unsupported.
  See https://docs.gradle.org/8.3-20230606222914+0000/userguide/configuration_cache.html#config_cache:requirements:use_project_during_execution
- Task `:smoke-test:DownloadSources` of type `org.gradle.api.DefaultTask`: cannot serialize object of type 'org.gradle.api.internal.project.DefaultProject', a subtype of 'org.gradle.api.Project', as these are not supported with the configuration cache.
  See https://docs.gradle.org/8.3-20230606222914+0000/userguide/configuration_cache.html#config_cache:requirements:disallowed_types

https://ge.gradle.org/s/fszpmm5plt7wo/failure

cc: @gradle/bt-execution

Sorry that you're having trouble with Gradle!
This is an issue with IntelliJ IDEA’s Gradle integration, which is developed by JetBrains and cannot be addressed in the Gradle Build Tool project.
Please open such IntelliJ-related issues on the JetBrains Issue Tracker at https://youtrack.jetbrains.com/newIssue?project=IDEA.

This is an issue with the DownloadSources task injected by IDEA.

Shouldn't we reach out to JetBrains to get them to fix this, just as we reach out to maintainers of other popular plugins to fix CC issues?


Can't connect to OPC PLC server - BadServiceUnsupported

Describe the bug
I am trying to connect to a simulated OPC PLC server, following the Postman tutorial.
After the service discovery, the TCP connection inside the PLC server closes.
To Reproduce
Steps to reproduce the behavior:

Install the full deployment with -type all -version 2.9
Create a VM for the PLC Server, open port 50000 for the TCP connection
After request as described here, the OCP PLC server throws errors:

 ██████╗ ██████╗  ██████╗    ██████╗ ██╗      ██████╗
██╔═══██╗██╔══██╗██╔════╝    ██╔══██╗██║     ██╔════╝
██║   ██║██████╔╝██║         ██████╔╝██║     ██║
██║   ██║██╔═══╝ ██║         ██╔═══╝ ██║     ██║
╚██████╔╝██║     ╚██████╗    ██║     ███████╗╚██████╗
 ╚═════╝ ╚═╝      ╚═════╝    ╚═╝     ╚══════╝ ╚═════╝

[09:53:49 INF] Current directory: /app
[09:53:49 INF] Log file: /app/cc8170bcfb50-plc.log
[09:53:49 INF] Log level: debug
[09:53:49 INF] OpcPlc v2.9.9 (SDK 1.4.372.56) starting up ...
[09:53:49 DBG] Informational version: v<IP_ADDRESS>+16d28ce723 (SDK 1.4.372.56+b90f76235135840774f898c59c20ff02ac994e2a from 08/25/2023 18:28:10)
[09:53:49 DBG] Build date: 09/18/2023 15:25:02
[09:53:49 INF] Alternate base addresses (for server binding and certificate DNSNames and IPAddresses extensions): ["opc.tcp://cc8170bcfb50:50000"]
[09:53:49 INF] Application Certificate store type is: Directory
[09:53:49 INF] Application Certificate store path is: pki/own
[09:53:49 INF] Rejection of SHA1 signed certificates is disabled
[09:53:49 INF] Minimum certificate key size set to 1024
[09:53:49 INF] Trusted Issuer store type is: Directory
[09:53:49 INF] Trusted Issuer Certificate store path is: pki/issuer
[09:53:49 INF] Trusted Peer Certificate store type is: Directory
[09:53:49 INF] Trusted Peer Certificate store path is: pki/trusted
[09:53:49 INF] Rejected certificate store type is: Directory
[09:53:49 INF] Rejected Certificate store path is: pki/rejected
[09:53:49 WRN] Automatically accepting all client certificates, this is a security risk!
[09:53:49 INF] Added security policy http://opcfoundation.org/UA/SecurityPolicy#Basic256Sha256 with mode SignAndEncrypt
[09:53:49 INF] Added security policy http://opcfoundation.org/UA/SecurityPolicy#Aes128_Sha256_RsaOaep with mode SignAndEncrypt
[09:53:49 INF] Added security policy http://opcfoundation.org/UA/SecurityPolicy#Aes256_Sha256_RsaPss with mode SignAndEncrypt
[09:53:49 INF] Added security policy http://opcfoundation.org/UA/SecurityPolicy#Basic256Sha256 with mode Sign
[09:53:49 INF] Added security policy http://opcfoundation.org/UA/SecurityPolicy#Aes128_Sha256_RsaOaep with mode Sign
[09:53:49 INF] Added security policy http://opcfoundation.org/UA/SecurityPolicy#Aes256_Sha256_RsaPss with mode Sign
[09:53:49 INF] LDS(-ME) registration interval set to 0 ms (0 means no registration)
[09:53:49 INF] The OPC UA trace mask is set to: 0x3A5
[09:53:49 INF] No existing application certificate found. Creating a self-signed application certificate valid since yesterday for 12 months, with a 2048 bit key and 256 bit hash
[09:53:49 INF] Checking application instance certificate.
[09:53:49 INF] Creating application instance certificate.
[09:53:49 INF] Imported the PFX private key for [EAC988B443270D0EE81B79880F55DAF136BBA5DB].
[09:53:49 INF] Certificate created for urn:OpcPlc:cc8170bcfb50. [CN=OpcPlc] [EAC988B443270D0EE81B79880F55DAF136BBA5DB]
[09:53:49 INF] Application certificate with thumbprint EAC988B443270D0EE81B79880F55DAF136BBA5DB created
[09:53:49 INF] Application certificate is for ApplicationUri urn:OpcPlc:cc8170bcfb50, ApplicationName OpcPlc and Subject is CN=OpcPlc
[09:53:49 INF] Application store contains 1 certs
[09:53:49 INF] 01: Subject CN=OpcPlc (thumbprint: EAC988B443270D0EE81B79880F55DAF136BBA5DB)
[09:53:49 INF] Trusted issuer store contains 0 certs
[09:53:49 INF] Trusted issuer store has 0 CRLs
[09:53:49 INF] Trusted peer store contains 0 certs
[09:53:49 INF] Trusted peer store has 0 CRLs
[09:53:49 INF] Rejected certificate store contains 0 certs
[09:53:49 INF] Application configured with MaxSessionCount 100 and MaxSubscriptionCount 100
[09:53:49 INF] Starting server on endpoint opc.tcp://cc8170bcfb50:50000 ...
[09:53:49 INF] Simulation settings are:
[09:53:49 INF] One simulation phase consists of 50 cycles
[09:53:49 INF] One cycle takes 100 ms
[09:53:49 INF] Reference test simulation: Enabled
[09:53:49 INF] Simple events: Disabled
[09:53:49 INF] Alarms: Disabled
[09:53:49 INF] Deterministic alarms: Disabled
[09:53:49 INF] Anonymous authentication: Enabled
[09:53:49 INF] Reject chain validation with CA certs with unknown revocation status: Enabled
[09:53:49 INF] Username/Password authentication: Enabled
[09:53:49 INF] Certificate authentication: Enabled
[09:53:50 INF] Server - Start application OpcPlc.
[09:53:50 INF] Server - CreateResourceManager.
[09:53:50 INF] Server - CreateRequestManager.
[09:53:50 INF] Server - CreateMasterNodeManager.
[09:53:50 INF] MasterNodeManager.Startup - NodeManagers=5
[09:53:50 INF] Creating 1 GUID node(s) of type: UInt
[09:53:50 INF] Node values will change every 1000 ms
[09:53:50 DBG] NodeId type is System.Guid
[09:53:50 INF] Creating 1 Fast nodes of type: UInt
[09:53:50 INF] Node values will change every 1000 ms
[09:53:50 INF] Node values sampling rate is 0 ms
[09:53:50 INF] Creating 1 BadFast nodes of type: UInt
[09:53:50 INF] Node values will change every 1000 ms
[09:53:50 INF] Node values sampling rate is 0 ms
[09:53:50 DBG] NodeId type is System.Byte[]
[09:53:50 INF] Creating 1 Slow nodes of type: UInt
[09:53:50 INF] Node values will change every 10000 ms
[09:53:50 INF] Node values sampling rate is 0 ms
[09:53:50 INF] Creating 1 BadSlow nodes of type: UInt
[09:53:50 INF] Node values will change every 10000 ms
[09:53:50 INF] Node values sampling rate is 0 ms
[09:53:51 INF] Server - CreateEventManager.
[09:53:51 INF] Server - CreateAggregateManager.
[09:53:51 INF] Server - CreateSessionManager.
[09:53:51 INF] Server - Session Monitor Thread Started.
[09:53:51 INF] Server - CreateSubscriptionManager.
[09:53:51 INF] Subscription - Publish Thread 00000009 Started.
[09:53:51 INF] Subscription - ConditionRefresh Thread 0000000A Started.
[09:53:51 INF] Server - Enter Running state.
[09:53:51 INF] Server - Started.
[09:53:51 INF] OPC UA Server started
[09:53:51 INF] PLC simulation started, press Ctrl+C to exit ...
[09:53:55 INF] ChannelId 1: in Connecting state.
[09:53:55 INF] TCPSERVERCHANNEL SOCKET ATTACHED: 01FA0ABC, ChannelId=1
[09:53:55 INF] ChannelId 1: in Opening state.
[09:53:55 INF] Security Policy: http://opcfoundation.org/UA/SecurityPolicy#None
[09:53:55 INF] Sender Certificate: (none)
[09:53:55 INF] ChannelId 1: Token #0 created. CreatedAt=09:53:55.068. Lifetime=3600000.
[09:53:55 INF] SECURE CHANNEL CREATED [.NET Standard ServerChannel UA-TCP 1.4.372.56] [ID=1] Connected To: opc.tcp://cc8170bcfb50:50000/
[09:53:55 INF] ChannelId 1: Token #1 activated. CreatedAt=09:53:55.068. Lifetime=3600000.
[09:53:55 INF] ChannelId 1: in Open state.
[09:53:55 WRN] SERVER - Service Fault Occurred. Reason=BadServiceUnsupported
[09:53:55 INF] TCPSERVERCHANNEL ProcessCloseSecureChannelRequest success, ChannelId=1, TokenId=1, Socket=01FA0ABC
[09:53:55 INF] ChannelId 2: in Connecting state.
[09:53:55 INF] TCPSERVERCHANNEL SOCKET ATTACHED: 03161E6F, ChannelId=2
[09:53:55 INF] ChannelId 2: in Opening state.
[09:53:55 INF] Security Policy: http://opcfoundation.org/UA/SecurityPolicy#None
[09:53:55 INF] Sender Certificate: (none)
[09:53:55 INF] ChannelId 2: Token #0 created. CreatedAt=09:53:55.183. Lifetime=3600000.
[09:53:55 INF] SECURE CHANNEL CREATED [.NET Standard ServerChannel UA-TCP 1.4.372.56] [ID=2] Connected To: opc.tcp://cc8170bcfb50:50000/
[09:53:55 INF] ChannelId 2: Token #1 activated. CreatedAt=09:53:55.183. Lifetime=3600000.
[09:53:55 INF] ChannelId 2: in Open state.
[09:53:55 WRN] SERVER - Service Fault Occurred. Reason=BadServiceUnsupported
[09:53:55 INF] ChannelId 1: in Closed state.
[09:53:55 INF] TCPSERVERCHANNEL ProcessCloseSecureChannelRequest success, ChannelId=2, TokenId=1, Socket=03161E6F
[09:53:55 INF] ChannelId 2: in Closed state.
[09:53:55 INF] ChannelId 2: closed
[09:53:55 INF] ChannelId 1: closed
[09:53:55 ERR] TCPSERVERCHANNEL ForceChannelFault Socket=FFFFFFFF, ChannelId=2, TokenId=1, Reason=BadConnectionClosed 'Remote side closed connection.'
[09:53:55 ERR] TCPSERVERCHANNEL ForceChannelFault Socket=FFFFFFFF, ChannelId=1, TokenId=1, Reason=BadConnectionClosed 'Remote side closed connection.'
[09:53:55 INF] ChannelId 2: in Faulted state.
[09:53:55 INF] ChannelId 1: in Faulted state.

Because of these errors, I am not able to activate the endpoints.

Am I missing something?

I looked at the tutorials and noticed that they have not been updated to 2.9, there is some work to do. Still the CLI tutorial should technically work, but the output will look different: https://github.com/Azure/Industrial-IoT/blob/main/docs/web-api/tut-use-cli.md.
There was also a new version of the PLC published, but I am fairly certain the error you are seeing is the attempt to call FindServerOnNetwork (to see if the server is a fully fledged LDS) during discovery, which it does not support (because it is none), so it should be ok and not the issue here.  There are no sessions opened once discovery finishes, so once it is done, the connection is closed.  So, the question is if the discovery operation worked and the OPC Publisher created the respective entries in IoT Hub.  You can test this by using apps list in the CLI or call the GetApplications operation on the controller.  If so, then it should be possible to subscribe or call methods, etc. on it.

Okay, thanks a lot. I can see the api changed. A mix of CLI and Postman brought me further.

The biggest change is the removal of everything "job" and "worker" related from the API (Apps should not have needed them in the first place), and the "Activate" and "Deactivate" API.  For the latter, while the HTTP resource path is the same, both methods have entirely different meaning now.  They are not necessary either, you can just skip over that part of the tutorial.
For the remaining capabilities, we have end to end tests that sit on top of the HTTP API directly, and they are passing on 2.8 and 2.9.  If, however, you see other changes that we might not be covering through our end-to-end tests, I would be curious to learn about those as we can explore then how to align them to past behavior.

We decided to remove the Postman tutorial.  The new HTTP API samples to call OPC Publisher directly, or use the IoT Hub samples to call API through IoT Hub are a partial replacement.  Furthermore, the CLI sample and openapi.json might be a good starting point for how to use the web-api.


chore(main): release 0.0.8

:robot: I have created a release beep boop
0.0.8 (2024-07-05)
Miscellaneous Chores

Prepare 0.0.8 release (3a4f8f7)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.0.8
:sunflower:


Is there a way to ignore some api call in the web ui

Hi, our app is under the nginx, so nginx will send many ping requests to our app, is there a way to ignore the ping call when collect the data? Thanks:)

Hi toxie641517
if receive ping with a specific url, consider the profiler.{was}.excludeurl option in pinpoint.config.

hi, koo-taejin, thanks a lot:)


more file ops support. eg.add/revome...

Hmr only watch file change,not watch other file ops.It is necessary for hmr.

A new file created is typically empty, triggering updates for this doesn't really mean much. Triggering update on file removal has a good chance of instantly cause an error, so it's better for the user to manually reload in this case.

Thanks your explaination.I am sorry    with I don't say my idea clearly.
For the sence of add file,eg. a.vue add an new import with b.ts.hmr update will be throw error with b.ts not found.The error should be disappear after we  add b.ts.
For the sence of delete file.The user don`t get error on now.Because the deleted file isn't be watced with delete event and isn't trrigger hmr. Actuary, the user need judge this sence with why the deleted sth. is still on page.Reload will be correct way for the user.Maybe we can watch delete file and trrigger hmr with reload, it can let user to saw the error.For the best hmr, the deleted file may be isn't import by the app,it is import someone of component,should not reload whole page.so this is maybe need some work todo.


How to incorporate other datasets like Gibson

Thanks for your work, could you please inform me where is the habitat rendering engine used? As I don't see "make_sim" function in the code. So I don't know how to use other dataset like Gibson.

Rendering engine is used in utils.scenedatasets.py. You need to download other datasets in datasets/, and include the folder address of the scenes you wanna use, as "scene_kwargs" and input it to xxxEnv. Maybe you can find an example in Complete Environment Definition.

Thanks, also I wonder if it is possible to train uav to perform tasks like image-goal-navigation/object-goal navigation like in habitat-lab?

Of course.
You need to define another Env.
Use RGB camera sensor, use drone local or external estimator(such as, the distance to object) to give appropriate reward, and provide success signal while achieving target (catch object in sight).
That's just my initial and immature thought, you should carefully design and enrich the details.

Great thanks for your advice

Correct me if I'm wrong. I noticed that although VecEnvis the base class, it is not used for parallel scene training with multi-thread communication. So I wonder if the parallelization of this env is achieved by training multiple agents in one scene. If that is the case, I wonder if the training speed here is as fast as those using multi-thread parallel scene training architechture?

It is available to trained multiple agents in one scene. This simulator just inherits VecEnv's interfaces. The bottleneck of speed is rendering the vision. In habitatsim, the graphical computation is deployed on GPUs. I am not sure which one could be faster because I dont know how much CPU resources will be used.

Thanks for your patient reply. Also, what happens if collisions occur in the simulation, will the drone be bumped back of the drone just slides to a place out of the obstacle? Or will the drone stop?

As you can see in the snapshot, the drone keeps hitting the obstacle and the trajectory seems to be twisted.

In a Gym-wrapped Env (requires_grad=False), drone will be reset back to its spawn position with random states (depends on settings) if it crashes or hits on the  wall.
we have sucessfully tested the auto-reset function in these datasets wiich are scanned realworld scenes. Maybe you can plot the distance to closest obstacle to show whether it is a bug or the drone just wander near the wall (not crash).
But indeed, this simulator is primarily validated on replicaCAD datasets, because the coordinates/frame of 3D reconstruction datasets (HM3D etc.) are not well uniformed. It is troublesome to specific initial position for each scene.

Thanks, again could you tell me how the parallel computation is achieved? Since I have not found codes related to multi-process or multi-thread computation.
Although, I noticed that increasing "num_agent_per_scene" can decrease training time.

The parallel compuataion is a standrard batch computation in torch, you can find it in dynamics.py.
You can consider it as:
scene1:
 agent(1):
...
 agent(num_agent_per_scene):
scene2:
...

scene(num_scene):

The states of all agents (num_scene * num_agnet_per_scene)  are package as a batch, and used to simulate physical process.

Thanks for your illustration. I noticed that you use enu coord frame for calculation, and the translation is calculated by:
I wonder if the frame axis relationship is: y_habitat = z_enu; x_habitat=x_enu; z_habitat=-y_enu? But your calculation suggests otherwise:

I wonder if my understanding about the two used frames are incorrect.

As I increases KPv:

Thank you for your illustration. However when I use velocity command, and set vx=vy=vz=0. The yaw command leads to great changes of v_z: could you tell me how to fix this?

Please correct me if I am wrong, It seems that you use "return th.stack([roll, pitch, yaw]).flipud()" in toEuler() function, which returns[yaw pitch roll]. But the cotroller calculates v_yaw_des by " yaw_spd_des = (command[0] - self._orientation.toEuler()[2]) * self._VELOCITY_PID.p[2]", where  self._orientation.toEuler()[2] corresponds to the roll angle, not yaw angle.


GET /v1/hosts/:id/instances never returns any instances

The instances link on a container never returns any instances.  Curiously include does work (GET /v1/hosts/:id/?include=instances)

Closing, will address in APIv2


Better error messaging for patron types that don't allow requesting

What is it and who's it for?
When a patron type that doesn't have the ability to request (e.g. P29) the error message that displays is very generic and not particularly helpful. It could be interpreted as a technical error as opposed to an issue with the account.
We need to update the error messaging to be more useful in these scenarios.
@DominiqueMarshall could you propose some wording? I think it still needs to be fairly generic to cover a number of account scenarios, but in essence a CTA to speak to library enquiries. Alternatively, we could state something along the lines of 'There's been an error. Please try again, if this doesn't work, please contact library enquiries'

"There was a problem processing this request. Please try again. If the problem persists, please contact library enquiries for assistance."

@DominiqueMarshall  - Speaking to Alex I think we can make this more specific (e.g. know when it's a P29/temp account). therefore the message can be more explicit about not having a full account / being able to request. Maybe something along the lines: "You're unable to request items. Please contact library enquiries to upgrade your account"

"Your account needs to be upgraded before you can make requests. Please contact library enquiries for assistance"

If you place a hold request in Sierra while you have a patronType of 29 (self registered), then currently that request is denied with an HTTP 500 containing the following:
{
  "code": 132,
  "specificCode": 2,
  "httpStatus": 500,
  "name": "XCirc error",
  "description": "XCirc error : You may not make requests.  Please consult Enquiry Desk staff for help."
}

We don't currently handle this case, and so we return a non-specific error from the requests API. Our current handling for errors from Sierra can be found in SierraRequestsService.scala.
We need to add a new case to the sealed trait called HoldRejected for when a self-reg user encounters this error, and associate it with the error message in @DominiqueMarshall's comment above.
Here is a rough shape of the main pieces of this work:

Add the new case to HoldRejected
In the pattern match in SierraRequestsService, add a new branch for this specific error. Although the code and specificCode already exist in one of the branches, the message here is different (the comment should be updated to reflect this too).
You could choose to match on the full error message, or you could use a pattern guard to say something like if description.includes("You may not make requests").
We should make sure that the user is self-reg before we return our new error - we don't know that this error won't occur for other reasons. If they aren't, we can return an UnknownReason.
Annoyingly, our Sierra API wrapper doesn't provide functionality for this, so we need to add it to SierraSource in the scala-libs repo.
The existing method, lookupPatronExpirationDate, is a useful example here for reading the patron record. Let's have our new method return the patronType as an Int directly, rather than introducing yet another abstraction. As well as adding some tests, the PR in scala-libs will need a release note.
Once the scala-libs PR is merged, weco-bot will automatically open a PR to bump the scala-libs version on the catalogue-api repo. It's a non-breaking change, so you should be fine to merge that and continue
Where were we? In the new branch for the error, we need to call our new patronType-getting method and see whether it's 29, in which case we can return our new HoldRejected case.
There's already a method called checkIfUserCanMakeRequests, but if it were to check for patron expiry (as it currently does) and patronType then there will always be 1 unnecessary request made for each case where it's used. Hmmm.
All we need to do now is to create the appropriate response for our new error type. That's in a trait called, uh, CreateRequest.

@melanierogan As this is in Done I presume this issue should be closed?


How to get status of my transaction..?

If I am sent 0.001 btc from my wallet to another address is successfully run but how can I get the status of that transaction.
Status means transaction is in which mode like pending,success,fails...etc
Using bellow code I found the whole discription of the transaction but not found the status of that transaction
$tx = $Blockchain->Explorer->getTransaction($hash);
Is it possible to get status ..?

I assume that by status you mean number of confirmations?
Usually a transaction can safely be regarded as 'successful' if it has 3 confirmations or more. The tx object in this api client does not have a field containing the number of confirmations, but you can look at the height of the block containing the transaction:
$latest_block_height = $Blockchain->Explorer->getLatestBlock()->height;
$tx_height = $Blockchain->Explorer->getTransaction($hash).block_height;
$num_of_confirmations = $latest_block_height - $tx_height + 1;


feat: added tRPC query for fetching all products

Added tRPC query for fetching all products

LGTM, nice work @kouamschekina


[DATA] Corrputed file: https://github.com/dsfsi/covid19za/blob/master/data/covid19za_timeline_testing.csv

Which Dataset
https://github.com/dsfsi/covid19za/blob/master/data/covid19za_timeline_testing.csv
Error Description
the file is corrupted: csv not formatted correctly
Suggested fixes

fix commas

I think c740e1263f158c48cbdef2240885a4cbf35ee1a6 closes this issue.  I built a scraper for that data, but the lines with multiple sources (with extra commas) confused my scraper.  I fixed it now. I also added private and public testing to this file, which I could easily scrape and thought it may be useful?

Thank you for the quick fix! It is working now 👍


Upgraded k8s version, modified ingress && service files in order to m…

Upgraded k8s version, and modified ingress && service files in order to make it compatible with 1.22.0 version of K8.
Also, removed the name keyword from service.yaml file, it was repeated

Thanks for the contribution! Tests are blocked until https://github.com/wiremind/wiremind-helm-charts/pull/229 is merged, I let you know when it is merged.

It should be good done, could you rebase to incorporate new ci pipeline?

Closing this since everything has been rebased and merged here:

https://github.com/wiremind/wiremind-helm-charts/pull/521

thanks for your contribution!


Add .nuspec file to project

Based #25 issue we can add .nuspec file to manage Nuget pack and publish. But to use this feature we need to change release.yml file and test flow.

closed: #25


[CT-1469] init pr for short term python version issue fix

resolves #https://github.com/dbt-labs/dbt-core/issues/6203
Description
Upgrade actions/setup-python from v2 to v4.3.0 to bring in all the updates for deprecations
Defines Python Version to use when installing dependencies
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

@emmyoop curious if the fix to core alone will fix this, not breaking in any other adapter from what we see, and dbt-bigquery doesn't seem to call upon mashumaro from what I could find.

@emmyoop curious if the fix to core alone will fix this, not breaking in any other adapter from what we see, and dbt-bigquery doesn't seem to call upon mashumaro from what I could find.


[Question] Usage with webpack 2 - dynamic import

Hey guys,
I'm a bit puzzled regarding the usage of this plugin with the "new" dynamic import implementation of webpack@2.
In our project we are using this plugin. Now that we are upgrading to webpack@2, should we keep using this plugin or is the preferred way now to use import? I couldn't find any mention about this in the documentation.
Thanks a lot

We migrated from bundle-loader to dynamic import(...) successfuly. For Babel we had to add babel-plugin-syntax-dynamic-import Babel plugin.


Enhancement: Add raw query example to documentation

Hi, as you can see in my latest issue (https://github.com/Kotlin/anko/issues/622), it is possible to do raw queries in Anko-SQLite. It would be really cool if someone could add an example (Check the one out from my comment on the issue referenced) to the documentation as it is really confusing.

Not needed any longer since Anko is deprecated.


[KARAF-6274] Karaf 4.2.5 maven plugin breaks if no archives are generated

Karaf 4.2.5 includes a line checking if archiveTarGz or archiveZip is
selected, throwing an error otherwise. Unfortunately, this line breaks
our build and a few of our use cases where we do not pack our
assemblies.
For example, we have a development assembly which we have end up in a
special folder unpacked and prepared to be launched. We could, of
course, pack and then automatically unpack it but that would just be a
waste of time.
We also have some assemblies which we pack with additional files
(documentation, …) which is handled separately. Again a use-case for
assemblies unpacked by the karaf-maven-plugin.
The change in question is commit 7da5044 (pull request #811)

Sure, no problem. Done.
Should pull requests always be based on master?

@lkiesow yes please, create the PRs based on master. I'm cherry picking on other branches once reviewed.

Thanks for the review.


Different idea for tracking aged syncs

This is a followup to https://github.com/integrations/jira/pull/312 and from my conversations with @tebriel to track aged syncs in different bins to create a histogram (a distribution) within datadog.
I'm not 100% solid this will create the visualization we're looking for, but I'm hopeful.

@tebriel and I are exploring getting this data from looker. I'm going to pull that thread first.

Looker looks like it'll be a great approach. Closing this unmerged. I'll also spin up a different issue to delete the datadog metrics collection method.


Remove type(?) expression customizations

Related to #1123
When the master branch contains the Querydsl 4 changes this PR should be ported also to be applied to the master branch.

This pull request looks good, the same goes for the forward ported one.
Shall we in the future (when PRs are some time apart) look at the oldest pull request, and merge them both when 2 days have passed?
What do you think?

Yes. That sounds fine.
On 21 Jan 2015 22:52, "Ruben Dijkstra"<EMAIL_ADDRESS>wrote:

This pull request looks good, the same goes for the forward ported one.
Shall we in the future (when PRs are some time apart) look at the oldest
pull request, and merge them both when 2 days have passed?
What do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/querydsl/querydsl/pull/1125#issuecomment-70919178.

Sorry for the delay.
I think we can merge these now?

Yes.


fixing user not being able to enter value as per original bootstrap component

As per original component http://eonasdan.github.io/bootstrap-datetimepicker/#minimum-setup the user is able to edit the input field and correct the date.
We were not able to implement that as the code filters out the invalid dates before calling the onChange method.
Is it possible to let me know if we can make this change ASAP?

Any news?

Hey @malliapi,
Thanks for contributing, I've been willing to implement that feature. However I tried your branch but I couldn't get it working with the expected behavior when running your branch.
Did you try running the examples ? http://localhost:8080/webpack-dev-server/basic/index.html

I've created another PR that — I think — fixes the behavior https://github.com/quri/react-bootstrap-datetimepicker/pull/22/files


Handling of empty objects

typespec: https://github.com/Azure/azure-rest-api-specs/blob/main/specification/apicenter/ApiCenter.Management/models.tsp#L492

swagger:
    "CustomProperties": {
      "type": "object",
      "description": "The custom metadata defined for API catalog entities."
    },

tsp:
@summary("Custom Properties")
@doc("The custom metadata defined for API catalog entities.")
model CustomProperties {}

Can you confirm that an empty object is the proper tsp to model any?

If name is generated and has no properties then it's an any type.

It should be the tsp definition issue.


Logging in forever

I use this plugin for my pixelmon server so i can have the spawn in a different world. But whenever the server restarted, when people log in, the server says they logged in fine, but they get stuck on the logging in screen forever. I need to kick them from the server for them to be able to join again, and then the world looks like it is a nether world, while it is not. In the client console, it spams an error:

Any way i can fix this?

I need more to go on than this. Are you sure it's this plugin causing this problem?

yes, before we had this plugin, everything worked fine, but now when people log out in the spawn world, it sometimes causes this problem.
Sponge version: 1.8.9-1890-4.2.0-BETA-1574
ProjectWorlds version: 0.9.4

You're over 100 SpongeForge builds outdated.

I just updates to the newest version, and it still has the same issue.

Idk without any errors related to this plugin I can't make any determinations. I can guess that it's probably pixelmon related. I'll setup a pixelmon test server and see if I can replicate the problem when I have time

Does this happen with a world created by this plugin? Are there any modifiers added to the world?


Recently sold tesla, all data from teslamate is gone

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
today I sold my MY, as soon as I removed the car from the Tesla app, my car disappeared from teslamate and grafana. I see my car is still on the database under the cars table. Is there a way of getting all the data to show up on Grafana for historical reference?
Expected Behavior
No response
Steps To Reproduce
No response
Relevant log output
N/A

Screenshots
No response
Additional data
No response
Type of installation
Docker
Version
1.29.1

Moving to discussion as this is not a bug / issue with TeslaMate.


Ember 2.8.0 and getAttrFor

Have just upgraded my project from Ember 2.7.3 to 2.8.0 and it looks like ember-bootstrap-switch broke.
Error:
Uncaught TypeError: this.getAttrFor is not a function
https://github.com/Panman8201/ember-bootstrap-switch/blob/master/addon/components/bootstrap-switch.js#L145

(sigh) They must have removed getAttrFor from ember, which was technically private but useful. I use that in other addons so I'll have to look into it, but I think the bootstrap-switch plugin is dead. Unfortunately there hasn't been any activity in over a year. Not sure when I'll get a chance to debug (I no longer use Bootstrap or this addon), but would welcome a PR.

getAttrFor: function(attrs, key) {
let val = attrs[key];
return val && val.value !== undefined ? val.value : val;
},
add this function into this component and it should work.

Cool.. works, i've created a PR for it.

Fixed by #18


Use dedicated assertRegExp assertions

Looks like this assertion isn't present in the 2.12 branch.

2.12 is using same PHPUnit that master branch


build(dependencies): update typescript to 4.9.4 and its dependencies to their latest version

EX-7289
Update typescript to 4.9.4 and its dependencies to their latest compatible versions.
This task was done with @tiborux . We've not introduced any new operator or done any changes in the code yet. We should probably start using new operators, such as satisfies, in order to benefit from them. With this update we are taking advantage of several performance improvements, including some of them targeted directly at monorepo setups.

Should we update prettier to its latest version?
Version 2.8.2 brings support for some Typescript 4.9 features, like the satisfies operator or the new auto accessor syntax.

Good job! 👌

Tried removing ts-lint and upgrading to prettier 2.8.2 and everything working fine

@ajperezbau @annacv Removed tslint and updated prettier to its latest version, thanks!

This is a link to Empathy Platform Docs Portal preview in a development environment.
⚠ Only for Empathy members. We’re sorry for the inconvenience!
Check out EPDocs preview

This is a link to Empathy Platform Docs Portal preview in a development environment.
⚠ Only for Empathy members. We’re sorry for the inconvenience!
Check out EPDocs preview


Log peer config upon startup

Log peer config upon startup.
Also log peer environment variables to help troubleshoot peer config.

@Mergifyio backport release-2.5

@Mergifyio backport release-2.2


fix(opt): :bug: fix gitRepo, gitOwner

Codecov Report

Merging #30 (07a2217) into main (7accc21) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #30   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines           29        29
  Branches         5         5
=========================================
  Hits            29        29

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 7d11592...07a2217. Read the comment docs.


Add location survey to main flow

Add states to QA and Prod flow results apps before merging/deploying this

👍


404 errors subsequent to renaming folder

Describe the bug
When renaming a Folder, the backend request successfully completes, however some subsequent backend calls temporarily return 404 which displays errors in the UI.
To Reproduce
Steps to reproduce the behavior:

Using the UI, log in as<EMAIL_ADDRESS>Open a Folder
Rename the Folder using the 'Edit Title' option
The UI displays a 404 not found error for the folder, a 'There was a problem getting the Subscribed Catalogues' message (404 getting SCs when rebuilding the navigation tree), but the folder rename was successful.

Expected behavior
Subsequent, non-overlapping backend API calls should not be affected after the folder rename has completed.
Screenshots
Screenshot below shows calls involved in error:
1 (labelled in screenshot). Folder rename (PUT /folders/df7b491b-be0a-49d9-b347-4202c5f89966) succeeds
2 (labelled). Subsequent GET of folder gives 404
3 (labelled). Subsequent GET of subscribedCatalogues gives 404
4. Then subsequent GET of subscribedCatalogues succeeds
5. Then (not in screenshot) manually reloading the folder again succeeds

Desktop (please complete the following information):

OS: macOS
Browser: Chrome 98
MDM: latest develop of mdm-core, mdm-ui and mdm-resources

Additional context
Similar to gh-196 folder rename issue, mc-9723.

annoyingly i cant replicate this on my imac, but i can replicate it on the CD environment

right ... figured it out...partly its not replicating on my system cos i tweaked the way admins have their policies built so its faster .. but its because when the folder rename happens the tree is called with a request to reload the policy(no cache) due to the folder rename ... this drops the policy and recreates it, but while it does that the other endpoints then get a no-access response as there no policy for the user


How do I distinguish dp.change event from the UI itself vs calling .date([newDate])?

The dp.change event fires when clicking on the picker itself as well as when calling .date([newDate]) programmatically. Is there a way to distinguish between the two?

Please look into this issue.
Even I have a situation where I want to update two date pickers on change of an dropdown. But don't want the change event to fire when I am doing it from javascript.
Can we have this control in line with the standard where the event is not fired when anything is updated from backend. Rather it is more sane to trigger manually at backend using $.trigger rather than forcing it on the user. It seems wrong when the developer is burdened with an event like this.
For now I have worked around by using dp.update event. But it is way too hacky at the moment, will be trying to change js file myself to get this working.


Add ContextMenuManager.remove

Description of the Change
This will add a remove method to atom.contextMenu to counter the add method
With this method one will be able to remove items from the context menu by specifying a selector(s) and label(s)
Why Should This Be In Core?
Because the only alternative is to override a global object atom.contextMenu
Benefits
This will finally give the ability to remove items that clutter the context menus from packages.
Similar to the benefits of disabling keybindings that packages automatically add
Possible Drawbacks
None that I can think of.
Applicable Issues
#9429
#7739

Thanks for contributing!
Unfortunately, we're not going to merge this for a few reasons:

There is already a way to remove added context menu items: using the Disposable returned by ContextMenuManager.add
The above is the primary method of removing something that was added using the Atom API, not matching remove methods
There is no precedent for having a Disposable returned to re-add something that was removed, only the reverse

If disabling context menu items from packages is your goal, you should mirror the design of the system used in disabling keybindings from packages: https://github.com/atom/atom/pull/8130
Thanks for the hard work you put into this and the interest you have in Atom!

Ok thank you. I'll take a look at that pull request


Consider using capsule as a packager?

capsule
very handy

A great idea, thank you! This should definitely be more convenient than using Appassembler.

This has now been done for the standalone REPL and merged into master.
I'm not particularly happy with having to name the REPL capsule from command line using
mvn package -DstandaloneFinalName=rembulan

to get the (executable) capsule in rembulan-standalone/target/rembulan-capsule.x, but this will mostly be useful for releases anyway. I think it's sufficient for building from sources, and it seems to work nicely.


Implement getTransaction methods

[x] starknet_getTransactionByHash
[ ] starknet_getTransactionByBlockIdAndIndex
[x] starknet_getTransactionReceipt

Specs: https://github.com/starkware-libs/starknet-specs/blob/cad17ef605ab337981ef11b28ba5343e51ad16ee/api/starknet_api_openrpc.json#L150-L239

@gabsn  starknet_getTransactionByBlockIdAndIndex is not implemented it seems.
This is what I get, when I try to call
{
    "jsonrpc": "2.0",
    "id": 0,
    "error": {
        "code": -32601,
        "message": "method 'starknet_getTransactionByBlockIdAndIndex' not found"
    }
}

@gabsn
Some of the pertaining issues/concerns with the following methods:

starknet_getTransactionByHash:

the response does not have the attribute type and is therefore not possible to distinguish between InvokeTxn, DeclareTxn and DeployTxn. Currently, the response is deserialized to InvokeTxn by default.
All the properties of Txn are made nullable, and this is because the specs do not mention which properties are required. We might have to take a look at this in the future.

starknet_getTransactionReceipt:

In the response, we do not get status_data , block_hash and block_number, even though it is mentioned as required in the specs. Hence, we have made these attributes Nullable in the TxnReceipt class.
As per the specs, the TxnReceipt is oneOf the [INVOKE_TXN_RECEIPT, DECLARE_TXN_RECEIPT, DEPLOY_TXN_RECEIPT, PENDING_TXN_RECEIPT] and there is no evident way of classifying the response into these types of receipt. Currently, we are manually checking for the presence of certain attributes for classification, which is not good practice.


:robot: Use renovate to update kairos version

@Itxaka good point, as long as they are accessible via a github release tag or a docker container version, it should be possible


CIV-1720 Fix hint text in hearing screen

Before creating a pull request make sure that:

[x] commit messages are meaningful and follow good commit message guidelines
[x] README and other documentation has been updated / added (if needed)
[x] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-1720
Related PR
https://github.com/hmcts/civil-general-applications/pull/159
Change description
Create textarea to populate the hint text in hearing screen
Does this PR introduce a breaking change? (check one with "x")
[x] Yes
[ ] No

the pipeline failure on investigation seems to be a random issue. will monitor the issue on the judges integration branch


A small systematization of Atlassian stuff

Maybe, I should place request page link somewhere else, so if you click on the logo, you'll be redirected to the product site, not on that page?

Done.

thanks bro! 👍


[Terraform] Editing data-sources/ec_deployment.md

Description
This PR proposes some editing suggestions.
Related Issues
#78
Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring (improves code quality but has no user-facing effect)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation

@elasticmachine update branch


⚠ Bofrost.es has degraded performance

In dbdec46, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1582 ms

Resolved: Bofrost.es performance has improved in 195989f after 32 minutes.


Add hardware description docs site

Add hardware overview with electrical and mechanical details and examples for connection to some single board computer development boards.

Personally, I feel like the Hardware description should come before the API docs. (explain "What is this" before? "How do I use this?")

silly nitpick: it's better if after a . you go to a new line.
This makes the pages easier to read on github or using an editor (because lines are shorter) while not affecting the rendered document

silly nitpick: it's better if after a . you go to a new line. This makes the pages easier to read on github or using an editor (because lines are shorter) while not affecting the rendered document

As you wish.


nixos/logstash: Listen on <IP_ADDRESS> instead of <IP_ADDRESS>

Motivation for this change
Fixes #32262
Things done

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

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

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

Does this need a disclaimer somewhere?

A release note entry would be helpful.

Thank you

@srhb Thank you


[MudStepper] Use LanguageResource constants instead of hardcoded strings

@henon I think this could be merged right away, as it's just a nit.


fix some test files test time is too long

because test_norm and  test_elemwise_binary_ops use  rand_shape_2d()。 by default, rand_shape_2d generate shape(0,0) to (10, 10), so it will generate too large shape. for some operator, it will consume much time. so i  limit it .
for test_laop_5, i modify for loop from (1, 10) to (1, 5), beacuse i it's ok!  so the test time from 71.902s to 16.569s.

@haojin2 , thx for reviewing


Revert "Implement Is() for multierrors"

Reverts grafana/mimir-prometheus#460
I'll send an upstream PR, and cherry-pick those commits here, to ensure we have the same commits.

See https://github.com/grafana/mimir-prometheus/pull/462 for follow up


Fix the counter of # of drew points in the screen

the counter is buggy because I did not knew how to properly update it.
so instead of updating, the system is adding it multiple times , trying to overwrite the previous one, but it is ugly and not cool.
please check it on below image...

@brunodrugowick

it is about the issue I've commented with you. The counter of number of dots added to the screen/image is not working correctly. Take a look and you'll remember 😜


chore: upgrade polkadot-sdk to stable 2409

Uplift base-parachain template to stable2409 - Release Changelog.

Reference Upgrade polkadot-sdk-parachain-template to stable2409

Commit history: https://github.com/paritytech/polkadot-sdk/commits/polkadot-stable2409/templates/parachain

Reference [Upgrade pop-ndoe to stable2409]: https://github.com/r0gue-io/pop-node/pull/412

Status

[x] Builds node & runtime successfully.
[x] Parachain node produce blocks with PASEO local.

Note for the reviewers: The upgrade in Polkadot SDK's templates change the constant variable names which I don't add to this PR. We discussed in the last PR that we prefer the current Pop templates' constant variable name style.


feat: golangci-lint.yml added

⚠   Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

[x] I have read through the Contributing Documentation
[x] This PR is using a label (bug, feature etc.)
[x] My code is has necessary documentation (if appropriate)
[x] I have added any relevant tests
[x] This section (⚠   Pre Checklist) will be removed when submitting PR

Summary

Key Points

[ ] This is a breaking change
[ ] New or existing documentation is updated

Description
This PR adds a linter for Go in our GitHub actions workflows directory. It has default configurations which are open to discussion (please comment)
Does this close any open issues?
This closes issue #272
Current Behavior
Current behaviour is that there is no linting mechanism on our Go code.
New Behavior
The new behaviour is that the Go linter should run in the GitHub CI on each new push.
Screenshots
Include any relevant screenshots here.
Other Information
Any other information that is important to this PR.

Proof of app still working after lint fixes:
POST request received:

DB record saved:


Add ability to validate an apps locale files

This PR is the first step to being able to validate an applications locale files. The checks revolve around making sure the locale files are in sync with each and that they are using the correct plural forms. Before these checks are implemented in the frontend applications, we should create documentation that explains what to do if one of the checks fail. I've provided a brief description but proper documentation is needed.
More info in the commits ->
Screenshot 1

Screenshot 2

Trello:
https://trello.com/c/Ma1ygjNc/2567-add-automated-tests-to-check-translations-5

Hello @ChrisBAshton 👋 Thanks for reviewing the PR. Gave me some really good suggestions which helped reshape / refine the approach.
I have split some of the logic into more classes which hopefully makes what each part does a bit clearer. For example, the AllLocales class and PluralForms class.

I'd be very tempted to just strip it out and force downstream apps to use i18n keys which don't clash with the plurality keys. What do you think?

I did think about doing this to start with but went with the approach I18n-tasks uses as I thought i did make sense to exclude these groups. It wasn't a requirement, just a feature I thought may be useful. However, on reflection I think we probably should force apps not to use plural keys if the key isn't a plural as it makes the check a bit simpler and as you say, less surprising.
Enforcing apps don't use plurals keywords incorrectly will create some extra work for us when we apply this validation to the frontend apps, as there are some examples of 'lookalikes' but we can address these as they crop up and work out if they are actually meant to be plurals 😅

Thanks for the comments @ChrisBAshton, it's appreciated! The PR is ready for another 👀


unable to connect

Sorry but I am unable to connect despite a valid token on linux (tested with the pagehttps://api.slack.com/methods/auth.test/test and it's ok ).
I have the following message :
not able to authorize client, check your connection and if your slack-token is set correctly
I've tested with cli flag and env variable too, but same error.
Any idea ?
Thank's in advance

Can take a look at: https://github.com/erroneousboat/slack-term/issues/72 and see if it is the same issue?

Thank's for your response
I don't use vpn and I'm at home with ISP router.
I'll test the application tomorrow from another access point, to see if it changes anything.

No connection problems with another slack client in node.js that also uses websockets and legacy token ...
Very strange.

Is really my system, no problem at friend's on the same distribution with same version
I'll find out why

Ok, good luck. Thanks for the feedback!

If anyone runs into this again, try removing the comments from the JSON config, that worked for me.


OmniAuth patch

We just updated the DMPRoadmap codebase this week to address the OmniAuth vulnerability (CVE-2015-9284)
The instructions for implementing the patch can be found here: https://github.com/omniauth/omniauth/wiki/Resolving-CVE-2015-9284
Basically we force all OmniAuth traffic to be a POST and which forces Rails to pass along the authenticity_token.

also discovered an issue with curation_activity.submit_to_stripe (while testing OmniAuth changes in the Dryad repo) which had not been updated when the Stripe payments service was updated to use the newer charge_user_via_invoice instead of charge_via_invoice


First step to the test suite

Add basic test suite structure, closes #71
Add tests for regressions before merging #70

We'll get the following picture after merging the URLPattern PR—no regressions! 🥳
P. S. Empty rows correspond to Dropbox and CodaAPI backend tests. There is nothing to test yet (no ref object different from the base one with the url field).

How was the experience of using hTest?

It went so smoothly. Finding a good name and a description for a group of tests was more challenging than writing the tests themselves! 🙃 Inheritance of some main test properties is something—it saved me so many keystrokes. I also worked with the data object in the first iteration of tests—it was a breeze to pass data to the run function and change its behavior depending on the context. In my humble opinion, hTest is a highly well-designed product. I bow to you!


fix: missing parens

oops sorry @ericnograles --had a bad merge when adding in the async calls.

Done @daniel-deutsch-zocdoc, published new version to NPM as 0.2.1 with this change.  I'll backfill the LTS branches too.

FYI, 3 releases cut to NPM with this change


closeTab warning (When more then one pane is open, moved from original)

Please note that this is moved from the original, #5874 since that one's commits are a mess, I'm recreating it with a new fork and branch.
Summary of the Pull Request
Adds a dialog that warns the user when tab close is requested and more than one pane is open. The feature can be disabled in settings.

References
Closes #5301
PR Checklist

[X] Closes #5301
[X] CLA signed. If not, go over here and sign the CLA
[X] Documentation updated (for setting to disable warning).
[X] Schema updated.

Validation Steps Performed
Manual

I did implement a Globalappsetings Boolean just like the warning with close all tab dialog

However, if you always want to be prompted I guess we can create a new issue for it (since the current Boolean cannot support that)

@zadjii-msft We could possibly add a second boolean setting: "alwaysWarnCloseTab"

@zadjii-msft Can you please review this?

As I mentioned before, I want to make sure there's a plan here for the actual settings here before signing off. The code in this PR isn't all that tricky, but I'm worried that we'll get the necessary approvals from other team members and auto-merge this without a plan. So I'm blocking on the "make a plan" element of this PR first and foremost.

But you can block this dialog from appearing in settings. Look at my commit. @zadjii-msft

I'm more concerned about making sure that we have a comprehensive plan for all the future variants of this scenario. I don't want to just slap another setting in there that'll make it harder for us to have a comprehensive solution to all the scenarios in #6549

Thank you so much for working on this! We've chosen to take the project in a slightly different direction, and make sure that this feature is fully specified before we commit to it. We'd love to have you back, of course, in the future! :smile:


HandHoverUpdate is not working

I realize with steamvr 2.5.0 the HandHoverUpdate was not working

Got to make it work


Upgrade to eslint 2.0

Closes #9

Deprecated rules were fixed before this upgrade. We may wish to investigate another way to handle the babel-eslint dependency — airbnb's config appears to do a nice job without it.

Great! Any ETA on when another release will get cut including this change?


z coordinate in Pose tracking

Requesting info about the integration of z coordinate / depth info on the landmarks of the pose tracker.

@RaiShra1,
Can you please elaborate? Thanks!

Currently in the pose tracking, z coordinate of the landmarks do not constitute for the depth of the subject. Depth information is a critical point for my research. So wanted to know if google would release the models with depth information from the landmarks in the future?

Just adding my two cents here, I agree it would be great to get an update on where things stand with the z-coordinate.
Citing the documentation:
We are actively working on improving BlazePose GHUM 3D’s Z prediction. It will allow us to use joint angles in the feature vectors, which are more natural and easier to configure (although distances can still be useful to detect touches between body parts) and to perform rotation normalization of poses and reduce the number of camera angles required for accurate k-NN classification.```

The documentation is also a bit inconsistent with what we should / shouldn't be doing with the z coordinate currently. [Here](https://developers.google.com/ml-kit/vision/pose-detection/classifying-poses) it says we could try extending angle calculation to 3 dimensions to see if it performs better (which I'm currently coding in to my application to test), but the z-coordinate [blurb ](https://developers.google.com/ml-kit/vision/pose-detection#z_coordinate) says this isn't a true 3D coordinate, so I have my doubts about the results I'll see. Probably better to replace the KNN and surrounding logic with a neural network or random forest classifier at the same time in case there's some strange non-linear scaling going on with it. Along those lines, the extended notebook example only uses the X and Y coordinate to normalize scale, and in the KNN distance calculations are a bit strange in that they're not actually calculating euclidiean distance, but the mean and max of the distances in each coordinate direction. And then, the distance in the z-direction is weighted at 0.2 of what X and Y get. Is that because the scale of this axis is 5x greater (on average) or that it's just so unreliable that you basically want to ignore it? If we can have any reasonable degree of confidence in Z, we should be able to just use np.linalg.norm to get the distance between points and then have a much simple embedding than this 3D one? I figure while I'm adding in the 3D angles, I may as well also try that, but I really wish there was a bit more info on the dependability / scale / upcoming changes to Z, especially since it's been almost a year since this all came out.

Where I'm coming from here is that I've been trying to get this to work for a more complex (martial arts) application, but detecting poses using the current embeddings is highly inaccurate. Without a dependable z-coordinate, we effectively need at least dozens (more like hundreds) of sample images from dozens of camera angles for each pose class to get this to work. And I have dozens of classes in my application, so I've spent hundreds of hours at this point filming and labeling my current dataset of 50k images, but the output is just all over the place. I'm playing whack-a-mole with my classes... getting more and more images from more and more angles for problem classes, filtering outliers, etc, but in the end I fear that it's going to just be wasted effort if all of a sudden you provide us with a more accurate z-coordinate since that will make the embedding much more generalizable and cut down on the need for all these different camera angles for each class. It'd be better for me to be focusing on capturing more of the nuance of the moves / poses.

Anyway, any update would be super appreciated.

Hi @RaiShra1 @M00NSH0T, Thanks for sharing your concern.  We have added the right owners who can resolve your query.

Any update on this?
Thank you.

Has any progress been made on this?

Hi. Recently we updated the released model to produced additional world 3D landmarks output with origin in hips center.
More details can be found in API: https://google.github.io/mediapipe/solutions/pose#pose_world_landmarks
And in the blog: https://blog.tensorflow.org/2021/08/3d-pose-detection-with-mediapipe-blazepose-ghum-tfjs.html
If you want to calculate relative depth between landmarks, this output should be enough for you. But if you want to calculate distance to the camera, extra work needs to be done. Having both 2D and 3D landmarks and knowing camera parameters you can apply some optimization (e.g. Procrustes analysis) to determine translation in 3D space.


Improve PeerDiscovery.discoverPeers at PeerManager hearbeat

Describe the bug
At https://github.com/ChainSafe/lodestar/blob/master/packages/lodestar/src/network/peers/discover.ts#L53, I added:
    console.log("@@@ discoverPeers, maxPeersToDiscover", maxPeersToDiscover, "notConnectedPeers", notConnectedPeers.length, "time", Date.now() - start);

in my environment, it printed out
@@@ discoverPeers, maxPeersToDiscover 24 notConnectedPeers 8092 time 1012

Normally it takes ~400ms but sometimes it jumps to more than 1s.


Xinput2 Backend

A backend implementation using X11's xinput version 2 extension.
Closes #10.
Unfortunately, this implementation relies heavily on heuristics to detect the class and capabilities of hardware. There is not - as far as I can tell - a better way to go about it, and no resource exists listing the set of standard values. While I have a sizeable collection of hardware to test on, the heuristics will very likely need updating as more hardware is tried (pads with several rings, pads with sliders, tools with roll/distance/wheel/slider, airbrushes..., etc.).
Once implementation is finished (not yet, some breaking changes are still pending), I would greatly appreciate testing help to ensure this detects the full capabilities on all hardware. I will probably put out a call for this when the implementation is ready prior to making a release :3

[x] Tool, Pad, Tablet hardware listing
[x] Tool events
[x] Pad events
[ ] Hardware Add/Remove Events
[X] Timeout based Out events
[ ] Mouse emulation
[X] Remove the use of Xinput 1.x APIs which "clients are requested to avoid mixing" with Xinput 2 (whoops)

Some necessary XI1 calls have no XI2 counterpart. I'm confused.

[ ] Refactor (a lot of this was written while I was pulling my hair out in utter bafflement)

Good news, after digging through several driver implementations (xf86-input-libinput, xf86-input-wacom, xf86-input-evdev, xwayland), it seems things are more standardized than is exposed through public documentation, and the things that aren't standardized are presented through driver-specific properties. These have been implemented and it's all around more scrumptious, however the heuristics must unfortunately remain for when the drivers aren't available. Getting there!

Thank you for the interest ^^. I am very pleased to see this crate put to use :3
I got a bit caught in the weeds of implementation details on this PR, so I haven't touched this in a bit. I will say having someone in need of this feature will be great motivation for getting it done!
I believe I have most of the protocol and hardware/driver-specific quirks mapped out in my notes, which was the hard part.


fix: css chunk loading missing chunkIds

Summary

Test Plan

Require Documentation?

[ ] No
[ ] Yes, the corresponding rspack-website PR is __

!canary

!canary


 component

Description
Breadcrumbs component is navigational component placed on top of the page for showing hieararchy of current page.
Visual style

Zeplin: https://zpl.io/anYQLWO
Interactions

Link will underline itself when hovered

Additional information

Last item is also URL

Accessibility

https://www.w3.org/TR/wai-aria-practices/examples/breadcrumb/index.html
https://gist.github.com/jonathantneal/4037764

Functional specs

Not sure how it works with React, but it would be great to have correct semantics for this component, so it shows nicely in Google. More info can be found  or here or here.

Google preview

@darkwindcz


removed filter/blur properties

fixes #6751

@straight-shoota this is your code, is this fine to remove?

It's not an essential feature. But blurring the background behind the modal window has some visual significance. I wouldn't simply remove it, but look for alternative ways to achieve this.
I haven't looked at the exact issue in detail (and won't be able to for another week). But I'm sure there should be a way to have a blurred background when the modal is active without any performance issues.


multi-processor zfile builder

What this PR does / why we need it:
Increase zfile compression speed by utilizing multi-core parallelism.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Please check the following list:

[x]  Does the affected code have corresponding tests, e.g. unit test, E2E test?
[ ]  Does this change require a documentation update?
[ ]  Does this introduce breaking changes that would require an announcement or bumping the major version?
[ ]  Do all new files have an appropriate license header?

awesome


v4 Action statuses change to a single "status" property

Hey, first of all, huge thanks for this pckg! Good job!
Problem
Quick question/suggestion. What's the background of this change?

Single status string property is a step back in terms of DX imho. JSX with string conditions (like status === 'executing') looks messier and more code to write. Noticed, I started manually creating isExecuting consts (and some for other statuses) all over the project.
Old api vs New api
const { status } = useAction(action);
const isExecuting = status === 'executing';

const { status, isExecuting } = useAction(action);

Extra
Tanstack Query Api could be great inspiration (for naming things as well) - I assume community got really used to it in past years.

I don't know about the actual background of this change but one issue with hasExecuted, isExecuting, hasSucceeded and hasError is that they are not discriminating.
From a static analysis standpoint they could be all true at the same time whereas status can only be in one state.

one issue with hasExecuted, isExecuting, hasSucceeded and hasError is that they are not discriminating.

Yes, this is exactly why I changed the API. Also, I think it's way cleaner to have just one status key.

Noticed, I started manually creating isExecuting consts (and some for other statuses) all over the project.

You can define an util function like this:
export function isExecuting(status: HookActionStatus): status is "executing" {
  return status === "executing";
}

Then import and use it in your Client Components. It's almost the same as having pre-v4 isExecuting key, the only difference is that you have to pass the action status to it.

Hope the code snippet above solved your problem. If you have any additional questions about this implementation, feel free to leave a comment here, and, if needed, I'll reopen the issue. Thank you!


Strange prediction results of SECOND model

I ran the SECOND model to transfer learning(start at SECOND model to learning kitti data set 20epoch, + 180 epoch to my original dataset(Lidar Sensor, Parking Car))
This is my result image to detection 3D-Box

How can I fix the model?

It seems that the model you trained works, since there are two matching predicted bboxes and ground truth bboxes in you results. As for the other predicted bboxes, can you check their predicted corresponding scores?

How can check predicted corresponing score?

you can print the detection results of https://github.com/open-mmlab/mmdetection3d/blob/c7272063e818bcf33aebc498a017a95c8d065143/mmdet3d/models/detectors/voxelnet.py#L104 to check the detected objects score.

Close due to inactivity.


Clang tidy: check capitalization

Enable a couple of checks for readability-identifier-naming that cover a majority of the code. Much more can be done about this topic, but I think this is a good start.
Respect, your capitalization of the checked identifiers was almost perfectly consistent already 👍

I like this. As I just saw this one the review. Could you open another pr where you replace the two lamdas that were touched here with auto ... and perfect forwarding.

Could you open another pr where you replace the two lamdas that were touched here with auto ... and perfect forwarding

Well spotted! PR #13 is open.


Low training accuracy and Errors when test videos

Hi, Mr. Xiong!

Now I can train the TSN on the dataset hmdb51_split_1_rgb with the default parameters using bash scripts/train_tsn.sh ucf101 rgb . After 2500 iteration, the train net loss = 0.17426, the test accuracy = 0.530679 and test loss = 2.30292.

Which parameters may affect this test accuracy, i.e., how could I get a similar accuracy as your work?

I met some problems when I test the TSN Network. As the file Readme, I input the code
python tools/eval_net.py hmdb51 1 rgb /raid/Public/zhoup_DS/HMDB51/TSN_hmdb51_frames/  models/hmdb51/tsn_bn_inception_rgb_deploy.prototxt models/hmdb51_split_1_tsn_rgb_reference_bn_inception.caffemodel --num_worker 3 --save_scores ZhouExample/hmdb51_split1_test.txt

However, it goes wrong with the following output:
....
6400 videos parsed
6600 videos parsed
frame folder analysis done
[libprotobuf ERROR google/protobuf/text_format.cc:245] Error parsing text-format caffe.NetParameter: 19:12: Message type "caffe.LayerParameter" has no field named "bn_param".
WARNING: Logging before InitGoogleLogging() is written to STDERR
F1216 09:33:18.682155 27370 upgrade_proto.cpp:79] Check failed: ReadProtoFromTextFile(param_file, param) Failed to parse NetParameter file: models/hmdb51/tsn_bn_inception_rgb_deploy.prototxt
*** Check failure stack trace: ***
[libprotobuf ERROR google/protobuf/text_format.cc:245] Error parsing text-format caffe.NetParameter: 19:12: Message type "caffe.LayerParameter" has no field named "bn_param".
......
*** Check failure stack trace: ***
WARNING: Logging before InitGoogleLogging() is written to STDERR
F1216 09:35:24.345011 27464 common.cpp:169] Check failed: error == cudaSuccess (10 vs. 0)  invalid device ordinal
....
When I check the cuda using TSN_master/lib/caffe-action/build/install/bin$ ldd caffe | grep cuda , it shows like
libcudart.so.7.5 => /usr/local/cuda/lib64/libcudart.so.7.5 (0x00007f387598e000)
libcurand.so.7.5 => /usr/local/cuda/lib64/libcurand.so.7.5 (0x00007f3872125000)
libcublas.so.7.5 => /usr/local/cuda/lib64/libcublas.so.7.5 (0x00007f3870846000)
libcudnn.so.5 => /usr/local/cuda/lib64/libcudnn.so.5 (0x00007f386ccfb000)
Have you met the errors? Could you give me some advice about that?
Thank you and looking for your reply. Best wishes!

See this

https://github.com/yjxiong/temporal-segment-networks/wiki/Frequently-Asked-Questions#testing-tsn

You are using another version of Caffe in your testing scripts. Please use the bundled Caffe for the Python scripts.

Thanks for reply! The Caffe version is downloaded in your github https://github.com/yjxiong/caffe and carried out according to the build_all.sh. I don't know whether the "bundled Caffe" is this one.
Thanks~

Look at this sentence in you log
19:12: Message type "caffe.LayerParameter" has no field named "bn_param".

This means the underlying Caffe libraries you are using in your Python scripts do not have the BNLayer in the bundled Caffe. This means you are linking to another version of Caffe installed on your machine.

According to your advice, in the file ~/.bashrc , I add
PYTHONPATH= /home/zhoupeipei/action-code/2016ECCV_TSN_master/lib/caffe-action/python:$PYTHONPATH and
LD_LIBRARY_PATH=/home/zhoupeipei/action-code/2016ECCV_TSN_master/lib/caffe-action/build/lib:$LD_LIBRARY_PATH.
After I rebuild the caffe-action based on build_all.sh, the errors disappear when I test videos again!
Thanks for your patience! It is very kind of you!

Looks good, normally the test scripts will then report the results shown in the paper.

@Xiao-G1023
have you got the correct results?

@yjxiong Dear Mr. Xiong, now I could test and train RGB_split_1 of hmdb51.  I got the rgb_score_file and the flow_score_file as ReadMe file. Here, I found that the SCORE_FILE is a file_name string instead of a directory path and the test result is a .zip files.
At last, using python tools/eval_scores.py RGB_SCORE_FILE.zip FLOW_SCORE_FILE.zip --score_weights 1 1.5, I obtain the accuracy of  hmdb51_split1 is 69.477124%.
Thanks for your attention~  Best wishes to you and to my experiments.

The score file is actually storing two numpy arrays in a compressed way.
See here for how they are stored
https://github.com/yjxiong/temporal-segment-networks/blob/master/tools/eval_net.py#L136
Currently, you can look at the tools/eval_net.py for ideas on how to analyze single videos. The script for this purpose is still on its way.

OK! Thanks a lot! I am very grateful for your kindly help!
祝熊老师及您的团队一切顺利，更上一层楼！

Good luck with your experiments!
Closing this. Please feel free to reopen it if you meet any further problem.

hello，i met with the same problem，and set the LD_LIBRARY_PATH and PYTHONPATH environmental variables. However,when i rebuild caffe-action, a new problem:'no module named caffe', do u met the problem?


wrong link is replaced by correct link#5436

as per user issue # 5436, I have replaced the wrong link to new link
wrong link
https://msdn.microsoft.com/library/windows/hardware/aa394505
new link
https://docs.microsoft.com/en-us/windows/win32/cimwin32prov/win32-videocontroller
also, i added the related link for reference

Ref. #5436


added drilldown module

Since the drilldown is in the demos it should probably be included.
Also HighCharts doesn't give any errors when you pass in a properly formatted drilldown series.

added .DS_store to ignore, that should be deleted from repo

We're trying to do something like this instead: https://github.com/MaazAli/Meteor-HighCharts/issues/12
Each additional functionality being a separate package that depends on this one.

@ClarenceL take a look at https://github.com/MaazAli/highcharts-gauge for an easy way to built the package. I dont know if @MaazAli wants to publish it under his name ... ?

Gotcha, ya I saw that too


Authorization issues with ChallengeQuestionManagementAdminService::setUserChallengeAnswers operation

We are allowing to set the challenge answer for a user if,

authenticated user and the user sent in request payload is the same.
authenticated user has /permission/admin/manage/identity permission.

Seems neither of these checks happens properly.
The authorization check should be done for the logged in user instead of the authenticated user [1].
Also, if the user is authenticated with the secondary user store domain appended to the user name, the user validation again fails.
[1] - https://github.com/wso2-extensions/identity-governance/blob/v1.1.42/components/org.wso2.carbon.identity.recovery/src/main/java/org/wso2/carbon/identity/recovery/services/ChallengeQuestionManagementAdminService.java#L191-L200

https://github.com/wso2-extensions/identity-governance/pull/259
https://github.com/wso2-extensions/identity-governance/pull/258


Shipping kernel modules in torcx

I think it would be a great idea to be able to ship kernel modules in torcx modules. Currently one can package the .ko file and insmod that in a systemd service, but complex modules (like ZFS on Linux or WireGuard) are a pain because of inter-module dependencies only properly handled by modprobe. modprobe doesn't work since the modules are not in the right path to be picked up by modprobe.
If we could extend torcx to symlink modules (difficult due to R/O /lib) or get modprobe to pick up torcx module that would be pretty cool. I don't have a very concrete plan for this yet, but just wanted to throw this out here to start a discussion which will hopefully lead to a better experience when deploying kernel modules on CoreOS.

This is something we'd like to explore.  https://github.com/coreos/bugs/issues/2215 has additional context in passing.

If it could somehow support DKMS, it gonna get very useful.
Ship kernel image once, and let CoreOS/torcx trigger DKMS on kernel upgrade.

I would actually like a simpler build mechanism more than DKMS, I don't like to compile modules on live nodes but would rather have a CI compile them beforehand and run acceptance tests and then shipping the same tested prebuilt torcx package. That's what I'm currently doing, it just involves a very weird bootstrap system and 3 layers of virtualization and lots of custom code and wrappers.

Currently one can package the .ko file and insmod that in a systemd service, but complex modules (like ZFS on Linux or WireGuard) are a pain because of inter-module dependencies only properly handled by modprobe. modprobe doesn't work since the modules are not in the right path to be picked up by modprobe.

Are you aware off: https://coreos.com/os/docs/latest/kernel-modules.html ? You can create a overlay to get around the RO issue, but I'm not sure if that helps, as modules.dep still needs to be updated.

If it could somehow support DKMS, it gonna get very useful.

I just stumbled on https://gist.github.com/dm0-/0db058ba3a85d55aca99c93a642b8a20 by @dm0- (CoreOS dev?), which could work for Wireguard but still a "lot" of custom code. + I will prefer access to the WireGuard tools from CoreOS somehow. It get messy quickly.

I would actually like a simpler build mechanism more than DKMS, I don't like to compile modules on live nodes but would rather have a CI compile them beforehand and run acceptance tests and then shipping the same tested prebuilt torcx package.
For me, that seems like a too complex solution. I would rather build it on the nodes, but WireGuard is also very small so that wouldn't take very long to do.

That's what I'm currently doing, it just involves a very weird bootstrap system and 3 layers of virtualization and lots of custom code and wrappers.

3 layers? That seems like a lot. CoreOS development container + systemd-nspawn coudln't that work?

I'm running ZFS on Linux with zvols and WireGuard, that requires a lot more than just kernel modules. I'm also not doing the overlays because that breaks updates, so I keep all the modules inside a torcx package. Layer-wise it's CoreOS developer with systemd-nspawn inside CoreOS inside KVM inside Kubernetes Pods (builds are automated).

If we could extend torcx to symlink modules (difficult due to R/O /lib) or get modprobe to pick up torcx module that would be pretty cool.

@lorenz if this the only step you are currently missing, I may have a few ideas involving bindmounts and overlays. How does your wireguard torcx addon looks like? Can you post the image somewhere (also mentioning the corresponding OS version)?
The general problem of providing a torcx-based solution to automatically build and prepare kernel modules (a-la DKMS) is a bit more complex though and we still haven't a proper plan to solve it.

You can get my ZFS module for 1520.8.0 from here. The build process is a giant hack and there are a lot of wrappers and custom stuff in there but it gets the job done much more reliably than the overlay mounts on /lib/modules. Look at the patched systemd services to see why not having modprobe and depmod support is making shipping kernel modules as torcx modules more complicated.

Two more things while I am at it:

Add the torcx path to the PATH that the default SSH shell is running with. I have to add a file to /etc/profiles.d to make that happen. Also make that path work for sudo, currently the wrapper scripts get themelves root because sudo torcx_binary doesn't work because the path is not included.

Pretty much everything that is not Go or Rust or tirivially simple requires dynamically loaded libraries. My wrapper currently deals with that (same as with the sudo workaround), but longer-term there should probably be a more ergonomic solution. Not sure yet what that will be though, maybe torcx should not link the binaries but generate wrappers in TORCX_BINDIR.

Also forgot the WireGuard module which is here.

A (very late) reply to @lorenz's comment:

items 1 and 2 are strictly related, and at the moment we are not confident enough to just expose torcx bindir into PATH. We explicitly documented it into devdocs. This is also tracked at https://github.com/coreos/torcx/issues/53.
item 3 is being taken care of via
I'm keeping this ticket open to later explore a more ergonomic approach to expose kernel modules (i.e. a generic to replace your zfs-import-scan.service)