problems installing node versions using nvm

Operating system and version:
macOS Big Sur 11.4
nvm debug output:

nvm --version: v0.38.0
$TERM_PROGRAM: vscode
$SHELL: /bin/zsh
$SHLVL: 1
whoami: 'adnanu'
${HOME}: /Users/adnanu
${NVM_DIR}: '${HOME}/.nvm'
${PATH}: /usr/local/opt/php@7.4/sbin:/usr/local/opt/php@7.4/bin:/usr/local/mysql/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/opt/php@7.4/sbin:/usr/local/opt/php@7.4/bin:/usr/local/mysql/bin
$PREFIX: ''
${NPM_CONFIG_PREFIX}: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'zsh 5.8 (x86_64-apple-darwin20.0)'
uname -a: 'Darwin 20.5.0 Darwin Kernel Version 20.5.0: Sat May 8 05:10:33 PDT 2021; root:xnu-7195.121.3~9/RELEASE_X86_64 x86_64'
checksum binary: 'shasum'
OS version: macOS 11.4 20F71
curl: /usr/bin/curl, curl 7.64.1 (x86_64-apple-darwin20.0) libcurl/7.64.1 (SecureTransport) LibreSSL/2.8.3 zlib/1.2.11 nghttp2/1.41.0
wget: not found
sed: /usr/bin/sed
cut: /usr/bin/cut
basename: /usr/bin/basename
rm: /bin/rm
mkdir: /bin/mkdir
xargs: /usr/bin/xargs
git: /usr/bin/git, git version 2.30.1 (Apple Git-130)
grep: /usr/bin/grep, grep (BSD grep) 2.5.1-FreeBSD
awk: /usr/bin/awk, awk version 20200816
nvm current: system
which node: /usr/local/bin/node
which iojs: iojs not found
which npm: /usr/local/bin/npm
npm config get prefix: /usr/local
npm root -g: /usr/local/lib/node_modules

nvm ls output:

->       system
iojs -> N/A (default)
node -> stable (-> N/A) (default)
unstable -> N/A (default)
lts/* -> lts/fermium (-> N/A)
lts/argon -> v4.9.1 (-> N/A)
lts/boron -> v6.17.1 (-> N/A)
lts/carbon -> v8.17.0 (-> N/A)
lts/dubnium -> v10.24.1 (-> N/A)
lts/erbium -> v12.22.3 (-> N/A)
lts/fermium -> v14.17.3 (-> N/A)

How did you install nvm?
followed instructions here: https://github.com/nvm-sh/nvm
What steps did you perform?
nvm install 12.13.0
What happened?
Downloading and installing node v12.13.0...
Downloading https://nodejs.org/dist/v12.13.0/node-v12.13.0-darwin-x64.tar.xz...
##################################################################################################################################################################################### 100.0%
Computing checksum with shasum -a 256
Checksums matched!
258938ef66a2a49a4a400554a6dce890226ae34c
rm: no such file or directory: pkg.*.tar.gz
What did you expect to happen?
nvm to install the new version... but it keeps running into that issue.
Is there anything in any of your profile files that modifies the PATH?

.zshrc file contents:
export PATH="/usr/local/mysql/bin:$PATH"
export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh"  # This loads nvm
[ -s "$NVM_DIR/bash_completion" ] && . "$NVM_DIR/bash_completion"  # This loads nvm bash_completion
export PATH="/usr/local/opt/php@7.4/bin:$PATH"
export PATH="/usr/local/opt/php@7.4/sbin:$PATH"

If you are having installation issues, or getting "N/A", what does curl -I --compressed -v https://nodejs.org/dist/ print out?

Trying <IP_ADDRESS>...
TCP_NODELAY set
Connected to nodejs.org (<IP_ADDRESS>) port 443 (#0)
ALPN, offering h2
ALPN, offering http/1.1
successfully set certificate verify locations:
CAfile: /etc/ssl/cert.pem
CApath: none
TLSv1.2 (OUT), TLS handshake, Client hello (1):
TLSv1.2 (IN), TLS handshake, Server hello (2):
TLSv1.2 (IN), TLS handshake, Certificate (11):
TLSv1.2 (IN), TLS handshake, Server key exchange (12):
TLSv1.2 (IN), TLS handshake, Server finished (14):
TLSv1.2 (OUT), TLS handshake, Client key exchange (16):
TLSv1.2 (OUT), TLS change cipher, Change cipher spec (1):
TLSv1.2 (OUT), TLS handshake, Finished (20):
TLSv1.2 (IN), TLS change cipher, Change cipher spec (1):
TLSv1.2 (IN), TLS handshake, Finished (20):
SSL connection using TLSv1.2 / ECDHE-RSA-CHACHA20-POLY1305
ALPN, server accepted to use h2
Server certificate:
subject: OU=Domain Control Validated; OU=PositiveSSL Wildcard; CN=*.nodejs.org
start date: Oct 21 00:00:00 2019 GMT
expire date: Jan 18 23:59:59 2022 GMT
subjectAltName: host "nodejs.org" matched cert's "nodejs.org"
issuer: C=GB; ST=Greater Manchester; L=Salford; O=Sectigo Limited; CN=Sectigo RSA Domain Validation Secure Server CA
SSL certificate verify ok.
Using HTTP2, server supports multi-use
Connection state changed (HTTP/2 confirmed)
Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
Using Stream ID: 1 (easy handle 0x7ffb2c809200)

HEAD /dist/ HTTP/2
Host: nodejs.org
User-Agent: curl/7.64.1
Accept: /
Accept-Encoding: deflate, gzip

Connection state changed (MAX_CONCURRENT_STREAMS == 256)!
< HTTP/2 200
HTTP/2 200
< date: Thu, 08 Jul 2021 01:45:37 GMT
date: Thu, 08 Jul 2021 01:45:37 GMT
< content-type: text/html
content-type: text/html
< cache-control: max-age=14400
cache-control: max-age=14400
< cf-cache-status: HIT
cf-cache-status: HIT
< age: 9515
age: 9515
< expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
< vary: Accept-Encoding
vary: Accept-Encoding
< server: cloudflare
server: cloudflare
< cf-ray: 66b5a2594adb42d1-LAX
cf-ray: 66b5a2594adb42d1-LAX
< content-encoding: gzip
content-encoding: gzip

<

Connection #0 to host nodejs.org left intact
Closing connection 0

What happens if you’re using a real terminal, as opposed to vscode’s?

hey ljharb,
Thanks for getting back to me on this, I think I figured it out and got it going.
I had node/npm installed on my system otherwise before installing nvm, it somehow didn't like that. I uninstalled node/npm and then tried installing the versions i needed through nvm, it let me install them then, everything works as expected now.

Although that should work fine, sometimes complex PATHs can make that tricky. Glad you figured it out.


sdl-* pkg-config packages

Saw the following errors when building sdl2-* stuff on the nightly.
sdl2-gfx: BuildFailureException Process exited with ExitFailure 1: ./Setup configure --package-db=clear --package-db=global --package-db=/var/stackage/work/builds/nightly/pkgdb --libdir=/var/stackage/work/builds/nightly/lib --bindir=/var/stackage/work/builds/nightly/bin --datadir=/var/stackage/work/builds/nightly/share --libexecdir=/var/stackage/work/builds/nightly/libexec --sysconfdir=/var/stackage/work/builds/nightly/etc --docdir=/var/stackage/work/builds/nightly/doc/sdl2-gfx-0.2 --htmldir=/var/stackage

Setup: The pkg-config package 'SDL2_gfx' version >=1.0.1 is required but it
could not be found.

sdl2-image: BuildFailureException Process exited with ExitFailure 1: ./Setup configure --package-db=clear --package-db=global --package-db=/var/stackage/work/builds/nightly/pkgdb --libdir=/var/stackage/work/builds/nightly/lib --bindir=/var/stackage/work/builds/nightly/bin --datadir=/var/stackage/work/builds/nightly/share --libexecdir=/var/stackage/work/builds/nightly/libexec --sysconfdir=/var/stackage/work/builds/nightly/etc --docdir=/var/stackage/work/builds/nightly/doc/sdl2-image-2.0.0 --htmldir=/var/stac

Setup: The pkg-config package 'SDL2_image' version >=2.0.0 is required but it
could not be found.

sdl2-mixer: BuildFailureException Process exited with ExitFailure 1: ./Setup configure --package-db=clear --package-db=global --package-db=/var/stackage/work/builds/nightly/pkgdb --libdir=/var/stackage/work/builds/nightly/lib --bindir=/var/stackage/work/builds/nightly/bin --datadir=/var/stackage/work/builds/nightly/share --libexecdir=/var/stackage/work/builds/nightly/libexec --sysconfdir=/var/stackage/work/builds/nightly/etc --docdir=/var/stackage/work/builds/nightly/doc/sdl2-mixer-0.1 --htmldir=/var/stacka

Setup: The pkg-config package 'SDL2_mixer' version >=2.0.0 is required but it
could not be found.

https://github.com/fpco/stackage/commit/50a89ebd5e7e420ed07c1c5f79424f3992fb9985 and https://github.com/fpco/stackage/commit/a2b809329d13f155c95a1c1a7e1be03b6c5c70de were for this. I didn't have time to follow up after that ... not sure what needs to be done beyond that?

The build is working correctly with the new docker image.


[Marker.io] Ghost CTA should be white

Can we change the ghost CTA to white? This pink doesn't pass AA at this size.
(I've asked Christina/Adrien in figma too)

Reported by: Holly<EMAIL_ADDRESS>- Contact via Marker.io
Source URL: https://na13-prezzee.webflow.io/en-us
Issue details: Open in Marker.io
Device typedesktopBrowserChrome <IP_ADDRESS>Screen Size2048 x 1152OSOS X 14.7.1Viewport Size1561 x 912Pixel Ratio@2x

🤖 Marker.io bot - This issue was marked as ✅ Resolved in Marker.io


[C++] Warning on Windows (ptrdiff_t converted to int)

Hi,
I'm compiling with the 1.0.0 PGV on Windows 10 64bits.

Visual Studio 17 2022
The C/CXX compiler is MSVC 19.35.32215.0

A Warning is fired which is (tranlated):
validate/validate.h(171,50): warning C4244: convert 'ptrdiff_t' to 'int', possible data loss...
The warning is not fired on GCC 10 in Linux Ubuntu 20.04.
Regards

Hello, thanks for your interest! As explained in the README, this project is in maintenance mode. We recommend upgrading to protovalidate.
I'm going to go ahead and close this issue, but feel free to open one against protovalidate if needed. Thanks!


[MGDOBR-535] Replace ValuesIn with typed filters

MGDOBR-535
Stuck due to https://issues.redhat.com/browse/DROOLS-6897
PR owner checklist
Please make sure that your PR meets the following requirements:

[ ] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[ ] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[ ] All new functionality is tested
[ ] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

@robpblake @r00ta a timely remind for your reviews of https://github.com/5733d9e2be6485d52ffa08870cabdee0/sandbox/pull/599.. as it'll possibly be impacted by this PR and vice-versa.

Waiting for green CI - Managed connector issue being addressed by that team


coi-serviceworker.js conflicts with index.service.worker.js when building as a PWA

Exporting the project as a PWA generates a separate service worker script as index.service.worker.js. This results in two separate service worker scripts. My understanding is that a given page ought to only have one.
Is it possible to integrate the behavior of coi-serviceworker.js with index.service.worker.js in case both are desired?

Good news on this, it sounds like the functionality of this plugin is effectively going to be built-in to the PWA functionality in Godot 4.3.
https://godotengine.org/article/progress-report-web-export-in-4-3/#easy-coop-coep-pwa-for-threaded-builds


Add benchmarks for BYTE_STREAM_SPLIT encoded Parquet FIXED_LEN_BYTE_ARRAY data

Which issue does this PR close?
Closes #6203.
What changes are included in this PR?
Adds new benches to arrow_reader and encoding in parquet/benches.
Are there any user-facing changes?
No.

Example usage and output:
% cargo bench -p parquet --bench encoding --all-features
    Finished `bench` profile [optimized] target(s) in 0.71s
     Running benches/encoding.rs (target/release/deps/encoding-fa07f9604f013e9b)
encoding: dtype=f32, encoding=BYTE_STREAM_SPLIT
                        time:   [44.223 µs 44.519 µs 44.860 µs]
                        change: [-1.0311% +0.0307% +1.0725%] (p = 0.95 > 0.05)
                        No change in performance detected.

dtype=f32, encoding=BYTE_STREAM_SPLIT encoded as 65536 bytes
decoding: dtype=f32, encoding=BYTE_STREAM_SPLIT
                        time:   [38.790 µs 38.997 µs 39.257 µs]
                        change: [-5.5214% -1.7711% +1.2431%] (p = 0.36 > 0.05)
                        No change in performance detected.
Found 15 outliers among 100 measurements (15.00%)
  4 (4.00%) high mild
  11 (11.00%) high severe

encoding: dtype=f64, encoding=BYTE_STREAM_SPLIT
                        time:   [108.12 µs 108.47 µs 108.85 µs]
                        change: [-54.122% -46.682% -39.381%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 8 outliers among 100 measurements (8.00%)
  4 (4.00%) high mild
  4 (4.00%) high severe

dtype=f64, encoding=BYTE_STREAM_SPLIT encoded as 131072 bytes
decoding: dtype=f64, encoding=BYTE_STREAM_SPLIT
                        time:   [81.265 µs 81.526 µs 81.789 µs]
                        change: [-10.273% -9.2926% -8.3289%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 4 outliers among 100 measurements (4.00%)
  1 (1.00%) high mild
  3 (3.00%) high severe

encoding: dtype=parquet::data_type::FixedLenByteArray, encoding=BYTE_STREAM_SPLIT
                        time:   [46.957 µs 47.171 µs 47.437 µs]
                        change: [-26.632% -23.632% -20.584%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 4 outliers among 100 measurements (4.00%)
  3 (3.00%) high mild
  1 (1.00%) high severe

dtype=parquet::data_type::FixedLenByteArray, encoding=BYTE_STREAM_SPLIT encoded as 32768 bytes
decoding: dtype=parquet::data_type::FixedLenByteArray, encoding=BYTE_STREAM_SPLIT
                        time:   [384.39 µs 385.17 µs 386.00 µs]
                        change: [-11.955% -10.898% -9.8426%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 10 outliers among 100 measurements (10.00%)
  5 (5.00%) high mild
  5 (5.00%) high severe

encoding: dtype=parquet::data_type::FixedLenByteArray, encoding=BYTE_STREAM_SPLIT #2
                        time:   [270.75 µs 274.65 µs 278.81 µs]
                        change: [-2.4969% -0.9976% +0.6427%] (p = 0.24 > 0.05)
                        No change in performance detected.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild

dtype=parquet::data_type::FixedLenByteArray, encoding=BYTE_STREAM_SPLIT encoded as 262144 bytes
decoding: dtype=parquet::data_type::FixedLenByteArray, encoding=BYTE_STREAM_SPLIT #2
                        time:   [542.87 µs 543.70 µs 544.61 µs]
                        change: [-16.128% -13.963% -11.712%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 6 outliers among 100 measurements (6.00%)
  6 (6.00%) high severe

% cargo bench -p parquet --bench arrow_reader --all-features -- BYTE_STREAM
    Finished `bench` profile [optimized] target(s) in 0.31s
     Running benches/arrow_reader.rs (target/release/deps/arrow_reader-f970ea21ed6b3164)
arrow_array_reader/BYTE_STREAM_SPLIT/Decimal128Array/byte_stream_split encoded, mandatory, no NULLs
                        time:   [3.4332 ms 3.4747 ms 3.5191 ms]
                        change: [+4.5989% +5.7616% +7.1541%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 2 outliers among 100 measurements (2.00%)
  2 (2.00%) high mild
arrow_array_reader/BYTE_STREAM_SPLIT/Decimal128Array/byte_stream_split encoded, optional, no NULLs
                        time:   [3.3593 ms 3.3818 ms 3.4055 ms]
                        change: [-5.2748% -4.1025% -2.8714%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
  5 (5.00%) high mild
arrow_array_reader/BYTE_STREAM_SPLIT/Decimal128Array/byte_stream_split encoded, optional, half NULLs
                        time:   [3.0576 ms 3.0768 ms 3.0976 ms]
                        change: [-5.6299% -4.1956% -2.8196%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild

arrow_array_reader/BYTE_STREAM_SPLIT/Float16Array/byte_stream_split encoded, mandatory, no NULLs
                        time:   [823.94 µs 827.82 µs 832.00 µs]
                        change: [-5.9189% -4.2386% -2.7745%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high severe
arrow_array_reader/BYTE_STREAM_SPLIT/Float16Array/byte_stream_split encoded, optional, no NULLs
                        time:   [845.52 µs 857.75 µs 873.06 µs]
                        change: [-2.3817% -1.0937% +0.1853%] (p = 0.10 > 0.05)
                        No change in performance detected.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high severe
arrow_array_reader/BYTE_STREAM_SPLIT/Float16Array/byte_stream_split encoded, optional, half NULLs
                        time:   [1.5441 ms 1.5657 ms 1.5884 ms]
                        change: [+1.2815% +2.5329% +3.8183%] (p = 0.00 < 0.05)
                        Performance has regressed.

Thanks again @etseidl


1.18?

https://github.com/ChloeDawn/BeaconOverhaul/releases/tag/1.5.0+1.18 https://www.curseforge.com/minecraft/mc-mods/beaconoverhaul/files/3561472
In the future, please do not use the issue tracker to request updates/ports.


Reproducible complete example with form included

I'm struggling to understand the workflow of the package. Would highly appreciate it if you a reproducible complete example with form included may be provided. Thanks

Hi!  The file example.qmd provides a description of how to configure the forms extension on a webpage, and if you render it to HTML, it should also provide a reproducible example of the forms in operation.  Is there part of it that isn't working properly or is unclear?

Got it. Thanks.


The string representation for Lazy should be Lazy([OBJ_NAME])

Currently if the return values from a stage don't match what's expected, and all the outputs are marked lazy, the error message only contains a bunch of <curifactory.caching.Lazy at 0x...> instances, instead of explicitly saying what object is what. (see image below.)

To make it easier to read, the __repr__ for a lazy instance should output Lazy('my_object'), where my_object is the name of the output.

Closed in https://github.com/ORNL/curifactory/commit/b35d8744fc05ee019e2386676e04e46a3563bc2e


On opening data tab of table, set focus on filter

Is your feature request related to a problem? Please describe.
When opening the data tab of a table, the usual thing I want to do when I type something, is to filter the table. However, the initial focus of the tab is on the first field in the table. So if I type something, instead of filtering I edit the existing data. I have done this numerous times, when rapidly switching between tables while searching for something.
IMHO that is not useful in any way. If I want to change some data, the chances that it is exactly the first field are almost 0, so I would have to change the field anyway.
Describe the solution you'd like
When opening the data tab of a table (or switching to it from another table), set the focus/cursor in the filter input, rather than the first field of actual data.

Thank you for sharing your use case. However, I think, it's controversial. Some users don't use this filter text field at all. Some open the data viewer to observe data only, and build filters automatically with UI.

Some open the data viewer to observe data only, and build filters automatically with UI.

Sure, but they wouldn't care either way, would they?

I think, that having focus on filter text field first is a usability issue for blind people

Hello @Khartir
You can try to use the shortcut to open the filter: Ctrl+Alt+Shift+T  for Windows, ⌃⌥⇧T for macOS.
You can change it to something more useful in Preferences —> Keys:


[Suggestion] Use GitHub actions after Public Alpha testing

Using GitHub Actions would auto-build binaries for development builds and I think it'll improve feedback and development for the program.

GyroShell is not far enough into development to justify an actions pipeline. When public alpha launches, I'll set up the pipeline.


Enhancement - Test coverage of groovy scripts

Would like to be able to drill into a node and see the groovy script and see the lines of script highlighted in green that are covered. Would like groovy script lines added to calculation of the coverage percentage.

Hi @madmax25,
thank you for the idea. I'm not sure this is intended by this extension. We speak about process coverage, not code coverage.
I think coverage of script code (Groovy, Javascript or any other) is out of scope of this extension for the moment.
Cheers,
Simon

Process coverage is a good start, but not a final answer. To achieve high coverage on the scripts, a gap will still need to be filled, perhaps by another open source project. @ingorichtsmeier


Timesheet - 12/04/2021 - Truyen Hoang

Task 1

Setup ECR reporitory for backend
Setup ECS cluster by terraform with fargate to saving cost
Refactor terraform project to support multiple environment
Test ECR by build backend and push to ECR
Try to deploy backend to ECS

Proof of Work

https://us-west-2.console.aws.amazon.com/ecs/home?region=us-west-2#/clusters/NonProduction/services
https://github.com/thenewboston-developers/iac/pull/1

Time Spent
4 hours

Task 2
Support and guide JustinN to access to dev environment
Proof of Work

N/A

Time Spent
1 hours

Total Time Spent
5 hours
Account Number
e3a94381f8db207ddad931391886d611d6f4c060d0db2b0e373738e2f4db96d6

Hey, @buckyroberts can you verify Task 1? I don't have a permission role for AWS.
Also, @truyenhv regarding Task 2, where you guys had the conversation? Discord or somewhere else? We need to have some proof for that too.

Hey yes @ma2b0043. We work through discord and zoom

Approved

Approved


Add test data & remove test data fetch

See explanation comment here: https://github.com/anchore/packageurl-go/pull/9#issuecomment-1371180663

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
25 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-10.8%) to 80.087%

Files with Coverage Reduction
New Missed Lines
%

packageurl.go
25
80.09%

Totals

Change from base Build<PHONE_NUMBER>:
-10.8%

Covered Lines:
185

Relevant Lines:
231

💛  - Coveralls


Date picker value reset

Fix for being able to delete date from datepicker field. At the moment typing is disabled.

Datepicker resets when the selected day is selected again. Closing.


Constrain sqlite3 gem for Rails

The v2.0.0 release of the sqlite3 gem today seems to be causing some issues with Rails main (example). Let's temporarily constrain it until that is resolved.

It's also affecting the normal Gemfile so I've update it too (was likely only passing due to caching).


CSharpCompiler not working on Linux

CSharpCompiler doesn't seem to work for Oxide Rust linux server on Ubuntu 12.04
./CSharpCompiler: error while loading shared libraries: libmonoboehm-2.0.so.1: cannot open shared object file: No such file or directory
Steps to reproduce:
wget https://raw.githubusercontent.com/OxideMod/Snapshots/master/Oxide-Rust_Linux.zip
Unzip, run ./CSharpCompiler
If i install the missing library that changes to:
Corlib not in sync with this runtime: expected corlib version 111, found 138.
Loaded from: mscorlib.dll

CSharpCompiler is not meant to be manually ran. Just install it and let Oxide run it when it needs to.

The problem is: the plugins don't compile.
21:20 [Error] InfoPanel plugin failed to compile!
21:20 [Error] Compiler disconnected.

No, that would be a missing system dependency most likely. See http://oxidemod.org/threads/rust-server-on-linux-natively.16255/#post-182752.

Like I wrote in the forum:
I've found the solution: Oxide doesn't seem to respect the custom LD_LIBRARY_PATH I've set.
After I copied the libraries to rust_dedicated/RustDedicated_Data/Managed/x86 and rust_dedicated/RustDedicated_Data/Managed/x64 the plugins do compile now.

The libraries should already exist in those paths, if not then the install isn't complete.

I do mean the System libraries like libc6, not the libmono-thing I mentioned at the start. In my current setup I dont have them in the normal System library path, but in an extra steam folder. Works fine for rust, but the plugins don't compile this way.
Anyways it's solved, thanks for pointing me in the right direction.
----- Ursprüngliche Nachricht -----
Von: "Luke Spragg"<EMAIL_ADDRESS>Gesendet: 09.03.2016 21:43
An: "OxideMod/Oxide"<EMAIL_ADDRESS>Cc: "XL-Reaper"<EMAIL_ADDRESS>Betreff: Re: [Oxide] CSharpCompiler not working on Linux (#651)
The libraries should already exist in those paths, if not then the install isn't complete.
—
Reply to this email directly or view it on GitHub.


/code command needs to be a rich embed

We are in the process of moving our commands  to a Rich Embed feature that makes the commands more obvious, and take up less space in the chat. An example of this is below.

Our /code command will need to have a title, a description, and a regular field for the typical code snippets markdown syntax (the backticks). Focus on changing the format, but not changing the copy where possible. We will want additional inline-fields for Repl.it (Javascript/Ruby), CodeSandBox (Webpack/React), and Codepen (Basic HTML/CSS/JS).
Completion Criteria:

We leverage the Discord.MessageEmbed() constructor
We have an embed with the aforementioned fields
updated snapshot
correct grammar and spelling
All tests pass

I'll take a crack at this

This one will be a little more involved, and require you run an instance of the bot so we can see what the embed looks like. You good with that?

Like run an instance of the bot in my own server to see what the embed looks like?
Is there any time-bound on this?

Yep, we want to make sure we're good with the layout or if we want to make changes. No, not particularly.

Yup, I'm good with it.

This is the basic embed that I came up with. I have fill in the aforementioned fields, but I
am not too sure this is the look that we want.
I'd appreciate some feedback so I can implement

There’s a typo with “snippets”

Oops, yeah. I will fix it before committing it.
But overall the look of the embed is okay?

I like the embed. I know you said you’ll proofread, but let’s change “to the others” in the first sentence to “with others”.
Let’s change the next header to “Sharing Code On Discord”.
I think showing multi line code and then showing it again with syntax highlighting is redundant, we want people to highlight by default so let’s just combine the first bit of text but use the syntax highlighting example. I know I said we wouldn’t change the copy but since we’re already here and making this command better, let’s go for it.

ah, I actually just realized that we also mention users in this command. Let’s not forget to add that capability too. It can go in the message under the first header.

Closed by #144


Update command-line-tools.rst

Channel search: https://github.com/mattermost/mattermost-server/pull/10093
Team rename: https://github.com/mattermost/mattermost-server/pull/10083
Webhook show: https://github.com/mattermost/mattermost-server/pull/10123

@jasonblais  - feedback addressed.  Thank you!


Pattern: Media - Prod level version

This epic contains links to the issues needed to create the Caption sub-pattern needed for the Solutions page template

Iceboxing all Media component issues.

Decided to close all of the Media manager issues.


Revert "[Typechecker] Diagnose key paths with contextual root type but no leading dot"

Reverts apple/swift#30164

@theblixguy Let's revert and go with parser identification of such cases, sounds good?

@swift-ci please smoke test

+1 thank you!

Sounds good! I’ll put up a new PR shortly!

No worries, take your time and thank you again!

@theblixguy Just wanted to ask you whether you are still interested in working on alternative solution for this?

@xedin Yes, sorry been a little busy in the past few days. I’ll put up a PR for the other approach shortly!

@theblixguy No worries, take your time!


Add direct access option to avoid caching certain paths

What changes are proposed in this pull request?
Datalake formats such as Iceberg requires frequent changes to certain files, it is better not to cache them at all to avoid frequent invalidations.
Why are the changes needed?
for correct functionality using iceberg
Does this PR introduce any user facing changes?
one user property that is by default empty, alluxio.user.file.direct.access

@dbw9580 @apc999 as I am adding more to this PR, I find that I am changing all the entry point that path-specific configuration is changing. I am not sure that this change is any less complex than path-specific config when it is fully done.

alluxio-bot, cherry-pick this to branch-2.10 please

@yuzhu Do we need this in main? Thanks!

Not needed in main, will require a very different implementation.


GLES, How to add anti-aliasing ?

Platform : Android
Solution :  Face Mesh
In effect_renderer.cc, I changed dst_texture's color container from glTexImage2D() to glTexStorage2DMultisample(), but it does not work.

@wingdi,
Are you still looking for the resolution on this issue? If yes, then would you let us know the complete steps followed to reproduce the issue from our end. Feel free to close if no longer issue from your end.


CheckDocumentType in Table 179 is not public

Hi,
please expose function CheckDocumentType in table 179 Reversal Entry.
Thank you for your help.

I assume you mean in the Italian version, as I can only find that function in that version.

Yes, the function come from the italian version, but I found that in the daily build of business central.
I assume that it will be present in the release, that why I asked to expose it.

It will be part of the release, yes. But only the Italian one. I will expose that function for you :)


Create instalable package of the FRAME

Provide built packages of the FRAME for normal users to install and use.
Packages to consider:

PyPi (platform independent)
Snap (linux only)
Exe installer (windows only)
compressed standalone archive (platform independent)

PyPI, snap and source release are ready
Exe installer is currently postponed


popmon working with hgr v1.0.23

popmon working with hgr v1.0.23

removal of all HistogramContainer code
all unit tests working
fix all flake8 errors

Tests are failing because of this issue: https://github.com/python/importlib_metadata/issues/293

Thanks Simon!


Sign up request getting stuck

Describe the bug
Sign up request getting stuck similar to this issue https://github.com/better-auth/better-auth/issues/320
I suspect something similar to this is the issue. But have removed everything except fastify cors and better-auth routes without luck.
To Reproduce
    const fastify = Fastify({});
    fastify.register(cors, {
      origin: ['http://localhost:3000'],
      credentials: true,
    });
    fastify.all('/auth/*', (req, res) => {
      return toNodeHandler(auth)(req.raw, res.raw);
    });
    await fastify.listen({
      port,
    });

export const auth = betterAuth({
  database: prismaAdapter(db, {
    provider: 'postgresql',
  }),
  baseURL: `${process.env.NEXT_PUBLIC_API_URL}/auth`,
  logger: {
    level: 'debug',
  },
  trustedOrigins: ['http://localhost:3000', 'http://localhost:3333'],
  emailAndPassword: {
    enabled: true,
  },
});

Using signUp method
Request get stalled

Expected behavior
The request should return 200 or any other status
Screenshots

Desktop (please complete the following information):

OS: MacOS 14.6.1
Browser Chrome

Additional info
The logger does not seem to work since I don't get any logs (atleast not for sign up, /auth/get-session or /auth/ok requests)

Attempt 2
Tried to use fastify-standard-request-reply to convert req and reply to Request and Response. No stalled request but I get errors that email is invalid. I assume the body is not picked up.
import cors from '@fastify/cors';
import { auth } from '@repo/auth/server/auth';
import Fastify from 'fastify';

import {
  createStandardRequest,
  sendStandardResponse,
} from 'fastify-standard-request-reply';

import { logger } from './utils/logger';

const startServer = async () => {
  logger.info('Starting server');
  try {
    const fastify = Fastify({});
    fastify.register(cors, {
      origin: ['http://localhost:3000'],
      credentials: true,
    });
    fastify.all('/auth/*', async (req, reply) => {
      const request = createStandardRequest(req, reply);
      const response = await auth.handler(request);
      await sendStandardResponse(reply, response);
    });
    await fastify.listen({
      port: 3333,
    });
  } catch (error) {
    console.log('Sad hest');
  }
};

startServer();

me with express got stucked too, lol

I was getting the same behavior. In my case I tracked it down to this line in a dependency: https://github.com/Bekacru/better-call/blob/53df74c594cb4f6ed557f3a7a59a2effa7806b30/src/utils.ts#L31
It just hangs there for me when trying to send magic links.
Decided to just spin up a separate express app just for this lib. It worked there immediately.

Rolled my own auth in the end but this is great. Hope to give this library another shot for another project!


Secrets app giving other OTPs than password manager with the same secret

I have issues with getting the secrets app to work on my Nitrokey 3C NFC, or at least it is not working as I would expect it to. I did the following:

$ nitropy nk3 version
Command line tool to interact with Nitrokey devices 0.4.36
v1.3.1

$ nitropy nk3 secrets register MyApp $(base32 MyApp)
Command line tool to interact with Nitrokey devices 0.4.36
Please touch the device if it blinks
Current Password (8 attempts left):

$ nitropy nk3 secrets list
Command line tool to interact with Nitrokey devices 0.4.36
Please touch the device if it blinks
Current Password (8 attempts left):
b'MyApp'

$ nitropy nk3 secrets get MyApp
Command line tool to interact with Nitrokey devices 0.4.36
Please touch the device if it blinks
Current Password (8 attempts left):
Timestamp: 2023-04-11T08:45:56<PHONE_NUMBER>), period: 30
204269

The file 'MyApp' contains the trimmed OTP secret.
The OTP 204269 is different from what my password manager shows and is invalid for login.
What am I doing wrong?

The service might be using a different hash algorithm.  You can use the --hash option for register to select a different algorithm (default: SHA1).  If this does not help, we would need to know the service you use.  If you don’t want to share it here, you can also send a mail to me directly or to<EMAIL_ADDRESS>

I tried Gitlab and GitHub, both are wrong. I also added the secret to my YubiKey 5 with the default options (sha1, 6 digits, totp), the codes from the YubiKey are different from the Nitrokey codes and valid for login. can you reproduce this with a GitHub account?

I just tested it on GitHub and I did not have any issues.

the only difference between the Nitrokey and YubiKey procedure is that for Nitrokey I have to provide the secret base32 encoded whereas the YubiKey just takes the secret string.

How do you determine the secret?  The secret in the QR code generated by GitHub already is Base32-encoded.  Maybe you just have to drop the encoding step?

skipping the base32 part fixed it for GitHub but not for Gitlab.

Hi!
The OTP implementation used in the Nitrokey 3 firmware was tested against the test vectors provided in the specification for SHA1, so I expect this is some pynitrokey's UI/UX issue.
Can you provide the rest of the URL (with the secret removed) for Gitlab? Perhaps other parameters have changed as well, like the mentioned algorithm or the number of digits.
I think it would be a good idea to have the whole URL interpreted by pynitrokey directly, instead of asking user to divide it properly.
Moving to pynitrokey project.


zoomAtPointBy not working

Hello,
panBy, zoomBy, zoom and friends all work, but zoomAtPoint and zoomAtPointBy don't. Calling them does nothing.
Any ideas what are possible reasons for this? Not sure where to start digging

Hi,
Please provide a reproducible example as described here. Otherwise it's really hard to help.

Closing as not further details provided


Check whether a given point lies inside a triangle or not

Proposal
(A clear and concise description of what the proposal is.)

@ShrutiOmer This is already present Please check #3741


Update deploy.js

Coverage remained the same at 100.0% when pulling 6c12c417d73e7f04dbbdf1d51b5adf1ae24b312b on harrymitchinson-patch-1 into fc9a5a2c37b40ec6ea9811e6b7856b80eb9a9739 on master.


Update transaction memo

the USD wallet wording is fixed.
the BTC wallet wording is still incorrect

this should say "Received from" not "Received to"

@islandbitcoin
Updated transaction memo to Received and Sent.
Change Scan to Scan QR


show addresses as Google maps snippets

on user profiles the company address should be shown as a Google maps snippet beneath the current address section

need to check how to include the snippet if only the address or parts of the address is available
need to check how much information is needed for the snippet


U

Search Terms

Suggestion

Use Cases

Examples

Checklist
My suggestion meets these guidelines:

[ ] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[ ] This wouldn't change the runtime behavior of existing JavaScript code
[ ] This could be implemented without emitting different JS based on the types of the expressions
[ ] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[ ] This feature would agree with the rest of TypeScript's Design Goals.

You might want to fill out the issue template.... this time..? #41174

😓


CR3 Update

This patch provides an improved CR3 implementation

@connojd @JaredWright
This is not complete yet. I still have to add the helper functionality, but I wanted to get a sense for Codecov with these changes. Also, this would be a great time to review this code.


Scanning for projects on Windows takes forever

I used this tool on Linux and liked it, so was also trying to use it on my Windows machine. However, when I ran it on a (relatively large) git projects directory, it takes a very long time - not sure exactly how long yet, but at least 5 minutes and still going.
Running in verbose mode doesn't give me any hint as to what's taking so long - I definitely have some projects with large subdirectory trees, but I'd still not expect it to take near this long. I can do some local debugging if you want any more information.

Hi! Could you give some more information on the context concerning the following points:

Is the projects directory you want to clean located on HDD or SSD storage?
What is the active disk utilization shown in the Task-Manager? (The active time value for the drive in question on the performance tab)
How many files and subdirectories are in your projects directory? (This should be shown in the properties of the project directory)

As a reference, scanning my projects directory on fast SSD NVME storage with about 193k files and 26748 directories takes about 3 seconds. I tested this with Windows 11 and a Ryzen 7900x CPU.
If the disk utilization on your machine is at or near 100%, it might just be reaching the limits of the scanning technique that is used here. This tool is using the default std directory listing functions with optionally (default enabled) multithreading. Especially the multithreading could potentially even harm the performance when you are using a HDD. You could try to set the number of threads manually to a lower number using -t

I'm using an HDD, disk utilization is at 100%, and the total number of files is ~500k, directories ~75k, though I should be excluding ~376k and ~53k of those, respectively, unless the std method still scans those excluded directories?

Thanks for the answer. The full scan on HDD storage for this many directories will definitely take a pretty long time, especially if the cache is cold. To do that kind of a scan in a more reasonable time, the program would have to do something like what WizTree or Everything is doing. I would love to look into that at some point because it is pretty interesting in general but that is not really in scope for this project right now.
More relevant to your situation, you stated that it should be excluding a major part of that, which indeed should make it significantly faster. I assume you might have used something like cargo clean-all --ignore myprojects/cpp --ignore myprojects/javascript myprojects in order to clean the myprojects directory but skip over certain subdirectories and their respective fs trees.
If this is the case, that did not do what you were expecting. The help entry for --ignore is currently:
--ignore <IGNORE>    Directories that should be ignored by default, including subdirectories.
This will still detect the projects in those directories, but mark them to not be cleaned

The important part here being that projects will still be scanned, just not marked for cleaning.
I will admit that the argument is confusingly named and the possibility to actually skip over subdirectories completely would be extremely helpful here. The --ignore argument was designed together with the interactive mode in order to provide the same way of not cleaning certain projects while not using interactive mode. And since I personally have my stuff on pretty fast SSD storage and my project dir is a lot smaller, I never ran into the situation where fully skipping subdirs would make a difference.
Actions
I'll add a --skip option that will take arguments like --ignore but cause the scanner to fully skip over the specified directories, speeding up the scan time in the progress. While adding that I'll try to make the argument descriptions in the help page be clear on what --ignore and --skip do exactly and how it is different. I still want to keep the --ignore option to show how much potentially freeable space is being ignored

Awesome, thank you! I even read and knew the interactive mode might include them, I just didn't think hard enough about what that meant in terms of performance, so I'll take some blame for that. A --skip command sounds just like what I want, thank you! Until then I'll just manually handle it, or do some workaround moving the directory or something.

Hey! The --skip command is implemented in the new version 0.6.0, simply doing a cargo install cargo-clean-all should do the trick for updating.
Let me know if this helps with the scanning time.

Tried --skip and it worked amazing - tried with a similar number of directories, but skipping the largest ones, and it finished in less than 10 seconds. Thank you for the quick response, and the useful tool!


chore: use pipeline variable for windows gpu vm size

What type of PR is this?
/kind improvement

What this PR does / why we need it:
Use pipeline variable for windows gpu vm size so that we can change default value in pipeline variable without creating a new pull request when meeting capacity issues.
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2291

Relevant Lines:
2887

💛  - Coveralls


Add a create-cdd-id sub-command

Address first part of CRYP-166.

Ran the code and it works. But, don't merge it yet.
I am working on fixing the crypto-framework so that you don't have to pin a specific hash.

Ran the code and it works. But, don't merge it yet.
I am working on fixing the crypto-framework so that you don't have to pin a specific hash.

Great, once that goes in, I'll bring back --remote and rebase to your branch.

@ParnianAlimi check https://github.com/PolymathNetwork/crypto-framework/pull/12


bug: DeviceType filtering in api2

It seems device type filtering has been dropped with api2.
For exemple, in adding/editing a device, all device types are display when we select a Manufacturer.

Could you share some API request and outputs? Because I was not able to reproduce it.

I forgot to mention i'm using branch v2-develop
For exemple:

#curl http://<IP_ADDRESS>/api/dcim/device-types/?manufacturer_id=5
{
   "next" : null,
   "results" : [
      ...
      {
         "comments" : "",
         "id" : 2,
         "custom_fields" : {
            "serial-conf" : null
         },
         "is_console_server" : false,
         "model" : "DL160Gen9",
         "part_number" : "",
         "instance_count" : 7,
         "is_network_device" : true,
         "is_pdu" : false,
         "u_height" : 1,
         "is_full_depth" : true,
         "manufacturer" : {
            "id" : 4,
            "slug" : "hp",
            "url" : "http://<IP_ADDRESS>/api/dcim/manufacturers/4/",
            "name" : "HP"
         },
         "interface_ordering" : {
            "value" : 1,
            "label" : "Slot/position"
         },
         "slug" : "dl160gen9",
         "subdevice_role" : null
      },
      ...
   ],
   "previous" : null,
   "count" : 16
}

https://github.com/digitalocean/netbox/pull/1023/commits/8e1a34c853f42158fdb1b8e8409c960514e8f8ea Here is it :-)

I can't merge the linked commit as its part of a larger PR, but I'll make the change. Thanks for the report.


[example] CustomStrategy using passport-custom

Hi!
Could I create a custom passport strategy using passport-custom? How? Any basic example?
https://www.npmjs.com/package/passport-custom:  This module lets you authenticate using custom logic.
Thanks in advance

https://github.com/feathersjs/feathers-docs/issues/787

@kfern I added an example here feathersjs/feathers-docs#787

@zusamann Thanks 🎉💃


docs: add badges

resolves: #102

Note: to make the license badge work as we want it to, we need to update the parent package license to MIT and re-release.


WIP: Don't store file contents in the cache.

Don't store file contents in the cache.
We don't actually need or use them, and they take a lot of space. How much space? Let's find out...

Motivation

Reduce cache size and slightly speed up IDE startup w/ a cache.
Test plan

See included automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_Iyk6BR6YEtxTjt
→ https://go/builds/bui_Iyk6LaYt8sYuwC

Now ready for review.


[MWPW-129920] Marquee Large Updates

Implementation of new updates for Marquee Large.
Two new variants with min heights 560px (called standard variant) and 360px (called compact variant) are created.
Quiet Marquee updates will be pushed as a separate PR.
XD link: https://www.dropbox.com/sh/ote8tkbnhpggkpn/AADvJ5JF-7zmm-yEA8ZbquA8a?dl=0

Resolves: MWPW-129920
Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/drashti/marquee-std/marquee-large-final?martech=off
After: https://marquee-block--milo--drashti1712.hlx.page/drafts/drashti/marquee-std/marquee-large-final?martech=off

There are a number of css linting issues i'm seeing and if at all possible can we avoid using taxing css selectors like :not()?

The linting issues were resolved when the branch was updated, and I also removed the :not() selectors wherever possible.
Thanks!

@drashti1712 Thanks for looking at those scoped selectors. Good idea to have better performance first w/ regards to css selectors.
I'm still seeing linting errors on the marquee.css. Big picture is the format of that file should follow somewhat the dom structure of the block. Example. the first line in the marquee.css file should be simply .marquee etc and cascade down the selector specificity. This contribution goes against this paradigm and makes it hard to maintain and read. A block css file should never start w/ a very specific selector scope. ie .marquee.large .foreground.container .text See linting errors and order of selectors.

@ryanmparrish Thanks for clarifying this. I misunderstood the previous comment. Fixed these issues now!

Hello everyone,
This PR is blocked due to some confirmations pending from the consonant team regarding the designs. Hence, closing this PR for now. I will be raising another PR once the designs are revised.
Thank you for your valuable inputs.


Poor performance when testing

Describe the bug
I'm using react-query to fetch data from the backend, and in one of my test suites (using RTL) I am simulating a rejected promise.
const useUser = (id) => {
	return useQuery(['use-for-id', id], () => getUserForId(id));
}

const UserDetails = ({ userId }) => {
	const { data, isError, isLoading } = useUser(userId);
	if (isError) {
		console.error(error);
		return
Something went wrong :(
;
	}

	return
Hello, {data.user.name}
;
};

// in my test file

getUserForId.mockRejectedValue(new Error());
render(<UserDetails userId={1} />);
expect(await screen.findByText('Something went wrong :(')).toBeVisible();

The isError flag does not toggle to true quick enough and the test always times out. My workaround is to set jest's timeout at the start of the test.

Per default, react-query retries a failed promise 3 times with exponential backoff - so it will take some time. It would be best to turn off retries for tests. We’ve discussed this here: https://github.com/tannerlinsley/react-query/discussions/1090

Per default, react-query retries a failed promise 3 times with exponential backoff - so it will take some time. It would be best to turn off retries for tests. We’ve discussed this here: https://github.com/tannerlinsley/react-query/discussions/1090


Allow append of new exceptions to rules

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area rules

/area tests

/area proposals

What this PR does / why we need it:
PR introduces appending of new fully valid exception objects to existing rules.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

@sai-arigeli: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

30bae87 Allow append of new exceptions to rules

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: sai-arigeli
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @sai-arigeli! It looks like this is your first PR to falcosecurity/falco 🎉

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

30bae87 Allow append of new exceptions to rules
f05e4a0 Update negative regression tests and fix bugs

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: sai-arigeli
To complete the pull request process, please ask for approval from mstemm after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

30bae87 Allow append of new exceptions to rules
f05e4a0 Update negative regression tests and fix bugs
12e0e73 Addressing comments and minor indent fixes 1

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

c6d08b9 Merge branch 'falcosecurity:master' into sa-falco-21nov1

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sai-arigeli: Adding label do-not-merge/contains-merge-commits because PR contains merge commits, which are not allowed in this repository.
Use git rebase to reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

3d365ce Merge branch 'sa-falco-21nov1' of https://github.com/sai-arigeli/falco into sa-falco-21nov1
c6d08b9 Merge branch 'falcosecurity:master' into sa-falco-21nov1
e38f432 Removing redundant local variables

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Closing this PR due to merge issues. All commits squashed and comments addressed on https://github.com/falcosecurity/falco/pull/1780. Please continue the review there.


navbar

Description
updated the navbar and made it more responsive
Concise description of proposed changes, We recommend using screenshots and videos for better description
Additional Information
before

After

[ ] I have read the contributing docs (if this is your first contribution)
[ ] This is not a duplicate of any existing pull request

Related Issues
Closes #{issue number}
Note: If your changes are small and straightforward, you may skip the creation of an issue beforehand and remove this section. However, for medium-to-large changes, it is recommended to have an open issue for discussion and approval prior to submitting a pull request.
Your ENS/address:

Sorry, I'm not exactly sure what is updated here.  Can you provide a quick write up of the changes, or possibly screenshots/gif?

@KcPele reviewing the code seems that you are adding the connect button and the switch theme button to the header. Maybe we can add this information to the PR description and add some screenshots like @ZakGriffith suggested. The code looks good! Thanks!!

The search bar should be aligned vertically at the same place the buttons to the right.


PSDK

Use the preview tab to see the example given below. Edit the relevant information and preview the changes before submitting. You may delete these instructions.

Key
Value

ID
PSDK

Source
Vaaran Bhai Gurdas Ji

Page
24

Line
pYrI pY pwKwk hoie; grbu invwir grIbI AwxI [

Correction
THIS ≠ THAT

PROOF (EDITION)
Image:

Correction:
pwKwk ≠ pw Kwk
Proof:
Vaaran Bhai Gurdaas Ji | November 2011 | Shamsher Singh Ashok | SGPC, Sri Amritsar
Image:


Improve collective lessons

Also simplify e.g. use of printf in my examples elsewhere

i approved too early, have this question on the custom.py change


SSH connection auto disconnected after a few minutes idle

Version: 1.0.104
Platform: win32 10.0.17134
Plugins: none

It may depends on a server.
But there is a bug with follow up behavior.
After resetting connection in a tab, that tab disappears! And there is no "reconnect" option in the menu.


Please make this work on Brave with custom filtering for mobile.

What website(s) would this filter apply to?
startpage.com
What content should be blocked?
To be able to block any social media domains. Like, the Lizard's Meta domains. Or any domain of the user's choosing on mobile version of startpage.
What options should be offered?
No response
Did you write a custom uBlock filter already?
No response
Additional notes
No response

Hello @linlinxza, Brave browser is not and will not be supported. See https://www.spacebar.news/stop-using-brave-browser/ for why.

Hello @linlinxza, Brave browser is not and will not be supported. See https://www.spacebar.news/stop-using-brave-browser/ for why.

Anyway, if you can recommend a browser with custom content blocking (similar to that of Ublock), then I'd ditch Brave.

Yes, I am dead serious. I am free to decide what I spend my unpaid hobby time on, and supporting Brave is not something I'll willing to spend time on. I encourage you to read https://snarky.ca/the-social-contract-of-open-source/

Every commit of open source code should be viewed as an independent gift from the maintainer that they happened to leave on their front yard for others to enjoy if they so desire; treating them as a means to and end for their open source code is unethical.

Even if I wanted to support Brave Browser, its built-in cosmetic filtering engine is missing a lot of the advanced features that our rules use.

on Android devices, Firefox mobile supports extensions, including uBlockOrigin
on Apple devices, the paid-for AdGuard for iOS works OK for several users, although we only support it as "best effort", see https://github.com/letsblockit/letsblockit/discussions/577


[Usage]: Are there any batch size requirements for offline batch inference? For example, is 10,000 okay?

Your current environment

Specifically, is there a requirement for the length of the prompts list in the picture above?
How would you like to use vllm
I want to run inference of a [specific model](put link here). I don't know how to integrate it with vllm.
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

vLLM will automatically batch the prompts when sending them to the model. You don't have to worry about how many prompts you pass into LLM class.


Creating by appending gist ID doesn't apply color scheme

Expected Behavior
Creating a carbon from a gist like this https://carbon.now.sh/865b2cdf970d69153559eca39b52727f would apply the color scheme.
Actual Behavior
Carbon has no color scheme applied.
Latest Chrome, latest macOS

Selecting all the code, Cmd-X, then pasting it back in does apply the color scheme.


Cpc 437 get bills by owner

JIRA: link to jira ticket
Context:
Add access to view all bills by a customer by customer ID
Changes

create view bills by customer ID method in api gateway
add a bill by customer ID page to the UI
add a button to view bills by customer ID to owner details and to bill history

Before and After UI (Required for UI-impacting PRs)
If this is a change to the UI, include before and after screenshots to show the differences.
If this is a new UI feature, include screenshots to show reviewers what it looks like.
Dev notes (Optional)

Specific technical changes that should be noted

Linked pull requests (Optional)

pull request link

In the pr you mention you have changed some UI elements yet no pictures were provided for a before and after of the UI. Please remember to add images to the pr if UI changes are made to help visualize the difference.


Workspace Layout

I like the info presented for each project in the woprkspace: can we lose the dark grey bars, and just bold the titles? It looks a but ugly as it it is.

Dine in new design


updateof does full rm -rf/git clone of Sub-Zero

This may have been to resolve an earlier issue, however it removes and recreates every week as part of updatewkly.sh.  I've noticed the git clone command hangs for as long as an hour during this part of the script (presumably the git repo is slow to respond)

Fixed in 2.8


Feature/cookie

Again, I see no point to keep Drab.Utils as a separate module, as encode_value actually repeats what is done in Drab.Coder. The ideal API for me would be:
set_cookie cookie, value # no encoding
set_cookie cookie, value, encode: true # default &Drab.Coder.encode/1
set_cookie cookie, value, encode: Drab.Coder.URL # calling Drab.Coder.URL.encode
set_cookie cookie, value, encode: Any.Other.Module # calling Any.Other.Module.encode

cookie cookie # no encoding
cookie cookie, decode: true # default decode
cookie cookie, decode: Drab.Coder.URL

Let's cut Drab.Utils with Ockham's razor ;)

Again, I see no point to keep Drab.Utils as a separate module

Was not intentional, I've pushed a bad commit forgetting that it would be update also the pr, sorry
I'm going to remove Drab.Utils

Hi @guidotripaldi, don't worry, I understand you don't have time for this. I am going to do a cookie functions myself. Thanks for all the help! You really helped me a lot.


SearchPane Options

Is there a way to specify what shows in the SearchPane by default? If not can we add it?

Currently, it's not configurable and has a bug https://github.com/EvotecIT/PSWriteHTML/issues/193
There is a lot of options to play with https://datatables.net/reference/option/#searchpanes but that would require some work to get this added.
However, I believe Search Builder will mostly mitigate the need for search panes - https://datatables.net/blog/2020-09-01
It's implemented in 0.0.133. Of course, it would be great to have Search Panes working and configurable, but it's not there yet.

Thank you. Is there a guide on how to install the latest? The install-module looks to install 0.0.132.
Thanks

You would need to download it straight from GitHub. I would wait if I were you. It should be soon released - but I am working on some other stuff and this isn't priority at the moment.

Ok. Thanks


Ghost projectiles

On some maps, you will often see shots fired through a wall.  These are client-side only and don't actually do damage.  We should find a way to eliminate these.

I'm 90% sure this is due to chunking.  If the level is not loaded in client-side, you can see ghost projectiles go through walls.
The solution to this is to sync every projectile like we do for devastators and novas so that when the projectile's life comes to an end on the server it tells all clients to remove it.  However, this has a bandwidth cost that will need to be researched.  I imagine it won't be as large as the sniper packets bandwidth cost, but it'll be worth investigating.

OK, figured out what we need to do
In ChunkManager.ActivateChunks, there are 3 places that check a segment's GameObject.activeSelf and if it's not correct it calls GameObject.SetActive(bool) on it to fix it.  This needs to change to set thje segment's GameObject.GetComponent<MeshRenderer>.enabled to the appropriate value.
In ChunkManager.ForceActivateAll, the 3rd for loop calls a segment's GameObject.SetActive(true) function.  In addition to that, it needs to also set the segment's GameObject.GetComponent<MeshRenderer>.enabled to true.
This is well beyond my ability to transpile.  @tobiasksu would you be able to try doing this?

Actually it's Renderer and not MeshRenderer, but otherwise this appears to work.  Thanks @tobiasksu!


sample code does not work

The sample code does not work.
Apart from the variable naming which is not correct
e.g. $client is initialised but then $apiClient is used
same goes for $services and $apiService
even after fixing the above, i do get the following exception
Wallee\Sdk\ApiException: [1217B46F-7AC4-2272-1409-586DE1476CF9] The entity query filter requires the type property. in Wallee\Sdk\Service\AccountService->countWithHttpInfo() (Zeile 147 in /vendor/wallee/sdk/lib/Service/AccountService.php).

@zehnplus-giannis We have fixed the example.

The example now gives me the following error
Wallee\Sdk\ApiException: [99280D14-A860-76C3-4BC5-AA548DCF371B] Error 401 connecting to the API (https://app-wallee.com:443/api/space/search) in Wallee\Sdk\ApiClient->callApi() (Zeile 466 in /vendor/wallee/sdk/lib/ApiClient.php).

This means, you application user in missing the permissions to read the space list.

I just assigned my application-user with all possible roles for Account, Subaccount and Space.
Error still appears.
Is there any documentation which guides through this whole process in detail?

Having the same issue. @zehnplus-giannis were you able to solve this?

@bummzack
Can you please open a new ticket and paste your error information in the new ticket.
Regards,
@vttn

@vttn It's the same issue as this… trying to run the transaction example (second code example from here: https://github.com/wallee-payment/php-sdk/#configuring-a-service) and getting the following:
Uncaught Wallee\Sdk\ApiException: Error 401 connecting to the API (https://app-wallee.com:443/api/transaction/create?spaceId=XYZ) : Error // We need to force a reload when the back button is used in the browser. window.addEventListener( "popstate", function ( event ) { window.location.reload(true); }); Ooops! Sorry, something went wrong! Error code: 401 Please contact us and we will try to solve this issue.

(I replaced the spaceID with XYZ)

Hello @bummzack
If this issue is still occurring for you, please reach out to our support https://wallee.com/support.html .
It is most likely that your user credentials are not setup properly.
Regards,
@vttn

Hi @vttn
Thanks for your reply.
I'm trying to integrate Postfinance Checkout, maybe it's got something to do with that?
It's also not very helpful that the link that should point to "Learn how to authenticate and connect to our web service." is returning a 404:
https://checkout.postfinance.ch/doc/payment/doc/api/web-service#_authentication
https://app-wallee.com/doc/payment/doc/api/web-service#_authentication

Dear Roman,
Please get in contact with the guys from PostFinance checkout regarding this issue. They might be able to help you with this. The correct URL is: https://checkout.postfinance.ch/doc/api/web-service#_authentication
Regards,
Thomas

For anybody having this issue with the Postfinance Checkout, just use https://github.com/pfpayments/php-sdk instead.


Improve view granularity

Good evening,
I added this component to my in-development website, for cooking purposes. While the component works well, its view options are far too broad to work. Even at GanttViewType.day my recipe just shows everything together.
Adding additional types like GanttViewType.minute would be beneficial for these niche use-cases.

I think the customized view can solve your problem
for reference example:

http://gantt.ngnice.com/components/custom-view
https://github.com/worktile/ngx-gantt/tree/master/example/src/app/gantt-custom-view

Because the minimum unit before is day, the dragging of the minute view may be affected, and we may improve this in the future

Thanks for your reply.
I have created a custom view, though it looks like primaryDatePoints loads and provides a default start date a week of last Sunday and a default end date of next Monday.
I added logging and see that primaryDatePoints runs before my log in the constructor which shows the correct start/end dates.
Is there a way to update the chart once the new dates are determined? Right now this adds a lot of empty space.

<EMAIL_ADDRESS>supports hour view.  hour view can solve your problem. you can set viewOptions to set the cellWidth of hour view to a larger value

Thanks! This change has definitely improved the component functionality.


More items to tab-sync

[ ] "Current uploads" -- remove custom implementation
[x] "Watch History"

"Current uploads" -- remove custom implementation

Rather than trying to make it fit in with new tab-sync system, leave the implementation as is. Replace with a new implementation instead (the uploads code is not good. my bad).


Changing game size with scale mode FIT results in a canvas with a incorrect aspect ratio.

Version

Phaser Version: 3.22.0

Operating system: Android & Linux (Ubuntu-based)

Browser: Chrome (mobile & desktop emulator)

Description

I'm starting up the game with the following settings

{
width: 1777,
height: 999,
scale:
{
mode: Phaser.Scale.FIT,
autoCenter: Phaser.Scale.CENTER_HORIZONTALLY,
},

. In my primary scene I'm listening for the 'orientationchange' and adjusting the game size accordingly as the code snippet below shows

this.scale.on('orientationchange', (Orientation)=> {
    console.log(Orientation);

    if (Orientation === Phaser.Scale.PORTRAIT)
    {
        this.scale.setGameSize(999,1777);
    }
    else if (Orientation === Phaser.Scale.LANDSCAPE)
    {
        this.scale.setGameSize(1777,999);
    }
}, this );

When orientation is set to landscape and the setGameSize arguments match the original game size, everything works fine, however in portrait mode the resulting height is completely incorrect, resulting in a actual size of 980x550 with an aspect ratio of 1.78 instead of the expected 0.56 on my device for a game size of 999x1777.
Example Test Code

Game Config

{
width: 1777,
height: 999,
scale:
{
mode: Phaser.Scale.FIT,
autoCenter: Phaser.Scale.CENTER_HORIZONTALLY,
},

Add this code to create in an active scene.

this.scale.on('orientationchange', (Orientation)=> {
    console.log(Orientation);

    if (Orientation === Phaser.Scale.PORTRAIT)
    {
        this.scale.setGameSize(999,1777);
    }
    else if (Orientation === Phaser.Scale.LANDSCAPE)
    {
        this.scale.setGameSize(1777,999);
    }
}, this );

Additional Information

This issue appears to be very similar to a previously resolved issue here
I dont fully understand the code however I believe the issue is stemming from the "constrain" function. On line 76998
height = width / this.aspectRatio;
this.aspectRatio appears to still be the old aspect ratio of 1.7 instead of 0.5 resulting in the incorrect height.

This issue has been mentioned on Phaser. There might be relevant details there:
https://phaser.discourse.group/t/am-i-using-setgamesize-wrong-not-working-as-expected/3890/5


DVD Version with MPEG-2 MCI driver do not run (The X-Files Game)

This game has a DVD version with some better video quality in MPEG-2 format, but they are not running on Windows 11 (actually, none windows after Win98).
https://www.pcgamingwiki.com/wiki/The_X-Files_Game
the game is not unplayable because by not running MPEG-2 it apparently plays another video of worse quality in its place, but the person cannot experience the high quality of the DVD version.
The 'read me' of the game says: "The MPEG2 player must include a Windows MCI driver"
this MCI support seems to be ended after Windows98.
When searching solutions i saw that this case is similar to the other games like "Star Trek: Starfleet Academy" and "Wing Commander 4" that also has a DVD version with MPEG-2 better videos,
Fortunately, these 2 games have a Patch:
http://www.si-gamer.net/gulikoza/dxmci.html
which converts the MCI commands to the current way that Windows works, but it seems that this tool needs to be done game by game, so "X-Files" must not be supported, and the patch author must have been off the internet for over 10 years to ask for an update to this game.
So it would be great if a fix was added for this "MCI Driver" case, so that X-Files becomes playable too, perhaps based on this code for other games that are already ready.
To test, the DVD version can be downloaded at Archive.org and myabandonware-com
in DVD folder you can see the "vob" folder with the best quality videos, that include the Opening.

Fixed here:
https://github.com/FunkyFr3sh/cnc-ddraw/issues/315

I checked if this could be built in to DDrawCompat, but it looks like those registry keys are cached by the system and not actually read from the game process. Anyway, at least I fixed another presentation issue with this game in v0.5.3.


Using {Action Type}_FAILED

http://martyjs.org/guides/action-creators/index.html refers to {Action Type}_FAILED and similar types.
Reading the description it seems these default actions could be used to (more) easily implement progress and error tracking - However the payload of these actions seem to only contain an internal actionId generated by Marty.js which can't really be used to reference anything.
Am i going about this the wrong way? I would assume the payloads to contain the same thing as payload or similiar - Would it better to manually dispatch error events from the state source to the source action creators?

I'd also like to know more about purpose of these automatically dispatched actions that do not carry any application relevant data.

hey, sorry for the delay in getting back to you on this. According to the code we do pass the error as one of the arguments however there might be a bug there. I will have a look at this over the weekend. I'm not entirely happy with how error handling works right now and I think its probably something I'm going to fix in the up coming release. For now I'd say it probably makes more sense to handle errors yourself as I will probably move back to that approach

@jhollingworth It does contain that, but no reference to the original action, hence no way to map errors/successes back to initial requests.
But i will go forward with handling it manually with sourceActionCreators and the state sources.
Thank you for a great project, made it much easier to get started with Flux

@mickhansen Hopefully #163 will resolve your issues

@jhollingworth Yeah looks like it :)


Reputation: Indicate when a reputation is at maximum (i.e. reset available)

When you hit max rep, you stop accumulating rep, so it's good to remind ourselves to go and reset that rep!

Fixed in https://github.com/rslifka/reckoner-app/commit/a2d0d18cbd8e54429d1094e1931adf9a6bad2fdd


0091 Recognize when an account can fully sign

Description:
Added code and testing for methods determining suitability of an account to be a fee payer for a transaction that must be sent immediately
Related issue(s):
Closes #91
Checklist

[ ] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

From what I can tell, those key files do not contain real (sensitive) key and they are just for testing purposes. Still, I think it's better to change the tests to create those keys on the fly, then clean up (wipe them out) at the end of the tests.

That is a good point. In other places I have done exactly that, however I noticed that generating the keys slows down the test considerably.


Remove Struct's reliance on scripts + Proftt

Currently hpcstruct invokes shell scripts to implement brand new use cases. Measurement directory support (#421) runs a Makefile, which in turn invokes hpcproftt to parse *.hpcrun files. Cache support (#507) runs renamestruct.sh to adjust the binary name when pulling from the cache.
While these approaches "work," they are fraught with design concerns:

Most critically, shells do not handle errors gracefully. Subcommand failures are often ignored (eg. in a pipeline). This is generally very difficult to fix, and failing to handle errors results in confusing logs or worse, unusable output. These are critical issues for a user-facing tool like hpcstruct.
hpcproftt is primarily a development tool, while it is relatively simple it was not written to be fast or robust. Relying on it in a user-facing tool like hpcstruct, or in general requiring it as part of a complete install, does not match it's original purpose.
/bin/sh is not a stable target, on Red Hat it's Bash but on Debian it's Dash (a minimal POSIX-compliant shell). Bash-isms sometimes leak in (like #429) and break the tool in mysterious ways (when running on Debian like me).

Also, using scripts that need to be installed alongside hpcstruct complicates running uninstalled (out of the build tree). While this case isn't supported by the current Autotools build system, it is by the Meson port (#320). (Handling this on the side makes my life easier updating the port to the latest develop.)
See the commit log for details on the new approach.
Current known issues:

Control-C on hpcstruct --gpucfg yes x.gpubin does not stop, because system() blocks SIGINT in the parent (see man system). With this PR, this causes Control-C on hpcstruct --gpucfg yes measurements_dir to only stop the "overseer" process and not the "worker" processes, orphaning the last batch of worker processes. How best to solve one or both of these issues requires some investigation.
UPDATE: Control-C will now wait for the current batch of workers to complete. If --gpucfg yes is used, this may not be immediate and may cause unnecessary NVIDIA warnings.

*.warnings files are no longer generated, the errors from workers are output directly to the shared stderr. This may cause inconvenience when nvdisasm spews thousands of lines of error. Whether this should be handled here or as part of the nvdisasm invocation requires discussion.
UPDATE: *.warnings files are now generated.

In response to @jmellorcrummey:

the reason for the warnings files is that we have seen nvdisasm spew 100K lines of errors. having that appear on stdout is not useful.

The same issue is present when parsing a single binary (although it's a far less common use case), so IMHO it's the error/output handling around nvdisasm at fault here.
Maybe we could redirect its output to a --log-file (on request) and leave a (short) warning in stderr, but I'd rather wait until #522 is merged before messing with that code. (Ping @Jokeren)

making complete result files appear atomically and respecting results files that are present (with the exception of when the —gpucfg argument to hpcstruct was changed) lets one control-c hpcstruct and restart it without throwing away what it may have already computed.

IMHO the cache has the same effect, and the implementation is far more appropriate (no sensitivity to timestamps, no auxiliary not-quite-temporary files, proper cleanup of outputs on crash...). The output is also clearer IMO since it lists all the binaries every time.
We already recommend using a cache, but maybe we could add a cache within the measurements directory by default?

The same issue is present when parsing a single binary (although it's a far less common use case), so IMHO it's the error/output handling around nvdisasm at fault here.

When the errors are due to a single binary, the answer is obvious: restart and put the output in a log file. When the errors spew from analysis of a measurement directory, you provide no choice but to restart analysis of everything and then hunt through the mass of output to see if you can figure out which file is the culprit. Also, from what your comments describe (everyone uses stdout/stderr), the errors from analysis of multiple files are likely to be interleaved (and not necessarily at the line level either). For that reason, when analyzing multiple files, dumping the output in a warnings file is much better.
It used to be the case that the Makefile would create a warnings file for every file analyzed and then remove any warnings file that was empty after analysis was complete. Marty's changes to use them to record cache state made them always non-empty, which was a downgrade.

https://github.com/HPCToolkit/hpctoolkit/pull/522 does not report any problem with merging into develop. I am not sure what test cases are necessary though before merging. Maybe I should contact Marty?

*.warnings files are now generated as before, except the exit code is used to pass the cache state back to the "overseer" so there should only be *.warnings when there are real warnings. This should resolve @jmellorcrummey's concerns.
I've also added some code to handle ^C as best as possible without adjusting nvdisasm invocation, which handles the other issue I knew of.
If there are no other issues, ready for review.

Updated and rebased on top of #598

This repository has moved to GitLab, please continue this PR there.

The previous link was incorrect, apologies. The corrected link is here.


negative assertions

Negative assertions, e.g. asserting that a certain call was not performed or a certain event was not emited are not very useful on their own (because they might pass for the wrong reasons, e.g. a negative assertion always passes if you simply don't do anything of course).
However, when paired with positive assertions (e.g. asserting that the tested functionality actually performed the desired state mutation), they can be very useful to assert that a certain code path was skipped in doing so. This can be useful to test e.g. that a certain conditional was skipped.
I realize that this can already be achieved at the test level with the testFail prefix, but that comes with different ergonomics vs. negating individual assertions.

Good idea. Does any other project do it, so that we can have some example? What would be your ideal syntax?
(cc @brockelmore @mds1 @transmissions11)

I like this idea, agreed it'd be useful, but I don't have any examples offhand. It's easy to extend DSTest to add an assertNeq method so I think we can ignore that for the scope of this issue. Then we have expectEmit, expectCall, and expectRevert. Inverting expectRevert isn't necessary, so we're down to just calls and events, but let me know if there's anything we might be missing here.
It would also be useful to test that you don't jump to a certain function within your contract (and similarly have an expectJump), though I'd guess this is currently difficult for the same reasons as #432
Some syntax ideas:

a vm.expectInverse() cheat code that watches for the next vm.expect*() and inverts it. With an approach like this I worry we're cluttering up tests a bit, e.g. you now need 4 lines to test an event is not emitted, and because of the order required you can't wrap these into a helper method
similar to 1 but instead called as vm.expectInverse(vm.expect*()), which saves a line, but I think we'd have to abi encode this without a preprocessor—also this then becomes the only cheatcode that takes another cheat code as input, but maybe that's ok

There's probably another question of how topics should be considered when expecting an event not to be emitted. I'd guess most or all use cases only are really interested in only ensuring that topic0 is not found, as opposed to ensuring topic0 is emitted but that topic1/data does not have some value. The easiest behavior is to just invert the result of expectEmit, but then everyone would be calling vm.expectEmit() with 4 unused input args every time they want to assert an event isn't emitted which feels a bit gross

@mds1 Agreed wrt 1 being a bit verbose / cluttered. I like 2, but maybe we can make that a bit shorter: What about vm.not(vm.expectCall(...)) ?
Slightly off topic, but for events in particular, wouldn't it make sense to also have a more ergonomic version that just tests for topic0 via vm.expectEmit("Transfer(address, address, uint)").

Slightly off topic, but for events in particular, wouldn't it make sense to also have a more ergonomic version that just tests for topic0 via vm.expectEmit("Transfer(address, address, uint)")

That'd be a useful overload here, because then we can shorten it to s single line vm.not(vm.expectEmit("Transfer(address, address, uint)")). IIRC we discussed an expectEmit syntax like that but didn't implement it since it doesn't extend nicely to taking input args of the expected params, though I may be misremembering / @brockelmore might recall

+1 for vm.not before expectNotEmit. For some protocols (like Nomad), where the functionality is heavily based on event emission, some times there are no other signals (or variables to be read) that can verify that something did NOT happen, except the NOT emission of an event.

this is definitely useful and we should add this imo.
otherwise, you can't test the inverse of expectEmit.
flipping the impl and adding expectNotEmit should be easy.
will try to squeeze this in

+1

def a +1 from my side for this change, too.
Hardhat has a similar functionality that I found really easy and handy to use:
expect(<call).to.not.emit(, ).withArgs(....)
the ".withArgs" part is optional. So either I check only if an event was emitted or not or I can also check if the args match (or dont).

+1
Another way to go about this would be with cheatcodes that count the number of times a function is called or event emitted. That way we can not only assert that call/emit was made multiple times, but also if it was not made at all using the same cheat.
There is an issue for expectCallCount (https://github.com/foundry-rs/foundry/issues/4513), but we could also do something similar for events as well.
Eg. expectEmit(bool,bool,bool,bool, uint256 count)

I'm supportive of adding overloads with a count arg for both the expectEmit and expectCall cheats, since that accomplishes both this issue and #4513 with the same UX/set of cheats. I saw in the support telegram that you'd be interested in working on this, should I assign to you?
Also if anyone has objections to that syntax and has a reason to prefer e.g. expectNoEmit please share 🙂

Yep, you can assign it to me @mds1

cc @mds1 i guess we went for the count syntax arg (which needs to be fixed)—do we still wanna implement negative assertions?

cc @grandizzy possibly relevant to recent cheatcode additions

yep, is on my cheatcode expect* list to tackle


Discord icon is not appearing

Hi - thanks for creating this package. For some reason the Discord icon doesn't seem to appear - all the others I tested are fine.

I cannot reproduce this issue using a discord.com url. Can you make sure you are on package version 5+ (npm i react-social-icons@latest)? Can you please share your test code and the version number of react-social-icons you are using? And include the url and other props you are passing to <SocialIcon />?

Thanks @couetilc, it works with the latest version. I thought it might be because the TLD for Discord sharing is .gg - both seem to work now though.

Ok, thank you for following up. I will close this issue, feel free to update with other issues you run into in the future, please just add some example code or a reproducible case.


Align all L2 resource names to L1 resoruce names (e.g. "EncryptionKey" => "Key")

Duplicate: https://github.com/awslabs/aws-cdk/issues/1920

Superseded by #2267


💭 [REQUEST] - Refactor PUML Functionality

Reference Issues
#773
#774
#775
#776
#777
#778
#779
#780
#781
Summary
As an AaC Developer, I want to refactor current AaC gen-puml plugin, so that I provide a useful AaC tool for end users leveraging the updated API.
DoD:

[ ] gen-puml plugin supports object diagram creation leveraging the updated API.
[ ] gen-puml plugin supports component diagram creation leveraging the updated API.
[ ] gen-puml plugin supports sequence diagram creation leveraging the updated API.
[ ] gen-puml plugin supports requirements diagram creation leveraging the updated API.

Basic Example
Needing to reincorporate the PUML plugin by refactoring it to utilize the new 0.4.0 structures, data object, and API.
What are the drawbacks/impacts of your feature request?
If this does not get completed in the next iteration it will become a blocker for end users.
Unresolved questions
No response

Duplicated feature request, so closing this one.


Enhance Shortcut Search to Include Any Part of Title, not only beginning

Overview:
This pull request introduces an improvement to the shortcut search functionality. Previously, the search was limited to matching only the beginning of shortcut titles. With this update, users can now search for shortcuts by using terms that appear anywhere within the shortcut title, for example: previously, in order to find "Company Information", you couldn't just type "Info", but now you can. This change better aligns with Salesforce's setup search functionality witch allows searching for terms that appear anywhere in the node title. In addition this change is beneficial since users don't always remember the exact title and it's beginning.
Testing
This feature has been tested across different environments to ensure compatibility and smooth integration with existing functionalities. I encourage further testing to validate its efficiency and usability enhancements.
Documentation
I have documented this change in the CHANGES.md file
Request for Review
I kindly request a review of this pull request and am open to making any necessary adjustments based on your feedback. Thank you for considering this valuable enhancement to the user experience.

Hi @toly11,
Thanks for your contribution !
Welcome to the team ! 👏

@tprouvot Thank you!


update readme - add https to composer link

The link to download composer on the README currently takes me to https://github.com/jikan-me/jikan-rest/blob/master/getcomposer.org/download, added https to fix that.

Thanks!


Fix custom data source Sandcastle example.

The clustering property is now required on all DataSources. Updated the Sandcastle example.

Looks good, the demo works fine.


REQUEST

Is your feature request related to a problem? Please describe.
I made a batch script, its take a screenshot of the computer BUT its use powershell and the grabber completly break my powershell code :/
Describe the solution you'd like
Add an "Powershell code finder" and obfuscate the powershell code diferaly as the BatchCode
Describe alternatives you've considered
For the Powershell obfuscation you can use this ! [https://github.com/JoelGMSec/Invoke-Stealth]
Additional context
https://github.com/JoelGMSec/Invoke-Stealth

Can u turn debug mode on and also try changing some of the settings?

@Appolon24800 I see the issue. I just pushed out a fix a few mins ago and it should all work now


Issue with PVC when renaming devfile component

/kind bug
What versions of software are you using?
Operating System:
Linux Fedora
Output of odo version:
$ odo version
odo v3.0.0-alpha1 (96ea52e88)

How did you run odo exactly?
After running odo dev a first time, I update the metadata.name in the devfile.
Actual behavior
After running odo dev and stopping it, the PVC is still present. No ownerReference are set in the PVC.
The problem is that the name in the devfile and the name in the odo env file differ.
The pvc is created with the name in the env file then updated to add ownerReferences by using the name in the devfile.
Expected behavior
Any logs, error output, etc?

This is fixed in v3.13.0 (or previously)


JAMES-3926 quota current value recomputation for sieve

this PR is based on https://github.com/apache/james-project/pull/1707

1) [Guice/MissingImplementation]: No implementation for UploadRepository was bound.

Requested by:
1  : JMAPCurrentUploadUsageCalculator.<init>(JMAPCurrentUploadUsageCalculator.java:37)
      \_ for 1st parameter
     at RecomputeCurrentQuotasService.<init>(RecomputeCurrentQuotasService.java:180)
      \_ for 7th parameter
     at WebadminMailboxTaskSerializationModule.recomputeCurrentQuotasTask(WebadminMailboxTaskSerializationModule.java:76)
      \_ for 1st parameter
     at WebadminMailboxTaskSerializationModule.recomputeCurrentQuotasTask(WebadminMailboxTaskSerializationModule.java:76)
      \_ installed by: Modules$OverrideModule -> Modules$OverrideModule -> Modules$OverrideModule -> Modules$OverrideModule -> Modules$OverrideModule -> Modules$CombinedModule -> Modules$CombinedModule -> Modules$CombinedModule -> Modules$CombinedModule -> Modules$CombinedModule -> MailboxRoutesModule -> WebadminMailboxTaskSerializationModule
2  : JMAPCurrentUploadUsageCalculator.<init>(JMAPCurrentUploadUsageCalculator.java:37)
      \_ for 1st parameter
     at RecomputeCurrentQuotasService.<init>(RecomputeCurrentQuotasService.java:180)
      \_ for 7th parameter
     at UserQuotaRoutes$RecomputeCurrentQuotasRequestToTask.<init>(UserQuotaRoutes.java:85)
      \_ for 1st parameter
     at InconsistencyQuotasSolvingRoutesModule.configure(InconsistencyQuotasSolvingRoutesModule.java:36)
      \_ installed by: Modules$OverrideModule -> Modules$OverrideModule -> Modules$OverrideModule -> Modules$OverrideModule -> Modules$OverrideModule -> Modules$CombinedModule -> Modules$CombinedModule -> Modules$CombinedModule -> Modules$CombinedModule -> Modules$CombinedModule -> InconsistencyQuotasSolvingRoutesModule

Please have a look into these failures.
1) [Guice/MissingImplementation]: No implementation for UploadRepository was bound.

Requested by:
1  : JMAPCurrentUploadUsageCalculator.<init>(JMAPCurrentUploadUsageCalculator.java:37)

It's a guice architecture problem. RecomputeCurrentQuotasService should not have a hard dependency to JMAPCurrentUploadUsageCalculator. We would need to hide it with a multi-binder...

The Guice failures it's likely because this PR was built on top of https://github.com/apache/james-project/pull/1707 that had the same issue but is being fixed atm. So I would not worry too much and wait for this PR to be updated after the first one becomes valid ;)

test added


[Enquiry] Does this docker provider suppose Docker Compose deployment?

Description
Does this docker provider suppose Docker Compose deployment?
References
No response
Would you like to implement a fix?
None

Nope, sorry! This provider focuses on managing Docker-hosted resources (such as repositories, teams, organization settings), not on running containers.


feat: add "promoted" label and new naming for isbsvc, similar to other Rollouts

Modifications
This doesn't implement the full Blue/Green strategy for InterstepBufferService, but does implement the initial commit, which sets up the main ("promoted") InterstepBufferService using the same process we follow for PipelineRollout and MonoVertexRollout. One goal for this initial PR is to avoid backward incompatibility issues as we migrate it to Blue/Green.
Just like Pipeline and MonoVertex:
The new InterstepBufferService has:

"parent-rollout-name" label
"upgrade-state"="promoted" label
And the new ISBServiceRollout has:
Status.NameCount  to keep track of name indexing

The tricky thing that's different is that the Pipeline that's created cannot use the InterstepBufferServiceName which is defined in the PipelineRollout Pipeline spec, but rather must resolve the InterstepBufferServiceName based on the isbsvc child of the ISBServiceRollout.
How does it know which one to use?:
per the original design:

the "promoted" isbsvc is used for the "promoted" Pipeline
for an "upgrading" Pipeline, if there's an "upgrading" isbsvc, it uses that; otherwise, it uses the "promoted" isbsvc

Verification
Performed progressive testing, creating ISBServiceRollout and PipelineRollout; updated PipelineRollout and observed that "upgrading" pipeline also used the isbsvc.
See "Backward incompatibilities" section below for additional testing that was performed.
Backward incompatibilities
The clusters are all running v0.9.1. For these clusters, no isbsvc has "parent-rollout-name" label or "upgrade-state" label.
We need to be able to transition them to use these labels, and we want to avoid a new isbsvc being created. Therefore, what I did was create a temporary init container for numaplane, which updates the InterstepBufferServices across the cluster to have the new labels. I've also made sure that the code works to have an isbsvc that doesn't end in dash+number.
To test all of this, I have:

run v0.9.1 locally
created isbsvc and pipeline
switched to new branch, and run numaplane
observed that init container updated my isbsvc to have the new labels
observed that pipeline continued to use the same isbsvc
tested PPND in this state, updating both PipelineRollout and ISBServiceRollout, making sure everything worked correctly

Note, I won't merge this until after we create a PR for @afugazzotto's recent change first.

I've marked as "Draft" so we don't accidentally merge it until after the new release is made.