Fails to compile without SDKROOT env var set

In a fresh clone of this repo on my machine:
RubyGems Environment:
  - RUBYGEMS VERSION: 3.4.17
  - RUBY VERSION: 3.2.2 (2023-03-30 patchlevel 53) [arm64-darwin22]
  - INSTALLATION DIRECTORY: /Users/segiddins/.gem/ruby/3.2.2
  - USER INSTALLATION DIRECTORY: /Users/segiddins/.gem/ruby/3.2.0
  - RUBY EXECUTABLE: /Users/segiddins/.rubies/ruby-3.2.2/bin/ruby
  - GIT EXECUTABLE: /opt/homebrew/bin/git
  - EXECUTABLE DIRECTORY: /Users/segiddins/.gem/ruby/3.2.2/bin
  - SPEC CACHE DIRECTORY: /Users/segiddins/.gem/specs
  - SYSTEM CONFIGURATION DIRECTORY: /Users/segiddins/.rubies/ruby-3.2.2/etc
  - RUBYGEMS PLATFORMS:
     - ruby
     - arm64-darwin-22

`rake compile` output:
cd tmp/arm64-darwin22/oxi-test/3.2.2
/Users/segiddins/.rubies/ruby-3.2.2/bin/ruby -I. ../../../../ext/extconf.rb
checking for clang... yes
checking for clang++... yes
checking for ar... yes
cd -
cd tmp/arm64-darwin22/oxi-test/3.2.2
/usr/bin/make
generating /Users/segiddins/Development/github.com/oxidize-rb/oxi-test/target/release/liboxi_test.dylib (release)
cargo rustc  --manifest-path /Users/segiddins/Development/github.com/oxidize-rb/oxi-test/ext/Cargo.toml --target-dir /Users/segiddins/Development/github.com/oxidize-rb/oxi-test/target --lib --profile release -- -C linker=clang -L native=/Users/segiddins/.rubies/ruby-3.2.2/lib -L<EMAIL_ADDRESS>-L native=/opt/homebrew/opt/readline/lib -L native=/opt/homebrew/opt/libyaml/lib -L native=/opt/homebrew/opt/gdbm/lib -C link-arg=-Wl,-multiply_defined,suppress -l pthread -C link-arg=-Wl,-undefined,dynamic_lookup
   Compiling rb-sys v0.9.71
error: failed to run custom build command for `rb-sys v0.9.71`

Caused by:
  process didn't exit successfully: `/Users/segiddins/Development/github.com/oxidize-rb/oxi-test/target/release/build/rb-sys-f2b7b7be3bee2786/build-script-main` (exit status: 101)
  --- stdout
  cargo:rerun-if-env-changed=RUBY
  cargo:rerun-if-env-changed=RBCONFIG_ruby_version
  cargo:rerun-if-env-changed=RBCONFIG_platform
  cargo:rerun-if-env-changed=RUBY_ROOT
  cargo:rerun-if-env-changed=RUBY_VERSION
  cargo:rerun-if-env-changed=RUBY
  cargo:rerun-if-changed=build/features.rs
  cargo:rerun-if-changed=build/version.rs
  cargo:rerun-if-changed=build/ruby_macros.rs
  cargo:rerun-if-changed=build/main.rs
  cargo:rerun-if-env-changed=RUBY_STATIC
  cargo:rerun-if-env-changed=RBCONFIG_ENABLE_SHARED
  cargo:rerun-if-env-changed=RBCONFIG_rubyhdrdir
  cargo:rerun-if-env-changed=RBCONFIG_rubyarchhdrdir
  cargo:rerun-if-env-changed=RBCONFIG_CPPFLAGS

  --- stderr
  Using bindgen with clang args: ["-I/Users/segiddins/.rubies/ruby-3.2.2/include/ruby-3.2.0", "-I/Users/segiddins/.rubies/ruby-3.2.2/include/ruby-3.2.0/arm64-darwin22", "-fms-extensions", "-fdeclspec", "-O3", "-fno-fast-math", "-ggdb3", "-Wall", "-Wextra", "-Wextra-tokens", "-Wdeprecated-declarations", "-Wdivision-by-zero", "-Wdiv-by-zero", "-Wimplicit-function-declaration", "-Wimplicit-int", "-Wmisleading-indentation", "-Wpointer-arith", "-Wshorten-64-to-32", "-Wwrite-strings", "-Wold-style-definition", "-Wmissing-noreturn", "-Wno-cast-function-type", "-Wno-constant-logical-operand", "-Wno-long-long", "-Wno-missing-field-initializers", "-Wno-overlength-strings", "-Wno-parentheses-equality", "-Wno-self-assign", "-Wno-tautological-compare", "-Wno-unused-parameter", "-Wno-unused-value", "-Wunused-variable", "-Wundef"<EMAIL_ADDRESS>"-I/opt/homebrew/opt/readline/include", "-I/opt/homebrew/opt/libyaml/include", "-I/opt/homebrew/opt/gdbm/include", "-D_XOPEN_SOURCE", "-D_DARWIN_C_SOURCE", "-D_DARWIN_UNLIMITED_SELECT", "-D_REENTRANT"]
  /Users/segiddins/.rubies/ruby-3.2.2/include/ruby-3.2.0/ruby/defines.h:16:10: fatal error: 'stdio.h' file not found
  thread 'main' panicked at 'generate bindings: ClangDiagnostic("/Users/segiddins/.rubies/ruby-3.2.2/include/ruby-3.2.0/ruby/defines.h:16:10: fatal error: 'stdio.h' file not found\n")', /Users/segiddins/.cargo/registry/src/github.com-1ecc6299db9ec823/rb-sys-0.9.71/build/main.rs:53:6
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
make: *** [/Users/segiddins/Development/github.com/oxidize-rb/oxi-test/target/release/liboxi_test.dylib] Error 101
rake aborted!
Command failed with status (2): [/usr/bin/make...]
/Users/segiddins/.gem/ruby/3.2.2/gems/rake-compiler-1.2.4/lib/rake/extensiontask.rb:195:in `block (2 levels) in define_compile_tasks'
/Users/segiddins/.gem/ruby/3.2.2/gems/rake-compiler-1.2.4/lib/rake/extensiontask.rb:194:in `block in define_compile_tasks'
/Users/segiddins/.gem/ruby/3.2.2/gems/rake-13.0.6/exe/rake:27:in `<top (required)>'
Tasks: TOP => compile => compile:arm64-darwin22 => compile:oxi-test:arm64-darwin22 => copy:oxi-test:arm64-darwin22:3.2.2 => tmp/arm64-darwin22/oxi-test/3.2.2/oxi_test.bundle
(See full trace by running task with --trace)

 `SDKROOT=$(xcrun --sdk macosx --show-sdk-path) rake compile` output
cd tmp/arm64-darwin22/oxi-test/3.2.2
/Users/segiddins/.rubies/ruby-3.2.2/bin/ruby -I. ../../../../ext/extconf.rb
checking for clang... yes
checking for clang++... yes
checking for ar... yes
cd -
cd tmp/arm64-darwin22/oxi-test/3.2.2
/usr/bin/make
generating /Users/segiddins/Development/github.com/oxidize-rb/oxi-test/target/release/liboxi_test.dylib (release)
cargo rustc  --manifest-path /Users/segiddins/Development/github.com/oxidize-rb/oxi-test/ext/Cargo.toml --target-dir /Users/segiddins/Development/github.com/oxidize-rb/oxi-test/target --lib --profile release -- -C linker=clang -L native=/Users/segiddins/.rubies/ruby-3.2.2/lib -L<EMAIL_ADDRESS>-L native=/opt/homebrew/opt/readline/lib -L native=/opt/homebrew/opt/libyaml/lib -L native=/opt/homebrew/opt/gdbm/lib -C link-arg=-Wl,-multiply_defined,suppress -l pthread -C link-arg=-Wl,-undefined,dynamic_lookup
   Compiling rb-sys v0.9.71
   Compiling oxi-test v0.1.0 (/Users/segiddins/Development/github.com/oxidize-rb/oxi-test/ext)
    Finished release [optimized] target(s) in 5.42s
cd -
mkdir -p tmp/arm64-darwin22/stage/lib/oxi/test
cp Cargo.lock tmp/arm64-darwin22/stage/Cargo.lock
cp Cargo.toml tmp/arm64-darwin22/stage/Cargo.toml
cp LICENSE.txt tmp/arm64-darwin22/stage/LICENSE.txt
cp README.md tmp/arm64-darwin22/stage/README.md
mkdir -p tmp/arm64-darwin22/stage/ext
cp ext/Cargo.toml tmp/arm64-darwin22/stage/ext/Cargo.toml
cp ext/build.rs tmp/arm64-darwin22/stage/ext/build.rs
cp ext/extconf.rb tmp/arm64-darwin22/stage/ext/extconf.rb
mkdir -p tmp/arm64-darwin22/stage/ext/src
cp ext/src/lib.rs tmp/arm64-darwin22/stage/ext/src/lib.rs
cp lib/oxi/test.rb tmp/arm64-darwin22/stage/lib/oxi/test.rb
cp lib/oxi/test/version.rb tmp/arm64-darwin22/stage/lib/oxi/test/version.rb
/usr/bin/make install sitearchdir=../../../../lib/oxi/test sitelibdir=../../../../lib/oxi/test target_prefix=
generating /Users/segiddins/Development/github.com/oxidize-rb/oxi-test/target/release/liboxi_test.dylib (release)
cargo rustc  --manifest-path /Users/segiddins/Development/github.com/oxidize-rb/oxi-test/ext/Cargo.toml --target-dir /Users/segiddins/Development/github.com/oxidize-rb/oxi-test/target --lib --profile release -- -C linker=clang -L native=/Users/segiddins/.rubies/ruby-3.2.2/lib -L<EMAIL_ADDRESS>-L native=/opt/homebrew/opt/readline/lib -L native=/opt/homebrew/opt/libyaml/lib -L native=/opt/homebrew/opt/gdbm/lib -C link-arg=-Wl,-multiply_defined,suppress -l pthread -C link-arg=-Wl,-undefined,dynamic_lookup
    Finished release [optimized] target(s) in 0.14s
installing oxi_test.bundle to ../../../../lib/oxi/test
/opt/homebrew/bin/ginstall -c -m 0755 oxi_test.bundle ../../../../lib/oxi/test
cp tmp/arm64-darwin22/oxi-test/3.2.2/oxi_test.bundle tmp/arm64-darwin22/stage/lib/oxi/test/oxi_test.bundle

Note that compiling without SDKROOT will continue to work after it has worked once, until rake clean is run, at which time rake compile will revert to failing to find system headers.
This looks to be an issue of -isysroot somehow not being included in the clang invocation, despite it being a default added by the clang frontend for /usr/bin/clang:

Random `clang` invocation:
❯ clang code.c --output libCode.dylib -dynamiclib -v
Apple clang version 14.0.3 (clang-14<IP_ADDRESS>.1)
Target: arm64-apple-darwin22.5.0
Thread model: posix
InstalledDir: /Applications/Xcode-14.3.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin
 "/Applications/Xcode-14.3.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang" -cc1 -triple arm64-apple-macosx13.0.0 -Wundef-prefix=TARGET_OS_ -Wdeprecated-objc-isa-usage -Werror=deprecated-objc-isa-usage -Werror=implicit-function-declaration -emit-obj -mrelax-all --mrelax-relocations -disable-free -clear-ast-before-backend -disable-llvm-verifier -discard-value-names -main-file-name code.c -mrelocation-model pic -pic-level 2 -mframe-pointer=non-leaf -fno-strict-return -ffp-contract=on -fno-rounding-math -funwind-tables=1 -fobjc-msgsend-selector-stubs -target-sdk-version=13.3 -fvisibility-inlines-hidden-static-local-var -target-cpu apple-m1 -target-feature +v8.5a -target-feature +crc -target-feature +lse -target-feature +rdm -target-feature +crypto -target-feature +dotprod -target-feature +fp-armv8 -target-feature +neon -target-feature +fp16fml -target-feature +ras -target-feature +rcpc -target-feature +zcm -target-feature +zcz -target-feature +fullfp16 -target-feature +sm4 -target-feature +sha3 -target-feature +sha2 -target-feature +aes -target-abi darwinpcs -fallow-half-arguments-and-returns -mllvm -treat-scalable-fixed-error-as-warning -debugger-tuning=lldb -target-linker-version 857.1 -v -fcoverage-compilation-dir=/var/folders/p3/950_j_ps7xq47vrzdj1qvg940000gn/T/t -resource-dir /Applications/Xcode-14.3.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/clang/14.0.3 -isysroot /Applications/Xcode-14.3.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk -I/usr/local/include -internal-isystem /Applications/Xcode-14.3.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/local/include -internal-isystem /Applications/Xcode-14.3.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/clang/14.0.3/include -internal-externc-isystem /Applications/Xcode-14.3.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include -internal-externc-isystem /Applications/Xcode-14.3.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include -Wno-reorder-init-list -Wno-implicit-int-float-conversion -Wno-c99-designator -Wno-final-dtor-non-final-class -Wno-extra-semi-stmt -Wno-misleading-indentation -Wno-quoted-include-in-framework-header -Wno-implicit-fallthrough -Wno-enum-enum-conversion -Wno-enum-float-conversion -Wno-elaborated-enum-base -Wno-reserved-identifier -Wno-gnu-folding-constant -fdebug-compilation-dir=/var/folders/p3/950_j_ps7xq47vrzdj1qvg940000gn/T/t -ferror-limit 19 -stack-protector 1 -fstack-check -mdarwin-stkchk-strong-link -fblocks -fencode-extended-block-signature -fregister-global-dtors-with-atexit -fgnuc-version=4.2.1 -no-opaque-pointers -fmax-type-align=16 -fcommon -fcolor-diagnostics -clang-vendor-feature=+disableNonDependentMemberExprInCurrentInstantiation -fno-odr-hash-protocols -clang-vendor-feature=+enableAggressiveVLAFolding -clang-vendor-feature=+revert09abecef7bbf -clang-vendor-feature=+thisNoAlignAttr -clang-vendor-feature=+thisNoNullAttr -mllvm -disable-aligned-alloc-awareness=1 -D__GCC_HAVE_DWARF2_CFI_ASM=1 -o /var/folders/p3/950_j_ps7xq47vrzdj1qvg940000gn/T/code-6a0225.o -x c code.c
clang -cc1 version 14.0.3 (clang-14<IP_ADDRESS>.1) default target arm64-apple-darwin22.5.0
ignoring nonexistent directory "/Applications/Xcode-14.3.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/local/include"
ignoring nonexistent directory "/Applications/Xcode-14.3.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/Library/Frameworks"
#include "..." search starts here:
#include <...> search starts here:
 /usr/local/include
 /Applications/Xcode-14.3.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/clang/14.0.3/include
 /Applications/Xcode-14.3.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include
 /Applications/Xcode-14.3.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include
 /Applications/Xcode-14.3.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/System/Library/Frameworks (framework directory)
End of search list.
 "/Applications/Xcode-14.3.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ld" -demangle -lto_library /Applications/Xcode-14.3.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/libLTO.dylib -no_deduplicate -dynamic -dylib -arch arm64 -platform_version macos 13.0.0 13.3 -syslibroot /Applications/Xcode-14.3.0.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk -o libCode.dylib -L/usr/local/lib /var/folders/p3/950_j_ps7xq47vrzdj1qvg940000gn/T/code-6a0225.o -lSystem /Applications/Xcode-14.3.0.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/clang/14.0.3/lib/darwin/libclang_rt.osx.a

This causes the cargo builder tests in rubygems to fail consistently on contributors machines, as well.


Add video embed support for updates

Related to https://github.com/opencollective/opencollective/issues/4298
Depends on https://github.com/opencollective/opencollective-frontend/pull/6394

Would be great to have a preview for anchor.fm, people will be surprised if it just disappears from the content when they send emails.
Also, we really need unit tests to make sure future regressions on this code don't break the emails.

@Betree : Working on this. I don't think we could get a preview image for anchor.fm as in YouTube. So I am gonna put generic image there. Hope that's adequate for this purpose. 🤔 🐨

@Betree : I've added the anchor preview as well. It looks looks like this. Let me know your thoughts. I wonder if this is the best placeholder to show but I felt this is good for now? 🤔 🐨

Would be great to have a preview for anchor.fm, people will be surprised if it just disappears from the content when they send emails.
Also, we really need unit tests to make sure future regressions on this code don't break the emails.

@Betree : Working on this. I don't think we could get a preview image for anchor.fm as in YouTube. So I am gonna put generic image there. Hope that's adequate for this purpose. thinking koala

Yes, absolutely! Ideally, it should be something that lets you know you can click on it to read the track but for a first version any image just to make sure the content doesn't disappear is fine :wink:

@Betree : I've corrected the remaining issues and added some tests as well. Let me know how it looks and if anything else needs to be done. 😄

@Betree : Also in the final commit; 5d66ec0 I've removed one test which I've added. This isn't a YouTube link or a AnchorFM link and hence this kinda of iframe isn't valid and the API will not add such content at all.
https://github.com/opencollective/opencollective-api/blob/f1648e68812a7d724e9bb7b6a29f744cd73debbc/server/lib/sanitize-html.ts#L90
Hence the test will always fail for such a test I think. 😉

@Betree : Also in the final commit; 5d66ec0 I've removed one test which I've added. This isn't a YouTube link or a AnchorFM link and hence this kinda of iframe isn't valid and the API will not add such content at all.
https://github.com/opencollective/opencollective-api/blob/f1648e68812a7d724e9bb7b6a29f744cd73debbc/server/lib/sanitize-html.ts#L90
Hence the test will always fail for such a test I think. wink

This test was there on purpose, the goal was to make sure iframe from other domains are stripped out. We should add the test back.

Looking great! Small feedback below. Let's then add back the test and we'll be good to go.

Thanks for the explanation, that makes sense. I've added the test back and adjusted the count here,
https://github.com/opencollective/opencollective-api/blob/5b31336033ee042a36a03c3b6e439f1aa37aee95/test/server/lib/notifications.test.ts#L170-L174
so that we don't consider the final test (with www.test.com) as that one is stripped out. Let me know if everything looks good now to merge. 👍🏽


Error when running Laravel Mix as a global package

Laravel Mix Version (npm list --depth=0): v1.7.2
Node Version (node -v): v9.3.0
NPM Version (npm -v): v5.5.1
OS: Windows 10 v10.0.16299.192

Description:
I am attempting to run Laravel Mix as a Global Package with NodeJS. However, I am receiving the following error:
@ production E:\webserver\apache\htdocs\service_framework\resources
cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --no-progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js

cross-env is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @ production: cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --no-progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @ production script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      E:\webserver\nodejs\_logs\2018-01-14T21_30_56_445Z-debug.log

No issues when I install Laravel Mix in the local directory using npm install. Just easier and uses less disk space by having this as a global package.
Steps To Reproduce:

Open Command Prompt.
Run npm install -g laravel-mix.
CD to the directory where Laravel Mix is required.
Run npm link laravel-mix.
Run npm run production.

This morning, I came across this post which unfortunately did not have a satisfying solution. After a day searching, I still don't have the solution, but perhaps a bit more information that might be useful.
I agree with the above given reasons, to which I can add one (don't hate me): I store the projects in a Dropbox folder, and don't have 10GB to store the exact same node_modules in each project... (yes I use git as well 👍)
node.js looks for folders higher in the tree that have a child with the name "node_modules". Furthermore it checks the location of PATH (enviromental variable in windows). So, in theory it should be possible to just move node_modules to any parent folder, and all should still work... but no...
I've seen the above error allot today:
Error: spawn node_modules\webpack\bin\webpack.js ENOENT

which basically means that it cannot find webpack...
What I did (for now) is simple: I created a new package.json in a parent folder of my project. (Project: c:/Dropbox/webserver/projects/laravel/projectname/, the package was created at c:/)
cd /
npm init

Install all packages that are shared among the projects, the most important one is ofcourse laravel-mix
npm install laravel-mix cross-env axios vue vue-i18n vue-loader bootstrap bootstrap-vue

and any other package you like to be shared.
Now copy the folders cross-env, laravel-mix and webpack to the node_modules of your project. Or, I guess a symlink should work as well. Also make sure to copy /.bin/cross-env and /.bin/cross-env.cmd to your local project node_modules.
When running npm run watch from your local project, it will/should work.
Notes:

In my test i actually made the "shared" node_modules in the parent of the project folder, but that should not matter
I guess you can install the packages as global as well (I cleared out my global folder %appdata%/npm/node_modules)
If any of the 3 folders does not exist, it wont work...

My thoughts: somehow the configuration passed to cross-env, laravel-mix and webpack, influence the way node.js searches for those folders. It prevents searching in parents or global (=PATH) folder...

Wow, @matalina's suggestion works for me, thanks!
Here is my webpack.config.js version (copied from laravel-mix@2.1.11):
/**
 * As our first step, we'll pull in the user's webpack.mix.js
 * file. Based on what the user requests in that file,
 * a generic config object will be constructed for us.
 */
let laravelMixPath = process.env.NODE_PATH + '/laravel-mix/';
let mix = require(laravelMixPath + 'src/index');

let ComponentFactory = require(laravelMixPath + 'src/components/ComponentFactory');

new ComponentFactory().installAll();

require('./webpack.mix');

/**
 * Just in case the user needs to hook into this point
 * in the build process, we'll make an announcement.
 */

Mix.dispatch('init', Mix);

/**
 * Now that we know which build tasks are required by the
 * user, we can dynamically create a configuration object
 * for Webpack. And that's all there is to it. Simple!
 */

let WebpackConfig = require(laravelMixPath + 'src/builder/WebpackConfig');

module.exports = new WebpackConfig().build();

Note: in my case NODE_PATH is refer to global packages, see https://nodejs.org/api/modules.html#modules_loading_from_the_global_folders

This works for me:

Install node_modules on test laravel project:
npm install

move node_modules directory to a global path on your computer, save this global path on a global environment variable (run command with administrator previllege):
setx NODE_PATH "your/path"

Close cmd and open again to reload new env var created in last point.

Install cross-env and jquery globally:
npm install -g cross-env jquery

Copy laravel-mix from %NODE_PATH%/laravel-mix to myLaravelProject/node_modules

cd myLaravelProject
xcopy "%NODE_PATH%/laravel-mix" ./node_modules/laravel-mix /e /i /h

Update scripts in your laravel project myLaravelProject/package.json to:

    "scripts": {
        "dev": "npm run development",
        "development": "cross-env NODE_ENV=development %NODE_PATH%/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
        "watch": "npm run development -- --watch",
        "watch-poll": "npm run watch -- --watch-poll",
        "hot": "cross-env NODE_ENV=development %NODE_PATH%/webpack-dev-server/bin/webpack-dev-server.js --inline --hot --config=node_modules/laravel-mix/setup/webpack.config.js",
        "prod": "npm run production",
        "production": "cross-env NODE_ENV=production %NODE_PATH%/webpack/bin/webpack.js --no-progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js"
    },

Hi All,
Thank you for your suggestions.
I have replaced the website name with  due to a confidentiality agreement.
I have managed to compile JavaScript files but the SASS compiler throws the following error:
ERROR in Entry module not found: Error: Can't resolve 'webpack.config.js' in 'E:\webserver\nodejs'
ERROR in multi ./node_modules/laravel-mix/src/builder/mock-entry.js ../apache/htdocs/<website name>/resources/assets/scss/app.scss
Module not found: Error: Can't resolve 'style-loader' in 'E:\webserver\nodejs'
 @ multi ./node_modules/laravel-mix/src/builder/mock-entry.js ../apache/htdocs/<website name>/resources/assets/scss/app.scss

My package.json file:
{
  "private": true,
  "scripts": {
    "dev": "npm run development",
    "development": "node E:/webserver/nodejs/node_modules/cross-env/dist/bin/cross-env.js NODE_ENV=development E:/webserver/nodejs/node_modules/laravel-mix/node_modules/webpack/bin/webpack.js --progress --hide-modules config=webpack.config.js",
    "watch": "npm run development -- --watch",
    "watch-poll": "npm run watch -- --watch-poll",
    "hot": "node E:/webserver/nodejs/node_modules/cross-env/dist/bin/cross-env.js NODE_ENV=development E:/webserver/nodejs/node_modules/laravel-mix/node_modules/webpack-dev-server/bin/webpack-dev-server.js --inline --hot config=webpack.config.js",
    "prod": "npm run production",
    "production": "node E:/webserver/nodejs/node_modules/cross-env/dist/bin/cross-env.js NODE_ENV=production E:/webserver/nodejs/node_modules/laravel-mix/node_modules/webpack/bin/webpack.js --no-progress --hide-modules config=webpack.config.js"
  }
}

My webpack.config.js file:
let laravelMixPath = process.env.NODE_PATH + '/laravel-mix/';
let mix = require(laravelMixPath + 'src/index');
let ComponentFactory = require(laravelMixPath + 'src/components/ComponentFactory');
new ComponentFactory().installAll();
require('./webpack.mix');
Mix.dispatch('init', Mix);
let WebpackConfig = require(laravelMixPath + 'src/builder/WebpackConfig');
module.exports = new WebpackConfig().build();

Any further assistance would be appreciated. Thank you.

Hi All,
Thank you for your suggestions.
I have replaced the website name with <website name> due to a confidentiality agreement.
I have managed to compile JavaScript files but the SASS compiler throws the following error:
This dependency was not found:
* E:\webserver\apache\htdocs\<website name>\resources\assets\scss\app.scss in multi E:/webserver/nodejs/node_modules/laravel-mix/src/builder/mock-entry.js ./resources/assets/scss/app.scss
To install it, you can run: npm install --save E:\webserver\apache\htdocs\<website name>\resources\assets\scss\app.scss
ERROR in Entry module not found: Error: Can't resolve 'webpack.config.js' in 'E:\webserver\nodejs'
ERROR in multi ./node_modules/laravel-mix/src/builder/mock-entry.js ../apache/htdocs/<website name>/resources/assets/scss/app.scss
Module not found: Error: Can't resolve 'style-loader' in 'E:\webserver\nodejs'
 @ multi ./node_modules/laravel-mix/src/builder/mock-entry.js ../apache/htdocs/<website name>/resources/assets/scss/app.scss

My package.json file:
{
  "private": true,
  "scripts": {
    "dev": "npm run development",
    "development": "node E:/webserver/nodejs/node_modules/cross-env/dist/bin/cross-env.js NODE_ENV=development E:/webserver/nodejs/node_modules/laravel-mix/node_modules/webpack/bin/webpack.js --progress --hide-modules config=webpack.config.js",
    "watch": "npm run development -- --watch",
    "watch-poll": "npm run watch -- --watch-poll",
    "hot": "node E:/webserver/nodejs/node_modules/cross-env/dist/bin/cross-env.js NODE_ENV=development E:/webserver/nodejs/node_modules/laravel-mix/node_modules/webpack-dev-server/bin/webpack-dev-server.js --inline --hot config=webpack.config.js",
    "prod": "npm run production",
    "production": "node E:/webserver/nodejs/node_modules/cross-env/dist/bin/cross-env.js NODE_ENV=production E:/webserver/nodejs/node_modules/laravel-mix/node_modules/webpack/bin/webpack.js --no-progress --hide-modules config=webpack.config.js"
  }
}

My webpack.config.js file:
let laravelMixPath = process.env.NODE_PATH + '/laravel-mix/';
let mix = require(laravelMixPath + 'src/index');
let ComponentFactory = require(laravelMixPath + 'src/components/ComponentFactory');
new ComponentFactory().installAll();
require('./webpack.mix');
Mix.dispatch('init', Mix);
let WebpackConfig = require(laravelMixPath + 'src/builder/WebpackConfig');
module.exports = new WebpackConfig().build();

Any further assistance would be appreciated. Thank you.

it show "File is a CommonJS module; it may be converted to an ES6"


error TS2315: Type 'Buffer' is not generic

Environment Information

OS: Mac M3 Sonoma 14.6.1
Node Version: 20.14.0
NPM Version: 10.7.0
confluent-kafka-javascript version: 0.5.2

Steps to Reproduce
Update existing Typescript app with schemaregistry dependency from 0.6.1 to 1.0.0 and compile.
> tsc --outDir ./dist --project ./tsconfig.json

../../node_modules/@confluentinc/schemaregistry/dist/rules/encryption/encrypt-executor.d.ts:41:32 - error TS2315: Type 'Buffer' is not generic.

41     static readonly EMPTY_AAD: Buffer<ArrayBuffer>;
                                  ~~~~~~~~~~~~~~~~~~~

../../node_modules/@confluentinc/schemaregistry/dist/serde/serde.d.ts:8:34 - error TS2315: Type 'Buffer' is not generic.

8 export declare const MAGIC_BYTE: Buffer<ArrayBuffer>;
                                   ~~~~~~~~~~~~~~~~~~~

Found 2 errors in 2 files.

Errors  Files
     1  ../../node_modules/@confluentinc/schemaregistry/dist/rules/encryption/encrypt-executor.d.ts:41
     1  ../../node_modules/@confluentinc/schemaregistry/dist/serde/serde.d.ts:8
npm error Lifecycle script `build` failed with error:
npm error Error: command failed

This can be worked around by enabling skipLibCheck in the tsconfig.json...
  "compilerOptions": {
    "skipLibCheck": true
  }

...but that feels like bypassing a legitimate type issue.
As the error says the built-in Buffer type doesn't support generics and I can't find any definition of a non-built-in Buffer type in the project.
$ grep  -r "Buffer<" .
./dist/rules/encryption/encrypt-executor.d.ts:    static readonly EMPTY_AAD: Buffer<ArrayBuffer>;
./dist/serde/serde.d.ts:export declare const MAGIC_BYTE: Buffer<ArrayBuffer>;

@apeloquin-agilysys the change for Buffer becoming generic looks like it's coming from TypeScript 5.7 and es2024, see https://devblogs.microsoft.com/typescript/announcing-typescript-5-7/#typedarrays-are-now-generic-over-arraybufferlike.
Are you using those?


Add BUILD rule to create a shared library

#108 (https://github.com/tensorflow/tensorflow/issues/108#issuecomment-167847860)
cc @martinwicke

Hmm, that's odd. Just realized this is in the wrong repo. Will fix this weekend.

Hmm, that's odd. Just realized this is in the wrong repo. Will fix this weekend.


Requests have item_ids that match no Item

Summary
Related to the bug found in https://github.com/rubyforgood/diaper/pull/1962
We discovered that there are Request with item_ids that match no Item. This is problematic because we cannot find the corresponding Item that a request was meant for. This issue is meant to be a starting off point to figuring out the next steps. Here are some topics that we should consider:

How can we ensure that we don't lose Item data like this again. (Revisit how we were soft deleting Item records)
How many item_ids specified in requests that match no Item? (Audit & assess the magnitude of the data inconsistency)
What should we do about the requests that have item_id that don't match any Item anymore? (Maybe we can remove them since the Item had been deleted already?)

Here are some examples of ids of Request and Items that fit this description
Request - 1781
Item -  441, 457, 441, 457, 441, 457
Things to Consider
We should be mindful of how we treat data moving forward. Data inconsistency can be really painful to handle for engineers and users.

SO! Some fun app history here.
This problem was anticipated a long time ago and the solution we had was by using partner_key (the base_item type). (cf. https://github.com/rubyforgood/diaper/wiki/Base-Items#integration-with-the-partner-application ) At some point, someone changed the Requests to work based on actual item instead. Not sure when it happened, but my guess is sometime in the last year? I think it was in response to a request from an organization to show specific quantities to the partners.
We could probably fix this by doing a similar lookup to what we do for Barcodes.
https://github.com/rubyforgood/diaper/blob/d1e03866ade774953207a8dd9e3156dbffafa21e/app/controllers/barcode_items_controller.rb#L47 (pasting here for convenience):
def find
    # First, we do a naive lookup
    @barcode_item = current_organization.barcode_items.all.find_by!(value: barcode_item_params[:value])
    # Depending on whether or not the result is solely a global barcode, we may need additional queries
    # Global barcodes don't explicitly map to organization items, so we can do a lookup to clarify that
    if @barcode_item.global?
      # So in this case, we need to do an item lookup. #593 clarifies that we should fall through to the
      # *oldest* item that the organization has that matches this base item type.
      base_item = @barcode_item.barcodeable
      @item = current_organization.items.by_base_item(base_item).order("created_at ASC").first
    else
      # It was a local barcode_item, which maps directly to a known org item. We're set!
      @item = @barcode_item.item
    end
    respond_to do |format|
      format.json { render json: { barcode_item: @barcode_item, item: @item }.to_json }
    end
  end

Basically, look up the item first -- if it fails, then look up by the base_item and find the oldest item that matches the type (this one is most likely to be the most generic)
That would probably work, in this case. What do you think?

@armahillo hey there sorry for the late reply. Thanks for the historical context :)
Giving it some thought. I think falling back to the generic base item in cases where the item has been removed might lead to some confusion. Since the Item <-> BaseItem association can be made by users, it could open up cases in which a missing Item matches a BaseItem that the diaperbank doesn't actually have.
Ideally, I'd like to retain as much information as possible without deleting the original records so our historical data stays intact. I noticed that the Item model has something that is like a soft delete but might be missing one case (hence the missing Item records).
  # in item.rb

  # Override `destroy` to ensure Item isn't accidentally destroyed
  # without first being disassociated with its historical presence
  def destroy
    if has_history?
      update(active: false)
    else
      super
    end
  end

Here are my thoughts on how to proceed:

Ensure we aren't deleting Items with any records. Let's ensure the case found doesn't occur.
(Not awesome but...) I think we should just move forward without fixing the Unknown Item issue. It is going to be very difficult (I dare say impossible) to restore the information.

Thoughts @armahillo ?

This is inevitable when the Partner is allowed to select Items explicitly; that was the whole reason we did the partner_key in the first place (this was actually the result of about a week of discussion between a few of us, a couple years ago). has_history? safeguards against deleting items that have been used in any kinds of transactions within diaperbase. Inbound Requests weren't checked because they hadn't yet been brought into the system (and at the time, it wasn't relevant because requests were always made based on the base item type / partner key).
Merging the two applications will make this a more avoidable mistake since the Partner requests would be able to more easily be tracked and sync'd (and incorporated into has_history?)

it could open up cases in which a missing Item matches a BaseItem that the diaperbank doesn't actually have.

That shouldn't ever happen. The only case where it would is if every item with that BaseItem type was missing (in which case I believe there is a failsafe to provide an indicator that it's missing).

Ensure we aren't deleting Items with any records. Let's ensure the case found doesn't occur.

has_history? IIRC, checks for the appearance of an item_id in the LineItem and InventoryItem models. Request was built using a hash to track the items that were included because we were doing the partner key thing. You could make Request be another Itemizable, and then the items wouldn't be deleteable, implicitly.

(Not awesome but...) I think we should just move forward without fixing the Unknown Item issue. It is going to be very difficult (I dare say impossible) to restore the information.

Retroactively fixing requests with missing items would be hard to do, especially if the partner_key isn't tracked anymore.
Going forward, another possibility would be to include a mapping of the item_id & name of the item with the submitted request. This would at least ensure the data is persisted.
Making Request another Itemizable model is the way forwards, I think. When the apps are merged, that would still persist as a useful refactoring.


deny to create non-ref fields with kind RecordID

hide DataKind_RecordID?

@host6 can you give an example of undesirable code?

@host6 can you give an example of undesirable code?

https://github.com/untillpro/airs-bp3/blob/bad4b72909cb9a83aae54aad4e17ab310048dc5b/packages/air/dashboard/provide.go#37
https://github.com/voedger/voedger/blob/874c8cfac957166061eb2045850bc0c050320755/pkg/sys/authnz/workspace/provide.go#L81
https://github.com/untillpro/airs-bp3/blob/bad4b72909cb9a83aae54aad4e17ab310048dc5b/packages/air/xzreports/impl_savexzreport.go#L19

@host6 parser do not creates fields of type RecordID directly. They are only added with function appdef.IFieldsBuilder.AddRefField which is used for adding reference fields, to either TABLE, TYPE or VIEW definitions.
Regarding the examples provided by you for air app, it will be solved by moving definitions to SQL

It is not important since it is not possible to make this using vSQL


Ability to set pool_recycle args for db connection handling

Is this a BUG REPORT or a FEATURE REQUEST? (choose one): Feature Request
If this is a FEATURE REQUEST, please:
Add ability to specify more db connection pool settings, including at least the ability to set the pool_recycle timeout to ensure stale connections are recycled before being used and causing an error if the connection was closed by the server or some intermediate network gear (e.g. a load balancer).
This is useful for cases where load balancers have short idle connection timeouts like 5-10 minutes.
Should add a generic db_engine_args section and handlers for config.yaml to allow per-deployment customization.
e.g.
credentials:
  database:
    db_connect: ...
    db_connect_args:
      timeout: 120
      ssl: false
    db_pool_size: 30
    db_pool_max_overflow: 100
    db_engine_args:
      <key-values to pass>
      pool_recycle: 600

Closed by #642


Added error handling for unknown ALSA device errors

Instead of taking the easy way out and killing the whole program by panicking, device enumeration and stream creation will now report the error variant 'Unknown' as suggested in #216
Also added an "InvalidArgument" variant for the infamous -22 error (this should resolve #247 and make #206 more handable)

LGTM I was playing with something like this myself in https://github.com/tomaka/cpal/pull/223#issuecomment-435906472

I'm just going to take this opportunity to moan about the awful error reporting in alsa, and C libs in general. They give you nothing to work with but a meaningless posix error code.

@derekdreery I really like your idea to get the error message directly from alsa. That should enable more differentiated error handling so that if you get some weird error, at least you get somewhat of an idea what went wrong.
That beeing written, let me add some rant as well:
I very much dislike the prominence with which Rust advertises panics - I think they should not be used in library crates at all. If a library fails its task, there is absolutely no reason to crash a whole thread or maybe even the whole program. That's what Results/Options are for!

@RoflCopter24 I agree with you that panicking is often the wrong thing to do for stable libraries, but all libraries started out as prototypes, and when you're prototyping you don't want to spend ages on errors. It just needs someone to improve error handling here, which anyone can do I think.

What is holding this Pull Request back?

Personally, I think this should be merged, as it is an improvement on the current state of things.

@tomaka Can you comment?


Will Microsoft Lists be added as an approved client app?

We were excited to finally see the Microsoft Lists app make it to the Apple App store last week, however it is not listed as an approved client app, so we cannot use it with our Conditional Access policies. Is there a timeline for when it might be added to the approved app list so that our users can begin using it without registering their personal devices in InTune? If those monitoring this page don't know the answer, are you able to point me to the team at Microsoft who might know?
Thank you!

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

ID: bd934939-4dbb-3a92-f996-56b9557f49ac
Version Independent ID: 3cda6b81-26e8-c9ac-e768-d912636a461b
Content: Grant controls in Conditional Access policy - Azure Active Directory
Content Source: articles/active-directory/conditional-access/concept-conditional-access-grant.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@GitRomney
Thanks for your feedback! We will investigate and update as appropriate.

@GitRomney
Thanks for your feedback! We will investigate and update as appropriate.

Same issue in our company. Cannot use Microsoft Lists right now

Same issue in our company. Cannot use Microsoft Lists right now

@GitRomney and @diu618
Thank you both for adding to this thread! I've reached out to my team and to the author, to see if we can provide you with any information/point you in the right direction regarding this ask.

@GitRomney and @diu618
Thank you both for adding to this thread! I've reached out to my team and to the author, to see if we can provide you with any information/point you in the right direction regarding this ask.

@GitRomney and @diu618
Thank you both for your time and patience throughout this issue.
We received an update from our engineering team and the app is not an approved app yet. However, we're working to enable App protection policy for it.
Since we're currently working on this internally, I'll go ahead and close out this issue.
Thank you again for your time and patience.

@GitRomney and @diu618
Thank you both for your time and patience throughout this issue.
We received an update from our engineering team and the app is not an approved app yet. However, we're working to enable App protection policy for it.
Since we're currently working on this internally, I'll go ahead and close out this issue.
Thank you again for your time and patience.

Thank you. Can you provide a timeline, please?

On Jan 26, 2021, at 1:08 PM, James Tran-MSFT<EMAIL_ADDRESS>wrote:

@GitRomney and @diu618
Thank you both for your time and patience throughout this issue.
We received an update from our engineering team and the app is not an approved app yet. However, we're working to enable App protection policy for it.
Since we're currently working on this internally, I'll go ahead and close out this issue.
Thank you again for your time and patience.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thank you. Can you provide a timeline, please?

On Jan 26, 2021, at 1:08 PM, James Tran-MSFT<EMAIL_ADDRESS>wrote:

@GitRomney and @diu618
Thank you both for your time and patience throughout this issue.
We received an update from our engineering team and the app is not an approved app yet. However, we're working to enable App protection policy for it.
Since we're currently working on this internally, I'll go ahead and close out this issue.
Thank you again for your time and patience.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@GitRomney
Thank you for the quick follow up!
Unfortunately, there's no ETA as of right now. However, I'd recommend monitoring this doc page or our Azure Updates page if you'd like to get the latest updates on Azure products and features.

@GitRomney
Thank you for the quick follow up!
Unfortunately, there's no ETA as of right now. However, I'd recommend monitoring this doc page or our Azure Updates page if you'd like to get the latest updates on Azure products and features.


Title don't reactive with TAPi18n?

Title don't reactive
title: function () {
        return TAPi18n.__('core.exchange.title');
    },

I try to change language in the same route, it don't reactive.

Returned value from this Function must be reactive (if you want to have it reactive)

What is returned by TAPi18n.__() method?
What do you actually expect?
How you change the title? (I guess you trying to change title outside of the Router, right?)

Just to make sure you use this lib with flow-router-extra, right?

Sorry I am late, It work fine.


Trying to connect to private IP addresses

From .coda-config/coda.log :
{"timestamp":"2020-06-10 00:38:34.603187Z","level":"Spam","message":"Output from process $child_name: $line","metadata":{"child_name":"libp2p_helper-stderr","handle":"stderr","line":"{"id":1045,"level":2,"message":"couldn't connect to dht peer map[addrs:[/ip4/<IP_ADDRESS>/tcp/8302 /ip4/<IP_ADDRESS>/tcp/8302 /ip4/<IP_ADDRESS>/tcp/8302 /ip4/<IP_ADDRESS>/tcp/8302] peerID:12D3KooWBoUzBKdvJ4Nx8hyRQTFeTuouEa7P24H652YXJR4P8xGx] (maybe the network ID mismatched?): failed to dial : all dials failed\n  * [/ip4/<IP_ADDRESS>/tcp/8302] failed to negotiate security protocol: connected to wrong peer\n  * [/ip4/<IP_ADDRESS>/tcp/8302] failed to negotiate security protocol: connected to wrong peer\n  * [/ip4/<IP_ADDRESS>/tcp/8302] dial tcp4 <IP_ADDRESS>:8302-\u003e<IP_ADDRESS>:8302: i/o timeout","module":"codanet.Helper","time":"2020-06-10T02:38:29.361426487+02:00"}","pid":12753,"process":"libp2p_helper"}}
{"timestamp":"2020-06-10 00:38:34.603251Z","level":"Spam","message":"Output from process $child_name: $line","metadata":{"child_name":"libp2p_helper-stderr","handle":"stderr","line":"{"id":1046,"level":4,"message":"error reading rpc from 12D3KooWBFkd5LeJdoKs7aV2yRBBSXaSN6hqZByCQCo2oyPucuVi: stream reset","module":"pubsub","time":"2020-06-10T02:38:32.47751189+02:00"}","pid":12753,"process":"libp2p_helper"}}
{"timestamp":"2020-06-10 00:38:34.603319Z","level":"Spam","message":"sending line to libp2p_helper: $line","metadata":{"host":"<IP_ADDRESS>","line":"{"seqno":378,"method":"listPeers","body":{}}","peer_id":"12D3KooWPpStBnkyf6trfGuQYc7YDEMMuJk6G6d7M3ACfjbUEy2e","pid":12753,"port":8302}}
{"timestamp":"2020-06-10 00:38:34.603372Z","level":"Spam","message":"Output from process $child_name: $line","metadata":{"child_name":"libp2p_helper-stderr","handle":"stderr","line":"{"id":1047,"level":4,"message":"got error on dial: dial tcp4 <IP_ADDRESS>:8302-\u003e<IP_ADDRESS>:8302: i/o timeout","module":"swarm2","time":"2020-06-10T02:38:34.369179794+02:00"}","pid":12753,"process":"libp2p_helper"}}
The <IP_ADDRESS>/8 address range is marked privated (i.e. used in LAN).
WARNING : some hosting providers might flag this as abussive behaviour, a programme trying to compromise their internal network.

Interesting, this looks like the Libp2p peer discovery behavior, @emberian any thoughts about this?

Believe this is libp2p always checking on local first to give preference to local peers but need to double check with @emberian

Closing this issue as it is stale for a long time.


CRDs should not be uninstalled when helm uninstall is run

From looking at the helm chart docs, it is standard practice that running helm uninstall does not delete the CRDs previously installed. Specifically saying:

Operators who want to upgrade or delete CRDs are encouraged to do this manually and with great care.

When the service catalog chart is combined with other charts that follow the standard practice, it leads to a strange mix of behaviours.
The questions we have are the following:

What is the reason behind implementing a CRD uninstall hook?
If there is a reason you believe this behaviour can be desirable would you be open to parameterising it so that charts which want to depend on service catalog can disable it?

cc @edwardecook
Thanks!

/remove-lifecycle stale

Hi @gmrodgers
Before CRDs we had the api-server with underlying etcd server. When it was uninstalled then all resources were also deleted (ServiceInstance, ServiceBindings etc).
We just wanted to have the same behavior with CRDs as we didn't want to have a breaking change.
Pros when removing CRDs and its instances:

you do not leak any resources which cannot be later managed
it's not misleading to the user as you do not leave any CRDs which cannot be used as backend(controller) was removed

But I can also imagine that leaving CRDs also has some benefits. But the worst thing is that you cannot say if you want to keep those resources when doing the helm delete action because it does not take any params. You need to take that decision while doing helm install which is another problem.
We can think about creating some ConfigMap with some value before doing helm delete and base on that we will know if delete or not the CRDs but it's making this process more complex.
We also need to think about removing CRDs we have to think about it in conjunction with ServiceCatalog reinstallation.  If you will not delete and install ServiceCatalog once again then it will fail because CRDs will be already on the cluster.
TBH I still do not know which behavior is better, delete or not. IMO we can make it optional but also take care of all aspects that I mentioned above.
@gmrodgers are you interested to create a proposal/POC?
We are definitely open for feedback and ideas :)

I'd be interested to hear what the use case of keeping CRDs while uninstalling the controller for these CRDs actually is.
A possible way to allow implicit control over this behaviour could be to separate out the CRDs and controller into their own Helm charts and add both as child charts to a third.
Installing the child charts would support the "keep CRs around" use case while installing the parent chart would keep the current behaviour of deleting them.
From my point of view, we should not adapt the Helm 3 crds folder concept, as this offers no support at all for updating CRDs.
What we should think about is adopting Kubernetes 1.16s CRD versioning, so that we minimize the risk of breaking clients with changes in the CRDs.

This issue in question is the difference between helm charts for v2 and v3, as the v3 charts treat CRDs with more caution.
We may update our chart to use the new schema, as the old schema is no longer supported; but that is not the top priority.
Closing the issue.
/remove-lifecycle rotten
/close

We have just discovered this feature, luckily in a temporary environment.

I'd be Interested to hear what the use case of keeping CRDs while uninstalling the controller for these CRDs actually is.

Let's say we have a working environment with multiple ServiceInstances running, and for whatever reason (helm release was damaged, tiller glitch, networking issue, helm release became unrecoverable), we have to helm delete service-catalog and install it again. Upon deletion, all CRDs get destroyed and existing ServiceInstances too.
We were testing out this specific scenario and found out all the resources gone.
I believe whatever tool was used to create certain entities should be also used to remove them, for the sake of consistency.


Latest products - Get top 10 latest products

An API endpoint to retrieve top 10 latest products

I can work on this


Fixed theme_download variable initialization.

I had problem with downloading themes via your plugin and found that variable theme_download is not properly initialized. The condition data.download_url_v2 != "" was actually true for non-existing url (is is undefined, not empty string).
Also you had duplicated line, so I removed it :)

thanks for the heads-up! @toomyem
I couldn't take your pull request tho, because I had other fixes already done.
pushing the new stuff the next few minutes 😉


feat: configure project to use Docker Compose

Create Docker files to run project also on two containers: api and mongodb

https://github.com/marciovsena/bibleapi/pull/115


added antibody to alternatives

Replaced the alternative list for a link to a more complete and up to date list of frameworks and plugins.
Thanks.


Can you share the code for predicting？

Hi, I'm not sure the meaning of predicting. Actually, you can follow the procedures in Installation and utilize Detectron2 for training and evaluation.

Hi, I'm not sure the meaning of predicting. Actually, you can follow the procedures in Installation and utilize Detectron2 for training and evaluation.

I mean, input a set of photos and use the trained panoramic segmentation model to predict the objects in these photos. Thank you for your reply!

Hi, I'm not sure the meaning of predicting. Actually, you can follow the procedures in Installation and utilize Detectron2 for training and evaluation.

I mean, input a set of photos and use the trained panoptic segmentation model to predict the objects in these photos. Thank you for your reply!

Hi, for demo visualization, you can utilize the demo/demo.py in Detectron2, with a simple modification. Please to the comment in this issue.

Hi, for demo visualization, you can utilize the demo/demo.py in Detectron2, with a simple modification. Please to the comment in this issue.

Ok, thank you.


Line 34 Error

Traceback (most recent call last):
File "/Users/user/Downloads/YouTube-shorts-generator-main/main.py", line 34, in
os.rename(directory + os.listdir(directory)[0], directory + "main_clip.mp4")
IndexError: list index out of range

The way this line is supposed to work is as follows:
os.rename takes old directory of file, and places it in new directory with new name.
directory + os.listdir(directory)[0] --> temp_clips/[name of file downloaded using redvid, as it's first item in the directory]
directory + "main_clip.mp4" --> temp_clips/main_clip.mp4
If you're getting indexError, this isn't due to directory not existing, per os.rename documentation.
This means issue is probably within downloading reddit video. Make sure config.py is configured correctly. If you think it's a bug in the code, lmk how I can replicate it!
Hope that helped,
Zack.


Add GitHub Actions

I recently announced ha-blueprint:

With formatting (black + isort for python, prettier for js)

It’ll go ahead and pull it, format it, and amend the changes to the last commit.

With lint

JS: Run ESLint to catch syntax errors
Python: It runs Hassfest (to catch invalid integrations), HACS (to catch invalid HACS integrations), and flake8 (to catch invalid python).

https://community.home-assistant.io/t/made-a-custom-integration-or-card-ive-created-an-all-in-one-github-action-for-you/235041
This makes it so it auto-checks for HACS, and ESLint so you won't have to worry about it being invalid.

Hey thanks for this PR but I'm not sure how this is going to help here.
Card is written in TS so ESLint is not needed - syntax errors are caught at the build time.
Is there anything else what your action can catch in case of the lovelace cards?

Forgot to change the text. It can run prettier too.

Maybe it makes more sense for integrations but here it is little bit useless. So sorry but I think I won't use it - it would be just waste of computation power

Well, it doesn't cost you anything, and for public repositories GitHub Actions has an unlimited quota.

But what is the point of doing it? Just because it is free for you it doesn't mean it doesn't cost money someone. GH has to pay for computation power (electricity, machines, service etc). If everyone starts using random actions, without any reason, eventually GH will start running out of the quota and they may decide to cut the free Actions for public repos.
Please do not send such PRs to random repos, just because they are related to HA. I believe that your Action can be helpful for some of them (like integrations) but for plugins/cards written in TS it doesn't give any value.

I get your point.

Just curious, is there anything making you particularly upset? Your tone of voice indicates that you're angry.

Maybe I was too harsh saying that it doesn't give any value. Prettier can be helpful actually but I wasn't (am not) aware how does it work exactly here.
So if someone sends PR, with badly formatted code, will it automatically fix it and send update?

Well, once you merge it, it amends the last commit. There's plenty of other GitHub Apps that have the ability to create a PR on the PR, and this GitHub Action doesn't do it.

Ok thanks for the info. Anyway I think this project is too small - TBH I don't expect many PRs coming so I don't mind fixing formatting or asking someone to do it.

Got it. Thanks for letting me know.


fix: compile down to ES5

I think it's not the time that we can publish an npm package as ES2017 code if it is intended to use this on a browser 😭

Thanks 👍

Thank you for PR! That’s exactly true.


[Bug]: Bash prompt not colored in pure mode

Current Behavior (bug)
When using Bash as a shell and running devbox shell, the prompt is colored (green and blue). But when running devbox shell --pure, the prompt is not colored (white).
Expected Behavior (fix)
The prompt should be colored in pure mode. This should just be a matter of adding TERM to the list of environment variables that are passed through from the parent shell.
Additional context
Output of devbox version -v:
Version:     0.5.5
Platform:    linux_amd64
Commit:      891e8de7a8ab7a3fec42fad069326d68cb84603e
Commit Time: 2023-06-19T19:18:00Z
Go Version:  go1.20.5
Launcher:    0.2.0

This issue should be fixed in v0.5.6


add widget for access_logging

Cherry-pick #256 with no conflicts

looks like an indentation issue on this and the original, otherwise 👍 lgtm

I fixed the spacing here. I'll submit a new PR to fix the indent on develop


fastlane commands not running

New Issue Checklist

[x ] Updated fastlane to the latest version
[ x] I read the Contribution Guidelines
[ x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description

I want to generate login session using fastlane command "fastlane spaceauth -u<EMAIL_ADDRESS>for Continuous Integration. It was working fine previously on my MacOS. But when I update Ruby version from 2.3 to 2.7, now no fastlane command is running and it is giving me following error;
Trying to register Bundler::GemfileError for status code 4 but Bundler::GemfileError is already registered (ArgumentError). I have tried to update gem, bundler and fastlane but could not get any solution.
Complete output when running fastlane, including the stack trace and command used

**fastlane spaceauth -u<EMAIL_ADDRESS>[⠙] 🚀 /Users/vsts/.rvm/gems/ruby-2.7.0/gems/faraday_middleware-0.13.1/lib/faraday_middleware/response_middleware.rb:14: warning: Capturing the given block using Proc.new is deprecated; use `█` instead
[⠼] 🚀 /Users/vsts/.rvm/gems/ruby-2.7.0/gems/bundler-2.1.4/lib/bundler/vendor/fileutils/lib/fileutils.rb:105: warning: already initialized constant Bundler::FileUtils::VERSION
/Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/bundler/vendor/fileutils/lib/fileutils.rb:105: warning: previous definition of VERSION was here
/Users/vsts/.rvm/gems/ruby-2.7.0/gems/bundler-2.1.4/lib/bundler/vendor/fileutils/lib/fileutils.rb:1284: warning: already initialized constant Bundler::FileUtils::Entry_::S_IF_DOOR
/Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/bundler/vendor/fileutils/lib/fileutils.rb:1284: warning: previous definition of S_IF_DOOR was here
/Users/vsts/.rvm/gems/ruby-2.7.0/gems/bundler-2.1.4/lib/bundler/vendor/fileutils/lib/fileutils.rb:1568: warning: already initialized constant Bundler::FileUtils::Entry_::DIRECTORY_TERM
/Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/bundler/vendor/fileutils/lib/fileutils.rb:1568: warning: previous definition of DIRECTORY_TERM was here
/Users/vsts/.rvm/gems/ruby-2.7.0/gems/bundler-2.1.4/lib/bundler/vendor/fileutils/lib/fileutils.rb:1626: warning: already initialized constant Bundler::FileUtils::OPT_TABLE
/Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/bundler/vendor/fileutils/lib/fileutils.rb:1626: warning: previous definition of OPT_TABLE was here
/Users/vsts/.rvm/gems/ruby-2.7.0/gems/bundler-2.1.4/lib/bundler/vendor/fileutils/lib/fileutils.rb:1685: warning: already initialized constant Bundler::FileUtils::LOW_METHODS
/Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/bundler/vendor/fileutils/lib/fileutils.rb:1685: warning: previous definition of LOW_METHODS was here
/Users/vsts/.rvm/gems/ruby-2.7.0/gems/bundler-2.1.4/lib/bundler/vendor/fileutils/lib/fileutils.rb:1692: warning: already initialized constant Bundler::FileUtils::METHODS
/Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/bundler/vendor/fileutils/lib/fileutils.rb:1692: warning: previous definition of METHODS was here
[⠴] 🚀 Traceback (most recent call last):
	24: from /Users/vsts/.rvm/gems/ruby-2.7.0/bin/ruby_executable_hooks:24:in `'
	23: from /Users/vsts/.rvm/gems/ruby-2.7.0/bin/ruby_executable_hooks:24:in `eval'
	22: from /usr/local/bin/fastlane:23:in `'
	21: from /usr/local/bin/fastlane:23:in `load'
	20: from /Users/vsts/.rvm/gems/ruby-2.7.0/gems/fastlane-2.139.0/bin/fastlane:23:in `'
	19: from /Users/vsts/.rvm/gems/ruby-2.7.0/gems/fastlane-2.139.0/fastlane/lib/fastlane/cli_tools_distributor.rb:41:in `take_off'
	18: from /Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
	17: from /Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
	16: from /Users/vsts/.rvm/gems/ruby-2.7.0/gems/fastlane-2.139.0/fastlane/lib/fastlane.rb:27:in `'
	15: from /Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
	14: from /Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
	13: from /Users/vsts/.rvm/gems/ruby-2.7.0/gems/fastlane-2.139.0/fastlane/lib/fastlane/plugins/plugins.rb:9:in `'
	12: from /Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
	11: from /Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
	10: from /Users/vsts/.rvm/gems/ruby-2.7.0/gems/fastlane-2.139.0/fastlane/lib/fastlane/plugins/plugin_manager.rb:3:in `'
	 9: from /Users/vsts/.rvm/gems/ruby-2.7.0/gems/fastlane-2.139.0/fastlane/lib/fastlane/plugins/plugin_manager.rb:4:in `'
	 8: from /Users/vsts/.rvm/gems/ruby-2.7.0/gems/fastlane-2.139.0/fastlane/lib/fastlane/plugins/plugin_manager.rb:5:in `'
	 7: from /Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
	 6: from /Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:72:in `require'
	 5: from /Users/vsts/.rvm/gems/ruby-2.7.0/gems/bundler-2.1.4/lib/bundler.rb:7:in `'
	 4: from /Users/vsts/.rvm/gems/ruby-2.7.0/gems/bundler-2.1.4/lib/bundler.rb:7:in `require_relative'
	 3: from /Users/vsts/.rvm/gems/ruby-2.7.0/gems/bundler-2.1.4/lib/bundler/errors.rb:3:in `'
	 2: from /Users/vsts/.rvm/gems/ruby-2.7.0/gems/bundler-2.1.4/lib/bundler/errors.rb:20:in `'
	 1: from /Users/vsts/.rvm/gems/ruby-2.7.0/gems/bundler-2.1.4/lib/bundler/errors.rb:20:in `'
/Users/vsts/.rvm/gems/ruby-2.7.0/gems/bundler-2.1.4/lib/bundler/errors.rb:9:in `status_code': Trying to register Bundler::GemfileError for status code 4 but Bundler::GemfileError is already registered (ArgumentError)
gem env
RubyGems Environment:

RUBYGEMS VERSION: 3.1.2
RUBY VERSION: 2.7.0 (2019-12-25 patchlevel 0) [x86_64-darwin18]
INSTALLATION DIRECTORY: /Users/vsts/.rvm/gems/ruby-2.7.0
USER INSTALLATION DIRECTORY: /Users/vsts/.gem/ruby/2.7.0
RUBY EXECUTABLE: /Users/vsts/.rvm/rubies/ruby-2.7.0/bin/ruby
GIT EXECUTABLE: /usr/bin/git
EXECUTABLE DIRECTORY: /Users/vsts/.rvm/gems/ruby-2.7.0/bin
SPEC CACHE DIRECTORY: /Users/vsts/.gem/specs
SYSTEM CONFIGURATION DIRECTORY: /Users/vsts/.rvm/rubies/ruby-2.7.0/etc
RUBYGEMS PLATFORMS:

ruby
x86_64-darwin-18

GEM PATHS:

/Users/vsts/.rvm/gems/ruby-2.7.0
/Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/gems/2.7.0

GEM CONFIGURATION:

:update_sources => true
:verbose => true
:backtrace => false
:bulk_threshold => 1000

REMOTE SOURCES:

https://rubygems.org/

SHELL PATH:

/Users/vsts/.rvm/gems/ruby-2.7.0/bin
/Users/vsts/.rvm/gems/ruby-2.7.0@global/bin
/Users/vsts/.rvm/rubies/ruby-2.7.0/bin
/usr/local/bin
/usr/bin
/bin
/usr/sbin
/sbin
/Applications/git-tf/
/usr/local/share/dotnet
/Library/Frameworks/Mono.framework/Versions/Current/Commands
/Users/vsts/Library/Android/sdk/platform-tools
/Users/vsts/Library/Android/sdk/tools
/opt/gradle/bin
/Users/vsts/.gem-cache/gems/fastlane-2.118.1/bin
/Users/vsts/.rvm/bin

bundle env
Environment
Bundler             2.1.4
  Platforms         ruby, x86_64-darwin-18
Ruby                2.7.0p0 (2019-12-25 revision 647ee6f091eafcce70ffb75ddf7e121e192ab217) [x86_64-darwin18]
  Full Path         /Users/vsts/.rvm/rubies/ruby-2.7.0/bin/ruby
  Config Dir        /Users/vsts/.rvm/rubies/ruby-2.7.0/etc
RubyGems            3.1.2
  Gem Home          /Users/vsts/.rvm/gems/ruby-2.7.0
  Gem Path          /Users/vsts/.rvm/gems/ruby-2.7.0:/Users/vsts/.rvm/rubies/ruby-2.7.0/lib/ruby/gems/2.7.0
  User Home         /Users/vsts
  User Path         /Users/vsts/.gem/ruby/2.7.0
  Bin Dir           /Users/vsts/.rvm/gems/ruby-2.7.0/bin
Tools
  Git               2.21.0 (Apple Git-122)
  RVM               1.29.9 (latest)
  rbenv             not installed
  chruby            not installed
  rubygems-bundler  (1.4.5)

Bundler Build Metadata
Built At          2020-01-05
Git SHA           32a4159325
Released Version  true

gem which fastlane
/Users/vsts/.rvm/gems/ruby-2.7.0/gems/fastlane-2.139.0/fastlane/lib/fastlane.rb
</pre>
</details>

### Environment

<!-- Please run `fastlane env` and copy the output below. This will help us help you :+1:
If you used `--capture_output` option, please remove this block as it is already included there. -->

<details>
  <pre>[INSERT OUTPUT HERE]</pre>
</details>

It seems like you have not included the output of fastlane env
To make it easier for us help you resolve this issue, please update the issue to include the output of fastlane env 👍

Because fastlane command is not running and it gives the same error as I mentioned above.

I have solved the issue by simply deleting the latest bundler version and using old default version;
gem uninstall bundler


Using grid

Hi dart team
Is there any grid components available for angular2dart?
Thanks

Hi, thanks for filing an issue @kostonstyle.
We don't have any plan to provide grid components immediately, as we're not sure we can do so in a way that will perform well and not have code size implications. Obviously we welcome any community contributed package if someone feels strongly about it.
It might be nice to hear what you intend to build and what your needs are.

Ping @kostonstyle

I'm closing due to lack of response. Feel free to re-open if this becomes an issue.


feat: update the edit project route to use the PUT method

User description
Fixes #55

PR Type
Enhancement

Description

Updated the /update-project route in routes.go to use the PUT method instead of POST.
Modified the project update form in projects.html to use JavaScript for handling PUT requests.
Added a new JavaScript function updateProject to handle the PUT request and reload the page on success.

Changes walkthrough 📝
Relevant filesEnhancement

routes.goChange update project route to use PUT method

cmd/web-ui/routes.go
Updated the /update-project route to use the PUT method instead of POST.

+1/-1

projects.htmlUpdate project form to use PUT method via JavaScript

cmd/web-ui/templates/projects.html
Modified the form method to use JavaScript for handling PUT requests.  Added updateProject JavaScript function to handle the PUT request.

+17/-1

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

/review
--pr_reviewer.num_code_suggestions="4"
--pr_reviewer.inline_code_comments=true
-i
--pr_reviewer.require_score_review=true


Add missing steps to "run locally" instructions

pip install -r requirements-dev.txt was missing, which will cause errors in a new repo.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Just an additional thought, would it make sense to change step 3 to use pip install -e? Then maybe people don't have to pip install every time. Haven't tested this but if that works how I think it does it might be another nice quick addition to this PR or a future one.

Eh, maybe. I've been burned before trying to help users with -e and I personally tend to stay clear of it. But I may be unfairly biased against it. From https://developers.redhat.com/articles/2021/05/26/can-we-consider-editable-bad-practice#editable_dependencies_and_project_thoth, it looks like the issue is using -e for dependencies, not the project itself


Next 6 month road map as software engineer?

I am a backend developer that approximately 3 years of hands-on experience as a java developer.
After failure recent interview processes I decided to improve my skills on these branches.
I am open any suggestion to make reform my road map. Thank in advance.
*Algorithms
Learn implementation of most known algorithms(Sorting, searching, etc.).
Practice hackerrank, leetcode, dailycodingproblem.
Practice whiteboard coding.
*DataStructures
Learn all available DS in CS.
Learn how to implement your own data structure.
*Java
Improve core java skills including multithreading, serializable, socket programming and etc.
Improve framework skills (Spring boot).
Design patterns.
Unit testing.
*Operations
Linux from Evi Nemeth.
Learn about DevOps tools and technologies.
Learn network fundamentals. (OSI layers, network protocols, subnetting, etc.)
Architectural design.

Your roadmap is pretty solid and also tough I would say. You are a hero if you can do it. Also maybe include some FrontEnd stuff optionally?

There are some front end technologies like angular, react in my backlog. I decided to learn the mentioned technologies when I feel comfortable at the backend. for my prediction, this will happen in the first quarter of 2020.

@vgrch I dont think its right one, let me do one for u this evening.

I made the schedule:
Mondays and Thursday: Algorithms and DS.
Tuesday and Friday: Core Java and framework
Wednesday: Operations
I can dedicate about 2 hours every day.
It would be better to hear any recommendation from you.

So, will go to more details, for now you can start:

Language: Python3
Algorithm https://github.com/goupaz/0-to-hero/blob/master/algorithms_plan.md
Leetcode: Easy ones only
Data Structures (coming)
OS: Ubuntu (coming)
1 Open source contribution: written in Python3, preferable Django Framework

Update the current state. Currently preparing Java8 OCA exam.
Watching algorithms & data structure course
Reading this book

*Algorithms
[X] - Learn implementation of most known algorithms(Sorting, searching, etc.).
[X] - Practice hackerrank, leetcode, dailycodingproblem.
[ ] - Practice whiteboard coding.
*DataStructures
[X] - Learn all available DS in CS.
[ ] - Learn how to implement your own data structure.
*Java
[X] - Improve core java skills including multithreading, serializable, socket programming and etc.
[X] - Improve framework skills (Spring boot).
[ ] - Design patterns.
[ ] - Unit testing.
*Operations
[ ] - Linux from Evi Nemeth.
[X] - Learn about DevOps tools and technologies.
[ ] - Learn network fundamentals. (OSI layers, network protocols, subnetting, etc.)
[ ] - Architectural design.

Good job @vgrch please, keep sharing your experiences. We are here to support you!


Issue with replaceWith method

When I am using replaceWith on an element, it changes the size of childNodes the element has.
On closely examining I found that element ito be replaced with has previous element as  "br" or "br" followed with textnode with just newline (\n) are now after replacement only textnode with newline"\n".  Is it replacing any prev "br" elements and newlines?
Is this a known behvaiour or an issue? Can i change this behaviour or is there a workaround?

Can you add a test case please, to better understand the issue.

Please reopen if you have a testcase.

Please reopen if you have a testcase.


Fuselag Notam (Independent Publisher)

Description: This will retrieve NOTAM (Notice to airmen) data from the FAA using a valid API key.
See further instructions in the ReadMe file.
Options:
I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[x] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[x] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[x] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[x] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[x] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

@vmanoharas Why are we submitting a CLA license agreement for a connector self created?

Issues corrected
summary property and contact /info property
license/cla does not work as expected

@vmanoharas , the modifications were made to the connector. However, there are issues with the cla/license agreement. I constantly received a message stating that the license is not agreed upon. I have tried submitting the following parameters:
@microsoft-github-policy-service agree and
@microsoft-github-policy-service agree company="Double Axis"
What are these parameters attempting to sync or map to? Our profile name, etc?

@microsoft-github-policy-service agree company="Microsoft"

Renamed PR from: Proposal - FAA Notam - Double Axis to Fuselag Notam (Independent Publisher)

Modified the PR title
Checked the boxes on the PR form
Modified Readme file and added screenshots demonstrating use of the feature
Showed successful test inside of the readme file

@vmanoharas  Please review the requested modifications. All checks have passed.

@vmanoharas , Removed the images from the ReadMe file and from the directory

@vmanoharas , Please review

Hi @SalesLighter,
Thank you very much, please remove independent-publisher-icon as well.  We will add it during certification.

@vmanoharas - Removed the independent publisher icon from the directory per your request. Please review.

[[certify-connector]]

Hello, I see that the certification is still in progress. Any updates on a certification date?

Hello @SalesLighter,
Your connector is now deployed to the Asia region. It will be deployed to all the regions on or before 22nd December.

Hello Partner,
I hope you are doing well.
Congratulations, the deployment of your connector and documentation page was already completed, We have a short survey on the connector certification process that we would like for you to complete. Your feedback is appreciated and will help us improve the program.
Thank you very much for working with us.


bootstrap popover & tooltip on hover doesn't display when cursor enters button at the edge where the popover-caret would be placed

When doing exactly what is described below, this happens 95% of the time. However I noticed sometimes there are periods where it doesn't happen 10x in a row.
This happens even on the bootstrap example itself:
https://ng-bootstrap.github.io/#/components/popover/examples
This only happens when entering slowly with the cursor at the red marked line here:

This also only happens if the movement of the cursor is slow.
It is especially problematic with small buttons, as the popover-caret size is absolute and thus takes up a bigger percent of the button-edge.
Also you can see the cursor quickly switching between default and pointer around 100 times per second when exactly over the red line. This is likely a byproduct of the bug.

I see the gif, but I cannot replicate it in windows & linux (chrome, brave, firefox) . Can you provide more details?

I think it's not a big deal. It's only happening when the mouse enter so slowly.

I see the gif, but I cannot replicate it in windows & linux (chrome, brave, firefox) . Can you provide more details?

Yes I guess it doesn't happen on all devices. I tried it on my work laptop with chrome and it happened but now on my personal (newer/better) one it doesn't happen.
My work laptop is a think pad.

I think it's not a big deal. It's only happening when the mouse enters so slowly.

Cleary this doesn't happen 95% of the time on your setup (but it did on the bootstrap wesbite using my work laptop).
This does happen like 80% of the time on my application actually, and this even without going "unrealistically" slow, just normal user speed slow.

https://user-images.githubusercontent.com/81622754/183368666-8375c580-b1a0-402e-9c8f-89420fff1429.MOV
This is what it looks like in my application. As you can see it only appears twice when trying like 15-20 times.
(Sry no screen record/gif cause no ShareX cause work laptop)

same issue after update<EMAIL_ADDRESS>to<EMAIL_ADDRESS>

same issue after update<EMAIL_ADDRESS>to<EMAIL_ADDRESS>

How exactly is this the same issue? Should the popover/tooltip appear on top of the caret you circled but it doesn‘t? Where is the mouse cursor, from where did you enter etc. from the picture you sent not much can be related for me.

Is there any chance of this getting fixed soon or rather not?
Can I provide any additional information for you to be able to fix this?
@GeoSot
I see the gif, but I cannot replicate it in windows & linux (chrome, brave, firefox) . Can you provide more details?
What details exactly would be helpful to solve this? I will try listing some:

We are using ng-bootstrap for our application. But as @Nudasoft commented it also happens on the general boostrap tooltip-page.
I am eager to help solving this, can also use discord if this helps to provide more information or better explain the problem scenario.

Was this fixed by adding this gap between the element and the tooltip/popover-caret-arrow?

If so, if someone could confirm, then I think this can be closed.


v1.4 Updated collated files

@inGall After scanning the code, I think your code quality is fine.  Please refer to this: https://github.com/nus-cs2103-AY1718S1/forum/issues/173 for more tips to improve the code quality.


Version error on install.

In godot 3.5.3, importing gives this error dialog

Please advise?

I downloaded 3.2 (thank the gods that godot is so self contained) and received this error dialog

Let's see what happens.

I tried to be thorough in the troubleshooting and detailing the process. Hope it helps.

Sorry, but since I've brought my project forward to 4.0, it wouldn't be compatible with Godot 3.2. However, you can go download a past release here, which should compatible with Godot 3.2: https://github.com/Pocket-Godot/Pokemon-Battle/releases

Ok. Then you should change the readme. That's why I tried it to begin with.


Change matchers to accept raw dom nodes as arguments

I found it a bit unnerving that I have to always meticulusly make sure that all my arguments to these matchers are for sure dom nodes, when jquery itself doesn't care and just returns itself when wrapping a jquery object again.
So thats why I wrote this patch, in the hopes that you accept it.
I found my modification of the testsuite not so great, but I also didn't see a better way to do it. :/
Hope you like it!
Regards,
Martin

+1, I'd be interested in this change. Several times had to double check to see if I needed jQuery objects or just DOM nodes, and the overhead for this is negligible.

ping...
Hello dear maintainer - do you have any intention of merging this?
Regards,
Contributor

Ping....
Dear Maintainer,
do you have any intention on pulling this in?
Regards,
Your friendly contributor

The toBe and toContain matchers currently rely on receiving a jQuery object in order to tell if they should use the jQuery semantics or call through to the original Jasmine matchers. I see that you backed that check up with the jasmine.isDomNode function to test whether the actual result is a DOM node, but that function is pretty naive. It returns true if the argument has a nodeType property that compares > 0. I could see that leading to false positives, which would produce some unpredictable behavior for those matchers. For example, this would fail:
var o = {nodeType: "2"};
expect(o).toBe(o)

Is there a more robust way to distinguish DOM objects?

Well, we could try to go for the inheritance hierarchy of dom nodes, but then again that has problems with old IE versions and you get even more problems when you get dom objects across iframe boundaries.
On the other hand checking the nodeType to see what kind of dom node you have before you is used quite a lot in many places.
I would guess that sending a patch to jasmine to enhance isDomNode to check more properties would be a good idea, but I personally wouldn't worry about it till someone actually trips  up on this.


Fix the SIGSEGV for large index

Description
Upgrade the larray version to include the fix: https://github.com/xerial/larray/pull/52
Also fix the issue of not able to load large index if it shows up first in SingleFileIndexDirectory

Codecov Report

Merging #6577 (06c26bd) into master (1beaab5) will decrease coverage by 1.01%.
The diff coverage is 60.64%.

@@            Coverage Diff             @@
##           master    #6577      +/-   ##
==========================================
- Coverage   66.44%   65.43%   -1.02%
==========================================
  Files        1075     1342     +267
  Lines       54773    65983   +11210
  Branches     8168     9620    +1452
==========================================
+ Hits        36396    43174    +6778
- Misses      15700    19733    +4033
- Partials     2677     3076     +399

Flag
Coverage Δ

unittests
65.43% <60.64%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...e/pinot/broker/api/resources/PinotBrokerDebug.java
0.00% <0.00%> (-79.32%)
:arrow_down:

...pinot/broker/api/resources/PinotClientRequest.java
0.00% <0.00%> (-27.28%)
:arrow_down:

...ot/broker/broker/AllowAllAccessControlFactory.java
71.42% <ø> (-28.58%)
:arrow_down:

.../helix/BrokerUserDefinedMessageHandlerFactory.java
33.96% <0.00%> (-32.71%)
:arrow_down:

...ker/routing/instanceselector/InstanceSelector.java
100.00% <ø> (ø)

...ava/org/apache/pinot/client/AbstractResultSet.java
66.66% <ø> (+9.52%)
:arrow_up:

...n/java/org/apache/pinot/client/BrokerResponse.java
100.00% <ø> (ø)

.../main/java/org/apache/pinot/client/Connection.java
35.55% <ø> (-13.29%)
:arrow_down:

...org/apache/pinot/client/DynamicBrokerSelector.java
82.85% <ø> (+10.12%)
:arrow_up:

...n/java/org/apache/pinot/client/ExecutionStats.java
68.88% <ø> (ø)

... and 1203 more

Continue to review full report at Codecov.

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


Release v0.26.0 on April 06, 2022

Checklist

[ ] Create the new release branch with minor version, i.e. release-0.26 for v0.26.0.

You can use the existing branch for patch releases.

[ ] Tag release by running Tag action in Github UI against release branch, i.e. release-X branch.

Use v-prefixed version, like v0.26.0, for both description and tag.

[ ] Run CI action against release branch.
[ ] Tag the next pre-release by running Tag action in Github UI against master branch, if it's not a patch release.

The tag should be v0.27.0-rc.0 if you're releasing v0.26.x.

[ ] Validate that you can install the published version, basic sanity check.
[ ] Run Promote action to promote it in alpha channel if it's pre-1.0.
[ ] Use Github UI to generate release notes.

Make sure to scan all PRs marked with breaking-change and add instructions for users to handle them.

[ ] Create the next release issue with a title that has its date, 4 weeks after the current release day.
[ ] Announce in Slack, Twitter etc.

The release is delayed because of https://github.com/crossplane/provider-aws/pull/1250


Non-writable IO device error when intersecting BAM and BED files

I have a sorted BAM file that I want to filter for reads not overlapping regions in a BED file. However, the following error occurs:
bedtools intersect -a sample.bam -b regions.bed -v > reads.not.in.regions.bam
terminate called after throwing an instance of 'std::runtime_error'
  what():  BgzfStream::Write() - trying to write to non-writable IO device

I have plenty of space to write the files, and if I do not specify an output file it prints to the console.

Odd. Do you have permissions to write to the space?

Yes I do, all other bedtools commands, especially intersect, appear to work fine when using BED files

That's odd. Let me see if I can get that error.

Two other users have had similar issues in the past:

https://code.google.com/p/bedtools/issues/detail?id=125
https://code.google.com/p/bedtools/issues/detail?id=150

Could you tell us what version you are using?  If you are not using 2.23.0, coulr you upgrade and retry? If it persists, could you provide a minimal test file?

Couldn't get the error with the small example I tried. Yes, could you please send your input files to me at<EMAIL_ADDRESS>Thanks. Below is what I tried:
[nek3d@d-128-109-67 data]$ samtools view a.bam
a1      0       chr1    11      255     10M     *       0       0       *       *
a2      16      chr1    101     255     100M    *       0       0       *       *
[nek3d@d-128-109-67 data]$
[nek3d@d-128-109-67 data]$ more b.bed
chr1    15      25
[nek3d@d-128-109-67 data]$ ~/2.23space/bt2-release/bin/bedtools intersect -a a.bam -b b.bed -v > out.bam
[nek3d@d-128-109-67 data]$ samtools view out.bam
a2      16      chr1    101     255     100M    *       0       0       *       *

I am using v2.23.0, I'll send you some sample files which reproduce the problem.

Hi James, I am SO sorry, I gave you the wrong e-mail address. It is<EMAIL_ADDRESS>I sincerely and profusely apologize for that.

Hi again,
Thanks for sending those files. I ran them, but can't reproduce the error. To confirm, you sent me sample.bam and sample.bed. In your first post, the bed file was named regions.bed.  I just want to make sure it was the same file, just with a different name. Thanks.

Hello again, yes the files I sent were just samples of the larger ones. The sample files reproduced the problem for me. I've just tried the command on another computer, running the same OS (CentOS 6.6), and it works correctly. I guess I'll spend some time debugging the environment to see what is causing the problem. Sorry to bother you. Thanks!

No, no bother.  Hmm, the exact same OS worked on one machine but not another? My next guess would be to check the compiler versions. I've seen seg faults in the past with one compiler but not another, typically due to an overwritten buffer. So at a command prompt, type:
g++ --version
To see which version of the compiler is installed.

My apologies, the OS it works on is not CentOS 6.6 but actually Ubuntu 14.04, running g++ 4.8.2 using bedtools v2.17.0. I updated bedtools to v2.23.0 and it works correctly. The compiler on the CentOS machine is g++ (GCC) 4.4.7. I probably won't be able to upgrade the compiler directly as I do not have root, but shall try a local install of the latest compiler.

Ok, thanks for checking that. When you say you upgraded from 2.17 to 2.23 and it worked, does that mean the original error you were getting went away?

No I mean that it worked on both v2.17.0 and v2.23.0 on Ubuntu, but did not work on v2.23.0 on CentOS

Thanks. I'm using gcc 4.8.3 - I'll try downgrading to 4.4x and see if it errors then. My OS is Red Hat 4.8.2-1, so if that gets the error, we'll know it's compiler dependent. If not, it's OS dependent.

Hi James, I was finally able to get gcc-4.4.7 installed. Unfortunately, the problem still doesn't show up. This is on a machine with Red Hat 4.8.2-1. Did the problem go away when you upgraded the compiler on the CentOS machine? If not, I'm afraid there isn't too much else I can do on my end.

Closing, as this appears to not be reproducible. Please reopen if proven otherwise!

Hi Aron,
This happened to me as well. My OS is Red Hat Enterprise Linux Server release 6.6. When I pipe output from sambamba view to bedtools, the same error occurred. Could you look into this?
sambamba view -f bam -L NA12878-5-noanalysis_blocks.bed NA12878-5/5_130327_iplat-sort.bam | bedtools intersect -abam - -b NA12878-5-noanalysis_blocks.bed -f 1.0 -nonamecheck > 5_130327_iplat-sort-noanalysis-prep.bam
terminate called after throwing an instance of 'std::runtime_error'
what():  BgzfStream::Write() - trying to write to non-writable IO device
Aborted
If I piped into samtools, the it is perfectly OK.
sambamba view -f bam -L NA12878-5-noanalysis_blocks.bed NA12878-5/5_130327_iplat-sort.bam | samtools view - | head -n 4
ERR091571.70803150      163     1       16821821        0       101M    =    16822096 376     GATTCCTCTTAGGAATATGGCTAAAAGCCGTTTCCCAGACTCCCTTGCAGTCAGATACGGCCATGTGGTCAAGTTCTGGTCCATAGAATTAAGACAGAAAG ?B@D;?DDFFDHDCFHGGGGGIJJJIEEE):1C9:CHIIJJGIIEHI@BDHFGG@GGIHDGEBA?HCEC?DB>DDEDA;AC>@>CCCCCA::(:>A(:AC: NM:i:1  MD:Z:29A71    AS:i:96 XS:i:96 RG:Z:5  XA:Z:1,-17244430,101M,1;
Or, the below command is completely ok as well.
sambamba view -f bam -L NA12878-5-noanalysis_blocks.bed NA12878-5/5_130327_iplat-sort.bam | samtools view - > test.txt
Thanks!!

Hello again, I'm still unable to output results in BAM format. Below is a very simple example which fails:
$ cat temp.bed
chr1	10068	10118	HISEQ:127:C7HJ7ANXX:1:1312:20025:49343/2	30	+
chr1	10074	10124	HISEQ:127:C7HJ7ANXX:1:1312:20025:49343/1	30	-
chr1	10163	10213	HISEQ:127:C7HJ7ANXX:1:1313:9157:25504/1	30	+
$ bedtools bedtobam -i temp.bed -g hg38.txt > tmp.bam
terminate called after throwing an instance of 'std::runtime_error'
what():  BgzfStream::Write() - trying to write to non-writable IO device
Aborted
If I run the command without directing to a file it prints the compressed BAM contents to the screen. I have access to write to the file system and every other command in bedtools works except trying to do intersections with BAM files (as was the original issue).
I compiled bedtools with gcc version 5.2.0 and I'm running on CentOS release 6.7

I encountered this error with bedtools 2.17 and 2.18 (didn't try later versions).
It only happens when writing to the NFS-mounted home directory, but not when writing to a local disk or a tmpfs, e.g. to /tmp.

@rekado thanks for this - I was having this error as well when writing to an NFS-mounted drive. can write to /tmp without error. not sure why this is.

Same problem here, cannot directly output bam to NFS storage, only bed. Local storage is fine.
My current workaround is pipe to stdout and then use samtools view.

The problem still exists in version 2.26.

I just experienced this, but only when utilizing NFS v4. Switching to v3 solves the issue.


Adding a worker node to an existing cluster in a VDC takes forever

VDC name: jetserthing
I launched a numbetr of marketsplace solutions (2) and wen I tried to launch the 3rd one, I got a message that told me that there were no more resources.

Wanted to add a worker node to see of that would lead to more capacity being available in the cluster that would allow me to launch this peertube container.  The progress screen was slow to progress and got stuck at this forever:

I pressed "Reload page" a couple of times but always got back to the same rotating page.
Also when CLI asking for the nodes to display the kbectl command hangs:
john@RescuedMac:~/.kube$ kubectl get nodes -w
NAME         STATUS   ROLES    AGE     VERSION
k3os-21973   Ready    <none>   3m41s   v1.19.4+k3s1
k3os-31241   Ready    <none>   22h     v1.19.4+k3s1
k3os-4283    Ready    master   22h     v1.19.4+k3s1
k3os-21973   Ready    <none>   5m31s   v1.19.4+k3s1
k3os-31241   Ready    <none>   22h     v1.19.4+k3s1
k3os-4283    Ready    master   22h     v1.19.4+k3s1
^Cjohn@RescuedMac:~/.kube$ kubectl get nodes -w
NAME         STATUS   ROLES    AGE     VERSION
k3os-21973   Ready    <none>   8m59s   v1.19.4+k3s1
k3os-31241   Ready    <none>   22h     v1.19.4+k3s1
k3os-4283    Ready    master   22h     v1.19.4+k3s1

After a very long time the update / addition of the worker node failed:

CLI kubectl command still shows
^Cjohn@RescuedMac:~/.kube$ kubectl get nodes -w
NAME         STATUS   ROLES    AGE   VERSION
k3os-21973   Ready    <none>   23m   v1.19.4+k3s1
k3os-31241   Ready    <none>   22h   v1.19.4+k3s1
k3os-4283    Ready    master   22h   v1.19.4+k3s1

And then hangs.

After a very long time the update / addition of the worker node failed:

CLI kubectl command still shows
^Cjohn@RescuedMac:~/.kube$ kubectl get nodes -w
NAME         STATUS   ROLES    AGE   VERSION
k3os-21973   Ready    <none>   23m   v1.19.4+k3s1
k3os-31241   Ready    <none>   22h   v1.19.4+k3s1
k3os-4283    Ready    master   22h   v1.19.4+k3s1

And then hangs.

Tried to add a large kuberneted worker, and now I get a message about no capacity in the farm to deliver that:

Tried to add a large kuberneted worker, and now I get a message about no capacity in the farm to deliver that:

With the second worker node deployed, I tried to deploy the peertube container and it worked.

With the second worker node deployed, I tried to deploy the peertube container and it worked.

we reserve capacity for 1 hour first for the node, then after the node is successfully deployed, we update the expiration for 2 weeks. that's why the node is shown when you list k8s nodes
this is taking long time because payments are failing and the chat keeps retrying depending on how many pools are being extended, each pool can take up to 5 minutes before it fails.

we reserve capacity for 1 hour first for the node, then after the node is successfully deployed, we update the expiration for 2 weeks. that's why the node is shown when you list k8s nodes
this is taking long time because payments are failing and the chat keeps retrying depending on how many pools are being extended, each pool can take up to 5 minutes before it fails.

will explicitly show that payments are failing and the chat is retrying

will explicitly show that payments are failing and the chat is retrying

let's add text to show that we're retrying

let's add text to show that we're retrying


Uncaught TypeError if Json.Decode is not imported (module loading order problem?)

I experience an issue which might be related to #683.
I created a SSCCE to illustrate the problem.
Im working on Windows 10. I use Elm 0.17.1, core 4.0.3.
Description:
I have a port module where I have defined all my app's ports. If this module does not import "Json.Decode", I get an Uncaught TypeError exception:

I am aware of this discussion. I do not use elm-webpack-loader.

It appears that the presence of a port module alone doesn't inform the compiler of the need to import Json.Decode. Moving the line import Ports exposing (..) below the Html imports fixes the issue as the Html modules import Json.Decode internally. Here's a succinct SSCCE for which you can examine the generated code and note the absence of Json.Decode
port module Main exposing (..)

port example : (String -> msg) -> Sub msg

value =
    1

@lukewestby, feel free to close issues and open nicer ones if you figure things out like this! I did this with #703.

Also running into this issue with React Native talking to Elm program via ports. The error is
Can't find variable _elm_lang$core$Json_Decode$string

Adding import Json.Decode solves the problem. See relevant code here.


fix: dialcode is changed to country code in entry form

Description
Motivation
https://github.com/reach4help/reach4help/issues/831
Testing Guidelines
Release Checklist

[ ] This code has unit tests
[ ] I have a plan to verify that these changes are working as expected after deploy
[ ] I have a plan for how to revert, rollback, or disable these changes if things are not working as expected

Security Checklist
This PR creates, modifies, or deletes:

[ ] API routes: API routes, parameters, or user authorization
[ ] Authentication: Authentication mechanism
[ ] Credentials: Server side credentials, or secrets in configuration / source code
[ ] Cryptography: Encryption, hashing, certificates, signatures, random numbers, etc.
[ ] User data: personal information handling, logs, error messages, etc.

Additional Notes

@Tapudp I am closing this pull request.  You should not have changed both files.  Please start again by creating a new fork.