Azure AD Auth: Invalid Compact JWE

Steps to reproduce
Steps:

npx create-toolpad-app@latest --studio dog-app
cd dog-app
npx run dev
Setting up Azure AD Authentication
Sign in with Azure AD

Current behavior
Redirected to MS, selected the account and returned to the login page with an error in the console:
[auth][error] JWTSessionError: Read more at https://errors.authjs.dev#jwtsessionerror [auth][cause]: JWEInvalid: Invalid Compact JWE at compactDecrypt (<>/dog-app/node_modules/jose/dist/node/esm/jwe/compact/decrypt.js:13:15) at jwtDecrypt (<>/dog-app/node_modules/jose/dist/node/esm/jwt/decrypt.js:5:29) at Object.decode (<>/dog-app/node_modules/@auth/core/jwt.js:67:31) at Module.session (<>/dog-app/node_modules/@auth/core/lib/actions/session.js:17:39) at AuthInternal (<>/dog-app/node_modules/@auth/core/lib/index.js:35:38) at async Auth (<>/dog-app/node_modules/@auth/core/index.js:111:34) at async <>/dog-app/node_modules/@toolpad/studio/dist/cli/index.mjs:366:24
Expected behavior
Expects to log in.
Context
I registered the application in Azure, set env according to the instructions.
Your environment

npx @mui/envinfo
  System:
    OS: Windows 11 10.0.22631
    Browser: Microsoft Edge for Business 130.0.2849.46
  Binaries:
    Node: 20.15.1 - ~\Node\node.EXE
    npm: 10.7.0 - ~\Node\npm.CMD
    pnpm: Not Found
  Browsers:
    Chrome: Not Found
    Edge: Chromium (127.0.2651.74)
  npmPackages:
    @emotion/react:  11.13.3
    @emotion/styled:  11.13.0
    @mui/base:  5.0.0-beta.59
    @mui/core-downloads-tracker:  6.1.5
    @mui/icons-material:  6.1.4
    @mui/lab:  6.0.0-beta.12
    @mui/material:  6.1.4
    @mui/private-theming:  6.1.5
    @mui/styled-engine:  6.1.5
    @mui/system:  6.1.4
    @mui/types:  7.2.18
    @mui/utils:  6.1.4
    @mui/x-charts:  7.21.0
    @mui/x-charts-vendor:  7.20.0
    @mui/x-data-grid:  7.21.0
    @mui/x-data-grid-premium:  7.21.0
    @mui/x-data-grid-pro:  7.21.0
    @mui/x-date-pickers:  7.21.0
    @mui/x-date-pickers-pro:  7.21.0
    @mui/x-internals:  7.21.0
    @mui/x-license:  7.21.0
    @mui/x-tree-view:  7.21.0
    @toolpad/core:  0.8.1
    @toolpad/studio: latest => 0.8.1
    @toolpad/studio-components:  0.8.1
    @toolpad/studio-runtime:  0.8.1
    @toolpad/utils:  0.8.1
    @types/react:  18.3.12
    react:  18.3.1
    react-dom:  18.3.1
    typescript:  5.5.4

Search keywords: azure authentication

Hi, thanks for reporting this issue!
Did the Azure authentication use to work before updating to v0.8.1 of @toolpad/studio, or have you ever only tried in this last version?
Just making sure it's not a regression due to any latest changes.
Also just in case before I look further into this: did you try clearing your cookies? Just to be sure!

I tested only on the latest version. Interestingly, it uses three accounts that give different results, and I did not set different permissions for them:
Account 1:
[auth][cause]: JWEInvalid: Invalid Compact JWE     at compactDecrypt (file:///C:/Users/dominik.olczak/Documents/GitHub/dog-app/node_modules/jose/dist/node/esm/jwe/compact/decrypt.js:13:15)     at jwtDecrypt (file:///C:/Users/dominik.olczak/Documents/GitHub/dog-app/node_modules/jose/dist/node/esm/jwt/decrypt.js:5:29)     at Object.decode (file:///C:/Users/dominik.olczak/Documents/GitHub/dog-app/node_modules/@auth/core/jwt.js:67:31)     at Module.session (file:///C:/Users/dominik.olczak/Documents/GitHub/dog-app/node_modules/@auth/core/lib/actions/session.js:17:39)     at AuthInternal (file:///C:/Users/dominik.olczak/Documents/GitHub/dog-app/node_modules/@auth/core/lib/index.js:35:38)          at async Auth (file:///C:/Users/dominik.olczak/Documents/GitHub/dog-app/node_modules/@auth/core/index.js:111:34)     at async file:///C:/Users/dominik.olczak/Documents/GitHub/dog-app/node_modules/@toolpad/studio/dist/cli/index.mjs:366:24
Account 2:
[auth][error] AccessDenied: AccessDenied. Read more at https://errors.authjs.dev#accessdenied     at handleAuthorized (file:///C:/Users/dominik.olczak/Documents/GitHub/dog-app/node_modules/@auth/core/lib/actions/callback/index.js:405:15)     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)     at async Module.callback (file:///C:/Users/dominik.olczak/Documents/GitHub/dog-app/node_modules/@auth/core/lib/actions/callback/index.js:63:30)     at async AuthInternal (file:///C:/Users/dominik.olczak/Documents/GitHub/dog-app/node_modules/@auth/core/lib/index.js:27:24)     at async Auth (file:///C:/Users/dominik.olczak/Documents/GitHub/dog-app/node_modules/@auth/core/index.js:111:34)     at async file:///C:/Users/dominik.olczak/Documents/GitHub/dog-app/node_modules/@toolpad/studio/dist/cli/index.mjs:366:24
Account 3:
Successful login


Add filename and line number to logs

Is this a BUG REPORT or FEATURE REQUEST?: FEATURE REQUEST
We should enable logging https://github.com/argoproj/argo/blob/6c18039be962996d971145be8349d2ed3e396c80/workflow/controller/operator.go#L95 with filename and line # in argo https://github.com/sirupsen/logrus/issues/63#issuecomment-433746888 with short file path https://github.com/sirupsen/logrus/issues/63#issuecomment-476486166 .
in order to do, this we have to upgrade our logrus pkg to v1.2.0 or after https://github.com/sirupsen/logrus/issues/63#issuecomment-435566044

Would you like to submit a PR for this?


Migrate to Alice v3

It's time to prepare HautelookAliceBundle 2.x. Nelmio Alice 3.0 beta is coming out this weak, the main features being a much more robust loader, more performant as well. The major change affecting HautelookAliceBundle besides a few changes in the interfaces is that the persistence layer has been dropped. To fill the gap I released https://github.com/theofidry/AliceDataFixtures.
Now given those two libraries, I don't HautelookAliceBundle need to be re-focused, but rather all the tweaks and patched made for alice 2.x and the persistence layer should now be removed, drastically simplifying the bundle.
@Soullivaneuh anything else in mind for HautelookAliceBundle 2.x?

Is there a timeline to migrate to v3 of Alice?

I don't think there is much left to do in #249, I still need to add some tests but the gist of it is there. So we can hope for a beta release soon (in the two coming weeks?).
There still a lot to finish before a stable release, in part due to the fact that I have to keep up with four libraries:

https://github.com/nelmio/alice
https://github.com/theofidry/AliceDataFixtures
https://github.com/theofidry/AliceBundleExtension
https://github.com/hautelook/AliceBundle (this library)

but the beta for HautelookAliceBundle should be close.


WIP: Feature: specify subdir in repo as template

This change allows to specify a sub-folder inside a git repository for use as the template.
By doing so, it allows us to e.g. generate projects from examples inside a repo, or to have a repo that contains multiple templates.
Relates to #47 and #78
This pull-request will also make #295 obsolete, as there is no need to specify folders as templates, and thus we avoid a lot of the problems specified in #295 and #291, allowing for easier implementation of #291 (and thus #211).
@sassman Sorry for the long timeout here - as it seems we get no feedback for #295, I suggest this one instead. I still like the idea of --pick for #291, but it kinda conflicted and this is my proposal to remedy that!

@sassman After the changes to 0.7.0 with SSH, I can't run tests neither locally nor here on Github - any ideas?

Locally should work. What's the error message?
The github action right now failed because of access to a secret variable.. I will fix it ASAP

test basics::ssh_remote::it_should_support_a_public_repo ... FAILED
test basics::ssh_remote::it_should_support_a_private_repo ... FAILED
failures:
---- basics::ssh_remote::it_should_support_a_public_repo stdout ----
thread 'basics::ssh_remote::it_should_support_a_public_repo' panicked at 'Unexpected failure.
code-1
stderr="Error: ⛔   Git Error: Failed to retrieve list of SSH authentication methods: Failed getting response; class=Ssh (23)\n"
command="/home/johnny/projects/github/cargo-generate/target/debug/cargo-generate" "generate" "--git" "git@github.com:ashleygwilliams/wasm-pack-template.git" "--name" "foobar-project"
code=1
stdout="🔧   Creating project called `foobar-project`...\n💡   Using private key: `$HOME/.ssh/id_rsa` for git-ssh checkout\n"
stderr="Error: ⛔   Git Error: Failed to retrieve list of SSH authentication methods: Failed getting response; class=Ssh (23)\n"
', /home/johnny/.cargo/registry/src/github.com-1ecc6299db9ec823/assert_cmd-1.0.7/src/assert.rs:155:13
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
---- basics::ssh_remote::it_should_support_a_private_repo stdout ----
thread 'basics::ssh_remote::it_should_support_a_private_repo' panicked at 'Unexpected failure.
code-1
stderr="Error: ⛔   Git Error: Failed to retrieve list of SSH authentication methods: Failed getting response; class=Ssh (23)\n"
command="/home/johnny/projects/github/cargo-generate/target/debug/cargo-generate" "generate" "--git" "git@github.com:cargo-generate/wasm-pack-template.git" "--name" "foobar-project"
code=1
stdout="🔧   Creating project called `foobar-project`...\n💡   Using private key: `$HOME/.ssh/id_rsa` for git-ssh checkout\n"
stderr="Error: ⛔   Git Error: Failed to retrieve list of SSH authentication methods: Failed getting response; class=Ssh (23)\n"
', /home/johnny/.cargo/registry/src/github.com-1ecc6299db9ec823/assert_cmd-1.0.7/src/assert.rs:155:13
failures:
basics::ssh_remote::it_should_support_a_private_repo
basics::ssh_remote::it_should_support_a_public_repo
test result: FAILED. 37 passed; 2 failed; 1 ignored; 0 measured; 0 filtered out; finished in 10.91s
error: test failed, to rerun pass '--test integration'

Famous words: It works on my machine(s) 😄

Hmm, I probably shouldn't have force pushed my latest commit with the review changes - but I thought: hey @sassman is keen on having a clean git history, so I'll make it easy...
And now I don't know how to get Github to recognize I've made the needed changes 😦

I recommend squashing at the end of the review when all things are settled.

good job and many thanks @taurr


Fix UserProvider to use Laravel 6 Array Helper

After upgrading Laravel to version 6 the UserProvider no longer worked.
When trying to log in I started getting the following error:
Call to undefined function Sburina\Whmcs\array_has()
To fix this I have included the Array Helper in Laravel 6 and updated aaray_has to Arr::has.

I've somehow missed that one.
Thank you, @tecreasey !


[FEATURE] Item Purchase Indicators with item name

Hey mate,
Sorry for not using the template, but it doesn't refer to the description of the suggestion. I hope that's not a problem.
I asked the community and one suggestion to improve your tool would be to add a bar with the name of the item you purchased. CBLOL takes advantage of something like this and it really improves the reception and draws attention. I would like to thank you because you are doing a really good job.
Example from CBLOL:

Greetings.

Support for item name indicators added in latest release


Successfully Integrated Date Picker

Good evening sir, I have successfully integrated a Date Picker in the dashboard.

@Arnab55555 Let me review it, and then I will merge it.

@Arnab55555 Everything looks good so far. Can you also add a time-picker? We need to include both a date-picker and a time-picker.

Sir I have added the Time Picker as requested

@Arnab55555 Everything looks good after testing.


About us and Team

Hey, Ankita
It looks for good for mobile but not responsive for laptops like Macbook 13 (width of 1400px * height 896px)

@Ankitapanda25 The issue persists on my laptop's resolution - [2560:1600]


Git tag v2.1.0 missing

The Git tag for the latest release (v2.1.0) is missing. This causes all the links in the documentation that point to the source code to return 404's. For example
https://hexdocs.pm/phoenix_gen_socket_client/Phoenix.Channels.GenSocketClient.html#joined?/1 currently links to this broken link:  https://github.com/Aircloak/phoenix_gen_socket_client/blob/v2.1.0/lib/gen_socket_client.ex#L175

Good catch, thanks for reporting! The tag is now pushed.


Feature/ankr rpcs

Description
Added Ankr rpc provider support and documentation alphabetically.
Additional Information

[x] I read the contributing docs (if this is your first contribution)

Your ENS/address:
codingwithmanny.eth

@tmm Did you want to wait for the other testnets to be added to ethers to revise this to include the testnets or fair to proceed with these ones and add the additional testnets in a separate PR?
Ethersjs commit with Ankr Providers
https://github.com/ethers-io/ethers.js/commit/d9f45b3b9db92c72f9c606bab8315d0eb02fec70

@codingwithmanny can wait if cool. @jxom is looking into a streamlined provider and chains API too that might inform some of this.

@tmm would this replace both infura and alchemy providers and potentially support quicknode all in one fell swoop :) ?
If you think it's going to take a while, would be ok if we merged it in the meantime, I have a use-case for it where I want to work with Ankr's premium API which apparently has a load balancer that uses all providers, if one fails, it goes to the next.

Good news! Testnets were merged in ethers.js 5.6.5
https://github.com/ethers-io/ethers.js/issues/2949#issuecomment-1115749557

@tmm this the direction you were talking about? If so 🔥
https://github.com/tmm/wagmi/pull/408

@tmm this the direction you were talking about? If so 🔥 #408

you are correct!


126 unrecognized jsdoc tag

PR for issue #126.
This is a quick and dirty fix to avoid frustration when using jsdoc versions producing a list of files for a package ("file:"). This attribute is not interpreted yet however, just filtered out.

This is a lint issue related to code previously here.

I'm going to close this because the original author cannot update the pull request, and we have no way of testing the changes.
However someone else with the original issue can feel free to pick this up and make a fix themselves.


feat: propagate doc comments on flags and arguments

the crux of the pr is https://github.com/near-cli-rs/interactive-clap/pull/26/commits/ed605cb80e33cec124dea1627df525f40e1cd818
and https://github.com/near-cli-rs/interactive-clap/pull/26/commits/a6406dac3318aa48c8f7d01e2e4ab637fee04580
the rest is some refactoring and splitting big chunks of code into smaller functions

introspect feature can be used to debug how derive proc macros work similar to the following:
# interactive-clap-derive folder
cargo test test_doc_comments_propagate --features introspect -- --nocapture
# from repo root
cargo run --example struct_with_subargs --features interactive-clap-derive/introspect


Update sorting field descriptions

Update sorting field descriptions - add info where channel slug is required.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-6753 Sort products by PUBLISHED_AT don't work as expected


Getting 404 Error

after following the Usage given in GiT i am Getting 404 Error like below not sure what i missed.

I believe your issue is most likely the result of #19.  The compiled JS is not currently include in the NPM package and so your project is unable to locate the file.
The current workaround is to tsc from within the package directory, which will generate the necessary files.  This should be resolved in the next package version, though.


Verdaccio is always listening on IPv6 port on ubuntu-latest on GitHub Actions

Your Environment

verdaccio version: @latest
node version [12.x.x, 14.x.x]:
package manager: [npm@7, pnpm@6, yarn@2]
os: ubuntu-latest on GHA
platform: GHA

Describe the bug
Hey all,
Nicola Corti from the React Native Team at Meta.
We do use Verdaccio on GitHub actions on our setup
I've just noticed that the latest deployment of the ubuntu-latest runner on GitHub actions (version 20240609.1.0) is breaking our CI on React Native causing it to take 6+ hours to complete.
This happens because Verdaccio starts to listen on a IPv6 address.
We do use Verdaccio in our CI setup to test NPM packages. For some reasons, Verdaccio is always listening on a IPv6 port since 20240609.1.0.
I'm unsure what could be causing this, but I tried all possible ways to force Verdaccio to listen on a IPv4 address/port. It seems like to always fallback to an IPv6.
I've also reported this bug to the runner-images repo (see https://github.com/actions/runner-images/issues/10061). I'm wondering if you folks have any insights on what could be happening here.
I also created a reproducer here:
https://github.com/cortinico/repro-ipv6-gha-issue/
You can see in the latest run how I'm running Verdaccio and using net-stat to show the open ports:
https://github.com/cortinico/repro-ipv6-gha-issue/actions/runs/9503903039/job/26195373761
Seems like Verdaccio is listening on a IPv6 port:

To Reproduce
Minimal Reproducer is here: https://github.com/cortinico/repro-ipv6-gha-issue/
Expected behavior
Verdaccio should listen on http://localhost:4873 IPv4 by default
Configuration File (cat ~/.config/verdaccio/config.yaml)
https://github.com/cortinico/repro-ipv6-gha-issue/blob/main/verdaccio.yml

I can't tell. Did you try setting listen to both ipv4 and ipv6?
https://github.com/orgs/verdaccio/discussions/4086#discussioncomment-7393623
https://verdaccio.org/docs/configuration#listen-port

Is it working now?


Added support to get parameters from Query

I need access to the parameter values which were sent along with the request. Would it be OK to add access to the key values like this?
Alternatively,  access to the whole map would also work, I just thought because the values can already be accessed by get() then it would be at least helpful to get the set of keys as well.

Hello, thanks for the contribution !
I agree with you. I think we could have both. If we don't want to directly expose the Query's underlying map, maybe could we add iterator support ? So that you could at least iterate over the list of all pairs ?

For sure, that would work too. I already thought that exposing is not desirable, which is why in this PR a copy of the parameters is returned. But providing const iterators would certainly be an alternative solution. Would you like me to modify this PR with iterators instead?

Yes I will modify the PR, hopefully will get around to it in the next couple of days, will put it on top of my TODO  list :)

Added access to begin() and end() iterators, is it something like this you had in mind?

Ok, formatting adjusted.

Oh no, how silly, this must have snug in with copy and paste :( sorry, I really should have looked more closely.
Since this is already merged, would you like to just update it in your master branch directly, or should I submit a new PR for the fix?

Jennifer, please submit a PR to fix it.  I'll merge it immediately.


paket.dependencies with multiple sources

In paket.dependencies file, Is it possible to mix  different sources from:

https://api.nuget.org/v3/index.json
local subdirectory containing .nupkg files?

The following configuration seems to fail:
storage: none
framework: net8.0
source https://api.nuget.org/v3/index.json

nuget Microsoft.AspNetCore.App.Ref 7.0.15
nuget Microsoft.NETCore.App.Ref 7.0.15

group local
    framework: net8.0
    source other-packages

    nuget my.local.nuget 13.0.3

I try doing this:
csharp_binary(
    name = "hello-world",
    srcs = [
        "Program.cs"
    ],
    target_frameworks = [ "net8.0" ],
    targeting_packs = [
<EMAIL_ADDRESS><EMAIL_ADDRESS>    ],
    deps = [
<EMAIL_ADDRESS>    ],
)

But the "bazel build" command failed:
ERROR: /private/var/tmp/_bazel_pta/1e6e2676e185910c118ab175fa8c6d5c/external/paket.local/newtonsoft.json/13.0.3/BUILD.bazel:5:15: @paket.local//newtonsoft.json/13.0.3:13.0.3 depends on @nuget.newtonsoft.json.v13.0.3//:newtonsoft.json.13.0.3.nupkg in repository @nuget.newtonsoft.json.v13.0.3 which failed to fetch. no such package<EMAIL_ADDRESS>java.io.IOException: Bad URL: other-packages/package/newtonsoft.json/13.0.3

No, this is not possible. Paket does support referencing file with http file://path/to/file but that is not supported in rules_dotnet.


Shadow Dom Conflict when integrating with Angular2

How to ask Polymer to work nicely with Angular2?
I am facing the content distribution issue.

Can you please add more details to this bug. A code snippet would be nice too.
There's an experimental patch-dom feature that you can try to make other frameworks place nicely with Shady DOM. Example usage: https://github.com/Polymer/polymer/blob/c263ad590639b565fadc7f94f162b398f219d55a/test/smoke/patch/patch-dom.html

Rob discussed this content distribution issue on AngularU.
Let me know if you want to look into my repo?

@ebidel , After using webcomponets.js instead of lite one and with below code, that issue got resolved.
But I want to know how to use this patch-dom and how this approach is different than current solution??
	var Polymer = Polymer || {};
	Polymer.dom = 'shadow';

Fair enough, When filing a bug though, you need to help maintainers understand "this content". We don't have context from AngularU :) Unless you have a snippet of code that's not working, I think this bug post would a better question for slack/mailing list.
To answer your question, webcomponents.js is the full shadow dom polyfill. Angular/other frameworks work well with shadow dom b/c the composition model is transparent to them. If you use it, you don't need Polymer.dom = 'shadow'; because the polyfills will use shadow dom (or poyfill it in non-supported browsers). In wc-lite.js, you get shady dom instead of shadow dom. Polymer.dom = 'shadow'; opts you in to native shadow dom if it's available in the browser.
In 1.0, shady dom requires usage of the Polymer.dom() APIs to work with the DOM. Angular/other frameworks don't know about these APIs, therefore things can break. An example is using Angular's data binding system to stamp content. Under the hood, it will try to do so using node.appendChild() instead of Polymer.dom.appendChild(). patch-dom.html wraps the native DOM methods with the correct Polymer.dom() calls. In that way, frameworks making those DOM calls will let shady dom know things like distribution have changed.

Thanks for the explanation @ebidel,
Sorry regarding the less context(I thought its a common issue). Will take care next time :)

Hi Eric, this explanation of how shady dom works and opts in to use the native shadow dom when available and the overall explanation of the patch-dom is really great. I feel like an explanation on several occasions on the polymer-project site would probably be very useful until shadow dom is widely supported!

Have you seen @sjmiles most excellent article? https://www.polymer-project.org/1.0/articles/shadydom.html

@tapas4java
Have you been able to get a sample app working that combines Angular2 with Polymer and uses this patch-dom?  It would be great to see it in action. Thanks.

@Analect
I didn't tried patch-dom approach however native shadow dom approach works for me.

@tapas4java
Thanks. I don't suppose you have a public repo with Angular/Polymer together you could share?


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/1052 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

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

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_main_2022-02-18-03-36-44 has been deleted or has been force pushed

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


Fails to copy necessary scripts if opts.scripts.length > 9

I've been debugging this one for a while and can't seem to find any reason this would happen, so I thought I'd see if it makes any sense to you since you're much more familiar with the code.
I'm trying to add a bunch of scripts so that my examples will run, and it seems that if the array of scripts I pass can have up to 9 scripts, but as soon as I pass the tenth (doesn't seem to matter which scripts) it copies all my scripts over but begins to fail copying over angular-bootstrap.js, docs.js, and prettify.js for some reason.
Any idea what's going on?

I have encountered this also. I worked around it by adding a step to my build that concatenates all the scripts first. It's really annoying to have to do this and makes debugging harder since I can't find out how to make the sourcemaps load.


RUN error: count - ERROR  Command failed with exit code 134

Hi, I encountered some errors when running the code below:
dnbc4tools rna run   --cDNAfastq1 cDNAlib-A3_raw_1.fq.gz   --cDNAfastq2 cDNAlib-A3_raw_2.fq.gz   --oligofastq1 oligolib-A3_raw_1.fq.gz   --oligofastq2 oligolib-A3_raw_2.fq.gz   --genomeDir ref   --name A3 --threads 50

The error is depicted in the picture:

I noticed a similar issue reported by others. However, based on the solution provided, my "CB_UB_count.txt" file is not empty. Thus, I'm unsure about the exact problem or what steps to take next. Could you offer any guidance? Thank you!

Screenshot 01.data

ls -lh，look at the storage space of the file
and the contents of the alignment_report.csv and anno_report.csv files.

01.data:

alignment_report.csv：

anno_report.csv：

There is something wrong with your gtf file。

Thank you for your solution!
I have two GTF files; this time, I used the first one. I thought it meets the conditions mentioned above.
However, if the first one is not okay, is the other one correct? Thank you!
The first one:

The other one:

Oh, I must mention that, initially, I ran the code from your tutorial:
dnbc4tools tools mkgtf --ingtf ref.gtf --output genes.filter.gtf --type gene_type
However, the output file genes.filter.gtf is empty, and subsequent code failed. Consequently, I skipped that step and used the original GTF as the input for the "dnbc4tools rna run". I'm uncertain if this has any impact.

The filtering step is optional.
The other one is OK

Thank you for your help! I'll give it a try.

Excuse me, I used the other GTF file, but I encountered another error...

Your environment is not installed properly. You can test whether the R library (DropletUtils) is installed successfully. If the installation is not successful, reinstall dnbc4tools. If there is no problem with the installation, it may be that the .Rprofile file exists in the home directory or the R-related path exists in ~/.bashrc, causing the location of the identified package to be specified.
if reinstall, you can modify the mirror in yaml to ensure that the installation does not go wrong.
name: dnbc4tools
channels:
  - http://mirrors.bfsu.edu.cn/anaconda/pkgs/r/
  - http://mirrors.bfsu.edu.cn/anaconda/pkgs/main/
  - http://mirrors.bfsu.edu.cn/anaconda/pkgs/free/
  - http://mirrors.bfsu.edu.cn/anaconda/cloud/conda-forge/
  - http://mirrors.bfsu.edu.cn/anaconda/cloud/bioconda/
  - bioconda
  - conda-forge
  - http://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/msys2/
dependencies:

Thank you! I've resolved the issue. When I tried to load DropletUtils, it prompted me about the missing S4Arrays package. After installing it, everything worked smoothly, and the code is now running fine.


v0.15.14

Please review, I think this may fix an issue with purs-nix: https://github.com/purs-nix/purs-nix/issues/49

@JordanMartinez, @f-f, @justinwoo , would someone review at their convenience?

I've pushed a 0.15.14 tag here. Sorry about not doing so earlier.

Thank you


Stats component libraries

What library do we want to use for stats - options from class were scikit-learn (Pets team chose this one due to documentation ease of understanding), statsmodels, seaborn can do statistical analysis
What library do we want to use for viz - options from class include plotly (more interactive), seaborn

Results: Using seaborn for viz and scipy for statistics


Feature: Frontend Update for Live Stream Support

As needed by (#3), live stream support needs to be added to the frontend. This also necessitates adding an option to choose between live stream or standard offline processing.
Objectives:

Add an option for users to select either "Download Video" or "Live Stream."
Update input fields dynamically based on selection (YouTube link or a stream URL).
Validate the proper playback of the processed feed

would like to work on this.

Hi @AschalewMathewosDamtew, thanks for your interest!
I've assigned the issue to you, and we have someone for #3 as well now, which should help in testing the full functionality.
Until that is ready, we can already test if we can loop back live streams, that is, without filtering via the MediaProcessor


Add tabBarProps prop to Tabs and accept Tabs of only 1 element

Closes #159

Could you also add the information in the docs (configuration table)?
Thanks :)

Not sure you get a notif when there is a new @tleunen, but done ;-)

hello @tleunen - could you maybe publish a new version of react-mdl including this PR ? My site doesn't look that pretty without it :) Thanks !

Released as v1.1.0. Sorry for the delay @slybridges

Top, merci Tommy ;-)


How do I display the complete list items with focus on the input?

I want to be display the list items by typing a letter and also displayed the complete list items by focus on the input too.
How can I do this?

Now, just typing a letter in the list is displayed.

I think your concern is about this
hideResults={query.length < 3 ? true : false}
you can change the length to get your result

I think your concern is about this
hideResults={query.length < 3 ? true : false}
you can change the length to get your result

thank you,
But I mean to open the list with focus on input without typing.

I used:
ref={el => this._dropdown_3 = el}
and call this._dropdown_3.show() in button

I use the following solution
<Autocomplete hideResults={this.state.hideResults} onFocus={() => this.setState({ hideResults: false })} renderItem={({ item, i }) => (   <TouchableOpacity onPress={() => this.setState({ hideResults: true })}>     <Text>{item.name}</Text>   </TouchableOpacity> )} />

working for me:
hideResults={isHideResultsAutocomplete}
onFocus={() => setHideResultsAutocomplete(false)}
onEndEditing={() => setHideResultsAutocomplete(true)}

working for me:
hideResults={isHideResultsAutocomplete}
onFocus={() => setHideResultsAutocomplete(false)}
onEndEditing={() => setHideResultsAutocomplete(true)}

Yes,
I used the same
Anyway, thank you

The issue with this approach is that if you select an item, but then delete it and try typing again, the results does not show. I've tried adding "onEditing={() => setHideResults(false)}" but that hasn't solved that edgecase for me. Anyone else encounter this issue?


Incorrectly highlighted regex

Issue
In some conditions a regex is highlighted like a comment and vice versa. The issue is originally filed at Chromium project. Relevant issues are this, this, this, this and this. Almost all issues have been merged into the last one due to similarity.
Test cases
The highlighter fails for the following test cases:
[.../\//];

void/\//;

do/\//
while(0);

if(0);else/\//;

`${async++}//`;
`${await++}//`;
`${let++}//`;

+{1:+{}/undefined};

`${{}/ /\//}`;

+class extends{1:{}/ /\//}{};

(a={}/ /\//)=>a;

{}/./gym?
{}/ /\//:
{}/ /\//;

Explanation
Each of the test cases represents a valid JavaScript code according to the latest ES standard specification. However, such examples, although they are valid, they will never appear in a serious script, until someone's intent is to craft a programming puzzle. For example, the first test case is useless until someone implements RegExp.prototype[Symbol.iterator]. Another example (the second test case) makes no sense, because it is equivalent to undefined and there's no need for a regex at all.
Therefore, if fixing these issues will yield a huge impact on the performance, I strongly recommend not to spend time on fixing it. It would be good to cover all test cases, but if it would require major changes, impact performance or cause complexity problems, then probably it would be better not to bother with this at all.
I've just reported this because the members of Chromium project have suggested it, but I'm aware that the fix may probably have a big impact on the performance. So, you will decide if it is worth fixing the issue or not.

Are these the output of a  fuzzer?
Most of them weren't too hard to fix, and are fixed in 036beb54. Distinguishing between keyword let/await and regular identifiers by that name isn't something this mode does yet, since it seems the world is moving on from using them as identifiers. It's probably also not too hard, but it seems like it's not worth it.

Thanks for the fix. As answer to your question, no fuzzer is used. I found the examples manually.


[bug] Enumerate hidden directories on linux

Describe the bug
Hi, I'm using in my code
readDir(path, { dir: BaseDirectory.Home, recursive: false });

from module "fs" and there is impossible to specify all subdirectories in "allowlist" -> "scope"
"**" can browse non-hidden directories like
"$HOME/**/*.*"

, but not hidden directories.
".**" is not accepted by tauri. And print error:
thread 'main' panicked at 'error while running tauri application: GlobPattern(PatternError { pos: 10, msg: "recursive wildcards must form a single path component" })', src/main.rs:52:10

I think all the hidden, non-hidden directories can be allowed by e.g.:
$ echo {.**,**}/{.**,**}/{.*,*}

I haven't found a way, how to do this. I need to find all files across all the directories.
Reproduction
In description. You can try search e.g. directory with .git directory
Expected behavior
Enumerate hidden directories.
Platform and versions
$ npm run tauri info
<EMAIL_ADDRESS>tauri
> tauri info

[✘] Environment
    - OS: Fedora 38.0.0 X64
    ✔ webkit2gtk-4.0: 2.40.2
    ✘ rsvg2: not installed
      Visit https://tauri.app/v1/guides/getting-started/prerequisites to learn more about tauri prerequisites
    ✔ rustc: 1.70.0 (90c541806 2023-05-31) (Fedora 1.70.0-1.fc38)
    ✔ Cargo: 1.70.0
    ⚠ rustup: not installed!
      If you have rust installed some other way, we recommend uninstalling it
      then use rustup instead. Visit https://rustup.rs/
    ⚠ Rust toolchain: couldn't be deteceted!
      maybe you don't have rustup installed? if so, Visit https://rustup.rs/
    - node: 18.16.0
    - yarn: 1.22.19
    - npm: 9.5.1

[-] Packages
    - tauri [RUST]: 1.2.4
    - tauri-build [RUST]: 1.2.1
    - wry [RUST]: 0.23.4
    - tao [RUST]: 0.15.8
    - @tauri-apps/api [NPM]: 1.3.0
    - @tauri-apps/cli [NPM]: 1.3.1

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../dist
    - devPath: http://localhost:1420/
    - framework: React
    - bundler: Vite
npm notice
npm notice New minor version of npm available! 9.5.1 -> 9.7.1
npm notice Changelog: https://github.com/npm/cli/releases/tag/v9.7.1
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice

### Stack trace

_No response_

### Additional context

I'm doing my own synchonization file manager and I need sync all the files. Not skip some of them because of this issue.

In the next release, there will be an option to specify require_literal_leading_dot so you will be able to do this:
"scope": {
	"allow": ["$HOME/**"],
	"require_literal_leading_dot": false,
}

Sorry, which version it will be? I can't find a commit. I have done upgrade of the project and nothing changed
Also, I didn't understand the construction:
"scope": {
	"allow": ["$HOME/**"],
	"require_literal_leading_dot": false,
}

Does it mean this will work for my purposes?
"scope": {
	"allow": [
          "$HOME/**/.*",
          "$HOME/**/*"],
	"require_literal_leading_dot": true,
}

or this?
"scope": {
	"allow": [
          "$HOME/{.**,**}/{.**,**}/{.*,*}"
        ],
	"require_literal_leading_dot": true,
}

I need to get access for every subfolder in directory, regardless they are hidden (with . prefix) or not.

This is available in 1.4 and this should be enough for your usage:
"scope": {
	"allow": ["$HOME/**"],
	"require_literal_leading_dot": false,
}

Hmm, I'm not sure, if it is working properly, like it is supposed to.
with $HOME/** it doesn't work.
/home/ksj/ is $HOME
[Log] path not allowed on the configured scope: /home/ksj/TRASH/syncdirs/.git
[Log] path not allowed on the configured scope: /home/ksj/TRASH/syncdirs/src-tauri/target/debug/.fingerprint
...

But I have found out, this seems to be working:
        "scope": {
          "allow": [
          "$HOME/**",
          "$HOME/**/*.*",
          "$HOME/**/*.*/**"
        ],
        "require_literal_leading_dot": false
        }

This works fine for me
"scope": {
	"allow": ["$HOME/**"],
	"require_literal_leading_dot": false,
}

if still having issues, please make a minimal repro

Like I have shown in previous comment. With your code it fail in reading hidden subdirectories. Have you tried something like me? so, the git repo or project (with hidden directories) and not in $HOME directory, but lower?

that's interesting.
I tried your code again, and it is the same:
[Log] path not allowed on the configured scope: /home/ksj/syncdirs/.git

The command which returns these errors is:
readDir(path, { dir: BaseDirectory.Home, recursive: false });

I tried to remove dir: BaseDirectory.Home and set recursive to true, but it has the same result.
It is strange, that it works for you and not for me.
I can use my functional config, but it will be better to have shorter code and make this working.
I have no idea, where can be the issue. The tests are similar...

That's why I need you to make a minimal repro that doesn't work for you, along with the directory structure you're trying to read

Ok, there:
syncdirs.zip
I have hardcoded to code /home/ksj/syncdirs, so change it to place where you extract the directory.  /home/ksj is $HOME
just run "yarn && yarn tauri run" and open console window.

Please don't upload an archive of a project, instead make a public repository and upload the code there.

It is not whole project. I removed a lot of code. The short code, where we are looking for issue is just in App.tsx. Everything else is just default configuration or unused. Except the tauri config a package.json.
I put it it there, if you want to check the configuration.
I will put my code in some public repository in future, when i will check the init version is working properly.
But make a repository for just one file of code? Which cannot be used for anything else than check the issue and then will be deleted/made obsolete? I don't like it much...

But make a repository for just one file of code? Which cannot be used for anything else than check the issue and then will be deleted/made obsolete? I don't like it much...

That is pretty much what a minimal reproduction repo means, you can use a different account to host it or any other GitHub alternative. Archived files could be dangerous to download for me as a maintainer and anyone who happens to pass by this issue.
And I would recommend reading theses resourse:

https://stackoverflow.com/help/minimal-reproducible-example
Why reproduction is required

Ok, next time i will upload only app.tsx. it is only file you need. It is up to you if you want to use anything else:-).
I just wanted to create working solution for you to speed it up, when you asked for minimal reproducible solution. This code exactly reproduces the same issue i have.
So, are there any error in app.tsx, which explains the issue?

Ok, next time i will upload only app.tsx. it is only file you need. It is up to you if you want to use anything else:-).

I don't need only app.tsx, I need a fully working project but contains the minimalist code to reproduce the issue, that I can run without needing much setup which would allow me to investigate the issue faster.

I just wanted to create working solution for you to speed it up, when you asked for minimal reproducible solution. This code exactly reproduces the same issue i have.
So, are there any error in app.tsx, which explains the issue?

I haven't used the archive you posted so I don't know what is the exact issue yet, I am waiting on a public repo. No offense but anonymous archives could be dangerous to open.

For me zip package and some repo link is similarly dangerous. But ok.
There you have it in repo https://github.com/podhorsky-ksj/tauri-fs-bug

For me zip package and some repo link is similarly dangerous. But ok.

Not really, one could be investigated before-hand, and one could run arbitrary code when even opened in a zip viewer before extraction.

found the issue and it is my bad, sorry about that, the config should be:
"scope": {
	"allow": ["$HOME/**"],
	"requireLiteralLeadingDot": false,
}

confirm, with this config it is now ok.
Are you going to put this to fs documentation? https://tauri.app/v1/api/js/fs/

it is documented here https://tauri.app/v1/api/config#fsallowlistscope.requireliteralleadingdot

you shouldn't rely on the config option in the JS module documentations because they are written manually and can get out-of-sync, while the config documentation is auto-generated and will always be updated with new releases.

Hey @amrbashir, how this can be configured for Tauri 2.0? Doc is really confusing: https://v2.tauri.app/references/config/#fsscope
fs plugin seems to not have "requireLiteralLeadingDot" key in capabilities settings.

@iSuslov  in v2, the option is moved to the plugins config
{
  ..
  "plugins": {
    "fs": {
      "requireLiteralLeadingDot": false
    },
    ..
  },
  ..
}


[BUG] Change audio input/output device methods don`t work.

I'm trying to implement audio input/output device selection. Getting a list of devices works, but changing device doesnt, even though I get 0 as a result from SetAudioPlaybackDevice method.
Here is a sample of my code:
audioPlaybackDeviceManager = mRtcEngine.GetAudioPlaybackDeviceManager();
audioPlaybackDeviceManager.CreateAAudioPlaybackDeviceManager();
string deviceName = string.Empty;
string deviceId = string.Empty;
audioPlaybackDeviceManager.GetAudioPlaybackDevice(deviceIndexFromList, ref deviceName, ref deviceId);
var result = audioPlaybackDeviceManager.SetAudioPlaybackDevice(deviceId);

Desktop:

OS: Windows 11
Browser: Chrome
Version 105.0.5195.127 (Official Build) (64-bit)
Unity 2021.3.4f1

Thanks in advance!

We are having similar issues. Even demo app have this issue https://webgl.agoraguru.net/.
if customer join from headphone wired or USD, and remove the headphone during the call there is no audio for remote participants of this user.  For audio to work again he has to rejoin.
Can you please tell any tentative timelines to fix issue?

@friskyfoxy  please see if this device manager demo has the right behavior for you.
Please check the latest merged refactor7 for a change.
cc @adhawan18


Re-enable NATS test with higher timeout

Summary
Re-enable tests for the NATS messaging channel; running this with go test -count 200 ./internal/events/nats produced 5 failures with the previous 5s timeout.  Increasing the timeout to 15s avoided all timeouts.
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[x] Refactoring or test improvements (no bug fixes or new functionality)

Testing
go test -count 200 ./internal/events/nats
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[x] Included tests that validate the fix or feature.
[x] Checked that related changes are merged.

coverage: 53.132% (+0.5%) from 52.658%
when pulling eff53feae6091ea42f10fdefcbc40ece23e16277 on evankanderson:fix-nats-tests
into 87aae863100147925af55bdda575fb5bc2031c62 on stacklok:main.


Use DirectClient in BB/BE reconciler

How to categorize this PR?

/area quality
/kind bug
/priority normal
What this PR does / why we need it:
If the CachedRuntimeClients feature gate is enabled, the backup entry/bucket reconciler in the gardenlet might not execute any reconciliation the first time the backup bucket/entry is deployed.
That's because we are using an Informer created by the SharedInformerFactory here for receiving watch events, but we are actually using a cached runtime client for retrieving the object in the reconciler which uses a separate watch. This in turn means, that the cached runtime client might not have received the add event for a backup bucket/entry, while the Informer already has received an event and enqueued the object for reconciliation. The cached runtime client the gets a NotFound error and the reconciler exits early.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
In general, we should always use the same informers for registering event handlers which we also use in our controllers to retrieve the object in the beginning of the reconciliation. For example an Informer from the SharedInformerFactory and the respective Lister or an Informer from the controller-runtime cache and the cached runtime client.
I will do this refactoring in a separate PR, as it requires to not invalidate the Garden ClientSet in any case (also for version changes) which I will have to tackle first.
/cc @timuthy
Release note:

A bug has been fixed, that caused that newly created `BackupBuckets`/`BackupEntries` where not reconciled by the `gardenlet` immediately when the `CachedRuntimeClients` feature gate was enabled.

Probably we should cherry-pick this to release-v1.7.


New TV Backlight 3 Lite

Which model is this? Need the Hxxxx number.
H6099

https://eu.govee.com/collections/tv-gaming-lights/products/govee-tv-backlight-3-lite

Which type of device is this? Light, heater, leak sensor, etc.
TV Backlight Lightstrip

Which connection methods does this device have? Wifi? Bluetooth? LAN mode? All?
Wifi, Bluetooth

Anything else?

[RGBIC TV Backlight 3 Lite] is not currently supported, do create a Github issue with the following info:
{"device":"18:54:D9:38:32:34:3D:17","deviceName":"RGBIC TV Backlight 3 Lite","model":"H6099","httpInfo":{"groupId":0,"device":"18:54:D9:38:32:34:3D:17","sku":"H6099","spec":"","versionHard":"3.05.01","versionSoft":"1.00.05","deviceName":"RGBIC TV Backlight 3 Lite","deviceExt":{"deviceSettings":"{"wifiName":"MYWIFI","address":"D9:38:32:34:3D:17","bleName":"Govee_H6099_3D17","topic":"GD/c5593a1e476795e2064cb65cf29f36bd","wifiMac":"D4:AD:FC:B4:67:94","pactType":1,"pactCode":1,"wifiSoftVersion":"1.00.10","wifiHardVersion":"1.06.00","ic":36,"secretCode":"gZTZODI0PRc=","boilWaterCompletedNotiOnOff":1,"completionNotiOnOff":1,"autoShutDownOnOff":1,"sku":"H6099","device":"18:54:D9:38:32:34:3D:17","deviceName":"RGBIC TV Backlight 3 Lite","versionHard":"3.05.01","versionSoft":"1.00.05","playState":false}","lastDeviceData":"{"online":false}","extResources":"{"skuUrl":"https://d1f2504ijhdyjw.cloudfront.net/sku-img/784ef9170ab66ccdaf137d29963f5476-add_list_type_device_6099.png\",\"headOnImg\":\"https://d1f2504ijhdyjw.cloudfront.net/sku-img/42d0585f82d3c6c93b50534f9e9b3804-new_light_title_h6104_alexa_on.png\",\"headOffImg\":\"https://d1f2504ijhdyjw.cloudfront.net/sku-img/5098709790bf157447ed1f8e0b65c2ad-new_light_title_6104_alexa_off.png\",\"ext\":\"\",\"ic\":50}"},"pactType":1,"pactCode":1,"share":0,"goodsType":191,"attributesId":1,"supportScene":1}}

+1 for this


Add composer-plugin-patchset to the plugin list

Even though there's already a lot of patching plugins listed I think this one has a unique approach - among others it allows easy distribution of whole patchsets as composer packages. Additionally it's got a functional test suite.
Disclaimer: I'm the author.

Thank you for contributing, Filip! 👍


Trigger Jenkins build on successful Travis run

Right now I believe Jenkins polls for any PRs to run. Ideally we'd like Jenkins to only be triggered when Travis passes. Hopefully there are Travis triggers and Jenkins APIs to make this possible.

I remember that we tried this before in #183.  That attempt failed and I backed it out because it only used a single, static trigger.  Once that Jenkinsfile got merged into master and other users eventually rebased their working branches, every commit they made to those branches would cause an explosion of build requests in unrelated branches, like open PRs to upstream, upstream master, and everywhere in between.
I'm happy to revisit this but it will take me a little while to understand how to do it correctly in order to avoid the disaster that resulted last time.

Takeaway from yesterday's discussion: @crd477 will instead explore a multi-stage Jenkins pipeline and we'll ditch Travis entirely.

I don't know if we settled on what the initial pass/fail stage would look like in terms of build steps and environment.  Could you describe that or point me to a Travis stage that I should use as a model?

I think the travis.yml in the repo is a good place for us to start for the first stage. On all systems it runs:
make
make check-format
make check-lint
make check-censored

That seems like a good first stage. The second stage could be the Jenkins pipeline we have now.
A theoretical third stage would be a scalability test on a beefy EC2 box if everything else passes, but we don't have the infrastructure for that yet.

Sorry for the delay in getting back to this.  Does the "shape" depicted here match the goal?
This is just a mocked up example but if it's right in principle, I'll submit the actual change.

Yes, this looks like what we want.


"Show address" doesn't show last character on a Nano S

When using MEW and clicking the link/button to show the address on the device, the last nibble is not displayed.
So, for example, if my address was
0x04b3fAAA7c8127A80eB6D24672CfdAF4AeCABbF8
the device scroll-shows
0x04b3faaa7c8127a80eb6d24672cfdaf4aecabbf

yes, the problem is well known. Will be fixed soon.

Fixed in 1.0.21


android: avoid connect for USB target

The "adb connect" command is required only for USB connected devices.
This patch is a small update which avoid the connect for non IP accessed
devices.
Signed-off-by: Patrick Bellasi<EMAIL_ADDRESS>

Your commit message/PR description is incorrect. The first line should read "The "adb connect" command is required only for non-USB connected devices."


[utils] Specify "main" as the default branch in hrp2_drc_description

Otherwise the script fails at git clone phase

@mehdi-benallegue The issue came from https://github.com/jrl-umi3218/mc_rtc/pull/166. Sorry for bothering this.


Rename ThrowHelper classes so they don't clash when merged into on assembly

Looks awful but get it in so we can get unbroken.

We can revert this change after 2.0.


Typo issue

There is a typo issue in this article: https://github.com/WillowInc/opendigitaltwins-building/blob/main/Samples/02-Assets.md.
The Lease in the first pictrue.It would be Land.

https://github.com/WillowInc/opendigitaltwins-building/blob/main/Samples/Images/AssetLocation-Example1.png


EOF Error

Describe the bug
When downloading a series, I receive the following error at a random point in time. (Usually halfway through downloading episode three or four of a series)
I get the following error: An error occurred: error while downloading: unexpected EOF
To Reproduce
Steps / command to reproduce the behavior:
$ crunchy archive -l en-US -l ja-JP -s en-US -d "documents" -o "S{season_number}E{episode_number} {title}.mkv" https://www.crunchyroll.com/series/GEXH3WGGV/otherside-picnice

Expected behavior
Expected to download the complete 12 episode series with English and Japanese Audio as well as English Subtitles.
Screenshots
If applicable, add screenshots to help explain your problem.
Client (please complete the following information):

OS: Windows
Version 2.3.6

Additional context
No issues until a few hours ago. Have tried reinstalling the program and restarting computer. Happens with any series/film.

This might be a temporary Crunchyroll issue. Does it still appear?
If so, can you retry with the latest dev release (v3.0.0-dev.1) and see if it still occurs with this version?

It seems to be working fine now (still version 2.3.6). Sorry for being so quick to post an issue and thank you!


add the search bar to the main page

[SP]=2
it needs to be an input box following by a button SEARCH
[________] {button}
when the person click the SEARCH button and there is nothing inside the input box, should display a pop-up saying {you search nothing}-> up to the designer to change this sentence.
if there is something inside send him to the 3pages -> list.php

Low risk and low value since it's only css for now.


feat(Embed): return the new embed at Embed.clear_fields()

Summary

close #624
Checklist

[ ] If code changes were made then they have been tested.

[ ] I have updated the documentation to reflect the changes.

[x] This PR fixes an issue.
[ ] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

Doc can be updated to show return type.

what happened to the return type

All of the existing methods in this lib that follow the builder design pattern seem to just have a comment saying:
"This function returns the class instance to allow for fluent-style chaining."
and they are not documented with "Returns".

oh i see i see


Reading incorrect amount of columns in csv file.

I'm using the asdataset functionality to read a csv file and populate a table and for 8/10 csv files it works perfectly. However I have a couple csv files where it reads that there is only 2 columns when there is 6 just like the over csv files.
`        private bool LoadCSV(string FilePath)
{
FileStream stream;
bool success;
try
{
            stream = File.Open(FilePath, FileMode.Open, FileAccess.Read);
            IExcelDataReader reader = ExcelReaderFactory.CreateCsvReader(stream);

            var conf = new ExcelDataSetConfiguration
            {
                ConfigureDataTable = _ => new ExcelDataTableConfiguration
                {
                    UseHeaderRow = false,
                    FilterColumn = (rowReader, columnIndex) => { return true; }
                }
            };

            var dataset = reader.AsDataSet(conf);
            var dataTable = dataset.Tables[0];

            int f = 1;
            for (int i = 5; i < dataTable.Rows.Count; i++)
            {
                if (i % 2 == 0)
                {
                    ZoneTable.RowGroups[0].Rows.Add(new TableRow());
                    for (int j = 0; j < ZoneTable.Columns.Count; j++)
                    {
                        TableRow currentRow = ZoneTable.RowGroups[0].Rows[f];
                        TableCell currentCell = (new TableCell(new Paragraph(new Run(dataTable.Rows[i][j].ToString()))));
                        int g = dataTable.Columns.Count;
                        currentCell.BorderThickness = new Thickness(2, 2, 2, 0);
                        currentRow.Cells.Add(currentCell);
                    }
                    f++;
                }
            }
            reader.Close();

            return success = true;
        }
        catch (Exception ex)
        {
            MessageBox.Show(ex.ToString());
            return success = false;
        }
    }`

What version of ExcelDataReader and do you have an example we can use to reproduce it?

I'm using 3.7.0. I can't give you a copy of the project but I can assist with giving code snippets of how its used like what I pasted above. I've attached 2 csv files. Example one gives me the error and example 2 works.
HX1 Vacuum Zone 10.csv
HX1 Vacuum Zone 1.csv

Did some messing around and found that resaving the first csv file allowed it t open normally.

Not sure what is going on then, I can't reproduce the issue with Zone example 1.csv. It gives me 13 columns and 13 rows.

@CanduD That code got some potentially nasty side effects that makes bold assumptions about what's in the  ZoneTable variable. F.ex it will insert columns on the wrong row unless ZoneTable has exactly one row before you call this function, and it will only load as many columns as there are in the ZoneTable. I don't think your problems are with ExcelDataReader.

@andersnm when I read the csv file into a dataset and then set a datatable equal to that dataset. The number of columns in the datatable is usually correct except for a couple files.
@appel1 try this csv I just attached it should cause this issue.
HX1 Vacuum Zone 10.csv

It's the header that causes the problem. If you set ExcelReaderConfiguration.AutodetectSeparators to [',', ';', '\t', '|'] instead, that is without the #, it might work better.
Or you could remove those header rows first.
Or if you know that it is always comma separated you can set AutodetectSeparators  to just [ ',' ].

@appel1 Thank you so much that fixed it! Cheers


Typeahead with show-hint enabled focuses wrong input field on label click

Bug description:
When using the typeahead component with the flag typeahead-show-hint set to true and a label, clicking on the label will focus the wrong input field (the one that is used to show the hint text). This is probably because the ID is passed to the wrong input field.
Link to minimally-working plunker that reproduces the issue:
http://plnkr.co/edit/YPDrR76uEnXoyYO5F9pP?p=preview
Version of Angular, UIBS, and Bootstrap
Angular: v1.4.3
UIBS: v1.3.3
Bootstrap: v2.3.1

@grobgl, this is a good catch.  the issue is that (and you can see this if you look at the generated HTML) there are two elements with the same id - which is a recipe for badness in a browser.
The culprit is this line where we clone the element but never change the ID, if present.
I'm going to play around with this some more, but I believe the fix simply to remove the ID from the original element until after it's cloned - which would be a breaking change should anyone be relying on it.
@wesleycho, if this sounds like the right fix, I'll make a PR.

I'm fine with that fix


use AssetSymbolRenderer to show AMM stats

Issue: https://taiga.sovryn.app/project/sovryn-light/task/79

Closing as deprecated


Add complex128 support

Added complex128 support, based on the guidence from @girving and others in PR #2244. All (python) tests passed.
Complex128 support is not added for these including ops with input: complex64 and output: complex64:

Real
Imag
Conj
*FFT*

Can one of the admins verify this patch?


IndexError: list index out of range (Hue Play Gradient)

Just recently the integration stopped working for some unknown reason.
I just recently (FINALLY) installed the new Hue Play Gradient lightstrip. I'm assuming the issue is related to this.
Error log
2021-05-14 01:51:09 ERROR (MainThread) [homeassistant.components.media_player] Error adding entities for domain media_player with platform huesyncbox
Traceback (most recent call last):
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/helpers/entity_platform.py", line 364, in async_add_entities
    await asyncio.gather(*tasks)
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/helpers/entity_platform.py", line 558, in _async_add_entity
    await entity.add_to_platform_finish()
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/helpers/entity.py", line 581, in add_to_platform_finish
    self.async_write_ha_state()
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/helpers/entity.py", line 334, in async_write_ha_state
    self._async_write_ha_state()
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/helpers/entity.py", line 370, in _async_write_ha_state
    attr.update(self.state_attributes or {})
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/components/media_player/__init__.py", line 868, in state_attributes
    value = getattr(self, attr)
  File "/Volumes/HomeAssistant/custom_components/huesyncbox/media_player.py", line 338, in media_title
    return f"{self.get_mode().capitalize()} - {self.sound_mode.capitalize()}"
  File "/Volumes/HomeAssistant/custom_components/huesyncbox/media_player.py", line 315, in sound_mode
    attributes = self.device_state_attributes
  File "/Volumes/HomeAssistant/custom_components/huesyncbox/media_player.py", line 210, in device_state_attributes
    'entertainment_area': self._get_selected_entertainment_area()
  File "/Volumes/HomeAssistant/custom_components/huesyncbox/media_player.py", line 193, in _get_selected_entertainment_area
    id = parts[1]
IndexError: list index out of range
2021-05-14 01:51:09 ERROR (MainThread) [homeassistant.components.media_player] Error while setting up huesyncbox platform for media_player
Traceback (most recent call last):
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/helpers/entity_platform.py", line 239, in _async_setup_platform
    await asyncio.gather(*pending)
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/helpers/entity_platform.py", line 364, in async_add_entities
    await asyncio.gather(*tasks)
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/helpers/entity_platform.py", line 558, in _async_add_entity
    await entity.add_to_platform_finish()
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/helpers/entity.py", line 581, in add_to_platform_finish
    self.async_write_ha_state()
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/helpers/entity.py", line 334, in async_write_ha_state
    self._async_write_ha_state()
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/helpers/entity.py", line 370, in _async_write_ha_state
    attr.update(self.state_attributes or {})
  File "/Volumes/HomeAssistant/.venv/lib/python3.9/site-packages/homeassistant/components/media_player/__init__.py", line 868, in state_attributes
    value = getattr(self, attr)
  File "/Volumes/HomeAssistant/custom_components/huesyncbox/media_player.py", line 338, in media_title
    return f"{self.get_mode().capitalize()} - {self.sound_mode.capitalize()}"
  File "/Volumes/HomeAssistant/custom_components/huesyncbox/media_player.py", line 315, in sound_mode
    attributes = self.device_state_attributes
  File "/Volumes/HomeAssistant/custom_components/huesyncbox/media_player.py", line 210, in device_state_attributes
    'entertainment_area': self._get_selected_entertainment_area()
  File "/Volumes/HomeAssistant/custom_components/huesyncbox/media_player.py", line 193, in _get_selected_entertainment_area
    id = parts[1]
IndexError: list index out of range

I changed line 196 to:
id = parts[0]
and it seems to be working fine now.
hue_target = self._huesyncbox.api.execution.hue_target
was returning a list with on item in the format of: ['47XX285f-XXXX-492e-XXXX-51bXXXXa590c']
So string split was not affecting anything and a one item list was being returned.

There seem to be some API updates.
I had already been working on some improvements and fixes for this integration and the first one was already solved when I simplified some code. I just also fixed the second issue you mentioned.
I tagged version 1.11.0 which should be available soon. Could you check if that solves the issues?

I can confirm no errors and everything works beautifully after updating to 1.11.0.
Thank you!


Check target existance before to remove original files

dotbot version: 28959a3f31d8632975de4e18d0a0e47476b5a413 dotbot (v1.9.3)
When the force option is enable the original files are removed independently from the existence of the relative targets.
Reproducible steps:

create a file in your home
touch ~/.will_be_deleted
create the configuration files that contains the non existent targetcat <<'EOF' > test2.yaml
- link:
    ~/.will_be_deleted:
       force: true
       path: nonexistent
EOF

run dotbot
./install -c test2.yaml

You will get this output:
Removing ~/.will_be_deleted
Nonexistent target for ~/.will_be_deleted : /home/ael/.dotfiles/nonexistent
Some links were not successfully set up

==> Some tasks were not executed successfully

The original file ~/.will_be_deleted has been deleted and then, only after that, it found out that the targets specified for that file was not existent.
My suggestion is to make some sanity check on the target before to delete the correspondent original file
Note: It will be the same if ~/.will_be_deleted it would be a folder

Fixed in b482cbda585848c0d5f5345711fe338e319c0adf. Thanks for reporting the issue!

I think that we still have some problem. I mean the problem is fixed but now we have really confusing logs:
If I try to reproduce the bug with the new 1.9.4 version that includes https://github.com/anishathalye/dotbot/commit/b482cbda585848c0d5f5345711fe338e319c0adf you get the following log:
~/.will_be_deleted already exists but is a regular file or directory
Some links were not successfully set up

==> Some tasks were not executed successfully

The logs should inform the user that is trying to link to an non-existent target. But instead we've got an already exists error, even if we are setting  the force variable to true.

with PR #107 the logs will be:
Skipping nonexistent target "~/.will_be_deleted" -> "nonexistent"
Some links were not successfully set up

==> Some tasks were not executed successfully

Good catch!


add sip5 binding install directory check

resolves #80
affected downstream issues:
https://github.com/ros-noetic-arch/ros-noetic-python-qt-binding/issues/2
https://github.com/ros-melodic-arch/ros-melodic-rviz/issues/9
Basically adds the sip5 default binding install directory to the check, since this will be the new place of bindings and some distros have already switched causing errors on those systems without adding this check.
Added comments in code for clarity.

@dirk-thomas can you take a quick look at this patch?

@dirk-thomas @claireyywang @sloretz it has been more than 2 week since this is open. Can you provide closure to this PR?
It'll be quick ;)

I am not maintaining this repository anymore. That is why I assigned the new maintainers to review this PR.

@claireyywang @sloretz it has been around a month since this is open. Can you provide closure to this PR?

@claireyywang @sloretz friendly ping

@claireyywang @sloretz friendly ping

@acxz Sorry about the delay, I'll take a look soon!

@claireyywang @sloretz it has been around 2 months since this PR has been opened, sorry to keep bugging you guys about it, but it really is a quick fix. It would be much appreciated if you guys can provide closure to this PR.
Thank you.

@claireyywang @sloretz would be much appreciated if you can take a glance at this.

@claireyywang @sloretz another ping.

@claireyywang @sloretz another ping.

@claireyywang @sloretz it has been a couple months now if you guys could take just a quick look at this, it would be much much appreciated!

@claireyywang @sloretz another ping.
:(

thank you for considering my PR!

@sloretz Would it be possible to get a ROS 1 release which includes this change? We can use it from the hash in the short term, but we need it for the environment we're building in.

@mikepurvis released in https://github.com/ros/rosdistro/pull/30232 and https://github.com/ros/rosdistro/pull/30233

Awesome, thanks!


Never receiving reinvite event after hold function

I'm migrating from 0.7.X to 0.9.X and i'm having problems with detecting response to hold/unhold functions. Until now there were specific events for achieving these, however now a new event reinvite is suposed to be emited after successful response.
After reviewing the code, the response to C_INVITE request is handled by receiveReinviteResponse function which never emitts reinvite event after successful but ack event.
In older code the response to C_INVITE was directly handled by receiveRequest which it calls to receiveReinvite function where you can find reinvite event
On the other hand, we have to control the local hold state to detect if the response of the action is hold or unhold, haven't we?
Thank you

So you want to know when a reinvite is successful or not? This seems pretty reasonable, I will try and get it into a future release.

Thank you. I would like to update visual interface depending on the response to invite request.


Panic after wifi initialization

Hello,
I added wifi initialization to my code based on IDF 6873c19 and got this panic:
I (2650) wifi: wifi firmware version: 8a1ad86
I (2650) wifi: config NVS flash: enabled
I (2650) wifi: config nano formating: disabled
I (2670) wifi: Init dynamic tx buffer num: 32
I (2670) wifi: Init dynamic rx buffer num: 64
I (2670) wifi: wifi driver task: 3ffd5490, prio:23, stack:4096
I (2670) wifi: Init static rx buffer num: 10
I (2680) wifi: Init dynamic rx buffer num: 0
I (2680) wifi: Init rx ampdu len mblock:7
I (2680) wifi: Init lldesc rx ampdu entry mblock:4
I (2690) wifi: wifi power manager task: 0x3ffda85c prio: 21 stack: 2560
I (2690) wifi: wifi timer task: 3ffdb8e0, prio:22, stack:3584
I (2740) wifi: mode : sta (18:fe:34:6a:b0:e2)
/home/andrey/test/espressif_3rdparty/esp32/esp-idf/components/freertos/./queue.c:715 (xQueueGenericSend)- assert failed!
abort() was called at PC 0x4008450f on core 0
0x4008450f: xQueueGenericSend at /home/andrey/test/espressif_3rdparty/esp32/esp-idf/components/freertos/./queue.c:2034
Guru Meditation Error: Core  0 panic'ed (abort)
Backtrace: 0x40008155:0x3ffd52a0 0x40007d16:0x3ffd52c0 0x400f8c55:0x3ffd5300 0x400f9c0a:0x3ffd5320 0x400fe55d:0x3ffd5340 0x400ffcac:0x3ffd5390 0x4010028e:0x3ffd53c0 0x4010f732:0x3ffd53e0
0x400f8c55: esp_event_send at /home/andrey/test/espressif_3rdparty/esp32/esp-idf/components/esp32/./event_loop.c:76
0x400f9c0a: wifi_send_event at ??:?
0x400fe55d: wifi_restart_process at ??:?
0x400ffcac: wifi_start_process at ??:?
0x4010028e: ieee80211_ioctl_process at ??:?
0x4010f732: ppTask at ??:?
Any ideas what can cause that?
I'm attaching sdkconfig i used just in case it contains anything useful for the analysis.
sdkconfig.txt

Answering my own question. Not registering system event handler through esp_event_loop_init causes this. Is it required? If so can you add a more specific diagnostic message?

You can close it, you answered it yourself.
As you can see events will be posted by wifi to a non existing Queue
From esp-idf/components/esp32/event_loop.c
esp_err_t` esp_event_loop_init(system_event_cb_t cb, void *ctx)
{
    if (s_event_init_flag) {
        return ESP_FAIL;
    }
    s_event_handler_cb = cb;
    s_event_ctx = ctx;
    s_event_queue = xQueueCreate(CONFIG_SYSTEM_EVENT_QUEUE_SIZE, sizeof(system_event_t));

    xTaskCreatePinnedToCore(esp_event_loop_task, "eventTask",
            ESP_TASKD_EVENT_STACK, NULL, ESP_TASKD_EVENT_PRIO, NULL, 0);

    s_event_init_flag = true;
    return ESP_OK;
}

Yes, i could close, but the point was doing an extra check in wifi implementation internals to let dev know if he/she didn't do necessary initializations (i.e. didn't assign the event callback). At the same time i may not need the sys event callback at all and may expect wifi initialization to work even without informing me through callback.

When you call esp_wifi_init with WIFI_INIT_CONFIG_DEFAULT, you are passing a pointer to the default event processing function, esp_event_send, to the WiFi driver. For this reason, event loop must be initialized. You may very well not use a callback, in this case you can pass two NULL arguments to esp_event_loop_init function. Good point about adding an explicit check to esp_event_send and failing via abort inside esp_event_send instead of xQueueGenericSend.

Hi @Sermus , is this problem solved? Can we close this issue?

I dunno. The Ivan's explanation is clear and this is not an issue for me anymore. But as Ivan said, it would be good to add an explicit check and a bit more specific debug message for the situation i faced.
If your internal dev process allows doing this after closing the github ticket, then i'm fine with this.

I have a PR which includes an explicit check for this in pipeline. Let's keep this open for now.


Fix regression in 0.3.0

This fixes #137.

Shouldn't we rather adapt the code instead of reverting to an outdated library?

BTW: readdirp seems to be outdated anyway :rofl:

What do you think about using glob?

this patch does not work for my PR, ie, the format option still does not work.
$ sloc --format=csv .

---------- Result ------------

            Physical :  4
              Source :  4
             Comment :  0
 Single-line comment :  0
       Block comment :  0
               Mixed :  0
 Empty block comment :  0
               Empty :  0
               To Do :  0

Number of files read :  1

----------------------------

... this should now be obsolete, since #143 is merged.
Thanks anyway!


add Yandex OAuth provider #34

I added the Yandex OAuth provider, altered the UI and documentation, added unit-tests and check the Yandex login flow manually.
The list of permissions has Access to user avatar and Access to username, first name and surname, gender. The latter is needed to get a friendly username. In fact a yandex user can have 3 different names:

login – same as first part of email address, this one is always required,
real_name – the combination of First and Last names, optional,
display_name – the name is used for displaying user name in yandex services, this name is visible to other yandex users.
So more preferable name for us is display_name. If it's empty we use real name, otherwise login.
The doc is here: https://tech.yandex.com/passport/doc/dg/reference/response-docpage/#norights_1

Thx for the PR. I'm not sure yet if we really need Yandex as another provider, but I'll think about it. Github related change will be nice to have as a separate PR and I could merge it right away.

@umputun, got it. I will create another PR with github refactoring.

Regarding Yandex scope - is it possible to minimize it? We need just 3 things:

some ID
display name
avatar

Things like first name and surname, gender, real name I'm trying to avoid, but not sure if it even possible with Yandex.

@igoradamenko can you take a look at js related part of this PR?

@anatolym could you resubmit with the recent master? We have a conflict on Gopkg.lock

Regarding Yandex scope - is it possible to minimize it? We need just 3 things:

@umputun, unfortunately nope. If we don't request the scope "Access to username, first name and surname, gender", we can use user login only. Display name comes together with gender and other stuff of the scope.
I'm not sure that's correct behavior since yandex users can hide login name on public services and use his Display name.

could you resubmit with the recent master? We have a conflict on Gopkg.lock

Sure, gimme a minute.

thx, merged. Btw, if you could take care on twitter auth this will be really great.

@umputun sure, I will take a look at twitter api.


What about stripping and passing in styles?

this is really nice and seems the best one out there.. but need basic styling like stripping and table lines.. is there a way to set those?

Hello @rchancey! Thank you for the kind words. The styles are totally within your control via the generic css import. If you are meaning something else, please elaborate or submit a proof-of-concept. I'm happy to consider new approaches.

Closing for now. Feel free to reopen with more information.


Support headless Chrome

Now that Chrome 59 has been released with support for headless mode, it'd be great if chromedriver-helper used the headless mode (by default). Or at the least, documented how to enable headless mode. I realize that this may require waiting on an updated version of chromedriver itself, but I wanted to make sure we don't forget about this nice new feature.

I found this article from Thoughtbot and tried following the instructions there. I verified that I'm using chromedriver 2.30 and Chrome 59, but when I go to chrome://version/, I don't see --headless in the list.
Also worth mentioning is the Getting Started with Headless Chrome article from Google and the ChromeDriver Capabilities docs.

OK, I got it working. Oddly, I had to change chromeOptions: in the Thoughtbot example code to 'chromeOptions' => to get it to work.
I still think it's worth mentioning how to get this working in the docs.

I love Ruby more than most (it is by far my favorite programming language), but I gotta say the "symbol vs. string" hash key issue is one of the biggest warts on the language. 😭

If you're running on OSX - make sure to avoid the remote-debugging-port=9222 option, which causes chrome to hang, resulting in a Net::ReadTimeout. (chrome 59.0.30, chromedriver 2.3)

@booch
Can you share your driver config, please?
I followed https://robots.thoughtbot.com/headless-feature-specs-with-chrome instructions along with converting chromeOptions key to string as you suggested and nothing happened? Capybara stucks without any log information.
PS My version is:
Chromium 59.0.3071.109 Built on Ubuntu , running on Ubuntu 16.04

Hey guys. I'm in the same boat as the others here. I've got Chromedriver working with Capybara, but I cannot, for the life of me, get the options working. Anytime I try to introduce the desired_capabilities key, everything falls apart. I've tried changing symbols to keys, different options, chanting, yelling... nothing has worked yet.
Capybara.register_driver :selenium do |app|
  Capybara::Selenium::Driver.new(
    app,
    browser: :chrome,
    desired_capabilities: Selenium::WebDriver::Remote::Capabilities.chrome(
      chromeOptions: {
        args: %w(headless disable-gpu)
      }
    )
  )
end

I've got Chromedriver 2.3 and selenium-webdriver in the gemfile.

I was not able to get this to work with chromedriver version 2.28.  When I updated to 2.31, it worked.  Note that the path for the chromedriver bin is set by chromedriver-helper and is in your gem path.  I'd forgotten about this at first and so was inadvertently checking the version of chromedriver I had installed elsewhere.
Here is the code that I'm using in my cucumber features/support/env.rb file that is working:

# Headless chrome browser.
# Use @selenium
Capybara.register_driver :selenium do |app|

  options = Selenium::WebDriver::Chrome::Options.new(args: ['headless', 'disable-gpu'])

  Capybara::Selenium::Driver.new(
      app,
      browser: :chrome,
      options: options
  )
end

chromedriver-helper v. 1.1.0
selenium-webdriver 3.4.4

@weedySeaDragon Your config works for me! I've got headless chrome running. Now I have all kinds of problems related to changing from Poltergeist to Chrome, but progress is being made.
🍺 🌮 🎉  Thanks very much.

It appears that when running headless, JavaScript is disabled. Has anyone else had this problem?

Thanks for opening this issue, and apologies for not responding earlier.
Honestly, I'm not sure it's within the ability of chromedriver-helper to opt into headless mode. All this gem does is download the correct executable for your platform and run it -- configuration needs to be done within your testing framework (e.g., Capybara).
I'm going to close this; though if I'm wrong and there's a PR that can be submitted to do this, I'm open to it.

@flavorjones No problem at all, this is absolutely unrelated to your work. I'm going to go bark up the right tree now. Thanks very much for the gem!

Thanks @weedySeaDragon, your config works. Did try ~ a dozen different config variants, to no avail.
@flavorjones Maybe how to pass options to the Driver (for example to run it headless) could be added to the Readme. I could open a PR, if desired.

I had to downgrade selenium-webdriver to v 3.4.4. My system test config looks like this. Note how I am forcing a version downloaded by the helper too. I found there is no class level property to set for future binstub run to take advantage of. This seemed like a good way to be consistent.
require 'test_helper'

driver = Chromedriver::Helper.new
driver.update('2.30')

Selenium::WebDriver::Chrome.driver_path = driver.binary_path

class ApplicationSystemTestCase < ActionDispatch::SystemTestCase
  Capybara.register_driver :headless_chrome do |app|
    Capybara::Selenium::Driver.new(
      app,
      browser: :chrome,
      desired_capabilities: Selenium::WebDriver::Remote::Capabilities.chrome(
        'chromeOptions' => {
          'args' => ['headless', 'disable-gpu', 'no-sandbox', 'window-size=1280,1024'],
          'binary' => ENV['GOOGLE_CHROME_SHIM']
        }.reject { |_k, v| v.nil? }
      )
    )
  end

  driven_by :headless_chrome
end

Ah, I am using latest selenium-webdriver gem now and driver.update('2.36') version now.


[FEATURE] Add tested or recommend kernel version of each tutorial

Is your feature request related to a problem? Please describe.
Some tutorial may require a higher version of kernel.
Describe the solution you'd like
A clear and concise description of what you want to happen.
We should list the minimal version or tested of each tutorial.

I would like to run tutorials and test them on Ubuntu 22.04 with kernel version of 6.5.0-18-generic.


Enable mocking for extension methods in mgmt SDKs

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Resources


Add an initial (MVP) version of quickstart application

This PR adds an initial version for the Spine server application quickstart.

@alexander-yevsyukov PTAL.

@alexander-yevsyukov All the changes discussed vocally are now applied. PTAL.

@alexander-yevsyukov @vladimir-moiseiev PTAL.

@alexander-yevsyukov now the codebase and build settings are configured for Java 7. PTAL.


[RFC] PHPLIB-225: DuplicateKeyException wraps BulkWriteException

https://jira.mongodb.org/browse/PHPLIB-225
Not sure if we want to merge it, but I wanted to open it for discussion purposes to demonstrate the complexity of actually implementing this in PHPLIB.

@jmikola Any plans to revisit this one?

@lindelius: This isn't currently a priority (vs. other development tasks), but I'm also open to coming back to it at a later date, which is why I've not closed the issue. This may be something for @alcaeus to follow up on after he joins the team in a few months.

Closing this: as explained in the linked ticket, there is no sensible way to wrap or replace the BulkWriteException with a DuplicateKeyException, since it would only work in a small subset of cases.

@lindelius: If you still need this functionality, I think the best solution may be to implement a utility function (e.g. has_duplicate_key_error()) that takes a MongoDB\Driver\BulkWriteException and returns true if its code is 11000 or if any of its constituent write errors have the code 11000. The latter check would be done by calling getWriteResult(), iterating on getWriteErrors(), and then checking the code for each.
While researching this, I believe @alcaeus determined that our exception is assigned a code from the first write or write concern error (behavior from libmongoc). Therefore, simply checking the exception's top-level code is not sufficient if there was a duplicate key error somewhere within a batch of multiple write operations.


[AUTOPATCHER-CORE] Upgrade pkgconf to 1.9.4 to fix CVE-2023-24056 -

[AUTOPATCHER-CORE] Upgrade pkgconf to 1.9.4 to fix CVE-2023-24056
Upgrade pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=302022&view=results
AMD64 build -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=302026&view=results
ARM64 build -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=302027&view=results

Closing in favor of https://github.com/microsoft/CBL-Mariner/pull/4732 which does not require a version bump.


revised resource templates do not show up in other sessions when uploaded

Upload a revised version of a resource template (i.e., a resource template with an ID Sinopia already has in its available list, but which has changed content, such as a revised Template Name or anything else different).
When you go to an existing session in another browser, you don't see the changes. You don't see the revised Template Name (if that was revised) and when you click on the Template Name to open it in the Linked Data Editor, you are seeing the old version of the template.
per @mjgiarlo "A container's entity tag is not updated when one of its children it updated. And this isn't a Trellis bug, either. I don't think an LDP container has changed when one of its children is updated."

Consider using web-based sockets for this, integrating the editor directly with the pipeline. E.g., https://www.valentinog.com/blog/socket-react/

@michelleif A user can handle this by a hard refresh; solving the problem would take a significant effort. OK to close?

ok to close, this is already documented in known issues


Fix module import

When try run pymux cannot import ctrl module so this patch fix this.
Traceback (most recent call last):
  File "/usr/bin/pymux", line 9, in <module>
    load_entry_point('pymux==0.5', 'console_scripts', 'pymux')()
  File "/usr/lib64/python2.7/site-packages/pkg_resources/__init__.py", line 558, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib64/python2.7/site-packages/pkg_resources/__init__.py", line 2682, in load_entry_point
    return ep.load()
  File "/usr/lib64/python2.7/site-packages/pkg_resources/__init__.py", line 2355, in load
    return self.resolve()
  File "/usr/lib64/python2.7/site-packages/pkg_resources/__init__.py", line 2361, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/usr/lib64/python2.7/site-packages/pymux/entry_points/run_pymux.py", line 28, in <module>
    from pymux.main import Pymux
  File "/usr/lib64/python2.7/site-packages/pymux/main.py", line 18, in <module>
    from .arrangement import Arrangement, Pane, Window
  File "/usr/lib64/python2.7/site-packages/pymux/arrangement.py", line 11, in <module>
    from .process import Process
  File "/usr/lib64/python2.7/site-packages/pymux/process.py", line 13, in <module>
    from .stream import BetterStream
  File "/usr/lib64/python2.7/site-packages/pymux/stream.py", line 7, in <module>
    from pyte import ctrl
ImportError: cannot import name ctrl

Thank you! I just pushed a new release for this.


Terratest AWS module for RDS has issue with GetParameterValueForParameterOfRdsInstance function

Bug report:  github.com/gruntwork-io/terratest/modules/aws/rds.go has issue with GetParameterValueForParameterOfRdsInstance function and it's dependent function GetParameterValueForParameterOfRdsInstanceE. The process loads all the parameters from the parameter group for the specified RDS instance and tries to locate the specified parameter in the set, but in fact, it is not loading all parameters.  The dependent AWS API call DescribeDBParameters, by default, only returns the first 100 parameters (in alphabetical order) and if your parameter group has more than 100 parameters and the parameter you are looking for is not in the first 100, the function throw errors,  incorrectly reporting the parameter does not exist.
Recommend the functions support pagination using the Marker parameter to scan all the parameters, 100 at a time.

Thanks for reporting! Would you be up for a PR to fix this?

I have modified the related modules to make this work but it's a bit messy.  Being very new to Go I had difficulties with combining the results of the DescribeDBParameters call in the GetAllParametersOfRdsInstanceE function.   I wanted to have this function loop through the pages of the available parameters.  So to get it working I setup the GetParameterValueForParameterOfRdsInstanceE function to also return the Marker token to the calling process and then setup a loop in the GetParameterValueForParameterOfRdsInstanceE function to use the token to reference the next set. I can submit  PR with this fix.....

PR welcome, thank you! Note that we probably have other code in this repo, and code in many other Go repos, that use the Marker token pattern to fetch all results if you need a reference.


Does openssl 1.1.1 need callback?

I want to invoke a https request with libcurl + openssl.
I have read some documents, such as https://curl.haxx.se/libcurl/c/threadsafe.html.
In openssl <= 1.0.2, we need 2 callbacks:
CRYPTO_set_id_callback((unsigned long (*)())thread_id);
CRYPTO_set_locking_callback((void (*)(int, int, const char *, int))lock_callback);

In openssl >= 1.1.0, we need another callback:
CRYPTO_THREADID_set_callback((unsigned long (*)())thread_id);

But I look the macro definition:

It seems they do nothing. So I want to ask whether openssl 1.1.1 still need callbacks function to set or not in multithread environment.
Some document is welcome. Thank you.🙂

And I check the macro is set to non:

The locking callbacks have been removed in 2e52e7df518d80188c865ea3f7bb3526d14b0c08, see also the note in CHANGES, L934:
  *) OpenSSL now uses a new threading API. It is no longer necessary to
     set locking callbacks to use OpenSSL in a multi-threaded environment. There
     are two supported threading models: pthreads and windows threads. It is
     also possible to configure OpenSSL at compile time for "no-threads". The
     old threading API should no longer be used. The functions have been
     replaced with "no-op" compatibility macros.
     [Alessandro Ghedini, Matt Caswell]

In openssl >= 1.1.0, we need another callback:

This is not correct. No threading callbacks are required in 1.1.0

@mspncp Thank you.
The pthreads threading mudel is #include<pthread.h> in Linux?

@mattcaswell Sorry, maybe I am wrong. I get this information from several blogs.


Ability to use local SQLite database, automatic database structure setup, Discord bot disabling

Added:

BotSettings 'DiscordBotEnabled' & 'DatabaseType' (sqlite or mssql)
DatabaseWrapper class that handles all requests to save/retrieve data depending on DatabaseType, with flexibility to add new storage types.
Sql Schema files for both mssql and sqlite which automatically set up necessary tables in database.

General flow:

User sets up database type in settings
On initialisation, depending on database type, new SQLite database is created and/or local .SQL script run to create all necessary tables
On data request, repository passes arguments into corresponding database wrapper method.
Database wrapper methods contain logic that retrieves data based on database type and passes it back through in expected format.

Ahh, sorry I'm new to using these tools.  I was only meant to include the
database stuff in the request, I didn't realise it'd include all of my
recent messing about as well.
I'll go back and clean it up, ignore for now I guess
On Thu, 15 Jun 2023, 04:14 Drew Kestell, @.***> wrote:

@.**** commented on this pull request.
In .gitignore
https://github.com/DrewKestell/BloogBot/pull/10#discussion_r1230369644:

\ No newline at end of file
+Bot
+/BloogBot/UI/App.xaml.cs

Why are we excluding this from the git repo? I think this needs to be
included.
—
Reply to this email directly, view it on GitHub
https://github.com/DrewKestell/BloogBot/pull/10#pullrequestreview-1480593459,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHZLL4LGVCCNSQRY77Z4G3XLJ4ZLANCNFSM6AAAAAAZB3PEGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Hardware reset

Is there a way to reset the HooToo device if I can't reach it due a bad configuration?
Is the reset button still working with OpenWrt?

The answer is yes to both questions but it's not specific to the configuration work I did. Best place to find the answer(s) will be on the OpenWRT website and forum.


-[NSURLSessionTask(_AFStateObserving) af_resume] + 48

谁能告诉我这个怎么解决？（who can tell me how to solve this?）

What AFNetWorking version are you using?

What AFNetWorking version are you using?
3.2.1

You can use 3.1.0


Elastic 6.4.3 is available

@vvanholl Please, release a compatible version. Thanks!

Thank you, @vvanholl!


Use gh's --jq option instead of the jq executable

This makes the script portable to environments where jq isn't installed.
See the documentation for gh api.

@spenserblack Can I work on this one?


update: cosign v1.10.0 to v1.11.0

Fixes #
Description

Checklist

[x] PR is rebased to/aimed at branch develop
[x] PR follows Contributing Guide
[x] Added tests (if necessary)
[x] Extended README/Documentation (if necessary)
[x] Adjusted versions of image and Helm chart in values.yaml and Chart.yaml (if necessary)

Codecov Report

Merging #757 (fe730af) into develop (ff82e00) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #757   +/-   ##
========================================
  Coverage    96.81%   96.81%
========================================
  Files           22       22
  Lines         1226     1226
========================================
  Hits          1187     1187
  Misses          39       39

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Multiple wire library errors

SF.Throw in a few cases, especially related to wirelinks. These cause library to error when wirelink execution isn't perfect.
Some cases wirelinks refuse to update their value unless you delete, and re-create the wirelink on the select input/output. Perhaps this is me using it incorrectly, in a 'think' hook, rather than the wiremod world of triggers or intervals?

Is this still an issue with #351 merged?

Perhaps, I need to review how wirelinks are behaving. I think attempting to update chips that have wirelinks or some preprocs will error it.


Bind nginx requests directly to docker internal IP

This would help avoid cases where users may not being using a firewall to block random ports and they are using VHOST-based deploys. Apps should only be routed to via the private docker ip address, not a public docker port.
Refs #207.
Refs #557.
@michaelshobbs I think you might be more familiar with how this works, so if you have time, would you mind taking a look at those PRs and modifying it to work in the new setup?

Sure thing. Probably later today or first thing tomorrow.

Sure thing. Probably later today or first thing tomorrow.


wsmancli: 2.6.0 -> 2.6.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/wsmancli/versions.
meta.description for wsmancli is: Openwsman command-line client
meta.homepage for wsmancli is: https://openwsman.github.io
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.6.2 in filename of file in /nix/store/yahqgifh9civjr44sm4dk2z5m7jw7max-wsmancli-2.6.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
wsmancli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yahqgifh9civjr44sm4dk2z5m7jw7max-wsmancli-2.6.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A wsmancli https://github.com/r-ryantm/nixpkgs/archive/212c3f86d003a5f6dbae0964c9d95d6b4c1a588f.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/yahqgifh9civjr44sm4dk2z5m7jw7max-wsmancli-2.6.2
ls -la /nix/store/yahqgifh9civjr44sm4dk2z5m7jw7max-wsmancli-2.6.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

wsmancli

Maintainer pings
cc @deepfire for testing.

@ofborg build wsmancli wsmancli.passthru.tests