ux: log when an api page is written

This PR adds a log message from the Builder, when a page is written. This means that when people run python -m quartodoc build --verbose, they get feedback on what pages are re-written.
This is important because for sites with hundreds of pages, re-writing pages means waiting for quarto to re-render them.

I see you locked a specific version of griffe in this PR. Can you merge that into main? When I build using main, I get errors due to the griffe version problem.


Upgrade banner should mention Zulip Standard sponsorship for non-business orgs

In organization settings, we show an upgrade banner next to settings that are not available on Zulip Cloud Free:

Non-business organizations are often eligible for sponsorship for Zulip Standard, so it would be very helpful if this banner mentioned sponsorship as an option for those categories of organizations.

Banner change: "Upgrade to access." ->  "Upgrade to access or request sponsorship."
Organization types: Any type other than "Business", including unspecified.

Related CZO thread

@zulipbot claim


Broken avatar and banner on profile

On Primal Android app v0.96.10, some avatar and banners are not showing up.
This issue is similar to the one reported on the web app repository but for Android: https://github.com/PrimalHQ/primal-web-app/issues/59
Thank you :)

The underlying library for image rendering is not working properly with your original profile url.
It's configured to follow redirects, but for some unknown reason it doesn't work.
This is currently not a high priority task, so it might take some time until this is fixed by Primal team.


feat: 🎸 Update dependencies

Description
Update dependencies to their latest compatible versions
Breaking Changes
NA
JIRA Link
DA-851
Checklist

[ ] Updated the Readme.md (if required) ?

@prashantasdeveloper could you check why Sonar says coverage is not enough? Maybe a test in conversion must be added.

I have added the missing line coverage but it seems the others are for V5 dual lines coverage

:tada: This PR is included in version 22.0.0-beta.4 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 22.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update README.md

Fixed a couple inconsistencies and a typo

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

Bump


the lang list is mismatch between view's definittion and codebehind's definition

view
Arabic,ChineseSimplified,ChineseTraditional ,Czech,Danish,Dutch,English,Finnish,French,German,Greek ,Hungarian,Italian,Japanese,Korean  ,Norwegian,Polish,Portuguese,Russian ,Spanish,Swedish ,Turkish
codebehind
"ara","chs","cht","cze","dan","dut","eng","fin","fre","ger","gre""hun","jap","kor","nor","pol","por","spa","swe","tur"
"ita","rus" would be lack.
and "jap" would be better "jpn"
So, if you run the sample with selecting "Italian" or later, will get un-expected result.
can repair oneself easy.

Thanks for this hint!


Write tests for the module

Jest + Svelte is absolutely possible, I should use that.
Also, I need some kind of dependency injection for API calls.

Besides Jest + Svelte for unit component tests I think I should use E2E framework to test the whole foundry integration
and have at least one green path test for every feature both in CSV and API mode.


Remove duplicate lint declaration, add vscode TS files to input.

Oops, sorry my mistake from https://github.com/google/wireit/pull/169


Ardb with LMDB crashing after startup

Hi,
We finished loading a big RDB file (37GB) to ardb with lmdb database. The loading part was OK but after ending ardb crashes. We start it up, it works for some minutes while continously increasing memory usage and almost no commands ran against it until it crashes again.
My guess is that it's running out of memory. There's nothing in the logs (DEBUG level).
We are running ardb master in a centos 6 ec2 amazon server.
We've seen that there are 3 configuration parameters but none seem to be related to max memory usage.
#lmdb's options
lmdb.database_max_size         200G
lmdb.readahead                 yes
lmdb.batch_commit_watermark    4096

it seems lmdb related bug since ardb do nothing after reopen lmdb.
And lmdb in ardb is upgraded to 0.9.18 which fixed many bugs since 0.9.10 may resolve this bug.


AutofacBootstrapper causes multiple Dispose of VMs and Self

I think these registrations are lacking ExternallyOwned specification
protected virtual void DefaultConfigureIoC(ContainerBuilder builder)
{
    var viewManagerConfig = new ViewManagerConfig()
    {
        ViewFactory = this.GetInstance,
        ViewAssemblies = new List<Assembly>() { this.GetType().Assembly }
    };
    builder.RegisterInstance<IViewManager>(new ViewManager(viewManagerConfig));

    builder.RegisterInstance<IWindowManagerConfig>(this); // <<< causes 'this' to Dispose twice
    builder.RegisterType<WindowManager>().As<IWindowManager>().SingleInstance();
    builder.RegisterType<EventAggregator>().As<IEventAggregator>().SingleInstance();
    builder.RegisterType<MessageBoxViewModel>().As<IMessageBoxViewModel>(); // <<< okay, but only as long as MessageBoxViewModel implements no IDisposable
    builder.RegisterAssemblyTypes(this.GetType().Assembly); // <<< both Stylet and Autofac will dispose every ViewModel implementing IDisposable
}

Aha thanks, I'm not too familiar with autofac.

Same as myself unfamiliar with other containers... And since nobody is able to know all of them anyway, wouldn't it be safer to just cover those cases by unit tests?

builder.RegisterAssemblyTypes(this.GetType().Assembly); // <<< both Stylet and Autofac will dispose every ViewModel implementing IDisposable

I don't think that's right - Stylet doesn't dispose transient instances.

@canton7 In our pseudo-MDI application we dynamically create VMs (resolve instances from IoC) and add them to Conductor<T>.Collection.OneActive. These VMs are presented as tabs on the UI. The VMs allocate some resources, so they implement IDisposable to perform the necessary cleanup. When the tab is closed, we call Conductor<T>.Collection.OneActive.CloseItem which in turn triggers VM.Dispose as part of StyletConductorExtensions.CloseAndCleanUp logic.
This behavior looks absolutely correct, BUT surprisingly we observe another call to VM.Dispose during app shutdown because Autofac (configured to own every resolved object implementing IDisposable because no ExternallyOwned is specified) is cleaning its items too.

Aha, gotcha. I think the changes I made just above fix this.


Error: android/amd64 requires external (cgo) linking, but cgo is not enabled

Hello,
I cannot release nicokosi/gh-collab-scanner, cf. this GitHub Actions' job:
Run cli/gh-extension-precompile@v1
  with:
    go_version: 1.22
Run actions/setup-go@v3
Setup go version spec 1.22
Found in cache @ /opt/hostedtoolcache/go/1.22.0/x64
Added go to the path
Successfully set up Go version 1.22
go version go1.22.0 linux/amd64

go env
Run if [ -n "$INPUT_TOKEN" ]; then
Run if [ -n "$INPUT_TAG" ]; then
Run if [ -n "$INPUT_TITLE" ]; then
Run ${GITHUB_ACTION_PATH//\\//}/build_and_release.sh
go: downloading github.com/cli/go-gh/v2 v2.5.0
✂...✂
go: downloading github.com/mattn/go-runewidth v0.0.14
go: downloading github.com/rivo/uniseg v0.4.4
android/amd64 requires external (cgo) linking, but cgo is not enabled
Error: Process completed with exit code 1.

The last release that succeeded, v1.5.3:

was using Go 1.20 instead of 1.22 ;
had module github.com/cli/go-gh which has not been updated to v2 yet ;
probably run on a previous version of the Ubuntu image.

I tried several changes: enable CGO (not sure it is correct), use a Windows runner etc. but I could not make the release succeed (see the GitHub Actions build history).
Do you have any advice/tip? Thank you!

No error after downgrading Go to 1.21, see gh-collab-scanner 1.5.4.
I think this issue can be kept open because it seems to be a work-around... 😅

Try with CGO_ENABLED=1

With CGO_ENABLED=1:
gcc_android.c:6:10: fatal error: android/log.h: No such file or directory
    6 | #include <android/log.h>
      |          ^~~~~~~~~~~~~~~
compilation terminated.

Go version: 1.22

Same error here. Since go v1.22 it requires cgo for at least those platforms:

android/386
android/amd64
android/arm/v7
ios/amd64
ios/arm64

@nicokosi @Biswa96 @baruchiro @the-hotmann : firstly, thank you all for raising this issue and starting exploring the nuances. ❤
I'm currently looking into this issue to see how this changing behavior with Go 1.22 should be taken into account here 🙇

Update
Doing a bit of digging, one option I need to carve out a little more time to try comes from https://github.com/golang/go/issues/42725#issuecomment-731438285, which highlights the need to override CC when compiling for android to point to Android NDK which is maintained on GitHub-hosted runners:

GOOS=android GOARCH=arm64 CGO_ENABLED=1 CC=$NDK_ROOT/21.3.6528147/toolchains/llvm/prebuilt/linux-x86_64/bin/aarch64-linux-android30-clang go build

This should live within $ANDROID_NDK_HOME on GitHub-managed runners

For what it's worth, I've been able to work around this problem by forking the action and removing Android build targets from build_and_release.sh:
platforms=(
  darwin-amd64
  darwin-arm64
  freebsd-386
  freebsd-amd64
  freebsd-arm64
  linux-386
  linux-amd64
  linux-arm
  linux-arm64
  windows-386
  windows-amd64
  windows-arm64
)

I'm uninformed regarding the rationale of maintaining Android as a build target for CLI extensions in the first place, but in our case, we decided that the accessibility of recent Go versions was more important than maintaining support for a build target that we'd expect to be used rarely (if ever) with by the end users of our particular extension.


Add support for scrolling a scroll view with a custom duration

Adds scrollWithCustomDurationTo() to ScrollView.js.
I found that this was necessary when moving a scroll view to accommodate a keyboard -- rather than using the default animation duration, I wanted to animate the scroll view in 250ms to match the duration of the keyboard animation.

I refactored scrollTo and also allowed for custom easing.

Maybe we could make scroll position a controlled value and do (sneak peak of the animation API i'm working on)
getInitialState: function() {
  return { scrollY: new Animated.Value(0) };
}
onWhatever: function() {
  this.state.scrollY.animateTiming({duration: 500, easing: Easing.quad});
}
render() {
  return <Animated.ScrollView scrollY={this.state.scrollY} />
}

Because Animated.Value is opaque, we don't even need to make it truly controlled. We can just serialize the animation and send it to native when it happens.
@ide what do you think?

@vjeux: what is the behavior of the scroll view after the animation is finished? Does it stay pinned to y=150, or does a finished animation stop controlling the scroll position?

It would not stay pinned. But you are right this is super weird
One other thing we can do is
scrollY(y, animation)
Where animation can be TimingAnimation, DecayAnimation, SpringAnimation (currently implemented but we can use any), pre-compute the key frames and send them to native to actually do the interpolation. Would that work?
This way, we don't have to implement yet another way to describe those easing functions

@vjeux does the TimingAnimation, DecayAnimation, etc. stuff already exist somewhere? I can't find it. Or is it in your private repo

Not yet, i'm iterating on it inside of facebook codebase, don't want to trash people while working on the api

@vjeux scrollTo(y, x, animation) would be a good stepping stone and preserve backward compatibility. The prop-driven API is interesting to me as an experiment though I'm not sure if it will work well in practice (I'm not against it - just don't know). Fine-grained control over the scroll position and content insets is so important for many apps.

other thoughts:
It's pretty interesting to have the scroll position not stay pinned. This notion of an "ephemeral value" could be used to replace imperative APIs and perhaps remove refs from React. To generalize, this code:
render() {
  return <X ref="x" />;
}
onEvent() {
  this.refs.x.performAction();
}

is now:
getInitialState() {
  return { performAction: new Animated.Boolean(false) };
}
render() {
  return <X performAction={this.state.performAction} />;
}
onEvent() {
  this.state.performAction.setEphemeralValue(true);  // true for one pass
}

This may also be the answer to making components like TextInput much more usable. Currently the asynchrony between JS and UIKit is janky and controlled TextInputs aren't usable in production. But I think there's a third type of TextInput -- semi-controlled -- where you generally don't need it to be controlled but occasionally want to set its value to a given string from JS, just like how the scroll view's position generally should not be controlled but sometimes we want to set the position from JS.

I agree that the prop driven API is interesting...but obviously in order to really make it viable, I think it would need to become idiomatic React...as right now, idiomatic React (at least, the public facing version) seems to be using refs to perform events on components.
I think @ide hits the nail on the head when he relates this API to making other components more usable and less imperative. I think there's a lot of power in that idea, and would be curious to see it explored further.
That being said...I think that keeping the scrollTo function and expanding on it is a good first step. I would love to add the animation stuff and agree it's better than passing in some weird options object...but I guess we'll just need to wait for what you come up with, @vjeux.
Should we keep this open just to remind us that it's here? Or should I close and revisit the refactor once the animation API you're talking about lands?

@ide your idea of semi-controlled components is fascinating! I have to sleep on it :) I'm sure @sahrens will enjoy it as well.
@skevy How urgent is it for you? I plan to have my api open sourced in the next few weeks. I'd love to revisit this once it is in a more advanced state. Would you mind keeping a fork for a while until things has settled?

It's urgent, but I don't mind keeping the fork.
Thanks for the feedback, and I look forward to seeing the new API!

The current TextInput is already basically semi-controlled because fully-controlled is janky due to async as you mention. If you don't update the value prop in js, the native value won't be overwritten, and if you continuously update the value, there is some buffering to try and reduce the jankiness by not overwriting the native value immediately.
Would be nice to have a better solution, perhaps a mechanism to provide a js code snippet to run synchronously on the main thread?

On May 19, 2015, at 8:15 PM, Adam Miskiewicz<EMAIL_ADDRESS>wrote:
It's urgent, but I don't mind keeping the fork.
Thanks for the feedback, and I look forward to seeing the new API!
—
Reply to this email directly or view it on GitHub.

Okay, I'll close the pull request for now and will open it back when Animated is released. Thanks for bringing this up!

@vjeux maybe it's time to open back this issue ?

perhaps a mechanism to provide a js code snippet to run synchronously on the main thread?

Is this something that could viably alleviate some performance issues? What difficulties are there in implementing such a thing?

Maybe we could make scroll position a controlled value

This idea is very interesting. Are there any inherent problems with making a scrollY an Animated.Value?

With the natively implemented Animated + gestures, this might actually make sense to do.
As for the original issue of specifying a custom duration, the scrollTo API now takes an object {x, y, animated} and could easily take


Not working on Raspberry Pi5

Hi,
for performance and RAM requirements of all my addons and integrations I decided to put my homeassistant instance on the new Pi5 with 8GB of RAM. How I still used this older integration to send an input to a remote, in order to open my gate, since that is shared and cannot be made smart in any other way.
Can someone figure out what needs to be changed on the new Pi5 hardware, in order to use a switch entity with this addon?
Here the errore I get on startup:
Error while setting up rpi_gpio platform for switch
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 356, in _async_setup_platform
await asyncio.shield(awaitable)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/rpi_gpio/switch.py", line 69, in setup_platform
RPiGPIOSwitch(
File "/config/custom_components/rpi_gpio/switch.py", line 100, in init
setup_output(self._port)
File "/config/custom_components/rpi_gpio/init.py", line 39, in setup_output
GPIO.setup(port, GPIO.OUT)
RuntimeError: Cannot determine SOC peripheral base address

Same here I control my relays using GPIO header and cant use RPI5 for that. Is there a way?

See https://github.com/thecode/ha-rpi_gpio/issues/225#issuecomment-1892114570
The current library RPi.GPIO did not add support for Raspberry PI 5 so there is nothing I can do. I don't have PI 5 and migrating to another library without the option to test my changes is not something I do. Someone try to replace to another library in the linked comment, I don't know what the status of it.

See #225 (comment) The current library RPi.GPIO did not add support for Raspberry PI 5 so there is nothing I can do. I don't have PI 5 and migrating to another library without the option to test my changes is not something I do. Someone try to replace to another library in the linked comment, I don't know what the status of it.

I found a drop-in replacement library.
https://rpi-lgpio.readthedocs.io/en/release-0.4/index.html
Can you maybe create a fork for pi5 and share the repository with me, so that i can test it?

I found a drop-in replacement library. https://rpi-lgpio.readthedocs.io/en/release-0.4/index.html Can you maybe create a fork for pi5 and share the repository with me, so that i can test it?

I don't understand what you ask, you can fork this repository from the GitHub interface, you don't need me to do it (and I can't)


Heroku Error R10 - Adonis v4.1

Error R10 (Boot timeout) -> Web process failed to bind to $PORT within 60 seconds of launch
I know this is duplicate but other solutions are not working. I am not using .env file. This is adonis api only. Heroku is not able to bind the port.

What is the value of PORT inside environment variables set with in heroku console?

Than you for the quick response. I figured it out. But to answer to your question, I have not set the PORT env variable at all. The problem was I wasn't ignoring .env file from .gitignore. I thought using ENV_SILENT is enough.
p.s. Awesome job with Adonis mate. Honestly never thought of Indian making an excellent Node Framework. single-handedly.

Closing since issue has been resolved 😄


Automatically escape quotes in the search query

I wanted to only take some articles, chosen by a label, as there are some that I prefer to read on my phone.
So I created a label and changed the search query: sort:saved-desc label:"Send to Kindle".
To escape it properly, I had to do double escaping - once for the JSON file, and once for the GQL request:
  "searchQuery": "sort:saved-desc label:\\\"Send to Kindle\\\"",

I think it would be nice if the script did the escaping-for-GQL on its own.

Hi Tamir,
Thanks for pointing out this shortcoming and creating a PR for it (#6).
I feel it might be a common enough requirement to warrant a dedicated config property. So I have created a PR for it (#7).
Please try it and let me know if it solves your requirement.

It doesn't solve the issue for me.
The label filtering now happens after fetching the articles.
This means that if I set my maxArticleCount to 10, and the first 10 articles don't have a matching label - I won't get any articles.
Additionally, it leads to extra API calls to fetch articles before checking their labels.
I think having a config option for labels is nice, but it should work towards the query, not post-filtering.
And since I can always create my "advanced" query string by doing the search on Omnivore, checking the results, and then copying the search query - I personally don't see the need for any support other than the query string itself at the moment.

Thanks for clearing it up. I keep my Omnivore library small by archiving read articles and don't use saved searches or query strings often. So I didn't think about this.
I will merge your version as its more versatile and add a note about escaping double quotes.


How to set the beta?

Hi, thanks for your nice work! However, I am confusing on how to set the beta

Hi @kojingying,
Your beta definition seems correct. You may want to ensure that the pooled x tensor will indeed be of oH,oW size.
For example, you can test if AdaPool2d works with beta=(1,1) (broadcasting the same value over the entire tensor as in issue #5 ).
Afterwards, if you would prefer to use beta as a matrix, you can find any difference between oH,oW and x.shape[2:].

self.pool = nn.AvgPool2d(ih // 32, 1)<——self.pool = AdaPool2d(kernel_size=ih // 32, stride=1, beta=(1, 1))
When I try to replace avgpool2d with adapool2d, the following error occurs:
“AssertionError: Only CUDA implementation supported!”
How can I use it correctly？

@Culturenotes it seems that either your input tensor or the beta is in the CPU. The current implementation only runs in CUDA, so you'll need to have all tensors used in AdaPool2d in CUDA.


enhancement: add xs size to status component

What does it do?
Adds a smaller XS variant to the Status component, in addition to the existing S and M versions
Why is it needed?
We need this size to apply the designs of the document status, for both history and preview features in the CMS
How to test it?
storybook link

Couldn't this also just be a styled Status component in the CMS content-manager for the two current uses cases? I guess it doesn't hurt to do it here though 🎈

It seems a bit overkill but the Figma status does have 3 size variants so I don't think it hurts to align the 2


Filtering with = operator on table clustered by property of object only returns subset of values inserted via UNNEST syntax

CrateDB version: 4.1.8
Environment description: docker 19.03.8 with crate:4.1.8
Problem description:
The documentation recommends to use the UNNEST syntax for insertion as described here.
When querying with a WHERE clause and the = operator on a table clustered by a property of an object for rows inserted with the UNNEST syntax only some of the rows in the table matching the comparison are returned, even though a full listing shows all objects have been inserted.
Querying using the LIKE operator returns all rows as expected. Same goes for the rows inserted with the value expression syntax. I created a minimal example that shows this behaviour:
Steps to reproduce:
Create table clustered on property of object:
CREATE TABLE IF NOT EXISTS crate.unnest_test(
    sample_object OBJECT(strict) as (
        some_id TEXT NOT NULL)
) CLUSTERED BY (sample_object['some_id']);

Insert rows with value expression:
INSERT INTO crate.unnest_test (sample_object)
    VALUES (
        { some_id='indeedworking'}
    );

Insert rows with UNNEST:
INSERT INTO crate.unnest_test (sample_object)
    (SELECT *
        FROM UNNEST(
            [ { some_id='notworking'} ]
        )
    );

After inserting several rows with above inserts, when I query:
-- next line returns all elements inserted via value expression as expected
SELECT *  FROM crate.unnest_test WHERE sample_object['some_id']='indeedworking';
-- next line returns only some elements inserted via unnest <-- UNEXPECTED
SELECT *  FROM crate.unnest_test WHERE sample_object['some_id']='notworking';
-- returns all elements inserted via unnest
SELECT *  FROM crate.unnest_test WHERE sample_object['some_id'] LIKE 'notworking';

Could this be related to https://github.com/crate/crate/issues/10270? I cant try this on crate 4.2.1 because of that issue...
I am trying to wrap my head around this for a while but cant see why this should not work. Would be thankful for any advice. Cheers!

@mtin We have just released CrateDB 4.2.2 testing version could you please try it out? The issue should be resolved there.

Can confirm this to be working with 4.2.2. Thanks!


Bump sw-precache version to v5

Hi @denar90,
Since sw-precache v5.0.0, an important change was introduced in response to a new browser restriction on redirected requests (see the v5.0.0 release notes). However, this Brunch plugin is currently locked at v4 (due to semver restrictions) in package.json.

Can we please bump the version number of sw-precache to the latest v5 release? I suppose since v5 is a major release, you would also need to push a new major release of this plugin to respect semver (so v2.0.0).
Thanks,
Caleb
P.S. I see a PR for this change has already been suggested in #10. Please consider merging it in.

Hi, thanks. I've also merged all stuff related to v2 in #9.

published 🚢
@caleb531 hope to get your feedback about how new version works for you :)

Yep, it works great! Though for what it's worth, I noticed that v2.0.0 is missing a GitHub release/tag. ;)
Anyway, thank you for the quick work, @denar90!

Thanks, tag has been created.


allow for csp nodes and graphs to be defined in repl

Fixes #177 by reassembling the source code from the interpreter history
Needs tests

I tried 3 different ways to add tests:

exec -> uses <string>, added a code path to raise a more meaningful error
python -c -> also uses <string>
printf | python -> could not exit cleanly and tough to wrap in a test

None worked nicely.

would using dill.source.getsource() simplify the changes needed on our end?
I'm hoping it can just be as simple as calling dill.source.getsource() if inspect.getsource fails

would using dill.source.getsource() simplify the changes needed on our end? I'm hoping it can just be as simple as calling dill.source.getsource() if inspect.getsource fails

i had hoped so too but there appears to be a bug in dill getsource (which i tried to repro and fix). I can try again to repro/fix it but:

dill is an external dependency and the getsource code does a lot more than this (though it also handles a lot more cases than this)
if we made the upstream fix we'd be bound to a relatively new version of dill

https://github.com/uqfoundation/dill/pull/651

My patch to dill has been merged upstream, we can either require users that want this functionality to have a very latest dill, or keep the code in our own repo as the PR does currently.
Since defining nodes and graphs in a repl is not super common or practical, I think either option is fine.

My patch to dill has been merged upstream, we can either require users that want this functionality to have a very latest dill, or keep the code in our own repo as the PR does currently.
Since defining nodes and graphs in a repl is not super common or practical, I think either option is fine.

I'm a proponent of getting this working one way or another, as I think it improves the user experience (and maybe also because I try to define nodes in the repl more than most users). I don't have a super strong view about how it's done (i.e. force a relatively new version of dill in the dependencies or include the code in the repo), but I am in favor of having it work "by default", i.e. without users needing to opt-in (i.e. by manually installing dill).

My patch to dill has been merged upstream, we can either require users that want this functionality to have a very latest dill, or keep the code in our own repo as the PR does currently.
Since defining nodes and graphs in a repl is not super common or practical, I think either option is fine.

I'm a proponent of getting this working one way or another, as I think it improves the user experience (and maybe also because I try to define nodes in the repl more than most users). I don't have a super strong view about how it's done (i.e. force a relatively new version of dill in the dependencies or include the code in the repo), but I am in favor of having it work "by default", i.e. without users needing to opt-in (i.e. by manually installing dill).
@robambalu Which way are you leaning?

I would go with dill so we dont have to maintain this extra code, as long as we can tell when we're in a case that dill is required and we can then do a version check / raise an exception if its not supported

My patch to dill has been merged upstream, we can either require users that want this functionality to have a very latest dill, or keep the code in our own repo as the PR does currently.
Since defining nodes and graphs in a repl is not super common or practical, I think either option is fine.

I'm a proponent of getting this working one way or another, as I think it improves the user experience (and maybe also because I try to define nodes in the repl more than most users). I don't have a super strong view about how it's done (i.e. force a relatively new version of dill in the dependencies or include the code in the repo), but I am in favor of having it work "by default", i.e. without users needing to opt-in (i.e. by manually installing dill).
@robambalu Which way are you leaning?

I would go with dill so we dont have to maintain this extra code, as long as we can tell when we're in a case that dill is required and we can then do a version check / raise an exception if its not supported

That's fine with me, as long as it's super clear to users what they need to do when they encounter the error. Maybe down the road we can include the new-enough dill as part of the requirements, but I agree it's probably too new to do that now.


distance unit is km not m

A client pointed out that the distance property seems to be in km.
Our reverse docs say "Each result returned has a distance from the query point (in meters)".

This was a good catch. I now wonder how consistent we are, both across our code and documentation, in using km vs meters. I would not be surprised to learn that at least some of the code uses meters as well.

I noticed the same. Example take from https://github.com/pelias/documentation/blob/master/reverse.md#example-requests
https://pelias.github.io/compare/#/v1/reverse%3Fpoint.lat=51.5081124&point.lon=-0.0759493
"properties": {
        "id": "node/4200057922",
        "gid": "openstreetmap:venue:node/4200057922",
        "layer": "venue",
        "source": "openstreetmap",
        "source_id": "node/4200057922",
        "name": "White Tower",
        "confidence": 0.9,
        "distance": 0.009,
        "accuracy": "point",
...
      }

It can't be 0.009 meters.

hi @acondrat, thanks for confirming, would you be interested in contributing a pull request which corrects the documentation?
https://github.com/pelias/documentation/blob/master/reverse.md#distance-and-confidence-scores-for-the-results

https://github.com/pelias/documentation/pull/269

resolved in https://github.com/pelias/documentation/pull/269


Bug Reporting Form

Add a form that enables users to automatically inform developers of bugs in the system.

Reopening this because I've realised that no information is stored in the report about which of our deployments the fault is from. Will fix today and then close.

The fault report now includes the deployment id from which the fault was reported. Incidentally if you want to report the deployment id at any point you can now using the "DEPLOYMENT" env variable in abacus, api and frontend.


Some model components Versions are the wrong part of the spec string

In the web interface of the build provenance DB, the model components of access-esm1p5 have an incorrect Version field of access-esm1.5.

This is due to https://github.com/ACCESS-NRI/experiment-metadb/blob/6c78365816419a5bda8e5f1fa116456e53764e49/frontend/src/app/release-provenance/page.js#L33-L34 taking split("=")[1] of spec. This is usually okay (see other models), but in access-esm1.5 we specifically set the right side of the equals to something that isn't the version.
Solution
Update the above linked split(...) function to instead take the first capture of the regex ^.*@(?:git.)?([^=]+)(?:=.*)?$ instead.
For a spec string of the form<EMAIL_ADDRESS>we essentially want the TAG:

^.*@ is the model name and start of version specifier (ex. mom5<EMAIL_ADDRESS>(?:git.)? is a noncapturing optional git. section
([^=]+) is the only captured group, anything before a = (ex. 2024.03.01, 13, development...)
(?:=.+)? is a noncapturing optional VERSION section (ex. =2024.03.02, =access-esm1.5...)

Do we need the spack parsing functionality in more than one place? If so I'm starting to think it would be good to have a little python library of functions to call for this stuff. It could be something that exposes existing functionality in spack, or if that isn't convenient, then roll something ourselves.


Update: Status

https://issues.redhat.com/browse/RHCLOUD-34198

[ ]  !important override being used to control margin. It is set to 16px but should be updated to 8px using: global/spacer/gap/text-to-element: 8px/spacer-200
[ ]  update icon and text to be top aligned when there is 2 or more lines of text

--pf-t--global--spacer--control--horizontal--compact
margin semantic spacer


[BUG] JDBC Driver doesn't return TIMESTAMP fractional seconds

What is the bug?
Timestamps with fractional seconds have the fractions stripped through the JDBC driver.
How can one reproduce the bug?
Steps to reproduce the behavior:

Load data into a timestamp index with fractional seconds
Connect using the JDBC driver
Issue a SELECT * query on the table
Notice no fractional seconds for any values

What is the expected behavior?
Fractional seconds match the inserted values
What is your host/environment?

OS: macOS
Version JDBC <IP_ADDRESS>
Plugins

Test data:
test_data.zip

The problem is here.

This is the definition of DATE_TIME_FORMATTER

The correct solution should use the formatter from specified in the mapping for the field or whatever is set as the default formatter for the cluster.

I believe this was actually a red herring - it appears to have invoked the v1 SQLEngine which doesn't have timestamp, and thus couldn't return the precision. It's now working correctly after some other changes.


AllOf should report which condition failed

Feature summary
See https://stackoverflow.com/questions/78745304/assertj-how-to-know-which-conditions-failed-when-using-allof/78747459#78747459
Example
Condition<String> condition1 = new Condition<String>(text -> text != null, "Input not null");
Condition<String> condition2 = new Condition<String>(text -> text.equalsIgnoreCase("abc"), "Input not matching");
assertThat(someString).satisfies(allOf(condition1, condition2));

It looks interesting ! Can i get this issue?

Sure thanks @JunHyungJang

Now we can check which condition failed inside allOf 👍


Could not find image named undefined.png

Getting many of these warnings. Images still show though. Don’t get this with Image.
Could not find image named undefined.png

@kirkstrobeck could you send me a snippet that would allow me to reproduce this bug?

I’m not sure that I can without hours of work. Feel free to close as it only produces a warning. Perhaps there can be a check that somewhere says thing must not be undefined. There is a guard around the image iterator that ensures there is a proper url on my end.

@kirkstrobeck Any change you could send me the props to CachedImage? What is undefined in the props you send to the component?

Here is an example
{
  hasImage1
  && <CachedImage
    blurRadius={30}
    source={{
      uri: photos[1],
    }}
    style={{
      bottom: 0,
      left: 0,
      position: 'absolute',
      right: 0,
      top: 0,
      zIndex: 9,
    }}
  />
}

@kirkstrobeck and photos[1] is undefined?

No, *.png isn’t specified in my code. The ajax data gets .jpg files. I have a couple local .png files that I was running thru CachedImage, but it wasn’t working so now I run them thru Image. Perhaps uninstalling the app would blow away the cache? but that image isn’t referenced any longer so not sure whats causing this? Looks like the pattern is that whenever I use CachedImage it warns about an error case from the past and doesn’t forget.

@kirkstrobeck I ran a couple of tests on IOS and couldn't reproduce the issue. Is it possible that you experienced the issue on android? Please feel free to re-open if this is the case of if you have more information on how to reproduce the issue.

My hunch was correct. Removing the app and reinstalling resolved the error as it wiped the cache.

I am also seeing this issue on iOS... just using
<CachedImage source={{uri: item.image_url}} style={{flex:1, resizeMode: 'stretch'}} />
the image_url has a png extension and guaranteed to exist.

@wcandillon further debugging - the warning is on only when you provide a style prop to it.

@dillonykyang @kirkstrobeck The issue should be fixed in 1.1.7. Please keep me posted and thank you so much for your help.

Upgraded, but it seems to still be present. 😭

Yeah I'm seeing this issue as well and I think it's because of this line: https://github.com/wcandillon/react-native-img-cache/blob/master/src/index.tsx#L101
I actually see render() getting called twice before the component finally has a value at this.state.path, so the built in Image component is throwing this warning when it can't load undefined.png.  I have to think it's just because RNFetchBlob.fs.exists() is a Promise and it's delaying anything until the next loop, which is going to miss that first render() call?
For what it's worth, I don't think the style prop has anything to do with this issue as mentioned above.  All that does is allow RN to determine it doesn't even need to attempt to load the image because it has no dimensions and won't be displayed anyways, so the warning is never thrown.
I'm not exactly sure what the right solution is here, but one idea is to allow for passing a placeholder image path that could be set as the initial value of this.state.path so that something is displayed while it pulls the image from cache or downloads the image.
For those who are affected by the warning, you can disable that specific yellow box warning with console.ignoredYellowBox = ['Could not find image'] somewhere near the entry point of your app.  This will prevent the yellow box from being displayed, but you'll still see the warnings in your debugging console.

@thorsonmscott Thank you so much for providing this information, I will look into it.

@thorsonmscott @kirkstrobeck Can you check version 1.1.8?

Good for a while, then many warnings like
Could not find image named undefined/Users/Foo/Library/Developer/CoreSimulator/Devices/AB229581-586B-40CF-864C-09A66A9A13BF/data/Containers/Data/Application/8C807149-EE98-413A-9DD3-1D5EE79FF3D5/Library/Caches/b253a77b492fc10de6c4c2075c5c17bc2ddee4cd.jpg

@kirkstrobeck should be fixed in 1.1.9, sorry about this regression!

No worries, that’s what software does. 1.1.9 appears to be fixed. 🎉


Issue #62: make AST nodes always Node subclasses

Description

Fixes for #62 and related issues:

ensure that the following Nodes have children which are always a subclass of Node:

IndexedId (as noted in https://github.com/QISKit/qiskit-sdk-py/issues/62#issuecomment-328586459)
BinOp and Prefix (as noted in https://github.com/QISKit/qiskit-sdk-py/issues/62#issue-256569802). This involved introducing new nodes (BinaryOperator and UnaryOperator) to represent the children nodes.
Magic (discovered during testing).

additionally fix the ^ operator, as it seems it produced a syntax error on the current version of the lexer as it was not considered a valid literal.

The operators are still considered plain literals by the lexer - I'm not sure if it would be arguably cleaner to consider them proper tokens, and in the end I went with the option that seemed to fit into the current structure of the parser.
I would welcome a thorough review by the developers more involved in the parser, as while the tests are passing and I have introduced a new one for checking that the qasm/example.qasm file produces only Node subclasses, I'm not fully aware of potential side effects.
Motivation and Context

Fix #62.
How Has This Been Tested?

make test, and also make test on the OpenQASM repository.
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

I have a qasm file which seems to parse with a non-node child,
OPENQASM 2.0; include "qelib1.inc"; qreg qr0[1]; creg cr0[1]; h qr0[0]; measure qr0[0] -> cr0[0]; if(cr0==0) x qr0[0]; h qr0[0]; reset qr0[0]; measure qr0[0] -> cr0[0]; h qr0[0]; reset qr0[0]; u1(0.562367489586523) qr0[0]; measure qr0[0] -> cr0[0]; reset qr0[0]; reset qr0[0]; reset qr0[0]; h qr0[0]; measure qr0[0] -> cr0[0]; if(cr0==1) x qr0[0]; h qr0[0]; u1(0.083178154631269) qr0[0]; h qr0[0]; reset qr0[0]; measure qr0[0] -> cr0[0];

Thanks, Erick! The if Nodes contained an integer instead of an Int Node, it should be addressed by 967983d, which also includes adding your sample qasm file to the TestParser.test_all_valid_nodes().


Expression evaluation errors

Hello and thanks for the useful library, but i've got an issue with it.
I have a quite simple expression:
([X] >=<PHONE_NUMBER> && [X] <=<PHONE_NUMBER>) || ([X] >= 168100864 && [X] <= 168118271)
If i pass X =<PHONE_NUMBER>, which satisfies first expression in parenthesis, the expression is for some reason evaluated to FALSE.
If i PASS X = 168100865, which satisfies second part of an expression in parenthesis, i get TRUE.
If i swap the sub-expressions, then i get the inverted behaviour.
If i use smaller numbers, like:
([X] >= 0 && [X] <= 10) || ([X] >= 20 && [X] <= 30)
then the expression is evaluated right in all cases.
I think the problem is with numbers being converted to float64, however looking at tokens after expression compilation the numbers seem fine:
{Kind:13 Value:40} {Kind:7 Value:X} {Kind:10 Value:>=} {Kind:2 Value:2.887057408e+09} {Kind:11 Value:&&} {Kind:7 Value:X} {Kind:10 Value:<=} {Kind:2 Value:2.887122943e+09} {Kind:14 Value:41} {Kind:11 Value:||} {Kind:13 Value:40} {Kind:7 Value:X} {Kind:10 Value:>=} {Kind:2 Value:1.68100864e+08} {Kind:11 Value:&&} {Kind:7 Value:X} {Kind:10 Value:<=} {Kind:2 Value:1.68118271e+08} {Kind:14 Value:41}
And this does not explain why the behaviour changes if i just reverse the parts before and after || operator...

My fault, one correction: when i swap the sub-expressions to get (in the very simple case):
([X] >= 10 && [X] <= 20) || ([X] >= 30 && [X] <= 40)
Then the whole expression evaluates correctly when using X from both ranges (15 and 35 for example).
So it seems that if expressions goes from smaller to bigger it works fine.

Further research... As i have no time now to dig into this bug myself, i tried to walk around it like this:
    functions := map[string]govaluate.ExpressionFunction{
	"InRange": func(args ...interface{}) (interface{}, error) {
	    return (args[0].(float64) >= args[1].(float64)) && (args[0].(float64) <= args[2].(float64)), nil
	},
    }

    x, _ := govaluate.NewEvaluableExpressionWithFunctions("InRange([X], 30, 40) || InRange([X], 10, 20)", functions)
    t, _ := x.Evaluate(p);
    fmt.Printf("%+v\n", t)

This way it works fine.
But when i add another condition to the expression, like this:
[X] == 1 || InRange([X], 30, 40) || InRange([X], 10, 20)
Then I get panic:
panic: interface conversion: interface is []interface {}, not float64

goroutine 1 [running]:
panic(0x4dbd20, 0xc42004c3c0)
	/opt/go/src/runtime/panic.go:500 +0x1a1
main.main.func1(0xc42000e5c0, 0x2, 0x2, 0xc42009bb10, 0xc42000a528, 0x469609, 0x4d1620)
	/usr/src/go-test/main.go:22 +0x176
mt/govaluate.makeFunctionStage.func1(0x0, 0x0, 0x4d0160, 0xc42000e5e0, 0x5622e0, 0xc42000a510, 0x0, 0x0, 0x0, 0x0)
	/root/go/src/mt/govaluate/evaluationStage.go:215 +0x179
mt/govaluate.evaluateStage(0xc420014320, 0x5622e0, 0xc42000a510, 0x4d1620, 0xc42000a520, 0x0, 0x0)
	/root/go/src/mt/govaluate/EvaluableExpression.go:181 +0xf5
mt/govaluate.evaluateStage(0xc420014550, 0x5622e0, 0xc42000a510, 0x40e648, 0x8, 0x10, 0x10)
	/root/go/src/mt/govaluate/EvaluableExpression.go:154 +0x404
mt/govaluate.evaluateStage(0xc4200145a0, 0x5622e0, 0xc42000a510, 0xc42009bde8, 0x46fffe, 0xc4200145a0, 0xc4200145a0)
	/root/go/src/mt/govaluate/EvaluableExpression.go:147 +0x48f
mt/govaluate.EvaluableExpression.Eval(0x4fb03e, 0x22, 0xc420092000, 0x15, 0x20, 0xc4200145a0, 0x4fce69, 0x38, 0x562460, 0xc420016480, ...)
	/root/go/src/mt/govaluate/EvaluableExpression.go:138 +0x5c
mt/govaluate.EvaluableExpression.Evaluate(0x4fb03e, 0x22, 0xc420092000, 0x15, 0x20, 0xc4200145a0, 0x4fce69, 0x38, 0xc420016480, 0xc42003bf38, ...)
	/root/go/src/mt/govaluate/EvaluableExpression.go:113 +0x62
main.main()
	/usr/src/go-test/main.go:40 +0x1d6
exit status 2

The value that is passed to the function is for some reason [9.0 [30.0 40.0]], which is not of course float64, but [][]float64... Looks like another bug?

Looks like this is due to some post-processing that happens when the expression builds the evaluation stage tree. There is logic built as part of #21 which rearranges the tree to make sure that identical operators of the same precedence work as written - but it looks like it's messing up the order of comparators. The first case evaluates the very last compare ([X] <= 168118271) as the root of the tree, so it ends up ignoring all  the rest.
I was able to put together a quick fix by disabling the rearrangement logic for logical operators and comparators, but there's still a larger problem with the stage rearrangement logic.

Can i peek at the fix? :) By the way, i already tried some time ago to disable reorderStages(), it just reverses the behavior - the first comparison works, while the second does not.

Just now pushed 84358896c8e45bda07e23537a50ea59886dd5e5e to master, since the tests are good to have and the fix works for these specific cases. I'm positive that it doesn't completely correct the root problem though.
It seems to be a problem with logical operators, specifically. A case like (true && true) || (true && false) also exhibits the same issue.

Thank you very much, it works like a charm now!
By the way it also fixed the above issue with a function being passed a [][]float64 slice.

After scrutinizing it more, it looks like it really was just a problem with stage reordering and the logical operators. I can't find any other cases that exhibit the behavior, and those are the only operators which should never be reordered, so it looks like there's no followup.
Thanks for using the library, and for the detailed report!


[Application] App control support

Appcontrol part of application plugin were not implemented.
Introducing following function of application API:

Application.getRequestedAppControl()
RequestedApplicationControl.replyResult()
RequestedApplicationControl.replyFailure()
ApplicationManager.launchAppControl()
ApplicationManager.findAppControl()

BUG=XWALK-991
Requires:

https://review.tizen.org/gerrit/#/c/35215/
https://review.tizen.org/gerrit/#/c/35689/
https://review.tizen.org/gerrit/#/c/35700/

@halton

And I bet you need update the test case as well.

https://crosswalk-project.org/jira/browse/XWALK-1490?focusedCommentId=28601&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-28601

@tiwanek there is example to application api under examples/application.html.

About examples. I'm not sure what I should do about it.
Each function, I have added, bases its behaviour on other apps being installed on tizen (it searches for them or launch them).
If you have some suggestions, please, let me know. I will try to fix it on next PR.

Initially, API owner write test case (example/api.html) along with  the implementation. There is tizen-extensions-crosswalk-examples.rpm will be built out if you run gbs build. We should keep the implementation and example aligned all the time.

After short look on examples. I will add "handleFindAppControl" and "handleLaunchAppControl" in separate PR.

Fine with separate PR for the test case. LGTM and this PR will merging soon.


[Bug] Start-DbaAgentJob verbose output "Cannot convert value" error for job step

Report

Host used

[ ] powershell.exe
[ ] ISE
[ ] VS Code
[X] Other (please specify) pwsh.exe

Errors Received

InvalidArgument: C:\Users\myusername\Documents\PowerShell\Modules\dbatools\1.0.153\allcommands.ps1:71864
 Line |
71864 |              [int]$currentStepId = $currentstep.Split('()')[0].Trim()
      |              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      | Cannot convert value "7 (My Job Step Name)" to type "System.Int32". Error: "Input string was
      | not in a correct format."

PSMessageDetails      :
Exception             : System.Management.Automation.RuntimeException: Cannot convert value "7 (My Job Step Name
                        )" to type "System.Int32". Error: "Input string was not in a correct format."
                         ---> System.Management.Automation.PSInvalidCastException: Cannot convert value "7 (My Job
                        Step Name)" to type "System.Int32". Error: "Input string was not in a correct format."
                         ---> System.FormatException: Input string was not in a correct format.
                           at System.Number.ThrowOverflowOrFormatException(ParsingStatus status, TypeCode type)
                           at System.Number.ParseInt32(ReadOnlySpan`1 value, NumberStyles styles, NumberFormatInfo
                        info)
                           at System.ComponentModel.Int32Converter.FromString(String value, NumberFormatInfo
                        formatInfo)
                           at System.ComponentModel.BaseNumberConverter.ConvertFrom(ITypeDescriptorContext context,
                        CultureInfo culture, Object value)
                           --- End of inner exception stack trace ---
                           at System.Management.Automation.LanguagePrimitives.ConvertStringToInteger(Object
                        valueToConvert, Type resultType, Boolean recursion, PSObject originalValueToConvert,
                        IFormatProvider formatProvider, TypeTable backupTable)
                           at CallSite.Target(Closure , CallSite , Object )
                           at <ScriptBlock>(Closure , FunctionContext )
                           --- End of inner exception stack trace ---
TargetObject          :
CategoryInfo          : InvalidArgument: (:) [], RuntimeException
FullyQualifiedErrorId : InvalidCastFromStringToInteger
ErrorDetails          :
InvocationInfo        : System.Management.Automation.InvocationInfo
ScriptStackTrace      : at <ScriptBlock>, C:\Users\myusername\Documents\PowerShell\Modules\dbatools\1.0.153\allcommands.ps1: line 71864
                        at Start-DbaAgentJob<Process>, C:\Users\myusername\Documents\PowerShell\Modules\dbatools\1.0.153\allcommands.ps1: line 71959
                        at <ScriptBlock>, C:\Users\myusername\Documents\PowerShell\Data
                        Refresh Delay reload data Prod.ps1: line 90
                        at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo : {}

Steps to Reproduce

Start-DbaAgentJob -SqlInstance $server -SqlCredential $creds -Job $jobname -Wait -WaitPeriod 60 -Verbose
Expected Behavior
Display proper job step.
Actual Behavior
Error output, and blank or empty variable outputs no job step name.
VERBOSE: [12:28:50][Start-DbaAgentJob] Server: MyServer - My Agent Job is Executing, currently on Job Step 'My Job Step Name' (7 of 40), and has tried 0 of 0 retry attempts

see /functions/Start-DbaAgentJob.ps1 line 140:
            [int]$currentStepId = $currentstep.Split('()')[0].Trim()

Environmental information
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨
Please provide the output of the below script
[pscustomobject]@{
'PowerShell Version' = $PSVersionTable.PSVersion.ToString()
'dbatools latest installed' = (Get-InstalledModule -Name dbatools).Version
'Culture of OS' = (Get-Culture)
} | fl -force

PowerShell Version        : 7.1.3
dbatools latest installed : 1.0.153
Culture of OS             : en-US
SQL Server:
Microsoft SQL Server 2017 (RTM-CU22-GDR) (KB4583457) - 14.0.3370.1 (X64)
	Nov  6 2020 18:19:52
	Copyright (C) 2017 Microsoft Corporation
	Standard Edition (64-bit) on Windows Server 2016 Standard 10.0 <X64> (Build 14393: ) (Hypervisor)

and

Microsoft SQL Server 2019 (RTM-CU8-GDR) (KB4583459) - 15.0.4083.2 (X64)
	Nov  2 2020 18:35:09
	Copyright (C) 2019 Microsoft Corporation
	Standard Edition (64-bit) on Windows Server 2019 Standard 10.0 <X64> (Build 17763: ) (Hypervisor)

us_english

ouch. there's a nicer way but the interface is wonky . will work on that if you can wait a few days.

@wsuhoey : would you give this a whirl ? https://github.com/sqlcollaborative/dbatools/pull/7445

I realize I should know this, but how do I test the updated code? Do I edit the code in Start-DbaAgentJob.ps1 or do I find the section in allcommands.ps1? I don't normally do this :)

Confirmed fixed in 1.0.158


Cleanup dotenv files

📖 Description
While merging back changes from the boilerplate to existing codebases, I realized a few keys were left behind following recent enhancements.
I made a little cleanup and ordered them correctly!
🦀 Dispatch

#dispatch/elixir

🦀 Requesting reviewers for this pull request:

@remi (contributor with 3 commits in the last 90 days and 180 commits overall)
@simonprev (expert reviewer for the elixir stack)


[.vscode-server] Installing an extension creates EACCES error.

Version
Microsoft Windows [Version 10.0.19044.2604]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
Linux version 4.4.0-19041-Microsoft<EMAIL_ADDRESS>(gcc version 5.4.0 (GCC))
Distro Version
Ubuntu 22.04
Other Software
Visual Studio Code (1.75.1) with WSL extension v0.75.3
Repro Steps
Not sure.
Simply install Ubuntu 22.04 as the distro, install the WSL extension, and tried to install another extension.
Expected Behavior
N/A. Simple download.
Actual Behavior
EACCES: EACCES: permission denied, rename '/root/.vscode-server/extensions/.dbcca4b5-7c2b-4473-ba38-6bad1c859460' -> '/root/.vscode-server/extensions/botloader.botloader-vscode-0.11.0'
Diagnostic Logs
2023-02-28 18:15:04.072 [info] [remote-connection][ExtensionHost][b6ad5…] high roundtrip time: 1013ms (1 of 5 recent samples)
2023-02-28 18:18:27.041 [info] Electron sandbox mode is enabled!
2023-02-28 18:18:27.188 [warning] Via 'product.json#extensionEnabledApiProposals' extension 'github.vscode-pull-request-github' wants API proposal 'commentsResolvedState' but that proposal DOES NOT EXIST. Likely, the proposal has been finalized (check 'vscode.d.ts') or was abandoned.
2023-02-28 18:18:28.471 [info] Invoking resolveAuthority(wsl)...
2023-02-28 18:18:28.474 [info] [LocalProcess0][resolveAuthority(wsl,1)][0ms] obtaining proxy...
2023-02-28 18:18:32.240 [info] [LocalProcess0][resolveAuthority(wsl,1)][3766ms] invoking...
2023-02-28 18:18:33.897 [info] [LocalProcess0][resolveAuthority(wsl,1)][5423ms] waiting...
2023-02-28 18:18:34.985 [info] [LocalProcess0][resolveAuthority(wsl,1)][6510ms] waiting...
2023-02-28 18:18:36.513 [info] [LocalProcess0][resolveAuthority(wsl,1)][8040ms] waiting...
2023-02-28 18:18:37.246 [info] [LocalProcess0][resolveAuthority(wsl,1)][8772ms] waiting...
2023-02-28 18:18:38.248 [info] [LocalProcess0][resolveAuthority(wsl,1)][9775ms] waiting...
2023-02-28 18:18:39.244 [info] [LocalProcess0][resolveAuthority(wsl,1)][10771ms] waiting...
2023-02-28 18:18:39.603 [info] [LocalProcess0][resolveAuthority(wsl,1)][11130ms] returned <IP_ADDRESS>:57704
2023-02-28 18:18:39.605 [info] resolveAuthority(wsl) returned '<IP_ADDRESS>:57704' after 11133 ms
2023-02-28 18:18:39.926 [info] Creating a socket (renderer-Management-e7c99c9e-2df8-439d-a1ab-8a15f65c3590)...
2023-02-28 18:18:39.940 [info] Creating a socket (renderer-ExtensionHost-0bf9cbba-c04f-4ae6-82d7-bef273078f57)...
2023-02-28 18:18:41.113 [info] Creating a socket (renderer-Management-e7c99c9e-2df8-439d-a1ab-8a15f65c3590) was successful after 1187 ms.
2023-02-28 18:18:41.562 [info] Creating a socket (renderer-ExtensionHost-0bf9cbba-c04f-4ae6-82d7-bef273078f57) was successful after 1622 ms.
2023-02-28 18:18:46.146 [warning] Webgl could not be loaded. Falling back to the canvas renderer type. {}
2023-02-28 18:18:47.843 [error] Cannot read properties of undefined (reading 'setAtlas'): TypeError: Cannot read properties of undefined (reading 'setAtlas')
    at p._refreshCharAtlas (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/node_modules.asar/xterm-addon-webgl/lib/xterm-addon-webgl.js:1:20362)
    at p._handleOptionsChanged (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/node_modules.asar/xterm-addon-webgl/lib/xterm-addon-webgl.js:1:19062)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/node_modules.asar/xterm-addon-webgl/lib/xterm-addon-webgl.js:1:15765
    at t.EventEmitter.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/node_modules.asar/xterm/lib/xterm.js:1:139525)
    at a.i (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/node_modules.asar/xterm/lib/xterm.js:1:266845)
    at e.Terminal.s (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/node_modules.asar/xterm/lib/xterm.js:1:273006)
    at new h (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1889:9476)
    at o.j (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:611:1296)
    at o.createInstance (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:611:788)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2542:38375
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1882:21348
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at L.handleReady (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1092:20599)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1697:8388
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at Z (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:793)
    at A.t (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:1577)
    at A.s (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:1471)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:552:1351
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at r.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:15219)
    at E (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:19285)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:17273
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at d.acceptChunk (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:12950)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:12237
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:642:38242
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at i.onload (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:642:36509)
2023-02-28 18:18:48.187 [info] [perf] Render performance baseline is 662ms
2023-02-28 18:18:56.973 [error] Cannot read properties of undefined (reading 'setAtlas'): TypeError: Cannot read properties of undefined (reading 'setAtlas')
    at p._refreshCharAtlas (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/node_modules.asar/xterm-addon-webgl/lib/xterm-addon-webgl.js:1:20362)
    at p._handleOptionsChanged (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/node_modules.asar/xterm-addon-webgl/lib/xterm-addon-webgl.js:1:19062)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/node_modules.asar/xterm-addon-webgl/lib/xterm-addon-webgl.js:1:15765
    at t.EventEmitter.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/node_modules.asar/xterm/lib/xterm.js:1:139525)
    at a.i (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/node_modules.asar/xterm/lib/xterm.js:1:266845)
    at e.Terminal.s (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/node_modules.asar/xterm/lib/xterm.js:1:273006)
    at new h (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1889:9476)
    at o.j (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:611:1296)
    at o.createInstance (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:611:788)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2542:38375
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1882:21348
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at L.handleReady (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1092:20599)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1697:8388
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at Z (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:793)
    at A.t (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:1577)
    at A.s (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:1471)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:552:1351
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at r.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:15219)
    at E (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:19285)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:17273
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at d.acceptChunk (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:12950)
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:553:12237
    at vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:642:38242
    at g.invoke (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:145)
    at n.deliver (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:2029)
    at a.fire (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:86:1667)
    at i.onload (vscode-file://vscode-app/c:/Users/hp/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:642:36509)
2023-02-28 18:19:20.494 [error] EACCES: EACCES: permission denied, rename '/root/.vscode-server/extensions/.dbcca4b5-7c2b-4473-ba38-6bad1c859460' -> '/root/.vscode-server/extensions/botloader.botloader-vscode-0.11.0'
    at Y.t (/root/.vscode-server/bin/441438abd1ac652551dbe4d408dfcec8a499b8bf/out/vs/server/node/server.main.js:149:86362)
    at async Y.extractUserExtension (/root/.vscode-server/bin/441438abd1ac652551dbe4d408dfcec8a499b8bf/out/vs/server/node/server.main.js:149:83665)
    at async U.s (/root/.vscode-server/bin/441438abd1ac652551dbe4d408dfcec8a499b8bf/out/vs/server/node/server.main.js:149:89446)
    at async U.h (/root/.vscode-server/bin/441438abd1ac652551dbe4d408dfcec8a499b8bf/out/vs/server/node/server.main.js:149:90672)
2023-02-28 18:19:20.504 [error] EACCES: permission denied, rename '/root/.vscode-server/extensions/.dbcca4b5-7c2b-4473-ba38-6bad1c859460' -> '/root/.vscode-server/extensions/botloader.botloader-vscode-0.11.0': EACCES: EACCES: permission denied, rename '/root/.vscode-server/extensions/.dbcca4b5-7c2b-4473-ba38-6bad1c859460' -> '/root/.vscode-server/extensions/botloader.botloader-vscode-0.11.0'
    at Y.t (/root/.vscode-server/bin/441438abd1ac652551dbe4d408dfcec8a499b8bf/out/vs/server/node/server.main.js:149:86362)
    at async Y.extractUserExtension (/root/.vscode-server/bin/441438abd1ac652551dbe4d408dfcec8a499b8bf/out/vs/server/node/server.main.js:149:83665)
    at async U.s (/root/.vscode-server/bin/441438abd1ac652551dbe4d408dfcec8a499b8bf/out/vs/server/node/server.main.js:149:89446)
    at async U.h (/root/.vscode-server/bin/441438abd1ac652551dbe4d408dfcec8a499b8bf/out/vs/server/node/server.main.js:149:90672)
2023-02-28 18:22:43.026 [info] [remote-connection][Management   ][e7c99…] high roundtrip time: 1011ms (1 of 5 recent samples)
2023-02-28 18:22:43.027 [info] [remote-connection][ExtensionHost][0bf9c…] high roundtrip time: 1096ms (1 of 5 recent samples)
2023-02-28 18:23:43.058 [info] [remote-connection][Management   ][e7c99…] high roundtrip time: 1120ms (2 of 5 recent samples)
2023-02-28 18:23:43.060 [info] [remote-connection][ExtensionHost][0bf9c…] high roundtrip time: 1122ms (2 of 5 recent samples)
2023-02-28 18:30:57.973 [info] [remote-connection][ExtensionHost][0bf9c…] high roundtrip time: 15023ms (3 of 5 recent samples)
2023-02-28 18:30:57.979 [info] [remote-connection][Management   ][e7c99…] high roundtrip time: 15030ms (3 of 5 recent samples)

The vscode remote guys should probably have a first look at this.
https://github.com/microsoft/vscode-remote-release


Rename components/models/services/etc from navi-{name} to yavin/{name}

[ ] Core @cythrawll
[ ] Data @willmsn98
[ ] Directory @kevinhinterlong
[ ] Reports @jkusa
[ ] Dashboards @holderb5 ??
[ ] Search
[ ] WS Models
[ ] Admin

Is this issue still open for contributions? If yes, then I would like to contribute. But please also add some additional info about what exactly is to be done and how.

@jkusa Hi, I am trying to setup the project on my machine but having never used npm before, I am unable to set it up. So could you please help me set it up or atleast provide me with some links to tutorials related to it? Also sorry for the late response.


RAPL Devices Unknown Devices - Please report on Github Intel i9-10980XE

Brief description of your issue
Under RAPL Devices, direction to report the device on github.
Steps to reproduce (optional)
Options-> RAPL Devices
Expected behavior (optional)
No response
Actual behavior (optional)
No response
Environment (optional)
Nightly 3.0.7310 https://github.com/winsiderss/systeminformer/commit/bc98048
Windows 22621.2506

Getting same with all 8 Cores of my
AMD Ryzen 7 7800X3D 8-Core Processor
Only other difference to above report is OS
Windows 11 Pro v23H2 Build 22631.2428

Duplicate of #1558


174236904 header ui

Update the CLUE app header to match the new UI specs:

update header bg color
update member area colors and styling
add (currently disabled) problem select
update unit/investigation information in upper left and revise styling
add updated content divider
update student name area styling

Prototype: https://om2cbt.axshare.com
Specs - web: https://zpl.io/boLO9EX

@kswenson I made changes to address code comments.  It's probably worth waiting on the ProblemSelect issues until that component is no longer in the disabled state and is actually used as a select.


Add Toolbar component

This PR adds an accessible Toolbar component and update website to use it.
Preview: https://deploy-preview-174--reakit.netlify.com/components/toolbar

Codecov Report

Merging #174 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #174   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          23     23
  Lines         258    258
  Branches       63     63
=====================================
  Hits          258    258

Impacted Files
Coverage Δ

src/enhancers/as.js
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Connect with KeyValue files and password

Hi,
First of all; thanks for FluentFTP.
I need to connect to a FTPS server using a KeyValue pair file (.ppk), username and password for the file. Didn't see an example on the https://github.com/robinrodricks/FluentFTP/wiki/FTPS-Connection page.
Is it possible with FluentFTP?
Here is an example of FileZilla configuration:

Thanks,
Matthijs

I'm not sure how this is implemented. If you can share more technical details perhaps we can support it.

I have no clue ;-) Will look into this.

found something here that seems to work: https://stackoverflow.com/a/59783368/369122

We don't support SFTP which is a completely different protocol. You'll need SSH.NET for that. I think keyfiles are used for SFTP and not FTPS.


Have a section in docs referencing video tutorials/presentations

The idea here is to have a section in docs, which will be a hub for video content on ASP.NET Core related topics.
There should be a way to identity/separate Microsoft owned as well as community-owned videos.
To clarify, this is not about hosting the videos in the docs page, rather just being a catalog of such videos.
Why is this important?
People prefer to learn differently. Some prefer written content like books, articles and others prefer video tutorials. Some people, however, who are in the first group, still prefer to have a way to watch some content on-demand.

This was admonished in the past due to the stability of video content and our lack of control over it ...

It's a stable medium, which is rarely updated, and thus becomes stale rather quickly.

It's also difficult to know when a video has become stale because we have no way to track video content and keep tabs on it relative to the framework.

Even if we knew that a part (or several parts) of a video has become stale, is the doc also supposed to call that out a list of video sections that the viewer should refrain from consuming?

There was a discussion in this vein years ago (👴 Yikes! :smile:) about blogs. DR said ...

Our official docs need to be authoritative, updated and persistent, while blog content is more point in time.

https://github.com/dotnet/AspNetCore.Docs/issues/125#issuecomment-109008003
The same can probably be said for video content, at least for video content that the docs team itself doesn't produce and maintain.

Thanks for the feedback, @guardrex.
Content becoming stale is manageable. We can version these materials on our own. So that some videos will show up through v2.1 docs, some through v3.2 etc.
As for the video becoming unavailable, somebody will notice that and file an issue, and we can be reactive here, it's ok.

I can see the updated docs landing page now and it does have the new ASP.NET Core video tutorials section, which is a link to a https://www.youtube.com/playlist?list=PLdo4fOcmZ0oW8nviYduHq7bmKode-p8Wy channel. This is a great first step.
I'm wondering what will it take to take this to the next level and do what I've asked for above?
Do I need to start an email thread with "higher-powers" ?

Video tutorials need to be curiated, typically by the engineering team. Obviously video's delivered by the engineering team don't need to be curated.  If you find a video that should be added, create an issue with the link and we'll get it added within 2 business days. We don't have the resources to search for videos and curate them.
In a poll of 20 ESL customers, with all 20 having strong english skills, all 20 said they found our English docs very useful but videos totally useless.

Thanks @Rick-Anderson. That's actually very interesting data you have there. What does ESL stand for?

English as a Second Language

And which videos were they referring to?

Professionally done videos, including those by @shanselman. The audio moves too fast. With english text, there is no ambiguity, no accents, and they can read at their own rate. They were emphatic that video didn't work for them, no matter how well done. They were all MVP's attending a MVP conference in Seattle.
cc @JasonCard

Let's pause on this. I can't ignore that data. Let me follow up on this offline. I'm wondering whether MVPs are not the right audience for this, given they're already experts. Anyway, let's pause on this for now till I get back.

@JasonCard  do you have any data non-english speakers and video?

MVPs are not the right audience for this, given they're already experts.

But these MVPs has excellent English skills, far better than most customers reading doc's in their native language. My guess is the odds are worse for non-English speakers, more like 100/100 don't like video.

hi @Rick-Anderson
I'm a little lost in this thread, but may I ask which videos we're discussing about and what kind of data are you looking for?
PV or CSAT on localized videos?

I took a survey of 20 ESL Microsoft MVPs and everyone of them said videos in English are worthless. I'm guessing you don't have any data on how non-English speakers view videos done in English.

I see. I don't think we have any data on that, but we do localize some videos in captions only. For example, videos in Microsoft Learn. I wonder if the videos would work for some non-English speakers if audio is English, but the captions are localized?

@mkArtakMSFT  another problem is video's get stale very quickly. When we find a good video, we do at it to the
Additional Resources

This Great video

Let me know if you'd like to keep this open.  cc @tdykstra

@mkArtakMSFT  I might have hijacked this thread with ESL (not english speakers) don't like video. Many of our customers are English speaking and they do like videos. Those that don't like videos just skip the video link.


Isolate FastDifferentiation pullback bug

@brianguenter here you have a PR where the test suite is reduced to the thing that makes FastDifferentiation error
If you want to try and run locally:

Clone the repo from the gd/fastdiff_fail branch
Activate the test environment with TestEnv.activate()
Include the file test/test_imports.jl
Then include the file test/first_order.jl

See the test failure https://github.com/gdalle/DifferentiationInterface.jl/actions/runs/8527052431/job/23357604893?pr=132#step:6:471


win32/msvc support + script + native dlfcn + console lib

feel free to tweak and/or discuss

ping @skeeto

Sorry I never took the time to address your port. It's a lot of code — one order of magnitude more than the original demo — so I kept putting it off and then never got around to it. The important part isn't all the (unfortunate) complexity of drawing to the terminal/console, but dynamic reloading. And to properly demonstrate a Windows version of the same concept, I'd orient it around the Win32 API rather than create a POSIX-like wrapper.
This demo involves a dynamic loading interface, file monitoring interface, and a terminal/console interface, each of which varies between OS APIs, and greatly in the case of the second two. So I think being portable between Win32 and POSIX for this program basically boils down to having two entirely different implementations. It could be simplified by inventing a common API and providing two implementations of it, but that would defeat the purpose of a simple educational demo. It's a fools' errand.

yep np :+1:


ECMA script - investigate a quick fix

What
Why
Who needs to work on this
Done when

No fix identified


Enable Ag-Grid Enterprise Features in GUI2

The current set up of the Table Viz in GUI2 imports a minimal set of Ag Grid packages but this is disabling some of the enterprise functionality (such as copy and paste).
This ticket is to re-enable all of these features by importing the full package.
Initial testing of re-enabling these packages showed issues with the focus management. Once you select into a grid it was very hard to get the selection back onto the graph rather than the grid. This meant Copy and Paste shortcuts was working inside the grid not in the graph.

there is a useAutoBlur utility to automatically blur (unfocus) an element when focus moves outside of its root element:
https://github.com/enso-org/enso/blob/ffa06c9476866bbfb69b46b01e5d3599c69f8f6e/app/gui2/src/components/CodeEditor.vue#L37-L38
not 100% sure it's the correct solution in this case, but it's probably worth a try


[New] Support else in template

Show else as a valid attribute.
If possible on control + click we take them to the if.bind that it matches to

Completed here https://github.com/CollinHerber/AureliaStorm/pull/8


Failed to take continuous backup using wal-g for missing env values

At first, pod-0 is primary and pod-1 is standby.
In this case, if pod-0 restarted, pod-1 should be selected as primary.
But, if pod-0 again selected as primary, then it causes a problem. Because of this
if [ ! -e "$PGDATA/PG_VERSION" ]; then
fi

So the problem is we do not configure ENV for wal-g again
export WALE_S3_PREFIX=$(echo "$ARCHIVE_S3_PREFIX")
set_walg_env "/srv/wal-g/archive/secrets"

We need to do this

fixed with https://github.com/kubedb/postgres/pull/126


Custom button to open ConfigComponent inside DisplayComponent

I was looking for a way for the drawer for the ConfigComponent to open from the click of a button inside the DisplayComponent.
Are there possibilities to disable the click in the entire DisplayComponent and generate a custom one within it?
The idea is to be able to have several buttons within the Display, the issue is that when you click anywhere in the Node, the ConfigComponent opens

I was looking for a way for the drawer for the ConfigComponent to open from the click of a button inside the DisplayComponent. Are there possibilities to disable the click in the entire DisplayComponent and generate a custom one within it?
The idea is to be able to have several buttons within the Display, the issue is that when you click anywhere in the Node, the ConfigComponent opens

You can disable opening the Drawer  in the entire DisplayComponent by stopPropagation in your DisplayComponent. And then call the setSelectedNode method in BuilderContext, so that you can customize the way to open the Drawer.

I was looking for a way for the drawer for the ConfigComponent to open from the click of a button inside the DisplayComponent. Are there possibilities to disable the click in the entire DisplayComponent and generate a custom one within it?
The idea is to be able to have several buttons within the Display, the issue is that when you click anywhere in the Node, the ConfigComponent opens

You can disable opening the Drawer in the entire DisplayComponent by stopPropagation in your DisplayComponent. And then call the setSelectedNode method in BuilderContext, so that you can customize the way to open the Drawer.
Here is a demo https://codesandbox.io/s/8wpqx7

Extraordinary, it was what I needed
Thank you!


WIP: feat(build): add cache invalidation mechanism for js/css files

The js/css files are the result of a build done by webpack. Each
time they are updated, browsers will get an outdated version from
cache on the webserver or CDN which are difficult to invalidate.
By appending as querystring a unique identifier calculated on the
content of the file we make sure browsers always get the updated
version.

Have you looked at https://github.com/owais/django-webpack-loader ? We use it in edx-platform to bridge webpack and Django, and it supports optimized builds.

We can now merge this PR as the disturbing file references in CSS have been removed.
@ormsbee It's not clear to me what django-webpack-loader would improve to the current Webpack build we have? As for cache invalidation, the ManifestStaticFilesStorage included in Django seems to do a good job. It caches the mapping between each file and its renamed version in a json file manifest.


Typo in Overview.md

What would you like to be added:
The following step could help us to better integrate with sklearn, which means we could treat the customized feature selector as a mudule of the pipeline.
should be
The following step could help us to better integrate with sklearn, which means we could treat the customized feature selector as a module of the pipeline.
Why is this needed:
the typo made me wonder if mudule was a word that I did not know. it confused me.
Without this feature, how does current nni work：
it works fine.
Components that may involve changes:
Docs
Brief description of your proposal if any:

thanks @DanielMcSheehy  and @AnshuTrivedi for the fix.


DHT11 (Temp/Humidity) Plugin

Hi
Thank you for this great software! I've enjoyed building a pixelix clock and working with the base. I've created a DHT11 plugin that displays temperature and humidity as read by a dht11 sensor (connected to IO5). I'm happy to have it reviewed and maybe included.
I've also implemented a bitcoin/usd exchange rate plugin - happy to submit that as well. TLS/SSL support in the AsyncHTTP Client library is really needed for more/better choice of data sources :-)
Let me know how you want the submissions.
Have a nice day

Hi, thanks for the praise. :-)
You can create a pull-request in the PluginDevelopment branch and we will review it (according to our coding style too :-) ).
In our PCB currently IO5 is not used, therefore we will add the plugin configurable via preprocessor. Have to think about how we handle this in the Board.h too.
Waiting for some time now for TLS/SSL support and hopefully me-no-dev will have time after esp32-arduino v2.0.0 release. I try to avoid to implement it by my own, like the AsyncHttp client. We will see. On other hand e.g. if node-red is running locally, you can request data from https sources and push it to the display.
Have a nice week!

Something to discuss: It's becoming apparent that the DHT11 is very inaccurate. However it is cheap and well sourceable. However I've now ordered a replacement (SHT31-D). Now the question: Should I then setup a second plugin for the new sensor? Or should I rather create a TempHumid plugin and make the sensors definable at compile time? One thing to note - DHT11 is a kind of onewire protocol, the new one is i2c.
What's your opinion?

Some months ago we thought about having a universal sensor platform separate (separate HW/SW), which push the information to the display via REST API. This idea is freezed at the moment, because lack of time.
For now I would prefer having one TempHumidPlugin, where the sensor can be choosen. In the first step at compile time, e.g. via platformio.ini and maybe later via web interface at runtime. All depends on the available flash size and right now we are 85% full, but this will increase with SSL/TLS stuff for sure.

I've now made the basic import of the source code. How should I handle the Compile-Time switches?
Two switches (one for the module itself and one for the sensor) or one just for the sensor and compile the module without sensor if none is chosen?
Regarding the flash size:
I've read that it's possible to setup custom partition tables to increase the available space for the program (as we are running with 4MB, 2MB Flash/2 MB Data could be an option to increase the firmware size?)

I just realized you proposed to create a class for each sensor.
Where should I put the TempHumidDrv class definition? Into the Plugin/Plugins folder? Or some other place?

I've now made the basic import of the source code. How should I handle the Compile-Time switches?
Two switches (one for the module itself and one for the sensor) or one just for the sensor and compile the module without sensor if none is chosen?

Does your plugin already supports more than just the DHT-11 sensor? If the DHT-11 sensor is the only one, you won't need the compile time switch. You may come back with, after the plugin is extended. This would make it easier for the review and integration too.

Regarding the flash size:
I've read that it's possible to setup custom partition tables to increase the available space for the program (as we are running with 4MB, 2MB Flash/2 MB Data could be an option to increase the firmware size?)

Yes, custom partitions are possible. Don't forget that because of the over-the-air update concept, you need storage size twice. And right now in the filesystem we have 35% unused from 1378241 byte. My guess is, that with integration of SSL/TLS it will be full, but we will see.

I just realized you proposed to create a class for each sensor.
Where should I put the TempHumidDrv class definition? Into the Plugin/Plugins folder? Or some other place?

The classes in the 3rd party package are just placeholders for the 3rd party libs you will require via platformio.ini

Okay, I will submit it like this. I do have the new sensor, but I need to get it connected and everything first.
One question: How do I easily access the "duration" available for display from inside the plugin? I'd like to switch the temperature/humidity display after 1/2 duration.

Please have a look at the DateTimePlugin (updateDateTime()) this plugin also switches its content after 1/2 duration.


Could not refresh api access token

I recently started getting the following errors in the logs:
[31/03/2022, 19:33:43] Homebridge v1.4.0 (HAP v0.10.0) (Homebridge 341A) is running on port 51460.
[31/03/2022, 19:33:43] [CarrierInfinity] Could not refresh api access token:  getaddrinfo ENOTFOUND www.app-api.ing.carrier.com
Status:  undefined
Data:  undefined
[31/03/2022, 19:33:43] [CarrierInfinity] Failed to fetch updates (axios):  getaddrinfo ENOTFOUND www.app-api.ing.carrier.com
[31/03/2022, 19:33:43] [CarrierInfinity] Could not discover devices: Cannot read property 'locations' of null

Sorry looks like I was having a networking problem with my Pi. Closing as this is not an issue.


clidb.py variable_list fails for ocean

There's a few changes needed to make this command works with the ocean data files.

Fixed, the issue was linked to having to take into account multiple frequencies for um files, umfrq wasn't defined for other realms


PHPUnit70Migration ruleset

One of the changes in phpunit 7.0 is that @test annotation is checked a bit more strict. This means that
/**
 * Big @test, Only run if you have time
 */

Is no longer accepted as test annotation.
I propose a fixer that would change this into
/**
 * @test
 * Big @test, Only run if you have time
 */

This should probably be a risky fixer, since it might change functions that you don't want to be tests into tests.
See: https://github.com/sebastianbergmann/phpunit/blob/master/ChangeLog-7.0.md

I guess this could be implemented as well:

@scenario is no longer an alias for @test

@BackEndTea sounds like great idea, could you create a PR ?

Another thing is that methods like setUp, setUpBeforeClass, tearDown and tearDownAfterClass must declare void as return type. There are some more signature changes listed here: https://phpunit.de/announcements/phpunit-7.html


No submenu options in shader settings as shown in screenshot/demo video

Using Unity 2020.3.17f1
Also has Hidden/InternalShaderError on the Material when first importing. Not sure if that may cause the error.

Thank you for the report. It is not happening here, so if there is a way to reproduce the issue, feel free to re-open the issue.

This is what I see on the Shader options on the Inspector panel for Unity 2020.3.17f1, nothing else.
https://share.getcloudapp.com/12u7GobK
All instructions to use were followed based on README.
The same applies to a newly created Unity 3D URP project as well.

Oh, those options are provided in newer versions of ShaderGraph and Unity. It is not a thing I can do much about that.
P.S. you can share you images directly in github. just copy yout image in your clipboard and paste (ctrl-v) in the comment. it will be something like this:
![image](https://github.com/omid3098/Unity-URP-GlassShader/assets/6388730/3945c831-1c51-471c-9f73-a0ca888a3744)

and you can preview the image in the preview tab:


Improvements to System Requirements doc

Added note to review w/ operations team
Specified disk allocation per directory
Added minimum total VM count for each configuration to improve clarity on total requirements

Thanks @sundippatel


Example using Azure  Powershell

Hi , Using the Managed identity from VM , i need to access to Storage account to download / upload the data . Can we get example using Powershell ?

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

ID: e5ce8d88-79ef-1c9a-fd65-167e3ca2f36a
Version Independent ID: 2c78ff72-b8c6-2b1f-de73-114594a8dbf1
Content: Access Azure Storage using a Windows VM system-assigned managed identity
Content Source: articles/active-directory/managed-identities-azure-resources/tutorial-vm-windows-access-storage.md
Service: active-directory
Sub-service: msi
GitHub Login: @MarkusVi
Microsoft Alias: markvi

able to get the details using Connect-AzAccount -identity . MSI got authenticated able to connect the storage account .

@kgopi1 Thanks for the follow up. 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.


Neural net expects 'input' features with dimension 43 but you provided 40

Is this a problem caused by the model？

probably related to providing wrong input features / wrong config