Extension host crashes out of memory

VSCode Version: Code 1.16.0 (787b31c0474e6165390b5a5989c9619e3e16f953, 2017-09-06T16:19:37.192Z)
OS Version: Linux x64 4.4.0-53-generic
Extensions:

Extension
Author (truncated)
Version

vscode-jest
Ort
2.3.0

scss-scan
aar
0.1.67

scss-lint
ada
0.0.18

npm-intellisense
chr
1.3.0

vscode-eslint
dba
1.3.1

gitlens
eam
4.4.3

code-runner
for
0.7.1

vscode-javac
geo
0.0.46

docthis
joe
0.4.8

vscode-styled-components
jpo
0.0.8

node-module-intellisense
lei
1.4.0

code-beautifier
mic
1.7.2

prettify-json
moh
0.0.3

vscode-scss
mrm
0.6.2

sublime-keybindings
ms-
2.9.1

sort-imports
pet
2.4.0

java
red
0.9.0

vscode-icons
rob
7.12.0

autoimport
ste
1.2.2

gitblame
wad
2.2.0

(14 theme extensions excluded)

Steps to Reproduce:

Open VS Code
Wait for it to crash

Reproduces without extensions: No
Console output:
[uncaught exception]: Error: read ECONNRESET
onError @ index.js:5
index.js:5 Error: read ECONNRESET
    at exports._errnoException (util.js:1050:11)
    at Pipe.onread (net.js:581:26)
onError @ index.js:5
events.js:163 Uncaught Error: read ECONNRESET
    at exports._errnoException (util.js:1050:11)
    at Pipe.onread (net.js:581:26)
/usr/share/code/resources/app/out/vs/workbench/workbench.main.js:28 Extension host terminated unexpectedly. Code:  null  Signal:  SIGABRT
e._onExtensionHostCrashed @ /usr/share/code/resources/app/out/vs/workbench/workbench.main.js:28
/usr/share/code/resources/app/out/vs/workbench/workbench.main.js:28 Extension host terminated unexpectedly.
e.doShow @ /usr/share/code/resources/app/out/vs/workbench/workbench.main.js:28
/usr/share/code/resources/app/out/vs/workbench/workbench.main.js:28 Extension Host
/usr/share/code/resources/app/out/vs/workbench/workbench.main.js:28 FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node::Abort() [/usr/share/code/libnode.so]
 2: 0x7f726effbec7 [/usr/share/code/libnode.so]
 3: 0x7f726ed34bb7 [/usr/share/code/libnode.so]
 4: 0x7f726ed34b35 [/usr/share/code/libnode.so]
 5: v8::internal::Factory::NewTransitionArray(int) [/usr/share/code/libnode.so]
 6: 0x7f726e7fae1d [/usr/share/code/libnode.so]
 7: 0x7f726e918ea0 [/usr/share/code/libnode.so]
 8: 0x7f726e915e68 [/usr/share/code/libnode.so]
 9: 0x7f726e915cbb [/usr/share/code/libnode.so]
10: 0x7f726e92dad9 [/usr/share/code/libnode.so]
11: v8::internal::LookupIterator::PrepareTransitionToDataProperty(v8::internal::Handle<v8::internal::JSObject>, v8::internal::Handle<v8::internal::Object>, v8::internal::PropertyAttributes, v8::internal::Object::StoreFromKeyed) [/usr/share/code/libnode.so]
12: 0x7f726e91b5de [/usr/share/code/libnode.so]
13: 0x7f726e91f62a [/usr/share/code/libnode.so]
14: 0x7f726eccfbe9 [/usr/share/code/libnode.so]
15: 0x7f726eccf9c3 [/usr/share/code/libnode.so]
16: 0x7f726ecd1ba7 [/usr/share/code/libnode.so]
17: 0x7f726eccec26 [/usr/share/code/libnode.so]
18: 0x7f726eb5424e [/usr/share/code/libnode.so]
19: 0x2516d4a0415d

<--- Last few GCs --->

[29336:0x2337be422000]   160430 ms: Mark-sweep 2057.1 (2207.6) -> 2057.1 (2187.1) MB, 3510.6 / 0.0 ms  (+ 0.0 ms in 0 steps since start of marking, biggest step 0.0 ms, walltime since start of marking 3511 ms) last resort
[29336:0x2337be422000]   164006 ms: Mark-sweep 2057.1 (2187.1) -> 2057.1 (2187.1) MB, 3575.3 / 0.0 ms  last resort

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x153f97ab959 <JS Object>
    0: builtin exit frame: stringify(this=0x153f9790771 <a JSON with map 0x3f1fa4d06199>,0x82336802311 <undefined>,0x2d4924437931 <JS Function r (SharedFunctionInfo 0x4133d5c6739)>,0x1a2fc107f0b1 <JS Array[3]>)

    1: arguments adaptor frame: 2->3
    2: callOnRemote [/usr/share/code/resources/app/out/vs/workbench/node/extensionHostProcess.js:~28] [pc=0x2516d54d0433](this=0x11275e472039 <a e wi...

I was uninstalling some plugins to find out which one was causing this and saw that after uninstalling Language Support for Java(TM) by Red Hat it stopped crashing.

@aeschli Would you perhaps know some specifics about the Java extensions, is it possible it has a memory leak in the extension code ?

@fbricon Fred FYI.
@melloc01 Do you have an example project where you see this?

Thanks guys,
@aeschli would you suggest an open source project to test whether it crashes or not?
This one is private.

Yesterday I uninstalled it and no crashes happened until I installed it back - pretty sure it is the extension.

@melloc01 did it crash after opening a Java project or java files?

Java project @fbricon

@melloc01 a sample project reproducing the error would be mighty helpful.
@aeschli the vscode-java extension is fairly simple. I'm not a javascript or typescript expert but I don't see anything obvious. Any advice on how to track memory leaks in vscode extensions?

@fbricon yeah, I know, I'm a Java rookie tho, I use this extension to help me patching any small issues.
I just tested in this project: https://github.com/gabrielf/maven-samples and it works fine.
Seems to be related to the project.
I managed to attach a strace on the process, not sure if it'll help but this was what I got:
strace -s 800 -p 27457
strace: Process 27457 attached
futex(0x7fa810df79d0, FUTEX_WAIT, 27458, NULL <unfinished ...>
+++ exited with 13 +++

Which is an EACCES.
hmm.. maybe my permissions are messed up on some place the plugin uses for caching?

With the latest insider build it's now very easy to profile the extension host. Just open the dev tools, and check the console for the chrome debug URL that lets you open the chrome dev tools on the extension host node process.
I tried that with a simple project, but didn't see hangs or any big memory allocations.

Closing. @melloc01 Please file an issue against the Java extension with an example project.


Individual UK flags

Hi, any chance of adding Wales, England, Northern Ireland and Scotland?

I'm in general against anything that is not on the ISO 3166-1-alpha-2 code list.. but maybe due to the popular demand I'll have to start adding more flags.. we'll see :)

really need this

+1
These flags are widely used. For example, you can see Scotland. Wales and Northern Ireland flags on Miss World page (http://www.missworld.com/contestants)

So if we are about to do it.. I will need some help in terms of codes and what is what :) For someone that is not from GB it might be confusing.. so lets move this conversation to #120


Revert workaround in compas.robots.OrientationConstraint.transform()

What type of change is this?
Outdated workaround now that pinned version of compas has been bumped.
Checklist

I added a line to the CHANGELOG.rst file in the Unreleased section under the most fitting heading (e.g. Added, Changed, Removed).
[x] I ran all tests on my computer and it's all green (i.e. invoke test).
[x] I ran lint on my computer and there are no errors (i.e. invoke lint) .
I added new functions/classes and made them available on a second-level import, e.g. compas_fab.robots.Configuration.
I have added tests that prove my fix is effective or that my feature works.
I have added necessary documentation (if appropriate).

@tetov thx!


[GitHub Enhancement] Issue Posting Guidelines

Make guidelines & to-dos before posting a issue. Such as updating to the latest build, providing diagnostics log, etc

I can help add Issue Templates

Also done #627 so closing this.


Can easily support .NET Framework 3.5

By Reference Theraot.Core

Thanks for the suggestion, but why would I want to support a 20 year-old framework? Does anyone still maintain an app targeting .NET 3.5?


Don't allow switching to edit mode when a transform is in effect

Related bugs:

if you switch from a transform to display mode and back out of display mode (ctrl-shift-d twice), the transform is still applied, but the toolbar is the standard toolbar instead of the transform toolbar
Saving when a transform is applied (ctrl-shift-s) should not be allowed, because it blows away the dashboard definition! Oops.


Beta v1.6 Won't load on Android Companion App

Upgraded to beta 1.6 today. Got it running on iOS by reloading the cache. Couldn't get it running on Android companion app at all. It would just load a blank screen.
Tried deleting cache, then data then reinstalling the whole app. Wouldn't load until I reverted to v1.5.3.

Hi and thanks for your feedback.
This issue should be fixed in the new 1.6.0-beta.2 🤞
https://github.com/Clooos/Bubble-Card/releases/tag/v1.6.0-beta.2

Issue is still occurring on beta 2. It actually showed up in 1.5.3 when I downgraded too. Not sure what's going on?

Just upgraded to beta 3. It appears to be an issue with the dashboard loading incredibly slowly on Android.
I'm using optimised mode. Is it meant to be listed as v1.6.0-beta.3 in the config?

Can you try in regular mode? I still don't understand your issue because It's really fast on Android on my side.
Have you checked in your logs if there are errors?
And I'm not sure to understand your last question.

Regarding the last question, is that the format I should be putting the version down as for beta versions?
The logs aren't throwing anything up. The dashboard just seems to be loading really slow or not at all on my partner's galaxy phone.
I'll try reloading it as regular version.

I wonder if it was the header I used from Mobile First. Since the card mod update I removed the graphic and image and things seem to be working quicker. I'll do some more testing...

I just had another look on my partner's phone. It still doesn't seem to be loading. Any thoughts?

Can you try the latest beta?
https://github.com/Clooos/Bubble-Card/releases/tag/v1.6.0-beta.5


ignore newline EOF for svg

We have svg image files in the website repo, and need to ignore this files
cc @gabo1208

/lgtm

/assign @julz

/approve


backporting: Highlight ambiguous backport suggestions

Recently, the check-stable script has suggested every single possible
match for commits where the name does not uniquely identify the commit.
This can be a bit confusing to backporters since it looks like there are
many commits to backport as part of this PR, but the second and later
ones are not necessary to backport.
 * PR: 16589 -- vagrant: Bump all Vagrant box versions (@pchaigno) -- https://github.com/cilium/cilium/pull/16589
   Merge with 1 commit(s) merged at: Tue, 22 Jun 2021 12:36:17 -0700!
     Branch:     master (!)                          refs/pull/16589/head
                 ----------                          -------------------
     v (start)
     |  edf76fb1ef6b58d5ef90b439d54134f314ed086e
5bef5d77137a9ecc5d3f2b72149307ffdd52cd42
4dc60e6faf654d7424ee959867a774205b3fed13
816b3231cdbc39f4bcdd3e6f5b40a056459a478c
51826b31087496d108044f3bffbf304580fffb4a
df8238d451d755d5be75e202be89b4f88067c77b
a4e7bc6c1f0e96078793458b6719b9a3999b89db via fb723f8133c40faa068a5a401f594622668b2753 ("vagrant: Bump all Vagrant box versions")
     v (end)

It's likely that the most recent commit is the correct one to backport,
so this commit suggests that one by default.
Plausibly at some point we might want to choose a different commit
rather than the most recent one. In order not to lose this content,
track the set of commits with the same name in the past one year and
print those in a short message after the existing text.
Here's the new format:
 * PR: 16589 -- vagrant: Bump all Vagrant box versions (@pchaigno) -- https://github.com/cilium/cilium/pull/16589
   Merge with 1 commit(s) merged at: Tue, 22 Jun 2021 12:36:17 -0700!
     Branch:     master (!)                          refs/pull/16589/head
                 ----------                          -------------------
     v (start)
     |  edf76fb1ef6b58d5ef90b439d54134f314ed086e via fb723f8133c40faa068a5a401f594622668b2753 ("vagrant: Bump all Vagrant box versions")
     v (end)
   Found the following ambiguous commits:
     edf76fb1ef6b<EMAIL_ADDRESS>2021-06-18      vagrant: Bump all Vagrant box versions
     5bef5d77137a<EMAIL_ADDRESS>2021-04-21      vagrant: Bump all Vagrant box versions
     4dc60e6faf65<EMAIL_ADDRESS>2021-04-19      vagrant: Bump all Vagrant box versions
     816b3231cdbc<EMAIL_ADDRESS>2020-11-25      vagrant: Bump all Vagrant box versions
     51826b310874<EMAIL_ADDRESS>2020-11-13      vagrant: Bump all Vagrant box versions
     df8238d451d7<EMAIL_ADDRESS>2020-09-30      vagrant: Bump all Vagrant box versions
     a4e7bc6c1f0e<EMAIL_ADDRESS>2020-07-23      vagrant: Bump all Vagrant box versions

In this case it seems fairly obvious that the commits with conflicting names
are pretty innocent and the selected one with the date 2021-06-18 should be
the right one to select, but at least we could see that there was an
ambiguity and potentially choose a different commit if necessary.
Fixes: #16572

:+1: good feedback @pchaigno , and thanks for being patient with me in Slack :sweat_smile:
I plan to respin this some time this week.


interp1d: add option for fill_value to 'extrapolate'

I'm not sure if this is too harsh a change but it would remove the bounds checking for nearest neighbor interpolation. This leads to consistent behavior of 1D and ND nearest neighbor interpolation.
In light of the discussion in https://github.com/scipy/scipy/issues/4304 about deprecation of interp1d another solution might be to use another 1D interpolator or cKDTree directly (although the performance of that is probably not as good as a special 1D case)
fix #4233

Something like fill_value="extrapolate" probably would be better and not cause backward compatibility issues.

I've changed the PR according to your suggestion.
Do you consider it surprising that interp1d discards out of bound values when the method is set to nearest? If so we might add a note to the documentation about this.

I've implemented the necessary checks and tests.

Fixed the linting issue.

Ideally, fill_value=΅extrapolate" would work for all interpolation kinds (I'm guessing that's what Pauli meant). This is not very easy for spline-based kinds ATM, because splmake/spleval does not do extrapolation out of the box:
In [29]: x
Out[29]: array([0, 1, 2, 3, 4])

In [30]: for order in [1, 2, 3]:
    tck = splmake(x, x)
    print spleval(tck, [-1, 6])
   ....:
[ 0.  0.]
[ 0.  0.]
[ 0.  0.]

If this option is added to kind='nearest', it needs a test. At least as simple as
In [33]: interp1d(x, x, kind='nearest', fill_value='extrapolate', )([-1, 6])
Out[33]: array([ 0.,  4.])

It might also make sense to add extrapolation to kind="linear".
All in all, I'm +0 on merging this once there is a test for interp1d

Thank you @cpaulik --- your commits were merged as a part of gh-5021. Just to make it clear: your authorship is preserved, congratulations with what I believe are your first scipy commits. Keep them coming :-).


cytnx.linalg.Lanczos_Gnd_Ut need to be checked

We may want to check Lanczos_Gnd_Ut algorithm for the following reason. In computing the ground state of transverse-field Ising model by DMRG, using Lanczos_Gnd_Ut obtains the correct energy, but it's "harder" to get a symmetry-breaking state for a small transverse field. Using Scipy Lanczos or ITensor Davison algorithm tends to obtain symmetry-breaking states more easily.

It is found that Lanczos_ER can also reproduce symmetry-breaking state well (the reason why Lanczos_Gnd_Ut fails to do so might be its bad ability to converge)

I thought Lanczos_ER and Lanczos_Gnd_Ut are doing the same thing but just having different input types. What are their differences?

I thought Lanczos_ER and Lanczos_Gnd_Ut are doing the same thing but just having different input types. What are their differences?

Lanczos_ER uses explicitly restarted while Lanczos_Gnd is the naive Lanczos implementation. Lanczos_ER and  Lanczos_Gnd takes the cytnx.Tensor type as input while Lanczos_Gnd_Ut takes cytnx.UniTensor.

Lanczos_Gnd_Ut can also produce symmetry-breaking state, just need slightly more dmrg sweep than Lanczos_ER to converge.


No Drag Handles

I am not seeing any drag handles for text, so the only thing you can do is "Select All". Long pressing a word does not select anything, once I select all, I can not modify the selected region after that.
Is this intentional? Some screenshots from the past show TextHandles, so I'm confused:
https://github.com/singerdmx/flutter-quill/issues/5
Is there a way to enable this? Or has it been removed in recent versions?
Testing on iOS w/ the /example demo and latest stable.
https://user-images.githubusercontent.com/736973/231244261-f010d3bd-07c6-4e15-9a1e-4f116e9d2239.mp4

This is working in the current version.
If you update to the current version and still have a problem, please reopen this issue and give us more information so we can reproduce the problem.


RHIDP-3022: Document how to track changes to catalog database

Version(s):
1.2.2
Issue:
https://issues.redhat.com/browse/RHIDP-3022
Link to docs preview:
https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-371/getting-started-rhdh/#con-audit-log-track-changes-catalog_assembly-audit-log
Reviews:

[x] SME: @kim-tsao @schultzp2020
[ ] QE: N/A - conceptual info only
[x] Docs review: @hmanwani-rh

Additional information:
Blurb about tracking catalog database changes has also been added to the RNs draft doc

lgtm

@linfraze Added a few comments below. Please take a look and let me know if you've any question.
Quick question: If the tracking changes section lacks detailed information, do you think we should include this information in the introduction paragraph instead? I’m not sure if having a separate section for such minimal content is necessary. What do you think?

Sure, I will go ahead and incorporate this content into assembly, but leave the con module in the repo in case it is needed in the future. This will also resolve the gerund heading for now.

/cherrypick 1.2.x


Porting Redmine ticket 26713 to v3.7.7

Include on-demand CVMFS provisioning info in GWMS documentation

Added documentation regarding on-demand CVMFS (the internal specifics) and how the dynamic creation/selection of the cvmfsexec distribution takes place.

Abandoning and closing this PR since a single PR has been created (#122) that includes this ticket to avoid conflicts.


Release Zebra 1.0.0-beta.12

Related Tickets
Close #4651
Depends-On: #4680
(waiting on review by @dconnolly)
Versioning
Which Crates to Increment
To check if any of the top-level crates need version increments:

go to the zebra GitHub code page: https://github.com/ZcashFoundation/zebra
check if the last commit was a Zebra version bump, or something else

Alternatively you can:

use the github compare tool and check the main branch against the last tag (Example)
use git diff --stat <previous-release-tag> origin/main

Once you know which crates have changed:

[x] Increment the crates that have new commits since the last version update
[x] Increment any crates that depend on crates that have changed
[x] Keep a list of the crates that haven't been incremented, to include in the PR

How to Increment Versions
Zebra follows semantic versioning.
Semantic versions look like: MAJOR.MINOR.PATCH[-TAG.PRE-RELEASE]

Pre-Release Crates
Pre-Release versions have a TAG like "alpha" or "beta". For example: 1.0.0-alpha.0

Increment the PRE-RELEASE version for the crate.

Unstable Crates
Unstable versions have a MAJOR version of zero. For example: 0.1.0

Follow stable crate versioning, but increment the MINOR version for breaking changes

Stable Crates
For example: 1.0.0
Increment the first version component in this list, and reset the other components to zero:

MAJOR versions for breaking public API changes and removals

check for types from dependencies that appear in the public API

MINOR versions for new features
PATCH versions for bug fixes

includes dependency updates that don't impact the public API

Reviewing Version Bumps
Check for missed changes by going to:
https://github.com/ZcashFoundation/zebra/tree/<commit-hash>/
Where <commit-hash> is the hash of the last commit in the version bump PR.
If any Zebra or Tower crates have commit messages that are not a version bump, we have missed an update.
Also check for crates that depend on crates that have changed. They should get a version bump as well.

Version Locations
Once you know which versions you want to increment, you can find them in the:

[x] zebra* Cargo.tomls
[x] tower-* Cargo.tomls
[x] zebra-network protocol user agent: https://github.com/ZcashFoundation/zebra/blob/main/zebra-network/src/constants.rs
[x] README.md
[x] book/src/user/install.md
[x] Cargo.lock: automatically generated by cargo build

Version Tooling
You can use fastmod to interactively find and replace versions.
For example, you can do something like:
fastmod --extensions rs,toml,md --fixed-strings '1.0.0-beta.11' '1.0.0-beta.12'
fastmod --extensions rs,toml,md --fixed-strings '0.2.26' '0.2.27' tower-batch tower-fallback

Ignore any fastmod updates to previous release notes in CHANGELOG.md.
README
We should update the README to:

[x] remove any "Known Issues" that have been fixed
[x] update the "Build and Run Instructions" with any new dependencies.
Check for changes in the Dockerfile since the last tag: git diff <previous-release-tag> docker/Dockerfile.

Change Log
Important: Any merge into main deletes any edits to the draft changelog.
Once you are ready to tag a release, copy the draft changelog into CHANGELOG.md.
We follow the Keep a Changelog format.
We use the Release Drafter workflow to automatically create a draft changelog.
To create the final change log:

[x] Copy the draft changelog into CHANGELOG.md
[x] Delete any trivial changes. Keep the list of those, to include in the PR
[x] Combine duplicate changes
[x] Edit change descriptions so they are consistent, and make sense to non-developers
[x] Check the category for each change

prefer the "Fix" category if you're not sure

Change Categories
From "Keep a Changelog":

Added for new features.
Changed for changes in existing functionality.
Deprecated for soon-to-be removed features.
Removed for now removed features.
Fixed for any bug fixes.
Security in case of vulnerabilities.

Create the Release
Create the Release PR
After you have the version increments and the updated changelog:

[x] Push the version increments and the updated changelog into a branch
(name suggestion, example: v1.0.0-alpha.0-release)
[x] Create a release PR by adding &template=release-checklist.md to the
comparing url (Example).

[x] Add the list of deleted changelog entries as a comment to make reviewing easier.
[ ] Also add the list of not-bumped crates as a comment (can use the same comment as the previous one).

[ ] Turn on Merge Freeze.
[ ] Once the PR is ready to be merged, unfreeze it
here. Do not
unfreeze the whole repository.

Create the Release

[ ] Once the PR has been merged, create a new release using the draft release
as a base, by clicking the Edit icon in the draft
release.
[ ] Set the tag name to the version tag, for example: v1.0.0-alpha.0
[ ] Set the release to target the main branch
[ ] Set the release title to Zebra  followed by the version tag, for example:
Zebra 1.0.0-alpha.0
[ ] Copy the final changelog of this release to the release description
(starting just after the title ## [Zebra ...)
[ ] Mark the release as 'pre-release' (until we are no longer alpha/beta)
[ ] Publish the release

Final Testing

[ ] After tagging the release, test that the exact cargo install command in
README.md works (--git behaves a bit differently to --path)
[ ] Turn off Merge
Freeze for the
whole repository.

If the build fails after tagging:

fix the build
increment versions again, following these instructions from the start
update README.md with a new git tag
update CHANGELOG.md with details about the fix
tag a new release

I bumped all the crate versions.
I made tower-fallback the same version as tower-batch to simplify version updates.
I deleted almost all the dependency version bumps (there were a lot).
I also deleted some trivial documentation changes, and merged some docs into the related PR's changelog entry.


Fix the name of the "JobTime(out)Error" class

PR #771 introduced the "JobTimeError" class along with several references to a "JobTimeoutError" class (note the additional "out" text).
Presumably, the new references were meant to refer to the newly defined class, but the names differed.
Since there were more callers than definitions, and the callers are all to JobTimeoutError, I believe that was the intended name, and JobTimeError was a typo.
This change fixes that by switching the class name to the longer JobTimeoutError.

This change is

/assign jlewi
Adding Jeremy, since he wrote the original PR and would know best which name he intended to use.

Coverage increased (+0.07%) to 58.026% when pulling 10c5f848fbc4d17875e8bae5be8ae03ad332487e on ojarjur:patch-1 into 78cfba16da8dbd07360135253aad1091f0eec305 on kubeflow:master.


Deeplink change default root in ion-nav

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

Deeplink change default [root] in ion-nav
Possible to make path like: [host]/#/home/home/home/home/other if home Is a IonicPage and have ion-nav tag

Expected behavior:

ion-nav [root] don't  changed with deep links.

Steps to reproduce:
Create file like at Related code
write path like [host]/#/home/home/home/other
Related code:
APP
app.component.ts
...
this.rootPage = 'HomePage';
...

app.html
<ion-nav [root]="rootPage"></ion-nav>
HOME
home.ts
...
@IonicPage()
...
rootPage: any = 'OtherPage';
...

home.module.ts
...
@NgModule({
  declarations: [HomePage],
  imports: [IonicPageModule.forChild(HomePage)],
  exports: [HomePage]
})
export class HomePageModule {}

home.html
HOME
<ion-nav [root]="rootPage">
OTHER
other.ts
...
@IonicPage()
...

other.module.ts
...
@NgModule({
  declarations: [OtherPage],
  imports: [IonicPageModule.forChild(OtherPage)],
  exports: [OtherPage]
})
export class OtherPageModule {}

other.html
Other
Other information:
Sometimes there is an error:
Runtime Error
Cannot read property 'indexOf' of null
Stack
TypeError: Cannot read property 'indexOf' of null
at NavControllerBase._cleanup (http://localhost:8100/build/main.js:44474:59)
at NavControllerBase._transitionFinish (http://localhost:8100/build/main.js:44376:18)
at t.invoke (http://localhost:8100/build/polyfills.js:3:14529)
at Object.onInvoke (http://localhost:8100/build/main.js:4498:37)
at t.invoke (http://localhost:8100/build/polyfills.js:3:14469)
at n.run (http://localhost:8100/build/polyfills.js:3:9741)
at NgZone.run (http://localhost:8100/build/main.js:4367:62)
at Array. (http://localhost:8100/build/main.js:44324:25)
at MDTransition.Animation._didFinish (http://localhost:8100/build/main.js:23589:29)
at MDTransition.Animation._playProgress (http://localhost:8100/build/main.js:22928:18)
at MDTransition.Animation._playDomInspect (http://localhost:8100/build/main.js:22893:14)
Ionic Framework: 3.0.1
Ionic App Scripts: 1.3.0
Angular Core: 4.0.0
Angular Compiler CLI: 4.0.0
Node: 7.6.0
OS Platform: Linux 4.4
Navigator Platform: Linux x86_64
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.100 Safari/537.36

Hello, thanks for using Ionic! We will look into this.

I have come across a even worse variant.
I have a split pane with 2 side menu's and 1 content (it's meant for (really) large tablets and desktops) and as soon as the deeplink is accessed all three get their root set to the nav target (a tabbed page which should display in the content pane not any of the side panes). This results in some unexpected and unwanted behavior, since the tab pane tries to set the menus to correspond to the current view (this actually happens 3 times, once in each pane). This in turn destroys the tab (nav) controller in the 2 side panes which results in a navcontroller destroyed error error because the actual menus get loaded and these in tern try to send some information to the tab nav controller (using observables).
I would expect that (at least in split view) the nav component would either not listen to the current route (and make the developer responsible for triggering the correct side menu) or that each nav component is targeted separately in the deeplink uri (using a separator symbol which targets each nav item individually)...

I met the same problem. Deep link definitely should not change the rootPage. Instead, the deep linked page should be a navigated page from the root page. @jgw96


Brain

This should basically be a multi layer perceptron, possibly with memory cells or signal feedback to allow periodic tail gaits.

Requires #11

The current model can give rise to swimming behavior & sensory input.


Motion detection Stop not working

The problem
Your Environment
Core
2024.1.6
Supervisor
2023.12.1
Operating System
11.4
Frontend
20240104.0

tuya-device-sharing-sdk Version used:
Device diagnostic file:

Expected Behavior
Once motion detector has Stopped detecting motion it should trigger the event
Current Behavior
When motion detector stops detecting motion it should trigger the event to allow automation to run.

in addition, if I reload tuya smart in HA, it resets back to 'clear' correctly, and when it spots motion, it correctly triggers the event, however, it never triggers the 'no motion detected'


XR Streaming SDK for Unity

Streaming SDK for XR Devices.
Supporting HoloLens 2, Magic Leap 2, Quest 2, 3, Pro, iPad, iPhone
https://github.com/Holo-Light-GmbH/Hololight-Stream-SDK-Trial

Great solution. Thanks for letting me know that. I added it under "Solutions/Cloud XR, Remote Rendering & Streaming" with the official product page link.


IDA-like default variable names in decompiler

Current ghidra default variable names are verbose, especially those contain stringified address in it. Most of the time, those addresses are not useful. We should strip them out to provide a cleaner decompiler output.
Example (ghidra):

Same function in IDA:

Ignore the array analysis and the alloca thing. v2 is definitely visually better than those uStack77832 thing. Nobody cares about 77832 kinda thing.
This functionality could be also provided to upstream. But in our case, we could set it to enabled by default but official upstream should have it disabled.
My previous commit can be an illustration of how this can be done properly. But that's not a complete commit, as it does not cover all of the variable name generation algorithm.
One can implement this by following my commit and complete the whole variable generation.
Also, better simpler variable name genration algorithm is also welcome.

Close due to project restructure.


Error running AdaptVQE

I'm having an error when running the attached script. It works nicely without AdaptVQE but when I insert it in the stack and launch the job on QLMAAS the result object contains the following error:
qat.comm.exceptions.ttypes.PluginException: PluginException(code=None, modulename='qat.plugins.junction', message='Caught exception from junction: "<class 'super'> returned a result with an error set" [see /tmp/tmpnqozg3oy.junction.log for traceback info]', file=None, line=None)
I'm sending the job to qlm35e and my version of qlmaas is 1.7.2 running on python 3.9.16
adapt_error.txt

Dear @Buffoni,
The x0 argument of ScipyMinimizePlugin is expected to be a list of numbers (not qat-lang variables). Our plugin rely on the minimize function from scipy and this parameter is used to define the initial value for each angle composing your circuit.
This x0 parameter is not mandatory, then, you can simply define your plugin like that:
# Define plugin without x0 parameter
optimizer_plugin = ScipyMinimizePlugin(method="COBYLA", tol=1e-3, options={"maxiter": 200})

Then, your circuit is useable on both myQLM and QLMaaS.


While debugging device simulation dotnet on vsCode, error occu

While debugging device simulation dotnet on vsCode, error occurs.
The error is on config data cs  . saying Environment variable 'PCS_STORAGEADAPTER_DOCUMENTDB_CONNSTRING' not found.
when creating simulated devices using postman also not getting any response. -

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

ID: 91e85ea3-e6f9-8f21-c542-410b2cbe0b27
Version Independent ID: 23969f86-7f28-4205-bc3a-e0dd518eedea
Content: Device simulation with IoT remote monitoring - Azure
Content Source: articles/iot-accelerators/iot-accelerators-remote-monitoring-create-simulated-device.md
Service: iot-accelerators
GitHub Login: @dominicbetts
Microsoft Alias: dobett

@developersenakim thank you for your feedback. Did you follow the step here?

Please also make sure that you completed all the Prerequesites:

Thank you.

that has been assigned. I am getting this.

@developersenakim what is the result when you run Visual Studio Code in Administrative mode?
Thank you!

@developersenakim did you have the time to test it running Visual Studio Code in Administrative mode? Let us know if it worked?
Thank you!

Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


GH-38757: [C#] Implement common interfaces for structure arrays and record batches

What changes are included in this PR?
New interface IStructType is implemented by both Schema and StructType.
New interface IArrowStructArray is implemented by both RecordBatch and StructArray.
These changes make it easier to write code which handles both types of structures, which are virtually identical to each other.
Are these changes tested?
Yes.
Are there any user-facing changes?
New interfaces have been added to access existing structures.

Closes: #38757

w interface IStructType is implemented by both Schema and StructType.

Overall, I think this change makes sense, considering what is needed in ADBC and that C++ as both a To/From StructArray method on RecordBatch. I prefer the IArrowRecord and IRecordType to the original proposed naming.


Glow shows a blank screen

Whenever I try to preview any markdown file using :Glow command, a blank floating window pops up.
I am using the default configuration.

@dasShounak are you using latest plugin version? What OS and neovim version are using as well? Can you share a minimal config to reproduce it?

@dasShounak Updating to the latest plugin version fixed the issue for me.

@johnhampton ahh… I might sound dumb, but how do you update a plugin to the latest version?

@ellisonleao I just installed the plugin with packer
use {"ellisonleao/glow.nvim", config = function() require("glow").setup() end}

I am using nvim v0.8.3 on fedora

Can you call :PackerSync just to make sure you are on latest?

Yeah, I'm on the latest

Did you already installed the glow binary (not the plugin) before installing the plugin?

No I didn't.

@dasShounak Can you show the output of
:lua vim.pretty_print(require('glow').config)

please?

Sure. Here's the output:
{
  border = "shadow",
  glow_path = "/home/c3rb/.local/bin/glow",
  height = 100,
  install_path = "/home/c3rb/.local/bin",
  mouse = false,
  pager = false,
  style = "dark",
  width = 100
}

Seems okay. unfortunately, i can't reproduce your issue on my side using a fresh install. Can you try:

remove the glow installation from /home/c3b/.local/bin/glow
call :PackerSync again
call :Glow for an unmodified md file
call :Glow <file> for an unomodifield md file in another path then pwd
call :Glow in a modified md file (open some md file, make some modifications, do not save, call :Glow)

It worked! Thanks a lot.


Provide an API for comparing a file between the two revisions

For example:
compare_file "Podfile.lock" do |before, after|
  reduced_deps =  ":tada: - Great work on reducing our incidental complexity"
  message(reduced_deps) if before.lines_of_code < after.lines_of_code
end

I wonder what attributes could be available?

I like the :tada:

Dupes - https://github.com/danger/danger/issues/282
both have different APIs too

Does #385 fix this?

ish, it would let you see the diff but not a before/after for those specific files - you'd need to check them out and pull out the text. Think there's still a space for this to co-exist.

We do this on Danger.systems
diff = git.diff_for_file("plugins.json")
if diff
  current_plugins = JSON.parse(diff.blob(:dst).contents)
  pr_plugins = JSON.parse(diff.blob(:src).contents)

  new_plugins = current_plugins - pr_plugins


Web Chat not able to communicate with bot on some Mac machines

Describe the bug
Some Mac users are experiencing issues when trying to communicate with a bot via the embedded Web Chat in Composer. From the logs, it looks like the runtime is getting a 403 when trying to reply to an activity.
OS Versions experiencing the bug: macOS Monterey 12.1
Note: We are not seeing this issue on Linux & Windows
Version
We know 2.1.1 to be a version in which we have observed the bug. However, we have not determined the earliest release in which the bug is present.
Browser

[x] Electron distribution

OS

[x] macOS
[ ] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Create or open a bot
Start the bot
Open the Web Chat pane
Try to communicate with the bot by typing messages in the Web Chat send box
Observe that the messages are not sending successfully in the Web Chat tab of the log

Expected behavior
The user should be able to send messages to the bot and receive responses on Mac.
Additional context
Here are the runtime errors we were seeing on the server side of Composer when observing this bug (you can see this output by starting the Composer executable from a terminal window):
PUT /api/projects/56762.55293620476/files/appsettings.json 200 48.405 ms - 34
POST /api/telemetry/events 200 0.774 ms - 2
POST /v3/conversations 201 1.617 ms - 302
POST /api/telemetry/events 200 0.693 ms - 2
2022-02-01 11:28:44.257 Bot Framework Composer[44452:2687805] Warning: Expected min height of view: (<NSPopoverTouchBarItemButton: 0x7fa1c3025130>) to be less than or equal to 30 but got a height of 32.000000. This error will be logged once per view in violation.
  composer:extensions:localpublish:44516 fail: Microsoft.Bot.Builder.Dialogs.Adaptive.Runtime.CoreBotAdapter[0]
      [OnTurnError] unhandled error : Operation returned an invalid status code 'Forbidden'
Microsoft.Bot.Schema.ErrorResponseException: Operation returned an invalid status code 'Forbidden'
   at Microsoft.Bot.Connector.Conversations.ReplyToActivityWithHttpMessagesAsync(String conversationId, String activityId, Activity activity, Dictionary`2 customHeaders, CancellationToken cancellationToken)
   at Microsoft.Bot.Connector.ConversationsExtensions.ReplyToActivityAsync(IConversations operations, String conversationId, String activityId, Activity activity, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.CloudAdapterBase.SendActivitiesAsync(ITurnContext turnContext, Activity[] activities, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.TurnContext.<>c__DisplayClass31_0.<<SendActivitiesAsync>g__SendActivitiesThroughAdapter|1>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.Bot.Builder.SetSpeakMiddleware.<>c__DisplayClass3_0.<<OnTurnAsync>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.Bot.Builder.TurnContext.SendActivityAsync(IActivity activity, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.Adaptive.Actions.SendActivity.BeginDialogAsync(DialogContext dc, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.DialogContext.BeginDialogAsync(String dialogId, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.Adaptive.Actions.ActionScope.BeginActionAsync(DialogContext dc, Int32 offset, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.Adaptive.Actions.ActionScope.BeginDialogAsync(DialogContext dc, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.DialogContext.BeginDialogAsync(String dialogId, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.DialogContext.ReplaceDialogAsync(String dialogId, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.Adaptive.Actions.IfCondition.BeginDialogAsync(DialogContext dc, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.DialogContext.BeginDialogAsync(String dialogId, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.Adaptive.Actions.ActionScope.BeginActionAsync(DialogContext dc, Int32 offset, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.Adaptive.Actions.Foreach.NextItemAsync(DialogContext dc, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.Adaptive.Actions.Foreach.BeginDialogAsync(DialogContext dc, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.DialogContext.BeginDialogAsync(String dialogId, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.Adaptive.Actions.ActionScope.BeginActionAsync(DialogContext dc, Int32 offset, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.Adaptive.Actions.ActionScope.BeginDialogAsync(DialogContext dc, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.DialogContext.BeginDialogAsync(String dialogId, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.Adaptive.AdaptiveDialog.ContinueActionsAsync(DialogContext dc, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.Adaptive.AdaptiveDialog.BeginDialogAsync(DialogContext dc, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.DialogContext.BeginDialogAsync(String dialogId, Object options, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.DialogExtensions.InnerRunAsync(ITurnContext turnContext, String dialogId, DialogContext dialogContext, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.DialogExtensions.InternalRunAsync(ITurnContext turnContext, String dialogId, DialogContext dialogContext, DialogStateManagerConfiguration stateConfiguration, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.DialogExtensions.InternalRunAsync(ITurnContext turnContext, String dialogId, DialogContext dialogContext, DialogStateManagerConfiguration stateConfiguration, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.DialogExtensions.RunAsync(Dialog dialog, ITurnContext turnContext, IStatePropertyAccessor`1 accessor, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.Dialogs.Adaptive.AdaptiveDialogBot.OnTurnAsync(ITurnContext turnContext, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.SetSpeakMiddleware.OnTurnAsync(ITurnContext turnContext, NextDelegate next, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.MiddlewareSet.ReceiveActivityWithStatusAsync(ITurnContext turnContext, BotCallbackHandler callback, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.BotAdapter.RunPipelineAsync(ITurnContext turnContext, BotCallbackHandler callback, CancellationToken cancellationToken)
+5s

@OEvgeny could you look into this when you get a chance?

@tonyanziano don't have a Mac machine. I am going to try to repro on Linux

Ahh, my mistake. I thought you had a Mac. It seems like users on Mac Monterey (12.1) are experiencing this issue.

Ok, no problem. I see no any sign it can be reproed outside of Mac

Short term fix @nightscape is to turn off "Airplay Receiver" in your system preferences, then restart the Bot Composer.
If you nlsof -i :5000 before and after you'll see that port become part of the Bot Composer.
It would be useful to allow this port to be something other than 5000 on a Mac.


[ENH] Make explicit that "task" metadata applies to "beh" modality

Fixes #795
Adds task metadata that exist in other modalities to:

behavioral
PET

All metadata is RECOMMENDED.
Making TaskName REQUIRED (like it is for other modalities) could break backward compatibility, no?
I used a pretty global approach for beh assuming that these metadata could go in any of JSON flles on can find in this folder: should we have a more fine grain approach to this?

[ ] update validator according to this PR (see this comment)

@melanieganz and @mnoergaard I am adding you as reviewers as this also relates to PET. :smile:

LGTM, probably something to remind ourselves of in #774

Good point.
Once this is merged I will update #774 (and also the validator ?)

Mmh yes ... about updating the validator: Would you add it to the events.json schema, or the beh.json schema? The latter doesn't exist yet, but I think it'd be a better choice, and also something to clarify in this PR. WDYT?
Otherwise we'd have to specify it in the events.json schema: https://github.com/bids-standard/bids-validator/blob/master/bids-validator/validators/json/schemas/events.json

I would be more in favor of creating a beh.json schema.
This will need to happen at some point anyway.

This PR will have to be updated with respect to #774 --> metadata tables are now rendered via macros directly from the schema.

Updated the PR to use the macros and the schema definitions.
The problem is that now the definitions for TaskName and Instructions both make reference to resting state which does not apply to behavior (see @effigies comments above).
Suggestions: shorten the base definitions and add the "resting state" mentions where necessary (func, eeg, meg, ieeg).
@tsalo does that make sense to you?

:+1: That works for me!

Tempted to add some info in the contributing.md on how to formulate the call to the macros to use the metadata description contained in the schema.
Just so that people are aware that those base description can be extended.

Tempted to add some info in the contributing.md on how to formulate the call to the macros to use the metadata description contained in the schema.
Just so that people are aware that those base description can be extended.

I think that's a good idea 👍  we'll have to refer people to such docs quite some times in the future

Thanks @Remi-Gau :tada: let's get the validator PR fixed and merged next

yup.
in the meantime I will update #839


Expandable row table doesn't work with striped

Issue
Seems like the striped property of the react bootstrap table doesn't work on tables with expandable row option enabled.
Example
Even the examples of the expandable row tables that have the striped property with value true aren't actually striped tables.

it's a known issue, check this. Currently, i still can not find a good solution, but there's a workaround, let me know if you have any questions.  Thanks


Image occlusion and always different questionnaire

Hello first of all great plugin, I see that it will incorporate questions based on images and pdf in future versions, so I share my ideas

The possibility of incorporating image occlusion type cards

Use multimodal models to recognize text in images to cover said text automatically, I would like this to be used with the excalidraw plugin so that it is compatible, here is an example of how it can be made simple
https://github.com/user-attachments/assets/2b86a7d7-6d44-40fa-aa7b-21dbe4886c1c

Always different questions

this is related to testo mode and Question Variety, the idea is that sometimes there are concepts that must be memorized, there is no other way, but on other occasions they must only be understood in their different situations, the clearest example of this is mathematics, providing the LLM as a source the notes or PDFs (and other sources if necessary) so that in each new study session, the exercises are always new and based on this evaluation the next session will be more difficult 🫡

For your second idea, do you mean the generated questions change depending on what questions were generated previously, and the difficulty changes depending on how many questions you answered correctly?

Yes, I thought about it like this, if a question that was created was later evaluated and answered well, then in the next session that same question will increase in difficulty. Here I show you a diagram, I hope I express myself well, I am not a programmer.
graph TD
    A[Create Question] --> B[Deck]
    B --> C[Evaluate Question]
    C --> D{How did you respond?}
    D --> E[Bad or so-so]
    E --> F[Remains the same]
    F --> C
    D --> G[Good]
    G --> H{How many times did you answer correctly?}
    H --> I[Only once]
    H --> J[More than once]
    I --> F
    J --> K[Increase difficulty of this card only]
    K --> A

Oh, I see. I think I have a pretty good idea of what you're talking about (the diagram is very helpful). This is a really interesting idea so I'll likely include it as part of the Test Mode feature.

Great, to clarify further I see that I forgot to add after more than once that said card that has already been answered correctly more than once will return to the deck but in "learned" mode so that it is not shown again, but also not deleted, I will only be able to see it if I unlock it for the final review 😉


how to change the text background color?

Do you mean as in drawing a colored rectangle behind the text? If so, then the library itself does not support that (yet?).
Instead you'd have to "manually", issue the needed draw calls to draw a rectangle behind the text beforehand. You can use the functions gltGetTextWidth and gltGetTextHeight to query the bounds of the text, for the rectangle.


Preloading in Storybook and adding more font Chromatic stories

Closes #4165 #4125 #3834
Adding a story to test #3983 which failed. Will be handled in another sprint.

Copied the Chromatic font preloading to Storybook.
Switched preloading of fonts for both Chromatic and Storybook to one typekit web project.
Updated the font family definitions to be more "correct" and prevented CCJK fonts from being italics. Added chromatic stories to cover reported instances of components with italics that weren't correct with CCJK fonts.
Adding CCJK fonts for docs, for someone using the docs from another local and the translated strings use the correct fonts.
Pulled ActionButton, TextArea, and Textfield font stories into new languages package.

Note: Chrome dev tools sometimes show the font name and sometimes an id. The Firefox dev tools have a fonts tab that shows both the id and the font name.
✅ Pull Request Checklist:

[x] Included link to corresponding React Spectrum GitHub Issue.
[x] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[x] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:
Run chromatic and confirm that the CCJK fonts are not italic.
Run chromatic locally and check via dev tools that the correct fonts are used.
Check the new provider locale zh-Hant font family story to see the rendered font via dev tools.
🧢 Your Project:
RSP

Note: let's test off the PR for this one because merging it would affect the docs in production.

Note: let's test off the PR for this one because merging it would affect the docs in production.

I can move


Add basic infrastructure for generic overlays on player

includes some minor vertical responsiveness

as well as infrastructure to allow us to support ad takeovers, endslate, and a redesigned share overlay, for both the large version:

and small version of the player:

This PR includes CSS vars which there is no way to make angular-cli work with so needs to be manually handled before merging. I wanted to leave them in place in case changes were requested

looks a little funky for playlists! margin collapses between the scrub bar and the top ep.

https://play.prx.org/e?uf=http:%2F%2Ffeeds.99percentinvisible.org%2F99percentinvisible&sp=
vs.

http://play.prx.dev/e?uf=http:%2F%2Ffeeds.99percentinvisible.org%2F99percentinvisible&sp=

Hello, @gcampo88 and @cavis ! I just joined the team at RadioPublic and just added two commits to address two items:
• Fix the playlist margin funkiness that Gigi pointed out
• Replace the CSS vars that Chris added during development
Like Chris mentioned, angular-cli currently doesn't expose a way to add the postcss-custom-properties plugin (although an issue was recently opened for that). I had considered making player.component.css an scss file in order to install Sass to use variables, but I decided to defer that  since I'm not sure yet how you two feel about preprocessors.

hey @pkarman been a while! I assume this assignment means it is up to me to push that big green button? any objections from @gcampo88 or @cavis?

No objections ... assuming that you've re-verified that thing @gcampo88 was seeing.

does still seem to take away the rounded corners on the player when showing playlist!
(left: dev, right: prod)

but not sure how crucial that border-radius is down there on the playlist view...

@gcampo88 , thanks for spotting that. I'm taking a look at a fix for the border-radius now.

also, hi @BinaryNate! Congrats on joining RP!

Thanks, @gcampo88 ! I just pushed a small tweak to bring back the bottom border-radius.

Nice!
Oops, just noticed this -- the builder offers a 500x500 "square" version of the player (which translates to it just being 500px wide and the usual height tall), and with this PR the background artwork extends down to 500px tall and looks bad. e.g.:

That said, we do have this ticket #153 to just get rid of that "square" option, since it's not actually a real option (we don't have a 500x500 version to offer them). So maybe we should just kill that now and give this the 👍 ? @cavis is out today -- perhaps he can weigh in early next week and then we can merge this thing!

@gcampo88 , thank you for catching that. I just pushed a commit to fix it by adding the max-height of 200px like @cavis  had suggested.

Yeah, let's not worry about the square view for now.  (Other than not extending the BG-image into it, which seems to be taken care of now)


Quake II with VKQuake2 won't open (but keeps running)

Describe the bug
I try to run Quake II using VKQuake2 as engine, and it shows it's running, but no window is open. I have to stop it manually from Steam.
To Reproduce
Steps to reproduce the behaviour:

Run Quake II from Steam and select VKQuake2 as engine.
Nothing happens, but it shows it keeps running
Tried over and over and nothing changes.

Expected behaviour
Quake II should run with VKQuake 2 and a window should open.
Additional context
Using Nvidia GTX 1650 with privative drivers, which supports Vulkan. I use Manjaro as distro.
In the meantime, I'm using Yamagi Quake 2 with Vulkan support and works flawlessly.

The only log I can leave here is this output, which is repeated infinitely until I stop the game from the Steam library:
------- Loading ref_vk.so ------- LoadLibrary("ref_vk.so") failed: libXxf86dga.so.1: cannot open shared object file: No such file or directory Only Vulkan renderer is supported on Linux.
I guess is the ref_vk.so inside the vkQuake2 directory, because I don't have the same issue with Yamagi Quake 2, which it has its own ref_vk.so (which I had to compile it apart).

Ok, that's been fixed in this https://github.com/luxtorpeda-dev/packages/pull/410
It looks like libXxf86dga is not included on all systems, it's a x11 package. I added compilation steps so that it is included with the engine.
Next time you launch it should work. Let me know if you have any issues.

Hi, now it launches, and I noticed that I haven't installed libxxf86dga, so I did it and now it works. Thank you!


Update to Big Sur

I wonder if we can update to Big Sur without having any problem.
Did somebody tried to do it?
I've read on a french forum some people using OpenCore 0.6.3 and updating successfully:
https://forum.macbidouille.com/index.php?s=56ae222578c674832685e2ee1cc104d3&showtopic=418642

I'm trying to update to big sur, also I had patched my edid to force 48hz, under calatina patched edid works well, but on big sur it's hanging after loading whatevergreen.

Got BigSur with 4k on Dell XPS 2020 working, only trackpad make problems, best Regards
I'll create later a own repo on Github for this

What kind of issue do you have with trackpad? Did you install BigSur from scratch or from Catalina with update tool?

Trackpad is solved for me.

@BenDevelopment from scratch, no issue at the moment the only working version for my touchpad and touchscreen is 2.4.4 from voodooi2c

@BenDevelopment from scratch, no issue at the moment the only working version for my touchpad and touchscreen is 2.4.4 from voodooi2c


Confluent.Kafka.ConsumeException: Local: Timed out

Description
We are experiencing the below error intermittently while consuming the messages from Kafka
Confluent.Kafka.ConsumeException: Local: Timed out
at Confluent.Kafka.Consumer2.ConsumeImpl[K,V](Int32 millisecondsTimeout, IDeserializer1 keyDeserializer, IDeserializer1 valueDeserializer)  at Confluent.Kafka.Consumer2.Consume(CancellationToken cancellationToken)
How to reproduce
The below configuration is using in our application
return new ConsumerConfig
{
BootstrapServers = _consumerConfiguration.BootstrapServers,
GroupId = _consumerConfiguration.GroupId,
SessionTimeoutMs = 100000,
AutoOffsetReset = AutoOffsetReset.Earliest,
EnablePartitionEof = true,
EnableAutoCommit = _consumerConfiguration.EnableAutoCommit
};
Checklist
Please provide the following information:

[ ] A complete (i.e. we can run it), minimal program demonstrating the problem. No need to supply a project file.
[ ] Confluent.Kafka nuget version.
[ ] Apache Kafka version.
[ ] Client configuration.
[ ] Operating system.
[ ] Provide logs (with "debug" : "..." as necessary in configuration).
[ ] Provide broker log excerpts.
[ ] Critical issue.

i don't have time to trace this through to be 100% sure, but this is due to an op on the fetch queue having this error set, which I would guess is due to a protocol fetch request timing out. so indicative of network connectivity / broker issues.
also, this seems to be a case of librdkafka leaking a re-tryable error (only non-retryable errors should be reported to the user in this way). we need to audit all this and precisely document / tweak.
you can safely ignore this (perhaps just log it).

ed-alertedh, I am seeing similar issue, have you got to the solution to it?
After these errors, some consumers doesn't resume message/record processing even if partition has records and consumer is healthy and polling.

"error":"Local: Timed out","error":"commit error" : This is happening on commit.
"error":"GroupCoordinator: 1 request(s) timed out: disconnect (after 6847266ms in state UP, 1 identical error(s) suppressed)"
socket.timeout.ms is set to 60000ms right now.

Any luck with this challenge yet?

We have the same issue in Producer, Exception like this:
Confluent.Kafka.ProduceException2[System.String,System.String]:  Local: Message timed out    at Confluent.Kafka.Producer2.ProduceAsync(TopicPartition topicPartition, Message`2 message, CancellationToken cancellationToken)


Retest only test that fail and not entire package

Correct me if I'm mistaken, but it seems that colcon test --retest-until-fail N colcon test --retest-until-pass will rerun all tests for a failing package, as opposed to only the specific tests in the package that failed. Could this be changed so the results from a previous job be used as a block list to scope which specific tests need to be repeated, not merely the package. This could help speed up CI pipelines with flaky tests.
@SteveMacenski, could you link to a previous job that exhibits the behavior you described to me offline.

... as opposed to only the specific tests in the package that failed

The option --retest-until-fail doesn't ever re-run a test which has failed. It only re-runs all tests when in the initial / previous run all tests have passed. If any test fails no more re-runs are attempted.
The use case of the option is to check for flaky tests. So passing --retest-until-fail N ensures that all tests passed N+1 times and never failed.

Woops, I typo-ed, I meant to reference --retest-until-pass.

For CMake packages the option --retest-until-pass maps to the CTest option [--rerun-failed](https://cmake.org/cmake/help/latest/manual/ctest.1.html#options which only re-runs the tests which have failed: https://github.com/colcon/colcon-cmake/blob/b1963768a8484aebdf827fdf10303ebe73003b45/colcon_cmake/task/cmake/test.py#L105-L107
For Python packages using pytest the option --retest-until-pass maps to the pytest-rerunfailures option --reruns: see https://github.com/colcon/colcon-core/blob/2a12b22aa2b4635e2be5bd31138795a73ad63614/colcon_core/task/python/test/pytest.py#L126-L137

@ruffsl https://app.circleci.com/pipelines/github/ros-planning/navigation2/3553/workflows/67da27fb-e87b-46d9-b33b-840882c9bb19/jobs/13927/steps this is one such job. You can see it fails the first time on nav2_system_tests after about 10 minutes (about as long as it takes to run all of the tests), then it re-runs and take another 10 minutes to fail. No single test takes 10 minutes so that was re-running the full suite.

The log shows a completely separate second invocation of colcon test with the argument --packages-select-test-failures which will run all tests for the packages which has test failures in the first invocation of colcon test.
This kind of repetition has nothing to do with the re-running of tests based on the arguments --retest-until-fail / --retest-until-pass.

The log shows a completely separate second invocation of colcon test

This may have gotten lost in the line nose thanks to RCUTILS_CONSOLE_STDOUT_LINE_BUFFERED.
Although, from https://github.com/ros-planning/navigation2/issues/1271#issuecomment-545654248 , not sure why time ordering of console messages is lost without it.
Thanks for spotting this @dirk-thomas !
Looks this is a side effect from https://github.com/ros-planning/navigation2/pull/1284 . I'll close this and move discussion there.


Ideas for CI-only tests

Certain tests we could run would require installing actual software into specific locations, thus are unfeasible to run on contributors' machines. If someone could figure out how to set up a CI server with the necessary environment, I could write the tests.
Testing nodeBin behavior
This would involve installing various versions of Node into known locations.

Install three different versions of Node:

One bleeding-edge
One LTS
One explicitly too old to support module syntax (maybe?)
Maybe use NVM for this? Not sure how well it plays with CI environments

Set nodeBin to the LTS version via package settings

Open a project with ESLint v8 or greater

lint a file; expect results
run the debug action; get output with a Node version number that matches what we expect

Create a .linter-eslint file with { "nodeBin": "[path to bleeding-edge node]" }

save the .linter-eslint file; wait a few seconds
lint a file; expect results
run the debug action; get output with a Node version number that matches what we expect

Modify the .linter-eslint file to read { "nodeBin": "[path to too-old node]" }

save the .linter-eslint file; wait a few seconds
run the debug action; get output with a Node version number that matches what we expect
lint a file; expect… not sure? failure of some kind

Delete our .linter-eslint file

expect behavior to match what we saw above with LTS node

Change the node path in package settings via atom.config.set to bleeding-edge

expect behavior to match what we saw above with bleeding-edge node

Change the node path in package settings to something nonexistent

expect a warning message

Testing behavior with various ESLint versions
We read ESLint's package.json to figure out what version we're dealing with. So to write specs against this behavior, we'd have to have at least three different dummy projects, each of which installs a different version of ESLint:

One v8 or greater (latest stable version)
One v7 (to test overlap behavior)
One <v7 (to test eslint-too-old behavior)

Things to test:

Open the v8-or-greater version with linter-eslint package installed, but linter-eslint-node disabled

Expect a notification that prompts us to upgrade

Open the v8-or-greater version with linter-eslint package enabled and linter-eslint-node enabled

Ensure no notification appears that prompts us to upgrade
Ensure the “Debug” command correctly identifies linter-eslint-node as handling the linting for this project

Open the v7 project with linter-eslint disabled

Ensure linter-eslint-node lints as expected

Open the v7 project with linter-eslint enabled

Ensure the “Debug” command correctly identifies linter-eslint as handling the linting for this project
Ensure linter-eslint-node doesn't try to lint; explicitly calling provideLinter().lint should produce zero results, even on a file with a syntax error

Open the <v7 project with linter-eslint disabled

Expect a notification that prompts us to install linter-eslint

Repeat steps, but with relevant setting unchecked; expect no notification

Ensure linter-eslint-node doesn't try to lint; explicitly calling provideLinter().lint should produce zero results, even on a file with a syntax error

Open the <v7 project with linter-eslint enabled

Expect no notification
Ensure linter-eslint-node doesn't try to lint; explicitly calling provideLinter().lint should produce zero results, even on a file with a syntax error

We can setup GitHub Action workflows for the different setups. If we have some tests we only want to run in CI we can use process.env.CI.
something like:
if (process.env.CI) {
  describe('CI tests' () {
      ...
    }
  );
}


Fix shortcut table

Noticed that the table looked like this

GH does not represent it like on our website.
In the unmodified file it looks fine:
https://github.com/FreeTubeApp/FreeTube-Docs/blob/master/usage%2Fkeyboard-shortcuts.md


Update Spanish translation

Spanish translation didn't make any sense.
Fixed Issues
$ https://github.com/Expensify/App/issues/5990
Tests

Navigate to add bank account flow (app must be in spanish)
Add a Verifying bank account (must go through onfido flow)
Reach the "Let's finish in chat" Modal.
Spanish copy should have been updated, instead of "Acabemos en el chat" and "Gracias por hacer eso", it should say "Continuemos en el chat" and "Gracias" respectively.

QA Steps
No QA.
Tested On

[ ] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

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

🚀 Deployed to staging by @yuwenmemon in version: 1.1.8-10 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


fix: upgrade pnpm/action-setup

This fixes the Run_Svelte_Check_on_PRs action and gets the Run_Playwright_Tests_on_PRs a bit further.


Create ReverseLinkList

Description

Checklist

[ ] An issue was first created before opening this pull request
[ ] The new code follows the contribution guidelines
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests to ensure that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

您好，您的邮件我已经收到，会尽快处理，谢谢！中建八局设计管理总院 胡伟


get latest state data for media player entity

Hello,
using your example code with SubscribeToEvents/ReadEventAsync, I can get media player data like this  every time play/pause/stop status changes:
media_player.plex_plexamp_dev5: playing->playing
is_volume_muted False
media_content_id 18220
media_content_type music
media_duration 295
media_position 15
media_position_updated_at 2022-07-22T10:34:36.717561+00:00
media_title Space cowboy
media_artist Steve Miller Band
media_album_name Ten Songs for you
media_album_artist Steve Miller Band
media_track 1
media_library_title Music
player_source session
username mhwlng
entity_picture /api/media_player_proxy/media_player.plex_plexamp_dev5?token=bc9a7bfac19719e4b8051f2890c457eef6f779bf6d2eb43f489096e792b2eb54&cache=44e228a1b6fa7fd2
friendly_name Plex (Plexamp - DEV5)
supported_features 131584
The problem is that media_position , media_position_updated_at is only updated whenever play/pause/stop status changes.
I tried to call, every few seconds :
var stateMessage2 = await  client.GetState("media_player.plex_plexamp_dev5");
but that does not return the current media_position , media_position_updated_at  value,
I get the same data from the last state_changed event (play/pause/stop)

It seems this behavior is by design:
https://community.home-assistant.io/t/no-media-position-update/200406/2
so, never mind....


some providers are mangling $http's default content-type

Our app makes extensive use of $http and it defaults to application/json.  When a user tries to login with AzureAD, the following code will override the default header and cause things to mess up.
in oauth.azuread.js:
  $http.defaults.headers.post['Content-Type'] = 'application/x-www-form-urlencoded';

Rather than set the global header, I have only set it on the specific request like so:
$http({
  method: "post",
 headers: {'Content-Type': 'application/x-www-form-urlencoded'},
...
});

I noticed a bunch of other OAuth providers seem to have the same issue.  I'll try to update them all and do a pull request.

Hi, I updated the pull request to point to the development branch.  (I'm not the most skilled with GitHub, so I hope I've done it correctly).

PR https://github.com/nraboy/ng-cordova-oauth/pull/267 is pending due to errors in the Travis CI tests.

looks like the PR was merged in dev. is there an estimate for when these changes will be merged to master and released?
just spent the evening trying to figure out why after using a new oauth service, the rest of my http calls were being mangled.

I usually merge the development branch on the weekend.  I like to do it when I know I'll have time to troubleshoot, should things not work.  I have no doubt that these changes will work flawlessly.  Just more of a personal habit.
Thanks for being patient.
Best,

This is now in version 0.2.10.  Thanks for being patient :-)


Custom Element Status

It would be useful to have a "status" field (free text). I'd use it for something like this:

alpha
beta
stable
deprecated

I don't really think we could regularize this well, personally. Since npm packages are semver versioned, packages can have pre-release suffixes like any other.
I presume a catalog-type app would not show preview releases by default, but could be configured to as an option.

an npm package could contain many components... some could be stable, some could be alpha... so a "state" per component could be useful to display to users in a readme or documentation site 🤗

I would love this. Right now we are using this plugin to add a custom property to the manifest through a jsDoc tag. It would be great if it were part of the actual schema.


[Fix #9689] Treat parens around array items the same for children and deeper descendants

Previously there was a disparity between these two code blocks:
[
 (
   1
 ),
 2
]

[
 x = (
   1
 ),
 y = 2
]

The first was accepted as "plausible" but the second registered an offense (and was autocorrected to a syntax error because the comma in an array cannot be on its own line).
This change allows both to be treated the same by looking for any ancestor to be an array or hash rather than just the direct parent.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Looks good! Thanks!


FIX Update ghostAPI.ts to use environment URL instead of package URL when available

Fix setting of ghostURL to use user set URL instead of default
Currently script prioritizes CONF_URL which is a package default.
set CONTENT_API_URL to have priority to ensure user set URL has priority.

Hi, The recommended setup is to define the actual ghostURL within your astro.config.mjs instead of .env ( example ) since its not considered a protected key/secret by GhostCMS.  Though from my own testing it should default to what ever option you have set without needing this change as i do not set a default ghostURL myself.
Side note... I will be rebuilding the Astro-GhostCMS integration system here soon as i have learned a ton from my developing StudioCMS


Add config option to disable flat bedrock.

The flattening bedrock feature is interfering with a mod I'm using to remove the ceiling of the nether. Also, you should document somewhere that netherex flattens the nether's bedrock. I was only able to find out it was this mod by disabling every other mod I had.

Flat bedrock has been removed in newer versions.


upgrade(fleet): Support fleet policies for the injector

What does this PR do?
Supports the APM Injector for Fleet policies
Motivation
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes

/merge


Racer error every few minutes

rust stable 1.5 / Windows 10
code completion seems to work, but every few minutes this error pops up. After dismissing it continues to work.
Last command:
complete-with-snippet 50 -1 c:\Users\nathan\repos\program_name\src\file.rs C:\Users\nathan\AppData\Local\Temp\tmp-9576Sj9wVh3anPK7.tmp

Racer Output:
error: 'c:\Users\nathan\repos\program_name\src\file.rs ' isn't a valid value
USAGE:
	racer complete-with-snippet <fqn>
	racer complete-with-snippet <linenum> <charnum> <path> [substitute_file]
Please re-run with --help for more information
Racer Error:
Racer Output:
error: 'c:\Users\nathan\repos\program_name\src\file.rs ' isn't a valid value
USAGE:
	racer complete-with-snippet <fqn>
	racer complete-with-snippet <linenum> <charnum> <path> [substitute_file]
Please re-run with --help for more information
Racer Error:

Do you have a way to reproduce this? that -1 column number is really strange, I can put bound checks on the command triggers, but I would like to know the cause.

I will try to get a simple case to reproduce it. So far it seems fairly random.

If I slowly type the following invalid code into Visual Studio Code, the Racer error appears:
fn main {
    let t = "".();
}

If it doesn't appear right away, try removing and adding the parentheses again.
This is the error I get:
Last command:
complete-with-snippet 135 -1 /home/nikon/Documents/Testing/euler3/src/main.rs /tmp/tmp-23302salenBkuEKqh.tmp

Racer Output:
error: '/home/nikon/Documents/Testing/euler3/src/main.rs' isn't a valid value
USAGE:
	racer complete-with-snippet <fqn>
	racer complete-with-snippet <linenum> <charnum> <path> [substitute_file]
Please re-run with --help for more information
Racer Error:
bogofile:0:1: 0:1 error: unexpected token: `<eof>`
bogofile:0:1: 0:1 error: unexpected token: `<eof>`
bogofile:0:1: 0:1 error: unexpected token: `<eof>`
bogofile:0:1: 0:1 error: unexpected token: `<eof>`
bogofile:0:1: 0:1 error: unexpected token: `<eof>`

I'm having the same problem with "," it appears if you type a comma in a line without an opening parens.

I'll look into it.

New version is out, please update extension and let me know if i fixed this issue or not :)

The fix remove the crash for me :+1:

Yeah, it seems OK for me too :+1: Awesome!

Looks like it has been fixed for me. Thanks.


Failed to parse error

My code works fine If I'm running it using VScode with the Python virtualenv
Python version: [3.8.5] 64-bit
however if I run the same code in the terminal using Python [3.8.5] 64-bit
I get this error here
nick-pc@nickpc-HP-xw4300-Workstation:~/Documents/pythonprojects/DCAbot$ /usr/bin/python3 /home/nick-pc/Documents/pythonprojects/DCAbot/DCAETHbot.py
Traceback (most recent call last):
File "/home/nick-pc/.local/lib/python3.8/site-packages/requests/models.py", line 382, in prepare_url
scheme, auth, host, port, path, query, fragment = parse_url(url)
File "/usr/lib/python3/dist-packages/urllib3/util/url.py", line 392, in parse_url
return six.raise_from(LocationParseError(source_url), None)
File "", line 3, in raise_from
urllib3.exceptions.LocationParseError: Failed to parse: https://api.binance.com/api/v3/ping
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/nick-pc/Documents/pythonprojects/DCAbot/DCAETHbot.py", line 8, in
client = Client(keys.api_key, keys.api_secret)
File "/home/nick-pc/.local/lib/python3.8/site-packages/binance/client.py", line 105, in init
self.ping()
File "/home/nick-pc/.local/lib/python3.8/site-packages/binance/client.py", line 392, in ping
return self._get('ping', version=self.PRIVATE_API_VERSION)
File "/home/nick-pc/.local/lib/python3.8/site-packages/binance/client.py", line 237, in _get
return self._request_api('get', path, signed, version, **kwargs)
File "/home/nick-pc/.local/lib/python3.8/site-packages/binance/client.py", line 202, in _request_api
return self._request(method, uri, signed, **kwargs)
File "/home/nick-pc/.local/lib/python3.8/site-packages/binance/client.py", line 196, in _request
self.response = getattr(self.session, method)(uri, **kwargs)
File "/home/nick-pc/.local/lib/python3.8/site-packages/requests/sessions.py", line 555, in get
return self.request('GET', url, **kwargs)
File "/home/nick-pc/.local/lib/python3.8/site-packages/requests/sessions.py", line 528, in request
prep = self.prepare_request(req)
File "/home/nick-pc/.local/lib/python3.8/site-packages/requests/sessions.py", line 456, in prepare_request
p.prepare(
File "/home/nick-pc/.local/lib/python3.8/site-packages/requests/models.py", line 316, in prepare
self.prepare_url(url, params)
File "/home/nick-pc/.local/lib/python3.8/site-packages/requests/models.py", line 384, in prepare_url
raise InvalidURL(*e.args)
requests.exceptions.InvalidURL: Failed to parse: https://api.binance.com/api/v3/ping

Python version: [3.8.5] 64-bit
Virtual Env: [virtualenv]
OS: [Ubuntu]
python-binance version [0.7.9]

uh nevermind I think I solved this issue by updating urllib3
python -m pip install --upgrade urllib3


[Feature Request] A callback function prop that executes after the marquee has finished rendering

It would be really nice if there was a callback prop with the name along the lines of onRenderComplete or onRenderFinish so we can run some logic after the component has been successfully rendered.
An example use case for this would be to refresh the scroll trigger of the GSAP library so it can recalculate measurements after the <Marquee /> component has been rendered.

@rubek-joshi this issue causes a layout shift which makes Gsap's measurements slight off, correct?


Fix snapping when zoomed in Polygon2D UV editor

After reading the code I concluded that uv_draw_ofs is not affected by the uv_draw_zoom, hence should be multpliplied by it as well when snapping.
Also simplified some code when reading.
Fixes #94385.

Before(v4.3.rc2.official [3978628c6])
After(this PR)

Thanks!

Cherry-picked for 4.3.1.


Use Maven to generate a Jandex index instead of indexing at every build

I think this should be avoided:
https://github.com/quarkiverse/quarkus-vault/blob/main/deployment/src/main/java/io/quarkus/vault/deployment/VaultProcessor.java#L61
and a Jandex index created by Maven via the Jandex Maven plugin for the quarkus-vault-model artifact instead.
/cc @vsevel

good catch @gsmet :)


KeepAlive

Verification
Before opening a bug make sure that the following conditions are met.

Performance issues are also appearing in RELEASE mode of your application.
An increased memory consumption or high CPU load also happens when your application runs in RELEASE mode AND logging is DISABLED.
Client issues with not received messages, connection drops etc. can be reproduced via another client application like "MQTTnetApp" (https://github.com/chkr1011/MQTTnetApp) or similar.
The bug also appears in the VERY LATEST version of this library. There is no support for older version (due to limited resources) but pull requests for older versions are welcome.

Describe the bug
A clear and concise description of what the bug is.
I have set the KeepAlivePeriod to 20 seconds but it does not seems to trigger a disconnect when the network is down.   Currently a disconnect will only trigger after 18 minutes of the network going down.
Which component is your bug related to?

ManagedClient version <IP_ADDRESS> on dotnet 6.0

To Reproduce
Steps to reproduce the behavior:

Using this version of MQTTnet <IP_ADDRESS> on dotnet 6.0.

 var options = new MqttClientOptionsBuilder()
           .WithClientId(_clientID)
           .WithTcpServer((opt) =>
           {
               opt.Server = "broker.emqx.io";
               opt.Port = 8883;

               opt.TlsOptions = new MqttClientTlsOptions
               {
                   UseTls = true,
                   SslProtocol =
                       System.Security.Authentication.SslProtocols.Tls12 |
                       System.Security.Authentication.SslProtocols.Tls13,
               };

           })
           .WithKeepAlivePeriod(TimeSpan.FromSeconds(20))
           .WithTimeout(TimeSpan.FromSeconds(10))
           .Build();

Expected behavior
The keepalive to kick in very 20 seconds and to trigger the disconnect event when there is no response.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context / logging
Add any other context about the problem here.
Include debugging or logging information here:
\\ Put your logging output here.

Code example
Please provide full code examples below where possible to make it easier for the developers to check your issues.
Ideally a Unit Test (which shows the error) is provided so that the behavior can be reproduced easily.
\\ Put your code here.

I can confirm this issue. Looks like the async-wait for the ping reply packet is performed without any timeout.

There is a change on the Ping and merged  but not release on the latest version.

This is indeed already merged and will be fixed in version 4.1.1 which will be released soon.


Loki: Add yoecldp.F90 header that contains NCLV to fix SCC demotion

If Loki doesn't recognise NCLV as a parameter it will not demote the solver matrix arrays correctly.
@reuterbal This might be related to the last minute Scheduler changes, or might have been broken for a little while (not sure). However, with this in place, we get and the other Loki change we get equivalent performance between the respective "loki" and "gpu" variants again.

Closing now, as superseded by #80.


Training file

Thanks for this brilliant piece of work.
Can you provide the .py file you used for the training ? (for reproducing purpose )

we provide an inference script to validate our published scores
due to commercial limitations, we cannot share our full training script.
however, if you already have a multi-label framework, you can use ASL as a drop-in replacement for your loss.


Update TE-11.3

Changes:

Create a topology section to increase readability.
Refactored the wording to make it more readable.
Update existing test cases to add recursive lookup scenario.
Add decap-and-encap test scenario.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1181

Relevant Lines:
2064

💛  - Coveralls

Added changes as part of PR # 1335
https://github.com/openconfig/featureprofiles/pull/1335/

implementation is done per https://github.com/openconfig/featureprofiles/pull/1335


test: [M3-8471] - Allow Cypress region selection to work with Gecko options improvements

Description 📝
This updates the ui.regionSelect.findItemById() and ui.regionSelect.findItemByLabel() helpers so that they work when the upcoming Gecko changes are enabled and disabled. Previously, these helpers would fail to select a region when Gecko is enabled due to the layout improvements made to the region selection options.
Changes  🔄

Use RegEx to let ui.regionSelect.findItemById() and ui.regionSelect.findItemByLabel() work with and without Gecko

Target release date 🗓
N/A
How to test 🧪
Prerequisites:
Use a test account with Gecko enabled and set MANAGER_OAUTH in .env to a PAT belonging to the account
Reproduction steps
On develop:

Build & serve Cloud via yarn && yarn build && yarn start:manager:ci
Run yarn cy:debug
Search for the gdpr-agreement.spec.ts spec and run it
Observe that it fails when attempting to select a region

Verification steps
On this branch:

Build & serve Cloud via yarn && yarn build && yarn start:manager:ci
Run yarn cy:debug
Search for the gdpr-agreement.spec.ts spec and run it
Confirm that test passes
(Optional) Repeat steps with Gecko disabled, confirm tests still pass

ℹ Our CI will fail if there are any issues when Gecko is not enabled

As an Author I have considered 🤔
Check all that apply

[x] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[x] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

I couldn't get this to fail. I tried running this against develop following testing instructions and it passed every time 🤔

I got it to fail by manually adding
mockAppendFeatureFlags({
   gecko2: false,
});


Added explicite id where liquid var was included in title

Added explicit linking by ID for titles containing Liquid product name variable.

Fixed by #78 so this is not needed


upgrade references and framework to natively compile/use dotnet 6

Support for dotnet6 isolated.
Compile against recent (non-obsolete) dotnet5, dotnetcoreapp3.1
Upgrade to use more recent references.

@bobbyangers the library does support .NET 6/7. The targets using a lower version here only limit the set of APIs available to the SDK, but not the application.
We're also unable to drop support for TFM within a given major version of the library. Targets will be updated in the next major release of our packages.
If this is causing problems, can you share more information here?
We also have a change in PR to update all of our test and samples targets: https://github.com/Azure/azure-functions-dotnet-worker/pull/1203

@fabiocav although it does compile, it does not take advantage of the .net6 (or .net7) and linked to older and unsupported frameworks.
I do understand that you don't want to change within a minor release, maybe it's time to release a major version that leverage the framework/library of .NET 6 (LTS) and .NET 7;  and drop support for netcoreapp3.1 and net5.0 which are no longer supported.

We also have a change in PR to update all of our test and samples targets: #1203

I'll wait for this to push through and revisit my proposal.   Looking at that PR (#1203), mine new PR will be much smaller :-)

@fabiocav Please refer to this project as an example;
REF: Microsoft.Extensions.Http
They compile against:  net462, netstandard2.0, net6.0, net7.0;  not because they are incompatible, but because the different versions have different base.  Therefore it's better to multi-target and test against each.


Cannot access a disposed object. Object name: 'AndroidX.AppCompat.Widget.AppCompatImageView

Sometimes when choosing colors an error occurs: Cannot access a disposed object. Object name: 'AndroidX.AppCompat.Widget.AppCompatImageView.
Steps to reproduce the behavior
Steps to reproduce the behavior:

Open project editor.
Open any tile.
Select any color.
Repeat steps 1-3 many times.

The error is hard to reproduce. Try to complete the steps as quickly as possible.
Affected versions

v1.4 (20230608)

It looks like the problem is in the threads. But it is not yet clear how to reproduce the problem stably and, most importantly, how to fix it.

It looks like the problem is somewhere around toolkit:IconTintColorBehavior, but I can't find a way to further debug it :/

Testing in v1.4.

Tests on physical devices show that the problem is gone. Most likely IconTintColorBehavior is unstable, I just removed it.
I also noticed that in general the speed of the application has become better than it was with IconTintColorBehavior.
 We need to look for an alternative solution.
Fixed in v1.4 (20230608-1).


golang test failure on test/inputs/json/misc/734ad.json

$ FIXTURE=golang npm test -- test/inputs/json/misc/734ad.json

All regressions
    "734ad.json",
    "1b409.json",
    "0a91a.json",
    "combinations1.json",
    "combinations2.json",
    "combinations3.json",
    "combinations4.json",

null values are not present in output in go:


improvement(linter/unicorn): use root_node method in the no_process_exit rule.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#3091  👈
#3090
#3089
#3088
#3087
#3085
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rzvxa and the rest of your teammates on  Graphite

Merge activity

Apr 24, 8:43 PM EDT: @Dunqing started a stack merge that includes this pull request via Graphite.


Fix pre-pull images

What type of PR is this?
/kind bug
What this PR does / why we need it:
Because we tried to pre-pull images before upgrading kubeadm it failed
during the upgrade process for kubernetes v1.21 (prior v1beta3 kubeadm).
Does this PR introduce a user-facing change?:

Fix pre-pull images

/test pull-kubeone-e2e-aws-upgrade-1.21-1.22


train scripts with lora.

current problems in inferencing and saving.
optimizer adaw setted.
CrossAttention.to_out layer replaced with LoRADense layer.
Errors in inferencing and saving with lora.

duplicate with https://github.com/mindspore-lab/mindone/pull/22


[SPARK-46791][SQL] Support Java Set in JavaTypeInference

What changes were proposed in this pull request?

This patch adds the support of Java Set as bean field in JavaTypeInference.
Why are the changes needed?

Scala Set (scala.collection.Set) is supported in ScalaReflection so users can encode Scala Set in Dataset. But Java Set is not supported in bean encoder (i.e., JavaTypeInference). This feature inconsistency makes Java users cannot use Set like Scala users do.
Does this PR introduce any user-facing change?

Yes. Java Set is supported to be part of Java bean when encoding with bean encoder.
How was this patch tested?

Added tests.
Was this patch authored or co-authored using generative AI tooling?

No

The failures are irrelevant ones.
Merged to master for Apache Spark 4.0.0

Thank you, @viirya and @yaooqinn .

Thank you @dongjoon-hyun @yaooqinn


[DOCS] Add 8.0.0-rc2 release notes

Preview
https://elasticsearch-hadoop_1892.docs-preview.app.elstc.co/guide/en/elasticsearch/hadoop/master/eshadoop-8.0.0-rc2.html

@elasticmachine update branch

Do we usually put major deprecation information (#1845) into the release notes? Or is the breaking changes section enough?

Do we usually put major deprecation information (#1845) into the release notes? Or is the breaking changes section enough?

#1845 should be in the RNs too. I likely fumbled something in my original GitHub filter. Thanks for catching this.
Added with https://github.com/elastic/elasticsearch-hadoop/pull/1892/commits/8e4f1deb0e44eb693e568ad855a8e6d3237b68d5.
Here's a link to the GH filter I used for this round: https://github.com/elastic/elasticsearch-hadoop/pulls?q=is%3Apr+label%3Av8.0.0-rc2+is%3Aclosed+-label%3Anon-issue+-label%3Abackport+-label%3A>docs+


Feature Request: Add animation ease curves to key frame animations

Add the ability to specify the animation ease curve for a key frame animation. Currently it looks like the animations are hard-coded to ease-in / ease-out.

ease-curve.ts.txt

I think we could do this with a helper function to generate some keyframes between two points on a curve.

Currently keyframe animations are linearly interpolated, aren't they @eanders-MS ?

Yep, currently linear. But it would be straight forward to add an ease curve parameter to the keyframe.

That animation slows down at the top and bottom unless I'm missing something.

It does look like it's easing, yeah...

@stevenvergenz  what is the bug here?


If we improve /rotation/speculation to be a live table we can remove a lot of cruft

If we did this:
entry.mustache can lose the interestingness stuff
rotation.interesting can be removed
Some css to do with interestingness can be removed
We can get rid of the "key" stuff about interestingness
Could rename section.mustache to decksection or something more specific
Get rid of show_interesting

However there's no API endpoint that understands prices or most anything it needs to do.

It's probable that /rotation, /rotation/changes and /rotation/speculation are meant to be "one page" if we can come up with a good design for it.


[ADAP-767] [Bug] Model's contact validation fails on query data types change

Is this a new bug in dbt-bigquery?

[X] I believe this is a new bug in dbt-bigquery
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
Model data types are being detected incorrectly due to empty query caching in BigQuery, resulting in contract validation errors.
To get column data types, DBT executes a query like this:
select * from (
    -- model's body goes here
    select
        id,
        amount
    from project.dataset.payments
    --
) as __dbt_sbq
where false
limit 0

This query is cached in BigQuery, leading to incorrect data types detection in case of their change.
Expected Behavior
Model data types are detected correctly and contract validation works as expected.
Steps To Reproduce

Create a model that selects data from any table:

select
    customer_id,
    sum(amount) as total_amount
from {{ source('finance', 'payments') }}
where finished is true
group by customer_id

version: 2

models:
  - name: customer_payments
    config:
      contract:
        enforced: true
    columns:
      - name: customer_id
        data_type: string
      - name: total_amount
        data_type: integer

Run this model. Under the hood, DBT will execute the "empty" query mentioned earlier.
Change the customer_id data type in the payments source table.
Also, appropriately change the data type of customer_id for the model in the YAML config.
Run the model again. The dbt will not detect the customer_id data type change, and you will get the error:

This model has an enforced contract that failed.
Please ensure the name, data_type, and number of columns in your contract match the columns in your model's definition.
| column_name | definition_type | contract_type | mismatch_reason    |
| ----------- | --------------- | ------------- | ------------------ |
| customer_id | INT64           | STRING        | data type mismatch |

Relevant log output
No response
Environment
- OS: macOS 13.4.1
- Python: 3.10.2
- dbt-core: 1.5.2
- dbt-bigquery: 1.5.3

Additional Context
No response

Thanks for reporting this and opening up an associated PR @dementiev27 !
Maybe the BigQuery cache invalidation isn't working as we'd expect? It seems like BigQuery should invalidate its cache when an underlying table is rebuilt, but that doesn't appear to be happening.
I was able to reproduce the scenario you reported, and those details are in the "Reprex" toggle immediately below.

Reprex
Start with a model like this:
models/customer_payments.sql
{{ config(materialized="table") }}

select
    customer_id
from {{ ref('bogus_source_data') }}

Create the "source" data and the initial contracted model:
cat << EOF > models/bogus_source_data.sql
{{ config(materialized="table") }}

select
    cast(1 as integer) as customer_id
    -- '1' as customer_id
EOF
cat << EOF > models/_models.yaml
models:
  - name: customer_payments
    config:
      contract:
        enforced: true
    columns:
      - name: customer_id
        data_type: integer
        # data_type: string
EOF
dbt run -s bogus_source_data
dbt run -s customer_payments --full-refresh

Change the data type within the "source" data and the contracted model:
cat << EOF > models/bogus_source_data.sql
{{ config(materialized="table") }}

select
    -- cast(1 as integer) as customer_id
    '1' as customer_id
EOF
cat << EOF > models/_models.yaml
models:
  - name: customer_payments
    config:
      contract:
        enforced: true
    columns:
      - name: customer_id
        # data_type: integer
        data_type: string
EOF
dbt run -s bogus_source_data
dbt run -s customer_payments

Get an error like this:
22:59:24  Compilation Error in model customer_payments (models/customer_payments.sql)
22:59:24    This model has an enforced contract that failed.
22:59:24    Please ensure the name, data_type, and number of columns in your contract match the columns in your model's definition.
22:59:24
22:59:24    | column_name | definition_type | contract_type | mismatch_reason    |
22:59:24    | ----------- | --------------- | ------------- | ------------------ |
22:59:24    | customer_id | INT64           | STRING        | data type mismatch |

Updating models/customer_payments.sql like the following and re-running everything above seems work:
{{ config(materialized="table") }}

-- {{ modules.datetime.datetime.now() }}
select
    customer_id
from {{ ref('bogus_source_data') }}

Adding a SQL comment that generates a unique timestamp each time it is executed seems to successfully invalidate the BigQuery cache.
This feels kinda hacky, but I think we'd need to do something like this if BigQuery doesn't invalidate it's own cache when an underlying table is rebuilt.
One of the members of our engineering team will take a look at your PR -- thanks again for submitting it 🏆