"support" bundling AppImage when building in a container

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[x] No

The PR fulfills these requirements:

[x] It's submitted to the dev branch and not the latest branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[x] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
As discussed in this AppImage bug report, Using appimagetool under Docker (and friends) as an executable won't work because AppImage executes its bundle by FUSE mounting the embedded squashfs file system - and this does not work in a container becase FUSE is a kernel feature that is hard (and likely not a good idea anyway) to access from a container.
As the above discussion recommends, it is actually simple to workaround the problem by manually extracting the AppImage file system to a local directory and running it from there.
This drive-by PR adds a test using lsmod to see if we have fuse supported available in the kernel, and if not - uses the --appimage-extract workaround to run the AppImage tool. I have tested this patch by building tauri-theia under docker.
The lsmod detection is probably not ideal, but I'm not sure what other detection method would work better - I thought using an environment variable to trigger the new behavior, but Docker itself does not provide any such identifying environment variable so discovery by builders might be a problem.
Let me know if this feature is interesting but you want a different implementation and I can see what I can do better.

Do you have the dockerfile you used to try it?

Do you have the dockerfile you used to try it?

Working on it here: https://github.com/guss77/tauri-theia/blob/dev/Dockerfile
I'm having some problems with Github Actions (first time for me), but it builds fine locally.

Hmm I didn't see any problems with GitHub Actions and AppImage, but maybe you could use this as a guide if you're testing it: https://github.com/tauri-apps/tauri-action

Hmm I didn't see any problems with GitHub Actions and AppImage

Don't worry about that - my problems with Github Actions are definitely my own doing, and unrelated to AppImage, Taury or Theia ;-)

Looks good, thank you for your contribution!

@jbolda - is there a way to add this to the changelog post merge?

The change file will technically reference the wrong PR / commit, but you can point back to the correct one in the commit message.

Totally forgot about the PR/commit on the changelog :(

Maybe add that to the PR contribution guidelines?

Yeah maybe we just need a GH action to remind all of us.

@guss77 it is one of the check boxes that you checked 😄
Some kind of check would be nice though, but it would be a bit odd to fail if a change file doesn't exist as it is valid not to have one (e.g. updating a readme).

Maybe just a PR comment or something else


bump golang versions

Description

Checklist

[x] I have updated the CHANGELOG of any plugins updated
[x] I have updated the version file of any plugins updated
[x] I have added tests wherever appropriate
[ ] I have labeled this PR with a label per plugin updated

Taken care of in another PR


gns3-gui,gns3-server: 2.2.43 -> <IP_ADDRESS>

Description of changes

gns3-server: 2.2.43 -> 2.2.44
gns3-server: add meta.mainProgram
gns3-server: add passthru.tests.version
gns3-gui: 2.2.43 -> 2.2.44
gns3-gui: add passthru.tests.version

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2898


Custom Iconset by iconfiy

I notice that nuxt-ui-vue use iconify to support icon. That is good framework. Now I want to add my own icon collection, then use them in nuxt-ui-vue.
import { addCollection } from '@iconify/vue'
addCollection({
    prefix: 's24',
    icons: tranIcons,
    width: 24,
    height: 24
  }, 'b3d')

for example
toast.add({  title: '1234',  timeout: 0, icon: '@b3d:s24:cross-filled' })

I find that loadIcon throw error in UIcon component. Any example can show how to use custom iconset?

Yes, it does work. Here is an example:
<script setup lang='ts'>
import { addCollection } from '@iconify/vue';

addCollection({
	prefix: 'mdi',
	icons: {
		'account-box': {
			body: '<path d="M6 17c0-2 4-3.1 6-3.1s6 1.1 6 3.1v1H6m9-9a3 3 0 0 1-3 3a3 3 0 0 1-3-3a3 3 0 0 1 3-3a3 3 0 0 1 3 3M3 5v14a2 2 0 0 0 2 2h14a2 2 0 0 0 2-2V5a2 2 0 0 0-2-2H5a2 2 0 0 0-2 2z" fill="currentColor"/>',
		},
		'account-cash': {
			body: '<path d="M11 8c0 2.21-1.79 4-4 4s-4-1.79-4-4s1.79-4 4-4s4 1.79 4 4m0 6.72V20H0v-2c0-2.21 3.13-4 7-4c1.5 0 2.87.27 4 .72M24 20H13V3h11v17m-8-8.5a2.5 2.5 0 0 1 5 0a2.5 2.5 0 0 1-5 0M22 7a2 2 0 0 1-2-2h-3c0 1.11-.89 2-2 2v9a2 2 0 0 1 2 2h3c0-1.1.9-2 2-2V7z" fill="currentColor"/>',
		},
		'account': {
			body: '<path d="M12 4a4 4 0 0 1 4 4a4 4 0 0 1-4 4a4 4 0 0 1-4-4a4 4 0 0 1 4-4m0 10c4.42 0 8 1.79 8 4v2H4v-2c0-2.21 3.58-4 8-4z" fill="currentColor"/>',
		},
		'home': {
			body: '<path d="M10 20v-6h4v6h5v-8h3L12 3L2 12h3v8h5z" fill="currentColor"/>',
		},
	},
	width: 24,
	height: 24,
});
</script>

<template>
  <div class="grid place-items-center w-full min-h-screen">
      <UIcon name="mdi:account-box" />
  </div>
</template>

Yes, I test it and work well. But I want to use this with UNotification icon property. And it not work.

I have tested your demo, if I change the prefix or icons body, the icon show incorrectly. It seems like the UIcon loads svg from iconify library not from custom defination.

I check the code of UIcon in nuxt-ui-vue import { Icon } from '@iconify/vue/dist/offline', I think we should use import { Icon } from '@iconify/vue' for custom iconset.

Can you handle this by creating a PR?

I find @iconfiy/vue create a globa variable dataStorage to store the custom icons. If I call addCollections from @iconfiy/vue in my code, it will create another dataStorage which is not shared with previous one in nuxt-ui-vue. we should expose the functions from @iconfiy/vue.

PR  https://github.com/selemondev/nuxt-ui-vue/pull/145

Thanks for your contribution and this issue has been successfully resolved with the latest release of Nuxt UI Vue v1.0.1-beta.4


chore(shared): remove extra NaN logic

since x !== x  is the only true when x is NaN , so that indicate the oldValue and the value are both NaN
so remain one is fine

Why not just use Object.is?
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/is


[FEAT] Multiline flashcard with empty lines

Is your feature request related to a problem? Please describe.
I would like to use empty lines to separate paragraphs in an answer. For example:
Question?
?
First paragraph
Seconds paragraph
Describe the solution you'd like
Maybe we should add another separator for multiline cards that have a separator(---) to indicate the end of the card. Or we can use similar approach as neuracache
Additional context
Example of flashcard from neuracache:
Radii of stable orbits in the Bohr model #flashcard

$a_0 = \frac{{ \hbar ^2}}{{m_e ke^2}}$

$\Delta E = h\nu$

$c = \lambda \nu$

Today I decided to refactor my notes to support spaced repetition, but at the first note itself I faced this issue.
I have tables, multiple equations as answers.
Is there any active development going on in this.

Any updates to this?


ADD Header and footer in cart.html

Kindly review my issue and assign me under apertre24.

Update on Task?

Yes I will send PR for this by tonight


Add a 'reauthenticate' function

Many Firebase operations (account delete, update email, etc) require a recent (~5mins) authentication in order to succeed.  We should provide a convenience function/flow to re-authenticate the current user.
@amandle has already implemented this on Android (PR not yet merged, planned for version 2.0)
Android issue: https://github.com/firebase/FirebaseUI-Android/issues/563
Android implementation: https://github.com/firebase/FirebaseUI-Android/pull/583

FYI FirebaseUI-Android#583 was merged into our version-2.0.0-dev branch.

if I'm not mistake all mentioned operations are implemented in Account Management flow (https://github.com/firebase/FirebaseUI-iOS/pull/261) which has re-authenticate feature

@yramanchuk this is true.  But we need to expose the reauthenticate function separate from that flow to meet the original developer request.
It would also be good if you have time to implement reauth first since @amandle already did that and account management could take a lot longer.

Although we have put account management on the backburner, I would still like to get this done since the work has already been done for Android.
Yury / Morgan does either of you have time for this before the next release?  I could always make the Android version private and expose it in a later release but that would be a bummer since @amandle did good work.

I have used FUIPhoneAuth with signInWithPresentingViewController:
and now need to reauthenticateWithCredential:
My problem is I cannot get hold of the "verificationID", which I then can use to get
the FIRAuthCredential in order to re-authenticate.
How do I proceed to do this currently?
Some doc recommends this when not using FUIPhoneAuth
(1)
[[FIRPhoneAuthProvider provider] verifyPhoneNumber:userInput
UIDelegate:nil
completion:^(NSString * _Nullable verificationID ....
//save the verificationID in some user preferences...
(2)
FIRAuthCredential *credential = [[FIRPhoneAuthProvider provider]
credentialWithVerificationID:verificationID
verificationCode:userInput];
etc.

@wayneforrest you should be able to use that method regardless of whether or not you're using phone auth. Please file a separate issue for your question.


JoinLines command jsdoc support

Version: 1.54.0-insider (system setup)
Commit: 192c817fd350bcbf3caecae22a45ec39bae78516
Date: 2021-02-19T05:14:28.632Z
Electron: 11.2.3
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042
editor.action.joinLines command needs some tuning to work with jsdoc. Currently if you will try to join multiline jsdoc comment into a single line, it won't remove star symbols at the beginning of each line.
Before:
/**
 * @typedef {number} A
 */

Currently after:
/** * @typedef {number} A */

Expected after:
/** @typedef {number} A */

@alexdima @rebornix Adding you since joinLine is a basic text editing feature


Add user settings

We would like to have settings per user. This could be used for specifying a custom scheduler name. Then the dashboard's mutating admission controller could change all pod and deployment definitions to use the custom scheduler for the user.

Closing this, as there is no such need now. Reopen with more specific requirements.


Allowing u-turns in two way segments

Hello
First of all I'm so happy that I found an active repository of OSRM and I'm impressed by the functionalities that you're working on to add to OSRM.
I have an issue with OSRM in two way segments where the route can be both in forward direction (according to bearing) and a turn around and continuing backward direction. It seems that according to the driving rules in Europe it's not a legal action to do a u-turn in the middle of a two way segment but in my country it's a legal action and the resulting route of OSRM is kind of strange for our users.
Can you help me how can I fix this? Or maybe enable it?
Thanks

Could you please show us the concrete case that you met and the route you expected? Better to have pictures of your case, and location where the problem actually occurred.

@wangyoucao577 Thanks for your reply. This is a snapshot of the issue and this is the sample request
The user is driving towards the left side so the response route of OSRM is to go to the end of the road, turn around and come back. But the expected and legal response is to turn around at the start of the route and continue.
I've tried changing this line to always return true, but I'm not sure if it breaks other things as I haven't tested it much. What do you suggest?
I'm from Iran :)

Interesting issue. Firstly it's not caused by restriction since there's no turn restrictions around the origin/destination point of your request by checking osm map. Furthermore, the roads here mostly will be compressed as a single edge by OSRM's graph processing. So once you add bearings in request, the route can only go ahead to the deadend that allows u-turn.
From my perspective, it's not recommended to fix it in the routing engine since it may break OSRM's graph abstraction. Is it possible to remove the bearings option from your request?

I think it's not possible to remove the bearings, the driver requests a routing inside the navigator while driving and so it has a bearing. If I totally remove the bearings from the navigator it makes other issues, such as giving routes backwards the drivers direction while driving. Do you have any idea?
The issue isn't just related to these long and two way roads, it's also an issue in residential streets. Again it is legal to turn around in the middle of a residential and not to go at the end of a deadend for it.

I have no idea at the moment. Did you ask this issue on https://github.com/Project-OSRM/osrm-backend? Any other suggestion from other authors/contributors?

Yes, I've asked this in osrm-backend repository before (here) but the only idea was the same as you said to remove the bearing.
I think you are also using osrm for a navigator, how do you use the bearing? Do you even use it?


[Kata] Enable new dom0 loader

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Add new dom0 loader package (hvloader.spec) and bump kernel-mshv to a version of the source which supports the new loader.
Change Log

kernel-mshv -> 5.15.90.mshv2 (new dom0 loader feature branch commit c885081639d4e278ea77eb916258df07d62633ca )
Introduce hvloader.spec

Does this affect the toolchain?

NO
Test Methodology

https://dev.azure.com/mariner-org/mariner/_build/results?buildId=306323&view=results

I will merge if everything looks good and I pass the conformance/E2Es with the new kernel-mshv. kata-buildRPMs in progress right now:
https://dev.azure.com/mariner-org/mariner/_build/results?buildId=314758&view=results

This has been split into two separate PRs. Please see:

https://github.com/microsoft/CBL-Mariner/pull/5289
https://github.com/microsoft/CBL-Mariner/pull/5290


Removed unused field hello in Float32Array.hx

Fixes #1708
There's an unused field called hello in lime/utils/Float32Array.hx, which seems to have been added by mistake. This PR just removes it

Added all the way back in 2016, in an otherwise-unrelated commit. I assume it was used for quickly debugging something, and wasn't meant to be included in the commit. (I say this because I often do something similar, though I always call it debug.)


CLI template for qMR fitting

Until BIDS covers useful qMR data descriptions, we need to define a useful convention for this (title) purpose. The initial idea is to have 2 positional arguments for folder locations followed by a JSON file:
fitfunction input_folder output_folder opt_params.JSON
If we had a standard for file naming, it would not be necessary, but now we need users to somehow indicate which parameters (Flip Angle, for example) are related to which image sitting in input_folder.  I would like to hear your suggestions on this @mathieuboudreau and  especially @spinicist.

I'm not sure what to suggest here. Currently QUIT works on a per-file basis, rather than per-folder, e.g.
qidespot2 t1_map.nii ssfp_data.nii < params.json
I haven't got round to fully adopting BIDS yet - although I try to structure subjects/timepoints the BIDS way, for each session I keep everything in one folder as that is how it comes off our scanner. I'm in the middle of implementing nipype interfaces for QUIT, once that's finished I was going to take a more in-depth look at BIDS.

https://github.com/qMRLab/qMRWrappers
https://github.com/qMRLab/qMRFlow


Error after login

hi, I want to login with a user through query.
First i create feature file and i create query after that i fill the text file then i click login button after that its goes to the user dashboard page after that its show error
"Then I should be on the dashboard page # cucumber-websteps-0.10.0/lib/cucumber/websteps/browsing_steps.rb:56
expected: "/welcome/dashboard"
got: "/users/sign_in" (using ==) (RSpec::Expectations::ExpectationNotMetError)
./features/support/env.rb:94:in block in <top (required)>' features/login.feature:10:inThen I should be on the dashboard page'"
I want to perform some other action after login
can i show my code to you because i don't known what is error?
After create user i print user details on terminal and its shows your user created but after login break the page

expected: "/welcome/dashboard" got: "/users/sign_in"
This means that the Cucumber test was expecting to have been redirected to /welcome/dashboard, but was redirected to the login page. I recommend making sure that the user login process is working correctly.
Again, Github issues are for reporting bugs with Cucumber. As this isn't a bug with cucumber, please read the documentation I posted in a previous comment and have a read of
https://cucumber.io/support
I will close this issue. Please do not create another issue for support-related issues.


⚠ Mobitopup has degraded performance

In 56d6a6c, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3873 ms

Resolved: Mobitopup performance has improved in ea0e5e3.


IntelliJ Plugin CFLint Support

@TheRealAgentK, I am interested in a CFLint plugin for IntelliJ, how can I contribute?

Re CFLint plugin for intelliJ you can should be able to user the sonar cube plugin as it can use CFLint under the hood

Last time I tried, the SonarQube plugin was not usable for CFML.
Anyway, it would be a workaround, not a solution. An eventual lint plugin should work without dependency on SonarQube.

you could look at what was done to integrate VSC and CFLint

As far as integration, VSC is just using the command line and parsing the results. For a better solution, it might be best to hold off until the API lands.
Either way, you should check out the source for some of the existing third-party lint plugins. There are also some more official plugins such as stylelint and tslint.


npm install can't install dependencies (server) (development branch)

Describe the bug
npm install can't install dependencies (server)
To Reproduce
Steps to reproduce the behavior:

cd server
npm install
the error is shown in the screenshot

Expected behavior
to install dependencies
Screenshots

Additional context
Ubuntu 22.04

I believe required: {node: >= 14.0}  Means you need to update your nodejs from v12 to v14 or higher you type node -v` you can see your version.
My node version is 18
Are you using Ubuntu LTS (Long Term Support)
LTS versions are more stable, but the downside is that you get very old software packages.
LTS versions are not so good for coding and development, its more suited for regular desktop or production servers.
node version 12 is from 2019 for example and is no longer supported as of april 2022.
You can either use sudo apt-get install --only-upgrade nodejs
Or
sudo apt-get remove nodejs
curl -fsSL https://deb.nodesource.com/setup_current.x | sudo -E bash -
sudo apt-get install nodejs

You can also try
sudo apt update && sudo apt upgrade
If the above does not work try searching "How to update nodejs on ubuntu"
I hope this solves it. If not report back your results.


Added "--reprocess" for the update-pipeline

Use "--reprocess" for update-pipeline to manually trigger jobs for the update

Many thanks for the pull @sillystring13 before we can get this merged we'll need you to sign our CLA.
Other than that everything LGTM.

Apologies for the lateness - the form is signed

Actually, It looks like this was already updated (not quite the same way) on our end in the interim. Re-closing, but thanks again for the CLA and PR @sillystring13. We always love contributions like this!


How to prepare the training data

Hi,
"We fine-tune the 7B and 13B models with 80k and 18k conversations, respectively."
Could you provide more details about the training data? How the 80k data  are prepared? Are they all with length of 16k?
Is the data used for training longchat-v1.5  the same as previous version?

Same question about longchat-v1.5. Cannot find any details about the  longchat-v1.5.

@Mooler0410 oh it is the same, we just use the same data, but based on llama2


Show file transfer progress

Is there a way to show the file transfer progress on the node.js console after having started the http-server?

Because of how a file transfer works, the server doesn't exactly know how much has been transferred. So it makes more sense to show this on the client side.
Also, http-server is based on ecstatic, so ecstatic has to support it. I don't think this should be part of a small & lightweight HTTP server.

On the client, things like wget, curl, a WebDAV client or a Browser might show you progress.


Debugging tools: better tracebacks with cell links

Description
Debug cells that failed to run. Sometimes they are using functions defined in other cells, or imported modules.
Suggested solution
Multiple features would be helpful, below:
Alternative

%debug magic like Jupyter?
More readable tracebacks? Currently traceback has cells named with a hash, I wonder if this could be mapped to the 'cell number' that is displayed e.g. in dependency explorer (and/or if there's an option to toggle display of cell names)

Traceback (most recent call last):
  Cell /tmp/marimo_73071/__marimo__cell_XQaa__output.py, line 1
    f
NameError: name 'f' is not defined

Related, maybe better: when there's a traceback, make the trace levels in the notebook clickable to jump to the cell (line?) in question, as seems to already be supported when there's a cycle.

Additional context
No response

Closed by #1904 !!


Add documentation for extended division intrinsics

_udiv128, _div128, _udiv64, _div64 intrinsics were added in Visual Studio 2019.

@Qartar : Thanks for your contribution! The author, @, has been notified to review your proposed change.

@Qartar : Thanks for your contribution! The author, @, has been notified to review your proposed change.

Hi @Qartar
Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you review staging and fix all validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
For more information, see Should I work in a private or a public repo?
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.
Would you create a pull request in the private repo to make these updates? We'll close this PR. Thank you.
CC @corob-msft


[WIP] Implement a generic row iterator

Codecov Report

Merging #179 (dfff26b) into master (79539dc) will decrease coverage by 1.94%.
The diff coverage is 51.11%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master     #179      +/-   ##
==========================================
- Coverage   91.40%   89.46%   -1.94%
==========================================
  Files          28       29       +1
  Lines         896      940      +44
==========================================
+ Hits          819      841      +22
- Misses         77       99      +22

Files Changed
Coverage Δ

src/TableTransforms.jl
100.00% <ø> (ø)

src/tablerows.jl
50.00% <50.00%> (ø)

src/transforms/filter.jl
98.46% <100.00%> (ø)


feat(scanv4): Add upgrade E2E test using roxctl with scanner V4

Description
Add E2E test for upgrading using roxctl to scanner V4 version
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
TODO(replace-me)
Use this space to explain how you validated that your change functions exactly how you expect it.
Feel free to attach JSON snippets, curl commands, screenshots, etc. Apply a simple benchmark: would the information you
provided convince any reviewer or any external reader that you did enough to validate your change.
It is acceptable to assume trust and keep this section light, e.g. as a bullet-point list.
It is acceptable to skip testing in cases when CI is sufficient, or it's a markdown or code comment change only.
It is also acceptable to skip testing for changes that are too taxing to test before merging. In such case you are
responsible for the change after it gets merged which includes reverting, fixing, etc. Make sure you validate the change
ASAP after it gets merged or explain in PR when the validation will be performed.
Explain here why you skipped testing in case you did so.
Have you created automated tests for your change? Explain here which validation activities you did manually and why so.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/retest

/retest

/retest

/retest

/retest

/test gke-nongroovy-e2e-tests

/retest

Superseded by https://github.com/stackrox/stackrox/pull/9751


make some flags "global" (PersistentFlags)

Codecov Report

Merging #200 (cd9e169) into main (11828e8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #200   +/-   ##
=======================================
  Coverage   69.46%   69.46%
=======================================
  Files          44       44
  Lines        2224     2224
=======================================
  Hits         1545     1545
  Misses        436      436
  Partials      243      243

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 11828e8...cd9e169. Read the comment docs.


fix grails version detection for windows line endings

This change is to support Windows-style line endings when detecting the Grails versions in config file. It also extracts out the function to a new file for de-duplication and better testing. It uses the same style of version detection as the Play! buildpack, which does not care about the line ending itself, but rather stops when the value is not [A-Za-z0-9.-] and relies on grep for breaking into lines. I added the other matching characters because Grails sometimes uses version modifiers, like 1.2.3-RC, whereas Play! does not.
Note, the testing on this on a real Windows is still pending, but included unit tests create a Windows-style CRLF file.

I just tested this on a Windows instance on EC2 with both Grails 1.3.7 and 2.0.0. The issue is reproducible in both versions with the curent BP and is fixed with my pull request. Merge away!

Made requested changes.


Make pry plugins configurable

Currently, all pry plugins are required. In my case, I don't need some, for example pry-git, which holds the mime-types gem hostage. It would be great to be able to configure only those you need.
Ironically, this was somewhat possible while this bug was present ;)

@aried3r Gotcha. While what you are saying make sense i believe it would break the idea of this repo which is to be a gem to add everything for you without worries but at the same time i do agree that some of the dependencies are a pain in the ass because of bugs, just like one i encountered some time ago with yard and ENV variables.
So, with all this context, do you have a solution in mind? I believe we could work towards something like the jazz_fingers/setup file. What do you think?

Also at the moment pry-coolline fails to work on Windows ( https://github.com/pry/pry-coolline/issues/31 ) presumably because of a bug in coolline ( https://github.com/Mon-Ouie/coolline/issues/19 ).
So I had to uninstall jazz_fingers to be able to use pry again :(
If it was possible to choose which plugin to (not) load I could have avoided that..

jazz_fingers is just a thin layer on top of several other gems. If we have a problem with the dependencies i'm afraid that making them configurable sounds as a workaround instead of a fix.
Having this in mind i think that the only issues that need to be addressed in this repo are related to conflicts between those dependencies instead of trying to solve those issues in this repository.
As to be able to use jazz_fingers without those issues i think that one possible solution is to be pessimistic on the dependencies and avoid broken ones. Or not not have fixed dependencies at all in a way that the final user can easily fix any problem by forcing a specific version. What do you think?

jazz_fingers is just a thin layer on top of several other gems. If we have a problem with the dependencies i'm afraid that making them configurable sounds as a workaround instead of a fix.

It's just that the dependencies are old. pry-git hasn't been updated since 4 years ago which itself depends on grit which isn't maintained anymore. Maybe make the very old dependencies, beginning with pry-git optional? It's mostly this one which holds dependencies "hostage" in my case.
I'll have a closer look at the code of this repo what exactly it does and if I need the glue. I really enjoy this projects, but depending on your decision regarding dependencies, I'll probably fork or cook my own. This is not a threat, I really do appreciate your work on reviving jazz_hands! :)
That being said, I don't know enough about gem development but from what I've gathered, true optional dependencies aren't possible. Usually just using something like
begin
  require 'a'
rescue LoadError
  # The 'a' gem is not installed
  require 'b'
end

Got it. I think we should just get rid of pry-git then.
As for the dependency, is not that easy. The real problem is here.
If we do remove it from the gemspec then the user will have to load everything by hand and thus removing the need of this project. Maybe we can work on the dependencies and keep only the active and working ones. What do you think?

Keeping only the active ones sounds good as a first step. :)
Also: https://bibwild.wordpress.com/2015/09/09/optional-gem-dependencies
On Wed, Sep 9, 2015, 19:36 Paulo Henrique Lopes Ribeiro <
<EMAIL_ADDRESS>wrote:

Got it. I think we should just get rid of pry-git then.
As for the dependency, is not that easy. The real problem is here
https://github.com/plribeiro3000/jazz_fingers/blob/master/jazz_fingers.gemspec#L21-L29
.
If we do remove it from the gemspec then the user will have to load
everything by hand and thus removing the need of this project. Maybe we can
work on the dependencies and keep only the active and working ones. What do
you think?
—
Reply to this email directly or view it on GitHub
https://github.com/plribeiro3000/jazz_fingers/issues/9#issuecomment-138985991
.

Alright. Do you mind to send a PR for that @aried3r?
Otherwise i will do it myself but might take some time to do it.

Will try, taking a quick glimpse, I couldn't find anything relating to pry-git except for the gem spec file and https://github.com/plribeiro3000/jazz_fingers/blob/master/lib/jazz_fingers.rb#L7, so removing those should do it, right?

Yep. Exactly. =)

Closing this since #12 was already merged and released as in 3.0.0.
Thanks for your work here @aried3r !


Fix/monitoring flag

Proposed changes
Set the monitoring flag to true as default in the abci connection
Fixes
n/a
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

[x] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[ ] Breaking feature (breaking change which adds functionality)
[ ] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING doc
[x] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[x] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have locally run services that could be impacted and they do not present failures derived from my changes

Fixed here: https://github.com/valory-xyz/open-autonomy/pull/1112


Option have Payee always be Amazon.com or other default

I like to be able to track how much I spend on Amazon with the reports and looking at how much per payee.

Thanks for the feature suggestion, @dhcrain. I've added it in v1.5.0.

Thanks for the feature suggestion, @dhcrain. I've added it in v1.5.0.


enum constant chariot.model.Enums.VariantName.fromPosition missing

Hello,
the enum is missing a value for the fromPosition gamemode. The following code throws an error while trying to challenge a fromPosition match:
ClientAuth client = Client.auth(token);
Stream<StreamEvent> events = client.bot().streamEvents().stream();
events.forEach(event ->
{
    switch(event.type())
    {
        case challenge:
            ChallengeEvent challenge = (ChallengeEvent) event;
            System.out.println(challenge.challenge().variant());
            break;
        default:
            break;
    }
});

Ah, I hope this one got fixed in 0.0.38 - thanks for reporting!

Works!

Cool! Thanks for confirming!


TypeError at grad_cache/functional.py:39

I tried to follow your example in the functional approach.
But, I got the following TypeError.

TypeError                                 Traceback (most recent call last)
Input In [102], in <cell line: 1>()
1 for f, r in zip(closures_x, cache_x):
----> 2     f(r)
File ~/git/GradCache/src/grad_cache/functional.py:39, in cached..cache_func..forward_backward_func(cache_reps)
36     cache_reps = (cache_reps,)
37 assert len(reps) == len(cache_reps)
---> 39 surrogate = sum(map(lambda u, v: torch.dot(u.flatten(), v.grad.flatten()), zip(reps, cache_reps)), 0)
40 surrogate.backward()
TypeError: () missing 1 required positional argument: 'v'
Is it right to change the 39th line as like this ?
surrogate = sum(map(lambda u, v: torch.dot(u.flatten(), v.grad.flatten()), reps, cache_reps), 0)

Thanks for catching that.

Is it right to change the 39th line as like this ?
surrogate = sum(map(lambda u, v: torch.dot(u.flatten(), v.grad.flatten()), reps, cache_reps), 0)

That's correct. I guess was confused between map and starmap.

Hi there,
I encounted the same error today, could you help fix the bug in the main branch? or I can send a PR fix if you wish.
Thanks!

Hi @nforest I have pushed a fix to main.
As a side note, PRs are very welcome! (This one is very simple so I figure no need to bother..)

Thanks a lot!


[Remote Cluster State] Refactor BlobStoreformat classess

Is your feature request related to a problem? Please describe
There are 2 classes (ChecksumBlobStoreFormat and BaseBlobStoreFormat) which have the logic for serde of XContent objects. For serde of Writable objects, a new class ChecksumWritableBlobStoreForamt has been created. Some code has been duplicated across classes. and it difficult extend existing logic.
Describe the solution you'd like
Create a interface which provides generic methods to serde of different types of objects. The common logic will be to added header and footer with checksum.
Related component
Cluster Manager
Describe alternatives you've considered
No response
Additional context
No response

[Triage - attendees 1 2 3 [4]https://github.com/akolarkunnu/ [5]https://github.com/lukas-vlcek/ [6]https://github.com/shiv0408/]
Thanks for filing this issue @soosinha . Can you please update the issue to expand a bit on "Some code has been duplicated across classes" and on the proposal around what common logic is proposed to be moved?


Bais in the conv layer

Hi,
I find details about resnet 50 and resnet 18. In resnet 50, the conv layers don't have bais. However, in resnet 18, the conv layers have bais. They are all initialized from pre-training. Could you tell me why?

I don't think I have specifically made that distinction anywhere.

If you don't mean resnet50 vs resnet18 but just bias in general, I have probably added bias to compensate for the frozen batch norm. But that really shouldn't matter. And we did load the model with strict=False.

Thanks. I agree with you.


User model problem.

Greetings,
php artisan craftable-pro:generate-crud
When we run the users table with this command, the following columns were not found in available selection: 0, 1, 2, 3, 4, 5, 7, 8. gives an error and does not detect, but the table is attached in the database.

Hello,
this issue should be fixed in version v2.0.4. It was caused by one of the dependencies. Feel free to update! :)

Thanks


[Story] Share feature

https://www.fernleaf.com.my/en/stories-and-tips/susu-punca-anak-hiperaktif-kurang-selera-makan.html

@asthabh23 Share is not functional on live site, please confirm if its in scope and need to be fixed.

@mkbansal1 , already shared with customer, will follow-up on this. For now, just a placeholder for 'share' can be fine till we get confirmation


Generated config schema by Ballerina does not contain the exact type of the object

Description:
Generated config schema[1] by Ballerina does not contain the exact type of the object.
[1] -
 "config": {
   "type": "object",
   "properties": {
      "clientId": {
        "type": "string"
      }
    }
}

Describe your solution(s)
Include the exact type of the object in the json. As an example, type of config variable in the above mentioned example is drive:ListenerConfig
Related Issues (optional):
https://github.com/wso2-enterprise/choreo/issues/11859

With the above fix, you'll get a name field for the record type object in the schema as follows.
e.g. https://lib.ballerina.io/ballerina/http/2.2.1/records/BearerTokenConfig
code:
configurable http:BearerTokenConfig bearerToken = ?;
Portion of generated config schema JSON:
.......
            "bearerToken": {
              "type": "object",
              "properties": {
                "token": {
                  "type": "string"
                }
              },
              "additionalProperties": false,
              "required": [
                "token"
              ],
              "name": "ballerina/http:2.2.1:BearerTokenConfig",
              "description": ""
            }
.......

Seems like this change is not there in 2201.0.4 . @Dilhasha could you please check and confirm?

The fix is not available in 2201.0.4. It will be available in 2201.1.0. My apologies for the inconvenience caused.


Support for subquadratic attention methods such as Linear Attention

Hello!
As you probably know, there are developments proposing to switch away from the traditional transformer's attention architecture due to its quadratic context cost. While the approaches such as Mamba are too exotic and may be too complicated for the existing pipelines, such as ControlNet-Transformer, other sub-quadratic alternatives have been proposed recently. An example is ReBased Linear Transformers with Learnable kernels https://github.com/corl-team/rebased which seems to fare better than Mamba
Also it may be worth to take a look at Large World Model's ring attention https://github.com/lucidrains/ring-attention-pytorch enabling it to extend its context window to millions of tokens while reliably answering the needle in the haystack test
Here's my implementation for Latte https://github.com/Vchitect/Latte/pull/51

We will look into this, but not recently as there are tasks with higher priority.


Update analytic expression in variance test

Context:
A test case for a displaced squeezed circuit was added for calculating variances. However, the analytic expression provided was incorrect.
Description of the Change:
Updates the analytic expression used for the displaced squeezed variance test.
Benefits:
The tolerance value used can be decreased.
Possible Drawbacks:
N/A
Related GitHub Issues:
N/A

Was revisiting analytic expressions for https://github.com/XanaduAI/pennylane-sf/pull/42 and this turned out to be amiss. :slightly_smiling_face:
Yes, the increased tolerance made it pass. :confused: If the parameters for displacement and squeezing are high and the Fock backend is used (with lower cutoff values), then the difference between the obtained result for the variance and the analytic value can be greater than the standard tolerance used here. However, it turned out this was not the case.


Add Peek methods for NetDataReader class

Added peek methods. That will be enough?

That will be more than enough for me, thank you!


How to run background tasks within a custom APIRoute?

First check

[x ] I added a very descriptive title to this issue.
[ x] I used the GitHub search to find a similar issue and didn't find it.
[ x] I searched the FastAPI documentation, with the integrated search.
[x ] I already searched in Google "How to X in FastAPI" and didn't find any information.
[ x] I already read and followed all the tutorial in the docs and didn't find an answer.
[ x] I already checked if it is not related to FastAPI but to Pydantic.
[x ] I already checked if it is not related to FastAPI but to Swagger UI.
[x ] I already checked if it is not related to FastAPI but to ReDoc.
[ x] After submitting this, I commit to one of:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

I'm trying to work out how to add background tasks into my custom route handler. I read the part in the docs about using starlette response object when instantiating backround task on my own but am hoping its possible to do it the fastapi native way.
import time
from typing import Callable

from fastapi import APIRouter, FastAPI, Request, Response, BackgroundTasks
from fastapi.routing import APIRoute
from starlette.responses import JSONResponse

def bg_task(msg):
    time.sleep(2)  # simulate blocking behavior
    print(msg)

class TimedRoute(APIRoute):
    def get_route_handler(self) -> Callable:
        original_route_handler = super().get_route_handler()

        async def custom_route_handler(request: Request) -> Response:
            bg = BackgroundTasks()
            bg.add_task(print, "pre resp")
            bg.add_task(bg_task, "pre resp sleep")
            before = time.time()
            response: Response = await original_route_handler(request)
            duration = time.time() - before
            response.headers["X-Response-Time"] = str(duration)
            print(f"route duration: {duration}")
            print(f"route response: {response}")
            print(f"route response headers: {response.headers}")
            bg.add_task(print, "post resp")
            bg.add_task(bg_task, "post resp sleep")
            # await bg() ( this works but blocks.. )
            return JSONResponse(response.body.decode(), background=bg) # i would rather return the response directly. trying to avoid building the response myself.

        return custom_route_handler

app = FastAPI()
router = APIRouter(route_class=TimedRoute)

@app.get("/")
async def not_timed():
    return {"message": "Not timed"}

@router.get("/timed")
async def timed():
    return {"message": "It's the time of my life"}

@app.get("/bg-test", description="Am I Alive?")
async def ping(background_tasks: BackgroundTasks):
    background_tasks.add_task(bg_task, "i wont block")
    return {"ping": "pong"}

app.include_router(router)

I don't get it , write what you would like to have even if the code is not valide

I want to be able to use background_tasks as fastapi recommends (path parameter). However in order to use background_tasks without being a path param the docs say to instantiate it yourself via starlette docs(https://www.starlette.io/background/). Here's a code sample of what I would like to happen.
class AuditedRoute(APIRoute):
    def get_route_handler(self) -> Callable:
        original_route_handler = super().get_route_handler()

        # this will throw "TypeError: custom_route_handler() missing 1 required positional argument: 'background_tasks' "
        async def custom_route_handler(request: Request, background_tasks: BackgroundTasks) -> Response:
            try:
                response: Response = await original_route_handler(request)
            except RequestValidationError as exc:
                body = await request.body()
                detail = {"errors": exc.errors(), "body": body.decode()}
                raise HTTPException(status_code=422, detail=detail)
            background_tasks.add_task(send_message, 'foo bar')
            return response

        return custom_route_handler

Why do you need a custom route handler? Wouldn't you be able to do most of this with dependencies or a middleware?
I think it can help if you comment what's your intention, what you are trying to solve, what is the main problem, before thinking about the specific implementation. I think there's a high chance there's a better way to achieve what you need, but I'm not sure yet what you need.

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.

hey!  So I will preface this with I no longer need this solution. but for whats its worth, let me try again. my question was: given a custom api route class such as https://fastapi.tiangolo.com/advanced/custom-request-and-route/#custom-apiroute-class-in-a-router,  how would I inject the background_tasks object into the custom_route_handler?
the requirements from what i remember.. sorry its been a while 😬

i needed the response object so i could then submit that to a background task to be processed in a non-async way.
middleware wasn't option since at the time middleware could not be applied at the route level. however, looks like this is not the case anymore, and I would probably just use middleware for this now. https://github.com/tiangolo/fastapi/issues/1174#issuecomment-918322569
dependencies didn't work because I need the response object.

closing since I think this is resolved via middleware


Backspace doesn't work on Colemak keyboard layout

Issue Type: Bug
Hi, I use Colemak keyboard layout instead of Qwerty. I remapped Caps Lock to Backspace and now I have two backspaces, but remapped one(old Caps Lock) doesn't work on VSCode, it does nothing.
VS Code version: Code - OSS 1.36.0 (0f3794b38477eea13fb47fbe15a42798e6129338, 2019-07-09T11:32:10.645Z)
OS version: Linux x64 5.2.0-arch2-1-ARCH
Extensions (7)

Extension
Author (truncated)
Version

vscode-eslint
dba
1.9.0

beautify
Hoo
1.5.0

python
ms-
2019.6.24221

Go
ms-
0.11.4

java
red
0.46.0

vscode-wakatime
Wak
2.1.2

vscode-postfix-go
yok
0.0.12

resolved with "keyboard.dispatch": "keyCode"


text span selection problem when adding source of grounding

Paper ID: 1711.09576

s2.p3 last sentence "The RNN-acceptor"->

s2.p7.l7 "PyTorch framework"->

s2.p7.l7 " " (space between PyTorch and framework)->(result differs sometimes)

(~some parts of the incorrectly selected span does not allow "delete source"~ <- footenote)
(and sometimes page position moves)
S2.p3.13.m<IP_ADDRESS>.2 ("returns an accept or reject decision" -> as bellow)

Paper ID: 1711.09576

S5.F5.16.1.m1.1.1("a recurrent neural network" -> as below)

For 1711.09576 s2.p3 "The RNN acceptor"
its corresponding arxmliv is as follows:
on></semantics></math> and returns an accept or reject
decision.
The RNN-acceptor <math id="S2.p3.1

At least for this example, the problem is that "reject", "decision." and "The" are splitted by "\n" (newline) instead of " " (white space). That's why the preprocessing script (https://github.com/wtsnjp/MioGatto/blob/e3ff70e36e44950dbf7c6d6b0ffa0cc4c2d5a3ff/tools/preprocess.py#L49) does not work well.


Add Windows support and other enhancements

Add Windows support;
add prompt_color to all classes;
add HOME/END keys to Bullet and Check;
add HOME/END/PGUP/PGDN keys to Scrollbar.

This is my first try at creating a pull request.  I have found bullet helpful in my other console-based projects but needed Windows support and added the other updates as my need for them evolved.
I hope you can accept these updates and merge them into your fine contribution to python projects.
Regards,
Peter

Thank you so much @pjfarleyiii ✨ !!

Update: I published bullet including this PR under: winBullet.
Feel free to use it.


[Beta 4] Cannot make the research table

Hi,
I am having an issue when I try to create the research table. I have the scribing tools and the wood table. I place the wood table and I am trying to put the scribing tools on it, but it does not work. I am using more mods with a pack I made on the twitch app. Can it be some other mod that is making this?
This is the export of my profile, if it helps.
Omnius Pack 1.10-1.0.zip

Sorry about creating an issue without digging too much into it myself. I am not deleting the ticket because it may be it is of your interest as well. It looks like some other mod changes the way the player interacts with objects, which causes the behaviour I described before. I noticed that in the logs of my server (which I am posting with this). Could someone help me figure out what is the problem? Is it Survival Tweaks that is causing it?
11.02 19:13:26 [Server] Server thread/ERROR [FML]: Exception caught during firing event net.minecraftforge.event.entity.player.PlayerInteractEvent$RightClickBlock@236c93b9:
11.02 19:13:26 [Server] INFO java.lang.NullPointerException
11.02 19:13:26 [Server] INFO at thaumcraft.common.blocks.basic.BlockTable.func_180639_a(BlockTable.java:64) ~[BlockTable.class:?]
11.02 19:13:26 [Server] INFO at sorazodia.survival.mechanics.PlayerActionEvent.blockRightClick(PlayerActionEvent.java:124) ~[PlayerActionEvent.class:?]
11.02 19:13:26 [Server] INFO at net.minecraftforge.fml.common.eventhandler.ASMEventHandler_575_PlayerActionEvent_blockRightClick_RightClickBlock.invoke(.dynamic) ~[?:?]
11.02 19:13:26 [Server] INFO at net.minecraftforge.fml.common.eventhandler.ASMEventHandler.invoke(ASMEventHandler.java:90) ~[ASMEventHandler.class:?]
11.02 19:13:26 [Server] INFO at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:185) [EventBus.class:?]
11.02 19:13:26 [Server] INFO at net.minecraftforge.common.ForgeHooks.onRightClickBlock(ForgeHooks.java:1026) [ForgeHooks.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.server.management.PlayerInteractionManager.func_187251_a(PlayerInteractionManager.java:435) [lv.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.network.NetHandlerPlayServer.func_184337_a(NetHandlerPlayServer.java:679) [me.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.network.play.client.CPacketPlayerTryUseItemOnBlock.func_148833_a(SourceFile:55) [jj.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.network.play.client.CPacketPlayerTryUseItemOnBlock.func_148833_a(SourceFile:11) [jj.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.network.PacketThreadUtil$1.run(SourceFile:13) [fl$1.class:?]
11.02 19:13:26 [Server] INFO at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [?:1.8.0_152]
11.02 19:13:26 [Server] INFO at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_152]
11.02 19:13:26 [Server] INFO at net.minecraft.util.Util.func_181617_a(SourceFile:45) [h.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:668) [MinecraftServer.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.server.dedicated.DedicatedServer.func_71190_q(DedicatedServer.java:387) [ld.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:613) [MinecraftServer.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:471) [MinecraftServer.class:?]
11.02 19:13:26 [Server] INFO at java.lang.Thread.run(Thread.java:748) [?:1.8.0_152]
11.02 19:13:26 [Server] Server thread/ERROR [FML]: Index: 17 Listeners:
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 0: HIGHEST
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 1: ASM: com.attributestudios.wolfarmor.event.WolfArmorPlayerEventHandler@480f78b onPlayerInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 2: ASM: net.quetzi.morpheus.helpers.MorpheusEventHandler@612e4086 bedClicked(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 3: NORMAL
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 4: ASM: class vazkii.botania.common.block.string.BlockRedStringInterceptor onInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 5: ASM: vazkii.botania.common.item.material.ItemManaResource@71d22366 onPlayerInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 6: ASM: class vazkii.botania.common.item.relic.ItemLokiRing onPlayerInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 7: ASM: com.animania.events.ItemSeedHandler@2189c468 notify(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 8: ASM: vazkii.quark.vanity.feature.SitInStairs@d43a6b3 onInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 9: ASM: vazkii.quark.tweaks.feature.DeployLaddersDown@20f45be6 onInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 10: ASM: vazkii.quark.tweaks.feature.RemoveSnowLayers@4cb01bb5 onPlayerRightClick(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 11: ASM: vazkii.quark.tweaks.feature.RightClickSignEdit@7e5b76a1 onInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 12: ASM: biomesoplenty.common.handler.GrassPathEventHandler@63494c10 onBlockRightClick(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 13: ASM: class team.chisel.common.item.ChiselController onRightClickBlock(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 14: ASM: com.mrdimka.hammercore.event.TileHandler@599729dd playerInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 15: ASM: mods.railcraft.common.carts.MinecartHooks@29779ec3 onItemUse(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 16: ASM: com.pam.harvestcraft.addons.RightClickHarvesting@af0743a onPlayerInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 17: ASM: sorazodia.survival.mechanics.PlayerActionEvent@51851cec blockRightClick(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 18: ASM: thaumcraft.common.lib.events.ToolEvents@7a905042 playerRightClickBlock(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 19: ASM: toughasnails.handler.thirst.FillBottleHandler@6f3e6dd7 onRightClickCauldron(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 20: ASM: mod.chiselsandbits.events.EventPlayerInteract@7db7b145 interaction(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 21: ASM: com.enderio.core.common.handlers.RightClickCropHandler@50801fcc handleCropRightClick(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 22: ASM: mcmultipart.item.PartPlacementWrapper@1227201e onPlayerRightClickBlock(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 23: ASM: mcmultipart.item.PartPlacementWrapper@55b4baa5 onPlayerRightClickBlock(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 24: ASM: flaxbeard.cyberware.common.handler.EssentialsMissingHandler@46135cb0 handleRightClickBlock(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 25: ASM: com.jaquadro.minecraft.storagedrawers.core.CommonProxy@106d317c playerRightClick(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 26: ASM: com.pau101.fairylights.server.ServerEventHandler@343d7b8a onRightClickBlock(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 27: ASM: shetiphian.platforms.common.misc.EventHandler@6db3608c playerRightClickBlock(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 28: ASM: zmaster587.advancedRocketry.event.PlanetEventHandler@5320bd21 blockPlaceEvent(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 29: ASM: slimeknights.tconstruct.tools.ToolEvents@7bb561a4 onInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 30: ASM: betterwithmods.event.BucketEvent@7c5b853b fluidContainerUse(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 31: ASM: betterwithmods.event.LogHarvestEvent@20eec178 debarkLog(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 32: ASM: iguanaman.hungeroverhaul.module.event.HungerOverhaulEventHook@32d6f646 onRightClickBlock(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 33: ASM: biomesoplenty.common.handler.AchievementEventHandler@2b3907e9 onItemUsed(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 34: ASM: WayofTime.bloodmagic.util.handler.event.LivingArmourHandler@7e848823 onPlayerClick(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 35: ASM: WayofTime.bloodmagic.util.handler.event.GenericHandler@1d85dff7 onPlayerClick(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 36: ASM: minetweaker.mc1102.ForgeEventHandler@155886da onPlayerInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 37: ASM: toughasnails.handler.AchievementEventHandler@4993526e onItemUsed(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 38: ASM: appeng.parts.PartPlacement@1934de0e playerInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 39: ASM: pl.asie.computronics.integration.forestry.nanomachines.SwarmProvider@6c039d1c onPlayerInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 40: ASM: CoroUtil.forge.EventHandlerForge@1c45eec2 blockPlayerInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 41: ASM: sonar.core.SonarEvents@6c9c5978 playerInteracts(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 42: ASM: ZombieAwareness.ZAEventHandler@3a874cb1 interact(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 43: LOW
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 44: ASM: codechicken.multipart.minecraft.EventHandler@5064bb93 playerInteract(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 45: LOWEST
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 46: ASM: flaxbeard.immersivepetroleum.common.EventHandler@253741d9 handlePickupItem(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/ERROR [FML]: 47: ASM: elec332.core.main.ElecCore$1@68a3753d onItemRightClick(Lnet/minecraftforge/event/entity/player/PlayerInteractEvent$RightClickBlock;)V
11.02 19:13:26 [Server] Server thread/FATAL Error executing task
11.02 19:13:26 [Server] INFO java.util.concurrent.ExecutionException: java.lang.NullPointerException
11.02 19:13:26 [Server] INFO at java.util.concurrent.FutureTask.report(FutureTask.java:122) ~[?:1.8.0_152]
11.02 19:13:26 [Server] INFO at java.util.concurrent.FutureTask.get(FutureTask.java:192) ~[?:1.8.0_152]
11.02 19:13:26 [Server] INFO at net.minecraft.util.Util.func_181617_a(SourceFile:46) [h.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:668) [MinecraftServer.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.server.dedicated.DedicatedServer.func_71190_q(DedicatedServer.java:387) [ld.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:613) [MinecraftServer.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:471) [MinecraftServer.class:?]
11.02 19:13:26 [Server] INFO at java.lang.Thread.run(Thread.java:748) [?:1.8.0_152]
11.02 19:13:26 [Server] INFO Caused by: java.lang.NullPointerException
11.02 19:13:26 [Server] INFO at thaumcraft.common.blocks.basic.BlockTable.func_180639_a(BlockTable.java:64) ~[BlockTable.class:?]
11.02 19:13:26 [Server] INFO at sorazodia.survival.mechanics.PlayerActionEvent.blockRightClick(PlayerActionEvent.java:124) ~[PlayerActionEvent.class:?]
11.02 19:13:26 [Server] INFO at net.minecraftforge.fml.common.eventhandler.ASMEventHandler_575_PlayerActionEvent_blockRightClick_RightClickBlock.invoke(.dynamic) ~[?:?]
11.02 19:13:26 [Server] INFO at net.minecraftforge.fml.common.eventhandler.ASMEventHandler.invoke(ASMEventHandler.java:90) ~[ASMEventHandler.class:?]
11.02 19:13:26 [Server] INFO at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:185) ~[EventBus.class:?]
11.02 19:13:26 [Server] INFO at net.minecraftforge.common.ForgeHooks.onRightClickBlock(ForgeHooks.java:1026) ~[ForgeHooks.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.server.management.PlayerInteractionManager.func_187251_a(PlayerInteractionManager.java:435) ~[lv.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.network.NetHandlerPlayServer.func_184337_a(NetHandlerPlayServer.java:679) ~[me.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.network.play.client.CPacketPlayerTryUseItemOnBlock.func_148833_a(SourceFile:55) ~[jj.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.network.play.client.CPacketPlayerTryUseItemOnBlock.func_148833_a(SourceFile:11) ~[jj.class:?]
11.02 19:13:26 [Server] INFO at net.minecraft.network.PacketThreadUtil$1.run(SourceFile:13) ~[fl$1.class:?]
11.02 19:13:26 [Server] INFO at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) ~[?:1.8.0_152]
11.02 19:13:26 [Server] INFO at java.util.concurrent.FutureTask.run(FutureTask.java:266) ~[?:1.8.0_152]
11.02 19:13:26 [Server] INFO at net.minecraft.util.Util.func_181617_a(SourceFile:45) ~[h.class:?]
11.02 19:13:26 [Server] INFO ... 5 more

It does indeed seem to be breaking things.

I can confirm that removing Survival Tweaks was making that issue, as it changes the way the player interacts with blocks when right-clicking. Removing the mod fixed the issue.

SoraZodia has promoted this issue as a bug. It might be fixed from is side? We'll see
https://github.com/SoraZodia/Survival-Tweaks/issues/18


Alias 0.5.0

Added az alias import, az alias export, az alias remove-all commands + minor fixes.
This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

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

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT<EMAIL_ADDRESS>

With the current core and the current version of Alias, the following fails:
az alias create -n del-grp -c "group delete tjp-rg"
alias: Invalid Azure CLI command "group delete tjp-rg"

But this succeeds
az alias create -n del-grp -c "group delete -g tjp-rg"

@tjprescott how can I try the positional argument feature? I pulled the latest change from dev and ran python scripts/dev_setup.py. When I ran az group delete <resource_group_name>, I got the following error:
az group delete: error: the following arguments are required: --name/-n/--resource-group/-g

@tjprescott is it possible to have two positional arguments?

@chewong it depends what you mean.  You can only have one argument per command that is positional, but you COULD have that one argument be nargs='+' or nargs='*' where it could accept multiple things.  So whether you consider that "multiple positional" arguments are not is up to you.

@tjprescott I've incorporated alias with your positional argument change in core.


Read Until Prompt is not reading properly on windows

Hi,
I have simple code connecting to Windows Host (Through ssh) and trying to read .
Here is what i did
ENV_VAR on Windows Host is HELLO
Open Connection <win_host_ip> Prompt=>
${written}=Write echo %ENV_VAR%
${read_out}= Read Until Prompt
Log ${read_out}
I see following output : HELLO  ELLO LLO LO O
On unix host i see HELLO
Why read until is reading ELLO LLO LO O characters.
when i pass newline=CRLF in open connection, Read Until Prompt never reads and timesout.

You are using SSHLibrary, right? In this case this should be submitted to its issue tracker.


Certain key combinations are treated wrong in MacOS

VSCode Version: 0.10.11
OS Version: MacOS 10.11.2

Steps to Reproduce:

Install the 'Advanced New File' extension or create a shortcut that is listening for Mac on alt+cmd+n
It displays the keyboard shortcut correctly in the command palette
It doesn't trigger the respective command.
Changing it to `alt+cmd+`` will do the trick and trigger the shortcut when you press Alt+Cmd+N

It appears that this is happening because Alt+N is an actual special character and rather than triggering the actual keys it handles the special characters.

@egamma @bpasero could you share your concerns?

@dkundel What keyboard layout do you have?

I have a US keyboard layout. Not sure about the others. This has been reported by a couple of folks.

Same issue here, can't use neither cmd+alt+n nor alt+cmd+n for shortcut in vscode.
MacOS: 10.9.5
VSCode: 1.5.3
Keyboard Layout: US
At the same time cmd+alt+m & alt+cmd+m works fine

I @dkundel @ihorskyi Do other applications on your machine recognize alt+cmd+n. What happens when you press the key combo in the Define Keybinding widget?

It shows Cmd+Alt+` in the widget.

@dkundel I have just written a wiki describing in detail how we handle keybindings in VS Code -- https://github.com/Microsoft/vscode/wiki/Troubleshooting-Keybindings. Would you mind giving it a quick read so we end up talking about the same things.
From seeing cmd+alt+`, it appears that Chromium is sending our way a keydown event with the keyCode = VKEY_OEM_3 = 192. Would you mind checking that is indeed the case:
Go to https://www.w3.org/2002/09/tests/keys.html in Chrome and Safari and press the key combination cmd+alt+n. Please paste in here what events you see on your machine (a screenshot or copy-paste the results below show).

It indeed sends the keyCode 192 for N when it's used with cmd+alt
Here are the results:
keydown (0): [keyCode: 91][shift: false][ctrl: false][alt: false][meta: true]
keydown (1): [keyCode: 18][shift: false][ctrl: false][alt: true][meta: true]
keydown (2): [keyCode: 192][shift: false][ctrl: false][alt: true][meta: true]

I have validated that we now (i.e. after PR #22894 aka #17521 -- in today's upcoming Insiders build) appear to detect correctly this keypress.

Additionally, I have written the following keybinding rule in my keybindings.json:
{
    "key": "alt+cmd+n",
    "command": "actions.find",
    "when": "editorTextFocus"
}

And when pressing cmd+alt+n the find widgets pops up correctly.

Thank you!


Deprecation warning when trying to validate against no rules (after conditionals)

Laravel Version: 9.10.0
PHP Version: 8.1.5

Description:
Trying to validate a field that has no rules or no truthy conditional rules results in a deprecation warning:
local.WARNING: ErrorException: array_fill(): Passing null to parameter #1 ($start_index) of type int is deprecated in /app/vendor/laravel/framework/src/Illuminate/Validation/ValidationRuleParser.php:100
Steps To Reproduce:

Configure the deprecations log as per docs
Run the following:

validator([
	'field' => 'fff',
	'other' => 'fff'
], [
	'field' => Rule::when(false, [
		'string',
	]),
	'other' => [],
])->validate();

See two deprecation warnings in the deprecations log, one per field:

[2022-04-30 20:35:07] local.WARNING: ErrorException: array_fill(): Passing null to parameter #1 ($start_index) of type int is deprecated in /app/vendor/laravel/framework/src/Illuminate/Validation/ValidationRuleParser.php:100
[2022-04-30 20:35:07] local.WARNING: ErrorException: array_fill(): Passing null to parameter #1 ($start_index) of type int is deprecated in /app/vendor/laravel/framework/src/Illuminate/Validation/ValidationRuleParser.php:100

Thanks, I've sent in a fix for this: https://github.com/laravel/framework/pull/42213


When creating a new language, keywords do not work if they are not in English

Information:

Prism version: 1.21.0
Plugins: none
Environment: Chrome, npm 6.14.6,

Description
I made a new language  BSL (1C:Enterprise) https://github.com/Diversus23/prism
In Russia, the popular development environment 1C:Enterprise 8. It is possible to use two language in the development of Russian and English. For example:
While a = 1 Do
...
EndDo;

Analog in Russian:
Пока а = 1 Цикл
...
КонецЦикла;

I use flag /u for RegExp (for Unicode characters), but but it doesn't work for Russian language (for English ok)... How do I fix this?
Pattern:
	'keyword': [
		{
			// RU
			pattern: /\b(?:пока|для|новый|прервать|...|конеццикла)\b/uig,
			lookbehind: true
		},
		{
			// Eng
			pattern: /\b(?:while|for|new|break|try|...|enddo)\b/ig,
			lookbehind: true
		}
	],

Screenshots:

Example
An example which reproduces the bug. This can be a link to a website, a JSFiddle, an attached example project, etc.

The problem is \b.
\b means word boundary where a word is defined as a \w character. But \w only includes the letters A-Z (uppercase and lowercase), digits 0-9 (ASCII only), and the underscore (_). That's it; It doesn't include any non-Latin characters. This doesn't change when adding the u flag.
So from the perspective of \b, all Russian characters are non-word characters, meaning that there is no word boundary between ( and п because neither are word characters.
Workaround 1
As per specification, \b is the same as (?:(?<=\w)(?!\w)|(?<!\w)(?=\w)). If we want the boundary of letters and digits of any language, we can simply replace the \ws with [\p{L}\p{N}_] to get (?:(?<=[\p{L}\p{N}_])(?![\p{L}\p{N}_])|(?<![\p{L}\p{N}_])(?=[\p{L}\p{N}_])) (requires the u flag). You can use this monster as a drop-in replacement for \b.
The only problem with this workaround is browser support. Both Unicode properties and ES2018 lookbehinds are fairly new. While you can use Babel to transpile Unicode properties to support older browsers, ES2018 lookbehinds require native browser support.
Workaround 2
If you only need the boundaries for keyword lists (e.g. (?:пока|для|новый|прервать|...|конеццикла)), then we can use Prism's lookbehind groups to do so. It's done like this:
	'keyword': [
		{
			// RU
			pattern: /(^|[^\p{L}\p{N}_])(?:пока|для|новый|прервать|...|конеццикла)(?![\p{L}\p{N}_])/iu,
			lookbehind: true
		},
		{
			// Eng
			pattern: /(^|[^\p{L}\p{N}_])(?:while|for|new|break|try|...|enddo)(?![\p{L}\p{N}_])/iu,
			lookbehind: true
		}
	],

Again, the problem is browser support but if you use Babel to transpile the Unicode properties, you will be able to support older browsers. We didn't use ES2018 lookbehinds, so they aren't an issue.
I hope this solves your problem.

I hope this solves your problem.

Yes it work!
This is pattern:
(?:(?<=[\p{L}\p{N}_])(?![\p{L}\p{N}_])|(?<![\p{L}\p{N}_])
It looks like sorcery :)

One moment.
If use Workaround 1 not work "npm test"
If use Workaround not work both tests "npm run test:languages -- --language=bsl --pretty" and "npm test"

I will close this issue now because regex itself works. Let's continue the discussion on how to make the tests pass in #2520.


Make YASR settings part of persistent yasgui settings

not needed anymore. YASR settings are stored persistently independently.


Legends are overlapping in Firefox

Legends are overlapping in Firefox
i have attached screenshot .

Please let me know how to resolve this

This also happens for me on random occasions, on next reload the legends can be rendered properly.

@ergo it is happening on first load itself.
i tried to reload it again, but same Result.

For me it happens on random.

Same Problem here. Any Ideas how to fix?

See issue #1005

I'm now experiencing this issue on v4.10-rc1:
Firefox:

Chrome:

I'm sorry for the basic bug like this. Now it's been fixed in v0.4.10-rc2 and I'll release this fix in v0.4.10 after rc test.

Thank you

Bug  may not been fixed in v0.4.10

Test code : http://jsfiddle.net/nzw7449m/3/
Code :
var chart = c3.generate({
    data: {
        columns: [
            ['Country 1', 6, 5.95, 6.69, 7.47, 3.53, 0.92, 7.21, 4.02, 3.97, 4.18, 4.27],
            ['Country 2', 7.45, 7.31, 8.69, 8.74, 5.7, -6.15, 4.4, 4.67, 3.8, 2.02, 3.04],
            ['Country 3', 3.82, 3.06, 2.56, 4.43, 2.2, 1.23, 2.53, 2.4, 3.53, 2.62, 2.58],
            ['Country 4', 7.58, 8.57, 8.24, 7.54, 4.69, 3.68, 7.62, 6.55, 3.64, 4.02, 4.63],
            ['Country 5', 6.48, 5.78, 6.1, 6.51, 4.27, 1.15, 8.04, 4.57, 5.51, 4.99, 5.1]
        ],
    },
    legend: {
        show: true
    }
});
d3.selectAll("#chart .c3-legend-item").style("font-size","20px");

Response on StackOverflow, is due because font-size are setting after the chart render.One solution is to set the font size as regular CSS.


artist search error

I'm trying to search for artist like so but it's throwing KeyError:
query = 'year:2017'
results = sp.search(query, limit=50, type='artist')
artists = results['artists']
while artists:
for i, artist in enumerate(artists['items']):
print("%4d %s %s" % (i + 1 + artists['offset'], artist['id'],  artist['name']))
if artists['next']:
artists = sp.next(artists)
else:
artists = None
The error I get is:
for i, artist in enumerate(artists['items']):
KeyError: 'items'
While the same type of call for "user_playlist" works fine.
For example:
playlists = sp.user_playlists('spotify')
while playlists:
for i, playlist in enumerate(playlists['items']):
print("%4d %s %s" % (i + 1 + playlists['offset'], playlist['uri'],  playlist['name']))
if playlists['next']:
playlists = sp.next(playlists)
else:
playlists = None
Anyone got any idea?

Using sp.next() returns search results in a format similar to the original results.
So just like you had to retrieve the artists from the results in the initial search with artists = results['artists'] before you could use artists['items'], you have to do the same when you're reassigning artists.
You could use this instead and it should work:
if artists['next']:
    artists = sp.next(artists)['artists']

The structure of the response for playlists is different. The items key is immediately accessible from the results, so using playlists = sp.next(playlists) and then using playlist['items'] on the next iteration doesn't result in an error.

Thanks, I got this working like so:
`query = 'year:2017'
results = sp.search(query, limit=50, type='artist')
while results:
artists = results['artists']
for i, artist in enumerate(artists['items']):
artist_ids.append(artist['id'])
if results['next']:
    results = sp.next(artists)
else:
    results = None`


ci(sources/css): add unreset bundles' build

Pull Request checklist
🚨 Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a topic/feature/bugfix branch. Don't request your main!
[x] Make sure you are making a pull request against the main branch (left side).
[x] Check the commit's or even all commits' message styles matches our requested structure.
[x] Check your code additions will fail neither code linting checks nor unit test.

Description
The goal of this PR is to add alternative css bundles which does not include Tailwind's normalizing rules.
These bundles are needed when you want to embed your product inside another one and avoid overriding the legacy rules.
Does this introduce a breaking change?

No

Thanks for this @mgrisole, I understand your need!
I test it later today :)

For me it's ok, thanks Maxime!


支持运行开发好的 Spark jars（Support to run the developed spark jars.）

Is your feature request related to a problem? Please describe.
用户直接上传开发好的 Spark Jars 进行运行。
Describe the solution you'd like
当执行用户指定的 Jars 和 MainClass 时，Spark EngineManager 不通过提交 Spark Engine 启动，而是直接提交用户指定的 Jars 和 MainClass。
Describe alternatives you've considered
也可以实现 Shell Engine，在 DSS 上面通过封装 Shell Engine 节点实现。
Additional context
如何更好的管理这种 Spark 任务？
支持 spark 多版本。

Is your feature request related to a problem? Please describe.
users directly upload the developed spark jars to run.
Describe the solution you'd like
When the user specified jars and mainclass are executed, spark engine manager will not start by submitting spark engine, but directly submit the user specified jars and mianclass to start.
Describe alternatives you've considered
Of course, you can also implement shell engine, and then encapsulate the nodes in DSS.
Additional context
How to better manage this kind of spark task?
Support spark multi version.

dolphinscheduler spark node:

It is a good idea, we are planning to support it in Linkis1.0.0.
Multi version of one engine is certainly a very important feature in Linkis1.0.0.
If you are interested in those features, welcome to join us.


Use TLS whenever possible in Manager <-> MySQL connection

Feature request:
For communication from manager to mysql server, use tls=preferred parameter to enable TLS whenever possible. This should not be a breaking change, as per the library documentation the client will only attempt to use TLS if supported by the server. But this will fix the case where the manager cannot connect to a server that requires TLS.
Use case:
Currently Manager cannot connect to a MySQL database that requires TLS, without fully configuring certs/keys (functionality added in https://github.com/dragonflyoss/Dragonfly2/pull/1015). By adding this new default, TLS will be used whenever possible (even with only password), without breaking any existing functionality.

Thx 🚀🚀🚀


Add on_progress callback

This closes out issue https://github.com/typhoeus/typhoeus/issues/554.
The on_progress callback was added to ethon but as far as I can tell BackOrder's change was never added to typhoeus.
This adds the on_progress callback and some tests.

Hi there!
Any chances of this going in? We'd love to un-fork :)

Sorry, missed your PR!
Could you add your callback https://github.com/typhoeus/typhoeus/blob/master/lib/typhoeus/request/marshal.rb#L10 as well?
Looks great otherwise, will merge.

Thanks a lot @i0rek, no worries!
Done https://github.com/typhoeus/typhoeus/pull/600/commits/8c92f089f0652fda3517c1be809fcbf4fc22ac06

Thanks for your work!


Support rotation of kubeconfigs on Update for cx self-service

Which issue this PR addresses:
Supports self-service rotation of kubeconfigs should the user admin kubeconfig expire.
What this PR does / why we need it:
Update path now rotates kubeconfigs.
Test plan for issue:

create cluster
fetch kubeconfig, check expiration
call Update() code path
fetch kubeconfig, see if expiration is newer

How do you know this will function as expected in production?

/azp run ci, e2e

Success
$ az aro get-admin-kubeconfig -n bvesel -g v4-eastus
$ cat kubeconfig | yq .users[0].user.client-certificate-data | base64 -d | openssl x509 -enddate -noout
notAfter=Oct 29 17:19:58 2025 GMT
$ az aro update -n bvesel -g v4-eastus
$ rm kubeconfig
$ az aro get-admin-kubeconfig -n bvesel -g v4-eastus
$ cat kubeconfig | yq .users[0].user.client-certificate-data | base64 -d | openssl x509 -enddate -noout
notAfter=Oct 29 19:14:12 2025 GMT

/azp run ci, e2e


Stuck in python2 mode on macOS

Issue Type: Bug
Similar to https://github.com/microsoft/vscode/issues/89037#issue-553001407,
when launching code from the terminal, I get this error:
/usr/local/bin/code: line 10: ./MacOS/Electron: No such file or directory

I'm successfully using Python3 on this system, however since python2 is still installed in the (Catalina) OS, it seems code's runtime is defaulting there.
Your PYTHONPATH points to a site-packages dir for Python 3.x but you are running Python 2.x!

I will attach my trace results for more context.
VS Code version: Code 1.42.0 (ae08d5460b5a45169385ff3fd44208f431992451, 2020-02-06T10:51:33.119Z)
OS version: Darwin x64 19.3.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8569U CPU @ 2.80GHz (8 x 2800)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
7, 4, 6

Memory (System)
16.00GB (0.09GB free)

Process Argv

Screen Reader
no

VM
0%

As was suggested in #89037, I edited code to echo realpath. Here's what I got:
added realpath echo:
/usr/local/bin/code
Your PYTHONPATH points to a site-packages dir for Python 3.x but you are running Python 2.x!
     PYTHONPATH is currently: "/Users/bryandady/Library/Python/3.7/lib/python/site-packages/:/usr/local/lib/python3.7/site-packages"
     You should `unset PYTHONPATH` to fix this.

end realpath
Your PYTHONPATH points to a site-packages dir for Python 3.x but you are running Python 2.x!
     PYTHONPATH is currently: "/Users/bryandady/Library/Python/3.7/lib/python/site-packages/:/usr/local/lib/python3.7/site-packages"
     You should `unset PYTHONPATH` to fix this.
/usr/local/bin/code: line 15: ./MacOS/Electron: No such file or directory

Could you perhaps fix your PYTHONPATH? Or make sure Python3 is ahead in the PATH of your shell?

It's not clear to me that anything is wrong with my $PYTHONPATH (I'll include it below), and my $PATH only includes paths to Python 3.7 or 3.8 resources; there are no Python 2 statements to move them ahead of.
echo $PYTHONPATH
~/Library/Python/3.7/lib/python/site-packages/:/usr/local/lib/python3.7/site-packages

The only workaround that has worked for me is to edit my /usr/local/bin/code such that the
realpath() function calls python3 -c  instead of python -c.
I imagine their are more elegant long-term solutions, but this is what I can most easily do to make it work (but must repeat after a reboot).

and my $PATH only includes paths to Python 3.7 or 3.8 resources; there are no Python 2 statements to move them ahead of.

This seems to contradict with what's happening. If you run python --version do you get version 3?

Yes, when I run it at the prompt, I get Python 3.8.0.
But when I add python --version in my /usr/local/bin/code, it prints Python 2.7.16.

Interesting. So in the same shell, when running python --version you get 3, but when running code, it picks up 2? Can you show me the full contents of /usr/local/bin/code?

Sure. Here's my current copy that's working as expected:
#!/usr/bin/env bash
#
# Copyright (c) Microsoft Corporation. All rights reserved.
# Licensed under the MIT License. See License.txt in the project root for license information.

function realpath() { python3 -c "import os,sys; print(os.path.realpath(sys.argv[1]))" "$0"; }
CONTENTS="$(dirname "$(dirname "$(dirname "$(dirname "$(realpath "$0")")")")")"
ELECTRON="$CONTENTS/MacOS/Electron"
CLI="$CONTENTS/Resources/app/out/cli.js"
ELECTRON_RUN_AS_NODE=1 "$ELECTRON" "$CLI" "$@"
exit $?

@joaomoreno I shared my working copy of code a few days ago.
Do you have any updates or any additional questions I can address?

Sorry, @bcdady, I was out of office for a while.
Is your shell bash? In your shell, if you first run bash and then python --version, what's the result?

No, Apple changed up the default shell to ZSH https://www.theverge.com/2019/6/4/18651872/apple-macos-catalina-zsh-bash-shell-replacement-features
But if I get into BASH first, I get Python v2:
bash-3.2$ python --version
Python 2.7.16
bash-3.2$

I also use pwsh, and have the same symptom when in that shell.

That explains it then! I wonder how come python3 is in the path for zsh but not for bash?

That would be because I've got export PYTHONPATH set in my OhMyZSH profile. But since I don't launch bash, I never modified that profile.
So the right long-term fix is for VS Code for MacOS to embrace ZSH and Python 3, right?
It would be helpful if the realpath function knew to prefer Python3, and maybe only provide Python2 backward compatibility support via a parameter / argument? https://pythonclock.org/

Well, PYTHONPATH shouldn't affect PATH... that doesn't full explain why Python3 runs in zsh. There must be another change to PATH in your zshrc which gets Python3 in the path.
PYTHONPATH does affect the runtime behavior of Python 2 though... so have you considered removing it from your zshrc?

Sure, I can fix my environment.
Is it useful for me to submit a Python 2 deprecation issue, or is that already in the roadmap?

The thing is that script works in both 2 and 3, that's the point, so no need to deprecate 2 since it works for a lot of people. Python just doesn't like it if your PYTHONPATH is incompatible with whatever python is in the PATH. Thanks!


Experimental: expose the cursor in find() queries

I have been thinking for quite a while about being able to run queries without loading the entire result set into memory. Here's what I came up with (view the Changes):
collection.find_with_cursor returns a deferred which results in a tuple: (docs, deferred). If there are no more documents remaining in the current query, a deferred will be returned which results in this tuple: ([], None)
For example:
docs, d = yield collection.find_with_cursor()
# collection.find(cursor=True) also works

while docs:
    # Do something with this page of docs here
    for i in docs:
        pass
    # Get the next set of docs from the query.
    docs, d = yield d

I have already implemented this (here), but let me know what you think, or if there are any improvements that could be made on this, then I will send over a pull request

@psi29a Pull request submitted!


Added custom label settings for VSCode

This PR adds VSCode configuration information to the README.md file.
Having multiple -guard files open at once can become quite confusing. As such, making use of the custom labels feature in VSCode could be a good recommendation to users of this package.
Of course, these settings are optional.

thank for your contribution,


 Would you like to have a check?
Say q21 was slowed down from 289s to 229s in the above benchmark result table.

sure, let me check. Thanks.

cc: @FelixYBW

@zhli1142015
In TPC-H q21 join keys are all unique but semi/anti joins are included, so I guess we may skip your optimization in this case?

Here's the system metrics from our daily perf test. We observed that the used memory has been increased along with the performance regression.
| application_1712857130659_0001 | application_1712943012393_0001
-- | -- | --
runtime | 1,190.90 | 1,259.21
avg %user | 64.54 | 66.94
avg %system | 19.92 | 18.52
avg %iowait | 2.03 | 1.55
avg disk util | 51.39 | 47.26
total read (G) | 1,010.10 | 857.18
total write (G) | 1,896.31 | 1,909.39
avg read bw (MB/s) | 848.52 | 682.02
avg write bw (MB/s) | 1,592.96 | 1,519.20
cached mean | 5,880.09 | 5,606.80
cached 75% | 7,821.00 | 7,396.00
cached max | 8,886.00 | 9,060.00
used mean | 1,274.06 | 1,606.05
used 75% | 1,354.00 | 1,475.00
used max | 5,177.00 | 7,208.00

Hi @zhli1142015 , are you still able to have a look on this?

Hello @zhztheplayer ,
I think this issue is mentioned is because of #6789.
Here are some obsersavation from our side for the https://github.com/facebookincubator/velox/pull/9079.
It only has impact when there are duplicate rows in build side, this can be identified if hash probe output vectors > hash probe input vectors. And it would cause longer time of hash build but shorter time of hash probe. Please let me know if this is the case you observed.
BTW, most time of hash probe of the join in query 95 (1TB in our test) is spent in hash join filter. Internally we have counter for it. I think the PR may help for the case: https://github.com/facebookincubator/velox/pull/10464

Thanks for sharing this, then this maybe a different case. For such case we usually collect the profile graph to see which function is the bottleneck.

Let's close this one for now.


[Bug]: Graph view does not work in beta

What happened?
When selected from the file tree, everything works fine. But when any node is clicked in the graph view, it results in page not found.

To be clear, my vault has a computers folder, which has all of my notes.
I do not know if this might be relevant but -
Using 1.8.01, the web server export results in two folders - lib, and computers with the HTML.
Using 1.8.1-2b, the web server export results in a lib folder, and all the HTML on the same level (there is no computers folder).
Reproduction

Install plugin from BRAT
Select files to export and save
Export using the Online web server option

Last Working Version
1.8.01
Version
1.8.1-2b
Operating System
Ubuntu 22.04
Obsidian Version
1.6.7
Which browsers can you reproduce this in?
Chrome
Relevant log output
Log:
[INFO] 	Including stylesheet: css-settings-manager
[INFO] 	Including stylesheet: ADMONITIONS_CUSTOM_STYLE_SHEET
[INFO] 	Including stylesheet: css-settings-manager
[INFO] 	Including stylesheet: ADMONITIONS_CUSTOM_STYLE_SHEET
[WARNING] 	Creating an RSS feed requires a site url to be set in the export settings.
[INFO] 	Closing render window
[INFO] 	Including stylesheet: css-settings-manager
[INFO] 	Including stylesheet: ADMONITIONS_CUSTOM_STYLE_SHEET
[WARNING] 	Creating an RSS feed requires a site url to be set in the export settings.
[INFO] 	Closing render window

Settings:
	settingsVersion --------- 1.8.1-2b
	makeOfflineCompatible     false
	inlineAssets ------------ false
	includePluginCSS          []
	includeSvelteCSS -------- true
	titleProperty             title
	customHeadContentPath ---
	faviconPath
	documentWidth ----------- 40em
	sidebarWidth              20em
	minOutlineCollapse ------ 2
	startOutlineCollapsed     false
	relativeOutlineLinks ---- false
	allowFoldingHeadings      true
	allowFoldingLists ------- true
	allowResizingSidebars     true
	logLevel ---------------- warning
	minifyHTML                true
	makeNamesWebStyle ------- true
	onlyExportModified        true
	deleteOldFiles ---------- true
	addThemeToggle            true
	addOutline -------------- true
	addFileNav                true
	addSearchBar ------------ true
	addGraphView              true
	addTitle ---------------- true
	addRSSFeed                true
	siteURL -----------------
	authorName
	vaultTitle -------------- obsidian-vault
	exportPreset              website
	openAfterExport --------- false
	graphAttractionForce      1
	graphLinkLength --------- 10
	graphRepulsionForce       150
	graphCentralForce ------- 3
	graphEdgePruning          100
	graphMinNodeSize -------- 3
	graphMaxNodeSize          7
	showDefaultTreeIcons ---- false
	emojiStyle                Native
	defaultFileIcon --------- lucide//file
	defaultFolderIcon         lucide//folder
	defaultMediaIcon -------- lucide//file-image
	exportPath                /home/wahid/Documents/blog
	filesToExport ----------- 18
	filePickerBlacklist       [
	filePickerWhitelist ----- [

Enabled Plugins:
	Admonition
	Advanced Tables
	Omnisearch
	Style Settings
	Editing Toolbar
	Execute Code
	BRAT
	Webpage HTML Export

Additional Info
When files are accessed through the file tree:

When files are accessed via graph:

Ahh, I must not have updated the graph view to grab the correct path.  It is very likely this is fixed already in my development branch which hasn't been released, but thanks for the report and I'll make sure it works properly in the next release :)

Could you please include the option to allow nested folders if that is possible? That would be pretty useful as my vault has a lot of folders and I'd like to keep it that way. Thanks!

Could you please include the option to allow nested folders if that is possible? That would be pretty useful as my vault has a lot of folders and I'd like to keep it that way. Thanks!

I think you may misunderstand what I mean. It doesn't remove all the folders. It only removes folders which are not selected for export and are redundant. It still keeps folders as expected. You see that the folders are still there when you look at the file picker :)

Hey, I just released a beta version that should address this. If you could try it out and let me know if his issue is fixed and if you find any other issues. There are instructions for installing the beta in the readme :)
Thanks!