Thrift to Metadata filter proto design

Commit Message:
For apache thrift compatible HTTP requests and responses, this filter parses the thrift metadata and put them into filter dynamic metadata for other filter usage.
This is the initial proto design, which refers to other filters like json_to_metadata and payload_to_metadata.
Additional Description:
Risk Level: low
Testing: build
Docs Changes: yes
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] #29371
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@htuch could i have an early api review? the test failure is coverage which is expected

@zuercher is it ok to assign this one to you?

do you want to mark all the protobuf as not yet implemented and move forward with this PR just to get the protobuf API in place

Sure I'll have a very basic unit test to pass the coverage so we can move on the wip filter. Thanks for the great idea.

do you want to mark all the protobuf as not yet implemented

just notice i already set option (xds.annotations.v3.file_status).work_in_progress = true;

@zuercher I addressed all the comments except this https://github.com/envoyproxy/envoy/pull/33919#discussion_r1599303324 which needs some input. Also add some trivial unit test, hopefully pass the coverage test

do you want to mark all the protobuf as not yet implemented

just notice i already set option (xds.annotations.v3.file_status).work_in_progress = true;

Sorry -- I didn't realize that could be done at the file level vs. per-message.

/retest

we got approval and api reviewed so i think we're good to merge.
https://github.com/envoyproxy/envoy/pull/33919#pullrequestreview-2072613349


[getDefaultViewport] image has no property modalityLUT nor voiLUT

On getDefaultViewport() we are coping modalityLUT and voiLUT properties of image into the new created viewport like so :
 return {
    scale : 1,
    translation : {
        x : 0,
        y : 0
    },
    voi : {
        windowWidth: image.windowWidth,
        windowCenter: image.windowCenter,
    },
    invert: image.invert,
    pixelReplication: false,
    rotation: 0,
    hflip: false,
    vflip: false,
    modalityLUT: image.modalityLUT,
    voiLUT: image.voiLUT
};

Except we are never setting modalityLUT nor voiLUT on the image object anywhere on the library. So theses properties will most likely be undefined.
But on renderGrayscaleImage we are doing :
  image.lut.voiLUT = viewport.voiLUT;
  image.lut.modalityLUT = viewport.modalityLUT;

So we should either set modalityLUT and voiLUT into the image object or into the lut object (which is in fact an Int16Array).

modalityLUT and voiLUT are set on the image object in cornerstoneWADOImageLoader - does that explain things?

Ok this explain some things but taking for example voiLUT  we have

image.voiLUT which is some kind of default lut.
viewport.lut    which is the lut we want to apply during a display
image.lut.voiLUT which is a cache of the last lut used with this image on any enabledElement

While I can understand the first 2, the last one seems weird. In fact the whole image.lut property seems weird  because it's a rendering property not related to the image. I think image should be an immutable object.
So that issue is more a design concern than a code issue.

image.lut has been changed to image.cachedLut so this should be more clear now.


view: actions in template/iso zones

Fixes #279

@shwstppr instead can you remove the copy icon and move it to the template/iso zones component where we have the delete button already?

@shwstppr instead can you remove the copy icon and move it to the template/iso zones component where we have the delete button already?

@rhtyd not sure about template/iso zones component. I see UI as below,

We are just showing a tab for template zones and there is no additional delete button there. Can you please point me to the relevant code.

@shwstppr move the copy and delete buttons/action to the iso and template zones components.

@rhtyd movde template/ISO actions in zone item list

@shwstppr can you merge latest master on this branch and resolve conflict?

@rhtyd merged latest master

Test issues:

Don't poll for async job (both copy and delete operations), notification too early
should refresh after async job poll, for delete if there are no zones, run router.go(-1)

I've added support for template/iso filtering.

remark @shwstppr - this was an untested PR, actions didn't work or show errors/success without doing proper polling.

Reviewed and fixed. Tested:

Iso and template registration, fixes in form component
Filter by for template/iso
Copy and delete command for template/iso with job polling
Go to preview page on deletion of template/iso with a single zone


TraceabilityDriverService Controller Validation

We need to add validation of arguments on each API method on the controllers in the TraceabilityDriverService. This includes:

AccountController.cs
DigitalLinkController.cs
EPCISQueryController.cs
EventsController.cs
LocationController.cs
TradeItemController.cs
TradingPartnerController.cs

This has been added in.


nes-avatar add image-rendering pixelated

Is your feature request related to a problem? Please describe.
I was concerned that only the avatar image is rendering smoothly among the components of the dot.
Describe the solution you'd like
.nes-avatar {
    image-rendering: pixelated;
}

Additional context
Although it may not have a big influence, I thought that it might be able to offer something close to the value of NES.css when I put it in.
However, Can I Use says that since browser is not fully compatible, I don't think there is anything easy to introduce.
https://caniuse.com/#feat=css-crisp-edges
Please consider about this Issue :-)

Thanks for your issue 🙇
That's a good point.👍
But image-rendering depends on the user's image file and thing users want to do.
So It seems beter to set image-rendering in inline style instead of adding to nes-avatar.🤔
I'll try on the demo page as follows: https://github.com/nostalgic-css/NES.css/pull/277#issuecomment-461852242


Moving config classes to own java package

Simultaneously moving code currently under config to configgen.

Current coverage is 78.63% (diff: 100%)

Merging #559 into master will not change coverage

@@             master       #559   diff @@
==========================================
  Files           248        248
  Lines          9206       9206
  Methods           0          0
  Messages          0          0
  Branches       1212       1212
==========================================
  Hits           7239       7239
  Misses         1656       1656
  Partials        311        311

Powered by Codecov. Last update 94327e4...f806918

LGTM


Room name doesn't register special characters instead of throwing error

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Break down in numbered steps

Choose New Room from the create menu
Type in a special character (e.g. ! or . in the room title

Expected Result:
Describe what you think should've happened
The special character that was typed is shown, along with a validation error saying that character is not allowed.
Actual Result:
Describe what actually happened
The character is not shown at all.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: v1.2.26-0
Reproducible in staging?: y
Reproducible in production?: y
Issue reported by: @puneetlath
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1668173276175579?thread_ts=1668112427.854869&cid=C049HHMV9SM
View all open jobs on GitHub

@lschurr I'll fix this.

Will start on this today.

Cool @puneetlath - do I need to do anything? Should I be trying to reproduce?

Nope, it's reproducible.

Heyooooo any updates on this one @puneetlath?

I realized I need to refactor this page to use the new Forms component, so it's a little bit more involved then I initially thought. Hope to have a PR up today.

Still working on this, should be done soon.

@puneetlath - you're OOO this week right?

Gonna move to Weekly since Puneet is OOO.

Hey hey @puneetlath - any update on this one?

Working on it today! ETA Monday.

Still in progress. Working through an issue here.

Also assigning myself since I'm managing the tracking issue here

Still on hold, now for https://github.com/Expensify/App/issues/13915

@davidcardoza we need some error copy for when someone tries to enter an invalid room name. We only allow letters, numbers, and dashes in room names. All other characters are disallowed. I'm thinking:

Room names cannot contain punctuation or most special characters

For reference, here is Slack's:

How does that sound?

How about this, just a slight addition:

Room names cannot contain spaces, punctuation or most special characters

I think most users will try to create a room with spaces, but that's just my opinion.

If they try to enter a space we automatically replace it with a dash. So I don't think we need to include that.

That's awesome, I didn’t know that. Your copy looks good to me in that case.

@lschurr FYI this is internal, but @rushatgabhane is helping out with PR review and will need to be paid out (once this goes live).

@rushatgabhane could you apply for the job in Upwork? https://www.upwork.com/jobs/~01f8eb2fdbe370c1ac

Hey @puneetlath - Is this live? Should I pay and close this out or are we still waiting on something?

Yes! The PR hit production on Jan 9 so you're good to pay out @rushatgabhane on this one.

Awesome. Paid!


Atlantic Cozytouch not available ?

Hello, i'm trying to access the server altantic_cozytouch, but it tell me this error: Cannot connect to host apis.groupe-atlantic.com:443 ssl:default [The specified network name is no longer available.]
It's only me or not ?

Yes, i don't know if it's my username and password, or maybe something else.

This error is not related to your username and password, but to your network configuration. Would be good to double check if that host is reachable from other systems.

Do you know how to setup the proxy adress to use or not?

Ok it's good, i have access to my devices, i wanted to know if there is a getter for like the energy consumption etc.. ? i can't find it. @iMicknl


make poetry version work like npm version

update
Please see @mattyclarkson 's comment below for what this issue should be!

[X] I have searched the issues of this repo and believe that this is not a duplicate.

Issue

Interested to see what others feel about this. I've braino'ed poetry version a couple of times when I meant poetry --version. "version" is a noun in my head, and poetry inc_version would be more explicit.
However, I'm well aware this might just be me, so thought I'd ask/suggest here and see what others said :-)

Actually, that's not quite true, from chatting with a JS dev:
npm version outputs the current version of your project
npm version minor updates the third point
...etc...
I think that would be better for poetry too :-)

Run into same issue recently, as an npm user I expected the behavior from
https://github.com/sdispater/poetry/issues/629#issuecomment-439128806, when npm version call outputs the version of the package instead of increasing it, so I propose to change poetry version to work same as npm version

@mattyclarkson @abn - what do you think to @playpauseandstop 's suggestion. I like it :-)

@cjw296 fwiw, I do not think such a change is warranted. The option --version follows the expectations set by all other options in poetry and version  is a sub-command with a default behavior enacted on the project context. While poetry does share similarities with npm both these tools cater to different ecosystems with their own conventions and practices. Python command line tool options and arguments borrow heavily from conventions established by optparse and argparse. And this is anticipated quite naturally by developers working within the ecosystem.
Speaking for myself at least, the behavior of poetry version as opposed to poetry --version is natural and expected. This should not be changed.

@mattyclarkson - your description is perfect, let me update the issue title and point to your comment...

@mattyclarkson Just to be clear, I did not mean to allude that the purpose of the version sub-command in these tools serve different purposes but rather the tools sometimes cater to idiosyncrasies of the ecosystems they are a part of.
I, personally, have no objection to a change that updates the default behavior and in-fact do agree that having consistent behavior across environments is the way to go.
My comment was following from the issue description requiring a rename of the sub-command to inc_version. That, I think is not warranted.

@mattyclarkson, @abn - great, sounds like we're all on the same page: keep poetry version but make it display the current project's current version, and add something to explicitly do a third point increment.
@sdispater - would you welcome a PR that made this happen?

I think this issue deserves more thumb up's, given the update in the issue.
Right now I have to work around this by installing a toml library to read the pyproject.toml, and running a one-liner to get the project version. For anyone curious:
python -c 'import tomlkit; print(tomlkit.parse(open("pyproject.toml").read())["tool"]["poetry"]["version"])'

Since what @mattyclarkson suggests is a breaking change, putting it in the 1.0 release seems fitting.

poetry bump major
poetry bump minor

#1191 fixes this.


failed to install angular2-google-chart

when I tried on windows 7 and Ubunt6u, I got below error with npm install command:
d:\Test\ng2\angular2-google-chart>npm install angular2-google-chart
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "D:\nodejs\node.exe" "D:\nodejs\node_modules\npm\bin\npm-cli.js" "install" "angular2-google-chart"
npm ERR! node v6.6.0
npm ERR! npm  v3.10.3
npm ERR! code ENOSELF
npm ERR! Refusing to install angular2-google-chart as a dependency of itself
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     d:\Test\ng2\angular2-google-chart\npm-debug.log
thanks.

I am getting this

sh: 1: typings: not found
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm ERR! Linux 4.4.0-43-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install" "angular2-google-chart"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>postinstall: typings install
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'typings install'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the angular2-google-chart package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     typings install
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs angular2-google-chart
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls angular2-google-chart
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/cfontes/workspace/ask-the-world/src/main/resources/static/ask-the-world/npm-debug.log

cfontes, try it before install the library:
npm install typings --global


bacon cooking time

I'm not 100% sure. But are you cooking the bacon for 3 seconds a side?

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

ID: 17d27a0d-c072-8c9f-7e31-3de60bee9ae8
Version Independent ID: e0c09794-e712-19ff-cb9f-12648d60d4f6
Content: Asynchronous programming in C#
Content Source: docs/csharp/programming-guide/concepts/async/index.md
Product: dotnet-csharp
Technology: csharp-async
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@jon-searle
For this example code, yes. That means the sample app finishes in a reasonable amount of time. We do recommend properly cooking all breakfast meats. 🥓
I'll close this as an answered question.


mpfs: add opensbi support

Summary
BSD-2 license present in the following file:
arch/risc-v/src/mpfs/mpfs_opensbi_utils.S, due to function mpfs_exception_opensbi
This patchset introduces the necessary hooks for utilizing OpenSBI on the Polarfire Icicle platform.
Impact
Now it's possible to start U-Boot and Linux kernel via NuttX.
Testing
3 harts have been started using U-Boot / Linux kernel and a development rootfs on Polarfire Icicle board all to way to the rootfs prompt and beyond.

Hmm.. of course the compilation fails as the necessary 3rd party lib isn't present. Maybe I need to rethink this.


Ameerul / FEQ-2628 [[FE] - P2P Guide - Responsive] - [UI Bugs] AutoScroll Issue + Text readjustment in FAQ cards

Updated deriv-com/ui library to 1.35.6 where I had implemented the changes for Accordion
Added refs and function to enable scrolling to the Accordion when it is opened.
Removed scrolling ability in Accordion Content since it is not needed and when the user hovered over the content, it would add a scrollbar which made it look too clunky

https://github.com/user-attachments/assets/18678b8d-602a-4dfd-b1a5-5140c3ddc48f

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

12 of 20 (60.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.08%) to 85.607%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/pages/guide/screens/FAQs/FAQs.tsx
10
18
55.56%

Totals

Change from base Build<PHONE_NUMBER>8:
-0.08%

Covered Lines:
6405

Relevant Lines:
6974

💛  - Coveralls


Add e2e tests

Ticket
Resolves https://github.com/navapbc/template-application-nextjs/issues/358
Changes

add playwright e2e tests against preview env

Context

Working PR on nextjs test repo => https://github.com/navapbc/platform-test-nextjs/pull/85

The docs PR check error I don't think will resolve until this is merged

We've accumulated quite a few changes, do we want to apply these changes back to platform-test-nextjs?

We've accumulated quite a few changes, do we want to apply these changes back to platform-test-nextjs to ensure its working?

@rylew1 yeah definitely. I kind of assumed you were testing the changes, so yeah if you haven't yet then let's do one last final test to make sure it all works

Sorry accidentally closed the PR


allow env_servlet_processes to be empty dict

#1136  👈
main

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

Join @BelSasha and the rest of your teammates on  Graphite


Improvment to the submisison's related section

Update the data displayed - add current status and round/lab
Update the look and feel of it
Limit the title display to 2 lines.
Refractored the JS to be jquery independent and make it collapsing
behaviour generic, some concepts borrowed from https://rstacruz.github.io/rsjs/ for structuing the custom js code

Before:

After:

this looks fantastic! I'll do a deeper dive tomorrow but seems like a really solid improvement.


Use the common histogram interface.

Generalizing https://github.com/henryiii/uhi/blob/master/uhi/typing/plottable.py .

@henryiii This is intended to generalize the histogram protocol.
These are tests, demonstrating everything with the histograms I have in scikit-hep-testdata:
https://github.com/scikit-hep/uproot4/blob/jpivarski/use-the-common-histogram-interface/tests/test_0167-use-the-common-histogram-interface.py
The TAxis behaviors are defined here, implementing this protocol and the Sequence protocol (because I think that was desired):
https://github.com/scikit-hep/uproot4/blob/jpivarski/use-the-common-histogram-interface/uproot4/behaviors/TAxis.py
Histograms are defined in files for each class, but an abstract class is defined here in the file for TH1, as a way of getting docstrings to all the others. (Uproot 4 does not follow ROOT's class hierarchy, giving us more freedom in behavior implementations, like not having TH2 inherit from TH1, etc.)
https://github.com/scikit-hep/uproot4/blob/jpivarski/use-the-common-histogram-interface/uproot4/behaviors/TH1.py
This abstract class is mostly overwritten by the one for profiles, since the docstrings have to be very different.
https://github.com/scikit-hep/uproot4/blob/jpivarski/use-the-common-histogram-interface/uproot4/behaviors/TProfile.py
There's a proliferation of variance methods: errors, variances, values_errors, and values_variances. This is because regular histograms use values as part of the calculation for variances, which can be used to derive errors, and profiles use values as part of the calculation of errors, which can be used to derive variances. The methods that return two arrays, values and something else, are primary; the others are conveniences derived from them.
There are two FIXMEs in the the profile implementation, one for filling counts (I'm unclear about what it means) and the other for building a Boost histogram representing the profile. I'm 100% certain of the calculation from ROOT's C++ members to values and variances, but I'm unsure how to fill Boost. To start with, shouldn't a profile be "WeightedMeanVariance Storage," not "WeightedMean Storage"?


Turn Off/On

Hello, I'm having troubles turning off and after turn on tv.

Hi, thanks for your interest in the project. What errors do you see on the console when you ask to turn off tv (if any)? Can you share a little about your setup, such as if you are running the application locally or what endpoint are you using (Heroku, etc)?

im running application locally using ngrok. and i didn't found any error

so is your question that you are having trouble 1) turning on the device (after you have turned off) or are you 2) having trouble turning off the device?
If it's 1), this is a known issue because the TV loses it's IP address after a while. You can send a turn off command and immediately turn on device without much issue, but like I said it's going to be difficult to find the device on the network (all solutions are limited). Other users have raised and the team has advised that this program should be run with a TV that's already turned on - for example Amazon itself acknowledges as much when they created their fire tv cube, where it sends infrared commands to turn on the tv's that are sleeping.
If it's 2), can you share with me screen shots of your log after issuing a command like mute and then issuing turn off command. I'm not sure if you are issuing commands in the testing console in Amazon or on the device itself, but either way when a command is issued, ngrok receives and sends to your console that's running the application, I just want to confirm that the other commands are running to understand why turning off doesn't work.

its 1 but is there any plans for any solution?


E-Mail doesn't work after snap update

I use Snap-Update on Ubuntu 16.04.3 LTS for Wekan.
After every Snap-Update E-Mails from Wekan doasn't work any more. I need to go to the E-Mail-Settings and re-enter the password to get E-Mails working again.
BTW: Thanks for this great tool. I switched from Trello to Wekan for privacy reasons (EU-DSGVO) a couple of months ago and do not miss anything important.

I have a related issue.
I run Wekan as a snap on Ubuntu 16.04.4 LTS. Every morning I must go in to Admin Panel, Email and press Save or emails are not sent. I have set up a cron job run snap set commands every morning to address this.

@cyberfarer
What cron job you use? Source?

@cyberfarer I would also like to know the set-command for fixing this per cron :-)

@spassig @xet7
I created a file /etc/cron.d/wekan with the following contents:
0 7 * * * root /usr/bin/snap set wekan mail-url='smtp://localhost:25'
0 7 * * * root /usr/bin/snap set wekan mail-from='CLSA TASK BOARD<EMAIL_ADDRESS>I use postix as a smart relay to our organizational SMTP server. I found both commands necessary as executing the first replaced the from address with<EMAIL_ADDRESS>So far so good.

Hi,
I have the same problem: after every automatic Snap update email via the WebUI does no longer work. Saving the email settings without any changes solves this problem. Unfortunately, I have to do this after every Update. As described above, I created a cron tab to re-set the email settings daily (via snap set ...). But this does not affect the WebUI. If I want to invite new users, I have to save the settings at the WebUI before this, otherwise the new users does not get an email notification.
Are the email settings via "snap set ..." related to the email settings via WebUI? Can I solve the WebUI problems with the crontab?
Thanks and regards
Jochen

@spassig @cyberfarer @JHAServiceTrace
Please test does now snap set wekan mail... settings keep working with newest Wekan.

Hello,
my current Wekan version is 5.02.0. Now, the email settings in the Admin panel show only the button "Send a test email to yourself". After selecting the button, the error message "... SSL routines:ssl3_get_record:wrong version number:../deps/openssl/openssl/ssl/record/ssl3_record.c:332" appears in the web UI. I know this error from previous releases. In the past I could fix this problem by saving the email settings in the Admin panel. Without changing the properties.
Wekan is installed via Snap at an Ubunutu 18.04 host. I searched a lot via Google. Unfortunately, I cannot find an answer to this.
Regards
Jochen

https://github.com/wekan/wekan/wiki/Troubleshooting-Mail#if-you-get-sslv3-protocol-error

Hello,
thank you very much for the fast reply. I tried this settings. Now I get the following error: "Mail command failed: 451 5.7.3 STARTTLS is required to send mail".
Regards
Jochen

@JHAServiceTrace
Maybe this?
https://github.com/wekan/wekan/wiki/Troubleshooting-Mail#wrong-port
Please search that webpage for STARTTLS. Thanks!

I have disabled my cron jobs. I will report back tomorrow.

@cyberfarer
Thanks! I'm currently building new Wekan release, so we will then see does it work.

Hi,
starting with your following example I changed several parameters and checked the result:
snap set wekan mail-url='smtp://support%40example.com:password@mail.example.com:587/?ignoreTLS=true&tls={rejectUnauthorized:false}&secure=false'
We are using Microsoft Office 365. The following parameter set works for me.
snap set wekan mail-url='smtp://user:password@smtp.office365.com:587?ignoreTLS=false&tls={rejectUnauthorized:false}&secure=true'
I will change my crontab, respectively.
Thank you very much for your help.
Regards
Jochen

@JHAServiceTrace
Thanks a lot ! I added your Microsoft Office 365 example to https://github.com/wekan/wekan/wiki/Troubleshooting-Mail


Create Auth_with_Appwrite.md

A React-based secure authentication system utilizing Appwrite for user registration, login, password recovery, and session management, ensuring scalable and reliable user access control.
Hackathon Submission: Auth Using Appwite
Name of Team Lead
Raj Vardhan Chaudhary

GitHub handles of Team Members
Enter the GitHub handles of all your team members (including yourself) as separate bullet points

Are all of your team members registered on our Hackathon Signup page?
This is a mandatory step for all participants to be considered eligible for judging

[x] Yes
[ ] No

Rules and Code of Conduct
By submitting this PR, you agree to follow our Rules and Code of Conduct.

[x] Yes

Anything Else?
Any other feedback, queries or information you would like to share with us?

@adityaoberai  how i collect my gift?

@adityaoberai how i collect my gift?

Sorry, what gift?

@adityaoberai as mention there are some gift for submitting a project

@adityaoberai as mention there are some gift for submitting a project

Appwrite is providing prizes to top 3 projects after judging.
Since this hackathon occurred during Hacktoberfest, any digital prizes they provide will be available to you. That is not something we have control over, though.

@adityaoberai  can you provide me any digital certificate for shere on other social media?

@adityaoberai  can you provide me any digital certificate for shere on other social media?

That is not something we provide. Your merged PR is proof of submission.

@adityaoberai thankyou and happy diwali 🎇


Typing errors of Jupyter notebook still show after closing notebook

Environment data

Language Server version: 2021.6.0
OS and version: macOS 11.4
Python version (& distribution if applicable, e.g. Anaconda): 3.9.5

Expected behaviour
Typing errors of a Jupyter notebook should disappear when the notebook is closed.
Actual behaviour
Error messages still persist.
Screenshot

Before closing the notebook

After closing the notebook

The Jupyter extension manages the lifetime of these notebooks, so I'll transfer this to them. It's up to the client to open and close documents to ensure VS Code manages the diagnostics.


 It would be nice if we kept the sticky scroll around even though the original lines no longer exist.
Repro:

Set terminal.integrated.scrollback to 100
Run ls -lR . or tree, 🐛 sticky scroll shows briefly but then goes away

No plans to do this any time soon


chore: Add unit tests

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage
/sig storage

/kind enhancement
What this PR does / why we need it:
Add unit test
Which issue(s) this PR fixes:

None
Special notes for your reviewer:

None
Does this PR introduce a user-facing change?

other Note

/retest

/test


Revert "Hold a JVM-wide reentrant lock to grab mutex — helps ZK stale reads?"

Reverts vespa-engine/vespa#11815
@hakonhall let's revert if this doesn't help. We should know within a week or two.

Done elsewhere.


Axes: need a way to Remove() axes from multi-axis plots

Felxi in the Discord (#1966) noted there is no way to remove additional axes once added
https://github.com/ScottPlot/ScottPlot/blob/6380848be9dd73d62335feb4497ce3c967a08af9/src/ScottPlot5/ScottPlot5/AxisManager.cs#L14-L22
https://github.com/ScottPlot/ScottPlot/blob/6380848be9dd73d62335feb4497ce3c967a08af9/src/ScottPlot5/ScottPlot5/AxisManager.cs#L112-L125

Plot myPlot = new();

var sig1 =myPlot.Add.Signal(Generate.Sin(51, mult: 10));
var sig2 = myPlot.Add.Signal(Generate.Cos(51, mult: 1));

// create an additional axis and setup the second signal to use it
IYAxis secondYAxis = myPlot.Axes.AddLeftAxis();
sig2.Axes.YAxis = secondYAxis;

// remove the additional axis
myPlot.Axes.Remove(secondYAxis);

// tell the signal plot to use the original axis
sig2.Axes.YAxis = myPlot.Axes.Left;


DOC: add cookbook

Something like:
http://pandas.pydata.org/pandas-docs/dev/cookbook.html
Listing common idioms, one liners, FAQ ...

From @dengemann:
"We should somewhere write down some of our local style preferences in some contrib file. - return self - single quotes - generator / list comprehension instead of filter et al - explicit functional constructors for builtin containers, dict() et al - avoid nested functions if not necessary"

We should somewhere write down some of our local style preferences in some contrib file:

return self
single quotes
generator / list comprehension instead of filter et al
explicit functional constructors for builtin containers, dict() et al
avoid nested functions if not necessary and use private functions instead
when addinv viz methods, add public functions to the mne.viz package and link them
if not otherwise required, methods should deal with state while functions should return copies. Threre are a few justified excpetions though, e.g. equalize_channels, for memory reasons for example
update whats_new at the end, otherwise merge conflicts are garuanteed to occur
address one and only one issue per PR.
avoid **kwargs and *args in function signatures, they are not user friendly (inspection)
...

added some updates.

You got the above comment in your current PR @dengemann, and I added the residual "cookbook" idea to #1495


drivers/sensor: lis2mdl: use common SPI duplex option

Driver used a custom SPI duplex option. Replaced with the common one found in spi-device.yaml.

This looks like a good change, but need to fix the in-tree boards (this will resolve the failing twister checks) and add a note to the migration guide

Applied your suggestions but also realized duplex == 0 is full duplex.


View -> Appearance -> Fullscreen still says 'F11' despite F11 key doing otherwise.

Issue Type: Bug
I have no way of knowing exactly what of the 1000s of possible sources of incoming side-effects could contribute to this. The amount of information I can access allows me to be certain that it will be reproducible only where you have my exact system with my exact module set. The F11 key causes a debug session to initiate while I am working on this javascript file while a nodejs package source folder is open in the latest version of vscode downloaded into debian using apt according to the instructions on their wiki.
VS Code version: Code 1.66.2 (dfd34e8260c270da74b5c2d86d61aee4b6d56977, 2022-04-11T07:49:24.808Z)
OS version: Linux x64 5.10.106
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8700K CPU @ 3.70GHz (12 x 4602)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onrasterization: disabled_softwareraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
15.52GB (9.11GB free)

Process Argv
--unity-launch --crash-reporter-id e585ffe9-7db6-45fd-8db9-70a6acc77de2

Screen Reader
no

VM
0%

DESKTOP_SESSION
gnome

XDG_CURRENT_DESKTOP
GNOME

XDG_SESSION_DESKTOP
gnome

XDG_SESSION_TYPE
x11

Extensions (16)

Extension
Author (truncated)
Version

language-x86-64-assembly
13x
3.0.0

ebrew-vscode
498
0.1.4

vsc-material-theme
Equ
33.4.0

vsc-material-theme-icons
equ
2.2.1

customize-ui
ioc
0.1.61

monkey-patch
ioc
0.1.16

vscode-clangd
llv
0.1.15

remote-containers
ms-
0.231.6

remote-ssh
ms-
0.78.0

remote-ssh-edit
ms-
0.80.0

vscode-remote-extensionpack
ms-
0.21.0

cmake-tools
ms-
1.10.5

cpptools
ms-
1.9.8

hexeditor
ms-
1.9.6

vscode-typescript-next
ms-
4.7.20220421

cmake
twx
0.0.17

(2 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627cf:30244335
pythonvspyl392:30443607
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
pythonvsnew555:30457759
vscscmwlcmt:30465135
cppdebug:30475216
vscaat:30438848

please see https://stackoverflow.com/questions/29955146/unable-to-debug-with-f11-with-vscode-toggled-fullscreen

@weinand the OP's version info shows Linux rather than macOS.

On linux the following commands are assigned to the F11 key:

So the "Fullscreen" command uses the F11 key on linux.
Yes this is unfortunate, but we will not change this.

So why does the menu still put F11 text next to View -> Appearance -> Full Screen if F11 is assigned to something else? Even if node devs are going to get preference here, why lie to everyone else?

F11 is typically used for debugging as well. So there is a clash.
In VS Code F11 for debugging should only be enabled if the currently selected launch config is of type "node".
@isidorn @connor4312 do you remember the rational behind this?

@weinand probably the keybinding enablement of full screen is not dynamic enough and the menu does not pick it up.

In VS Code F11 for debugging should only be enabled if the currently selected launch config is of type "node".

That should be the case (and seems to be from the screenshot). It sounds like the enablement isn't quite right. That also seemed like a culprit in https://github.com/microsoft/vscode/issues/71800, where testing using cmd+h seemed to sometimes permanently start capturing that shortcut (though I have since moved to use a different shortcut in testing)


State of AFNetworking

Hi everyone,
I've been upgrading our project to latest version of AFNetworking and encountered number of issues. I am not blaming anyone, I know people are doing this in their free time and I am very thankful for that. I would just like to understand state of AFNetworking library.
Project currently has 183 open issues and 72 PRs.
3 most recent releases were:
3.2.0 - Dec 15, 2017 (No CocoaPods release as of yet)
3.1.0 - Mar 31, 2016
3.0.4 - Dec 18, 2015
There is way less activity then before - https://github.com/AFNetworking/AFNetworking/graphs/contributors
Alamofire (which looks much healthier at the moment) is I would guess part of the reason for this but AFNetworking is still useful I believe for many people.
If someone could clarify state of AFNetworking to community I would be thankful.
Thanks once again all contributors for hard work!

You’re correct that there is more effort behind Alamofire. AFNetworking should be considered to be in maintenance—if there are big issues that come up I’ll try to fix them, but I don’t have the time to invest in large new features. PRs are open if there’s something you want to work on!


Add async workflow integration test step to CI

What changed?
Async workflow integration test is introduced in #5678. This PR is adding it to CI pipelines for both pull request and master.
This new integration test only runs async workflow specific test cases and doesn't run the whole suit. Should finish within a few minutes. It also doesn't generate coverage because we don't need that for integration tests.

How did you test it?
Checked the new CI step running as part of this PR.

Pull Request Test Coverage Report for Build 018dce49-4aac-4c06-8363-21070aa8006c
Details

0 of 0   changed or added relevant lines in 0 files are covered.
76 unchanged lines in 16 files lost coverage.
Overall coverage increased (+0.01%) to 62.787%

Files with Coverage Reduction
New Missed Lines
%

common/task/weighted_round_robin_task_scheduler.go
1
88.06%

common/persistence/historyManager.go
2
66.67%

common/persistence/sql/common.go
2
72.73%

common/persistence/sql/sqlplugin/postgres/db.go
2
85.0%

common/persistence/sql/sqlplugin/postgres/task.go
2
73.4%

service/matching/db.go
2
73.23%

common/log/tag/tags.go
3
50.18%

common/persistence/nosql/nosqlplugin/cassandra/tasks.go
4
76.09%

service/history/task/transfer_standby_task_executor.go
4
87.01%

common/persistence/nosql/nosql_task_store.go
5
61.95%

Totals

Change from base Build 018dce2d-61e6-4ca6-bd29-a148af6ec27c:
0.01%

Covered Lines:
92799

Relevant Lines:
147800

💛  - Coveralls


UI: Allow commit message and metadata in merge action

Closes #6895
Change Description
Background
Allow adding commit message and metadata when merging from UI
Enhance functionality in UI
Testing Details
Tested manually, verified:

Merge works with empty commit message and metadata
Commit message and metadata exists when explicitly adding them in merge

How it looks:

Added random people for review - need only 1


Update documentation

Updates the whole documentation and modifies a little the user interaction example to have two buttons.

Codecov Report

Merging #152 into master will decrease coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #152      +/-   ##
=========================================
- Coverage   96.01%   95.9%   -0.11%
=========================================
  Files          30      30
  Lines         853     855       +2
  Branches      147     148       +1
=========================================
+ Hits          819     820       +1
- Misses         34      35       +1

Impacted Files
Coverage Δ

src/interaction-tracker.ts
96.26% <0%> (-0.94%)
:arrow_down:

src/util.ts
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Update Develocity plugin to 3.17.1

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

The merge queue build has failed. Click here to see all failures.

@bot-gradle test and merge


3d: explicitly fail with .3d files without entrypoint

Suggested by @msprotz at https://github.com/project-everest/everparse/pull/15#discussion_r437676574

https://github.com/project-everest/everparse/pull/25

Thanks @aseemr !


Updating information about an asset when it is first received

When the user makes the first deposit in DemoX DAO Staking, the user deposits the beams and receives the DEMOX assets. The dialog box displays just "ASSET" and should display the name of the DEMOX asset. Most likely, the fact is that this is a new asset for the user's wallet and the wallet does not have information about it and it needs time to figure it out. The wallet manages to understand quickly enough, but the dialog box is already open and is not updated. I think it is worth updating even while the dialog box is open.

Check Blocked by broken dapp contract on master. It Will be checked after the fix

checked


Support compilation for mirage unikernel

I think I need a review from @hannesm but this PR follows what he did on inhabitedtype/bigstringaf#13

And then, if all is good, it's ready to a first release (and decompress too, and ocaml-git, and the world ...)

Merging and release.


Fixed Padding - Alert-Dismissable-cross | Buttons

This pull request is invalid for 2 reasons:

You didn't edit the SCSS files
You added bootstrap.css, that file shouldn't be there

Closing per last comment.

@wolfy1339 - Okay, I'll provide .scss along with .css file.
After all Its my first pull request 😅.

No, don't include changes to the .css file.

From: Amandeep Singh<EMAIL_ADDRESS>Sent: March 26, 2018 9:26
To: twbs/bootstrap
Cc: wolfy1339; Mention
Subject: Re: [twbs/bootstrap] Fixed Padding - Alert-Dismissable-cross | Buttons (#25939)
@wolfy1339https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fwolfy1339&data=02|01||098420c3ecc74025a15b08d5931d2337|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636576675711079205&sdata=rkgelVtcJzYRDlj2zc3p1tPzOEVw4By59IgjdnXtg%2BI%3D&reserved=0 - Okay, I'll provide .scss along with .css file.
After all Its my first pull request 😅.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftwbs%2Fbootstrap%2Fpull%2F25939%23issuecomment-376164502&data=02|01||098420c3ecc74025a15b08d5931d2337|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636576675711079205&sdata=f1qi9vq6ocmEEIZwquX50eigzvKhmhv4G4YwRdH9mfA%3D&reserved=0, or mute the threadhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEYfFc6BWaYRgDgc0A14U3fpTCKy-kkFks5tiOxugaJpZM4S6Rys&data=02|01||098420c3ecc74025a15b08d5931d2337|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636576675711235445&sdata=Tjvj8DHkdDB4lcO992h8gjgLhqHFmVsjg5Z8%2B1ADMoM%3D&reserved=0.

@mdo
Should I edit the .scss files from bootstrap/scss only ?


MM-25003_Improve Onboarding screens for the desktop app - Intro Screen

Summary
Adding Intro Screen:

When starting the desktop app, if no servers are created, user will be prompted with a splash screen that will act as a ‘bridge’ in to the product
There are auto-advancing “slides” on a 5-second timer with controls to manually advance through the slides
There will also be a “Get started” button below that will take them directly to the Add Server screen (which consists of the add server modal).
The app window bar will show 'Mattermost' by default, and the Server dropdown will replace it when servers are created.

Ticket Link
https://mattermost.atlassian.net/browse/MM-25003
Checklist

[ x] Added or updated unit tests (required for all new features)
[x ] Has UI changes
[ x] read and understood our Contributing Guidelines
[x ] completed Mattermost Contributor Agreement
[ x] executed npm run lint:js for proper code formatting

Device Information
This PR was tested on: macOs Big Sur 11.6.6
Screenshots

Dark mode

Light mode

Release Note
Adding Intro Screen when starting the app without servers created.

/update-branch

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/desktop/21772
https://circleci.com/gh/mattermost/desktop/21771
https://circleci.com/gh/mattermost/desktop/21770

Successfully building:
https://circleci.com/gh/mattermost/desktop/21772
https://circleci.com/gh/mattermost/desktop/21771
https://circleci.com/gh/mattermost/desktop/21770

build-linux-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-linux-x64.tar.gz
build-windows-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-ia32.zip
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-x64.zip
build-macos-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-m1.dmg
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-x64.dmg

@matthewbirtch

Can you make sure the 'Get started' button uses Open Sans Semi Bold as the font?

Done.

Can we increase the pause time for each of the slides in the carousel by 1 full second? It just feels not quite long enough.

Done.

If the user interacts with the dots or arrows, can we make it so we stop all auto-advancing?

Done

Can we add -webkit-font-smoothing: antialiased; to make the body text render a little smoother?

Done

When I click on one of the arrow buttons or the dots, a border appears on them. Can we disable that and only use the border when focused via keyboard tabbing for accessibility? I'm hoping we can remove the border when clicking with a mouse.

Done

In the PR description you've shown screenshots of the dark and light modes, but I'm only seeing the dark blue background version in this build. It doesn't seem to be changing when I switch my OS theme between dark and light.

Done.

@matthewbirtch

Is it possible to remove the 'no servers configured' dropdown instead of greying it out? And then just having 'Mattermost' in the middle of the app window bar? Once we have the first server configured then we could bring in the server dropdown and remove the 'Mattermost' title. Let me know what's possible here though.

Checking feasibility.

If I click 'Get started', but choose not add a server and close the app, the next time I open the desktop app I don't get the welcome screen. I'm wondering if we could/should always show the Welcome screen on launch if the first server is not yet configured. Thoughts?

Would this also include when removing all the servers?

It feels a little odd to have the Mattermost logo placed where it is now since there is nothing balancing it on the other side. Wondering if we should anchor it to the top-left corner?

Would this also apply for the New Server screen? Otherwise we could have the logo jumping to the left when going from the welcome to the new server screen.

5. If I click 'Get started', but choose not add a server and close the app, the next time I open the desktop app I don't get the welcome screen. I'm wondering if we could/should always show the Welcome screen on launch if the first server is not yet configured. Thoughts?

Would this also include when removing all the servers?

I think so, I'd be good with bringing back the welcome screen once you remove all servers.

It feels a little odd to have the Mattermost logo placed where it is now since there is nothing balancing it on the other side. Wondering if we should anchor it to the top-left corner?

Would this also apply for the New Server screen? Otherwise we could have the logo jumping to the left when going from the welcome to the new server screen.

Yes, I agree - we don't want the logo to jump around.

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/desktop/21837
https://circleci.com/gh/mattermost/desktop/21836
https://circleci.com/gh/mattermost/desktop/21835

Successfully building: https://circleci.com/gh/mattermost/desktop/21837 https://circleci.com/gh/mattermost/desktop/21836 https://circleci.com/gh/mattermost/desktop/21835

build-linux-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-linux-x64.tar.gz
build-windows-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-ia32.zip
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-x64.zip
build-mac-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-m1.dmg
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-x64.dmg

@matthewbirtch All feedback has been addressed.

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/desktop/21872
https://circleci.com/gh/mattermost/desktop/21871
https://circleci.com/gh/mattermost/desktop/21870

Successfully building: https://circleci.com/gh/mattermost/desktop/21872 https://circleci.com/gh/mattermost/desktop/21871 https://circleci.com/gh/mattermost/desktop/21870

build-linux-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-linux-x64.tar.gz
build-windows-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-ia32.zip
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-x64.zip
build-mac-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-m1.dmg
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-x64.dmg

@devinbinnie @saturninoabril Test cases updated with corresponding Zephyr Scale. Thanks to @stevemudie!

@julmondragon Ran into two more issues while testing:

There's a little '0' visible when the Welcome Screen is visible.

On Windows (and presumably Linux) you can't open the 3-dot menu when there are no servers configured. This should definitely be allowed.

Building app in separate branch.

Successfully building: https://circleci.com/gh/mattermost/desktop/21970 https://circleci.com/gh/mattermost/desktop/21969 https://circleci.com/gh/mattermost/desktop/21968

build-windows-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-ia32.zip
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-win-x64.zip
build-linux-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-linux-x64.tar.gz
build-mac-pr:
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-m1.dmg
tmp/artifacts/mattermost-desktop-5.2.0-develop.1-mac-x64.dmg

@julmondragon Ran into two more issues while testing:

There's a little '0' visible when the Welcome Screen is visible.

On Windows (and presumably Linux) you can't open the 3-dot menu when there are no servers configured. This should definitely be allowed.

@devinbinnie  Solved both issues.

Thanks @julmondragon, I'll unblock this as all major issues are resolved. 2 more that I found that can be addressed at a later date:

Opening the 3-dot menu and going to Windows > Show Servers pops the dropdown menu even though it should be disabled.
On Windows, when you click the 3-dot menu and then click off of it, the button stays focused no matter where else you click

Thank you @devinbinnie !
Regarding the two issues, it looks like these are present in v5.1.1, should then be handled in bug tickets?

Thanks @julmondragon, I'll unblock this as all major issues are resolved. 2 more that I found that can be addressed at a later date:

Opening the 3-dot menu and going to Windows > Show Servers pops the dropdown menu even though it should be disabled.
On Windows, when you click the 3-dot menu and then click off of it, the button stays focused no matter where else you click

Thank you @devinbinnie ! Regarding the two issues, it looks like these are present in v5.1.1, should then be handled in bug tickets?

Yeah those would be fine.


ConsulSDConfig support for filter field

In docs https://docs.victoriametrics.com/sd_configs/#consul_sd_configs mentioned, that vmagent supports filter field, but operator don't - operator.victoriametrics.com/v1beta1, Kind=VMScrapeConfig): .spec.consulSDConfigs[0].filter: field not declared in schema.
Can this support be added?

The issue was fixed at v0.51.1 release


Add flash mode setting

Thanks for creating this project~

:beer:

Thanks for this, there is another PR for similar functionality but I feel as though this fits better with the current API.


PowerToys Run Unable to load DLL

ℹ Computer information

PowerToys version: <IP_ADDRESS>
PowerToy Utility: Power Toys Run
Running PowerToys as Admin: True
Windows build number: Microsoft Windows NT 10.0.19042.0

📝 Provide detailed reproduction steps (if any)

… Left PC on standby

❌ Actual result
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.19042.0
IntPtr Length: 8
x64: True
Date: 15/01/2021 00:10:56
Exception:
System.DllNotFoundException: Unable to load DLL 'vcruntime140_cor3.dll' or one of its dependencies: The specified module could not be found. (0x8007007E)
   at __std_type_info_destroy_list(__type_info_node* )
   at __scrt_uninitialize_type_info()
   at _app_exit_callback()
   at <CrtImplementationDetails>.LanguageSupport.DomainUnload(Object A_0, EventArgs A_1)
   at <CrtImplementationDetails>.ModuleUninitializer.SingletonDomainUnload(Object source, EventArgs arguments)
   at System.AppContext.OnProcessExit()

@otaconjh
open a command prompt and run "%PROGRAMFILES%\PowerToys\Tools\BugReportTool.exe" (the double quotes are important)
that will create a zip file on you desktop named like this PowerToysReport_2021-01-04-12-02-08.zip
You can drag and drop it in a GitHub comment.

@otaconjh
open a command prompt and run "%PROGRAMFILES%\PowerToys\Tools\BugReportTool.exe" (the double quotes are important)
that will create a zip file on you desktop named like this PowerToysReport_2021-01-04-12-02-08.zip
You can drag and drop it in a GitHub comment.

PowerToysReport_2021-01-19-11-59-12.zip

PowerToysReport_2021-01-19-11-59-12.zip

From log we can say that something caused application to terminate and during that System.DllNotFoundException was thrown.

From log we can say that something caused application to terminate and during that System.DllNotFoundException was thrown.

@otaconjh
have you updated to 0.33.1? Is the problem still happening?