esbuild: Failed to install correctly

Version
botframework-webchat": "^4.14.1 & "botframework-webchat": "^4.15.1" as well
nodejs: '14.16.0', npm: '6.14.11' and also tried with
nodejs: '16.10.0', npm: '7.24.0'
Describe the bug
We are using botframework-webchat's react webchat to extend the webchat functionality and publishing it as npm module and cdn lib.
with nodejs 12 it was working fine, when we upgraded our node js version it started giving below error on esbuild dependency.
16:37:43  npm ERR! throw new Error(`esbuild: Failed to install correctly
16:37:43  npm ERR! ^
16:37:43  npm ERR!
16:37:43  npm ERR! Error: esbuild: Failed to install correctly
16:37:43  npm ERR!
Steps to reproduce
Just install botframework-webchat": "^4.14.1 version with any react project.

@suraj5789, attempting a repro with node v14 and will let you know what I find. For clarity, is it failing when testing or after publishing?

@stevkan thanks for looking into the issue.
I am getting error in my repo installation at local and Jenkins and also my consumer application when they are installing my npm package.
Below are my dependencies and devdependencies.
"dependencies": {
"botframework-webchat": "^4.14.1",
"deepmerge": "^4.2.2",
"react-idle-timer": "^4.6.4",
"sonarqube-scanner": "^2.8.1",
"styled-components": "^5.3.3"
},
"devDependencies": {
"@types/chai": "^4.2.18",
"@types/mocha": "^8.2.2",
"@babel/cli": "^7.13.0",
"@babel/preset-react": "^7.12.13",
"mocha": "^9.0.0",
"mochawesome": "^6.2.2",
"nyc": "^15.1.0",
"@testing-library/jest-dom": "^5.11.9",
"@testing-library/react": "^11.2.5",
"@testing-library/user-event": "^12.7.1",
"react": "^17.0.1",
"react-dom": "^17.0.1",
"react-scripts": "4.0.2",
"ts-node": "^10.0.0",
"typescript": "^4.3.2",
"webpack": "4.44.2",
"webpack-cli": "^4.8.0",
"rewire": "^5.0.0",
"sinon": "^12.0.1",
"@material-ui/core": "^4.12.3"
},
"peerDependencies": {
"react": "^17.0.1",
"react-dom": "^17.0.1"
},

@suraj5789, I was able to dig down into this and I do not believe the issue is specific to your React setup nor is it necessarily tied to the node version. It looks like when Web Chat 4.14.1 was released, the 'markdown-it-attrs-es5' dependency was bumped from 1.2.0 to 2.0.0. This bump represented, among other things, a switch from using the 'babel-loader' and 'webpack' dependencies to 'babel-cli' and 'esbuild' for building the library.
Peeking at the readme file for 'markdown-it-attrs-es5', I noticed this comment:

When you update markdown-it-attrs, re-run npm install markdown-it-attrs-es5 to get the latest package transpiled.

Because using 4.14.1+ represents an update to the 'markdown-it-attrs' library, I went into '\node_modules\markdown-it-attrs-es5' and ran npm install. I then went back to my React project's root folder and attempted both npm install<EMAIL_ADDRESS>-S as well as @latest for 4.15.1. Both installed without issue and I was able to run my React project.
Can you try the same and see if it works for you? It's a bit messy, but if it works for you then that would be a good step forward. Then, I can see about getting the library to update as part of the install process.

@stevkan,
Thanks for your quick help. Tried your solution now my apps work on my local. But in Jenkins it is getting failed.
Jenkins Log :
17:44:50  + node -v
17:44:50  v16.14.0
17:44:50  + npm -v
17:44:50  8.3.1
17:44:50  + npm install
17:45:29  npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated
17:45:29  npm WARN deprecated<EMAIL_ADDRESS>Package moved to @redux-devtools/extension.
17:45:41  npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
17:45:45  npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
17:45:55  npm WARN deprecated<EMAIL_ADDRESS>This SVGO version is no longer supported. Upgrade to v2.x.x.
17:45:55  npm WARN deprecated<EMAIL_ADDRESS>You can now upgrade to @mui/system. See the guide: https://mui.com/guides/migration-v4/
17:46:05  npm WARN deprecated<EMAIL_ADDRESS>You can now upgrade to @mui/styles. See the guide: https://mui.com/guides/migration-v4/
17:46:44  npm WARN deprecated<EMAIL_ADDRESS>You can now upgrade to @mui/material. See the guide: https://mui.com/guides/migration-v4/
17:46:59  npm ERR! code 1
17:46:59  npm ERR! path /home/jenkins/workspace/I_uhc-webchat_Testing-Webchat-SG/node_modules/web-speech-cognitive-services/node_modules/p-defer-es5
17:46:59  npm ERR! command failed
17:46:59  npm ERR! command sh -c npm run postinstall:commonjs && npm run postinstall:esm
17:46:59  npm ERR! ><EMAIL_ADDRESS>postinstall:commonjs
17:46:59  npm ERR! > node ./scripts/esbuild.mjs p-defer ./src/index.js cjs ./lib/index.bundled.js && babel ./lib/index.bundled.js --config-file ./babel.config.commonjs.json --out-file ./lib/index.js
17:46:59  npm ERR! /home/jenkins/workspace/I_uhc-webchat_Testing-Webchat-SG/node_modules/web-speech-cognitive-services/node_modules/esbuild/bin/esbuild:2
17:46:59  npm ERR! throw new Error(`esbuild: Failed to install correctly
17:46:59  npm ERR! ^
17:46:59  npm ERR!
17:46:59  npm ERR! Error: esbuild: Failed to install correctly
17:46:59  npm ERR!
17:46:59  npm ERR! Make sure you don't have "ignore-scripts" set to true. You can check this with
17:46:59  npm ERR! "npm config get ignore-scripts". If that returns true you can reset it back to
17:46:59  npm ERR! false using "npm config set ignore-scripts false" and then reinstall esbuild.
17:46:59  npm ERR!
17:46:59  npm ERR! If you're using npm v7, make sure your package-lock.json file contains either
17:46:59  npm ERR! "lockfileVersion": 1 or the code "hasInstallScript": true. If it doesn't have
17:46:59  npm ERR! either of those, then it is likely the case that a known bug in npm v7 has
17:46:59  npm ERR! corrupted your package-lock.json file. Regenerating your package-lock.json file
17:46:59  npm ERR! should fix this issue.
17:46:59  npm ERR!
17:46:59  npm ERR!     at Object. (/home/jenkins/workspace/I_uhc-webchat_Testing-Webchat-SG/node_modules/web-speech-cognitive-services/node_modules/esbuild/bin/esbuild:2:7)
17:46:59  npm ERR!     at Module._compile (node:internal/modules/cjs/loader:1103:14)
17:46:59  npm ERR!     at Object.Module._extensions..js (node:internal/modules/cjs/loader:1155:10)
17:46:59  npm ERR!     at Module.load (node:internal/modules/cjs/loader:981:32)
17:46:59  npm ERR!     at Function.Module._load (node:internal/modules/cjs/loader:822:12)
17:46:59  npm ERR!     at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)
17:46:59  npm ERR!     at node:internal/main/run_main_module:17:47
17:46:59
17:46:59  npm ERR! A complete log of this run can be found in:
17:46:59  npm ERR!     /home/jenkins/.npm/_logs/2022-03-10T17_44_50_277Z-debug-0.log
My Package.json :
{
"name": "webchat-rc",
"version": "1.0.0",
"private": false,
"description": "Microsoft Botframework webchat",
"main": "./lib/index.js",
"dependencies": {
"-": "0.0.1",
"@material-ui/core": "^4.12.3",
"botframework-webchat": "^4.14.1",
"deepmerge": "^4.2.2",
"esbuild": "^0.14.25",
"markdown-it-attrs-es5": "^2.0.2",
"react": "^17.0.1",
"react-dom": "^17.0.1",
"react-idle-timer": "^4.6.4",
"react-scripts": "^5.0.0",
"sonarqube-scanner": "^2.8.1"
},
"devDependencies": {
"@babel/cli": "^7.13.0",
"@babel/preset-react": "^7.12.13",
"@testing-library/jest-dom": "^5.11.9",
"@testing-library/react": "^11.2.5",
"@testing-library/user-event": "^12.7.1",
"@types/chai": "^4.2.18",
"@types/mocha": "^8.2.2",
"mocha": "^9.0.0",
"mochawesome": "^7.1.2",
"nyc": "^15.1.0",
"rewire": "^6.0.0",
"sinon": "^12.0.1",
"ts-node": "^10.0.0",
"typescript": "^4.3.2",
"webpack": "^5.70.0",
"webpack-cli": "^4.8.0"
},
"peerDependencies": {
"react": "^17.0.1",
"react-dom": "^17.0.1"
},
"scripts": {
"start": "react-scripts start",
"build:js": "webpack --config webpack.config.js ",
"test": "react-scripts test --watchAll=false",
"eject": "react-scripts eject",
"build:npm": "rm -rf npm-package && mkdir -p npm-package/lib &&  babel src -d npm-package/lib --copy-files",
"coverage": "react-scripts test --coverage --watchAll=false"
},
"eslintConfig": {
"extends": [
"react-app",
"react-app/jest"
]
},
"browserslist": {
"production": [
">0.2%",
"not dead",
"not op_mini all"
],
"development": [
"last 1 chrome version",
"last 1 firefox version",
"last 1 safari version"
]
},
"author": "UHC",
"license": "UNLICENSED",
"publishConfig": {
"registry": ""
},
"jest": {
"collectCoverageFrom": [
"src/**/*.{js,jsx}",
"!src/index.js"
]
}
}
Any suggestions what mistake I am doing here.

I consulted with our primary Web Chat developer and he stated that Web Chat 4.15 should have the below packages. The previously included versions had a bug in them when used with certain versions of Node/npm. However, the upgraded packages correct this.

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Looking in the last entry in your Jenkins log, it appears you are still using<EMAIL_ADDRESS>which is only in v4.14 of Web Chat. Can you upgrade to v4.15 and try again?

Closing due to inactivity. Please feel free to reopen if the issue persists or you have new details to share.

I've encountered a similar issue which isn't resolved with v4.15.
Worth noting that the peer dependency web-speech-cognitive-services isn't actually using p-defer-es5 2.0.1, but rather 2.0.0 (at least from what I can see)
└─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
└─┬<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>And per the error log above (same as the one I get) it seems the error occurs with this package:
17:46:59 npm ERR! at Object. (/home/jenkins/workspace/I_uhc-webchat_Testing-Webchat-SG/node_modules/web-speech-cognitive-services/node_modules/esbuild/bin/esbuild:2:7)
Interestingly, my local build works fine. The issue only occurs when I try to build this as a docker image (which is probably related to why the original poster had it fail in Jenkins)
FROM node:16.15.0 as base
ENV CI=true
WORKDIR /var/www
COPY package*.json ./
RUN npm ci
COPY . .
RUN npm t -- --coverage --watchAll=false --reporters=jest-junit
EXPOSE 3000
ENV PATH /var/www/node_modules/.bin:$PATH
ENTRYPOINT [ "./init.sh" ]
Could this be reopened? It seems the dependency for the web-speech module should be updated to 2.0.1 as well.


Update: selectors syntax

Hi @lesky thanks for the PR. Haven't had a chance to look through it in depth yet but had an initial questions. Is it necessary to prefix the selector method with the class? i.e. can't you just say #selector(method) rather than #selector(foo.method)?
Other than that, I just need to verify that you got em all :)

You know that you always misspell my name? @lesky != @lesyk

facepalm
Sorry about that!
(and sorry other person I've been mentioning)

about #selector(method) rather than #selector(foo.method), Xcode this that more specific is better, but for you I can do your way @jbrophy1🎱

Sorry for the delay, everything looks good here


Is “gpu_hist” only supported by GPU-enabled machine?

Hi folks,
I just trained a model (XGBRegressor) using “gpu_hist” tree method on a GPU-enabled machine.
I used pickle.dump(…) to store the model.
I want to load and apply this model on a machine without any GPU. I am wondering if it is feasible?
What I am doing currently is to use pickle.load(…) to load the model. For example, XGBreg_load = pickle.load(…)
The "xgb.plot_importance(...)" still works and is reproducible.
However, when I use XGBreg.predict(…) to make the prediction, the results are not correct.
I am wondering if it is possible to make it correct on a machine without any GPU?

Can you post a small script to reproduce the issue?

Can you post a small script to reproduce the issue?

Sure.  @hcho3
On the GPU-enabled machine (where I got and saved the model):
XGBreg = pickle.load(open("./mode_XGB.dat","rb"))
[18:45:07] /workspace/src/learner.cc:362: Parameter 'predictor' has been recovered from the saved model. It will be set to 'gpu_predictor' for prediction. To override the predictor behavior, explicitly set 'predictor' parameter as follows:

Python package: bst.set_param('predictor', [new value])
R package:      xgb.parameters(bst) <- list(predictor = [new value])
JVM packages:   bst.setParam("predictor", [new value])

XGBreg.predict(df[vari_ls])
array([286.47766, 286.54575, 287.41147, ..., 274.29263, 273.1712 ,
275.02548], dtype=float32)
type(XGBreg)
xgboost.sklearn.XGBRegressor
One the machine without GPU
XGBreg = pickle.load(open("./mode_XGB.dat","rb"))
[11:46:23] /workspace/src/learner.cc:362: Parameter 'predictor' has been recovered from the saved model. It will be set to 'gpu_predictor' for prediction. To override the predictor behavior, explicitly set 'predictor' parameter as follows:

Python package: bst.set_param('predictor', [new value])
R package:      xgb.parameters(bst) <- list(predictor = [new value])
JVM packages:   bst.setParam("predictor", [new value])

XGBreg.predict(df[vari_ls])
array([0.5, 0.5, 0.5, ..., 0.5, 0.5, 0.5], dtype=float32)
type(XGBreg)
xgboost.sklearn.XGBRegressor
I am wondering how to modify the bst.set_param('predictor', [new value] for "gpu_hist" on the machine without GPU?

@zzheng93 Did you try using save_model() / load_model() instead?

@hcho3 Yes. I tried.
XGBreg = XGBRegressor(tree_method = 'gpu_hist',seed=66)
XGBreg.load_model("...l")
XGBreg.predict(df[vari_ls])
Same code, but different results on different machines.
I am wondering how to deal with the following hints?
[20:22:16] /workspace/src/learner.cc:362: Parameter 'predictor' has been recovered from the saved model. It will be set to 'gpu_predictor' for prediction. To override the predictor behavior, explicitly set 'predictor' parameter as follows:

Python package: bst.set_param('predictor', [new value])
R package:      xgb.parameters(bst) <- list(predictor = [new value])
JVM packages:   bst.setParam("predictor", [new value])

I got the following error
AttributeError: 'XGBRegressor' object has no attribute 'set_param'

This works for me on an XGBRegressor:
model.set_params(tree_method='hist')

Could you try:
model.set_params(tree_method='hist')

Hi,
I tried with CPU. Still gives a wrong result.

@zzheng93 I think #4454 may have address this issue. Can you try with latest code?

@zzheng93 I think #4454 may have address this issue. Can you try with latest code?

I believe it's fixed in #4516.  Feel free to reopen if the problem persists.


Add support for GET /api/recommend/sr/srnames

Partial implementation:  Models.Subreddits.Recommend

Resolved by https://github.com/sirkris/Reddit.NET/pull/55.


Zen Browser

How about taking a look at this new Firefox fork?
https://www.zen-browser.app
https://github.com/zen-browser

Interesting. Seems like it really blew up within the past few weeks. I do wonder how they can say Floorp has no sandboxing though, as sandboxing is very integral to Firefox these days.
It would be nice to see how these forks would look on Windows 2000/XP, But in Superfox I don't think I will deviate much from the main Firefox feature set just yet.


random line in Gulf of Mexico

just thought this looks sus

That line is this ferry route. We render ferries at low zoom because it's generally appropriate for really long-distance ferries like the Alaska Marine Highway.
I agree that this particular ferry looks dumb on the map because it isn't a public passenger ferry, which is what we would normally want to render. That's something we should tackle on the OpenMapTiles side to either differentiate them or remove these categories.

Blocked by openmaptiles/openmaptiles#1673


[Bug] Modmenu transparency

Modmenu supports transparency (the current netherite plus mod is not transparent)

not sure if you care, but I happened to notice it was transparent on the website/curse page but not in modmenu

Fixed

Fixed


Prevent word splitting

This PR prevents word splitting, e.g. when the repository is stored in a directory named /hello world/project.

Thanks! :+1:


feat: add debugComponent utility function

Describe your changes
See https://github.com/BearStudio/start-ui-web/discussions/398 for details
Screenshots

Checklist

[x] I performed a self review of my code
[x] I ensured that everything is written in English
[x] I tested the feature or fix on my local environment
[x] I ran the yarn storybook command and everything is working
[x] I created a PR or an issue on the documentation repository : https://github.com/BearStudio/start-ui-web-docs/issues/41

Storybook preview is empty on my side.


Log render duration

Allows the developer to see the rendering duration ^^

Thanks for the repo and hope you find this useful!

Nice, just need to add tests and docs to README.md and I'll merge.

@aexmachina not quite sure how to add tests to this feature, can you help me? :)

@aexmachina i just added some tests to check the number of times a component get rendered. Hope that the tests are valid for you ^^
Thanks!

Cool, thanks. That's sufficient. Would you mind also adding an entry to README as well?

Docs added :dancer:

Awesomez. Thanks!

Published v4.1.0

Thanks you for the repo!!


Add texlive (LaTeX compiler)

I'm using this docker image pretty frequently for compiling LaTeX documents in a container.
IDK if you want to add this to your repo but here it is.

Ah cool I love latex! I'll take a look later :)
On Saturday, February 6, 2016, Christian<EMAIL_ADDRESS>wrote:

I'm using this docker image pretty frequently for compiling LaTeX
documents in a container.
IDK if you want to add this to your repo but here it is.
You can view, comment on, or merge this pull request online at:
https://github.com/jfrazelle/dockerfiles/pull/111
Commit Summary

Add texlive (LaTeX compiler)

File Changes

A texlive/Dockerfile
https://github.com/jfrazelle/dockerfiles/pull/111/files#diff-0 (16)

Patch Links:

https://github.com/jfrazelle/dockerfiles/pull/111.patch
https://github.com/jfrazelle/dockerfiles/pull/111.diff

—
Reply to this email directly or view it on GitHub
https://github.com/jfrazelle/dockerfiles/pull/111.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3


Data Access Problem

Thank you for sharing your work.
I attempted to access the datasets, but received the following message:
"Unfortunately, your request has been denied. The following reason was provided: Full home/personal address including house number and street name is required."
However, when I try to enter my full address, I receive this error:
"Value for editable custom_field 'Address' is too long. Approximate max length: 75 characters per line, 1 line(s) (depending on character size). See https://developers.hellosign.com/docs/templates/walkthrough/#using-long-text-in-custom-fields."
Could you provide guidance on how to resolve this issue?

Hi @seohyun8825, the license server unfortunately cannot hold addresses longer than 75 characters. Can you please try a shortened version of the address that still includes all components? If that's not possible, please reopen this issue and we'll follow up to get you access.


Outdated, Incomplete and Incompatible

For anyone wanting to use this, just beware, it is a starting point, but for anything but trivial API usage it doesn't work.
It uses XML, some of the API calls don't  work with XML but require JSON.
Make sure you use Jackson and not JAXB for Jersey (This isn't the default), otherwise there are JSON formatting issues which make it incompatible.
Many of the types are wrong, they need to be Long or Double, not Int, or you get overflow issues or fields not populating.
Like I said, it is a starting point, but doesn't work out of the box.

"Make sure you use Jackson and not JAXB for Jersey" could you clarify this for me. As I'm not sure quite how to do that, and am running into issues related to the differences between Jackson and the default JAXB implementation.

I can't remember sorry!. I ended up tweaking the library a bit. I've attached the source here. I can't remember if there was any config needed elsewhere. We're not using this anymore.
You can probably do a diff to see what has changed and make any modifications.
Or you may be better off just starting a clean implementation and base it off their javascript docs.
com.agilecrm.zip


Can't install biome on Windows using aqua

Describe the bug
Can't install biome with aqua backend on Windows
mise ERROR failed to install<EMAIL_ADDRESS>mise ERROR unsupported env: windows/amd64
mise ERROR Run with --verbose or MISE_VERBOSE=1 for more information

While the releases in biome repo don't include the amd64 arch, aqua's registry file seems to use replacements: amd64: x64 to fix it.
To Reproduce
Run mise use aqua:biomejs/biome in Windows.
Expected behavior
Biome should be installed correctly.
mise doctor output
version: 2024.12.13 windows-x64 (c970664 2024-12-17)
shims_on_path: no

build_info:
  Target: x86_64-pc-windows-msvc
  Features: DEFAULT, NATIVE_TLS, OPENSSL
  Built: Tue, 17 Dec 2024 22:12:22 +0000
  Rust Version: rustc 1.83.0 (90b35a623 2024-11-26)
  Profile: release

shell:
  pwsh
  PowerShell 7.4.6

dirs:
  cache: ~\AppData\Local\Temp\mise
  config: ~\.config\mise
  data: ~\AppData\Local\mise
  shims: ~\AppData\Local\mise\shims
  state: ~\.local\state\mise

config_files:
  ~\usados\.python-version
  ~\usados\mise.toml

backends:
  aqua
  asdf
  cargo
  core
  gem
  go
  npm
  pipx
  spm
  ubi
  vfox

plugins:

toolset:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>(missing)
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
path:
  ~\AppData\Local\mise\installs\python\3.13.1
  ~\AppData\Local\mise\installs\uv\0.5.10
  ~\AppData\Local\mise\installs\ruff\0.8.3
  ~\AppData\Local\mise\installs\pipx-basedpyright\1.23.1\bin
  ~\AppData\Local\mise\installs\pipx-djlint\1.36.3\bin
  ~\AppData\Local\mise\installs\markdownlint-cli2\0.16.0
  ~\AppData\Local\mise\installs\taplo\0.9.3\bin
  ~\AppData\Local\mise\installs\node\23.4.0
  ~\scoop\apps\mise\current\mise\bin
  ~\scoop\apps\pwsh\current
  C:\Software\Python3\
  C:\Windows\system32
  C:\Windows
  C:\Windows\System32\Wbem
  C:\Windows\System32\WindowsPowerShell\v1.0\
  C:\Windows\System32\OpenSSH\
  C:\Program Files\Wolfram Research\WolframScript\
  C:\Software\MiKTeX\miktex\bin\x64\
  C:\Software\MATLAB\bin
  C:\Program Files\dotnet\
  C:\Program Files (x86)\Windows Kits\8.1\Windows Performance Toolkit\
  C:\Software\MEGA

  C:\MSDEV\BIN
  ~\scoop\apps\vscodium\current\bin
  ~\scoop\shims
  ~\AppData\Local\Microsoft\WindowsApps

env_vars:
  MISE_NPM_BUN=true
  MISE_PIPX_UVX=true
  MISE_SHELL=pwsh

settings:

No problems found

Additional context
mise use aqua:biomejs/biome --debug output
DEBUG ARGS: C:\Users\user\scoop\apps\mise\current\mise\bin\mise.exe use aqua:biomejs/biome --debug
DEBUG EnvResults
DEBUG config: C:\Users\user\usados\mise.toml
DEBUG config: C:\Users\user\usados\.python-version
DEBUG install_some_versions: aqua:biomejs/biome@latest
INFO<EMAIL_ADDRESS>install
Error:
   0: failed to install<EMAIL_ADDRESS>   1: unsupported env: windows/amd64

Location:
   src\backend\aqua.rs:559

Version:
   2024.12.13 windows-x64 (c970664 2024-12-17)

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.

Also, it's trying to install the last nightly version instead of the version aqua uses. I've tested with the aqua CLI tool and it does install the last stable version.

you need to submit a PR to aqua to include windows as a supported env:
https://github.com/aquaproj/aqua-registry/blob/e81bcf24528f64180cefeb2a0a96fa44d85d258a/pkgs/biomejs/biome/registry.yaml#L12-L14


Error when running ConFindr on unusual oganisms

Hi
I find ConFindr with --rmlst works very nicely for me most of the time, however when I try to use it on reads for more uncommon species, it will encounter an error and skip that sample. I have noticed this for Neisseria cinerea, Corynebacterium pseudodiphtheriticum, Listeria grayi, and Klebsiella variicola; all genre where ConFindr works very well to identify to the genus level most of the time.
I am using ConFindr v0.7.4 biopython 1.78 python 3.7
Error encounted was:
Traceback (most recent call last):
File "/home/rhh/miniconda3/envs/py37/lib/python3.7/site-packages/confindr_src/confindr.py", line 1067, in confindr
cross_details=args.cross_details)
File "/home/rhh/miniconda3/envs/py37/lib/python3.7/site-packages/confindr_src/confindr.py", line 629, in find_contamination
returncmd=True)
File "/home/rhh/miniconda3/envs/py37/lib/python3.7/site-packages/confindr_src/wrappers/bbtools.py", line 258, in bbduk_bait
out, err = run_subprocess(cmd)
File "/home/rhh/miniconda3/envs/py37/lib/python3.7/site-packages/confindr_src/wrappers/bbtools.py", line 16, in run_subprocess
raise subprocess.CalledProcessError(x.returncode, cmd=command)
subprocess.CalledProcessError: Command 'bbduk.sh in=220722_fastq/QC11_S11_L001_R1_001.fastq.gz in2=220722_fastq/QC11_S11_L001_R2_001.fastq.gz outm=220722_confindr2/QC11_S11_L001/rmlst_R1.fastq.gz outm2=220722_confindr2/QC11_S11_L001/rmlst_R2.fastq.gz ref=/home/rhh/.confindr_db/rMLST_combined.fasta threads=4' returned non-zero exit status 1.
Any advice?

Hi @cjdgug, could you please run that command shown below and paste the output here?
bbduk.sh in=220722_fastq/QC11_S11_L001_R1_001.fastq.gz in2=220722_fastq/QC11_S11_L001_R2_001.fastq.gz outm=220722_confindr2/QC11_S11_L001/rmlst_R1.fastq.gz outm2=220722_confindr2/QC11_S11_L001/rmlst_R2.fastq.gz ref=/home/rhh/.confindr_db/rMLST_combined.fasta threads=4

I still had this error on one of my samples on the larger machine. When I ran the bbduk.sh command that caused the error:
BBDuk version 37.62
Set threads to 24
Initial:
Memory: max=25918m, free=25377m, used=541m
Added 846536239 kmers; time: 	98.320 seconds.
Memory: max=24797m, free=10311m, used=14486m
Input is being processed as paired
Started output streams:	0.010 seconds.
Processing time:   		0.009 seconds.
Input:                  	16 reads 		2040 bases.
Contaminants:           	2 reads (12.50%) 	301 bases (14.75%)
Total Removed:          	2 reads (12.50%) 	301 bases (14.75%)
Result:                 	14 reads (87.50%) 	1739 bases (85.25%)
Time:   			98.350 seconds.
Reads Processed:          16 	0.00k reads/sec
Bases Processed:        2040 	0.00m bases/sec

sure: BBDuk version 37.62 Set threads to 4 Initial: Memory: max=866m, free=842m, used=24m
java.lang.OutOfMemoryError: Java heap space at kmer.AbstractKmerTable.allocLong1D(AbstractKmerTable.java:150) at kmer.HashArray1D.resize(HashArray1D.java:149) at kmer.HashArray.setIfNotPresent(HashArray.java:190) at jgi.BBDukF$LoadThread.addToMap(BBDukF.java:1924) at jgi.BBDukF$LoadThread.addToMap(BBDukF.java:1828) at jgi.BBDukF$LoadThread.run(BBDukF.java:1755)
This program ran out of memory. Try increasing the -Xmx flag and setting prealloc.
when I run the same command with -Xmx8g I get a different error: BBDuk version 37.62 Set threads to 4 Initial: Memory: max=8232m, free=8060m, used=172m
/home/rhh/miniconda3/envs/py37/bin/bbduk.sh: line 324: 3410 Killed java -Djava.library.path=/home/rhh/miniconda3/envs/py37/bbtools/lib/jni/ -ea -Xmx8g -Xms8g -cp /home/rhh/miniconda3/envs/py37/bbtools/lib/current/ jgi.BBDukF in=220722_fastq/QC11_S11_L001_R1_001.fastq.gz in2=220722_fastq/QC11_S11_L001_R2_001.fastq.gz outm=220722_confindr2/QC11_S11_L001/rmlst_R1.fastq.gz outm2=220722_confindr2/QC11_S11_L001/rmlst_R2.fastq.gz ref=/home/rhh/.confindr_db/rMLST_combined.fasta threads=4 -Xmx8g
I was also experiencing the same error on a bigger machine when it was running confindr v0.7.0, but I have now updated that one to v0.7.4 too. The bigger machine does not encounter the error, but it does still list the Genus as "NA" in the report. It also gives me the rMLST and contamination output files which this machine is not doing with the error.
So I guess a work around is not to use it on this machine with hardly any memory.

Indeed, it looks like Java is being killed because using -Xmx8g requests 8GB memory, which is the same as the total memory available on that machine. Because of other processes running in the background, the program is unable to access the full 8GB memory and is being killed. You could try running on that machine with -Xmx6g, for example, which might be enough memory for bbduk.sh to run without errors.

I still had this error on one of my samples on the larger machine, but this was a rubbish sample with no reads. When I ran the bbduk.sh command that caused the error:
BBDuk version 37.62 Set threads to 24 Initial: Memory: max=25918m, free=25377m, used=541m
Added 846536239 kmers; time: 98.320 seconds. Memory: max=24797m, free=10311m, used=14486m
Input is being processed as paired Started output streams: 0.010 seconds. Processing time: 0.009 seconds.
Input: 16 reads 2040 bases. Contaminants: 2 reads (12.50%) 301 bases (14.75%) Total Removed: 2 reads (12.50%) 301 bases (14.75%) Result: 14 reads (87.50%) 1739 bases (85.25%)
Time: 98.350 seconds. Reads Processed: 16 0.00k reads/sec Bases Processed: 2040 0.00m bases/sec

It looks like this command completed successfully. When you run ConFindr on the machine with larger memory, what error is ConFindr giving you?

Thanks
For the rubbish sample, the error is the same as when it runs out of memory: at lines 1067, 629, 258, and 16 with the bbduk returning a non zero exit, except when I run the offending command on its own it seems to work fine. So it is odd that it errors, rather than just giving an NA output. Obv. I shouldn't even need to run it for this sample, so this is NBD. For the others, it is still strange that it will not list the genus level ID in the report, despite giving me a contamination status, 20K+ bases examined, and an rMLST profile output that I can use to give me a species level ID on the website. Again, this is NBD, because I can work around it.

Hi @cjdgug, would you be able to attach a link to one or more of the problematic FASTQ file pairs?

This is not my own data, but I tried some from SRA. This is a Listeria grayi which has the same issue, i.e. gives NA in the report for genus ID, but does identify to the genus and species level with rMLST:
https://trace.ncbi.nlm.nih.gov/Traces/?view=run_browser&acc=SRR11343989&display=metadata

Hi @cjdgug, I'm sorry for the slow response!
I downloaded this genome and processed the FASTQ files using ConFindr v0.7.4. The genus is being reported as ND because 88/1000 shared-hashes of the L. grayi genome with L. monocytogenes are being detected using mash screen, and ConFindr looks for a cutoff of 150 shared-hashes by default (this can be adjusted using the -m/--min_matching_hashes flag). If you run ConFindr with the -k flag, you can keep the intermediate files produced by the pipeline, and inspect the 'screen.tab' file to see the number of shared-hashes being detected.
We changed ConFindr to report contamination even though an ND is being reported for the Genus so that users can individually decide whether to consider the SNVs being reported as contamination with caution. Hopefully this clarifies things!

Closing as a solution was provided.


Added support for specifying cookies with the -b flag

I added some code to accept a key=value pair for cookies using the --cookies flag or the -b shorthand. The cookies can still be added using the longer --header flag way.

I've written some code to implement this feature.

Originally posted by @7047payloads in https://github.com/epi052/feroxbuster/issues/442#issuecomment-1004090323

Hey there, thanks for taking the time to submit this PR! I'm trying to wrap a few things up, but will review it soon (hopefully this week).  Just wanted to let you know that I see it, thanks again!

Hey! I made a couple modifications. I'm keeping the multiple flags config set to true. Each key=value pair would be supplied to individual -b flags. These cookies are coerced together into a single cookie header, joined with semicolons, HTTP/1.1 compliant now.

Thanks for this! I merged your code along with a few minor tweaks and dependency updates. I'm waiting to get another PR or two merged before i actually tag the 2.5.0 release, but it shouldn't be too long, i don't think.
Thanks again!


[JS] API proposal to more efficiently get computed values

The problem
It seems that Yoga API calls are relatively expensive because of WASM overhead. I've noticed that getting all computed values of a tree takes almost as long as the initial Yoga layout calculation. Correct me if I'm wrong, but I believe this is because of the barrier between JS and WASM. To get all computed values, 13 API calls must be made for each node: getComputedPadding/getComputedMargin/getComputedBorder must all be called 4 times for each edge. getComputedLayout must also be called.
My use case
Every time Yoga executes, I must get all computed values and transfer them to my custom layout system.
Benchmarks
I did a few benchmarks by setting up a tree with N nodes, and then getting the computed values of each node. Environment: Windows 10, Chrome 120.0.6099.200, Yoga 2.0.1. The benchmarks in Firefox have lower times, but the ratio between Yoga execution and getting the computed results is the same, if not worse.
100 nodes:

Yoga execution: 1.1ms
Getting all computed values (13 API calls): 1.3ms
Calling getComputedLayout() only: 0.6ms

625 nodes:

Yoga execution: 4.8ms
Getting all computed values (13 API calls): 4.4ms
Calling getComputedLayout() only: 2ms

2,500 nodes:

Yoga execution: 18ms
Getting all computed values (13 API calls): 14ms
Calling getComputedLayout() only: 6.5ms

API proposal
I'm proposing a general getComputedValues() method that returns an object that includes all computed values for all edges so that the computed values of a node can be retrieved with only 1 API call. To make the method more versatile, there could be parameters that specify which values and edges to return. Perhaps a 'recursive' option could be added as well so that the computed values of an entire subtree could be returned with 1 API call.

This seems like a sensible proposal to me. Crossing the WASM boundary is indeed expensive. And in my testing, batching calls helps significantly (esp. in Firefox, which was already faster than Chrome). I have some performance experiments with WASM builds of Taffy and Yoga on this branch https://github.com/DioxusLabs/taffy/pull/394
It's also the case that Chrome's WASM implementation is just generally not very optimised compared to the one in Firefox. So if you're testing in Chrome then there is probably room for optimisation in the WASM-implementation itself.

Wow, that’s a lot of overhead.
One more layer we have in here is “embind”. https://emscripten.org/docs/porting/connecting_cpp_and_javascript/embind.html
I’ve been keen on removing it for some unrelated reasons (https://github.com/facebook/yoga/issues/1507), but a solution for both of these might be to move the bindings to Emscripten direct function calls. https://emscripten.org/docs/porting/connecting_cpp_and_javascript/Interacting-with-code.html#interacting-with-code-direct-function-calls
These supposedly lower call/bridging overhead, and the Yoga public API being C should have nice mappings for this most of the time.

Would you be able to share the code used for the example, where chunkier accessor improved performance?

If you are using Yoga in a case where your source may mutate over time, incremental Yoga layout can avoid the majority of the traversal, both for calculating, and later reading layout results.
But we are missing a trivial binding for the api for this  https://github.com/facebook/yoga/issues/681

Here's some more info about my benchmark. I updated my original post as well. Environment: Windows 10, Chrome 120.0.6099.200, Yoga 2.0.1.
Benchmark source code:
async function main() {
  const yoga = await loadYoga();
  postProcessingBenchmark(yoga, 2500);
}

function postProcessingBenchmark(yoga, numChildren) {
  const root = createRoot(yoga, numChildren);
  let start;

  start = performance.now();
  root.calculateLayout(10, 10);
  console.log("Yoga computation: ", performance.now() - start);
  start = performance.now();
  postProcessingLoop(root);
  console.log("Getting computed values: ", performance.now() - start);
}

function createRoot(yoga, numChildren) {
  const root = yoga.Node.create();
  root.setWidth(10);
  root.setHeight(10);

  for (let i = 0; i < numChildren; i += 1) {
    const node = yoga.Node.create();
    root.insertChild(node, i);
  }

  return root;
}

function postProcessingLoop(node) {
  const result = {
    paddingLeft: null, paddingRight: null, paddingTop: null, paddingBottom: null,
    marginLeft: null, marginRight: null, marginTop: null, marginBottom: null,
    borderLeft: null, borderRight: null, borderTop: null, borderBottom: null,
    computedLayout: null
  };

  result.paddingLeft = node.getComputedPadding(Edge.Left);
  result.paddingRight = node.getComputedPadding(Edge.Right);
  result.paddingTop = node.getComputedPadding(Edge.Top);
  result.paddingBottom = node.getComputedPadding(Edge.Bottom);

  result.marginLeft = node.getComputedMargin(Edge.Left);
  result.marginRight = node.getComputedMargin(Edge.Right);
  result.marginTop = node.getComputedMargin(Edge.Top);
  result.marginBottom = node.getComputedMargin(Edge.Bottom);

  result.borderLeft = node.getComputedBorder(Edge.Left);
  result.borderRight = node.getComputedBorder(Edge.Right);
  result.borderTop = node.getComputedBorder(Edge.Top);
  result.borderBottom = node.getComputedBorder(Edge.Bottom);

  result.computedLayout = node.getComputedLayout();

  for (let i = 0; i < node.getChildCount(); i += 1)
    postProcessingLoop(node.getChild(i));
}

I read the 'embind' documentation you posted Nick.
From the performance section at the bottom of the page:

The call overhead for simple functions has been measured at about 200 ns. While there is room for further optimisation, so far its performance in real-world applications has proved to be more than acceptable.

This makes perfect sense with what we're seeing. 200ns * 2500 nodes * 13 API calls = 6.5ms. I didn't account for the calls to getChildCount() and getChild() as well while iterating through nodes, so that adds even more time.


Typo in interface Docker

Method networs should be networks.

@llorllale I see, thanks, I'll keep it in mind for the next PR. I won't accept it as a valid Issue because it is a cosmetic matter :D

@llorllale these puzzles are still not solved: #41, #42.

@llorllale the puzzle #41 is still not solved; solved: #38, #42.

@llorllale the puzzle #87 is still not solved; solved: #38, #41, #42, #79.

@llorllale the puzzle #87 is still not solved; solved: #10, #38, #41, #42, #79.


revert transform_keys change

Closes https://github.com/oauth-xx/oauth-ruby/issues/271 by reverting the transform_keys change
omniauth ultimately defines options as a Hashie::Mash not a Hash:
https://github.com/omniauth/omniauth/blob/master/lib/omniauth/strategy.rb#L547
https://github.com/omniauth/omniauth/blob/master/lib/omniauth/key_store.rb#L5
While Hashie::Mash does implement transform_keys, it does not behave the same way as the Hash implementation does:
Hashie::Mash.new(one: "1", two: "2").transform_keys(&:to_sym)
{"one"=>"1", "two"=>"2"}

Hashie::Mash.new(one: "1", two: "2").to_hash.transform_keys(&:to_sym)
=> {:one=>"1", :two=>"2"}

🤔 An alternative could be to rewrite the existing line to:
@options = @@default_options.merge(options.to_hash.transform_keys(&:to_sym))

Never mind this idea 😬
Ah! Another option is to add `.with_indifferent_access` when `@@default_options` is created:
https://github.com/oauth-xx/oauth-ruby/blob/62d3f3e52daa565a04e508df915456a21a7faf37/lib/oauth/consumer.rb#L33-L73
irb(main):001:0> hh = {one: '1', two: '2'}
=> {:one=>"1", :two=>"2"}
irb(main):002:0> hm = Hashie::Mash.new(one: "one", two: "two")
=> {"one"=>"1", "two"=>"2"}
irb(main):003:0> mm = hh.merge(hm)
=> {:one=>"1", :two=>"2", "one"=>"one", "two"=>"two"}
irb(main):004:0> hh = {one: '1', two: '2'}.with_indifferent_access
=> {"one"=>"1", "two"=>"2"}
irb(main):005:0> mm = hh.merge(hm)
=> {"one"=>"one", "two"=>"two"}
irb(main):006:0> mm[:one]
=> "one"

I don't expect the removal of Hashie from omniauth to be possible, based on how purposeful they were in switching to it:
https://github.com/omniauth/omniauth/wiki/OmniAuth-1.0#changes-to-the-auth-hash

Closing in favor of https://github.com/oauth-xx/oauth-ruby/pull/273
Sorry for the noise

In the OAuth2 gem we actually decided to solve this by using the rash_alt gem, which is a subclass of Hashie::Mash, and it is working great there.  I think perhaps we could do the same thing here.

maintainers went with https://github.com/oauth-xx/oauth-ruby/pull/276


korean translation started

Korean translation started for fvtt

Thank you, but the system doesn't host other languages: for that you'll want to make a module. Please join us at https://discord.gg/Dv8YkbHZqM so we can help get you started.


[BUG] Handle dangling partitioned parameters

Detect dangling partitioned parameters of a module, probably at the end of deepspeed.initialize().
                    # TODO Delay error checking for dangling partitioned parameters to post module init
                    # raise RuntimeError(f"not all parameters for {module_to_apply_fn_to.__class__.__name__}, "
                    #                    f"were zero params, is it possible that the parameters were "
                    #                    f"overwritten after they were initialized? "
                    #                    f"params: {[p for p in module_to_apply_fn_to.parameters(recurse=False)]} ")

I met this error too.  In my case, the reason may originate from the recursive instantiate of the whole model. Some piece of codes are here:
with deepspeed.zero.Init():
        model = instantiate(cfg.model)

where instantiate is defined as :
def instantiate(cfg):
    """
    Recursively instantiate objects defined in dictionaries by
    "_target_" and arguments.

    Args:
        cfg: a dict-like object with "_target_" that defines the caller, and
            other keys that define the arguments

    Returns:
        object instantiated by cfg
    """
    from omegaconf import ListConfig, DictConfig, OmegaConf

    if isinstance(cfg, ListConfig):
        lst = [instantiate(x) for x in cfg]
        return ListConfig(lst, flags={"allow_objects": True})
    if isinstance(cfg, list):
        # Specialize for list, because many classes take
        # list[objects] as arguments, such as ResNet, DatasetMapper
        return [instantiate(x) for x in cfg]

    # If input is a DictConfig backed by dataclasses (i.e. omegaconf's structured config),
    # instantiate it to the actual dataclass.
    if isinstance(cfg, DictConfig) and dataclasses.is_dataclass(cfg._metadata.object_type):
        return OmegaConf.to_object(cfg)

    if isinstance(cfg, abc.Mapping) and "_target_" in cfg:
        # conceptually equivalent to hydra.utils.instantiate(cfg) with _convert_=all,
        # but faster: https://github.com/facebookresearch/hydra/issues/1200
        cfg = {k: instantiate(v) for k, v in cfg.items()}
        cls = cfg.pop("_target_")
        cls = instantiate(cls)

        if isinstance(cls, str):
            cls_name = cls
            cls = locate(cls_name)
            assert cls is not None, cls_name
        else:
            try:
                cls_name = cls.__module__ + "." + cls.__qualname__
            except Exception:
                # target could be anything, so the above could fail
                cls_name = str(cls)
        assert callable(cls), f"_target_ {cls} does not define a callable object"
        try:
            return cls(**cfg)
        except TypeError:
            logger = logging.getLogger(__name__)
            logger.error(f"Error when instantiating {cls_name}!")
            raise
    return cfg  # return as-is if don't know what to do


Unsupported operating system: centos

Currently only Ubuntu and Arch are supported, if you want to commit changes to support CentOS feel free, but as I only use Arch and Ubuntu I won't be adding this support myself.

Also the only thing that changes the support is a few packages that need to be installed, the issue arises because these distros use different package managers. I've tried to install as many packages with snap (even though I despise it) because it is distro agnostic once it's installed.

Closing as works as intended


reth db list SyncStage keys should be strings

Describe the feature
Currently, we type StageId = Vec<u8>
This has the unfortunate effect that the keys when doing reth db list SyncStage are printed as vec u8 array, which is not helpful.

Solutions:

Explore converting StageId to String
Explore converting T::Key to a nicely displayed value on the list method

Additional context
No response

I can take this one.


run_dien.py test reports "localhost/dnn/bias0/N10tensorflow3VarE does not exist"

NV A100 is used, try test on docker nvcr.io/nvidia/tensorflow:21.10-tf2-py3( or you can just install tensorflow==2.8.0)

install DeepCTR

cd DeepCTR/examples && python run_dien.py
Log see below:
File "run_dien.py", line 57, in
model = DIEN(feature_columns, behavior_feature_list,
File "/usr/local/lib/python3.8/dist-packages/deepctr-0.9.0-py3.8.egg/deepctr/models/sequence/dien.py", line 194, in DIEN
File "/usr/local/lib/python3.8/dist-packages/deepctr-0.9.0-py3.8.egg/deepctr/models/sequence/dien.py", line 76, in interest_evolution
File "/usr/local/lib/python3.8/dist-packages/deepctr-0.9.0-py3.8.egg/deepctr/models/sequence/dien.py", line 40, in auxiliary_loss
File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/keras/engine/base_layer_v1.py", line 784, in call
outputs = call_fn(cast_inputs, *args, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/tensorflow/python/autograph/impl/api.py", line 695, in wrapper
raise e.ag_error_metadata.to_exception(e)
tensorflow.python.framework.errors_impl.FailedPreconditionError: in user code:
/usr/local/lib/python3.8/dist-packages/deepctr-0.9.0-py3.8.egg/deepctr/layers/core.py:190 call  *
fc = self.activation_layers[i](fc, training=training)
/usr/local/lib/python3.8/dist-packages/keras/engine/base_layer_v1.py:732 call  **
base_layer_utils.create_keras_history(inputs)
/usr/local/lib/python3.8/dist-packages/keras/engine/base_layer_utils.py:175 create_keras_history
_, created_layers = _create_keras_history_helper(tensors, set(), [])
/usr/local/lib/python3.8/dist-packages/keras/engine/base_layer_utils.py:251 _create_keras_history_helper
constants[i] = backend.function([], op_input)([])
/usr/local/lib/python3.8/dist-packages/keras/backend.py:4031 call
fetched = self._callable_fn(*array_vals,
/usr/local/lib/python3.8/dist-packages/tensorflow/python/client/session.py:1478 call
ret = tf_session.TF_SessionRunCallable(self._session._session,
FailedPreconditionError: 2 root error(s) found.
(0) Failed precondition: Could not find variable dnn/bias0. This could mean that the variable has been deleted. In TF1, it can also mean the variable is uninitialized. Debug info: container=localhost, status=Not found: Resource localhost/dnn/bias0/N10tensorflow3VarE does not exist.
[[{{node dnn/BiasAdd/ReadVariableOp}}]]
[[dnn/BiasAdd/ReadVariableOp/_9]]
(1) Failed precondition: Could not find variable dnn/bias0. This could mean that the variable has been deleted. In TF1, it can also mean the variable is uninitialized. Debug info: container=localhost, status=Not found: Resource localhost/dnn/bias0/N10tensorflow3VarE does not exist.
[[{{node dnn/BiasAdd/ReadVariableOp}}]]
0 successful operations.
0 derived errors ignored.

I can run it when remove python from python.keras

same problem:
FailedPreconditionError: 2 root error(s) found.
(0) FAILED_PRECONDITION: Could not find variable attention_sequence_pooling_layer/local_activation_unit/dnn_1/bias0. This could mean that the variable has been deleted. In TF1, it can also mean the variable is uninitialized. Debug info: container=localhost, status error message=Resource localhost/attention_sequence_pooling_layer/local_activation_unit/dnn_1/bias0/N10tensorflow3VarE does not exist.
[[{{node attention_sequence_pooling_layer/local_activation_unit/dnn_1/BiasAdd/ReadVariableOp}}]]
[[attention_sequence_pooling_layer/local_activation_unit/dnn_1/BiasAdd/ReadVariableOp/_9]]
(1) FAILED_PRECONDITION: Could not find variable attention_sequence_pooling_layer/local_activation_unit/dnn_1/bias0. This could mean that the variable has been deleted. In TF1, it can also mean the variable is uninitialized. Debug info: container=localhost, status error message=Resource localhost/attention_sequence_pooling_layer/local_activation_unit/dnn_1/bias0/N10tensorflow3VarE does not exist.
[[{{node attention_sequence_pooling_layer/local_activation_unit/dnn_1/BiasAdd/ReadVariableOp}}]]
0 successful operations.
0 derived errors ignored.
Where to modify python.keras? in DIEN.py?

same problem: FailedPreconditionError: 2 root error(s) found. (0) FAILED_PRECONDITION: Could not find variable attention_sequence_pooling_layer/local_activation_unit/dnn_1/bias0. This could mean that the variable has been deleted. In TF1, it can also mean the variable is uninitialized. Debug info: container=localhost, status error message=Resource localhost/attention_sequence_pooling_layer/local_activation_unit/dnn_1/bias0/N10tensorflow3VarE does not exist. [[{{node attention_sequence_pooling_layer/local_activation_unit/dnn_1/BiasAdd/ReadVariableOp}}]] [[attention_sequence_pooling_layer/local_activation_unit/dnn_1/BiasAdd/ReadVariableOp/_9]] (1) FAILED_PRECONDITION: Could not find variable attention_sequence_pooling_layer/local_activation_unit/dnn_1/bias0. This could mean that the variable has been deleted. In TF1, it can also mean the variable is uninitialized. Debug info: container=localhost, status error message=Resource localhost/attention_sequence_pooling_layer/local_activation_unit/dnn_1/bias0/N10tensorflow3VarE does not exist. [[{{node attention_sequence_pooling_layer/local_activation_unit/dnn_1/BiasAdd/ReadVariableOp}}]] 0 successful operations. 0 derived errors ignored.
Where to modify python.keras? in DIEN.py?

change all tensorflow.python.keras to tensorflow.keras in deepctr, env tensorflow=2.13 python = 3.10


Object activity changes

Based on #556 I would like to fetch the activities and along with the changes.
Something along the lines of
$object = Model::findOrFail(1);
$object->activities->changes;

Should return activities + changes
Is is possible?

The changes are an appended model attribute.
https://github.com/spatie/laravel-activitylog/blob/808a3e5c119d31d1115ff7831efa866aa3fe9692/src/Models/Activity.php#L61-L64
So you have to access them per activity and not on the collection. But you can use any collection method and merge/reduce them, pluck single entries or whatever you need.
foreach($model->activities as $activity) {
    $activity->changes; // do what you want to do with the changes
}


Use RollUp to generate JS Bundles

As was done on the https://github.com/Polymer/polymer-bundler/tree/javascript-bundles branch, create a PR to synthesize a bundle document and feed into RollUp to get JS module bundle output.

This happened.  It's in master now.


Text Input enhancements

Describe your changes here.
Closes # https://github.com/github/primer/issues/501
Screenshots
Some additions
Error condition

Warning condition

Trailing Icon

Merge checklist

[x] Added/updated tests
[ ] Added/updated documentation
[x] Tested in Chrome
[x] Tested in Firefox
[x] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

Yep! Definitely looks like the composite component is our way forward seeing all the other component implementations too. My only qualm is that it gets verbose when composed at higher levels. If its a pattern we'll follow then our users should probably get used to it.
However, in this case I'm only fixing up current issues with TextInput. Do you think we want to change this api for beta?

What do you think of having a composable API for leading and trailing visuals similar to action list?

<InputGroup>
  <InputGroup.LeadingVisual><SearchIcon /></InputGroup.LeadingVisual>
  <TextInput placeholder="Search" />
</InputGroup>

What's the benefit of using that API here?

Yep! Definitely looks like the composite component is our way forward seeing all the other component implementations too. My only qualm is that it gets verbose when composed at higher levels.

I agree with @pksjce here, the additional verbosity doesn't seem to give us any benefits here.
I don't think composite components should not be our go-to API choice. They are useful when we're composing application components within primer components with. In TextInput, we tightly control "how" the Icon renders (size, color, spacing, etc) so <TextInput leadingVisual={SearchIcon} makes a lot of sense to me.


owners: Add Radim and Quique as owners to some sub directories

What this PR does / why we need it:
Add Radim and Quique as approvers to select directories
Release notes:

NONE

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
952

Relevant Lines:
1850

💛  - Coveralls

@qinqon you all will need to become members of the kubernetes-sig org first. You'll need two sponsors from separate companies to do this. I can sponsor you, but you'll need one more. Follow the directions here, https://github.com/kubernetes/community/blob/master/community-membership.md#member

@qinqon you all will need to become members of the kubernetes-sig org first. You'll need two sponsors from separate companies to do this. I can sponsor you, but you'll need one more. Follow the directions here, https://github.com/kubernetes/community/blob/master/community-membership.md#member

Done https://github.com/kubernetes/org/issues/3740

@qinqon can you update this PR to just have your name added to the approvers since Radim isn't in the org yet, that way we can at least get you in now.

/lgtm

/ok-to-test


Set up Jira widget through API

Hey guys,
I've been unsuccessfully trying to add a Jira widget into my dashboard through hygieia api. I figured out how add build, code analysis and bitbucket widgets so far but Jira I didn't.
Following the data I've been using:
  "featureEntries": [   	{     	"description": "<projectName>",         "options": {             "teamName": "<teamName>",             "projectName": <projectName>",             "projectId": <projectId>         },          "toolName": "Jira" 	}   ],
Does anyone know how to do it? Am I missing some field/information?

@caiohcl Please share what error you are getting? Also, This is a sample of the "featureEntries"
    "featureEntries": [
        {
            "description": "<projectName>",
            "options" : {
                "featureTool" : "Jira",
                "teamName" : "<teamName>",
                "teamId" : "<teamId>",
                "projectName" : "<projectName>",
                "projectId" : "<projectId>",
                "showStatus" : {
                    "kanban" : true,
                    "scrum" : false
                },
                "estimateMetricType" : "storypoints",
                "sprintType" : "scrumkanban",
                "listType" : "epics"
            },
            "toolName": "Jira"
        }
    ]
}```

Here are the potential values for the below properties

```"estimateMetricType" : "storypoints" or  "count" or "hours"
"listType" : "epics" or "issues"
"sprintType" : "scrum" or "kanban" or "scrumkanban"

Hi @subodhbattina , thanks for the replay.
Actually, Hygieia doesn't pop up any error message. Instead, the widget isn't set up when I call the API. It keeps like this:

The weird thing is that I've been successfully set up the others widget... I've got this issue only with Jira Widget....

Hi @caiohcl Sure, Np. Just to get some clarification, were you able to use the "featureEntries" format I shared earlier, are you still seeing this behavior with that format? Please share the exact message you have used on the call to the API. Thanks !

Hi @subodhbattina
Sure. I've been using the following code:
  "featureEntries": [
  	{
    	"description": "<ProjectName>",
        "options": {
            "teamName": "<ProjectTeam>",
            "featureTool": "Jira",
            "projectName": "<ProjectName>",
            "projectId": "<ProjectId>",
            "teamId": "<TeamId>",
            "showStatus" : {
            	"kanban" : true,
                 "scrum" : false
            },
            "estimateMetricType" : "storypoints",
            "sprintType" : "scrumkanban",
            "listType" : "epics"
        },
        "pushed": false,
        "toolName": "Jira"
	}
  ],

@caiohcl:
Lets do a few check here to see if the widget is created in the dashboard:

Pull up your hygieia dashbaord:
In the browser developer console: Look for the URL as below:

https://<host_name>/api/dashboard/<dashboard_id>

Make a note of the ID in the above url, go to your MongoDB dashbaord, and run the below query on the "dashboards" collection:

Sample Query: {_id : ObjectId("<dashboard_id>")}

Please check if you have widgets such as this in the document that pulls up:

"widgets" : [
{
"_id" :  ObjectId("<dashboard_id>")
"name" : "feature",
"componentId" : ObjectId(<component_id value>),
"options" : {
"id" : "feature0",
"featureTool" : "Jira",
"teamName" : "",
"teamId" : "<team_id>",
"projectName" : "<project_name>",
"projectId" : "<project_id>",
"showStatus" : {
"kanban" : true or false,
"scrum" : true or false
},
"estimateMetricType" : "storypoints",
"sprintType" : "scrumkanban",
"listType" : "epics"
}
}
]

Please make a note of the "projectId" and the "teamId" from the document above from MongoDB.

And then, in your mongo db, in the "feature" collection, see what data you have for that teamId and projectId

Sample Query: { "sProjectID": "", $and: [ { "sTeamID": "" } ] }

@caiohcl : If you are not seeing the data in MongoDB, please share the API logs from when added the Jira widget. We wont to see if there were any error or exceptions at all in the api when you tried to add the widget. Thanks

@caiohcl: Following up to check if you have any updates on this at all? Thanks

Hi @subodhbattina... I'm out of office until Friday. I'll get back to you when I get the information. Thanks for your help.

@caiohcl: Sounds Good. Thanks !

Hi @subodhbattina
I checked API and JIRA logs, and there's no exception or error message. Also, there's no data regarding JIRA widget into the database.
{
    "_id": "5ba515b04cedfd00074d1559",
    "_class": "com.capitalone.dashboard.model.Dashboard",
    "template": "Testing",
    "title": "New Dashboard Through Jenkins",
    "widgets": [
        {
            "_id": "5ba515b04cedfd00074d155b",
            "name": "build",
            "componentId": "5ba515b04cedfd00074d1558",
            "options": {
                "buildDurationThreshold": 3,
                "id": "build0",
                "consecutiveFailureThreshold": 5
            }
        },
        {
            "_id": "5ba515b04cedfd00074d155c",
            "name": "codeanalysis",
            "componentId": "5ba515b04cedfd00074d1558",
            "options": {
                "testJobNames": [
                    ""
                ],
                "id": "codeanalysis0"
            }
        },
        {
            "_id": "5ba515b04cedfd00074d155d",
            "name": "repo",
            "componentId": "5ba515b04cedfd00074d1558",
            "options": {
                "password": "<pwd>",
                "name": "repo",
                "id": "repo0",
                "scm": {
                    "name": "Bitbucket",
                    "value": "Bitbucket"
                },
                "userID": "<user>",
                "branch": "<branch>",
                "url": "<bitbucket url>"
            }
        }
    ],
    "owners": [
        {
            "username": "<user>",
            "authType": "STANDARD"
        }
    ],
    "type": "Team",
    "application": {
        "name": "New Dashboard Through Jenkins",
        "components": [
            {
                "$ref": "components",
                "$id": "5ba515b04cedfd00074d1558",
                "$db": ""
            }
        ]
    },
    "validServiceName": false,
    "validAppName": false,
    "remoteCreated": false,
    "scoreEnabled": false,
    "scoreDisplay": "HEADER",
    "activeWidgets": []
}

and the only message that appears in api log file is
`2018-09-21 16:23:44,045 INFO  org.mongodb.driver.connection - Opened connection [connectionId{localValue:31, serverValue:102}] to mongodb:27017
2018-09-21 16:25:16,921 INFO  org.mongodb.driver.connection - Closed connection [connectionId{localValue:26, serverValue:87}] to mongodb:27017 because it is past its maximum allowed idle time.
`

@caiohcl: Clearly the Feature Widget has not been created from the database document you have shared. Can you please send me the entire "post" payload you are using to create the widget? You may be missing some thing in there. Here's a sample payload to create the dashboard:
    "codeRepoEntries": [],
    "functionalTestEntries": [],
    "buildEntries": [],
    "libraryScanEntries": [],
    "securityScanEntries": [],
    "staticCodeEntries": [],
    "featureEntries": [
        {
            "description": "<projectName>",
            "options" : {
                "featureTool" : "Jira",
                "teamName" : "<teamName>",
                "teamId" : "<teamId>",
                "projectName" : "<projectName>",
                "projectId" : "<projectId>",
                "showStatus" : {
                    "kanban" : true,
                    "scrum" : false
                },
                "estimateMetricType" : "storypoints",
                "sprintType" : "scrumkanban",
                "listType" : "epics"
            },
            "toolName": "Jira"
        }
    ],
    "deploymentEntries": [],
    "metaData": {
       "template": "<Template>",
       "title": "<Title>",
       "type": "Team",
       "applicationName": "<Application Name>",
       "componentName": "<ComponentName>",
       "businessService": "<ApplicationName>",
       "businessApplication": "<ComponentName>",
       "owner": {
           "username": "<UserName>",
           "authType": "STANDARD or LDAP depending on what you use for the user name"
       }
    }
}```

@caiohcl: Do you have any other questions, or can we go ahead and close this issue, please let me know ? Thanks !

@subodhbattina
Following the entire body used on the API call.
By the way, I've been calling this API: <hygieia_url>/api/dashboard/remoteCreate
  "buildEntries": [
    {
      "description": "<job name>",
      "options": {
         "instanceUrl": "<jenkins url>",
         "jobName": "<job name>",
         "jobUrl": "<job url>"
         },
      "pushed": false,
      "toolName": "Hudson"
    }
  ],
  "staticCodeEntries":[
    {
      "description": "<job name> (<job url>)",
      "options": {
          "jobName": "<job name>",
          "jobUrl": "<job name>",
          "instanceUrl": "<job url>"
      },
      "pushed": false,
      "toolName":"JenkinsCodeQuality"
    }
  ],
  "featureEntries": [
  	{
    	"description": "<project name>",
        "options": {
        	"teamName": "<project team>",
            "featureTool": "Jira",
            "projectName": "<project name>",
            "projectId": "<project id>",
            "teamId": "<team id>",
            "showStatus" : {
            	"kanban" : true,
              "scrum" : false
            },
            "estimateMetricType" : "storypoints",
            "sprintType" : "scrumkanban",
            "listType" : "epics"
        },
        "pushed": false,
        "toolName": "Jira"
	}
  ],
  "codeRepoEntries": [
    {
      "description": "<repo name>",
      "options": {
         "password": "<pwd>",
         "userID": "<user>",
         "url": "<repo url>",
         "branch": "<branch>"
       },
      "pushed": false,
      "toolName": "Bitbucket"
    }
  ],
  "metaData": {
    "applicationName": "<dashboard name>",
    "componentName": "<dashboard name>",
    "owner": {
      "authType": "STANDARD",
      "username": "admin"
    },
    "template": "<template>",
    "title": "<dashboard name>",
    "type": "team"
  }
}'

@caiohcl: Ok, let me run this, and see what might be happening, thanks.

@caiohcl Thanks for bringing this to our notice, we'll work on fixing this and get back to you. Thanks !

@subodhbattina, no problem.
Do you have any idea of how long is this fix gonna take to be deployed on master?
Also, let me know if you need help.

Hey guys.. any update regarding the fix of this issue?  Thanks!!!

thanks for the replay @divashdr ! I'm gonna test it and let you know the result.

Could you contribute that back these changes in a pr?

yes @Sbrenthughes . Thanks

@Sbrenthughes
As discussed, here is the PR created to address this issue. Please review and merge as per. Thanks
https://github.com/Hygieia/Hygieia/pull/2545

@Sbrenthughes
Would you please review and merge the PR? Thanks

@caiohcl My sincere apologies for not being able to get back to this until now. I did make the changes to fix the issue, and have submitted a PR. I am following up on it.
@divashdr Thanks for your interest, really appreciate it. Since it involved a couple more changes, I went ahead and took care of if for now, thanks again !

Hey guys,
Sorry for the late reply. I've been testing the changes proposed by @divashdr, however I'm still not able to remotely create this widget.
I had changed the code, rebuilt API, rebuilt core, rebuilt the container... but I'm getting an error message that states "Jira collector does not support field: teamName".
I used the same call as @divashdr shared with us btw.
Any ideas?

@caiohcl : Bear with us please, working on getting the changes reviewed and merged at this point. Thanks

@caiohcl
To Find the teamid and projectid details
TEAM

PROJECT

Add the above details to the payload request
"featureEntries":[
{
     "toolName":"Jira",
     "description":"EBS: WFT: Engineering and Architecture(ENA)",
     "pushed":false,
     "options":{
        "teamName":"ENA",
        "featureTool":"Jira",
         "projectName" : "EBS: WFT: Engineering and Architecture(ENA)",
         "projectId" : "1542658246090",
         "teamId" : "1542658246085",
         "estimateMetricType":"count",
         "listType":"issues",
		 "sprintType" : "kanban",
          "showStatus" : {
                "kanban" : true,
                "scrum" : false
            }
     }
  }

],

Hey guys, I forgot to change a file. I did it, and it's working now.
@divashdr and @subodhbattina, I really appreciate your help guys!!

@caiohcl You are Welcome !
@divashdr and @caiohcl , Thanks for reaching out and your interest, and for your patience.


[k8s-runtime] Migrate graphrbac to MS Graph sdk

Replace soon-to-be-removed azure-graphrbac SDK to MS Graph SDK.

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

k8s-runtime


Resized the background image

closes #172
Now, this is how it works
https://github.com/Prajwal0225/My-Favorite-SuperHero/assets/102857774/daf4cc2a-41e1-42e5-8c9d-a2ee5c4bc0e6

Please resize this image ⤵  not the hero image generated using API.

You have to Resize the background image (bg.jpg) to match the dimensions of the generated superhero image

Please resize this image ⤵ not the hero image generated using API.

You have to Resize the background image (bg.jpg) to match the dimensions of the generated superhero image

Now can you check out, is it okay?

https://prajwal0225.github.io/My-Favorite-SuperHero/index.html

Thank you for your invaluable contributions to our open-source project! 🚀 If you like our project, please consider to start ⭐ the repo.


workflows: Add link to skipped action workaround

Actions that are skipped because of path filtering remain pending as
required on pull requests if they are set as required checks. Since the
required checks branch protection feature does not have a notion of
unnecessary checks a workaround is required.
This workaround is documented but hard to find so I added a note
pointing to it from the section that introduces path filtering for
workflows.
Fixes #16048

Why:
Closes issue link

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

How long do I need to wait for the "Automatically generated comment"? Does it not get generated for draft PRs?

@toonn Ah, that's a great catch. I didn't know the PR has to be "ready for review" for the auto generated comment. I will add that clarity in the PR template.
You can view your changes by clicking the "show environments" and "view deployment" button on the PR feed when the PR is in draft mode.

I think my side of this is fulfilled, waiting for a reaction.

@toonn Thanks for the comment. Could you please leave a comment next time when you convert your draft PR to ready for review? That way I will get notified and get this triaged 💖.
Thank you for opening this PR, I will get this triaged for a review ✨


luci-app-strongswan-ipsec: add package

I'm not an IPSec pro, so if there's anything missing or wrong please point it out and I'll be sure to address it. Should we include the example config in the LuCI package? I'm thinking no, it should be a part of the strongswan package. Thoughts?

The two packages strongswan and libreswan are largely synonymous, with slight behaviour differences which should not affect most users day to day.
I made the comment on the libreswan package effort a while back whether it would do as well to have strongswan also covered there. I don't actively use SS or LS so it's not in my purview to test it (wireguard floats my boat fine).
Something you could check? One GUI to rule them all...
https://github.com/openwrt/luci/pull/5932

What do you think of renaming the package to luci-app-strongswan since it generates uci configurations for both strongswan-ipsec and strongswan-swanctl?

This luci package also needs special treatment as @lvoegl did in #6700
See #6700 for inspiration as to how to mark cryptos as unsafe.

Do we still need this PR? @lvogel has taken this PR and extended and changed it, so that it generates the strongswan config for swanctl backend. The question is now do we still need or want to support via LuCI application the old IPsec strongswan config backend?The old backend is deprecated and we want to use the new one via swanctl.

No, if it's deprecated so vote to close this one.

Sorry I misread earlier. I vote no don't support deprecated features in new software.

@systemcrash Why did you merged that?
This is the old ipsec handling and not the new swanctl handling. I thought we wanted to support the new one?
Your already merged the new swanctl handing https://github.com/openwrt/luci/pull/6700

I did not merge this PR.
The master tree shows that I merged #6700.
This showing as 'merged' has to do with the fact that the first commit from #6700 uses this commit.
Weird huh?

This is spooky. But good to know. Then forget my comment :-)


Example of using this to bind locomotive as a proxy for GSAP ScrollTrigger

Long shot but does anyone have an example of how I would add Locomotive as a proxy for GSAP scrolltrigger with this package (as seen here with vanilla JS) -> https://codepen.io/GreenSock/pen/zYrELYe

As you get the scroll instance when you use useLocomotiveScroll you should be able to do whatever you like. I found this : https://greensock.com/forums/topic/24859-locomotive-scroll-direction-with-scrolltrigger/ I found this, it may possibly help you !

instead of this line,
 scroll.on("scroll", ScrollTrigger.update);
I've used this,
scroll.on('scroll', () => {
        ScrollTrigger.update()
        ScrollTrigger.refresh()
      })

it worked perfectly in my case. even with data-scroll-section s

@willslater How does this work with ScrollTrigger pin function? My content always disappears :(


APIv2: consider Message.IsValidValue

Should we have a way to test a Value for compatibility with a message field, along the lines of ExtensionType.IsValidValue?
This could either be a method on MessageType or Message.

Marking as blocks-v2 since this can't be changed after release.
Is this hypothetical, or did you have a real need for this?
I've seen the use for a semi-related validation function. Something like Message.IsValidFieldDescriptor, which takes in a FieldDescriptor and reports whether it is valid for that message or not.

I was updating a bit of code (previously Go reflection, now protoreflect) that copies a list of named fields from one message to another message of a different type. There's no simple way to catch a field type mismatch; you can compare the descriptors, but that won't catch the (admittedly unusual) case where the descriptors are the same but the Go types aren't.
It's nothing you can't work around, but got me wondering whether there should be an easy way to do this for non-extension fields.

Nice to have, but we both agreed that the use cases didn't seem all that compelling.


how to set the root path for production

on my prod server the app is hosted under "some-folder".
i would like that the build generated index.html will refer src scripts and styles to "some-folder"
<script type="text/javascript" src="some-folder/inline.bundle.js"></script>
<script type="text/javascript" src="some-folde/vendor.bundle.js"></script>
<script type="text/javascript" src="some-folder/main.bundle.js"></script>

Try
ng build -prod --base-href="/some-folder/"

thanks,
as .netGuro maybe you know how to use angular cli with vs2015 ? (question is not relation to this thread)

Closing as answered. Thanks @Meligy !

tiny fix: --prod instead of -prod
ng build --prod --base-href="/some-folder/"


Missing<EMAIL_ADDRESS>launch image

This old bug is still alive and kicking. I just tried this on a clean install of haxe (3.1.3), openfl (2.2.1) and xcode (6.1.1) using a project with no launch images defined in the project.
The XCode project generated by openfl update ios somehow messes up the reference to the<EMAIL_ADDRESS>as it is included in the project but only as a file and not as an actual launch image.

I believe this was either a bug in Xcode (when not using an asset catalog) or a missing entry in the Info.plist, though documentation on what tags were needed eluded my searches across the internet.
I migrated the project template to use an Xcode asset catalog, hopefully this doesn't hurt support on older Xcode releases. It seems to be much happier, and the way it is intended to be done now.
I also fixed one of the recommended changes, the only thing that's still is in the target architecture being hard-coded. I'm not sure that we should allow Xcode to choose it automatically, as that's what the architecture settings in the project files are for.

Here's the relevant commit https://github.com/openfl/lime/commit/af5396119e3d511d2c9713e0aa56f34b332baa64 :smile:

I just reverted the ONLY_TARGET_ARCH change, as it broke the builds (it wouldn't show any "eligible" target devices) and fixed a problem that affected command-line compiling. I think it's good now (if you ignore the "recommended setting" message) but I'm happy to hear feedback on how it's working out

I just tried this and the original warning when the project is generated is indeed gone. The images also seem to be included in the asset catalogs properly, however, I get these warnings once I compile the game:

I'll try and look into this tomorrow, I'm not quite sure how many of the images it picks up (the game icon is still properly displayed).

Try it now, the JSON files should be copying as intended after my last fix


React-router-redux related issues

Dispatching push has no effect.
Should apply routerMiddleware in Reducer
https://github.com/reactjs/react-router-redux/issues/366

Solved in #25 .


add latest dependencies

Codecov Report
Patch and project coverage have no change.

Comparison is base (94fb8e5) 97.79% compared to head (5e0972c) 97.79%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #60   +/-   ##
=======================================
  Coverage   97.79%   97.79%
=======================================
  Files           7        7
  Lines         726      726
  Branches      164      164
=======================================
  Hits          710      710
  Misses         13       13
  Partials        3        3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


DISTMYSQL-440: Orchestrator should use the new source-replica terminology for any higher version than 8.3

https://perconadev.atlassian.net/browse/DISTMYSQL-440
Problem 1:
Orchestrator uses source-replica queries only for 8.4.x versions.
For versions like 9.x it uses old master-slave queries.
Reason:
Query String Provider detects only 8.4.x versions as "new ones".
Solution:
Change "new version" detection method.
Problem 2:
MySQL 9.x removed native authentication plugin.
Now caching_sha2_password is used, and requres either secure SSL
connection or at least RSA-based password exchange during the client
authentication. If replication uses SSL channels, and we move the source
to be replica, we need to enable SSL on it. We do it by executing
'change replication source to source_ssl=1' if the new source has SSL
enabled.
However, if replication does not use SSL it means we used RSA, so after
making the source to be replica, we need to enable RSA-based login on it
by executing 'change replication source to get_source_public_key=1'.
Additionally, updated dbdeployer to be compatible with 9.0 (replication
user does not use native authentication)

https://ps80.cd.percona.com/job/mysql-orchestrator-pipeline/63

https://ps80.cd.percona.com/job/mysql-orchestrator-pipeline/78/


VuetifyTiptap sets internally "font-size: 20pxpx"

Duplicated "px" on load, which does not happen with VuetifyViewer
Steps:
1.
    <VuetifyTiptap
      v-else
      :modelValue="modelValue"
      @update:modelValue="updateModelValue"
    />

load with

sdfsdsdfsd

Edit the text, it will convert to:

sdfsdsdfsd

:tada: This issue has been resolved in version 2.4.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Spinner unstyled: displays in HTML but not visible for the user

Bug Description
Related: https://github.com/google/web-stories-wp/issues/790

Spinner is not visible for the user even though it's displayed in HTML.
Note that another Spinner bug is fixed within #836 , please wait until that's merged or bring the fix over from there when looking into this.
Expected Behaviour

Spinner should be visible for the user, too.
Steps to Reproduce

Create a new Story and click "Save Draft"
Spinner should be visible on the left of the button but it's not.

Screenshots

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria
QA Instructions

Noted in sprint 26 overview sheet


On web use "icon" link type instead of "shortcut icon" link type

While updating some icons in my index.html page I noticed when hovering over the rel  <link rel="shortcut icon" ... > it directs you to Mozilla's information on link types: https://developer.mozilla.org/en-US/docs/Web/HTML/Link_types
If you go down to icon it says this: The shortcut link type is often seen before icon, but this link type is non-conforming, ignored and web authors must not use it anymore.
Is there a reason Flutter still uses shortcut or should Flutter not use that anymore and switch to just using icon by default?

Fixed in #63154


flyway: 7.2.1 -> 7.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/flyway/versions.
meta.description for flyway is: "Evolve your Database Schema easily and reliably across all your instances"
meta.homepage for flyway is: "https://flywaydb.org/"
meta.changelog for flyway is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 7.3.0 with grep in /nix/store/5qzplksdqn415p3m3aajscr1lmsmb2iq-flyway-7.3.0

directory tree listing: https://gist.github.com/3069c6e22a8cbbfe79d6afadb6131b62

du listing: https://gist.github.com/7c725d0dc71d6ad8bb995598cf117e49

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
flyway

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

flyway

Maintainer pings
cc @cmcdragonkai for testing.

Result of nixpkgs-review pr 105644 run on x86_64-darwin 1

1 package built:

flyway


Narrow string allowlists in RestrictedControllerMessenger instances

Motivation
Some controllers have their allowed actions or events types for their RestrictedControllerMessenger set to string. This compromises allowlist security by enabling these controllers to gain unrestricted access to any external action or event.
Explanation

Fixes all instances in core of string being used to define RestrictedControllerMessenger allowlists.

Includes follow-up fixes to https://github.com/MetaMask/core/pull/4021.

Fixes accounts-controller allowlists being included in controller actions and events unions.
Fixes accounts-controller allowlists being exported at package-level.

References

Closes #4019

Changelog
@metamask/approval-controller
Fixed

BREAKING: Narrow ApprovalControllerMessenger generic arguments AllowedActions, AllowedEvents from string to never.

@metamask/keyring-controller
Fixed

BREAKING: Narrow KeyringControllerMessenger generic arguments AllowedActions, AllowedEvents from string to never.

@metamask/network-controller
Fixed

BREAKING: Narrow NetworkControllerMessenger generic arguments AllowedActions, AllowedEvents from string to never.

@metamask/permission-controller
Fixed

BREAKING: Fix SideEffectMessenger so that it's defined with a RestrictedControllerMessenger that has access to PermissionController allowed actions.

The messenger's Actions generic parameter is widened to include the PermissionController actions allowlist.
The messenger's AllowedActions generic parameter is narrowed from string to the PermissionController actions allowlist.

Checklist

[x] I've updated the test suite for new or updated code as appropriate
[x] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[x] I've highlighted breaking changes using the "BREAKING" category above as appropriate

Failing transaction-controller tests fixed by following changes. Merging https://github.com/MetaMask/core/pull/4013/ first should take care of this.
diff --git a/packages/transaction-controller/src/TransactionControllerIntegration.test.ts b/packages/transaction-controller/src/TransactionControllerIntegration.test.ts
index 72c91fa44..23bd723fa 100644
--- a/packages/transaction-controller/src/TransactionControllerIntegration.test.ts
+++ b/packages/transaction-controller/src/TransactionControllerIntegration.test.ts
@@ -114,7 +114,11 @@ const setupController = async (
   const unrestrictedMessenger: UnrestrictedControllerMessenger =
     new ControllerMessenger();
   const networkController = new NetworkController({
-    messenger: unrestrictedMessenger.getRestricted({
+    messenger: unrestrictedMessenger.getRestricted<
+      'NetworkController',
+      never,
+      never
+    >({
       name: 'NetworkController',
     }),
     trackMetaMetricsEvent: () => {
@@ -129,7 +133,11 @@ const setupController = async (
   assert(blockTracker, 'Provider must be available');

   const approvalController = new ApprovalController({
-    messenger: unrestrictedMessenger.getRestricted({
+    messenger: unrestrictedMessenger.getRestricted<
+      'ApprovalController',
+      never,
+      never
+    >({
       name: 'ApprovalController',
     }),
     showApprovalRequest: jest.fn(),

The unrestrictedControllerMessenger pattern newly introduced in the TransactionControllerIntegration tests seems to have exposed a flaw in the recent getRestricted fix (https://github.com/MetaMask/core/pull/4013).
Omitting the allowed{Actions,Events} options in getRestricted doesn't result in their types being inferred as never[]. Instead, these options are typed with the "unrestricted" action/event union.
A temporary fix applied here is to pass in empty arrays into allowed{Actions,Events}: https://github.com/MetaMask/core/pull/4031/commits/38d5e88b77d30f7f73e969c07ee702e1e1be5ca1
But getRestricted and/or RestrictedControllerMessenger will need to be fixed for a fundamental solution that lets us omit allowed{Actions,Events}. I'll create a ticket for this.


'states' method for united states returns several subdivisions that are not states

When I run ISO3166::Country['us'].states I get back a hash with 60 keys, which is 10 more than the 50 states the U.S. is commonly known to have. The issue is that the output includes not just states, but insular areas and U.S. military mail codes as well. I can understand why these would show up in a call to .subdivisions, but I don't think they have a place in .states. For context, I'm trying to use countries to build a select tag that allows a user in the U.S. to select their state. My first thought is that there could be more methods, such as .military_mail_codes and .insular_areas I'd be happy to implement them, unless you have another solution in mind.
Also this is my first time ever trying to do open source, so if I'm doing this wrong please have mercy 😅

@rposborne are you the maintainer for this repo now? i'm seeing above issue as well. ISO's website for the US shows a subdivision category (state, district, etc) that the ISO3166 struct doesn't seem to have. should this be added? just trying to do a filter for US states and found it challenging

Yea that's certainly a bit surprising.
It seems reasonable to track this category.  .states should be 50 for sure, the sixty-nifty United States just doesn't have the same ring to it.
I am more than open to a PR addressing this.  Though I think this would need to classify that a breaking change.

is there any update on this issue?

Nothing new. I took a quick look at the code, and the underlying issue is that states is just an alias of subdivisions https://github.com/countries/countries/blob/835e41c625cc9851f1fbc47bf75c7ce1883849c5/lib/countries/country.rb#L71, so the 60 or so results correspond to the ISO 3166-2 subdivisions of the US.
Separating these two methods to have different behaviours requires us to have some way to differentiate between the 50 states and the 'non-state' subdivisions, in some way that makes sense not just for the US, but for any other country that has more than one type of subdivisions.
At this point, we don't have any information in this gem on what the ISO 3166-2 subdivisions mean for different countries. Eg, for the US it's the 50 states, 1 district and 6 outlying areas, while PT has 18 districts and 2 autonomous regions, and there's probably several other designations for different types of subdivisions.

PR #720 removes the Armed Forces subdivisions that are not part of ISO data. The remaining subdivisions are all in ISO.
The subdivision object is in need of some refactoring to deal with countries that have different types/tiers of subdivisions at the ISO3166-2 level.

Addressed with #758 and Country#subdivisions_of_types(types)


Obtain route by providing string url

Would be nice if it was possible to get an instance of the route from the router by providing it with the string URL. My use case is a chat app where users can potentially post links. When rendering messages I find all the links, replace them with custom link components that on click either:

if it's a link to the external website display "Warning, you're leaving the website... YES/NO"
if it's a link to the page on the current website (basically when window.location.hostname matches the hostname from the link) I'd like to just redirect (ex. route.replace()) the user to that page without website reload.

I suggest exposing function (pathnameToRoute?) on the router instance that either takes pathname (ex. window.location.pathname) and returns Route | null.
I am pretty sure this functionality is already implemented and just needs to be exposed ;)

After digging a little bit, seems like it's already possible with router.session.replace(url.pathname).

After digging a little bit, seems like it's already possible with router.session.replace(url.pathname).

Yep, that's how I'd go about it!


github-actions bot: do not lock and limit issue

Provide a detailed description of the proposed feature
github-actions bot locked as resolved and limited conversation to collaborators
use case:
wanna add sth to the issue
What is the motivation for the feature?
I wanna contribute to a closed issue, now I have to duplicate it
How will the feature be relevant to at least 90% of Homebrew users?
90% of feedback will be in the same issue ticket
What alternatives to the feature have been considered?
spamming

so basically you don't mind if users don't do duplicate checks before posting? :|

@allanlaal for issues/PRs old enough to be locked: correct.


FIX: Test layer does not close Popen

If I run the cratelayer with zope-testrunner==4.9.2, the following errors appear:
Tearing down left over layers:
  Tear down crate.testing.layer.crate /usr/lib/python3.7/subprocess.py:858: ResourceWarning: subprocess 4904 is still running
  ResourceWarning, source=self)
Object allocated at (most recent call last):
  File "/home/work/fund-performance/eggs/crate-0.22.1-py3.7.egg/crate/testing/layer.py", lineno 295
    stdout=subprocess.PIPE)
sys:1: ResourceWarning: unclosed file <_io.BufferedReader name=3>

The pull request fixes the issue.
If this is not the preferred solution, I am happy to adapt the PR.

Sorry, this PR slipped through and we didn't notice it earlier. I incidentally made similar changes https://github.com/crate/crate-python/pull/315/files (release in 0.23.0) - could you test again to see if this resolves your issue?

@mfussenegger Yes, it solves me problem.


fatal error: Adafruit_I2CDevice.h: No such file or directory

Hi,
I tried to follow your instructions, but something wrong happened. it shows "Adafruit_I2CDevice.h: No such file or directory" It has been a long since I have been stuck in. Can you help to tackle it? I'm running the ESPHome version(2021.12.4)

Thanks for reporting this. I can confirm that I can reproduce this issue. Just to make sure the error message is searcheable, here is the output
Compiling .pioenvs/esp-43-fireplace/src/main.cpp.o
Generating LD script .pioenvs/esp-43-fireplace/ld/local.eagle.app.v6.common.ld
Compiling .pioenvs/esp-43-fireplace/lib0cb/ESPAsyncTCP-esphome/AsyncPrinter.cpp.o
Compiling .pioenvs/esp-43-fireplace/lib0cb/ESPAsyncTCP-esphome/ESPAsyncTCP.cpp.o
In file included from src/mlx90614_sensor.h:3:0,
                 from src/main.cpp:46:
.piolibdeps/esp-43-fireplace/Adafruit MLX90614 Library/Adafruit_MLX90614.h:18:32: fatal error: Adafruit_I2CDevice.h: No such file or directory

****************************************************************************
* Looking for Adafruit_I2CDevice.h dependency? Check our library registry!
*
* CLI  > platformio lib search "header:Adafruit_I2CDevice.h"
* Web  > https://platformio.org/lib/search?query=header:Adafruit_I2CDevice.h
*
****************************************************************************

 #include <Adafruit_I2CDevice.h>
                                ^
compilation terminated.
*** [.pioenvs/esp-43-fireplace/src/main.cpp.o] Error 1
======================================================================================================= [FAILED] Took 14.49 seconds ================

It seems as if more libraries need to be defined explicitly for the custom component to compile. On top of that a specific version of BusIO appears to be needed.
Can you please try the following configuration:
esphome:
  ...
  includes:
    - "custom_components/mlx90614_sensor.h"
  libraries:
    - SPI
    - Wire
    - adafruit/Adafruit BusIO @ 1.9.6
    - "Adafruit MLX90614 Library"

Yes, it's working perfectly after adding more libraries. Thanks a lot!

It seems as if more libraries need to be defined explicitly for the custom component to compile. On top of that a specific version of BusIO appears to be needed.
Can you please try the following configuration:
esphome:
  ...
  includes:
    - "custom_components/mlx90614_sensor.h"
  libraries:
    - SPI
    - Wire
    - adafruit/Adafruit BusIO @ 1.9.6
    - "Adafruit MLX90614 Library"

This worked! Thanks.

For users who use platformio:
Your  in platformio.ini file lib_deps  should be something like this :
As @exxamalte mentioned.

lib_deps =
	SPI
	Wire
	adafruit/Adafruit<EMAIL_ADDRESS>	adafruit/Adafruit GFX<EMAIL_ADDRESS>	adafruit/Adafruit<EMAIL_ADDRESS>	Adafruit MLX90614 Library


Exceptions in main thread actions

Currently, if you throw an exception in an ExecuteOnMainThreadAndWait callback, the exception is handled on the main thread (and probably crashes binja). We should wrap main thread callbacks and rethrow their exceptions in the context of the original caller so exceptions can be used.
In the case of an exception in ExecuteOnMainThread without waiting, we should just abort because nobody will handle it.

Will be fixed in 2.4.2876-dev


Hide wikipedia link in TaxonDetails if there's no URL

Describe the bug
There are some edge cases like this one where a taxon can have a wikipedia description but not a url. So we're showing a wikipedia link that the user can't open.
To Reproduce
Steps to reproduce the behavior:

Go to Strangalia acuminata Taxon Details screen via Explore
Tap on Read more on Wikipedia link
Nothing happens

Expected behavior
Right now, we're checking to see if a user can open the link and doing nothing if the wikipedia url doesn't exist. Instead, we should hide the "Read more on Wikipedia" link (or defer to @abhasinat on what the preferred UX is here).
Context (please complete the following information):
OS: iOS
App version: v0.34.0+97

Note, also merits some exploration of why the link for that specific taxon isn't working, b/c the wikipedia page exists.