Opening multiple VS Code windows causes connection to close

Description:
When two VS Code windows are open the connection get closed in both. And didn't recover until I disabled the extension in one workspace and restarted the other
Auto detecting Ballerina home.
Plugin version: 2.0.0
Ballerina version: Swan Lake Preview 9 SNAPSHOT
Using Ballerina CLI command 'bal' for Language server.
/usr/bin/bal: line 92: 150692 Killed                  $BALLERINA_HOME/bin/./bal "$@"
[Info  - 2:21:26 PM] Connection to server got closed. Server will restart.
/usr/bin/bal: line 92: 151350 Killed                  $BALLERINA_HOME/bin/./bal "$@"
[Info  - 2:21:27 PM] Connection to server got closed. Server will restart.
[Error - 2:21:32 PM] Server initialization failed.
Error: Connection is closed.
	at G (/home/chiran/.vscode/extensions/ballerina.ballerina-2.0.0-Alpha-SNAPSHOT/dist/extension.js:1:18779)
	at Object.onNotification (/home/chiran/.vscode/extensions/ballerina.ballerina-2.0.0-Alpha-SNAPSHOT/dist/extension.js:1:19609)
	at Object.onDiagnostics (/home/chiran/.vscode/extensions/ballerina.ballerina-2.0.0-Alpha-SNAPSHOT/dist/extension.js:1:67739)
	at /home/chiran/.vscode/extensions/ballerina.ballerina-2.0.0-Alpha-SNAPSHOT/dist/extension.js:1:63231
[Error - 2:21:33 PM] Server initialization failed.
Error: Connection got disposed.
	at Object.dispose (/home/chiran/.vscode/extensions/ballerina.ballerina-2.0.0-Alpha-SNAPSHOT/dist/extension.js:1:21204)
	at Object.dispose (/home/chiran/.vscode/extensions/ballerina.ballerina-2.0.0-Alpha-SNAPSHOT/dist/extension.js:1:67809)
	at /home/chiran/.vscode/extensions/ballerina.ballerina-2.0.0-Alpha-SNAPSHOT/dist/extension.js:1:64759

Suggested Labels:

Suggested Assignees:

Affected Product Version:
VS Code 1.52.1
Ubuntu 20.04
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

We were able to replicate the issue when there're multiple debug adapter terminate requests happen in a single VSCode environment.

As per the JSON-RCP implementation of VSCode and LS communication, the VSCode keeps a one-to-one mapping between the LS and the client connection. If LS stops, the client involves in restarting the LS for seamless communication. However, there are some scenarios where the client cannot restore the connection when the LS is consecutively and manually killed by another process.
The previous Ballerina Debug Adapter implementation had been killing the LS connection at each terminateRequest, hence caused the mentioned issue. Due to the auto-recovery nature of the underline VSCode-LS communication, this issue hasn't been noticed unless there're a significant number of debugging client terminate requests. The new debug adapter implementation (comes with SL Alpha2) has fixed this issue of killing LS instances of other VSCode instances.


Updated compute-basics to use LoginCredentials instead of Credentials

Didn't think it was worth creating an issue for this, let me know if I should have. I checked the changes by building the jar and running it a few times with different parameters.

good idea.  thanks!


New package: Lenovo.DiagnosticsTool version <IP_ADDRESS>

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/111894&drop=dogfoodAlpha

Supporting custom hardware in validation is "Not Planned".

https://github.com/microsoft/winget-pkgs/issues/199504


Remote unseal support

Unseal Vault via 3rd party (out-of-cloud) devices with sealed notifications

not relevant


new filters

There are some filters that I'd like to add to lume:

[ ] inline: to inline content in the html (css, javascript, svg, etc). Example:

This will return the content of the css file.

[ ] css: The css plugin could register the css filter, so we can not only inline css but also transform it:

[ ] bundle: We can do the same with the bundler plugin, to transform javascript/typescript.
[ ] svg: We can do the same with the svg plugin, to transform svg.

by 'bundler plugin', do you mean having a plugin to do deno bundle? swc, on which Deno is built, is working on implementing a port for Terser to add to this. I'm using denopack, which uses the browser version of Rollup/Terser, for bundling/minifying. Their roadmap mentions adding 'static generation', so perhaps there's scope for working together?

The idea is that existing processors like css (which uses postcss) or svg (which uses svgo) can be used also as template filters, to transform inline sources.
The bundle processor uses the built-in Deno.bundle() (see here: https://github.com/lumeland/lume/blob/master/plugins/bundler.js).
Any other bundler like denopack can be used (it's really easy to create more plugins for Lume). I'll take a look to Denopack.
Thanks!

ok. Maybe an example in the docs of how to create a plugin? There's a couple of issues with Denopack, but basically it works well. I'll look into creating a Terser plugin, which would be useful for minifying js without bundling.
I'd suggest standardising the docs to use the same terms throughout. For example, the home page talks of "transformers", but I don't think this word is used anywhere else. Does this mean both plugins and filters? https://lumeland.github.io/creating-pages/filters/ says, under md, "It's installed by the markdown plugin", which is correct, but at the top it says "New plugins must be registered" which I think should be 'filters'.

Yes, right. "New plugins must be registered" is a typo, it should say "New filters". I'm fixing it.
I have to complete the docs with info about extending Lume with new plugins, transformers, filters, etc.
I just added docs for loaders (https://lumeland.github.io/advanced/loaders/).

I just added some documentation for processors, engines and plugins.

For minification, we could use deno_swc which allows us to use SWC for creating an AST, and it also has built in minification in the stringification of that AST.
import { parse, print } from<EMAIL_ADDRESS>
const code = await Deno.transpileOnly("const x: string = 'asdf';");

const thing = parse(code, {
    syntax: "ecmascript"
});

console.log(print(thing, {
    minify: true,

}))

For minification, we could use deno_swc which allows us to use SWC for creating an AST, and it also has built in minification in the stringification of that AST.
import { parse, print } from<EMAIL_ADDRESS>
const code = await Deno.transpileOnly("const x: string = 'asdf';");

const thing = parse(code, {
    syntax: "ecmascript"
});

console.log(print(thing, {
    minify: true,

}))

looks like deno_swc uses Terser (via esm.sh and Wasm) for minification. Seems rather a roundabout way of using Terser :-). https://github.com/swc-project/swc/pull/1302 is swc's current work on adding a Terser port. https://github.com/denoland/deno/issues/6900 is Deno's issue for incorporating this in Deno bundle. denopack's using the browser version of Terser looks like the best way forward to me. A Lume plugin could easily do something like the shell script in https://github.com/terser/terser/issues/544#issuecomment-724663556.

looks like deno_swc uses Terser (via esm.sh and Wasm) for minification. Seems rather a roundabout way of using Terser :-). https://github.com/swc-project/swc/pull/1302 is swc's current work on adding a Terser port. https://github.com/denoland/deno/issues/6900 is Deno's issue for incorporating this in Deno bundle. denopack's using the browser version of Terser looks like the best way forward to me. A Lume plugin could easily do something like the shell script in https://github.com/terser/terser/issues/544#issuecomment-724663556.

Have you tried https://github.com/timreichen/Bundler?

Have you tried https://github.com/timreichen/Bundler?

looks interesting - something else for me to investigate. :-) Also uses Terser from esm.sh.

looks interesting - something else for me to investigate. :-) Also uses Terser from esm.sh.

AFAICS, Bundler does no tree-shaking. So I'm sceptical how much use it is for browsers. Might be ok for Deno though, where bundle size is not so important.

AFAICS, Bundler does no tree-shaking. So I'm sceptical how much use it is for browsers. Might be ok for Deno though, where bundle size is not so important.

Instead creating filters for every processor (css, bundle, svg etc), I finally created the inline processor, to insert automatically in the html the code of any resouce with the inline attribute. For example:
<link rel="stylesheet" href="styles.css" inline>

Is converted to:

<svg>
  /* Svg code here */
</svg>

Instead creating filters for every processor (css, bundle, svg etc), I finally created the inline processor, to insert automatically in the html the code of any resouce with the inline attribute. For example:
<link rel="stylesheet" href="styles.css" inline>

Is converted to:

<svg>
  /* Svg code here */
</svg>


trying to login results in "fatal--- uncaught exception, please report this"

Describe the bug
I am trying to enable the verdaccio-activedirectory plugin. Verdaccio crashes when I attempt to login and does not provide the error or call stack.
To Reproduce
Steps to reproduce the behavior:

Start Verdaccio with the activedirectory plugin
Try to login

Expected behavior
I'm sure my active directory configuration is incorrect and causing an error. But I expect an error message not a fatal crash with no details.
Configuration File (cat ~/.config/verdaccio/config.yaml)
storage: ./storage

listen:
    unix:/var/www/verdaccio/verdaccio.sock

url_prefix: https://npm.domain.com

web:
    title: Domain Private NPM
    enable: true

auth:
  htpasswd:
    file: ./htpasswd
    max_users: -1
  activedirectory:
    host: 'dc.domain.local'
    port: 389
    baseDN: 'DC=domain,DC=local'
    bindDN: 'CN=npm,CN=Managed Service Accounts,DC=domain,DC=local'
    bindCredentials: 'XXX'
    domainSuffix: 'domain.com'

# a list of other known repositories we can talk to
uplinks:
  npmjs:
    url: https://registry.npmjs.org/

packages:
  '@domain/*':
    # scoped packages
    access: $authenticated
    publish: $authenticated

  '*':
    access: $all
    publish: $authenicated
    proxy: npmjs

# log settings
logs:
  - {type: stdout, format: pretty, level: trace}
  #- {type: file, path: verdaccio.log, level: info}

Debugging output

$ NODE_DEBUG=request verdaccio display request calls (verdaccio <--> uplinks)
$ DEBUG=express:* verdaccio enable extreme verdaccio debug mode (verdaccio api)

> sudo NODE_DEBUG=request DEBUG=express:* -u verdaccio /usr/bin/node /usr/lib/node_modules/verdaccio/bin/verdaccio -c config.yaml
 warn --- config file  - /var/www/verdaccio/config.yaml
 warn --- Plugin successfully loaded: htpasswd
 info --- Active Directory plugin configuration:
 { host: 'dc.domain.local',
  port: 389,
  baseDN: 'DC=domain,DC=local',
  bindDN: 'CN=npm,CN=Managed Service Accounts,DC=domain,DC=local',
  bindCredentials: 'XXX',
  domainSuffix: 'domain.com' }
 warn --- Plugin successfully loaded: activedirectory
  express:application set "x-powered-by" to true +0ms
  express:application set "etag" to 'weak' +2ms
  express:application set "etag fn" to [Function: generateETag] +0ms
  express:application set "env" to 'development' +0ms
  express:application set "query parser" to 'extended' +0ms
  express:application set "query parser fn" to [Function: parseExtendedQueryString] +1ms
  express:application set "subdomain offset" to 2 +0ms
  express:application set "trust proxy" to false +0ms
  express:application set "trust proxy fn" to [Function: trustNone] +1ms
  express:application booting in development mode +0ms
  express:application set "view" to [Function: View] +0ms
  express:application set "views" to '/var/www/verdaccio/views' +0ms
  express:application set "jsonp callback name" to 'callback' +0ms
  express:application set "env" to 'production' +1ms
  express:router use '/' query +4ms
  express:router:layer new '/' +0ms
  express:router use '/' expressInit +1ms
  express:router:layer new '/' +1ms
  express:router use '/' corsMiddleware +0ms
  express:router:layer new '/' +0ms
  express:router use '/' log +1ms
  express:router:layer new '/' +0ms
  express:router use '/' errorReportingMiddleware +0ms
  express:router:layer new '/' +0ms
  express:router use '/' <anonymous> +0ms
  express:router:layer new '/' +0ms
  express:router use '/' <anonymous> +0ms
  express:router:layer new '/' +0ms
  express:router use '/' compression +1ms
  express:router:layer new '/' +0ms
  express:router:route new '/favicon.ico' +1ms
  express:router:layer new '/favicon.ico' +0ms
  express:router:route get '/favicon.ico' +0ms
  express:router:layer new '/' +1ms
  express:router use '/' <anonymous> +1ms
  express:router:layer new '/' +0ms
  express:router use '/' jsonParser +1ms
  express:router:layer new '/' +0ms
  express:router use '/' <anonymous> +0ms
  express:router:layer new '/' +0ms
  express:router use '/' encodeScopePackage +1ms
  express:router:layer new '/' +0ms
  express:router:route new '/whoami' +0ms
  express:router:layer new '/whoami' +0ms
  express:router:route get '/whoami' +1ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/whoami' +0ms
  express:router:layer new '/-/whoami' +0ms
  express:router:route get '/-/whoami' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/:package/:version?' +1ms
  express:router:layer new '/:package/:version?' +0ms
  express:router:route get '/:package/:version?' +1ms
  express:router:layer new '/' +0ms
  express:router:route get '/:package/:version?' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/:package/-/:filename' +0ms
  express:router:layer new '/:package/-/:filename' +0ms
  express:router:route get '/:package/-/:filename' +0ms
  express:router:layer new '/' +1ms
  express:router:route get '/:package/-/:filename' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/all(/since)?' +0ms
  express:router:layer new '/-/all(/since)?' +0ms
  express:router:route get '/-/all(/since)?' +1ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/user/:org_couchdb_user' +0ms
  express:router:layer new '/-/user/:org_couchdb_user' +0ms
  express:router:route get '/-/user/:org_couchdb_user' +1ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/user/:org_couchdb_user/:_rev?/:revision?' +0ms
  express:router:layer new '/-/user/:org_couchdb_user/:_rev?/:revision?' +0ms
  express:router:route put '/-/user/:org_couchdb_user/:_rev?/:revision?' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/user/token/*' +1ms
  express:router:layer new '/-/user/token/*' +0ms
  express:router:route delete '/-/user/token/*' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/_session' +0ms
  express:router:layer new '/_session' +0ms
  express:router:route post '/_session' +1ms
  express:router:layer new '/' +0ms
  express:router:route post '/_session' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/:package/:tag' +1ms
  express:router:layer new '/:package/:tag' +0ms
  express:router:route put '/:package/:tag' +0ms
  express:router:layer new '/' +0ms
  express:router:route put '/:package/:tag' +1ms
  express:router:layer new '/' +0ms
  express:router:route put '/:package/:tag' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/package/:package/dist-tags/:tag' +0ms
  express:router:layer new '/-/package/:package/dist-tags/:tag' +0ms
  express:router:route post '/-/package/:package/dist-tags/:tag' +0ms
  express:router:layer new '/' +1ms
  express:router:route post '/-/package/:package/dist-tags/:tag' +0ms
  express:router:layer new '/' +0ms
  express:router:route post '/-/package/:package/dist-tags/:tag' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/package/:package/dist-tags/:tag' +0ms
  express:router:layer new '/-/package/:package/dist-tags/:tag' +0ms
  express:router:route put '/-/package/:package/dist-tags/:tag' +1ms
  express:router:layer new '/' +0ms
  express:router:route put '/-/package/:package/dist-tags/:tag' +0ms
  express:router:layer new '/' +0ms
  express:router:route put '/-/package/:package/dist-tags/:tag' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/package/:package/dist-tags/:tag' +0ms
  express:router:layer new '/-/package/:package/dist-tags/:tag' +1ms
  express:router:route delete '/-/package/:package/dist-tags/:tag' +0ms
  express:router:layer new '/' +0ms
  express:router:route delete '/-/package/:package/dist-tags/:tag' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/package/:package/dist-tags' +0ms
  express:router:layer new '/-/package/:package/dist-tags' +0ms
  express:router:route get '/-/package/:package/dist-tags' +1ms
  express:router:layer new '/' +0ms
  express:router:route get '/-/package/:package/dist-tags' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/package/:package/dist-tags' +0ms
  express:router:layer new '/-/package/:package/dist-tags' +0ms
  express:router:route post '/-/package/:package/dist-tags' +0ms
  express:router:layer new '/' +1ms
  express:router:route post '/-/package/:package/dist-tags' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/:package/:_rev?/:revision?' +1ms
  express:router:layer new '/:package/:_rev?/:revision?' +0ms
  express:router:route put '/:package/:_rev?/:revision?' +0ms
  express:router:layer new '/' +0ms
  express:router:route put '/:package/:_rev?/:revision?' +0ms
  express:router:layer new '/' +1ms
  express:router:route put '/:package/:_rev?/:revision?' +0ms
  express:router:layer new '/' +0ms
  express:router:route put '/:package/:_rev?/:revision?' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/:package/-rev/*' +0ms
  express:router:layer new '/:package/-rev/*' +0ms
  express:router:route delete '/:package/-rev/*' +1ms
  express:router:layer new '/' +0ms
  express:router:route delete '/:package/-rev/*' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/:package/-/:filename/-rev/:revision' +0ms
  express:router:layer new '/:package/-/:filename/-rev/:revision' +0ms
  express:router:route delete '/:package/-/:filename/-rev/:revision' +0ms
  express:router:layer new '/' +0ms
  express:router:route delete '/:package/-/:filename/-rev/:revision' +1ms
  express:router:layer new '/' +0ms
  express:router:route new '/:package/-/:filename/*' +0ms
  express:router:layer new '/:package/-/:filename/*' +2ms
  express:router:route put '/:package/-/:filename/*' +0ms
  express:router:layer new '/' +0ms
  express:router:route put '/:package/-/:filename/*' +1ms
  express:router:layer new '/' +0ms
  express:router:route put '/:package/-/:filename/*' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/:package/:version/-tag/:tag' +0ms
  express:router:layer new '/:package/:version/-tag/:tag' +0ms
  express:router:route put '/:package/:version/-tag/:tag' +0ms
  express:router:layer new '/' +0ms
  express:router:route put '/:package/:version/-tag/:tag' +1ms
  express:router:layer new '/' +0ms
  express:router:route put '/:package/:version/-tag/:tag' +0ms
  express:router:layer new '/' +0ms
  express:router:route put '/:package/:version/-tag/:tag' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/ping' +0ms
  express:router:layer new '/-/ping' +0ms
  express:router:route get '/-/ping' +1ms
  express:router:layer new '/' +0ms
  express:router use '/' router +0ms
  express:router:layer new '/' +0ms
  express:router use '/' <anonymous> +11ms
  express:router:layer new '/' +0ms
  express:router use '/' securityIframe +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/static/:filename' +1ms
  express:router:layer new '/-/static/:filename' +0ms
  express:router:route get '/-/static/:filename' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/-/verdaccio/logo' +0ms
  express:router:layer new '/-/verdaccio/logo' +0ms
  express:router:route get '/-/verdaccio/logo' +1ms
  express:router:layer new '/' +0ms
  express:router:route new '/' +0ms
  express:router:layer new '/' +0ms
  express:router:route get '/' +0ms
  express:router:layer new '/' +0ms
  express:router use '/' router +0ms
  express:router:layer new '/' +0ms
  express:router use '/' urlencodedParser +6ms
  express:router:layer new '/' +0ms
  express:router use '/' <anonymous> +0ms
  express:router:layer new '/' +1ms
  express:router use '/' securityIframe +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/packages' +0ms
  express:router:layer new '/packages' +1ms
  express:router:route get '/packages' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/package/readme/(@:scope/)?:package/:version?' +0ms
  express:router:layer new '/package/readme/(@:scope/)?:package/:version?' +0ms
  express:router:route get '/package/readme/(@:scope/)?:package/:version?' +0ms
  express:router:layer new '/' +0ms
  express:router:route get '/package/readme/(@:scope/)?:package/:version?' +1ms
  express:router:layer new '/' +0ms
  express:router:route new '/sidebar/(@:scope/)?:package' +0ms
  express:router:layer new '/sidebar/(@:scope/)?:package' +0ms
  express:router:route get '/sidebar/(@:scope/)?:package' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/search/:anything' +1ms
  express:router:layer new '/search/:anything' +0ms
  express:router:route get '/search/:anything' +0ms
  express:router:layer new '/' +0ms
  express:router:route new '/login' +0ms
  express:router:layer new '/login' +0ms
  express:router:route post '/login' +1ms
  express:router:layer new '/' +0ms
  express:router use '/-/verdaccio/' router +0ms
  express:router:layer new '/-/verdaccio/' +0ms
  express:router:route new '/*' +0ms
  express:router:layer new '/*' +0ms
  express:router:route get '/*' +0ms
  express:router:layer new '/' +0ms
  express:router use '/' <anonymous> +1ms
  express:router:layer new '/' +0ms
  express:router use '/' final +0ms
  express:router:layer new '/' +0ms
 warn --- http address - unix:/var/www/verdaccio/verdaccio.sock - verdaccio/3.3.0
  express:router dispatching POST /-/verdaccio/login +4s
  express:router query  : /-/verdaccio/login +3ms
  express:router expressInit  : /-/verdaccio/login +2ms
  express:router corsMiddleware  : /-/verdaccio/login +1ms
  express:router log  : /-/verdaccio/login +2ms
 info <-- undefined requested 'POST /-/verdaccio/login'
  express:router errorReportingMiddleware  : /-/verdaccio/login +3ms
  express:router <anonymous>  : /-/verdaccio/login +0ms
  express:router <anonymous>  : /-/verdaccio/login +1ms
  express:router compression  : /-/verdaccio/login +0ms
  express:router router  : /-/verdaccio/login +1ms
  express:router dispatching POST /-/verdaccio/login +1ms
  express:router <anonymous>  : /-/verdaccio/login +0ms
  express:router jsonParser  : /-/verdaccio/login +1ms
  express:router <anonymous>  : /-/verdaccio/login +29ms
  express:router encodeScopePackage  : /-/verdaccio/login +0ms
  express:router router  : /-/verdaccio/login +2ms
  express:router dispatching POST /-/verdaccio/login +0ms
  express:router <anonymous>  : /-/verdaccio/login +0ms
  express:router securityIframe  : /-/verdaccio/login +1ms
  express:router trim prefix (/-/verdaccio) from url /-/verdaccio/login +1ms
  express:router router /-/verdaccio : /-/verdaccio/login +0ms
  express:router dispatching POST /login +0ms
  express:router urlencodedParser  : /-/verdaccio/login +1ms
  express:router <anonymous>  : /-/verdaccio/login +0ms
  express:router securityIframe  : /-/verdaccio/login +0ms
 fatal--- uncaught exception, please report this
undefined

$ npm -ddd prints:

npm info it worked if it ends with ok
npm verb cli [ '/usr/bin/node', '/usr/bin/npm', '-ddd' ]
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>
Usage: npm <command>

where <command> is one of:
    access, adduser, bin, bugs, c, cache, completion, config,
    ddp, dedupe, deprecate, dist-tag, docs, doctor, edit,
    explore, get, help, help-search, i, init, install,
    install-test, it, link, list, ln, login, logout, ls,
    outdated, owner, pack, ping, prefix, profile, prune,
    publish, rb, rebuild, repo, restart, root, run, run-script,
    s, se, search, set, shrinkwrap, star, stars, start, stop, t,
    team, test, token, tst, un, uninstall, unpublish, unstar,
    up, update, v, version, view, whoami

npm <command> -h     quick help on <command>
npm -l           display full usage info
npm help <term>  search for help on <term>
npm help npm     involved overview

Specify configs in the ini-formatted file:
    /home/bpage/.npmrc
or on the command line via: npm <command> --key value
Config info can be viewed via: npm help config

<EMAIL_ADDRESS>/usr/lib/node_modules/npm

$ npm config get registry prints:
https://registry.npmjs.org/

Additional context
Add any other context about the problem here.

Yes, you are right, a proper error message should be displayed. Could you help us debugging since your environment is a bit hard to emulate?
If you manage to provide where exactly is failing we can patch it.

I'm happy to. How can I provide the details you need?
Also, I sorted out the activedirectory config and everything is working. The error is most likely coming from the activedirectory module via the verdaccio-activedirectory module. It might be as simple as handling errors from authentication plugins.


Redundant flux-system namespace

Describe the bug
Running wego install | kubectl apply -f - creates two namespaces wego-system and flux-system when there should only be one namespace created.
Environment

wego: v0.0.0

Affects versions
wego: v0.0.1
To Reproduce
Steps to reproduce the behavior:

Get or build the latest wego binary
Run the command wego install | kubectl apply -f -
Observe the log and see there's two namespaces created
Logs

Expected behavior
There should only be one namespace created either default wego-system or the name provided by --namespace flag
Config and Logs
namespace/flux-system unchanged
namespace/wego-system unchanged
customresourcedefinition.apiextensions.k8s.io/alerts.notification.toolkit.fluxcd.io configured
customresourcedefinition.apiextensions.k8s.io/buckets.source.toolkit.fluxcd.io configured

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Since changes were made to wego install, the console output has changed into a different one. This bug is no longer an issue. New output screenshot has been provided.


Add publication section

A lot of academic CVs contain a list of publications, could we add support for a publication section?

perhaps also coursework and interests?

You can try adding a Publications optional section and submitting a PR.
But if you want the best results for your specific publications, download the Latex and add the publications with Bibtex manually. The least thing you want is GPT hallucinations for your co-authors


[Feature] add be config brpc_connection_type

Why I'm doing:
The single connection type may be problematic in some scenarios. (network not fully utilized, rpc queue stuck), we introduce config to modify the connection type.
What I'm doing:
default connection type is single. add connection type will help troubleshoot network problems.
Fixes #issue
What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

https://github.com/Mergifyio backport branch-3.2 branch-3.1 branch-3.1-cs


feat(ForeignKeySearchInput): wrap media files in static()

https://github.com/django-extensions/django-extensions/issues/1136

Coverage increased (+0.004%) to 71.357% when pulling bea5cdf2433436458d81885df04e8676ef637425 on matpfernandes:matpfernandes/admin-widget into e6df1174826f705e0a377048ff7ad1d55ab263b7 on django-extensions:master.

Could you please fix the build / indentation errors ?
I'd prefer to use something like:
    def _media(self):
        js_files = [
            static('django_extensions/js/jquery.bgiframe.js'),
            static('django_extensions/js/jquery.ajaxQueue.js'),
            static('django_extensions/js/jquery.autocomplete.js'),
        ]

        return forms.Media(
            css={'all': (static('django_extensions/css/jquery.autocomplete.css'), )},
            js=js_files,
        )

Thanks !


fix: Tabs tablist will forever expand despite the tab containers width

🐛 Bug Report
If you have too many tabs, the list of tabs along the top will expand forever, claiming more and more horizontal space irrespective of the width contraints of the Tabs container
💻 Repro or Code Sample
The "dynamic" example in the demo site has this issue. Keep adding tabs and you will see that they just extend off the end. https://www.fluentui-blazor.net/Tabs
🤔 Expected Behavior
Tabs should stop and instead display the overflow for any tabs overflowing.
😯 Current Behavior
The tab list expands forever

💁 Possible Solution
🔦 Context
If a user displays too many tabs, the overflow will not kick in.
🌍 Your Environment
Windows, latest Chrome

Overflow kicks in on resize of the window. We;ll see if we can make that work better.

Will be fixed with #1172


expose the vite-node bin

Clear and concise description of the problem
I'd like to run vite-node locally instead of pnpx vite-node, which pnpm has to resolve in runtime
Suggested solution
expose the bin file vite-node
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Duplicate of https://github.com/vitest-dev/vitest/issues/2554?

I've seen that, and it is true that solving #2554 might end up closing this as well, but just exposing the bin, AFAIK is pretty cheap, and it anyway it pushes #2554 forward.
Thoughts?

Bin was always exposed.


Added Hourglass Loader

Added new loader in loader component section .
Closes #486
Description👨💻
Added Hourglass loader in loader component section .

[X] New feature (non-breaking change which adds functionality)

How this has been tested✅
It has been tasted in the local machine and it works perfectly .
Checklist✅

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I am an Open Source Contributor

Screenshots/GIF📷

Merged 🎉 @bera474


observablehq-create should include a favicon

We're planning to serve a default one at /favicon.ico (https://github.com/observablehq/observablehq/issues/15747), but Visnu suggested that observablehq-create could also configure one for a new project.
See slack thread https://observablehq.slack.com/archives/C062YRED4TB/p1706990280011629

Since #692 has been fixed this should be easier.


[BugFix] ban partition clause on external table

Fixes #issue
ban query with partition clause on external table
select * from table partition(dt='2020-01-01');
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:disappointed: fail : 3 / 4 (75.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/QueryAnalyzer.java
1
2
50.00%
[959]

:large_blue_circle:
com/starrocks/scheduler/PartitionBasedMvRefreshProcessor.java
2
2
100.00%
[]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:disappointed: fail : 3 / 4 (75.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/QueryAnalyzer.java
1
2
50.00%
[959]

:large_blue_circle:
com/starrocks/scheduler/PartitionBasedMvRefreshProcessor.java
2
2
100.00%
[]

@Mergifyio backport branch-3.2

@Mergifyio backport branch-3.1


Fix read_index_ids to use nanos as block start and use std bytes pool

I think this was all addressed in #487


Learning to simulate: Generating new trajectories on different environments

Hi!
I have trained the model from Learning to Simulate using the WaterRamps dataset. Now I would like to use the trained model in a different environment (different location of sources and walls). I have two questions:

How should I code the new environment (where should I write the location of the different solid regions etc.)?
Which command should I use to generate the trajectories?

Thank you so much for your help.

Thanks for the message, unfortunately this code provided as is, but documenting dataset generation is beyond the scope of this reference implementation of the model.
That said, you may find some of the advice from https://github.com/deepmind/deepmind-research/issues/199 useful.


[UI] Support for OpenType text layout.

Is your feature request related to a problem? Please describe.
When using TextBlock to display text written in certain script (in my case, Thai), text is not displayed correctly due to lacking of OpenType layout capability.
For example, this is from Stride <IP_ADDRESS>-beta2-0926

And this is from Notepad

Font in use is Sarabun.
notice that the tick (Mai-ek, for those who knows Thai) on the top of certain character is missing from a number of positions. This particular character is a mark which would attach to the character before itself. Without mark-to-mark or mark-to-base feature, it would be positioned statically and could be obstructed by surrounding character.
This character also have a smaller variant to be used in some case. This requires Glyph Substitution.
Describe the solution you'd like
To add support for OpenType text shaping. This could be done using either OS's text rendering service like DirectWrite or Core Text, or using opensource library like Harfbuzz.
Describe alternatives you've considered
There are fonts that designed specifically to avoid the limitation of TrueType-like text shaping. The output might not be perfect but at least it is readable.
Of course this would not work on Arabic or Persian scripts.
Additional context
From what I've researched, there are engines that support OpenType layout, and those not.

Unreal Engine uses Harfbuzz internally.
Godot Engine have a PR for including Harfbuzz. I don't know if this is already merged in or not.
Unity3D does not support OpenType shaping. A workaround (using offline Glyph Substitution) is currently in-use by most studio here. I think TextMate Pro also have some kind of workaround in-place (probably offline glyph substitution as well).

Personally I think ... implementing mark-to-mark and mark-to-base feature should suffice for most scripts (outside of those RTL of course). I will be looking more into the current spritefont implementation in Xenko and how could it be enhanced to support this. Please don't hold your breath though, I can't even got the source built at this point...


fix(*): Style updates, add edit issue body functionality

Various fixes/updates for #381

Changes color of mark all notifications button

Adds padding for no notifications message in notifications screen

Fixes this problem:

Allows editing of issue description

Makes title/subtitle of IssueDescriptionListItem better in terms of boldness

Removes the user title from repository list item (@lex111 - with this it's unlikely we'll see the private box being squeezed anymore as most repo titles aren't nearly as long)

Removes the user title from repository list item (@lex111 - with this it's unlikely we'll see the private box being squeezed anymore as most repo titles aren't nearly as long)

Let's remove this label and display this icon?

Adds padding for no notifications message in notifications screen

And why are they needed? There in fact there is an alignment on the center, and after addition padding, any more will not be absolutely on the center to be displayed.

@housseindjirdeh User titles are needed, at least for searched result or stared repositories. This feature is added by #254 in particular. And I also mentioned the redundancy, but you missed it. :)

Seconding @lex111, let's use that emoji and bring back username/reponame :
If I'm on some user profile, then check his repos, I'll have to remember the user name, or navigate back and forth to have the repo full name to clone it

@housseindjirdeh What do you think about handling with edit issue body in another screen. Maybe we could create an edit issue for edit the description and the title.

And why are they needed? There in fact there is an alignment on the center, and after addition padding, any more will not be absolutely on the center to be displayed.

Honestly I just felt that the message looks a bit squeezed for English copy?

Probably just imagining things, reverted it back and removed that padding :)

User titles are needed, at least for searched result or stared repositories. This feature is added by #254 in particular. And I also mentioned the redundancy, but you missed it. :)

Have no idea how I missed that point. Thank you, and I completely forgot that search and starred repos would have to show that. Updated it so now only the authenticated list of users repositories won't show (otherwise it will :))

Let's remove this label and display this icon?

Done:

@jouderianjr so you make a good point, and right now if you head to the settings screen you can modify a few issue settings (but not title or description)

We could have issue title and description modifiable here but honestly I actually prefer at least the description being modified the same way as any other comment. It feels intuitive as I still think of it as a comment.
Definitely curious if you think there's a better option however, as well as how we can add functionality to edit the issue title in the future 🤔

Honestly I just felt that the message looks a bit squeezed for English copy?

Oops, did not notice if padding left and right, then this should not affect the alignment, although at first glance it seemed to me that this is so. Excuse me, let's it return, but can we make them smaller?

Let's remove this label and display this icon?

Do not you think that the size is big? In my opinion it was better if you reduce the size of the icon?

Oops, did not notice if padding left and right, then this should not affect the alignment, although at first glance it seemed to me that this is so. Excuse me, let's it return, but can we make them smaller?

Cheers no worries. Added some padding but reduced it a bit :)

Do not you think that the size is big? In my opinion it was better if you reduce the size of the icon?

And nice you're probably right. Reduced it:

@housseindjirdeh I'd rather use type="default" (or no type) for the "Mark all as read" button.
Let's keep "success" for strong actions, like merge

@housseindjirdeh I'd rather use type="default" (or no type) for the "Mark all as read" button.
Let's keep "success" for strong actions, like merge

Done.
Although the default button looks interesting 🤔 Are these the intended colors? (just showing it defaulted for the merge button for demo, don't have any notifications in the moment that's why :))

@housseindjirdeh that icon should be black, otherwise it's the intended look.

We could have issue title and description modifiable here but honestly I actually prefer at least the description being modified the same way as any other comment. It feels intuitive as I still think of it as a comment.
Definitely curious if you think there's a better option however, as well as how we can add functionality to edit the issue title in the future 🤔

I got it, make sense, so we can release that in v1.3 and then reorganize this screen in order to fit edit title and body 😄
Nice job mate!

@housseindjirdeh that icon should be black, otherwise it's the intended look.

@machour fixed:

@jouderianjr Exactly what I was thinking :)


Remove etcd proposal alerts.

How to categorize this PR?
/area monitoring
/kind cleanup
What this PR does / why we need it:
Removes alerts which are built on the proposals_failed_total metric of etcd.
The information that these alerts currently convey are already conveyed to the operator through other means, and there is no action an operator can take to fix the issue that raises these alerts (primarily leader election among members of the etcd cluster). See #10514 for more information.
Which issue(s) this PR fixes:
Fixes #10514
Special notes for your reviewer:
/cc @shreyas-s-rao @ashwani2k @vicwicker @rickardsjp
Release note:

Alerts based on the `proposals_failed_total` metric of the etcd cluster are not raised anymore.

/retest-required

@rickardsjp @vicwicker could you take a look?

/lgtm, thank you!


docs: added 20 quiz questions for graphs and trees

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #56220 will close issue 56220

Added exactly 20 Questions which are tested.
All quiz questions are related to one of these topics:
a.  What are graphs? (types like scatterplot, bar graph)
b. Depth first and breadth first search
c. Matrices and adjaceny lists
d. Trees and tries
All questions are unique and contains different concepts of the above topics.
All questions have different difficulty levels.

You have not been assigned to the issue, stop openign PRs


Replace the deprecated RSSLink with the recommended workaround

I just upgraded to Hugo 0.58.3, and looks like .RSSLink is now deprecated:
https://gohugo.io/variables/page/#page-variables
Replacing with the recommended solution here to remove the warning.

@Xzya Would you mind having a look at this? Would love to remove the warning from the log output.

No longer a warning on 0.93, it's broken.


Example uses "For internal use only" method

The InputText based on the input event section uses CreateBinder(), which is labeled as "For internal use only."
Please update with a safe alternative. See https://github.com/dotnet/aspnetcore/issues/24883#issuecomment-710099785 for a related discussion.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: eef47457-c8cf-74a4-b884-c1fe5c2ef52d
Version Independent ID: 09a02d10-2834-467d-d8c2-3fd19ceb3863
Content: ASP.NET Core Blazor forms and validation
Content Source: aspnetcore/blazor/forms-validation.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Thanks @mrlife ... Let's ask ..........
@pranavkm @SteveSandersonMS  ... The original version of this (it was updated at least once along the way) was a Ryan bit that came at 3.0 Preview 9 ...

Issue: [Preview 9] Add sample for InputText with oninput (dotnet/AspNetCore.Docs #13912)
PR: Add sample for InputText with oninput (dotnet/AspNetCore.Docs #14123)

Do you want to keep it, replace it, or 🔪💀 kill it? 💀🔪

@SteveSandersonMS can you please look into this and see whether there is an alternative API we can use in this sample? Thanks!

@guardrex Do you remember any details about why the sample was written that way? My first thought is there's no obvious reason to use EventCallback.Factory.CreateBinder here - it looks like it could just use @bind. That is, CustomInputText.razor could be:
@inherits InputText
<input
    @attributes="AdditionalAttributes"
    class="@CssClass"
    @bind="CurrentValueAsString"
    @bind:event="oninput" />

Do you know of some reason that doesn't work?

It looks like it was just a workaround back when Ryan sent it over in August, 2019 (ages ago during Preview 9) ...
https://github.com/dotnet/AspNetCore.Docs/issues/13912

OK cool, looks like it could be updated then!

Thank you all, it works great!


Update sql-database-geo-replication-overview.md

#sign-off

#sign-off

@anosov1960 : Thanks for your contribution to the Azure documentation! The author, @anosov1960, has been notified to review your proposed change.

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.
It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We've closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions


Can't Dispatch Redux Action From Outside of Component (With Preloaded State)

Hello! I am having an issue where I am unable to dispatch actions from outside of a component. I have found many other people with this issue, but it seems that someone always submits an answer assuming that the "createStore" file always returns a store object:

https://stackoverflow.com/questions/45043219/calling-dispatch-function-from-a-blank-javascript-file
https://stackoverflow.com/questions/38460949/what-is-the-best-way-to-access-redux-store-outside-a-react-component
https://stackoverflow.com/questions/56498212/dispatching-a-redux-thunk-action-outside-of-a-react-component-with-typescript

However, my createStore file looks like this:
import { composeWithDevTools } from 'redux-devtools-extension';
import combinedReducers from './reducers/root-reducer';
import { load, save } from 'redux-localstorage-simple';
import { createStore, applyMiddleware } from 'redux';
import todosCustomMiddleware from './middlewares/todosCustomMiddleware';
import loginCustomMiddleware from './middlewares/loginCustomMiddleware';
import socketManager from './middlewares/socketManager';
import { ILoginState } from './reducers/login';
import { ITodosState } from './reducers/todos';
export interface IState {
  loginReducer: ILoginState,
  todosReducer: ITodosState
}
export default (preloadedState: IState) => {
  return createStore(
    combinedReducers,
    getLoadedState(preloadedState),
    composeWithDevTools(
      applyMiddleware(
        save({ states: ['loginReducer'] }),
        todosCustomMiddleware(),
        loginCustomMiddleware(),
        socketManager()
      )
    ),
  );
};
const getLoadedState = (preloadedState: IState | any) => {
  if (typeof window !== 'undefined')
    return {
      ...preloadedState,
      ...load({ states: ['loginReducer'], disableWarnings: true }),
    }
  return {
    ...preloadedState,
  }
}

And so then when I try to import it and call "dispatch" like this:
import store from './../state/createStore'
store.dispatch(loginSuccess())

Then the event is never handled by my middlewares/reducers, and I also get this typescript error:
Property 'dispatch' does not exist on type '(preloadedState: IState) => Store<{ loginReducer: ILoginState; todosReducer: { fetching: boolean; error: undefined; todos: any; } | { fetching: boolean; error: any; todos: never[]; }; }, AnyAction> & { ...; }'.
Is there any way for me to dispatch actions from outside of a component when I am syncing the state with localstorage? Thanks!

repo located here: https://github.com/JimLynchCodes/WaF-Repo/tree/master/waf-frontend-gatsby

Two observations:
First, we try to keep this repo reserved for actual bugs, not usage questions.
Second, as I just answered on Twitter, your code is wrong.
Your store.js is returning a function (call it configureStore) that actually generates and returns the store instance.
However, your import statement is treating that return value as the store itself, which is incorrect.
So, the correct usage would be:
import configureStore from './../state/createStore'
const store = configureStore()

store.dispatch(someAction)

Also, note that our new official Redux Toolkit package will do much of that store setup logic for you, and I strongly recommend you try using it.

Thanks!
Sorry, you may want to remove "Support / Usage Questions" from the options in the github issue template for this repo. 😁
Yes, you are correct @markerikson, After setting a default empty object preloadedState and making the properties of IState all optional I'm able to call configureStore with no typescript errors. Then actions are being handled properly. 👍 ❤
I'm curious about the Redux Toolkit, but I'm still confused as to what it actually is. Is it an npm package to use with redux? Is it a cli tool that generates code for you? something else?

Sorry, you may want to remove "Support / Usage Questions" from the options in the github issue template for this repo. 😁

@markerikson You might want to check out this nice PR in react. https://github.com/facebook/react/pull/18039

The point of that "Support/Usage Questions" template entry is that users are supposed to read what it says and go to Stack Overflow or Reactiflux :)
Redux Toolkit is an addon package that wraps the Redux core Please read through the Redux Toolkit docs that I linked, including the tutorials.

@dai-shi : nice.  Can you file PRs to add that to the Redux repos?
(May just be Redux and React-Redux atm, I don't think I've got issue templates set up for RTK.)

Pinging @ryota-murakami if he would be interested in filing PRs.

@dai-shi Thanks pinning 😀
@markerikson
I've just made PR and I chose reactflux url than stackoverflow., is it right?
https://github.com/reduxjs/redux/pull/3713
Fundamentally i think github_ discussion feature is ideal solution at the case.
AFAIK that's only available in Next.js repo, i hope public release this as soon as possible 🤔


Issue in Step :  15  there is no such method ViewModelProvider(this)

wordViewModel = ViewModelProvider(this).get(WordViewModel::class.java)
In above line, there is no such method ViewModelProvider(this) instead we should update it to as follows
wordViewModel =  ViewModelProviders.of(this).get(WordViewModel::class.java)
dependency used version is
 implementation "androidx.lifecycle:lifecycle-viewmodel:2.1.0"

Hi, I don't think this solution works. I still get a runtime error. This article helped me fix the issue https://medium.com/koderlabs/viewmodel-with-viewmodelprovider-factory-the-creator-of-viewmodel-8fabfec1aa4f
we need to make a new class, WordViewModelFactory then pass that to ViewModelProvider

Using by viewModels now.


tests-ci : drivers: counter: basic_api test failed

**Describe the bug**
basic_api test is failed on zephyr-v3.0.0-2262-ge8a369e7da50 on mimxrt685_evk_cm33

see logs for details

**To Reproduce**
1.
```
scripts/twister --device-testing --device-serial /dev/ttyACM0 -p mimxrt685_evk_cm33      --sub-test drivers.counter
```
2. See error

**Expected behavior**
test pass

**Impact**

**Logs and console output**

build zephyr-v3.0.0-2262-ge8a369e7da50  ***
Running test suite test_counter
===================================================================
START - test_multiple_alarms
Testing CTIMER_0
Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/counter/counter_basic_api/src/test_counter.c:569: test_multiple_alarms_instance: (2 not equal to cnt)
CTIMER_0: Invalid number of callbacks 1 (expected: 2)
FAIL - test_multiple_alarms in 268.463 seconds
===================================================================
START - test_all_channels
Testing CTIMER_0
E: channel already in use
Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/counter/counter_basic_api/src/test_counter.c:636: test_all_channels_instance: (0 not equal to 1)
CTIMER_0: Unexpected error on setting alarm
FAIL - test_all_channels in 0.22 seconds

**Environment (please complete the following information):**
 - OS: (e.g. Linux )
 - Toolchain (e.g Zephyr SDK)
 - Commit SHA or Version used: zephyr-v3.0.0-2262-ge8a369e7da50

@dleach02 @danieldegrasse @mmahadevan108

@hakehuang This issue should be able to be closed since #45075 merged, correct?

@danieldegrasse correct


[feature request] priority speaker

Assuming it's a feature mainly handled at the client side with the server only telling who has it, priority speaker shouldn't consume much resources. I'd be happy to contribute, but I'm not familiar with the codebase or C conventions in general.
Currently, requests to adjust the priority speaker status of a user seem to be ignored here:
https://github.com/umurmur/umurmur/blob/ae8259343df8cf5dfbe5eda22c58c222d36a5499/src/messagehandler.c#L452
The option to change somebody's priority speaker status becomes available in the Mumble UI when one adds the server's admin_password to access tokens, but the server responds to requests with the text "Not supported by uMurmur".
Thanks!

this is basic option in mumble and should be implemented.


ES6 support?

Hi guys!
I had this error when I tried to use some ES6 code:
ERROR in ./src/scripts/components/report.js Module parse failed: /home/remi/_Dev/JsCoverageQC/node_modules/react-hot-loader/index.js!/home/remi/_Dev/JsCoverageQC/node_modules/jsx-loader/index.js?harmony!/home/remi/_Dev/JsCoverageQC/node_modules/jsxhint-loader/index.js!/home/remi/_Dev/JsCoverageQC/src/scripts/components/report.js Line 134: Unexpected token [ You may need an appropriate loader to handle this file type. | 			// Used to change immutable data : https://facebook.github.io/react/docs/update.html | 			var objVariantChecked_ToAdd = { | 				[key]: { | 					$set: variant | 				}  @ ./src/scripts/components/JsCoverageQcApp.js 15:13-32
I guess this is related to ES6 and I have found an opened issue who seems to address this problem.
Could you guys confirm this? Or is it related to another problem?
If this is the problem, how do you think I can resolve this?

Confirmed, you have to wait for webpack 2 or fork it and switch from esprima to esprima-fb in the package.json. Did it to test, and works quite well.
You can close this ^^.

By the time, I looked deep further into this and found babel-loader supports Computed Property Names.
I replaced it by the js-loader I used before and it works well!
Thanks @Sinewyk ;)


Password protection

Am just wondering about the feasibility of adding some form of passphrase protection. Am well aware that it's a relatively weak form of protection but increasing the level of obfuscation even by a tiny bit could be helpful in slowing/ deterring would-be crackers.

I guess you are thinking about encryption. If it's possible for your webpages to decrypt the data, it probably means that there is enough information on the client side to decrypt it, so there is probably little purpose for this... If that's what you're looking for.
In any case, there are already good libraries out there for this: http://code.google.com/p/crypto-js/ comes to mind first. It can encrypt anything given current state-of-the-art algorithms. I suggest you process the output of lz-string with the CryptoJS algo that suit your need.
The purpose of lz-string is to compress and store efficiently. It's far from perfect on that front, so I'll focus on that for now.


WIP | Enable Configurable Retry Logic by default

It's not required to enable the safety switch "Switch.Microsoft.Data.SqlClient.EnableRetryLogic" for using configurable retry logic feature.
Related links:

Released: Microsoft.Data.SqlClient 3.0 Preview 1
Introducing Configurable Retry Logic in Microsoft.Data.SqlClient v3.0.0-Preview1
Configurable retry logic in SqlClient

Will this change the observed behaviour for existing users that do not have the feature enabled? Will they see commands retrying by default where they did not with previous releases?
Will this have a performance impact for users who do not have the feature enabled today?

My attempt was to take care of customers who don't use this feature. It shouldn't have a perf issue, though.
I'll run a perf test to find it out. In the meanwhile, I'd appreciate it if you could run your benchmark too.

If it's going to be the default then users will just have to live with it in terms of performance. My main concern is for code that doesn't expect to have retries enabled can easily be broken by the library changing to enable them by default.

If the feature needs to be more mainstream then adding a way to enable it which is easier to discover like a static property (or config file setting if no code changes is a required aspect) and then publicizing that would be preferable. It concerns me that a major behavioural change is going to be made without any evidence of user demand for it.
@roji will this change to enable retries by default have any impact on EFCore?

It concerns me that a major behavioural change is going to be made without any evidence of user demand for it.

I tend to agree - especially since I think this feature provides an illusion of resiliency without being able to deliver on it (e.g. transactions, any errors happening after the application already read some rows...). Seems like this did become a GA quite quickly...

@roji will this change to enable retries by default have any impact on EFCore?

I don't think so, aside from any perf penalties this feature brings (I have no idea about that). As far as I understand, this feature doesn't change SqlClient's visible behavior in any way except for not surfacing certain exceptions when a retry succeeds under the hood... EF Core's own resiliency would continue to work on top of this - if the user opted into using it - so we'd have two different retry strategies at the same time at different levels of the stack.
@ajcvickers or @AndriySvyryd may have some other thoughts.

It can potentially drastically change the amount of time for something to fail. For example, imagine that previously EF would retry 3 times, taking a total of 10 seconds before failing. (Numbers made up; I don't remember what the actual defaults are.) Now if SqlClient does several retries itself taking 10 seconds before failing and letting EF retry, now the total time to failure becomes 30 seconds--3 times longer than before. We had people run into this when the original connection retries were introduced. People reported applications hanging because things were being retried so many times.
I agree with @roji's other points, but this at the very least needs to be documented and publicized as a breaking change, since based on previous experience that's how many customers will experience it.

@Wraith2 @roji @ajcvickers
To clarify the behavior, we are not adding retries by default. This is simply removing the extra safety switch we put in front of the retry provider logic to ensure the "preview" feature didn't accidentally affect behavior in the GA release. We feel the feature is ready for GA, and there will still be zero retries by default, so no behavior change. There should be negligible impact on performance, if any. If users want to enable retries, they can do so via the app.config file. They just no longer need the extra AppContext safety switch.

So it is "Enable Configurable Retry Logic to be enabled by default" 🤔

So it is "Enable Configurable Retry Logic to be enabled by default" 🤔

Maybe a better title for this PR would be "Remove Configurable Retry Logic safety switch".

Thanks for clarifying that, @David-Engel

Me too, though I was trying to get the answer about whether it was being enabled for every user by default. If it's disabled by default it's fine.


Enh/acceptance tests

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Build (python setup.py install) was run locally and any changes were pushed
[x] Lint (black rocketpy) has passed locally and any fixes were made for failures
[ ] All tests (pytest --runslow) have passed locally

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

RocketPy's code did not have any unit tests setup.
What is the new behavior?

Implementing acceptance tests with two different rockets data

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

Not really happy with the fact that GitHub actions is currently failing since it is not using pandas. I'll check why it is not installing all the test requirements.

Added a few commits to #100 in order to fix the issue with the GitHub workflow. More specifically 9a230f6.

I will go ahead and accept it. Will see if GitHub action tests continue to fail when a PR from enh/acceptance_tests_fixtures_no_lfs to master is created.


bugfix: fix some comments(#3658)

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #3702 (302b0d2) into develop (7bd0243) will decrease coverage by 0.01%.
The diff coverage is n/a.

:exclamation: Current head 302b0d2 differs from pull request most recent head 0842020. Consider uploading reports for the commit 0842020 to get more accurate results

@@              Coverage Diff              @@
##             develop    #3702      +/-   ##
=============================================
- Coverage      51.68%   51.66%   -0.02%
- Complexity      3521     3523       +2
=============================================
  Files            640      640
  Lines          21420    21420
  Branches        2675     2670       -5
=============================================
- Hits           11070    11066       -4
- Misses          9241     9251      +10
+ Partials        1109     1103       -6

Impacted Files
Coverage Δ
Complexity Δ

...io/seata/core/rpc/netty/RmNettyRemotingClient.java
39.13% <ø> (ø)
10.00 <0.00> (ø)

...source/AbstractDataSourceCacheResourceManager.java
42.85% <0.00%> (-42.86%)
1.00% <0.00%> (-1.00%)

...erver/storage/file/session/FileSessionManager.java
51.82% <0.00%> (-5.11%)
26.00% <0.00%> (-1.00%)

.../main/java/io/seata/rm/BaseDataSourceResource.java
26.41% <0.00%> (-3.78%)
9.00% <0.00%> (-2.00%)

...in/java/io/seata/server/session/GlobalSession.java
83.47% <0.00%> (+0.43%)
76.00% <0.00%> (+1.00%)

...rage/redis/store/RedisTransactionStoreManager.java
64.38% <0.00%> (+0.45%)
38.00% <0.00%> (+1.00%)

...o/seata/server/coordinator/DefaultCoordinator.java
53.02% <0.00%> (+0.46%)
29.00% <0.00%> (+1.00%)

...io/seata/core/rpc/netty/AbstractNettyRemoting.java
14.76% <0.00%> (+1.34%)
5.00% <0.00%> (ø%)

...n/src/main/java/io/seata/common/util/IdWorker.java
83.33% <0.00%> (+6.25%)
12.00% <0.00%> (+1.00%)

由于前期Seata社区治理规范问题部分代码作者未签署CLA，可能引发社区知识产权风险问题。请所有在Seata社区贡献过代码（包含：主项目、官网、samples和多语言项目等）的 contributor 帮忙在这个链接登录github账号签署相应的开发者CLA，https://cla-assistant.io/seata/seata  。2023.1.31 未签署CLA的代码将会被rewrite，拜托大家帮忙签一下。


Disable auto block pr option of contributor-assistant/github-action

Enhancement
Partof https://github.com/kcl-lang/community/issues/18
This is suitable for good first issue

Closed by #256


Stripe PoC: checkout and order stubs

Cumbersome Stripe PoC for frontend part.
Test in conjunction with local BE instance, refer to Local Frontend + Backend instruction.
Test cards list: https://docs.stripe.com/checkout/embedded/quickstart?client=react#testing

Scenario 1: no address is specified
https://github.com/user-attachments/assets/65af043f-13db-4334-b776-e23700f94eeb
Scenario 2: address is specified
NOTE the issue with the items counter

@horprogs, спасибо! 🚀


Doesn't seem to install and use the correct version of Cypress

Our project references Cypress 8.7.0 as a dependency, which is checked out prior to running this action.  If we run this action to then install Cypress and then run npx cypress info it reports it is running 9.5.0.  How to we ensure it uses the correct version?

This is a stale issue.


ActiveRecord coupling fix

Just updated to latest master, and came across this error complaining about missing AR
  1) Supports API GET /supports?filter[proposal_id]=:proposal_id&filter[participant_id]=:participant_id support resource
     Failure/Error: subject { get "/supports?#{filter_params}", {}, headers  }
     NameError:
       uninitialized constant ActiveRecord
     # /Users/work/.rvm/gems/ruby-2.2.2/bundler/gems/jsonapi-resources-72168924065f/lib/jsonapi/resource.rb:689:in `block in _add_relationship'
     # /Users/work/.rvm/gems/ruby-2.2.2/bundler/gems/jsonapi-resources-72168924065f/lib/jsonapi/resource.rb:685:in `each'
     # /Users/work/.rvm/gems/ruby-2.2.2/bundler/gems/jsonapi-resources-72168924065f/lib/jsonapi/resource.rb:685:in `_add_relationship'
     # /Users/work/.rvm/gems/ruby-2.2.2/bundler/gems/jsonapi-resources-72168924065f/lib/jsonapi/resource.rb:346:in `has_one'
     # ./app/resources/support_resource.rb:4:in `<class:SupportResource>'
     # ./app/resources/support_resource.rb:3:in `<top (required)>'

This app is using mongodb/mongoid, and so far I had been fixing the AR dependency like this:
# app/controllers/my_resource_controller.rb
class MyResourceController < ApplicationController
  include JSONAPI::ActsAsResourceController

  def create_operations_processor
    TempMongoidOperationsProcessor.new
  end
end

# lib/temp_mongoid_operations_processor.rb
class TempMongoidOperationsProcessor < JSONAPI::OperationsProcessor

  private

    def transaction
      # if @transactional
      #   ActiveRecord::Base.transaction do
      #     yield
      #   end
      # else
      yield
      # end
    end

    def rollback
      # raise ActiveRecord::Rollback if @transactional
    end

    def process_operation(operation)
      # operation.apply(@context)
      operation.apply
    # rescue ActiveRecord::DeleteRestrictionError => e
    #   record_locked_error = JSONAPI::Exceptions::RecordLocked.new(e.message)
    #   return JSONAPI::ErrorsOperationResult.new(record_locked_error.errors[0].code, record_locked_error.errors)

    rescue Mongoid::Errors::DocumentNotFound
      record_not_found = JSONAPI::Exceptions::RecordNotFound.new(operation.associated_key)
      return JSONAPI::ErrorsOperationResult.new(record_not_found.errors[0].code, record_not_found.errors)
    end
end

Is my fix outdated? Is there a new recommended fix to get non-AR ORM/ODMs going?

@oliverbarnes Sorry about that. Can you please check if #362 fixes this?

@lgebhardt no worries, I've using #362's branch and it does fix it (I commented there a few days ago :) ). Thanks


VS Test task fails for data-driven tests that use TestCase as data source

I have the following setup:

Compiled my solution with my test assemblies in 15.9.0 Preview 3 (I have also tried 15.8.6)
VSTS Agent version 2.141.0 (I have also tried other agent versions including 2.140.2)
Test agents are Win10 OS
On-Premise TFS 2018 Update 3

I have a release definition with the following steps:

VsTest Platform Installer installing the latest pre-release (15.9.0)
Copy Test Assemblies - copies assemblies from a drop location to the agent
Visual Studio Test - runs tests from a test plan

I have followed the instructions listed here and created the Test.TestCaseAccessToken variable for my release definition:
https://docs.microsoft.com/en-us/visualstudio/releasenotes/tfs2018-update1#testpipeline
The Visual Studio Test task fails for test cases that have the DataSource as Microsoft.VisualStudio.TestTools.DataSource.TestCase are failing with the following error:
The unit test adapter failed to connect to the data source or to read the data. Could not load file or assembly 'Microsoft.TeamFoundation.TestManagement.Controller or one of its dependencies. The system cannot find the file specified.
Upon putting the missing dlls (there are 9 of them) into the C:\agent_work_tool\VsTest\15.9.0\x64\tools\net451\Common7\IDE\Extensions\TestPlatform folder on the agent, I get around those missing assemblies errors. However, I get to this error and don't know how to move further:
The unit test adapter failed to connect to the data source or to read the data. Error details: Object reference not set to an instance of an object.
I have attempted running the test using vstestconsole.exe via command line and I get the same results. However, if I run the test locally within my Visual Studio IDE, the test passes as expected.
I am not comfortable posting the vstest.console logs here. If you need additional logs, please provide me with an email address to send them to.

@NDDeveloper13 There were various issues in test case data driven based tests. We have resolved them in 15.9 preview 4 which will be publicly available on 23rd October.

Upon putting the missing dlls (there are 9 of them) into the C:\agent_work_tool\VsTest\15.9.0\x64\tools\net451\Common7\IDE\Extensions\TestPlatform folder on the agent, I get around those missing assemblies errors

You don't need to do this when you try in 15.9 preview 4 once available.

The unit test adapter failed to connect to the data source or to read the data. Error details: Object reference not set to an instance of an object.

You will be able to resolve this issue with 15.9 preview 4 along with using Test.TestCaseAccessToken variable.
Closing this issue for now. Please re-open in case you face any issues in 15.9 preview 4.

We are still working with old VS 2013 test controllers and test agents and have been trying to upgrade to use build/release definitions for years, but have been unable to because all of the features supported with lab center and automated testing within MTM were not supported with build/release definitions. The last piece of it was data-driven testing.
TFS 2018 RTW release notes states:
"Support for Lab Center and automated testing flows in Microsoft Test Manager has been removed."
https://docs.microsoft.com/en-us/visualstudio/releasenotes/tfs2018-relnotes#features-removed-in-tfs-2018-rtw
With XAML builds reintroduced back in 2018 Update 2, I was hoping Lab Center and automated testing flows in MTM were added back as well, but it appears not to be, according to the exception I get when attempting to add the lab environment to TFS 2018:
(QTController.exe, PID 12084, Thread 9) DEPLOY: PollDeploymentRuns : Handled Exception System.NotSupportedException: Using a test controller with Team Foundation Server 2018 for creating and managing Lab environments is no longer supported. You can use Build and Release Management for automated testing. Refer to https://aka.ms/xiu7ff for more details. --->
We are blocked from upgrading to TFS 2018, as a large percentage of our tests are data driven via test case. I was originally told that data-driven testing was fixed in VS 2017 Update 6, then VS 2013 Update 9 preview 3, now yet another later version. We have been blocked from upgrading to TFS 2018 because of this issue.

I have come up with a solution that removes the dependency on a fix being available for data driven tests using test cases. I'm posting it here so that maybe it will be useful to someone else.

Remove the DataSource attribute for data-driven tests using test cases and replace with this:
[TestProperty("DataSourceTestCaseId", "52550")]

In your test initialize, put the following code:
if (TestContext.Properties["DataSourceTestCaseId"] != null)
{
PopulateTestData(int.Parse(TestContext.Properties["DataSourceTestCaseId"].ToString()));
}
`
 private void PopulateTestData(int testCaseId)
 {
 	//Get the test case data via TFS
 	List<DataRow> testCaseDataRows;
 	using (TfsTeamProjectCollection collection = new TfsTeamProjectCollection(
 		TfsTeamProjectCollection.GetFullyQualifiedUriForName(ConfigurationManager.AppSettings["TFSServerURL"])))
 	{
 		ITestManagementService tcmService = collection.GetService<ITestManagementService>();
 		ITestManagementTeamProject project = tcmService.GetTeamProject(ConfigurationManager.AppSettings["TFSProject"]);
 		DataSet testCaseDataSet = project.TestCases.Find(testCaseId).DataReadOnly;
 		testCaseDataRows = testCaseDataSet.Tables[0].Rows.OfType<DataRow>().ToList();
 	}

 	//Set the DataRow property - it is read only, so need to use reflection in order to do this
 	FieldInfo fieldInfo = TestContext.GetType().GetRuntimeFields().First(x => x.Name.Contains("dataRow"));
 	fieldInfo.SetValue(TestContext, testCaseDataRows[0]);
 }

`
NOTE: We use test case parameters as a way to easily swap out data without a code change, and we currently don't have a case where we have multiple rows of data. This solution will not work if you have multiple rows of data. If you do have multiple rows of data, instead of setting the TestContext.DataRow, you could potentially set an entirely different TestContext property with the List object and have your test case manually iterate through the rows, but of course you will lose everything that comes along with the automatic iteration of rows which is the point of the DataSource attribute. Hopefully the data driven test issues are fixed soon.


Update README.md

It requires the Shapely module that is not required by HA itself, so you may need to install it.

I don't think that Shapely needs to be installed manually. It's part of the manifest (here) and therefore HA installs it automatically.

When installing through HACS, not when adding the component manually.

I don't think it matters. In both cases the dependencies get installed.

From what I saw in my setup, it didn't install them

The conclusion is that it happened because it's a venv environment which doesn't supporting installing requirements. Since this is an environment generic behavior, there is no reason to add a comment about it for every custom integration that is out there.
Closing the PR.


Querying table fails with  Value::LIST without providing a child-type requires a non-empty list of values. Use Value::LIST(child_type, list) instead.

Querying an Iceberg table throws the error
SELECT * from iceberg_scan('gs://path/to/metadata.json');

INTERNAL Error: Value::LIST without providing a child-type requires a non-empty list of values. Use Value::LIST(child_type, list) instead.
This error signals an assertion failure within DuckDB. This usually occurs due to unexpected conditions or errors in the program's logic.
For more information, see https://duckdb.org/docs/dev/internal_errors

The iceberg table is stored in GCS. I've verified that querying the underlying Iceberg parquet tables works, so I believe the issue is coming from the iceberg scan.
Has anyone seen this before?

Got to the bottom of it - this happened because the Iceberg table metadata was incorrect. A more intuitive error message could be thrown but that can be a separate issue.


Missung properies for multiple assemblies and dictionaries

In Wiki are samples for using assemblies and dictionaries
"{lex:Loc Key=Test, Assembly=MyAssembly, Dict=MyResource}"
But the Properties Assembly or Dict are not available
https://github.com/SeriousM/WPFLocalizationExtension/wiki/Multiple-assemblies-and-dictionaries

We'll need to update the documentation as this property was marked obsolete and consequently removed in one of the early releases. You need to use the attached properties of ResxLocalizationProvider.
@MartinEgli, thanks for pointing it out.

docu is changed


Fix function comments based on best practices from Effective Go

Hi, we updated some exported function comments based on best practices from Effective Go. It’s admittedly a relatively minor fix up. Does this help you?

merged; ty!


Change the way time is added to a model

This pull request changes the way time is added to a model:
time = pd.date_range('2000', '2000-1-4')
model_ds = nlmod.mdims.set_model_ds_time(model_ds, time=time)

<xarray.Dataset>
Dimensions:  (time: 4)
Coordinates:
  * time     (time) datetime64[ns] 2000-01-01 2000-01-02 2000-01-03 2000-01-04
Data variables:
    *empty*
Attributes: (12/13)
    time_units:                DAYS
    start_time:                1990-01-01 00:00:00
    nstp:                      1
    tsmult:                    1.0
    steady_start:              1
    steady_state:              0

When steady_start is True, a steady state stress-period is added before the first value of time with a stress period length of steady_start_perlen. The current ways to specify time (transient_timesteps or perlen) are still supported.
The variable time now always contains the time at the end of each stress period (not sure if this was the case before), so heads and other output can be easily added to the model_ds. To minimise double information, nper and perlen are removed from the attributes of model_ds, and are calculated when needed.

Nice improvements!
I did add something else when I went through the changes. I moved the attributes with time information ('time_units', 'start_time', 'nstp', 'tsmult', 'steady_start' and 'steady_state') from the model_ds to the time DataArray. So instead of model_ds.attrs['time_units'] = time_units you get model_ds.time.attrs['time_units'] = time_units.
I tested locally and it worked. So when the test pass this can be merged.

Good change. Maybe we can then also rename 'time_units' to 'units' and 'start_time' to 'start'?

Good one, added this in 8e54b1b


Prevent infinite recursion where draft history contains loops

Some draft histories (e.g., for RFC7982) contain loops, which can cause infinite recursion

Looks good – might be good to have a test case too?


Set up Patrol

Set up Patrol on Android, make a simple test pass.
See original PR here

too slow 🥵


[Question] How to extract filesize before download?

Checklist

[X] I'm asking a question
[X] I've looked through the README and FAQ for similar questions
[X] I've searched the bugtracker for similar questions including closed ones

Question

So I wanted to extract filesize information before download and it seems to work fine when I select a single format video. But when I merge multiple formats it always shows 'NA'. Although I can clearly see there is a filesize information in a json file, but I dont know how to extract it. Also is it possible to format filesize in a similar manner to formatting time? Or how to convert it to MB/GB?
File with multiple formats doesn't return filesize
yt-dlp LXb3EKWsInQ --print webpage_url --print "%(title)s (%(duration>%H:%M:%S)s)" --print "%(uploader)s (%(channel_url)s)" --print "Uploaded: %(upload_date>%d.%m.%Y)s              Filesize: %(filesize)d          Ext: %(ext)s" --print "Selected format(s): %(format)s" --print "VBR: %(vbr)d              FPS: %(fps)s                    Codec: %(vcodec)s" --print "ABR: %(abr)d                SR: %(asr)d                     Codec: %(acodec)s"

https://www.youtube.com/watch?v=LXb3EKWsInQ
COSTA RICA IN 4K 60fps HDR (ULTRA HD) (00:05:14)
Jacob + Katie Schwarz (https://www.youtube.com/channel/UCYq-iAOSZBvoUxvfzwKIZWA)
Uploaded: 12.06.2018		Filesize: NA		Ext: webm
Selected format(s): 335 - 1920x1080 (1080p60 HDR)+251 - audio only (medium)
VBR: 6734		FPS: 60			Codec: vp9.2
ABR: 132		SR: NA			Codec: opus

File with a single format works just fine
yt-dlp LXb3EKWsInQ -f 335 --print webpage_url --print "%(title)s (%(duration>%H:%M:%S)s)" --print "%(uploader)s (%(channel_url)s)" --print "Uploaded: %(upload_date>%d.%m.%Y)s               Filesize: %(filesize)d          Ext: %(ext)s" --print "Selected format(s): %(format)s" --print "VBR: %(vbr)d              FPS: %(fps)s                    Codec: %(vcodec)s" --print "ABR: %(abr)d                SR: %(asr)d               Codec: %(acodec)s"

https://www.youtube.com/watch?v=LXb3EKWsInQ
COSTA RICA IN 4K 60fps HDR (ULTRA HD) (00:05:14)
Jacob + Katie Schwarz (https://www.youtube.com/channel/UCYq-iAOSZBvoUxvfzwKIZWA)
Uploaded: 12.06.2018		Filesize: 264159085		Ext: webm
Selected format(s): 335 - 1920x1080 (1080p60 HDR)
VBR: 6734		FPS: 60			Codec: vp9.2
ABR: NA		SR: NA			Codec: none

Extracting all filesizes from multiformat file just to show it's there
yt-dlp LXb3EKWsInQ --print webpage_url --print "%(title)s (%(duration>%H:%M:%S)s)" --print "%(uploader)s (%(channel_url)s)" --print "Uploaded: %(upload_date>%d.%m.%Y)s              Filesize: %(filesize)d          Ext: %(ext)s" --print "All filesizes: %(formats.:.filesize)s" --print "Selected format(s): %(format)s" --print "VBR: %(vbr)d              FPS: %(fps)s                    Codec: %(vcodec)s" --print "ABR: %(abr)d          SR: %(asr)d                     Codec: %(acodec)s"

https://www.youtube.com/watch?v=LXb3EKWsInQ
COSTA RICA IN 4K 60fps HDR (ULTRA HD) (00:05:14)
Jacob + Katie Schwarz (https://www.youtube.com/channel/UCYq-iAOSZBvoUxvfzwKIZWA)
Uploaded: 12.06.2018		Filesize: NA		Ext: webm
All filesizes: [1915277, 1976157, 2635585, 5079894, 5199784, 456669599, 954222915, 998602858,<PHONE_NUMBER>, 217319942, 401868566, 526532833, 635852858, 3022967, 5526140, 8375984, 2509541, 2763272, 3182932, 11757449, 17535694, 4355021, 6139821, 6288588, 24268483, 37789471, 8668566, 13251952, 13438679, 25197684, 45945508, 72673441, 15886577, 22099213, 24210391, 45190880, 76313586, 97100288, 127189533, 170935530, 47531179, 60409340, 84831512, 134710778, 171661603, 204737404, 264159085]
Selected format(s): 335 - 1920x1080 (1080p60 HDR)+251 - audio only (medium)
VBR: 6734		FPS: 60			Codec: vp9.2
ABR: 132		SR: NA			Codec: opus

You can do this:
➤ yt-dlp b5xp4xMUhZQ -O "%(requested_formats.0.filesize+requested_formats.1.filesize)d"
955422295

but leave the issue open, since it honestly shouldn't be this hard

Also is it possible to format filesize in a similar manner to formatting time? Or how to convert it to MB/GB?

Not currently possible. Only +/- are currently supported inside the outtmpl, not division. You can approximate by stripping out the last few digits
➤ yt-dlp b5xp4xMUhZQ -O "%(requested_formats.0.filesize.:-6+requested_formats.1.filesize.:-6)dMB"
954MB

but again, let me see if I can add a simpler way to do this

You can do this:
➤ yt-dlp b5xp4xMUhZQ -O "%(requested_formats.0.filesize+requested_formats.1.filesize)d"
955422295

Thanks for help :)
Yeah, this works fine. But it's just flips issue around, meaning it does work with multiple formats, but doesn't work with single format.

I found that using --exec-before-download in combination with echo or printf works much better.
yt-dlp LXb3EKWsInQ --quiet --skip-download --exec-before-download 'size=$(numfmt --to=iec-i --suffix=B --format="%.2f" $(( %(requested_formats.0.filesize+requested_formats.1.filesize|0)d + %(filesize|0)d ))); echo "Size: $size"'

Size: 256,89MiB

Now you can just do -O filesize_approx

how you will get the name of file as well its size when you download playlists? yt-dlp https://www.youtube.com/playlist?list=PLQKyTXVHxaRPE3AkpcDxkTMZuimKXhGmq -O "%(requested_formats.0.filesize+requested_formats.1.filesize)d" > size.txt outputs online the size of the video not its name.

@estatistics how do you wanna format it? eg size-name, sizename, size name, name size, name:size etc

by name size or  name:size, thanks

Sorry, I just saw this message. I am assuming that by name you mean the filename of the output

name:size

-O %(title)s [%(id)s].%(ext)s:%(filesize,filesize_approx)s

name size

-O %(title)s [%(id)s].%(ext)s %(filesize,filesize_approx)s
If you are refering to just the title of the video, use:
-O %(title)s:%(filesize,filesize_approx)s and -O %(title)s %(filesize,filesize_approx)s respectively

--compat-opt manifest-filesize-approx

I wish without the --compat-opt manifest-filesize-approx, NA was printed instead of a totally wrong number.

Yeah, this could be a problem. @pukkandan what do you think?

is there by any chance a magic arg to get the human-readadble...

--print "%(filesize_approx&~)s%(filesize_approx,filesize)#D%(filesize_approx,filesize&B|)s"

I wish without the --compat-opt manifest-filesize-approx, NA was printed instead of a totally wrong number.

Yeah, this could be a problem. @pukkandan what do you think?

Personally I prefer current impl (showing faded and with ~). A lot of people however seems to have this confusion though and so I'm not completely opposed. But the same confusion would exist for bitrate, no? For bitrate, we don't even have a good visual indicator for it being approximate.
The root issue is that we don't have separate fields for tbr_approx etc like we do for filesize. So we take the bandwidth field from manifest and put it in tbr, which isn't actually correct. Sites set bandwidth much higher than the average bitrate so stream can play smoothly.

@pukkandan maybe I should have quoted more of the message I was replying to for context. I wasn't trying to ask if we should revert the manifest filesize approx removal. Let me reframe it:

[when video+audio formats are selected] --print filesize_approx returns size of only the audio stream (251): 911212.
[when filesize_approx isn't available for the video format] I wish ... NA was printed instead of a totally wrong number.

Let me reframe it:

[when video+audio formats are selected] --print filesize_approx returns size of only the audio stream (251): 911212.
[when filesize_approx isn't available for the video format] I wish ... NA was printed instead of a totally wrong number.

ah, I completely misunderstood. That is a bug in this logic:
https://github.com/yt-dlp/yt-dlp/blob/615a84447e8322720be77a0e64298d7f42848693/yt_dlp/YoutubeDL.py#L2331-L2332
Instead of filtering valid values, we should abort if any format lacks value

Also, for that same video, I just noticed that when using -S "height:720", the chosen formats are 247+251 both of which with a deterministic size; yet, -O filesize returns NA, and -O filesize_approx returns some value. Shouldn't it be the other way around?

This is intentional though. The filesize of the merged file need not be exactly the sum of audio+video sizes. So we only populate filesize_approx


max-tokens and GPU memory problems

I have two GPUs with memory of 12000Mib.
Run the translation model with fairseq, set max-tokens to 512 to run on a single GPU, and use about 10,000 memory; Set max-tokens to 256, run on two GPUs, and use about 10,000 video memories respectively.
Questions：

Why is the GPU's memory utilization rate not 10,000 in total but 10,000 respectively?
If it is 10000 respectively, isn't my video memory of 24000Mib equivalent to only 12000Mib?
Then I won't be able to run a model with more than 12000Mib of video memory?
For my model problem,How can fairseq be used on two GPUs to become a total of 10,000?

The following picture shows the script of the model running on a single  GPU：

Then a single GPU usage：

The following picture shows the script of the model running on a two GPUs：

Then two GPUs usage：

What's your environment?

fairseq Version : Latest
PyTorch Version: 1.6.0
OS: Linux
How you installed fairseq: Download on github
Python version: 3.6.12
GPU models and configuration:  2 TITAN Xp
CUDA/cuDNN version:  cuda-10.2

@myleott
cc: @shruti-bh
Thanks in advance!

Hi Surnal,
When you train a model on multiple GPUs, fairseq works in data parallel regime, that is model parameters and optimizer states are replicated across each GPU, while the data is split. This makes effective batch size bigger, and this increases the utilization of each GPU.
I don't quite understand why would you want to decrease utilization of GPUs, this will slow down training while keeping the overhead of data parallelism, but you can do it by decreasing the model size and decreasing the batch size (reduce --max-tokens )
If you want to keep GPUs underutilized, it would be much better to train on 1 GPU instead of 2 and keep 1 fully utilized and the other one free, since it will remove the cost of data parallelism and any communication and synchronization overheads.

Hi Surnal,
When you train a model on multiple GPUs, fairseq works in data parallel regime, that is model parameters and optimizer states are replicated across each GPU, while the data is split. This makes effective batch size bigger, and this increases the utilization of each GPU.
I don't quite understand why would you want to decrease utilization of GPUs, this will slow down training while keeping the overhead of data parallelism, but you can do it by decreasing the model size and decreasing the batch size (reduce --max-tokens )
If you want to keep GPUs underutilized, it would be much better to train on 1 GPU instead of 2 and keep 1 fully utilized and the other one free, since it will remove the cost of data parallelism and any communication and synchronization overheads.

Thank you very much for your answer, because my current model is too large, which leads to insufficient video memory, so I want to use 24000Mib video memory, that is, add two GPU video memories. But it can't be realized now.


Cassandra Store Catalog Function|| BE-Engine doesn't get kill if invalid keyspace is provided

Version Information

Software
Version(s)

BusinessEvents
6.3.0

OS Type?

OS Version?
NA

What is the expected behavior?
BE-Engine should not get started and shut down automatically.
What is the actual behavior?
If invalid keyspace name is provided, exception is thrown for the same but BE-Engine also gets started. Since keyspace is not valid, BE-Engine also should not get started. Please find attached logs for the same.
Please provide log files.
Cass_store_func_Test_invalid_keyspace.log

We are already throwing runtime exception from 'Connect' catalog function.
2023 Apr 03 17:41:45:540 GMT +05 Cass_store_func_Test_invalid_keyspace ERROR [main] - [runtime.service] [inference-class] Got runtime exception while invoking Action com.tibco.cep.runtime.session.impl.RuleFunctionsExecAction@11c25846 Objects <>
java.lang.RuntimeException: Error occurred while connecting to Store with url[<IP_ADDRESS>:9042]

Expected behaviour can be achieved by handling this exception in BE code. To shutdown engine - user can make use of Engine.shutdown catalog function.

As discussed with Rashmi if exception is thrown for any invalid combination then Engine will remain start, however if customer needs they can use Engine.Shutdown() catalog function inside try catch block in their codebase.


How does the library handle error messages?

Hey guys. The library has been working amazing so far. I just couldn't find a reference in the code or documentation that tells how would the code handle error frames that comes from CAN. Could you direct me to a resource and specific cod in the library that handles these errors?
Cheers

You can identify ERR message by the flag in ID
https://github.com/autowp/arduino-mcp2515/blob/master/can.h#L20


Modify inference engine

It seems that engine module under inference package is not well developed. This PR tries to

Replace assert with raise and additional information. Actually assert can be turned off sometimes.
Remove useless import
Fix a bug about return_tuple
modify the function name of _check_quantize_setting to _init_quantization_setting

More documentation about this module is needed.

ping @RezaYazdaniAminabadi

Hi @chunyang-wen
Thanks for going through the inference engine-API and make these nice modifications. I will put more information on the parameters description.
Thanks,
Reza

So should we merge this PR first and then you add further improvement or I just close it?

So should we merge this PR first and then you add further improvement or I just close it?

No, let me please add them and we can merge it soon. Thanks

So should we merge this PR first and then you add further improvement or I just close it?

No, let me please add them and we can merge it soon. Thanks

Sure, thanks.

Have no idea why the check fails.

@jeffra please merge or open auto merge. Thanks.

We had a CI runner failure earlier due to some new runners being misconfigured. Should be fixed now.


Fixes #5165

Fixes #5165
This updates the function 'parseEnhancedStereo' by fixing the whitespace parsing with regex of enhanced stereochem information in V3000 SD files. Now, the function can handle any amount of extra spaces at the end of a stereochem defining line in such a file. Additionally, there's a new test case and associated files to go along with this fix.

Thanks @gosreya ! I will merge this after the CI builds pass (assuming that they do, of course)

Sounds good, thank you :)


Not able to Deploy.

Describe the bug
It's not deploying my application.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-hic6cp
Steps to reproduce
It's not deploying my application.
Expected behavior
It's not deploying my application.
Screen Recording / Screenshot

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 119150389,
  "usedJSHeapSize": 110605325,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = saiprashanthbanda
Chat ID = b52d36800d6c

Additional context
Error:
An error occured while streaming this message.

Hi @Sai-Prashanth-Banda,
Appreciate the feedback. The team is aware of this issue with deployment which is being tracked here: #1022.


Issues with neeto-editor

@karthiknmenon _a https://www.loom.com/share/c5c10bb701c040e283da12add0278492
cc: @amaldinesh7 @labeebklatif

.neeto-ui-pane__close has position: absolute but it gets overridden by .neeto-ui-btn

@AbhayVAshokan _a Could you please take a loot at this issue?


⚖ Register Validator: InfStones

Network Choice
nemeton-1
Who are you?
We are InfStones, the ultimate web3 infrastructure platform.
For more information regarding our platform and services, please visit our website (https://infstones.com/).
gentx
{"body":{"messages":[{"@type":"/cosmos.staking.v1beta1.MsgCreateValidator","description":{"moniker":"InfStones","identity":"","website":"https://infstones.com/","security_contact":"zhucheng@infstones.com","details":"InfStones. Web3 made easy."},"commission":{"rate":"0.050000000000000000","max_rate":"0.200000000000000000","max_change_rate":"0.010000000000000000"},"min_self_delegation":"1","delegator_address":"okp410lmaxh6mmcjg76cmzr8cc73f80ykcttzs76w4d","validator_address":"okp4valoper10lmaxh6mmcjg76cmzr8cc73f80ykcttz9e28tv","pubkey":{"@type":"/cosmos.crypto.ed25519.PubKey","key":"JZPfXe8ZIhCCJjUqw7r9DI0MXVFsOBt17NqygBlDBaM="},"value":{"denom":"uknow","amount":"10000000000"}}],"memo":"4cbfeff6d88aa4cdb64631a30356a4fb5fb243c6@<IP_ADDRESS>:26656","timeout_height":"0","extension_options":[],"non_critical_extension_options":[]},"auth_info":{"signer_infos":[{"public_key":{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"AlfDVhufjsx21+UkqHR2zSdIUpoH5mkAqP0/N7s56Xi9"},"mode_info":{"single":{"mode":"SIGN_MODE_DIRECT"}},"sequence":"0"}],"fee":{"amount":[],"gas_limit":"200000","payer":"","granter":""},"tip":null},"signatures":["f2+jDHZhqCC4h8t4D4Af8TAvy+Ys/5NCiCD3eDZnFsltZKrlDSjh1B6RHnqzTfG57k8xM0QWjuFY/dXSsr2fuw=="]}

😉 Here is the corresponding PR: https://github.com/okp4/networks/pull/300


test: Change DROP_ALL to install a dummy policy.

DROP_ALL mode can be skipped in the default policy enforcement mode if
there are no imported policies, as in that case the policy will not
drop any packets. Augment the DROP_ALL test to install a policy that
does not apply to the endpoints being tested. With this the policy is
still allowing all traffic by default for the tested endpoints, but
only after it is known that the policy does not apply.
Signed-off-by: Jarno Rajahalme<EMAIL_ADDRESS>

@manalibhutiyani Blocks: #3933

test-me-please

@tgraf : I don't think the original CI test was marked as needs-backport ? We will need to back port both in that case.

@manalibhutiyani That's why I made this a separate PR - thinking this will not be backported either.

Thanks, removing label again


Remove TFLite specific reciprocal pattern

The reciprocal lowering pattern is redundant in TFLite due to the TFLite now including the legalize_tf TF to TF lowerings which itself contains the same lowering.

@smit-hinsu


Why use these codes, especially since they are all pass codes？

https://github.com/nik-dim/pamal/blob/497622ed01f853207c012f74ed50f53839e0933e/src/trainer/multi_forward_ensemble_trainer.py#L131
All functions in the callback.py and base_callback.py files have a 'pass' internally

Hello, most of these functions are indeed not doing anything. However, they can be useful if you want to add some monitoring callbacks, e.g., track the evolution of the model, metrics etc. We simply added callbacks for everything similar to PyTorch Lightning. Hope this helps.


1.0.1

Fixes

minDecimal had no effect if the number had no decimal part (#2)

New

bundles now have the header comment with @license

Regressions

bundle size is now 19% bigger (469 to 559 bytes)

Codecov Report

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

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 75f0f38...95df007. Read the comment docs.