Cannot add Qt 6.4.1 Android Arm64_v8a as a Qt Kit into QtCreator

Description
When trying to add Qt 6.4.1 Android Arm64_v8a as a Qt Version for Kit into QtCreator, I am getting an error:
The qmake executable /opt/Qt/6.4.1/android_arm64_v8a/bin/qmake could not be added: "/opt/Qt/6.4.1/android_arm64_v8a/bin/qmake" produced no output: /opt/Qt/6.4.1/android_arm64_v8a/bin/qmake: 7: /Users/qt/work/install/bin/qmake: not found
However, via an official installer I can add this Qt version without any issues and I can compile and deploy everything as usual.
To Reproduce

aqt install-qt -O /opt/Qt linux android 6.4.1 android_arm64_v8a --autodesktop
Open QtCreator 8.0.2, add /opt/Qt/6.4.1/android_arm64_v8a/bin/qmake as Qt Version.
See the error.

Expected behavior
Works the same as for the official installer.
Desktop:

OS: Ubuntu 22.04
aqt version: aqtinstall(aqt) v3.0.2 on Python 3.10.6 [CPython GCC 11.3.0]

UPD:
I don't have such problem with Qt 6.4.0 installed via aqtinstall.

Duplicated with #621

@miurahr
It's not a duplicate.
This is the aqt bug, because the behavior differs from the official installer.

#621 report  installation of  android_arm64_v8a with --autodesktop installs  "windows" binary because The Qt foundation published android binary with metadata only with windows SDK, and they close its issue ticket as won't fix.
It is why QtCreator on linux failed to  execute qmake,  but it is windows binary qmake.exe.

#621 report installation of android_arm64_v8a with --autodesktop installs "windows" binary because The Qt foundation published android binary with metadata only with windows SDK, and they close its issue ticket as won't fix.
It is why QtCreator on linux failed to execute qmake, but it is windows binary qmake.exe.

What I am missing here is that why then Qt 6.4.1 android_arm64_v8a from the official installer works in QtCreator but from aqt not?

You can do work around in a way that previous versions.

I completely removed Qt 6.4.1 installed with aqt and reinstalled it using 2 commands:
$ aqt install-qt -O /opt/Qt linux desktop 6.4.1
$ aqt install-qt -O /opt/Qt linux android 6.4.1 android_arm64_v8a
It did not help. I'm still getting the same error in QtCreator.
if I check cat /opt/Qt/6.4.1/android_arm64_v8a/bin/qmake , I see it's using some strange location to execute qmake /Users/qt/work/install/bin/qmake , while qmake from Qt installer points to the correct desktop qmake.

Upd:
I just manually replaced non-existent /Users/qt/work/install/bin/qmake with /opt/Qt/6.4.1/gcc_64/bin/qmake and it does work perfectly fine now.

@vkuznetsovgn Thank you. Now I understand a report.
when running command I observed
$ aqt install-qt  linux android 6.4.1 android_arm64_v8a --autodesktop
INFO    : aqtinstall(aqt) v3.0.2.dev11 on Python 3.10.6 [CPython GCC 11.3.0]
INFO    : You are installing the android version of Qt, which requires that the desktop version of Qt is also installed. Now installing Qt: desktop 6.4.1 gcc_64
WARNING : Specified Qt version is unknown: 6.4.1.
INFO    : Downloading qttools...
INFO    : Downloading qtbase...
INFO    : Downloading qtdeclarative...
INFO    : Downloading qtsvg...
INFO    : Redirected: ftp.yz.yamagata-u.ac.jp
INFO    : Redirected: ftp.yz.yamagata-u.ac.jp
INFO    : Redirected: ftp.yz.yamagata-u.ac.jp
INFO    : Redirected: ftp.yz.yamagata-u.ac.jp
INFO    : Finished installation of qtsvg-MacOS-MacOS_12-Clang-Android-Android_ANY-ARM64.7z in 3.58569239
INFO    : Downloading qttranslations...
INFO    : Finished installation of qttools-MacOS-MacOS_12-Clang-Android-Android_ANY-ARM64.7z in 5.86575145
INFO    : Redirected: ftp.yz.yamagata-u.ac.jp
INFO    : Downloading qtbase...
INFO    : Finished installation of qtbase-MacOS-MacOS_12-Clang-Android-Android_ANY-ARM64.7z in 10.55020444
INFO    : Redirected: ftp.yz.yamagata-u.ac.jp
INFO    : Downloading qtsvg...
INFO    : Finished installation of qttranslations-MacOS-MacOS_12-Clang-Android-Android_ANY-ARM64.7z in 10.60716895
INFO    : Redirected: ftp.yz.yamagata-u.ac.jp
INFO    : Finished installation of qtdeclarative-MacOS-MacOS_12-Clang-Android-Android_ANY-ARM64.7z in 15.21983997
INFO    : Finished installation of qtsvg-Linux-RHEL_8_4-GCC-Linux-RHEL_8_4-X86_64.7z in 4.99439451
INFO    : Downloading qtdeclarative...
INFO    : Downloading qttools...
INFO    : Downloading qttranslations...
INFO    : Redirected: ftp.yz.yamagata-u.ac.jp
INFO    : Redirected: ftp.yz.yamagata-u.ac.jp
INFO    : Redirected: ftp.yz.yamagata-u.ac.jp
INFO    : Finished installation of qtbase-Linux-RHEL_8_4-GCC-Linux-RHEL_8_4-X86_64.7z in 13.91680251
INFO    : Finished installation of qttranslations-Linux-RHEL_8_4-GCC-Linux-RHEL_8_4-X86_64.7z in 8.86879561
INFO    : Downloading qtwayland...
INFO    : Downloading icu...
INFO    : Redirected: ftp.yz.yamagata-u.ac.jp
INFO    : Redirected: ftp.yz.yamagata-u.ac.jp
INFO    : Finished installation of qttools-Linux-RHEL_8_4-GCC-Linux-RHEL_8_4-X86_64.7z in 25.95157577
INFO    : Finished installation of qtwayland-Linux-RHEL_8_4-GCC-Linux-RHEL_8_4-X86_64.7z in 21.82308230
INFO    : Finished installation of icu-linux-Rhel7.2-x64.7z in 19.52644032
INFO    : Finished installation of qtdeclarative-Linux-RHEL_8_4-GCC-Linux-RHEL_8_4-X86_64.7z in 33.75186424
INFO    : Patching Qt/6.4.1/android_arm64_v8a/bin/qmake
INFO    : Patching Qt/6.4.1/gcc_64/bin/qmake
... (continues)

then
$ strings 6.4.1/android_arm64_v8a/bin/qmake|grep work
/Users/qt/work/install/bin/qmake -qtconf "$script_dir_path/target_qt.conf" $*

Here should be some thing like
/opt/Qt/6.4.1/gcc_64/bin/qmake -qtconf "$script_dir_path/target_qt.conf" $*

It may need to patch  android qmake binary, it is not necessary for Qt 6.4.0 and before

I think the logic that needs to change is here:
https://github.com/miurahr/aqtinstall/blob/2c8b23d649a0c856dd575cf376438b3e9edf4beb/aqt/updater.py#L38-L39
On Linux, before Qt 6.4.1, the path that we needed to patch was /home/qt/work/install. Now, the qmake file looks a lot more like the Mac version, in which we need to patch the patch /Users/qt/work/install/bin/qmake.
Personally, I think it makes sense to just patch both paths on all OSes, so we are not affected by strange choices by the Qt devs like this.

@vkuznetsovgn Can you please try #627, and let us know if it fixes the problem for you with android_arm64_v8a?
OT: I'm having some difficulty with android_armv7, and right now it looks like it might not work anywhere other than Windows.


Uses functions from asterism

Fixes #19

@bonniegee should this PR be against development?

@bonniegee if you do pre-commit install you'll get the linting stuff happening in your git commit flow...


Cannot run _main.sh on Ubuntu/Debian

$ sh ./deployment/_main.sh
./deployment/_main.sh: 5: source: not found
Please check the details before proceeding
AWS Account:
AWS Region for deployment :
Please check the Karpenter version you have selected is available at
https://karpenter.sh
Also please check #Experiencing Issues# section before proceeding.
Casesenstive Press Y = Proceed or N = Cancel
Response:
Y
./deployment/_main.sh: 13: [[: not found
Cancel deployment

Please check the terminal path should be as per documentation. Please re-open if you still experience the issue


Added Code Geass OP/ED

Fixes #{144}
This PR is for [Code Geass].

[ ] This issue is for enhancement/bug fixing.
[x] This issue is an addition to JSON database.

[x] Pull Request Title is of form Added [Anime Name].

[ ] This addition is already on ongaku.

[ ] This PR is an improvement to current addition.

[x] I have checked my addition locally and it is working and won't break existing entries.
[x] I have gone through Contribution Guidelines.
[x] This pull request will be closed if I failed to update it even once in a week.

Fixes #144

This PR is for Code Geass.

[ ] This issue is for enhancement/bug fixing.
[x] This issue is an addition to JSON database.
[x] Pull Request Title is of form Added [Anime Name].
[ ] This addition is already on ongaku.
- [ ] This PR is an improvement to current addition.
[x] I have checked my addition locally and it is working and won't break existing entries.
[x] I have gone through Contribution Guidelines.
[x] This pull request will be closed if I failed to update it even once in a week.>

Make necessary changes @vanguard07


Support analysis for queries inside lambda expressions

I'm working with VS Code and the analyzer is working fine so far except it doesn't seem to analyze the queries inside lamda expressions.

Hi there @francesconi, that's an interesting edge case, I will have a look!

Fixes as of v3.18 of the analyzer, update the nuget package and it should now be able to detect the queries.
If the problem is resolved, please close the issue 😄

It's working great! Thank you for providing a solution that fast 🚀


x/tools/gopls: errors with non-existent directory in a multi-folder workspace

What version of Go are you using (go version)?

$ go version
go version go1.15.1 darwin/amd64

$ gopls version
golang.org/x/tools/gopls 0.4.4
<EMAIL_ADDRESS>h1:8djGYsaZ0ByP0vaXg4T+mnyfDcHpWKSZ+tpQSGv9ahk=

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users//Library/Caches/go-build"
GOENV="/Users//Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users//go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users//go"
GOPRIVATE="github.com/OceanCodes"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.15.1/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.15.1/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/b_/c6yh0ksn63d1yy192x2p9f4c0000gn/T/go-build483292181=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

Create a multi-folder workspace with one of the folders missing and others having go modules projects.
Open it in VS Code.
gopls will error:

Error loading workspace folders (expected 10, got 9)
failed to load view for file:///path/to/missing/folder: invalid workspace configuration: no such file or directory

What did you expect to see?
It should just ignore that folder. It's common to end up in a situation like that when working on multiple branches.
A workaround for now is to create that directory, leaving it empty. But that has to be done again from time to time when moving between branches.
What did you see instead?
It errors.

In this case, does gopls fail to work or does it just show an error message? I haven't tested this out yet, but based on the code it should just show an error and then continue to work for the other folders in the workspace.

It does seem to work. But I remember I had an issue with it also not working, but that might have been caused by something else as I'm not currently able to reproduce it not working. If I'll manage to reproduce a case where this specifically causes something to not work, I'll update here.

I just had a workspace where I had a missing folder and gopls will not respond to changes to any go file and refresh build errors when that happened.
Using gopls 0.5.1 and VS Code 1.50.0.

Thank you for the update, but unfortunately it's difficult to investigate without a log or a concrete repro case. Instructions on how to capture a log can be found here: https://github.com/golang/tools/blob/master/gopls/doc/troubleshooting.md#capturing-logs.

Closing, as there has not been any activity on this issue for over a month.


Improve releasing during v0.10.3

Looks like the sdk:go:lint failure can be reproduced by running a CLI v0.10.3 against an engine v0.10.2.
❯ docker ps
CONTAINER ID   IMAGE                               COMMAND                  CREATED         STATUS         PORTS     NAMES
c84ca47fb608   registry.dagger.io/engine:v0.10.2   "dagger-entrypoint.s…"   5 minutes ago   Up 5 minutes             dagger-engine-a0d01c49542e764e
❯ export _EXPERIMENTAL_DAGGER_RUNNER_HOST=docker-container://c84ca47fb608

❯ dagger-0.10.3 run --progress=plain ./hack/make sdk:go:lint
...
2: ./hack/make sdk:go:lint
2: [11.2s] Error: could not read updated file: open sdk/go/.changes/header.tpl.md: no such file or directory
2: [11.2s] exit status 1
2: ./hack/make sdk:go:lint ERROR: exit status 1

315: start dagger-entrypoint.sh
315: > in sdk > go > generate > service ri60dqkl8fdqa
exit status 1

Ohhhh. This is caused by #6909 (cc @sipsma)
If a new client connects to an old engine, then the engine won't send the merge value (which defaults to the zero value), so by default this would wipe the directory.
I don't think this is an issue - we don't support version compat between different dagger client/server versions.


How to Deploy Simple ERC20 Token on Polygon Mumbai using Defender Relay by OpenZeppelin

Topic Suggestion
How to Deploy Simple ERC20 Token on Polygon Mumbai using Defender Relay by OpenZeppelin
Writing sample(s):

Blcockchain series
personal blog

Proposal Submission
Proposed title of the article
How to Deploy Simple ERC20 Token on Polygon Mumbai using Defender Relay by OpenZeppelin
Proposed article introduction
A lot of people in tech are currently learning web3 especially smart contract development with solidity as their main programming language. In this article, I intend to write how to write an ERC20 Token and deploy it on Polygon Mumbai Testnet using the Relay Defender by Open-Zeppelin, hardhat as a development environment, and Alchemy's RPC for PolygonMumbai
Key takeaways

setting up a hardhat development environment and installing required dependencies
getting an RPC Link on Alchemy and API_KEY & Secret_Key on Relay Defender
configuring the RPC link, API_KEY & Secret_Key on hardhat.congif file & compiling
writing ERC20 Token
deploying ERC20 token successfully to Relay Defender and confirming transactions on Polygonscan

Article quality
the article is unique as the blockchain space is still growing and there are few resources linked to it. The article will include useful links and screenshots to help a user
References
I'll use the sites & documentation of hardhat, Alchemy, OpenZeppelin and Relay Defender

Thank you for submitting your topic suggestion. There seems to be many articles that already cover this topic. Specifically this. Can you tell me how yours will build upon them and be unique to add value to our readers?

I'm going to write the smart contract using ERC20 contracts and deploy it to Mumbai test-net using the Alchemy RPC and the Open-zeppelin's Defender Relay that specific sample article only deploys to Mumbai but doesn't use Defender Relay

@jebitok-dev
Closing due to inactivity - if you'd like to continue working on the topic. Please let us know.


Install cargo-generate Error!

When I use the command cargo install cargo-generate, I get an error:
couldn't read .cargo/registry/src/liquid-core-0.26.0/src/parser/parser/grammar.pest: No such file or directory(os error 2)
But, I find the file grammar.pest in the parent directory, like this:
.cargo/registry/src/liquid-core-0.26.0/src/parser/grammar.pest
So, I make a new directory parser under the old parser and move grammar.pest to there.
Although cargo install cargo-generate finally works, will this cause problems?

good question, but it seems to be a problem with liquid and not cargo-generate. We have no way of influencing the build of the liquid crate due to how Cargo works.

@BabaiLi could you please try to install via cargo install --locked cargo-generate and see if the issue persists?


fix: remove multiple call to locker

Type of Change

[x] Bugfix

Description

If we have hyperswitch_token as well as pmd, we get pmd against that hyperswitch token from basilisk.
We used to update the pmd again against that token w.r.t wallets and bank transfers, as both the data will be same, we get error.
Motivation and Context

Remove duplicate storage in locker
How did you test it?

Manual
Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed submitted code

@Sangamesh26 if we have hyperswitch_token and also PMD , we can choose to update PMD again ?


Fix variable names in README.md cue section

Setting vertical_vert_cue and vertical_vert_cue_color do not work, but vertical_cue and vertical_cue_color do.
Just a tiny detail to correct. I was struggling to figure out why it wasn't working and I almost filed an issue, but this fix is easiest.
Big fan of this prompt, by the way. I tried just about all of them but this is the only one that I wanted to keep.

Thanks, this was a good catch!


dbeaver/pro#2752 Add a new setting to Disable (grey out) the Automatic updates check checkbox in Preferences

Disable all automatic update operations

verified


Admin Product copy returns 404

Bug report
Issue Description
After upgrading bagisto to master branch. I can't use copy action from Admin Product.
Preconditions
1. Bagisto version 1.2.0.
2. commit id : 62b3aa99df1796a8cc7aff13ec418a18aae1814e

Steps to reproduce
1. Go to Admin , Select Catalog, Products
2. Select a product you want to copy
3. On actions column click copy icon

Here's an exception thrown when 404 appears
No query results for model [Webkul\Attribute\Models\Attribute].

Expected result

From website download version of bagisto, clicking copy would go to product creation with copied values from original product.

@kukocrunch
Is there any custom attributes or heavy size image inside the original product?? or what type of product you are trying to copied?
as this feature seems to be working fine in master https://prnt.sc/w23q3s

I don't have heavy size image with the original product and it's a simple type product.
And the products that I am trying to copy have custom attribute family am I missing something?

@kukocrunch
make sure you have done the below command after upgrading to master

composer dump-autoload

php artisan migrate

php artisan vendor:publish --force & press 0 to continue.

php artisan config:cache

I have executed the commands you mentioned earlier, but I still got 404 problem.

composer dump-autoload
Generated optimized autoload files containing 7584 classes

php artisan migrate
Nothing to migrate.

php artisan vendor:publish --force
Publishing complete.

php artisan config:cache
Configuration cache cleared! Configuration cached successfully!

@kukocrunch
if all the above command are working then please tell us the steps to reproduce the steps this issue.

@vaishaliwebkul
I think this caused the error when I upgraded from official download version to master.
My custom attribute family doesn't have product_number , tax_category_id, guest_checkout,

I can't reproduce the attribute family I have migrated to master version.

below is the custom attribute family I currently have

Code
Name
Type

sku
SKU
text

name
Name
text

url_key
URL Key
text

new
New
boolean

featured
Featured
boolean

visible_individually
Visible Individually
boolean

status
Status
boolean

ProductSeries
Series
select

CoreWidth
Core Width
select

QtyPerBundle
Qty/Bundle
select

@kukocrunch
Kindly update if the mentioned bug has resolved?

@vaishaliwebkul I'm still finding out how to resolve this issue I also tried downloading a recent commit from the master branch but the issue is still there.

@kukocrunch
I would recommend you to download the separate application from the master branch https://github.com/bagisto/bagisto/tree/master then verify the issue if you're still experiencing the same issue.
As in master, there are various system attribute has been added which are not added in your custom family.

@kukocrunch
I would recommend you to download the separate application from the master branch https://github.com/bagisto/bagisto/tree/master then verify the issue if you're still experiencing the same issue.
As in master, there are various system attribute has been added which are not added in your custom family.

Hello @vaishaliwebkul
I did compare missing fields from website-download and master
I'm missing Product Number and Brand so I did add those two to
admin/catalog/attributes now copy is working! 😅

Hello @vaishaliwebkul
I did compare missing fields from website-download and master
I'm missing Product Number and Brand so I did add those two to
admin/catalog/attributes now copy is working! 😅


Remove explicit dependency on gems in stdlib

json is included in the Ruby standard library for all supported versions of Ruby.

Thanks!


FR: Allow specifying metadata directly in DSL

I currently see 3 pieces of key information pulled out of the metadata.xml file:

entityID
x509 certificate
redirect SSO URL

I'd like to be able to specify these directly, rather than mess with an HTTP or file-based way.

This may require some rewrites to the underlying spring security SAML library, but would be a really good feature to have.

Agreed on all fronts.


Create functionality to create and manage local Rubrik user accounts

New-RubrikLocalUser
Remove-RubrikLocalUser
Set-RubrikLocalUser
Get-RubrikLocalUser
internal/playground/#!/47user/createLocalUser

Addressed by #452


Crash call delegate [didCaptureVideoSampleBuffer]

Hi, i got i crash with this delegate :
I really need this call, this is essential for my application.
How can i fix it ?
- (void)vision:(PBJVision *)vision didCaptureVideoSampleBuffer:(CMSampleBufferRef)sampleBuffer;

Thanks in advance.

This is a duplicate of https://github.com/piemonte/PBJVision/issues/254

Ok thanks. So what is your solution. Because I didn't find the solution there. And I don't want modify the code. This is a major bug.

thanks for reporting, someone will take a look. yes, is a duplicate.


Added type hints

Added type hints for better readability and correct types.
Related to issue https://github.com/firebase/firebase-admin-python/issues/626

Thank you @mohitCodepy for your contribution. Is this a duplicate of #627 ?

@lahirumaramba , Actually #627  has some missing type hints, so this PR currently in draft
I'll make changes to the remaining files and then open this PR for review.
Thanks

Thank you @mohitCodepy for your contribution. Is this a duplicate of #627 ?

@lahirumaramba , I saw that other guys also wake up almost after a year to continue this PR,
I want to complete this feature change, would you please decide the person for it?

@lahirumaramba ,
May I continue to push changes to this PR?
I want to work regularly


Respect region when finding profile

This means you can have more than one profile with the same account/role, but different regions.  Without this, the profile finding code always finds the first one and uses its region.

Codecov Report

Merging #57 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #57   +/-   ##
=======================================
  Coverage   87.64%   87.64%
=======================================
  Files           7        7
  Lines         259      259
=======================================
  Hits          227      227
  Misses         32       32

Impacted Files
Coverage Δ

src/lib/content.js
96.42% <100%> (ø)
:arrow_up:

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 744581c...ed7d079. Read the comment docs.

@craig-miskell-fluxfederation I understood this PR purpose, but I think it is better to embed the profile name itself in the input tag. Is there a problem with that?

Yes, that seems much better than checking arbitrary attributes.  Are you wanting me to implement that, or just asking for input?

It was a question. After thinking carefully, I thought that a custom input tag should not be included in a form, so I am fixing it by the another way.

I released v0.8.1 that fixed this issue.

Oh, nice, that's a much cleaner solution.  Thank you!


docs: Add guide for repository migration

@kailuowang
I've updated the README in accordance with #597.
I will merge this without waiting for a review,
but if you have any advice or comments,
please feel free to leave them and I will address them later.

@kailuowang
I'll leave it up to you whether you want to archive this repository or not,
but if you want to go back to development, please give us a shout so we can integrate it.


Minor fixes to native client setup of AAD quickstart webapi dotnet

Fixing ClientID details and permission drop down value for Native client piece of .Net Web API tutorial

Hi @gsacavdm, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@dstrockis - Please review this proposed change from @gsacavdm to your article in the public repository.

@tysonn this is fine, thanks.

#sign-off?

Ping :)


feat(nvmeadm): return subsystem object for representing nvme connections

ConnectArgs::connect() now returns a Subsystem object,  which simplifies
library usage and eliminates the need for library clients to manually parse
a cryptic connection string that represent a connection to get a Subsystem.
Signed-off-by: Mikhail Tcymbaliuk<EMAIL_ADDRESS>

bors merge


How to debug a 409 from the TRE CLI?

I'm trying to automate creating some shared services etc with the CLI, and getting a 409 return code with no error message. How can I debug this and find out what the problem is?
Steps to reproduce
My specific example is a certificate service for Nexus. I have a JSON file, as follows:
{
  "templateName": "tre-shared-service-certs",
  "properties": {
    "display_name": "nexus-ssl certificate service",
    "description": "automated deployment",
    "cert_name": "nexus-ssl",
    "domain_prefix": "nexus"
  }
}

I run the CLI:
tre shared-services new  --definition-file my-definition.json
and I get this response:
Creating shared_service...
{
  "status_code": 409,
  "body": ""
}

what can I do to debug this?
This is using the latest version of the CLI, installed just a few minutes ago.
N.B. Other commands work fine, creating workspaces etc, so my basic installation is good and I'm logged in to the TRE.

hi @tim-allen-ck. I'm not trying to register a shared service, I'm trying to deploy one from an existing template.
Can you tell me what fields are missing? I'm trawling the code to find out what should be in there.

My mistake. This line then: https://github.com/microsoft/AzureTRE/blob/9c59b8020af72ef90fdbab389816665694536f0f/api_app/api/routes/shared_services.py#L62-L63
template_schema.json
{
  "$schema": "http://json-schema.org/draft-07/schema",
  "$id": "./templates/shared_services/service_name/template_schema.json",
  "type": "object",
  "title": "New Shared Service",
  "description": "A shared service",
  "properties": {},
  "required": []
}

For the cert service I see
  "required": [
    "domain_prefix",
    "cert_name"
  ],

and I have both those, so that should be OK. Any idea what else could be causing my 409?

Are you trying to deploy multiple instances of the same shared service? Shared services are limited to one instance.
See #3997 and #2820 as a potential resolution.

Definitely nothing in the Cosmos DB? You will have to look at the API logs to get more details https://microsoft.github.io/AzureTRE/v0.19.0/troubleshooting-faq/app-insights-logs/

this was with a virgin TRE, I deployed from scratch (with a new TRE_ID) explicitly to have a clean environment.

Can you see anything in the logs?

no, nothing in the logs...


Add a contribute .md file

There exist before a contribute.md file, but it seems to have vanished in thin air. Either add it back, or create a new one.
One important point is - not allowed to ditch other people work. It should be added if it's not exist already.  I feel bad for what I just did in my latest push.

See: https://github.com/cherow/cherow/blob/master/.github/CONTRIBUTING.md


Hint optional dependencies

node_redis has an optional dependency to hiredis-node, the official c bindings for Redis, that it does not include in its package.json. I have been including hiredis as a dependency in my production deployments because I know the environment. When most everything was installed globally, hiredis could just be installed once and forgotten, but IMHO hiredis is very different from packages that are a good fit for global install post-1.0, such as express, forever and others.
I think it fits into something more akin to devDependencies. It's not in the core dependencies, but if you can, you should add it to your intall. What do you think?

+1
I have another case where this feature will be handy.

Lotta +1s.  Anyone wanna try writing a patch?  Dive in.  lib/install.js is where the "what to install" logic is.

More guidance:

lib/install.js Find the bits where it's deciding what to fetch and install.  Fold the optionalDependencies into that list.  If it's optional, maybe swap out the cb with a function that logs the failure, but calls the original cb without an error.
lib/utils/read-installed.js When reading the installed packages, don't fold optionalDependencies into the dependencies list (since they're not required), but also don't mark them as extraneous (so they won't be pruned.)

Thanks for the direction. I'll take a look over the holidays.

What do you guys think, should optionalDependencies do anything to handle dependency duplication? A duplicate entry in optionalDependencies would override the handling of a dependency defined in the dependencies block.

IMHO, it's up to the module to decide which dependency to require.

If a dependency is both listed in "dependencies" and "optionalDependencies", then that's an error.

What do you think the best way to detect if a certain optional dependency has properly installed?
I think I need to do conditional branching in my code depending on if the optional dependency is met...

@Marak Probably wrap require() in a try/catch.

@marak Check out how node_redis optionally incorporates hiredis. Personally, I wouldn't branch too extensively. I think optionalDependencies should augment the functionality of the code if that dependency is there, but the core functionality should work without it. Wrap it.

My use-case is exponentially more complicated...

@Marak Understood. Still think the impact surface area should be minimal within the rest of a module. The logic should be behind a wrapper, not branched throughout the code.

+1

w00t! Thanks @isaacs.

awesome.


Deploy docs automatically on a domain name

The goal of this branch is to publish the documentation automatically on merge.
This is work in progress.

https://beta.docs.aleph.im/ seems to have problems with setting up a secure TLS connection


[Helix] HttpSys.HttpsTests.Https_EchoHelloWorld_Success fails on Windows.10.Amd64.ClientRS4.VS2017.Open

Failed   Microsoft.AspNetCore.Server.HttpSys.HttpsTests.Https_EchoHelloWorld_Success
Error Message:
 System.Net.Http.HttpRequestException : An error occurred while sending the request.
---- System.Net.Http.WinHttpException : Error 12030 calling WINHTTP_CALLBACK_STATUS_REQUEST_ERROR, 'The connection with the server was terminated abnormally'.
Stack Trace:
   at System.Net.Http.HttpClient.FinishSendAsyncBuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
   at Microsoft.AspNetCore.Server.HttpSys.HttpsTests.SendRequestAsync(String uri, String upload) in /_/src/Servers/HttpSys/test/FunctionalTests/HttpsTests.cs:line 155
   at Microsoft.AspNetCore.Server.HttpSys.HttpsTests.Https_EchoHelloWorld_Success() in /_/src/Servers/HttpSys/test/FunctionalTests/HttpsTests.cs:line 63
--- End of stack trace from previous location where exception was thrown ---
----- Inner Stack Trace -----
   at System.Threading.Tasks.RendezvousAwaitable`1.GetResult()
   at System.Net.Http.WinHttpHandler.StartRequest(WinHttpRequestState state)
Failed   Microsoft.AspNetCore.Server.HttpSys.HttpsTests.Https_SendHelloWorld_Success
Error Message:
 System.Net.Http.HttpRequestException : An error occurred while sending the request.
---- System.Net.Http.WinHttpException : Error 12030 calling WINHTTP_CALLBACK_STATUS_REQUEST_ERROR, 'The connection with the server was terminated abnormally'.
Stack Trace:
   at System.Net.Http.HttpClient.FinishSendAsyncUnbuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
   at System.Net.Http.HttpClient.GetStringAsyncCore(Task`1 getTask)
   at Microsoft.AspNetCore.Server.HttpSys.HttpsTests.SendRequestAsync(String uri, X509Certificate cert) in /_/src/Servers/HttpSys/test/FunctionalTests/HttpsTests.cs:line 145
   at Microsoft.AspNetCore.Server.HttpSys.HttpsTests.Https_SendHelloWorld_Success() in /_/src/Servers/HttpSys/test/FunctionalTests/HttpsTests.cs:line 45

https://mc.dot.net/#/user/aspnetcore/pr~2Faspnet~2Faspnetcore/ci/20190213.29/workItem/Microsoft.AspNetCore.Server.HttpSys.FunctionalTests-netcoreapp3.0/wilogs
https://mc.dot.net/#/user/aspnetcore/pr~2Faspnet~2Faspnetcore/ci/20190213.18/workItem/Microsoft.AspNetCore.Server.HttpSys.FunctionalTests-netcoreapp3.0/wilogs

Closing as @jkotalik is reviewing our currently-flaky tests and RaaS is vigilant.


[SPARK-49385][K8S] Fix getReusablePVCs to use podCreationTimeout instead of podAllocationDelay

What changes were proposed in this pull request?
This PR aims to use podCreationTimeout instead of podAllocationDelay when getReusablePVCs excludes the newly created PVCs of previous batches.
Why are the changes needed?
K8s control plane pod creation can be delayed due to the unknown reasons. So, podAllocationDelay (default: 1s) is insufficient to say that the previous allocation batch's pods are created with their PVCs. We had better wait until podCreationTimeout.
Does this PR introduce any user-facing change?
This affects only the initial set of executors because the baseline is PVC's getCreationTimestamp. So, this fixes only a buggy situation where a PVC is shared by two executors due to the long pending executor pod.
How was this patch tested?
Pass the CIs with newly updated test cases.
Was this patch authored or co-authored using generative AI tooling?
No.

Thank you, @HyukjinKwon . Merged to master/3.5/3.4.


Data loss up failed auto-save

It would be nice if local storage could be implemented as a fail-safe. I just lost about 10 minutes worth of work when Tusk failed to auto-save to Evernote; this happens regularly for me although usually less data is lost at once (30 seconds worth in most cases).
A desirable flow may be local storage > autosave / upload success event > local storage clears out.

An auto-saving feature, running at a higher frequency than the already existing native Evernote one, would indeed be very useful.


Torchvision model state inconsistency in multi-gpu mode (distributed training)

🐛 Bug
I noticed that, when trying to train on multiple GPUs using Lightning 0.10.0, the outputs of the torchvision model fasterrcnn_resnet50_fpn were those described here for inference (eval mode). However, when using a single GPU on lightning 0.10, or multi-gpu on lightning 0.9, this behaviour stopped, and the model was in training mode as expected, with outputs of a forward pass in my training loop reflecting those documented for this mode.
A manual additional call to .train() set things right, but I then started seeing errors in my eval process, which leads me to believe that something is interfering with model state under circumstances which haven't changed except for the number of GPUs in use, like eval mode is being activated by default.
To Reproduce

Run a forward pass for the fasterrcnn_resnet50_fpn torchvision model (I'm not sure whether this extends to other torchvision or wider PyTorch models), using a single GPU, with lightning 0.10
Run a forward pass for the fasterrcnn_resnet50_fpn torchvision model, selecting multiple GPUs with lightning 0.10
Notice that model state appears to change between these circumstances, and in the case of this model, this causes the outputs of a forward pass to change in accordance with the torchvision documentation (linked above).

Expected behavior
E.g. for fasterrcnn_resnet50_fpn:
Expected output from forward pass at train time (taken from use of single GPU, lightning 0.10.0):
{
    'loss_classifier': tensor(0.5534, device='cuda:0', grad_fn=<NllLossBackward>),
    'loss_box_reg': tensor(0.1873, device='cuda:0', grad_fn=<DivBackward0>),
    'loss_objectness': tensor(0.1856, device='cuda:0', grad_fn=<BinaryCrossEntropyWithLogitsBackward>),
    'loss_rpn_box_reg': tensor(0.0244, device='cuda:0', grad_fn=<DivBackward0>)
}

Actual output from forward pass at train time (taken from use of multi-GPU, lightning 0.10.0):
{
    'boxes': tensor([[176.8907, 198.9963, 252.4170, 248.9736],
    [392.0787, 185.2145, 426.4388, 229.0993],
    [178.8200, 207.2800, 213.0467, 251.0375],
    [421.2797, 180.9008, 523.6691, 237.5523],
    [...]], device='cuda:1'),
    'labels': tensor([1, 1, 1, ...], device='cuda:1'),
    'scores': tensor([0.7446, 0.7164, 0.6861, 0.6852, 0.6635, 0.6364, 0.6237, 0.6221, 0.6205,
    0.6201, 0.6193, 0.6137, 0.6082, 0.6064, 0.6020, 0.6018, 0.6008, 0.5992,
    0.5986, 0.5983, 0.5903, 0.5840, 0.5731, 0.5713, 0.5696, 0.5636, 0.5627,
    ...], device='cuda:1')
}

These are the intended outputs from inference (eval mode) according to the documentation.
And when printing model.training I got the following:
For Lightning 0.10.0, single GPU:
Is the model in training mode?: True
For Lightning 0.10.0, multi-GPU:
Is the model in training mode?: False
🤔
Environment

CUDA:

GPU:

Tesla V100-PCIE-16GB
Tesla V100-PCIE-16GB

available:         True
version:           10.2

Packages:

numpy:             1.19.2
pyTorch_debug:     False
pyTorch_version:   1.6.0
pytorch-lightning: 0.10.0
tqdm:              4.50.2

System:

OS:                Linux
architecture:

64bit
ELF

processor:         x86_64
python:            3.8.5
version:           #32~18.04.1-Ubuntu SMP Tue Oct 6 10:03:22 UTC 2020

Thanks for the issue! Mind updating to 1.0.3 to see if issue persists?

Hi,
The issue seems to persist on upgrading to lightning 1.0.3.

Hey! Sorry for the late reply. Can you try to see if issue persists on boring model? https://github.com/PyTorchLightning/pytorch-lightning/blob/master/pl_examples/bug_report_model.py

Hey, sorry for the wait.
I noticed the boring model as it stands today fails when using Lightning 0.9.0 anyway. It required return statements in the training_epoch_end, test_epoch_end and validation_epoch_end functions to work, so I used return {} as a quick fix, as suggested in a separate issue.
When switching to 0.10.0,  I got the error:
pytorch_lightning.utilities.exceptions.MisconfigurationException: training_epoch_end expects a return of None. HINT: remove the return statement in training_epoch_end

So I reverted the changes.
I couldn't replicate the specific model state issue between 0.9.0 and 0.10.0 with the BoringModel. It seems to be in training mode throughout once the right modifications are made.
However, I notice the code I'm working with does return dicts from those functions.  It doesn't suffer the same error as the boring model, but clearly there were some fundamental changes between 0.9.0 and later versions which changes the expectation of whether the _epoch_end functions return anything.
Do you think this could be the cause? Of course I'm still confused that the outcome isn't a crash but inconsistent model state.

Are you able to try using master or 1.1?

Feel free to reopen if still persists!

Feel free to reopen if still persists!


torch.cat(): expected a non-empty list of Tensors

torch.cat(): expected a non-empty list of Tensors，What kind of garbage author and open source project closed without solving the problem?

First, learn some manners. Second, learn to report properly, this has zero information about anything, as you said... it's garbage.


proposed feature: more flexibility in "kustomize edit set image"

From #1981:

@Brightside56 Kargo Render passes images to the Argo CD Repo Server (which it uses as a library) i the form url=tag. Somewhere under the covers, the Argo CD Repo Server executes a kustomize edit set image url=tag. For Kargo's built-in Kustomize support, Kargo executes kustomize edit set image url=tag more directly...
I see your issue is that the image URL varies from Stage to Stage.
I believe your case can be addressed by adding more options to both Kargo Render and to Kargo's built-in Kustomize support such that the resulting command resembles this: kustomize edit set image placeholder-url=stage-specific-url:tag where placeholder-url and stage-specific-url are both things you specific in your promotion mechanisms on a Stage-by-Stage basis, which will leave the tag as the only part of the Freight that is applied from one Stage to the next.

Would this permit adding a prefix to the image registry? We operate an ECR pull through cache in production but GHCR in staging, so that would be nice to be able to have the ability.

Would this permit adding a prefix to the image registry?

I'm afraid I'm not entirely clear on what you are asking.
The proposal would permit the equivalent of kustomize edit set image url1=url2:tag where you would have full control over the values of url1 and url2 and tag would come from the Freight that is being promoted.

Sorry for being unclear.
What you describe should do the required functionality.

v0.9.0 promotion steps address this and it's addressed for legacy git-based promotion mechanisms as well.


provision: Rename --production-test-suite option in provision.

Follow up to https://github.com/zulip/zulip/pull/15195#issuecomment-639127551
@timabbott

Merged, thanks for backporting @arpit551!


Use cached state for continuous deployment instances

Motivation
If we want to deploy instances on every main branch push, they need to use a persistent state disk, or a copy of the CI cached state.
Otherwise, they will put a lot of load on the network by syncing all the time. And they won't be very useful for testing.
Doing this for the release and manual deploy instances also makes them available faster, and puts less load on the network. (Changing the state path will disable the cache for manual deploys, that's an existing input parameter.)
This would be useful to have before we tag the first stable release, but it's not a blocker.
Other Changes
We should re-enable deployments on main branch pushes as part of this PR, by reverting PR #6895.
Testing
Check the Google Cloud logs of a manually deployed instance, and look for the block height.

@gustavovalverde let's discuss when we can schedule this in

Hey team! Please add your planning poker estimate with Zenhub @arya2 @gustavovalverde @oxarbitrage @teor2345 @upbqdn

Even though this is marked as high-priority, I've removed it from the following sprint as we have some big tickets which we've been moving around for a while and are causing other pipeline changes harder to tackle. We should plan this for late September.

Even though this is marked as high-priority, I've removed it from the following sprint as we have some big tickets which we've been moving around for a while and are causing other pipeline changes harder to tackle. We should plan this for late September.

If we can delay it until late September, maybe it's not actually a high priority 🙂
What are we losing by not having these instances relaunch on every commit to main?
I think the Zebra team are mostly doing fine without it, but we might need to be slightly more careful to run full sync tests.

I'll put it in sprint 19 for now with the thought that it may move to Sprint 20


mate.caja: 1.21.0 -> 1.21.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/caja/versions.
These checks were done:

built on NixOS
ran ‘/nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin/caja -h’ got 0 exit code
ran ‘/nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin/caja --help’ got 0 exit code
ran ‘/nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin/caja-connect-server -h’ got 0 exit code
ran ‘/nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin/caja-connect-server --help’ got 0 exit code
ran ‘/nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin/.caja-wrapped -h’ got 0 exit code
ran ‘/nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin/.caja-wrapped --help’ got 0 exit code
ran ‘/nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin/.caja-connect-server-wrapped -h’ got 0 exit code
ran ‘/nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin/.caja-connect-server-wrapped --help’ got 0 exit code
found 1.21.1 with grep in /nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1
directory tree listing: https://gist.github.com/46fb531ddeedf504b1610ffcaf01f09f
du listing: https://gist.github.com/5396435fca3703ee457ac3c66b5c0121

meta.description for mate.caja is: '"File manager for the MATE desktop"'.
cc @romildo for testing.

Success on x86_64-linux (full log)
Attempted: mate.caja
Partial log (click to expand)
shrinking /nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/lib/libcaja-extension.so.1.4.0
shrinking /nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin/.caja-wrapped
shrinking /nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin/.caja-file-management-properties-wrapped
shrinking /nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin/.caja-autorun-software-wrapped
shrinking /nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin/.caja-connect-server-wrapped
gzipping man pages under /nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/share/man/
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/lib  /nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1/bin
patching script interpreter paths in /nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1
checking for references to /build in /nix/store/vpa6y0jj5wa85md64941j0mmd5h8w269-caja-1.21.1...

Success on aarch64-linux (full log)
Attempted: mate.caja
Partial log (click to expand)
shrinking /nix/store/pcqifnsw0chx0rxh3rg93r4y58xf21ha-caja-1.21.1/bin/.caja-file-management-properties-wrapped
shrinking /nix/store/pcqifnsw0chx0rxh3rg93r4y58xf21ha-caja-1.21.1/bin/.caja-autorun-software-wrapped
shrinking /nix/store/pcqifnsw0chx0rxh3rg93r4y58xf21ha-caja-1.21.1/bin/.caja-connect-server-wrapped
shrinking /nix/store/pcqifnsw0chx0rxh3rg93r4y58xf21ha-caja-1.21.1/lib/libcaja-extension.so.1.4.0
gzipping man pages under /nix/store/pcqifnsw0chx0rxh3rg93r4y58xf21ha-caja-1.21.1/share/man/
strip is /nix/store/gp7fylxwn18b7pl2c18ks89hsiaxyfvf-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/pcqifnsw0chx0rxh3rg93r4y58xf21ha-caja-1.21.1/lib  /nix/store/pcqifnsw0chx0rxh3rg93r4y58xf21ha-caja-1.21.1/bin
patching script interpreter paths in /nix/store/pcqifnsw0chx0rxh3rg93r4y58xf21ha-caja-1.21.1
checking for references to /build in /nix/store/pcqifnsw0chx0rxh3rg93r4y58xf21ha-caja-1.21.1...
/nix/store/pcqifnsw0chx0rxh3rg93r4y58xf21ha-caja-1.21.1


scrollIntoView-button-for-e2e-test

Signed-off-by: Chunxi Luo<EMAIL_ADDRESS>https://app.travis-ci.com/github/open-cluster-management/governance-policy-framework/builds/239073755

/lgtm


Unsaved message

It would be pretty nice if Fcus could alert me if I try to close the app without saving.

Thanks for letting me know! I am recently too busy on my work, so I probably need more time to fix these bugs and add new features.

This part of code is completed in the recent commit, so I will close this issue.


Update vshadow-tool-and-sample.md

fix markdown asterisk

@microsoft-github-policy-service agree

hello I know that when I  using vshadow.exe -p -nw -script=multi.cmd C: E: the content of multi.cmd is
@echo.
@echo [This script is generated by VSHADOW.EXE for the shadow set {e223de3c-8c50-46e7-aecd-53e606624990}]
@echo.

SET SHADOW_SET_ID={e223de3c-8c50-46e7-aecd-53e606624990}
SET SHADOW_ID_1={46e89eda-5658-4512-970f-bef0116b7908}
SET SHADOW_DEVICE_1=\\?\GLOBALROOT\Device\HarddiskVolumeShadowCopy6
SET SHADOW_ID_2={b7754370-ad0c-444a-8e21-1cea5655bfd7}
SET SHADOW_DEVICE_2=\\?\GLOBALROOT\Device\HarddiskVolumeShadowCopy7

I have a question is the SHADOW_ID_1 is C volume shadow and SHADOW_ID_2 is E volume shadow?
thanks @alvinashcraft

Hi @stardustman. I'm not the best resource to ask about details of using the vshadow tool. This would be a great question to ask in the Win32 section of Microsoft Q&A. The folks there can reach out to the product team and other experts in these tools.
If it turns out that the documentation is inaccurate or incomplete, please come back and either create another PR with the suggested changes or submit feedback with details about the docs issue using the "Feedback" button at the top of the page. I check those submitted feedback several times a week to create new work items. I'm closing this PR, as it originally related to the formatting of the page, but please follow up with what you find out. I'll be monitoring the PR for any replies. Thank you!


GoRoutines leak with 1.12.0-rc.3

In what area(s)?

/area runtime
What version of Dapr?

1.12.0-rc.3
Expected Behavior
The longhaul test application workflow-gen is showing a goroutine+memory leak

We can see the following in workflow-gen's dapr:
{
	"app_id": "workflow-gen",
	"instance": "workflow-gen-app-fbbfc94cc-twlxh",
	"level": "warning",
	"msg": "d9207b15::0::1: execution failed with a recoverable error and will be retried later: timed-out trying to schedule an activity execution - this can happen if too many activities are running in parallel or if the workflow engine isn't running: context deadline exceeded",
	"scope": "dapr.runtime.wfengine",
	"time": "2023-09-25T16:21:24.166784515Z",
	"type": "log",
	"ver": "1.12.0-rc.3"
}

Top 3 goroutines from pprof:
goroutine profile: total 8991
8825 @ 0x43e496 0x44ebbe 0x5185f50 0x51843c7 0x5183d68 0x512f2dd 0x5128fbb 0x513148b 0x5128ba9 0x512c634 0x513148b 0x512c375 0x512b858 0x511b8e8 0x472de1
#	0x5185f4f	github.com/dapr/dapr/pkg/runtime/wfengine.(*actorBackend).ScheduleActivity+0x8f		/home/runner/work/dapr/dapr/pkg/runtime/wfengine/backend.go:58
#	0x51843c6	github.com/dapr/dapr/pkg/runtime/wfengine.(*activityActor).executeActivity+0x206	/home/runner/work/dapr/dapr/pkg/runtime/wfengine/activity.go:165
#	0x5183d67	github.com/dapr/dapr/pkg/runtime/wfengine.(*activityActor).InvokeReminder+0x207		/home/runner/work/dapr/dapr/pkg/runtime/wfengine/activity.go:116
#	0x512f2dc	github.com/dapr/dapr/pkg/actors.(*internalActorChannel).InvokeMethod+0x2dc		/home/runner/work/dapr/dapr/pkg/actors/internal_actor.go:130
#	0x5128fba	github.com/dapr/dapr/pkg/actors.(*actorsRuntime).callLocalActor.func2+0x9a		/home/runner/work/dapr/dapr/pkg/actors/actors.go:561
#	0x513148a	github.com/dapr/dapr/pkg/resiliency.NewRunnerWithOptions[...].func2+0x42a		/home/runner/work/dapr/dapr/pkg/resiliency/policy.go:184
#	0x5128ba8	github.com/dapr/dapr/pkg/actors.(*actorsRuntime).callLocalActor+0x8e8			/home/runner/work/dapr/dapr/pkg/actors/actors.go:560
#	0x512c633	github.com/dapr/dapr/pkg/actors.(*actorsRuntime).doExecuteReminderOrTimer.func1+0x33	/home/runner/work/dapr/dapr/pkg/actors/actors.go:909
#	0x513148a	github.com/dapr/dapr/pkg/resiliency.NewRunnerWithOptions[...].func2+0x42a		/home/runner/work/dapr/dapr/pkg/resiliency/policy.go:184
#	0x512c374	github.com/dapr/dapr/pkg/actors.(*actorsRuntime).doExecuteReminderOrTimer+0x834		/home/runner/work/dapr/dapr/pkg/actors/actors.go:908
#	0x512b857	github.com/dapr/dapr/pkg/actors.(*actorsRuntime).executeReminder+0x37			/home/runner/work/dapr/dapr/pkg/actors/actors.go:851
#	0x511b8e7	github.com/dapr/dapr/pkg/actors/reminders.(*reminders).startReminder.func1+0x5a7	/home/runner/work/dapr/dapr/pkg/actors/reminders/reminders.go:962

100 @ 0x43e496 0x40618e 0x405d3d 0x5186d48 0x517788b 0x518236c 0x472de1
#	0x5186d47	github.com/dapr/dapr/pkg/runtime/wfengine.(*actorBackend).AbandonActivityWorkItem+0x127	/home/runner/work/dapr/dapr/pkg/runtime/wfengine/backend.go:147
#	0x517788a	github.com/microsoft/durabletask-go/backend.(*activityProcessor).AbandonWorkItem+0x4a	/home/runner/go/pkg/mod/github.com/microsoft/durabletask-go@v0.3.1/backend/activity.go:98
#	0x518236b	github.com/microsoft/durabletask-go/backend.(*worker).processWorkItem+0x70b		/home/runner/go/pkg/mod/github.com/microsoft/durabletask-go@v0.3.1/backend/worker.go:223

10 @ 0x43e496 0x44ebbe 0x42dcec6 0x42dce02 0x472de1
#	0x42dcec5	github.com/dapr/components-contrib/pubsub/redis.(*redisStreams).worker+0x145	/home/runner/go/pkg/mod/github.com/dapr/components-contrib@v1.12.0-rc.3/pubsub/redis/redis.go:193
#	0x42dce01	github.com/dapr/components-contrib/pubsub/redis.(*redisStreams).Init.func1+0x81	/home/runner/go/pkg/mod/github.com/dapr/components-contrib@v1.12.0-rc.3/pubsub/redis/redis.go:91

Actual Behavior

Steps to Reproduce the Problem
This is currently happening with the longhaul-release cluster, should be reproducible bringing a similar cluster up.
Release Note

RELEASE NOTE:

Attached the zip'ed http://localhost:7777/debug/pprof/profile?seconds=120" output workflow-gen.zip

hey @tmacam , Could you please share that whether same issue has been present on 1.12 previous RCs as well?
Just trying to zoom in and understand that what recent changes to focus on.

In an interesting turn of events, after 1) stopping workflow-gen, 2) cleaning up both the following 2 entries and 3) restarting the application, things have been looking fine.
c.id = "actors||dapr.internal.longhaul-test.workflow-gen.activity"
c.id = "actors||dapr.internal.longhaul-test.workflow-gen.workflow"
So far, with workflow-gen is keeping pretty stable at 70 goroutines even after some restarts.
Now, for something interesting: when this started and how it was before? Workflow-gen goroutines count was stable but started to get messy around Sep-21 16:40 UTC:

We can't get dapr version from prometheus but we can get from the logs:
(Last 7 days)

daprVersion
TimeGenerated

1.12.0-rc.4
2023-09-26T00:56:09.7194573Z

1.12.0-rc.3
2023-09-22T16:37:05.4251444Z

1.11.3
2023-09-21T01:40:49.8916543Z

ContainerLog
| where LogEntry has "hashtag-counter"
| where LogEntry has "starting Dapr Runtime"
| extend logEntryValues = parse_json(LogEntry)
| extend daprVersion = tostring(logEntryValues.ver)
| summarize arg_min(['TimeGenerated'], *) by daprVersion
| project  daprVersion, TimeGenerated
| order by TimeGenerated desc

We were running 1.11.3 and installed 1.12.0-rc.3 and that is when the problem started.
Right now we are running 1.12.0-rc.4 and after cleaning up the data everything seem to be just fine. I will keep monitoring but.. yeah 🙃

And some additional context: the number of activities inside the DB was approaching 11k (about the same number of goroutines that we saw) and some 700 entries workflows and the node reaching a point it was OOMKilled. Restarting the app made it so that it would quickly "recover" to this 11k-12k goroutines it would keep looping from started to OOMKilled. There was no option but to clean the data to take the app from this "poison pill" state it could not recover from.
It was speculated that this would happen on restarts but I restarted the app a few times after the clean up and things got back to a healthy state.

The remaining suspicion was that this could happen on an upgrade flow. I just tried to reproduce this by upgrading a cluster from 1.11.3 to 1.12.0-rc.4 and could not reproduce this issue. I think since we cannot reproduce this issue it would be safe to close this issue and if/when we find a way to reproduce this, we reopen.


Exception: HTTP request failed, statusCode: 403

I've downloaded the example project folder but I wasn't able to make it work (both on android and iOS)..
The project compiles without errors and the application launches as expected but the map won't show, and an exception for the http request is thrown with statusCode 403.
I'm using an unrestricted API key, and I'm certain it works, cause I've tested it on another Android project and the map is loaded.

That typically means that the API key is not set. You need to do that view the MapView API. For running on Android, you also have to add the API key to the AndroidManifest.xml

Thanks for the quick response, my problem was that I didn't enable "Google Static Maps API". You should add that to the readme cause it's only on the code.

Has anyone found a work around? The example works for me sometimes and sometimes not.
Most of the time I get the 403 when it request the static map.
Here's my flutter info:
[✓] Flutter (Channel master, v0.10.2-pre.99, on Mac OS X 10.13.6 17G65, locale en-US)
• Flutter version 0.10.2-pre.99 at /Users/jchasecrafton/flutter
• Framework revision c319b890b3 (13 days ago), 2018-10-29 19:44:36 -0700
• Engine revision 162b2e98c0
• Dart version 2.1.0 (build 2.1.0-dev.8.0 bf26f760b1)


To Do list add & remove Feature (rm2)

In this project, I implemented the CRUD (create, update, delete) methods. All the elements of the user interface are fully functional, details are below.
Requirements & Features

Create a JavaScript file for the new functionality
Implement a function for adding a new task (add a new element to the array)
Implement a function for deleting a task (remove an element from the array)
Implement a function for editing task descriptions (update content on the array)
By default, new tasks have the property completed set to false and the property index set to the value of the new array length (i.e. if you're adding a 5th task to the list, the index of that task should equal to 5)
Deleting a task should update all remaining items' indexes, so they represent the current list order and are unique(i.e. if you're deleting the first task index 1 from the list, the index of the next task(2) should be set to 1)
All changes to the To-Do List are saved in local storage

General Requirements:

Use webpack to bundle JavaScript
Use proper ES6 syntax
Use ES6 modules to write modular JavaScript
Check linter errors
Use the correct GitHub flow
Document my work in a professional way
Follow best practices for HTML, CSS & JavaScript
Open a pull request and ask for a review

@rloterh
Thank you so much for the feedback and review


adds a bookmark warning to the top of the app

Added a bookmark warning div that displays a bookmark warning as shown in the Figma design.

The feature is implemented by pushing the content down by 2 rem units. When displayed, the total page height is shrinked by the same amount to avoid showing a scrollbar. Tested in Firefox and Chrome.

Will check later today!


Problem with books that have number or more than one word

In the last update there are problems with books that start with a number or more than one word, that is, books that use quotation marks (").
I have checked in version b3663fba966549e196c20a5f0a275a7d941291ca of September 29, 2022 and it works, with some small problems, but it works.
The problem is from version 95d17cab07df0ad291b07e24ea74b71f436a6f8a onwards.

I think the issue is with the ruby code that reads the verses from internet. The solution I found was removing the blank spaces. I made a PR with that fix, but you can update line 220 with
book_no_spaces=$(echo "$book" | sed 's/ //g')
# Use the bg2md script to read chapter contents
chapter_content=$(ruby bg2md.rb $bg2md_flags -v $ARG_VERSION $book_no_spaces$chapter)

@Lucas-Froguel I'm also having this issue but know very little about scripting.  Is the line update supposed to go in bg2md.rb or bg2obs.sh? Line 220 doesn't make sense to me for either of those.

I think the issue is with the ruby code that reads the verses from internet. The solution I found was removing the blank spaces. I made a PR with that fix, but you can update line 220 with
book_no_spaces=$(echo "$book" | sed 's/ //g')
# Use the bg2md script to read chapter contents
chapter_content=$(ruby bg2md.rb $bg2md_flags -v $ARG_VERSION $book_no_spaces$chapter)

Thank you brother, worked perfectly and saved me a lot of time! 🙏

I just made a PR that pushes the full solution to the main code. After that is approved, I believe this issue can be closed.

Closed with #47, thanks @Lucas-Froguel !


Screen Ruler Black Screen

Microsoft PowerToys version
0.84.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Screen ruler
Steps to reproduce
Using the shortcut to launch the screen ruler and then selecting a ruler type. As soon as the measurements start showing up the screen goes completely black, apart from the line and values the Screen Ruler provides.
✔ Expected Behavior
No black screen.
❌ Actual Behavior
Ruler is engaging with applications in the background and my wallpaper. But its all hidden behind a black screen. I can scroll to change the pixel sensitivity, and i can see the ruler find contrast in different part of screen where an application's window might have ended.
Other Software
softwares running in BG

Netflix (installed via Edge)
Thorium
Whatsapp
Windows Phone Link

We've found some similar issues:

Black Screen Issue with Screen Ruler #26120 , similarity score: 89%
屏幕标尺bug #28455 , similarity score: 83%

If any of the above are duplicates, please consider closing this issue out and adding additional context in the original issue.

These issues seem to be resolved, but mine isnt.


How to use sass_func for computing convolution?

Thanks for your great work!
I just wanna to compare the latency of running convolution layer between anakin_sass and cudnn, and I have successfully ran convolution layer with cudnn.
However,  when I call api of direct_conv_bias_Kdivis4 in sass_funcs.h, I found that I just did not understand the meaning of parameters like img_in_channel_stride, img_in_height_stride and so on.
So, How should I set the img_in_channel_stride parameter properly?

please take https://github.com/PaddlePaddle/Anakin/blob/master/saber/funcs/impl/cuda/saber_conv_direct.cpp as an example


Release for ROS Noetic?

Please release for ROS Noetic.

Hi @tylerjw   Your question has been responded to on the ROS GitHub version of it.  Thank you very much.
https://github.com/IntelRealSense/realsense-ros/issues/1423

Case closed due to link to answer provided and no further comments received.
Doronhi the RealSense ROS wrapper developer added to the linked to discussion "Noetic version is planned, if everything will go according to plan, for December 2020".

Hi @tylerjw   RealSense ROS wrapper 2.2.20 has added Noetic support.
https://github.com/IntelRealSense/realsense-ros/releases/tag/2.2.20


Upgrade to ember 4.9 (in development)

This will allow us to use the preview types when resolving https://github.com/emberjs/ember-qunit/issues/957, which will help resolve https://github.com/emberjs/ember.js/issues/20254

Oops, merging this closed https://github.com/emberjs/ember.js/issues/20254 because I said "will help resolve" lolol


fix: remove 0x bytecode object prefix for CompilerOutput

Motivation
Closes #1422

Solution
adds custom serialization for Bytecode objects that omits the 0x prefix. This way the artifact types used by Forge (ConfigurableArtifact) are not affected and will still be output with 0x prefix

PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Updated the changelog

Thank you!


Add shimmer loader wherever possible

The components are loaded and are empty while the API is in pending state. adding a shimmer loader would look nice.

Looking into it

Go ahead @AqViolet


Where is the source code for the service

Hi,
this is a great tool, but where is the source code for the service it is? I can't find it in this repo?

okay, I found it.

okay, I found it.


[6.3.0] Spelling error in Filter mediator

Description:
When using a Filter Mediator from the tooling, we can edit the Condition Type property. The choices are XPath and Source and Regualar Expression. The last one should be Source and Regular Expression.
https://github.com/search?q=org%3Awso2+Source+and+Regualar+Expression&type=Code
Affected Product Version:
WSO2 Enterprise Integrator Tooling 6.3.0
OS, DB, other environment details and versions:
Windows

Moved to https://github.com/wso2/product-ei/issues/2573


User Role Permissions

Put together a sheet containing user roles and permissions, define who has access to do what.

Permissions.xlsx


[equation input] Suggestions don't appear after entering an arrow key on Android

It seems that the onChanged in Android only responds to the changes made by the user, however on iOS it reponds to changes made programatically (replacing -> by the UTF8 arrow).

This can be fixed with a forced refresh when the arrow gets converted.


Add bin folder

#30
Add bin folder with console script to load everything you to start playing with the library 🎉

@solnic can we merge this one?

@GustavoCaso I think we should change executables folder to exe in gemscpec.
https://github.com/dry-rb/dry-logic/blob/9d1ae6bc63e6c0a1bbd5159b3f9708f9eea5f7a3/dry-logic.gemspec#L13
as well as in dry-monads for example https://github.com/dry-rb/dry-monads/blob/master/dry-monads.gemspec#L25-L26

@artofhuman but the bindir is bin by default. In dry-monads is set to exe but the no exe folder in the project. Is that correct? Looks like we should change it in dry-monads as well.
Also why you think is better to name the folder exe just for curiosity.

Hm, i mean we should change spec.executables to
spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) } or remove this spec,
because as bundle say

If you use the --binstubs flag in [bundle install(1)][bundle-install], Bundler will automatically create a directory (which defaults to app_root/bin) containing all of the executables available from gems in the bundle.

@artofhuman is right, if we add this bin/ dir with some utility scripts, we either want to remove spec.executables = ... from the gemspec or tell it to look in exe/ instead of bin/.
Given that dry-logic is unlikely to ever ship its own executable, I think we could just remove it.

Thank you so much for the explanation @timriley and @artofhuman


add tests for lua functions that use integer types - v4

Related to task: https://redmine.openinfosecfoundation.org/issues/4182 use lua_pushinteger for pushing integer types as integers instead of floats.
Change compared to previous PR:

rebased, squashed file-compare commits

Previous PR: #523
Patch PR: https://github.com/OISF/suricata/pull/6352

Trying to figure out the failures...
master 5.0.x
===> ipv6-dos-with-ext-headers-6: FAILED: got exit code 1, expected 0
master 6.0.x
===> http2-deflate: Sub test #1: FAIL : expected 1 matches; got 0 for filter {'count': 1, 'match': {'event_type': 'fileinfo', 'http2.length': 30, 'fileinfo.size': 20}}
===> http2-upgrade: Sub test #4: FAIL : expected 1 matches; got 0 for filter {'count': 1, 'match': {'event_type': 'http', 'http.http2.stream_id': 1, 'http.version': '2', 'http.http_method': 'GET', 'http.url': '/robots.txt', 'http.request_headers[2].name': ':authority', 'http.request_headers[2].value': 'nghttp2.org', 'http.status': 200}}
===> ipv6-dos-with-ext-headers-6: FAILED: got exit code 1, expected 0

Trying to figure out the failures...

SV is currently in a state where its waiting on a backport it appears.

For 6.0: https://github.com/OISF/suricata/pull/6348
And probably one will be required for 5.0 if applicable, or that test marked with a min-version of 6.

Work continued in: https://github.com/OISF/suricata-verify/pull/537


cloud-regress is failing to get remote extension regress.so on staging

This job is failing since Oct 03
https://github.com/neondatabase/neon/actions/workflows/cloud-regress.yml
This is a sign of a problem with remote extension infrastructure.
This is a blocker for compute release, until we figure out what is the issue,
because this will potentially break customer's remote extensions.

this is not a compute issue, just missing configuration for test user
see details in slack https://neondb.slack.com/archives/C04DGM6SMTM/p1728570559142299


Server streaming with Callback after each streamed message

I need to know if it is possible to stream (Server --> client), in which after each streamed message i am able to release the resources before the next message streamed.
So i need something like a callback action that i can use it to release the resources of old streamed message.
Thanks for any Advice or Suggestion.

I might understand your issue wrong but if you use an async version you get an tag delivered once the message is sent. If you use a sync server you should be able to clean up the message once Write returns.

Hi,
Thanks for your answer, i think you have understood what i need.
You Said i should use a sync server to clean up the message once Write returns to the Server, it is a new Information for me that there are two different Server implementation Sync and Async , could you please show me how can i implement such a Sync Server
Thanks a lot in advance.

@MhdAna
https://grpc.io/docs/tutorials/async/helloasync-cpp/#async-server


Anyway to share an image with this plugin through twitter sharing?

Hello, love your plugin.  Just wondering if there is anyway to send an image using this plugin when sending off to twitter?  Is there an example of this someplace?
Thank you.

Thank you for the feedback!
I think this is the answer to your question https://stackoverflow.com/a/22395173/1140893


Issue with missing files

What is your rclone version (eg output from rclone -V)
rclone-stable - 1.36
rclone-latest - v1.36.49
Which OS you are using and how many bits
Linux Debian 8
Which cloud storage system are you using?
Google Drive

Missing files;
find . -type f | wc -l
usenet@web:/data/.gcd-crypt/tvshows$ find . -type f | wc -l
4893
root@web:/data/.gcd-crypt/movies# find . -type f | wc -l
1497
Total = 6390
Rclone Crypt mounted from PlexDrive
usenet@web:/data/.gcd-crypt-pd/tvshows$ find . -type f | wc -l
5168
usenet@web:/data/.gcd-crypt-pd/movies$ find . -type f | wc -l
1497

Nothing in the mount log with -v

root@web:/opt/plexgcd/logs# rclone --checkers 32 ls gcd-crypt:tvshows | wc -l
5917
root@web:/opt/plexgcd/logs# rclone --config /home/usenet/.rclone.conf --checkers 32 ls gcd-crypt-pd:tvshows | wc -l
6579

This is the same directory source from google too.

This is a duplicate folders issue, makes it harder causes it encrypted.


easy_install boto3 1.18.60 broken dependency on botocore 1.21.60

Describe the bug
AWS Elastic Beanstalk deployments are trying (and failing) to run easy_install boto3:
No local packages or working download links found for botocore<1.22.0,>=1.21.60
It's checking the pypi repo (https://pypi.org/simple/botocore/) which has the whl but is missing a tar.gz for the latest (1.21.60) release.
I don't know if this is more a boto3 problem or a botocore problem, but it seemed right to put it here
Steps to reproduce

Execute easy_install boto3

Expected behavior
Install!
Debug Logs
$ sudo easy_install boto3
Password:
Searching for boto3
Reading https://pypi.org/simple/boto3/
Downloading https://files.pythonhosted.org/packages/27/98/c9f627e7017ab4191818ef08a345390689b66e45d6fa3f4a57d1aa8c9a25/boto3-1.18.60.tar.gz#sha256=45709a04ec5fb67ce5a8eaade3eb0ab24d6eb08d9a9ca6bdb2153047896197fc
Best match: boto3 1.18.60
Processing boto3-1.18.60.tar.gz
Writing /tmp/easy_install-tipgbV/boto3-1.18.60/setup.cfg
Running boto3-1.18.60/setup.py -q bdist_egg --dist-dir /tmp/easy_install-tipgbV/boto3-1.18.60/egg-dist-tmp-uZK9pK
zip_safe flag not set; analyzing archive contents...
boto3.session: module references __file__
boto3.docs.service: module references __file__
creating /Library/Python/2.7/site-packages/boto3-1.18.60-py2.7.egg
Extracting boto3-1.18.60-py2.7.egg to /Library/Python/2.7/site-packages
Adding boto3 1.18.60 to easy-install.pth file

Installed /Library/Python/2.7/site-packages/boto3-1.18.60-py2.7.egg
Processing dependencies for boto3
Searching for s3transfer<0.6.0,>=0.5.0
Reading https://pypi.org/simple/s3transfer/
Downloading https://files.pythonhosted.org/packages/88/ef/4d1b3f52ae20a7e72151fde5c9f254cd83f8a49047351f34006e517e1655/s3transfer-0.5.0.tar.gz#sha256=50ed823e1dc5868ad40c8dc92072f757aa0e653a192845c94a3b676f4a62da4c
Best match: s3transfer 0.5.0
Processing s3transfer-0.5.0.tar.gz
Writing /tmp/easy_install-y7Qz1i/s3transfer-0.5.0/setup.cfg
Running s3transfer-0.5.0/setup.py -q bdist_egg --dist-dir /tmp/easy_install-y7Qz1i/s3transfer-0.5.0/egg-dist-tmp-7um8Q3
zip_safe flag not set; analyzing archive contents...
Copying s3transfer-0.5.0-py2.7.egg to /Library/Python/2.7/site-packages
Adding s3transfer 0.5.0 to easy-install.pth file

Installed /Library/Python/2.7/site-packages/s3transfer-0.5.0-py2.7.egg
Searching for botocore<1.22.0,>=1.21.60
Reading https://pypi.org/simple/botocore/
No local packages or working download links found for botocore<1.22.0,>=1.21.60
error: Could not find suitable distribution for Requirement.parse('botocore<1.22.0,>=1.21.60')```

I'm investigating this... it's strange that the source dist is available here:
https://pypi.org/project/botocore/#files
But not via pip or easy_install https://pypi.org/simple/botocore/
Trying to determine if this is an issue on the PyPI side or something we did.
It's worth noting though that this version of botocore / boto3 are not supported on Python 2.
You should pin to the last supported version of boto3 using: boto3==1.17.112, which was the last version to support Python 2.
It's also worth noting easy_install is deprecated as well:
https://setuptools.pypa.io/en/latest/deprecated/easy_install.html
Can you provide more context on your setup?


[FEAT] Support richer set of flashcard ordering during review; e.g. random card from random deck

Current Functionality
At present the plug-in has two methods of ordering flashcards during review. These are based on the setting:

Where this option enables randomization within a single deck.

Solution Implemented
I have implemented greater flexibility with the following options:

Feedback wanted
This enhancement was beta tested successfully for https://github.com/st3v3nmw/obsidian-spaced-repetition/issues/610.
However I would appreciate wider feedback from anyone before opening a PR.

Hi
Here is the code from:
https://github.com/ronzulu/obsidian-spaced-repetition
Or pre built main.js main_2023-12-14.zip
Note that this also contains post 1.10.5 fixes:

#776
#799
#800

Cheers
Ronny

Hi @st3v3nmw  thanks for including this on the latest release.

Random card from random deck breaks the heading. Normally you would get as heading the name of the deck you clicked with this option you get the sub deck it choose randomly in the deck you clicked.

Hi @4Source thanks for the prompt, it wasn't something I was particularly happy with.
For "Random card from random deck" ordering, what do you think of:
${deckName} (and ${subdeckCount} subdecks): ${cardCount}`
Where:
deckName - the name of the deck chosen by the user (doesn't change as cards from different subdecks are shown)
subdeckCount - count of subdecks under the deck chosen by the user. I'm leaning towards this being the remaining count, i.e. the count of subdecks that have remaining cards for review
cardCount - total number of cards remaining (I.e. not the count of cards remaining in the current randomly chosen deck).
Let me know what you think.
Cheers
Ronny

Sounds good, but subdeckCount counts only the top level of subdecks or count it each subdeck at each level below?
Maybe change the Format to:
${deckName}: ${cardCount}
and a subtitle:
Subdecks: ${subdeckCount}

Sounds good, but subdeckCount counts only the top level of subdecks or count it each subdeck at each level below?
Maybe change the Format to: ${deckName}: ${cardCount} and a subtitle: Subdecks: ${subdeckCount}

I was thinking that the ${subdeckCount} would be the total number of descendant decks. That way a person would see this number decrease every time all cards completed review in any descendant deck and see some progress during the review.


Ctrl-C / SIGINT fails to cancel parent on first try

What was wrong?

Expected Result
Cancellation of both spawned subprocesses and the parent in the traditional trionic fashion.
I'm going to see what I can do to provide a fix for this. Imho this is an absolutely critical problem!
PS: on another note if you're looking for any collaborators on this project I'd be willing to throw in my hat. I'm extremely interested in a rock solid process launching library based around trio not only because I have a project based on this but also because I think a resilient process spawner is something the Python community needs in general.

I believe this has been fixed in one of my recent commits. Now, if I Ctrl-C the following script
trio.run(run_in_process, trio.sleep_forever)

I get this immediately:
^CTraceback (most recent call last):
  File "issue.py", line 5, in <module>
    trio.run(run_in_process, trio.sleep_forever)
  File "/home/salgado/virtualenvs/trio-rip/lib/python3.8/site-packages/trio/_core/_run.py", line 1896, in run
    raise runner.main_task_outcome.error
  File "/home/salgado/src/snakecharmers/trio-run-in-process/trio_run_in_process/run_in_process.py", line 125, in run_in_process
    return proc.get_result_or_raise()
  File "/home/salgado/src/snakecharmers/trio-run-in-process/trio_run_in_process/process.py", line 161, in get_result_or_raise
    raise self._error
trio.MultiError: KeyboardInterrupt(), KeyboardInterrupt()

Details of embedded exception 1:

  KeyboardInterrupt

Details of embedded exception 2:

  KeyboardInterrupt

I believe this has been fixed in one of my recent commits. Now, if I Ctrl-C the following script
trio.run(run_in_process, trio.sleep_forever)

I get this immediately:
^CTraceback (most recent call last):
  File "issue.py", line 5, in <module>
    trio.run(run_in_process, trio.sleep_forever)
  File "/home/salgado/virtualenvs/trio-rip/lib/python3.8/site-packages/trio/_core/_run.py", line 1896, in run
    raise runner.main_task_outcome.error
  File "/home/salgado/src/snakecharmers/trio-run-in-process/trio_run_in_process/run_in_process.py", line 125, in run_in_process
    return proc.get_result_or_raise()
  File "/home/salgado/src/snakecharmers/trio-run-in-process/trio_run_in_process/process.py", line 161, in get_result_or_raise
    raise self._error
trio.MultiError: KeyboardInterrupt(), KeyboardInterrupt()

Details of embedded exception 1:

  KeyboardInterrupt

Details of embedded exception 2:

  KeyboardInterrupt

@gsalgado why don't you just prove it by merging the test in #5?
Also, what's with the necro-bump of this repo?
Aren't y'all doubling down on https://github.com/ethereum/aio-run-in-process ?

@gsalgado why don't you just prove it by merging the test in #5?
Also, what's with the necro-bump of this repo?
Aren't y'all doubling down on https://github.com/ethereum/aio-run-in-process ?


--force should silently ignore files not part of the project

Bug Report

TSLint version: 5.11.0
TypeScript version: 3.0.1
Running TSLint via: cli / pre-commit hook

TypeScript code being linted
There is no code in particular.
We invoke tslint as part of our pre-commit hook with the following options:

tslint --project . --format verbose --fix --force $FILES

Which then gives a non-0 return code with the following error:

'$FILE.ts' is not included in project.

The file in question is excluded via the tsconfig exclude option.
Actual behavior
The command returns a non-0 code, even though --force is specified
Expected behavior
--force should always succeed, no matter what.

@Swatinem you should not use --project . and files. use one or the other. not surprised there's an error if you pass files not included in the project, as the project takes precedence.

Hm, I need --project to support typechecking-dependent rules. And since I integrated tslint with prettier, --fix is run as part of the pre-commit hook to also correctly format all the code.
The way the pre-commit hook works, I am not sure I can avoid passing in the list of changed files.

Perhaps we should have better documentation here.

💀 It's time! 💀
TSLint is being deprecated and no longer accepting pull requests for major new changes or features. See #4534. 😱
If you'd like to see this change implemented, you have two choices:

Recommended: Check if this is available in ESLint + typescript-eslint ✅
Not Recommended: Fork TSLint locally 🤷♂

👋 It was a pleasure open sourcing with you!
If you believe this message was posted here in error, please comment so we can re-open the issue!

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


[Bug] - Layout breaking after going on "sleep" mode.

Installed and set up Orbit yesterday, today I've tried accessing it, and it shows like this:

All I can describe is yesterday it worked and looked as expected, and today after opening the MacBook noticed it was showing like this.
Force quitting it from the Activity Monitor and re-opening it fixed it.

Hi @Noitham, thanks for reporting this issue, I'll be looking into a fix later today

Figured out how to reproduce this consistently and I'll be opening a PR fixing it in a few
Steps to reproduce:

Open the Popover on a second monitor
Put your Mac to sleep from the second monitor
Unplug the monitor
Wake up your Mac and press the Orbit Icon on the menu bar

A new version of Expo Orbit has just been released! v0.1.1 includes fixes for this and other issues, you can check the full details on the releases page. Please test it out and let us know if you find any other problems.


[python client] fixed TaskClient.ackTask return value

The latest changes of the ConductorWorker.poll_and_execute function broke the client for me since the return value of TaskClient.ackTask is wrong (at least for python3.7). The result of the self.post function call in ackTask returns the boolean True instead of the expected 'true' string and therefore ackTask returns False even though the ack request was successful. The root of this is the __return function where resp.json() parses the value true of response body to the boolean True.

Pull Request Test Coverage Report for Build 2349

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 68.089%

Totals

Change from base Build 2345:
0.0%

Covered Lines:
8981

Relevant Lines:
13190

💛  - Coveralls


Slow start on Windows 7 for big folders (even on SSD)

We use grunt watch in our project with a config like this:
  grunt.initConfig({
    watch: {
      sync: {
        files: ['src/folder1/**', 'src/folder2/**', 'src/folder3/**'],
        tasks: ['sync'],
        options: {
          debounceDelay: 250
        }
      }
   ...

  grunt.registerTask('default', ['watch:sync']);

Those 3 folders amount to ~3000 files.
When I run grunt, it immediately prints that it's watching/waiting:
$ grunt
Running "watch:sync" (watch) task
Waiting...

but in fact, it doesn't "see" the changed files for at least first 10-15 seconds after startup. I.e. I change some files, but the change is not detected and the callback is not fired. It starts being notified about file changes only after some 10-15 seconds from startup.
The folder is located on an SSD drive.
Is this a known issue?
Is this a limitation of Windows?

So the problem about "Waiting..." being displayed too early is related to #332 - it's displayed immediately instead of when ready callback is fired by gaze.
The fact about long startup time is linked to the number of files watched. Globbing itself takes only about 1.5 seconds, but it's probably on adding listeners phase when it's so slow, and probably linear with the number of files being watched.
To improve perf, I've done this:

list explicitly files I am interested in, i.e.
files: ['src/folder1/**']

->
 files: ['src/folder1/**/*.js', 'src/folder1/**/*.css']

is much more performant

use 'cwd`:
files: ['very/long/path/**/*.js']

->
files: ['/**/*.js'],
options: {
    cwd: 'very/long/path'
}


website does not have Google Tag Manager activated

Current Behavior

Expected Behavior

Screenshots/Logs

Environment

Host OS: Mac Linux Windows
Browser: Chrome Safari Firefox Edge Opera

Contributor Guide and Resources

📚 Instructions for contributing to documentation

Layer5 documentation site and source

🎨 Wireframes and designs for Layer5 site in Figma (open invite)
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Analytics are now connected.


[BACKLOG-8221] switch/case mdi

@bmorrise @mdamour1976  please review

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dpentaho.resolve.repo=http://ivy-nexus.pentaho.org/content/groups/approved -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -Dpentaho.resolve.repo=http://ivy-nexus.pentaho.org/content/groups/approved -Dtestreports.dir=bin/reports/integration-test -f engine/build.xml jacoco-integration checkstyle publish-local
ant -Dpentaho.resolve.repo=http://ivy-nexus.pentaho.org/content/groups/approved -Dtestreports.dir=bin/reports/unit-test -f ui/build.xml clean-all resolve jacoco && ant -Dpentaho.resolve.repo=http://ivy-nexus.pentaho.org/content/groups/approved -Dtestreports.dir=bin/reports/integration-test -f ui/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/trans/steps/switchcase/SwitchCase.java
  engine/src/org/pentaho/di/trans/steps/switchcase/SwitchCaseMeta.java
  engine/src/org/pentaho/di/trans/steps/switchcase/SwitchCaseTarget.java
  engine/src/org/pentaho/di/trans/steps/switchcase/messages/messages_en_US.properties
  engine/test-src/org/pentaho/di/trans/steps/switchcase/SwitchCaseMetaTest.java
  engine/test-src/org/pentaho/di/trans/steps/switchcase/SwitchCaseMetadataInjectionTest.java
  engine/test-src/org/pentaho/di/trans/steps/switchcase/SwitchCaseTest.java
  ui/src/org/pentaho/di/ui/trans/steps/switchcase/SwitchCaseDialog.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change: -3.28%:small_red_triangle_down:
Line Change: -3.87%:small_red_triangle_down:

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -.66%:small_red_triangle_down:
Complexity Change: -.93%:small_red_triangle_down:
Instruction Change: -.13%:small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change:  + 1.37%
Complexity Change:  + .58%
Instruction Change:  + .50%
Line Change:  + .27%

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Branch Change: -1.32%:small_red_triangle_down:
Complexity Change: -1.43%:small_red_triangle_down:
Instruction Change: -.33%:small_red_triangle_down:
Line Change: -.54%:small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change:  + .10%

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change: -.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Instruction Change:  + .09%

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change:  + 2.00%
Complexity Change:  + 1.89%
Instruction Change:  + .17%

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -.06%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change:  + 2.35%
Complexity Change:  + 2.70%
Instruction Change:  + .18%
Line Change:  + .48%

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change:  + .07%

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta

Instruction Change: -.14%:small_red_triangle_down:

org.pentaho.di.trans.steps.switchcase.SwitchCaseMeta

Branch Change:  + 5.91%
Complexity Change:  + 3.84%
Instruction Change:  + 3.01%
Line Change:  + 3.42%
Method Change:  + 2.70%

org.pentaho.di.trans.steps.systemdata.SystemDataMeta

Branch Change: -2.04%:small_red_triangle_down:
Complexity Change: -2.04%:small_red_triangle_down:
Instruction Change: -.32%:small_red_triangle_down:
Line Change: -.81%:small_red_triangle_down:

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change: -.91%:small_red_triangle_down:
Complexity Change: -.82%:small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change:  + .53%

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.79%:small_red_triangle_down:
Complexity Change: -1.19%:small_red_triangle_down:
Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.trans.Trans

Branch Change: -.10%:small_red_triangle_down:
Complexity Change: -.14%:small_red_triangle_down:
Instruction Change: -.09%:small_red_triangle_down:
Line Change: -.11%:small_red_triangle_down:

License header violations

:heavy_exclamation_mark: Copyright year is not the current year on engine/src/org/pentaho/di/trans/steps/switchcase/SwitchCaseTarget.java. Found 2013. Was expecting 2016

:heavy_exclamation_mark: Copyright year is not the current year on engine/test-src/org/pentaho/di/trans/steps/switchcase/SwitchCaseTest.java. Found 2013. Was expecting 2016

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/engine/test-src/org/pentaho/di/trans/steps/switchcase/SwitchCaseMetadataInjectionTest.java

:heavy_exclamation_mark: Line has trailing spaces at Line: 95

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dpentaho.resolve.repo=http://ivy-nexus.pentaho.org/content/groups/approved -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -Dpentaho.resolve.repo=http://ivy-nexus.pentaho.org/content/groups/approved -Dtestreports.dir=bin/reports/integration-test -f engine/build.xml jacoco-integration checkstyle publish-local
ant -Dpentaho.resolve.repo=http://ivy-nexus.pentaho.org/content/groups/approved -Dtestreports.dir=bin/reports/unit-test -f ui/build.xml clean-all resolve jacoco && ant -Dpentaho.resolve.repo=http://ivy-nexus.pentaho.org/content/groups/approved -Dtestreports.dir=bin/reports/integration-test -f ui/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/trans/steps/switchcase/SwitchCase.java
  engine/src/org/pentaho/di/trans/steps/switchcase/SwitchCaseMeta.java
  engine/src/org/pentaho/di/trans/steps/switchcase/SwitchCaseTarget.java
  engine/src/org/pentaho/di/trans/steps/switchcase/messages/messages_en_US.properties
  engine/test-src/org/pentaho/di/trans/steps/switchcase/SwitchCaseMetaTest.java
  engine/test-src/org/pentaho/di/trans/steps/switchcase/SwitchCaseMetadataInjectionTest.java
  engine/test-src/org/pentaho/di/trans/steps/switchcase/SwitchCaseTest.java
  ui/src/org/pentaho/di/ui/trans/steps/switchcase/SwitchCaseDialog.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -.39%:small_red_triangle_down:
Complexity Change: -.51%:small_red_triangle_down:
Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.01%:small_red_triangle_down:
Complexity Change: -1.06%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.job.entries.getpop.MailConnectionMeta

Branch Change: -1.00%:small_red_triangle_down:
Complexity Change: -1.41%:small_red_triangle_down:
Instruction Change: -.30%:small_red_triangle_down:

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -.66%:small_red_triangle_down:
Complexity Change: -.93%:small_red_triangle_down:
Instruction Change: -.13%:small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change:  + 1.37%
Complexity Change:  + .58%
Instruction Change:  + .50%
Line Change:  + .27%

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Branch Change: -1.32%:small_red_triangle_down:
Complexity Change: -1.43%:small_red_triangle_down:
Instruction Change: -.33%:small_red_triangle_down:
Line Change: -.54%:small_red_triangle_down:

org.pentaho.di.trans.steps.combinationlookup.CombinationLookupMeta

Instruction Change: -.04%:small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change:  + 1.04%
Complexity Change:  + 1.05%
Instruction Change:  + .14%
Line Change:  + .37%

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change: -.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -2.35%:small_red_triangle_down:
Complexity Change: -2.70%:small_red_triangle_down:
Instruction Change: -.18%:small_red_triangle_down:
Line Change: -.48%:small_red_triangle_down:

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta

Instruction Change: -.14%:small_red_triangle_down:

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.switchcase.SwitchCaseMeta

Branch Change:  + 5.91%
Complexity Change:  + 3.84%
Instruction Change:  + 3.01%
Line Change:  + 3.42%
Method Change:  + 2.70%

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change:  + 1.79%
Complexity Change:  + 1.19%
Instruction Change:  + .07%

org.pentaho.di.trans.Trans

Branch Change:  + .10%
Complexity Change:  + .14%
Instruction Change:  + .09%
Line Change:  + .11%

Integration test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.trans.step.BaseStep

Branch Change: -.29%:small_red_triangle_down:
Complexity Change: -.18%:small_red_triangle_down:
Instruction Change: -.05%:small_red_triangle_down:
Line Change: -.08%:small_red_triangle_down:


items_custom.json should support "text_id" field

For replacement vanilla items, we have to duplicate the description, which is a bug / violates DRY.
Instead, we should specify the ID of the vanilla description to copy in the same way that we do with the graphic.

Fixed in a previous version


Localization - add Japanese translation

@issuehuntfest has funded $20.00 to this issue. See it on IssueHunt

@aashutoshrathi has submitted a pull request. See it on IssueHunt

@hasnayeen has rewarded $18.00 to @aashutoshrathi. (Total deposit: $20.00, Repository reward(0%): $0.00, Service fee(10%): $2.00) See it on IssueHunt


Feature request: hook for sanitization function

There are a number of potential attacks involving logs. They can be mitigated through sanitization. Users of the logrus package (and others) can sanitize data before passing it to logrus, but the broad spread of calls into the logging package means sanitization will sometimes be missed.
Compare the verbosity of
logger.Infof("Request %s %v %s\n", method, url.Path, extraReqInfo)
with
logger.Infof("Request %s %v %s\n",sanitize(method), sanitize(url.Path), sanitize(extraReqInfo))
...and consider how many log statements are created during the debugging process.
If sanitization is built into the logging package - for example in the form of a callback function provided as a configuration parameter - then security will be enhanced.

Closing in favor of https://github.com/sirupsen/logrus/issues/1307


Add couchpotato to incubator

What this PR does / why we need it:
Adds an incubator chart for CouchPotato
Which issue this PR fixes
N/A
Special notes for your reviewer:
This would be my first newly contributed chart, feedback welcome
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md

@unguiculus @lachie83 This is ready for review when you have a moment - feedback welcome


Ratelimiting

Feature request: add ratelimiting for the image upload endpoint.
ex: users can upload x images every x minutes or something. Configuration options available for it would also be just dandy.

I think @diced will migrate to Next 12 and rewrite the middleware, then there will be ratelimiting.

it'll be added to 3.3 don't get your hopes up that it'll be next 12 though

allah


[ARCTIC-1204][AMS] Start a new scheduled thread to refresh tables

Why are the changes needed?
fix #1204
Brief change log
When a brand-new table/enable self-optimizing table is added or a exsting table is dropped, AMS can update the managed table list in a timely manner.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before making a pull request

Documentation

Does this pull request introduces a new feature? (no)
If yes, how is the feature documented? (not documented)

Thanks for your contributions

LGTM