Unable to find type [Microsoft.PowerShell.Commands.WebRequestSession]

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
[Microsoft.PowerShell.Commands.WebRequestSession]::New()
#InvalidOperation: Unable to find type [Microsoft.PowerShell.Commands.WebRequestSession].
new-object Microsoft.PowerShell.Commands.WebRequestSession
<#Headers                : {}
Cookies                : System.Net.CookieContainer
UseDefaultCredentials  : False
Credentials            :
Certificates           :
UserAgent              : Mozilla/5.0 (Windows NT 10.0; Microsoft Windows 10.0.19045; it-IT) PowerShell/7.3.3
Proxy                  :
MaximumRedirection     : -1
MaximumRetryCount      : 0
RetryIntervalInSeconds : 0
#>

Expected behavior
They should both work

Actual behavior
`[Microsoft.PowerShell.Commands.WebRequestSession]::New()` doesn't work (it worked in PS 5)

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.3.3
PSEdition                      Core
GitCommitId                    7.3.3
OS                             Microsoft Windows 10.0.19045
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Windows Store version
Visuals
No response

In my environment all woks as expected.

This works for me too.

I found out it works if I input the two commands in reverse order
new-object Microsoft.PowerShell.Commands.WebRequestSession
<#Headers                : {}
Cookies                : System.Net.CookieContainer
UseDefaultCredentials  : False
Credentials            :
Certificates           :
UserAgent              : Mozilla/5.0 (Windows NT 10.0; Microsoft Windows 10.0.19045; it-IT) PowerShell/7.3.3
Proxy                  :
MaximumRedirection     : -1
MaximumRetryCount      : 0
RetryIntervalInSeconds : 0
#>
[Microsoft.PowerShell.Commands.WebRequestSession]::New()
<#Headers                : {}
Cookies                : System.Net.CookieContainer
UseDefaultCredentials  : False
Credentials            :
Certificates           :
UserAgent              : Mozilla/5.0 (Windows NT 10.0; Microsoft Windows 10.0.19045; it-IT) PowerShell/7.3.3
Proxy                  :
MaximumRedirection     : -1
MaximumRetryCount      : 0
RetryIntervalInSeconds : 0
#>

I guess you should import Utility module before.

it works correctly when using Import-Module Microsoft.PowerShell.Utility. I didn't have this problem in PS 5 what could be the origin of the issue?

It seems Windows PowerShell loads Utility module from start.

Should we do something about this? Or should I just close the issue?

Should we do something about this? Or should I just close the issue?

Let's wait MSFT team conclusion.
/cc @daxian-dbw @SeeminglyScience

It's the same behavior on Windows PowerShell 5.1, as long as the Utility module is not loaded:

So, I'm inclined to treat this issue as by design.

Thanks (I must have changed somethin in my PS 5), I'll close


style navbar

Describe the bug
A clear and concise description of what the bug is.
Scrolling navbar leads to blurness and contents are not visible

@Muskan-creator update ?


Fp16 bug HBH

Keep the alpha type as fp32 for HBH case.

Offline testing of this PR with rocBLAS-internal #1054 passed rocblas-test fully on gfx906, gfx908 and gfx90a.  It is ready for review.  Thanks.

@zaliu- On @cgmillette remarks: I already talked to Alex about the two concerns you mentioned. For the scope of this task, we decided not to move HBH to HHS_BH. I would be glad to continue working on the two concerns.

@benjaminulmer On your comments:
3-..... This PR and the corresponding rocBLAS one are NOT meant to finally have rocBLAS use the proper HHS_BH kernels, it fixes the wrong behavior of HBH, by not converting alpha/beta to F16 and back to F32. rocBLAS logic files are still HBH, and no HHS_BH will be added until we fully migrate HBH to HHS_BH.
Removes support for HHH + HPA kernels .... . The PR does not remove support for HBH.


Derive same attributes as MainEntity for RenderEntity

Spotted while working on updating bevy_egui.
Discord context: https://discordapp.com/channels/691052431525675048/1301212128115687454/1301469954465464320

@alice-i-cecile What happened here?

This was targeting the wrong branch; a sibling PR was made and merged.


Add missing migration following status choices changes

No real changes - the choices are actually the same, only the description change.


alpha0.5 producing incorrect output compared to README.md

With a system where tests are passing, we are not getting sensible output from the examples.
This is most concerning and the highest priority issue to fix right now.
type() not working:
>>> net = P.Worm().get_neuron_network()
>>> list(net.aneuron('AVAL').type())
[]
>>> list(net.aneuron('DD5').type())
[]

wrong receptors coming out:
FROM README:
>>>list(net.aneuron('AVAL').receptors())
['GLR-1', 'NMR-1', 'GLR-4', 'GLR-2', 'GGR-3', 'UNC-8', 'GLR-5', 'NMR-2']

ACTUAL:
>>> list(net.aneuron('AVAL').receptors())
['FLP-18', 'FLP-1']

Readme says this will be True
>>> 'MDL08' in P.Worm().muscles()
False

No receptors coming out even though we say there are:
FROM README:
>>>ader = P.Neuron('ADER')
>>>list(ader.receptors())
['ACR-16', 'TYRA-3', 'DOP-2', 'EXP-1']

ACTUAL:
>>> ader = P.Neuron('ADER')
>>> list(ader.receptors())
[]

Tested all these cases today and they work now!  Great job @mwatts15 !!


Error on parsing X-Instana-S and X-Instana-T

X-Instana-S and X-Instana-T headers may contain values between 0 and math.MaxUint64, but textMapPropagator.extract() uses strconv.ParseInt() to parse these headers instead of strconv.ParseUint().
This error cannot be observed if a golang service receives a request from another golang service, because instana/golang-sensor just generates IDs between 0 and math.MaxInt64 by using Rand.Int63(), so these IDs will be parsed well.

Hi PSanetra!  Instana actually generates and supports IDs with the range -9223372036854775808 to<PHONE_NUMBER>854775807 (signed 64 bit integer) which is why we use strconv.ParseInt() instead of the UInt variant.
With that said, we should have some tests in the package that confirm the translation from base10 IDs to their base16 equivalents.
Does this explain/clarify the issue at all?  Let me know if I missed something.

I tested a golang service, which received requests from a spring boot application. The spring boot application got automatically instrumented by instana and transmitted IDs, which were greater than math.MaxIn64 (for example 938a406416457535).

ok, I'm closing this issue since the actual problem is the automatic JVM instrumentation code, which serializes negative IDs as positive hex values. If the golang-sensor would parse the hex value with strconv.ParseUint(), it would return a wrong ID. Should I open another issue somewhere else?

this not an issue @PSanetra. the ID is opaque it can be anything in the storage. it is 64 bit. a 64 bit wide sequence of 1 and 0. The wire format is defined as unsigned hex encoded. The internal format is undefined.

Like @CodingFabian said - it's not the JVM code but an issue for this golang sensor.  I just spent some time looking into this and it's not properly converting incoming hex values.
It should treat incoming/outgoing on-the-wire hex values as unsigned hex when converting (which appears not to be the case currently).  So it was a good find :-)   Much appreciated.

Fixed and added a good amount of tests for ongoing validation.  Going to merge for more testing and will tag a release soon.  Thanks again @PSanetra!


Icon

That hopefully isn't just a screenshot

Any ideas?

It is unfortunate that the vanilla "ready" indicator already includes a plus, but I was thinking that with more pronounced colors.

Oh damn, that looks good. Not sure if I'd be tacky m, but maybe have the + be gold?

Sure.

Hmm, now that I'm at my pc, I'll try a more yellow-gold, and if depending on how that looks, I'll either use that, or the green one

Okay, this is will be the icon. I added more spacing between the sword and plus (4px), and then moved the whole thing up (2px). And thanks to tinypng, it's only 232 bytes!


Updates to retry definition

Many thanks for submitting your Pull Request :heart:!
Please specify parts this PR updates:

[ x] Specification
[ x] Schema
[ x] Examples
[ ] Extensions
[ ] Roadmap
[ ] Use Cases
[ ] Community
[ ] TCK
[ ] Other

What this PR does / why we need it:
Address requested changes in #137 & #136
Special notes for reviewers:
Additional information (if needed):

@jorgenj thanks for pr!

@jorgenj thanks for pr!

@jorgenj please also add a row to the roadmap/README.md file that points out this update. Thanks.

@jorgenj please also add a row to the roadmap/README.md file that points out this update. Thanks.

@jorgenj thanks for making the updates! It seems the DCO check is failing. Commit needs to be signed.

@jorgenj thanks for making the updates! It seems the DCO check is failing. Commit needs to be signed.


Use setuptools_scm for versioning

This PR implements versioning using setuptools_scm, resolving issue #25 (lack of good version numbering). This is similar to, but simpler than, PINT's use of Versioneer. I had been looking into using setuptools_scm for a different project and remembered that we had never settled the versioning issue for PINT Pal, so I decided to make this PR.
With this in place, you can get the version number at run time as either pint_pal.version or pint_pal.__version__. This version number is based on the tags in git -- setuptools_scm searches back through the git history for the most recent tagged commit, and builds the version based on that. If the current commit is the tagged one, it just uses the tag as the version number, and otherwise it tries to guess the next version number and adds .dev{distance}+g{hash} to the end of it, based on the distance from the tagged commit and the current commit hash. For example, the current version reports as 0.1.1.dev16+g950bd5b.

Closing in favor of #27.


NATIVE-587: Use release notes

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword NATIVE.

ready for merge :)


Getting Rate Limiting Statistics

As part of librdkafka, a throttle object is available from the statistics messages returned. this allows clients to detect when they are being rate limited by the client.
Is there a way to achieve this in, kgo? While the metrics are good, I would like to log when the client is receiving non zero throttle response times from the broker, to make it easier to quickly identify the issue is rate limiting.

https://pkg.go.dev/github.com/twmb/franz-go/pkg/kgo#HookBrokerThrottle
You can use this hook


gitignore Tycho pomless temp files

During a Maven build those files are shown in the staging view.

@Bananeweizen thanks!


Libpitt/156 status

Change log:

Add Incomplete status and theme
Adjust tooltip text to specs from https://github.com/sennetconsortium/portal-ui/issues/1197

@yuanzhou This update is related to a request Brendan made on the SenNet side for a new status called "Incomplete". No changes need to be made on the HuBMAP side. This PR adds styling to look like this:

@maxsibilla thanks for the heads-up. Since HuBMAP doesn't have this incomplete status, please feel free to merge and release on the SenNet side. I won't need to redeploy or update the config on the HuBMAP side.


Featues

Also, I kind of liked the "normal" look on the current deployed storybook (light grey and such).
We could also use that addon for storybook that allows us to change the title and link on the top left.
Another cool addon to add here is storysource


Add Air-Gapped documentation

Adds Air-Gapped documentation
How to test when there is no internet access
fixes #39
Signed-off-by: Vicente Zepeda Mas<EMAIL_ADDRESS>

Looks great!
Thank you for the contribution

Thank you for the project it is going to be really useful!


Web - Attachments Preview - Zooming in the edges of an image will lead to white spaces

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:

Open any chat
Send an image as attachment
Click on it in order to have larger preview
Click on any edges in order to zoom it

Expected Result:
Picture is zoomed and picture fills whole preview dialogue
Actual Result:
Picture is zoomed, but there is a lot of white spaces
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:
1.0.75-5
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/124826208-9379ca00-df42-11eb-9566-11d31750b082.mp4

Expensify/Expensify Issue URL: https://github.com/Expensify/Expensify/issues/170015
Upwork job posting: Posted to Upwork: https://www.upwork.com/jobs/~014d8cadbc82d362b0
View all open jobs on Upwork

Triggered auto assignment to @francoisl (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Looks like this can be external

Triggered auto assignment to @NicMendonca (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

What's the expected behavior here? Looks like the app centers the image on the point we're zooming in.
https://github.com/Expensify/Expensify.cash/blob/f11d808f6358ba57ae02a1d392229dbd38491afd/src/components/ImageView/index.js#L95-L104
I feel like this is the correct behavior, but it also means that if we zoom on the edge there will be blank spots.

@rdjuric that question might be better suited for #expensify-open-source.

Triggered auto assignment to @roryabraham (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

What's the expected behavior here

My thinking here is that we should center the image on the point we're zooming in on, but if we're within a specific distance of any edge, then we'd shift the zoomed-in section accordingly. Something like this:

Issue reproducible during KI retests

Hi @rdjuric! Based off @roryabraham's comment, are you keen to submit a proposal for this?

@NicMendonca Still researching this one 😅
Thanks for the picture, @roryabraham. So I think the behavior we want is what we have in the video below?
https://user-images.githubusercontent.com/53711423/125832688-777fcb5c-916b-4aff-9de5-7c274ad0c898.mov

It looks like it might not be working perfectly in the y-axis, but yeah that's prettymuch what I had in mind.

I have to reassign this to someone else while I'm OOO

Triggered auto assignment to @tgolen (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Issue reproducible during KI retests

@rdjuric Could you submit a proposal for this, please? It looks like the behavior in the video is pretty close to what's desired.

@rdjuric bump! Please let us know if you are still interested in submitting a proposal.

Issue reproducible during KI retests

@tgolen I'm back from OOO, so I can take this back now.

New Upwork job posting. Doubled the price since there hasn't been any proposals in a week: https://www.upwork.com/jobs/~0134dee8788c30ccc5

@roryabraham Can you help me a bit more here?
Looks like there're more issues at play

An image that is smaller than the size of the viewport isn't being zoomed
The zoom is done on the View that surrounds the image, so it doesn't correctly give a point in the image to zoom in.

Point (1) happens because we're "zooming" by increasing the height/width of our Image component to 250% while using resizeMode="center", this will only increase the size of the image if it was being scaled down by the resize - i.e, only when the image isn't bigger than the Image component.
Assuming that we fix (1), I don't think it's possible to "not show any blank spaces around the image" if our zoom is constant (like the 2.5x we use here) and our AttachmentModal is bigger than the image. Imagine that we have an image with size 100px x 100px, we zoom it so it's 250px x 250px. If our AttachmentModal (which has a size based on the window dimension) is bigger than 250px x 250px we're necessarily going to have white spaces around the image after zooming.
If we really don't want white spaces around the image, I think we must either:

Make the size of our AttachmentModal equal to the size of the attachment it's rendering
Change our zoom to always make the image fill the viewport. This would lead to different levels of zoom depending on the original size of the image.

@rdjuric Thanks for your investigation! Which platform did you observe these issues on? It seems like you're mostly talking about the web/desktop implementation, right? I notice we're using this react-native-image-zoom library on native, and while I'm not too familiar with this piece of the codebase I wonder if these problems you've described exist only in the web/desktop implementation?

this will only increase the size of the image if it was being scaled down by the resize

I would like to clarify my understanding of this issue with a small example. Let's assume we have an image that's 2x2 (four pixels of different colors), and a view that's 4x4. Then suppose we apply the style: {width: '200%', height: '200%'} – You're saying that, because the original size of the image is 2x2, RN will not scale the image up to 4x4? Is that correct?
If so, I think we'd maybe want to start with a "true" zoom that scales pixels:

"Fake" Zoom we have now
"True" zoom

2x2 image, 4x4 view

I'm sure there's some libraries that do exactly that (maybe even with some fancy interpolation to prevent the image from looking super blocky)

Make the size of our AttachmentModal equal to the size of the attachment it's rendering

Definitely don't want to do this.

Change our zoom to always make the image fill the viewport. This would lead to different levels of zoom depending on the original size of the image.

I think this makes sense to do, but I think before we can really discuss the solution here, I think that we would need to address how we're going to fix (1) right?
To be honest, I'm not sure I 100% understand your last comment, but here's my attempt at a summary of the features that would improve the image-zoom experience:

A "true" zoom
Variable scaling percentages to help smaller images take up more space in a large attachment view
The "gutter shifting" described in this comment

Does that sound correct?

As an aside, it seems like have pretty disparate implementations for ImageView on native v.s. web – I would love to someday see these implementations unified a bit if possible. Maybe a web-compatibility PR for react-native-image-zoom?

Issue reproducible during KI retests

@rdjuric can you confirm if you are interested in opening a PR for this?

Hey @NicMendonca!
I'm busy with other PR's and this one looks a bit complex, so I'm just helping us refine our understanding of the problem for now. If anyone wanna take this, please feel free to do so!

Issue reproducible during KI retests

No new proposals here

Issue reproducible during KI retests

Still no new activity here.

Sorry for the delay here y'all. I have doubled the price again. It's now set as $1,000 New job URL: https://www.upwork.com/jobs/~014d8cadbc82d362b0
I see your 👀 @parasharrajat 👀

I think it's layout issue. My opinion need to create new component with customized design and function and replace that one or modify original component
If you share source code, i can customize and make pull request
--- From Shane Watson-----

isZoom?

:

We can pass props isZoom and then activate zoom mode. Sorry it's rough logic but hope it will help.

As you can see screenshot, when zoom in, it is changing width to 250% and height to 250%

This is code for extends percent of width and height

I think need to change it to 'stretch' from 'resize' to 'stretch'
I hope it will work
---------- Shane Watson Upwork ---------------

https://user-images.githubusercontent.com/12084368/129867710-286dd60d-1baf-4f0a-b30d-e267388e3e20.mov

It's not solution @parasharrajat ?

Ok let me create video again with slow mode

https://user-images.githubusercontent.com/12084368/129905689-c9b4724b-4d51-4f62-aebc-70f43ea24896.mov
Please check this video again

Any feedback @parasharrajat ?

Better than the earlier video, I leave up to the engineer assigned to evaluate.

Thanks @parasharrajat

If you ok, i will create PR for it

@MTN718 it doesn't seem to me like you've solved the problem. Try again with an image with a more extreme aspect ratio, and you'll see what I mean. Also, I don't think we'll want to use stretch-resizing.

Ok. if you don't want use stretch, to keep aspect, another my idea is make attachment view size following image ratio

Attachment View Size should be fixed?

Please give me feedback video @roryabraham

I Found Solution :)
Logic Following:

Get Image Size in AttachmentView
Send parameter to Style for Zoom
Calculate width/height for zoom and set Zoom size
Adding Screenshot

This is final Video working
https://user-images.githubusercontent.com/12084368/130024591-b7d4bc42-253b-4770-8a96-02d0e003a87c.mov

This does not work properly. ImageWidth is calculated based on the wrapper. for example here we have 100% of the wrapper so image width is equal to the modal insets which is not how it should be. Original rendered are of image is far less than the modal insets. Hope it helps. I am also sticking my head around to get that working.

so problem is before zoom, blank area?

currently get image size before zoom and calculate size only when zoom image so i think it can be working

As you can see screenshot, image width/height is 576,600px but modal size is 1227*737px

Yeah, then it makes sense.

Yeah, then it makes sense.

Do you agree it's solution for issue?

I don't see anything proposed yet. It seems more of an analysis. Could you please post a formal proposal? you can look at contribution guidelines to learn about creating a proposal.

Proposal (Already submitted on upwork)
My name is Shane Watson from South Africa.
I suggest a way to fix issue following

Get Image Size when attachment view component mounted
Passing Image width/height parameter for zoom style
Calculate scale base on max width
Create new zoom width/height instead of 250% of view
Change Image resize mode property to "contains"

I already tested and you can see result above

Thanks for your analysis @MTN718. I think what you've done so far make sense, but it's very hard to tell from your screenshots because the image you've chosen has a white background. Can you please post a screenshot with a dark background so it's easy to see that the whitespace issue is eliminated? Thanks!

Added Videos for different size of images
https://user-images.githubusercontent.com/12084368/130188766-81c7b742-b919-4edc-affd-7009122a4382.mov
https://user-images.githubusercontent.com/12084368/130188829-abdeb72d-ce66-4f9f-bcb1-848acf5beaec.mov
https://user-images.githubusercontent.com/12084368/130188885-be29711d-e0b9-4273-9e98-8be6d2301896.mov

@MTN718 Looks great! Thanks for submitting those extra videos. @NicMendonca please hire @MTN718 for this issue on Upwork, and @MTN718 feel free to submit a pull request with the code changes as soon as you've been hired on Upwork.

My Upwork account name is Shane Watson from South Africa

@MTN718 hired!

Thanks. Let me create PR

Created PR. Waiting Feedback

Issue reproducible during KI retests

You mean issue again from PR?

PR is not merged yet, but I just re-reviewed it.

Issue reproducible during KI retests

@roryabraham my PR deployed in current KI retests?

Not yet

🚀 Deployed to staging by @robertjchen in version: 1.0.88-3 🚀

platform
result

🤖 android 🤖
cancelled 🔪

🖥 desktop 🖥
cancelled 🔪

🍎 iOS 🍎
cancelled 🔪

🕸 web 🕸
success ✅

🚀 Deployed to production by @roryabraham in version: 1.0.90-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

@NicMendonca Issue resolved?

@NicMendonca @roryabraham any update on upwork?

paid!


IE11 support

The demo doesn't work on IE11.

Thanks for bringing this to my attention, I'll look into it further and try to isolate the issue.

It’s a WebKit issue, I’ll release a fix and bump the version later today.


NetDocuments.ndSync version <IP_ADDRESS>71

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run

Manual Validation ended with:

NetDocuments® Terms of Use Acceptance of the TERMS OF USE THESE NETDOCUMENTS TERMS OF USE (THESE “TERMS OF USE”) STATE THE TERMS AND CONDITIONS UPON WHICH ACCESS TO OR USE OF THIS WEBSITE, THE NETDOCUMENTS SERVICE AND RELATED SERVICES (THE "SERVICE") OFFERED BY NETDOCUMENTS SOFTWARE INC. (“NETDOCUMENTS”) MAY BE PROVIDED TO INDIVIDUALS WHO USE OR SEEK TO USE THE SERVICE (EACH, A “USER” OR “YOU”).   ANY USE OF THE SERVICE IS SUBJECT TO THE TERMS OF CONDITIONS OF THE SERVICE AGREEMENT OF THE AUTHORIZING CUSTOMER, MAY BE TERMINATED BY THE AUTHORIZING CUSTOMER AT ANY TIME FOR ANY OR NO REASON AND MAY BE TERMINATED BY NETDOCUMENTS AS PROVIDED IN THESE TERMS OF USE. OTHER THAN AS EXPRESSLY PROVIDED HEREIN, THESE TERMS OF USE IMPOSE NO OBLIGATIONS ON NETDOCUMENTS TO ANY USER OR THE AUTHORIZING CUSTOMER, AND NO USER SHALL ACQUIRE ANY RIGHT AGAINST NETDOCUMENTS BY VIRTUE OF THESE TERMS OF USE OR THE USER’S USE OF THIS WEBSITE OR THE SERVICE. YOU MAY NOT USE THIS WEBSITE OR THE SERVICES WITHOUT ACCEPTING THESE TERMS OF USE.  Please read these Terms of Use carefully before your use any aspect of the Service.  By using www.netdocuments.com (including all subpages, this “Website”) or by clicking to accept or agree to these Terms of Use when the option is made available to you, you accept and agree to be bound and abide by these Terms of Use and our Privacy Policy found at http://www.netdocuments.com/en-us/TermsConditions/PrivacyPolicy and incorporated herein by reference. If you do not want to agree to these Terms of Use, you may not use this Website or the Service. These Terms of Use are in addition to any terms, conditions, representations, warranties and covenants in the Service Agreement with the Authorizing Customer.   This Website and the Service is offered and available to users who are 18 years of age or older, reside in the United States and have either been provided a username and password by an Authorizing Customer or are viewing the Website in order to evaluate a possible subscription to the Service. By using this Website or the Service, you represent and warrant that you are of legal age to form a binding contract with NetDocuments and meet all of the foregoing eligibility requirements. If you do not meet all of these requirements, you must not access or use the Website or the Service.   Changes to the Terms of Use  Subject to any restrictions in the Service Agreement, NetDocuments may revise and update these Terms of Use from time to time in its sole discretion. All changes to these Terms of Use are effective immediately when we post them and apply to all access and use of the Website and the Service thereafter. NetDocuments may require acceptance on the Website of such modified Terms of Use as a condition to any Users’ initial use or continued use of the Website and the Service. Notwithstanding the foregoing, your continued use of the Website and the Service shall constitute acceptance of any revisions and updates to these Terms of Use. You are expected to check this page from time to time so that you are aware of any changes these Terms of Use, as they are binding on you.  Accessing the Service and Account Security Subject to any limitations in the Service Agreement, NetDocuments reserves the right to modify the Website and the Service in its sole discretion without notice. NetDocuments will not be liable to any User if for any reason all or any part of the Website and the Service is unavailable at any time or for any period. From time to time, NetDocuments may restrict access to some parts of Website and  the Service, or the entire Website and the Service, to Users for scheduled maintenance or for other purposes.  You acknowledge that use of the Service requires compliance with certain minimum requirements for hardware, systems software and Internet browsers for User workstations and the download of certain desktop applications software on you workstation, as well as Internet connectivity. Such minimum recommended levels can be found at https://vault.netvoyage.com/neWeb2/helpRedirect.aspx?topic=service-requirements and may be amended from time to time by NetDocuments. Your failure to comply with these requirements may cause the Service not function at all or to function poorly. You are responsible for ensuring that you systems meet all minimum requirements, as modified from time to time.   To access the Service or some of the resources it offers, Users may be asked to provide certain registration details or other information. It is a condition of your use of the Service that all the information you provide on the Service is correct, current and complete, and you represent that all such information will be correct, current and complete. You agree that all information you provide to register with the Service or otherwise, including but not limited to through the use of any interactive features on the Service, is governed by our Privacy Policy at http://www.netdocuments.com/en-us/TermsConditions/PrivacyPolicy, which NetDocuments may update from time to time in its discretion, and you consent to all actions NetDocuments takes consistent with our Privacy Policy.   If you are provided with a user name, password or any other piece of information as part of our security procedures, you must treat such information as confidential and must not provide third parties with, or permit third parties to use, user names, passwords or other security information. You agree not to permit any other person to authorize the account associated with your user name and password (sometimes referred to as your “account”) and not to use the account of, or to impersonate, any other person. You agree to notify, or to cause the Authorizing Customer to notify, NetDocuments immediately of any unauthorized access to or use of your user name or password or any other breach of security with respect to the Service involving your account. You also agree to ensure that you exit from your account at the end of each session. Users should use particular caution when accessing account from a public or shared computer so that others are not able to view or record their respective password or other personal information.  NetDocuments shall have the right to disable any user name, password or other identifier, at any time in our sole discretion for any or no reason, including if, in our opinion, any User or Authorizing Customer has violated any provision of these Terms of Use.   Users to Look to Authorizing Customer for Support  NetDocuments shall have no obligation to provide support to any User with respect to the Service. Users shall direct requests for support to the IT personnel or help desk of the Authorizing Customer and shall not contact NetDocuments directly to request Support. If NetDocuments directly or indirectly provides any support to a User, no standards of service or other duties shall govern such support.  Deletion of Documents upon Termination of Service PRIOR TO THE TERMINATION OF THE AUTHORIZING CUSTOMER’S SERVICE AGREEMENT, IT IS THE OBLIGATION OF USER AND/OR AUTHORIZING CUSTOMER TO MAKE ANY REQUIRED COPIES OF DOCUMENTS AND DATA IN THE REPOSITORY OF THE AUTHORIZING CUSTOMER (“CUSTOMER DATA”, AND/OR TO TRANSITION SUCH CUSTOMER DATA ANOTHER DATABASE OR SERVICE.     Intellectual Property Rights The Service and its entire contents, features and functionality are owned by the NetDocuments, its licensors or other providers of such material and are protected by United States and international copyright, trademark, patent, trade secret and/or other intellectual property or proprietary rights laws. No right, title or interest in or to the Service or any rights associated with the service is transferred to Users. Users shall not copy, modify, sublicense, distribute, rent, lease, convey, translate, disassemble, reverse engineer or decompile any portion of the Service and related NetDocuments software and other intellectual property. Any use of the Service not expressly permitted by these Terms of Use is a breach of these Terms of Use and may violate copyright, trademark and other laws. If any User violates these Terms of Use, the rights of such User may be terminated immediately.  Trademarks The marks NetDocuments® and all related names, logos, products and service names, designs and slogans are trademarks of NetDocuments or its affiliates or licensors. You must not use such marks without the prior written permission of NetDocuments. All other names, logos, product and service names, designs and slogans on this Service or the Website (the “Website”) are the trademarks of their respective owners.  Prohibited Uses You may use the Service only for lawful purposes and in accordance with these Terms of Use. You agree not to use the Service:  • in any way that violates any applicable federal, state, local or international law or regulation (including, without limitation, any laws regarding the export of data or software to and from the US or other countries);  • to impersonate or attempt to impersonate NetDocuments, a NetDocuments employee, another user or any other person or entity; and • to engage in any other conduct that restricts or inhibits anyone's use or enjoyment of the Service, or which, as determined by us, may harm NetDocuments or users of the Service or expose them to liability. Additionally, you agree not to:  • use the Service in any manner that could disable, overburden, damage, or impair the Service or interfere with any other party's use of the Service, including their ability to engage in real time activities through the Service; • use any robot, spider or other automatic device, process or means to access the Service for any purpose, including monitoring or copying any of the material from the Service; • use any device, software or routine that interferes with the proper working of the Service; • introduce any viruses, trojan horses, worms, logic bombs or other material which is malicious or technologically harmful; • attempt to gain unauthorized access to, interfere with, damage or disrupt any parts of the Service, the server on which the Service is stored or the Website, or any server, computer or database connected to the Service;  • attack the Service via a denial-of-service attack or a distributed denial-of-service attack; or • otherwise attempt to interfere with the proper working of the Service. Legal Demand for Authorizing Customer and User Information NetDocuments disclaims any interest in the Customer Data. NetDocuments may from time to time receive a subpoena, order from a court, tribunal or government entity with jurisdiction or other legally binding demand (collectively, a “Legal Demand”) to obtain access to or copies of Customer Data. Upon such receipt, NetDocuments will, unless prohibited by the terms of the Legal Demand or governing law, promptly provide written notice to the Authorizing Customer, not to any Users, of such Legal Demand. To the extent the Authorizing Customer does not eliminate the legal efficacy, by means of a successful motion to quash or other action of the Legal Demand, NetDocuments may comply with the Legal Demand. This paragraph is not intended to, and does not, create a duty on the part of NetDocuments to preserve Customer Data if there is threatened or pending litigation, which shall be the responsibility of the Authorizing Customer and the User. NETDOCUMENTS WILL NOT BE LIABLE FOR ANY LOSS OR DAMAGE CAUSED BY NETDOCUMENTS ‘S PRODUCTION OF CUSTOMER DATA, OR ANY OTHER ACTION OF OMISSION, IN COMPLIANCE WITH THIS PARAGRAPH.  Reliance on Information Posted  The information presented on the Website or through the Service is made available solely for general information purposes. NetDocuments does not warrant the accuracy, completeness or usefulness of this information.   Changes to Website  NetDocuments may update the content on this Website from time to time in its discretion, but its content is not necessarily complete or up-to-date. Any of the material on this Website may be out of date at any given time, and we are under no obligation to update such material.  Linking to the Service and Social Media Features You may link to our homepage, provided you do so in a way that is fair and legal and does not damage our reputation or take advantage of it, but you must not establish a link in such a way as to suggest any form of association, approval or endorsement on our part without our express written consent.  Links from the Service If the Service or the Website contains links to other sites and resources provided by third parties, these links are provided for your convenience only. This includes links contained in advertisements, including banner advertisements and sponsored links. NetDocuments has no control over the contents of those sites or resources, and accepts no responsibility for them or for any loss or damage that may arise from your use of them. If you decide to access any of the third party websites linked to the Service or the Website, you do so entirely at your own risk and subject to the Terms of Use of use for such websites.  Geographic Restrictions NetDocuments is based in the state of Utah in the United States. NetDocuments provides the Service through this Website for use only by persons located in the United States. NetDocuments make no claims that the Service or any of its content is accessible or appropriate outside of the United States. Access to the Service may not be legal by certain persons or in certain countries. If you access the Service from outside the United States, you do so on your own initiative and are responsible for compliance with local laws.  Disclaimer of Warranties You acknowledge and agree that NetDocuments cannot and does not guarantee or warrant that files available for downloading from the Website or the Service, or any Customer Data, will be free of viruses or other destructive code. You are responsible for implementing sufficient procedures and checkpoints to satisfy your particular requirements for anti-virus protection and accuracy of data input and output, and for maintaining a means external to our site for any reconstruction of any lost data.    NEITHER NETDOCUMENTS NOR ANY PERSON ASSOCIATED WITH NETDOCUMENTS MAKES ANY WARRANTY OR REPRESENTATION TO ANY USER WITH RESPECT TO THE COMPLETENESS, SECURITY, RELIABILITY, QUALITY, FUNCTIONALITY OR AVAILABILITY OF THE SERVICE. WITHOUT LIMITING THE FOREGOING, NEITHER NETDOCUMENTS NOR ANYONE ASSOCIATED WITH NETDOCUMENTS REPRESENTS OR WARRANTS THAT THE SERVICE WILL BE RELIABLE, ERROR-FREE OR UNINTERRUPTED, THAT DEFECTS WILL BE CORRECTED, FREE OF VIRUSES OR OTHER HARMFUL COMPONENTS OR THAT THE SERVICES WILL OTHERWISE MEET THE NEEDS OR EXPECTATIONS OF ANY USER.          Indemnification You agree to defend, indemnify and hold harmless NetDocuments, its affiliates, licensors and service providers, and its and their respective officers, directors, employees, contractors, agents, licensors, suppliers, successors and assigns from and against any claims, liabilities, damages, judgments, awards, losses, costs, expenses or fees (including reasonable attorneys' fees) arising out of or relating to your violation of these Terms of Use or your use of the Website or the Service other than as expressly authorized in these Terms of Use.  Governing Law and Jurisdiction All matters relating to the Service and these Terms of Use and any dispute or claim arising therefrom or related thereto (in each case, including non-contractual disputes or claims), shall be governed by and construed in accordance with the internal laws of the State of Utah without giving effect to any choice or conflict of law provision or rule (whether of the State of Utah or any other jurisdiction).  Any legal suit, action or proceeding arising out of, or related to, these Terms of Use or the Service shall be instituted exclusively in the federal courts of the United States or the courts of the State of Utah in each case located in Salt Lake City and Salt Lake County. User waives any and all objections to the exercise of jurisdiction over you by such courts and to venue in such courts.    Waiver and Severability No waiver by NetDocuments of any term or condition set forth in these Terms of Use shall be deemed a further or continuing waiver of such term or condition or a waiver of any other term or condition, and any failure of NetDocuments to assert a right or provision under these Terms of Use shall not constitute a waiver of such right or provision.  If any provision of these Terms of Use is held by a court or other tribunal of competent jurisdiction to be invalid, illegal or unenforceable for any reason, such provision shall be eliminated or limited to the minimum extent such that the remaining provisions of the Terms of Use will continue in full force and effect.  Entire Agreement These Terms of Use and our Privacy Policy constitute the sole and entire agreement between you and NetDocuments with respect to the Service and the Website and supersede all prior and contemporaneous understandings, agreements, representations and warranties, both written and oral, with respect to the Service and the Website. There exists a separate Service Agreement between NetDocuments and the Authorizing Customer under which the Authorizing Customer may have certain rights and obligations, but you are not a party to, or intended third party beneficiary of such Service Agreement.  Designated Agent under 17 U.S.C. §512(c) Receiving Complaints and Infringement Claims  Pursuant to 17 U.S.C. §512(c), notice is hereby given that the person identified below is designated as NetDocument’s agent for receiving complaints concerning this Website, including for purposes of receiving notification of any claims of intellectual property infringement:  Legal Affairs Department
2500 W Executive Parkway Suite 350, Lehi, UT 84043, USA<PHONE_NUMBER><EMAIL_ADDRESS> NetDocuments  The Service is powered by NetDocuments Software Inc., a Delaware, USA corporation, located at 2500 W Executive Parkway Suite 350, Lehi, UT 84043, Phone<PHONE_NUMBER> (www.netdocuments.com). Answers to technical questions about the Service are found in Help which is accessed from within the Service. Any further questions should be submitted to the Authorizing Customer’s helpdesk.  All NetDocuments Services are: Copyright © 2020 NetDocuments Software Inc. and are covered by U.S. Patent No. 6,182,080. All rights reserved.

(Automated response - build 671.)

Hi @EmmaTheEmu,
This package appears to require the user to accept a license agreement before use. Is this the case?
Unfortunately, this kind of license usually requires explicit agreement, and including an installer switch to silently agree doesn't meet the requirement.

Hello @stephengillie
That is correct, when installing the application standardly you accept the license in the installer.
With the Winget package itself - during my testing the installer never prompted me for anything. With Windows Sandbox it installed without any additional pop ups, without sandbox it just prompted me for UAC.
Is there anything else I can/need to add in this case to make this package suitable?

@stephengillie
Still awaiting for your response regarding my question

Hi @EmmaTheEmu,
Sorry for the delay. During your testing, you didn't encounter an installer window such as the one in my previous comment above?
Unfortunately, we've recently had to remove a number of manifests from our repo because their installer included a license accepting installer switch. Most licenses do not support this kind of implicit agreement, and as you said instead require the user to explicitly agree to the license terms. We're working to add an Agreements section, which would allow developers to coordinate this type of agreement with a package manager install.
Here, a compounding issue is that the installer with the license terms also is not unattended. It appears that the Automatic Validation pipeline is mis-detecting the package as installed.

Do you encounter the above installer window in your testing?
Is there an installer switch to bypass the above installer window and complete the install without interaction? it's definitely blocking manual validation.
Do these installer switches include a switch to accept the terms, which might cause the situation described above?

This is how my installation looks like on Sandbox:
https://github.com/microsoft/winget-pkgs/assets/106916657/baac0434-7d4c-4a82-878f-9764c977bc1b
If required I can also add a video on how ndMail installs to that push request.

Was the license agreement accepted in a past installation? A previous installer might have saved that information, to streamline future installs.
Pipeline validations are performed on fresh installs of Windows.

The video was filmed on the Sandbox feature. Apart from installing winget, copying manifest and enabling manifest installs - nothing else was done.

I was wondering, would it be possible to provide the specifications of the system that the pop up appeared? I only tested this in a 22H2 Windows 11 Sandbox, but there is a separate x86 installer that might be tripping it up.

Sure:

Windows 10 Enterprise Evaluation
Version 21H2
Installed 1/4/2024
Build 19044.1288

Here is a screenshot of a list of installed applications: https://github.com/microsoft/winget-pkgs/issues/107639#issuecomment-1734683940


test flake: TestRemoteCacheRegistrySeparateImportExport - connection refused reaching registry

https://dagger.cloud/dagger/traces/321f2e777c47c9ca7651a28633a08420?span=79134fa449a2f409&logs#79134fa449a2f409:L3797`
Error: make request: input: container.from.withExec.stdout resolve: failed to copy: httpReadSeeker: failed open: failed to do request: Get "http://registry:5000/v2/test-cache-a/blobs/sha256:c74b3fa9b6e492458102dbd87edf33967a1863e12ac8a138fbc12c6f87483f18": dial tcp <IP_ADDRESS>:5000: connect: connection refused

Test:       	TestRemoteCacheRegistrySeparateImportExport

cc @sipsma

The registry service being used by the engines just stopped running in the middle of the test: https://dagger.cloud/dagger/traces/321f2e777c47c9ca7651a28633a08420?span=79134fa449a2f409&logs#79134fa449a2f409:L1802
So it makes sense the engine got a connection refused when trying to use it. But doesn't make sense that it just stopped running since it was a service bound to other running services.
Would need the logs of the outer engine to debug more, but don't see the name of engine connected to printed in the cloud logs, is that correct @vito?

Noticed it was pretty easy to repro the registry exiting so added some logs to the engine and found this:
time="2024-06-19T17:38:42Z" level=warning msg="!!! SERVICE EXITED" cmd="/entrypoint.sh /etc/docker/registry/config.yml" err="map[error:exit code: 141 kind:*errors.withStack stack:<nil>]" service="xxh3:ed1f507231e06760" session=ukwbmwy7kdsoxd42vkhep6t2j

(note: there's a non-!!! log that also shows this:)
time="2024-06-19T17:38:42Z" level=debug msg="> container done liq5slvzlt1lngha31pck1iw8 [/.init /entrypoint.sh /etc/docker/registry/config.yml]" client_hostname=buildkitsandbox client_id=dpuvp5cfclfplg9ktpzqtq9ra error="exit code: 141" session_id=ukwbmwy7kdsoxd42vkhep6t2j

141 = SIGPIPE :thinkspin:

[pid  1506] 13:52:28 write(2, "time=\"2024-06-19T17:52:28.414540923Z\" level=info msg=\"response completed\" go.version=go1.20.8 http.request.host=\"registry:5000\" http.request.id=133b3e63-7956-4a75-b579-50694c556be7 http.request.method=GET http.request.remoteaddr=\"<IP_ADDRESS>:57134\" http.request.uri=\"/v2/testcontainerpublishforcecompressionestargz/manifests/iebn5anrdi1gf9a1elxdudeju\" http.request.useragent=go-containerregistry http.response.contenttype=\"application/vnd.oci.image.manifest.v1+json\" http.response.duration=2.126706ms http.response.status=200 http.response.written=700 \n", 552) = -1 EPIPE (Broken pipe)

but why?

Managed to strace the runc process (note that this includes the registry process too since it's strace with -f):
strace-runc-happy.log
strace-runc-sigpipe.log
Strangely we can see that the other end of the pipe does read() the same data that the write() SIGPIPEs on:
[pid 196099] 14:34:20 read(13,  <unfinished ...>
[pid 196095] 14:34:20 <... futex resumed>) = 0
[pid 196095] 14:34:20 write(1, "<IP_ADDRESS> - - [19/Jun/2024:18:34:20 +0000] \"PUT /v2/testcontainerpublishforcecompressiongzip/blobs/uploads/580abfba-8502-4c28-a2b4-27cc6eee3be9?_state=tADWGXcmmevnL1bNklSe-PKnMYvwL7_cSMPrViccig97Ik5hbWUiOiJ0ZXN0Y29udGFpbmVycHVibGlzaGZvcmNlY29tcHJlc3Npb25nemlwIiwiVVVJRCI6IjU4MGFiZmJhLTg1MDItNGMyOC1hMmI0LTI3Y2M2ZWVlM2JlOSIsIk9mZnNldCI6MCwiU3RhcnRlZEF0IjoiMjAyNC0wNi0xOVQxODozNDoyMC4xOTEzNTYyNTFaIn0%3D&digest=sha256%3Ad25f557d7f31bf7acfac935859b5153da41d13c41f2b468d16f729a5b883634f HTTP/1.1\" 201 0 \"\" \"buildkit/v0.0-dev\"\n", 524 <unfinished ...>
[pid 196094] 14:34:20 <... epoll_pwait resumed>[], 128, 0, NULL, 0) = 0
[pid 196095] 14:34:20 <... write resumed>) = 524
[pid 196094] 14:34:20 futex(0xc000101148, FUTEX_WAKE_PRIVATE, 1 <unfinished ...>
[pid 196095] 14:34:20 read(11,  <unfinished ...>
[pid 196100] 14:34:20 <... futex resumed>) = 0
[pid 196094] 14:34:20 <... futex resumed>) = 1
[pid 196094] 14:34:20 nanosleep({tv_sec=0, tv_nsec=20000},  <unfinished ...>
[pid 196100] 14:34:20 futex(0xc000101148, FUTEX_WAIT_PRIVATE, 0, NULL <unfinished ...>
[pid 196094] 14:34:20 <... nanosleep resumed>NULL) = 0
[pid 196094] 14:34:20 futex(0xc000101148, FUTEX_WAKE_PRIVATE, 1 <unfinished ...>
[pid 196100] 14:34:20 <... futex resumed>) = 0
[pid 196094] 14:34:20 <... futex resumed>) = 1
[pid 196100] 14:34:20 futex(0xc000101148, FUTEX_WAIT_PRIVATE, 0, NULL <unfinished ...>
[pid 196094] 14:34:20 nanosleep({tv_sec=0, tv_nsec=20000}, NULL) = 0
[pid 196094] 14:34:20 futex(0x7f7cb54d6ea0, FUTEX_WAIT_PRIVATE, 0, {tv_sec=60, tv_nsec=0} <unfinished ...>
[pid 196099] 14:34:20 <... read resumed>"time=\"2024-06-19T18:34:20.562277231Z\" level=error msg=\"response completed with error\" err.code=\"manifest unknown\" err.detail=\"unknown tag=wqenwkle9vmxntn0b1vhui4br\" err.message=\"manifest unknown\" go.version=go1.20.8 http.request.host=\"registry:5000\" http.request.id=4e946782-3de8-47b3-b87e-8037c66a9e8b http.request.method=HEAD http.request.remoteaddr=\"<IP_ADDRESS>:41270\" http.request.uri=\"/v2/testcontainerpublishforcecompressiongzip/manifests/wqenwkle9vmxntn0b1vhui4br\" http.request.useragent=\"buildkit/v0.0-dev\" http.response.contenttype=\"application/json; charset=utf-8\" http.response.duration=1.281029ms http.response.status=404 http.response.written=115 vars.name=testcontainerpublishforcecompressiongzip vars.reference=wqenwkle9vmxntn0b1vhui4br \n", 32768) = 753
[pid 196095] 14:34:20 <... read resumed>"<IP_ADDRESS> - - [19/Jun/2024:18:34:20 +0000] \"HEAD /v2/testcontainerpublishforcecompressiongzip/manifests/wqenwkle9vmxntn0b1vhui4br HTTP/1.1\" 404 115 \"\" \"buildkit/v0.0-dev\"\n", 32768) = 173
[pid 196099] 14:34:20 futex(0x7f7cb54d6ea0, FUTEX_WAKE_PRIVATE, 1 <unfinished ...>
[pid 196095] 14:34:20 sched_yield( <unfinished ...>
[pid 196099] 14:34:20 <... futex resumed>) = 1
[pid 196095] 14:34:20 <... sched_yield resumed>) = 0
[pid 196094] 14:34:20 <... futex resumed>) = 0
[pid 196099] 14:34:20 write(2, "time=\"2024-06-19T18:34:20.562277231Z\" level=error msg=\"response completed with error\" err.code=\"manifest unknown\" err.detail=\"unknown tag=wqenwkle9vmxntn0b1vhui4br\" err.message=\"manifest unknown\" go.version=go1.20.8 http.request.host=\"registry:5000\" http.request.id=4e946782-3de8-47b3-b87e-8037c66a9e8b http.request.method=HEAD http.request.remoteaddr=\"<IP_ADDRESS>:41270\" http.request.uri=\"/v2/testcontainerpublishforcecompressiongzip/manifests/wqenwkle9vmxntn0b1vhui4br\" http.request.useragent=\"buildkit/v0.0-dev\" http.response.contenttype=\"application/json; charset=utf-8\" http.response.duration=1.281029ms http.response.status=404 http.response.written=115 vars.name=testcontainerpublishforcecompressiongzip vars.reference=wqenwkle9vmxntn0b1vhui4br \n", 753 <unfinished ...>
[pid 196095] 14:34:20 futex(0x7f7cb54d6db8, FUTEX_WAKE_PRIVATE, 1 <unfinished ...>
[pid 196099] 14:34:20 <... write resumed>) = -1 EPIPE (Broken pipe)

Interestingly that trace shows runc is getting a SIGPIPE from... itself? I thought that was showing the registry getting a SIGPIPE while writing to runc, but it's actually 100% runc in that trace. Attaching another that shows the process tree and command name proving it:
Uploading strace-runc-self.log…

Stepping away from this for now, here's my horrific scripting for posterity:
while true; do pid=$(ps auxf | grep -A1 '[/]usr/local/bin/runc' | tee /tmp/strace4.log | grep -B1 /registry | grep /runc | grep -v grep | awk '{print $2}'); if [[ -n "$pid" ]]; then strace --decode-pids=comm -s 2000 -vvtf -p "$pid" 2>&1 | tee -a /tmp/strace4.log; break; fi; done

while dagger-dev call --source .:default test custom --pkg ./core/integration --run '/ForceCompression' --count=100 --failfast; do date; done

This should be fixed by https://github.com/dagger/dagger/pull/7696

@sipsma might be wrong (typing from airport) but I thought that was for a separate test? This one had the registry abruptly exiting, never got to the bottom of it

@vito shit sorry you are right, I was going through my 1000 open tabs too quickly and misread this, re-opening!

I have not seen this one since July I think, so closing for now


OrbStack doesn't show stopped Docker

OrbStack doesn't show stopped Docker
The version of OrbStack I installed is Version 0.6.2 (1134)
I stopped a Docker in OrbStack, but my OrbStack interface is not stopped. To start the Docker again, I still need to enter the command line from the terminal. I see that you can start and stop Docker directly in OrbStack from the official website picture. How can I achieve this function?
Official website homepage picture

My OrbStack interface, here I have stopped 3 Dockers

That's right, thanks a lot!

All containers are now shown by default in v0.7.0.


many doubled "the" in comments etc

Fixes the vast majority of duplicated "the".
Also fixes a few other typos and grammar in the vicinity of these changes.
I left about 8 or 9 where I the comment was too unclear or ambiguous and I didn't want to risk a wrong assumption and make them worse.

Thanks, my concern with other versions of this PR we've gotten are that sometime the doubled word shouldn't just be removed, but replaced with a different word.  But it looks like you verified each change.


update junit versions

org.junit.jupiter:junit-jupiter-*  -> 5.4.1
org.junit.platform:junit-platform-console -> 1.4.1

awesome! i like when libs are updated! would you be please so kind to rebase; im on phone with limited internet and can do that only in 2 weeks

thank you!


[CI:DOCS] Overhaul image-table pr-comment workflow

Since this GHA workflow's previous update, support has been added to
Cirrus-CI such that downstream CI can reference EC2 images by
the same $IMAGE_SUFFIX value used in GCP.  Overhaul the image-table
posted to PRs by this workflow, such that it reflects this, and provides
a handy-dandy link directly to the build-log for each image.  Also,
sort the table rows by the stage + image name.
Signed-off-by: Chris Evich<EMAIL_ADDRESS>

Unfortunately there isn't an easy way to test these changes since the workflow has to run from the main branch.  Though I did (mostly) manually test the data-flows and output.  I'll merge this and submit new PRs for any issues that come up.


Refactor formula to have version in one place only

This does not seem to get the proper version for me. I'm not sure if I need to do anything in particular.
$ brew tap voelzmo/homebrew-tap
==> Tapping voelzmo/tap
Cloning into '/usr/local/Library/Taps/voelzmo/homebrew-tap'...
remote: Counting objects: 5, done.
remote: Compressing objects: 100% (5/5), done.
remote: Total 5 (delta 0), reused 4 (delta 0), pack-reused 0
Unpacking objects: 100% (5/5), done.
Checking connectivity... done.
"git hooks not installed in this repository. Run git hooks --install to install it or git hooks -h for more information."
Tapped 2 formulae (33 files, 9.6K)

$ brew install voelzmo/tap/bosh-init
==> Installing bosh-init from voelzmo/tap
==> Downloading https://s3.amazonaws.com/bosh-init-artifacts/bosh-init-0.0.81-darwin-amd64
Already downloaded: /Users/pivotal/Library/Caches/Homebrew/bosh-init-<IP_ADDRESS>-darwin-amd64
🍺  /usr/local/Cellar/bosh-init/0.0.81: 2 files, 12.3M, built in 0 seconds

@challiwill I guess this is due to https://github.com/voelzmo/homebrew-tap having 0.0.81 on the master branch?
could you please do the following:
brew tap voelzmo/homebrew-tap
cd `brew --repository`/Library/Taps/voelzmo/homebrew-tap
git checkout -b dry_version origin/dry_version
brew install voelzmo/tap/bosh-init

Seems good. Thanks.


Use yo-rc files to manage configuration

http://yeoman.io/authoring/storage.html

https://github.com/jhipster/generator-jhipster/blob/master/generators/app/index.js


Inherited class fields aren't recognized properly if parent class is not accessible/public

Description
I have a typical class hierarchy for a project that uses Hibernate. The "base" class is effectively package-private, abstract, and contains several "managed" fields. The actual entity classes are public.
I'm trying to set all the fields using Instancio.of for a given entity:
return Instancio.of(Category.class)
    .set(Select.field(Category::getId), UUID.fromString("SOME-VALID-UUID-VALUE"))
    ...
    .set(Select.field(Category::getName), "Some Name")
    .create();

But Instancio is unable to find the id member for Category, unless the parent class is made public :grimacing:
Expected Result
To be able to set all fields/members, as long as they are visible (from method reference).
Actual Result
Error creating an object
 -> at org.company.model.internal.CategoryFactory$Persistent.oneOf(CategoryFactory.java:20)

Reason:

Unable to resolve the field from method reference:
-> CategoryFactory$Persistent::lambda$oneOf$5709d914$1
   at org.company.model.internal.CategoryFactory$Persistent.oneOf(CategoryFactory.java:20)

Potential causes:
-> The method and the corresponding field do not follow expected naming conventions
   See: https://www.instancio.org/user-guide/#method-reference-selector
-> The method is abstract, declared in a superclass, and the field is declared in a subclass
-> The method reference is expressed as a lambda function
   Example:     field((SamplePojo pojo) -> pojo.getValue())
   Instead of:  field(SamplePojo::getValue)
-> You are using Kotlin and passing a method reference of a Kotlin class

Additional Information
// package org.company.model

@Getter
@MappedSuperclass
@NoArgsConstructor(access = AccessLevel.MODULE)
abstract class AbstractPersistable {
  @Id private UUID id;
  ...

  // @PrePersist + @PreUpdate implementations
}

@Getter
@Entity
@Builder
@Table(name = "categories")
@AllArgsConstructor(access = AccessLevel.PRIVATE)
@NoArgsConstructor(access = AccessLevel.PROTECTED)
public class Category extends AbstractPersistable {
  private String name;
  private String description;
  private String picture;
}

Java version: openjdk 21.0.4 2024-07-16 LTS
Instancio version: 5.0.1

@x80486 thanks for raising the issue! Unfortunately Instancio can't resolve the field since it's declared in the parent AbstractPersistable. In addition, AbstractPersistable is package private, so you can't use field(AbstractPersistable::getId) either.
In this case, you could target the id field using a predicate selector, e.g.
Select.fields().ofType(UUID.class).named("id")

Or if you want to be explicit (in case you have other classes with UUID id field) then you can do this:
Select.fields(f -> "id".equals(f.getName())
                && f.getDeclaringClass().getSimpleName().equals("AbstractPersistable"))

Here's an example:
@Test
void example() {
    Category res = Instancio.of(Category.class)
            .set(idSelector(), UUID.fromString("dfc5fcdd-a51c-4c94-849c-0e28ca7c45b2"))
            .create();
}

static TargetSelector idSelector() {
    return Select.fields(f -> "id".equals(f.getName())
            && f.getDeclaringClass().getSimpleName().equals("AbstractPersistable"));
}

Does this solve your issue?

Great! I wasn't sure if there was a way to overcome this. The first solution, Select.fields().ofType(UUID.class).named("id") works perfect for my use case.
Thanks! :raised_hands:

Awesome! Thanks for the confirmation :+1:


Upload.progress() fires 100% immediately

The progress event fires 100% complete as soon as the upload starts, and doesn't fire again.  The upload continues, maybe 20-30s for a large image, and everything works as expected after, but the progress indicator is useless.

Ok, it appears that the progress indicator relies on server response.  I thought that this plugin behaved like jQuery file upload and was able to report progress from the front end.

I've been trying this with HTML5 shim as well, no matter what I do, it immediately fires 100% upload complete.  For a 100mb upload, it fires 100% immediately, then waits 2 minutes to complete...

what browser/version?


Redis sentinals with a kubernetes clusterIP serivice?

I'm using the following redis operator for kubernetes: https://github.com/spotahome/redis-operator
The operator creates redis instances, sentinals and a ClusterIP service that load balances between the sentinals. Is it possible to connect to the sentinals through the service using the ioredis library?

When I do the following:
const redis = new Redis({
  sentinels: [
    { host: "rfs-redis", port: 26379 }
  ],
  name: "mymaster",
});

redis.set("foo", "bar");

I get:
[ioredis] Unhandled error event: Error: connect ECONNREFUSED <IP_ADDRESS>:6379
    at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1247:16)
[ioredis] Unhandled error event: Error: connect ECONNREFUSED <IP_ADDRESS>:6379
    at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1247:16)
[ioredis] Unhandled error event: Error: connect ECONNREFUSED <IP_ADDRESS>:6379
    at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1247:16

In fact if I try to directly connect to the sentinals:
const redis = new Redis({
  sentinels: [
	{ host: "<IP_ADDRESS>", port:26379 },
	{ host: "<IP_ADDRESS>", port:26379 },
  ],
  name: "mymaster",
});

redis.set("foo", "bar");

I get the same:
[ioredis] Unhandled error event: Error: connect ECONNREFUSED <IP_ADDRESS>:6379
    at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1247:16)
[ioredis] Unhandled error event: Error: connect ECONNREFUSED <IP_ADDRESS>:6379
    at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1247:16)
MaxRetriesPerRequestError: Reached the max retries per request limit (which is 20). Refer to "maxRetriesPerRequest" option for details.
    at Socket.<anonymous> (/code/node_modules/ioredis/built/redis/event_handler.js:182:37)
    at Object.onceWrapper (node:events:628:26)
    at Socket.emit (node:events:513:28)
    at TCP.<anonymous> (node:net:757:14)
[ERROR] 16:32:03 MaxRetriesPerRequestError: Reached the max retries per request limit (which is 20). Refer to "maxRetriesPerRequest" option for details.

I'm a bit confused as it tries to connect to <IP_ADDRESS>, i.e. localhost.
Next I'll try to connect using the redis-cli
> redis-cli -h rfs-redis -p 26379
rfs-redis:26379> SENTINEL get-master-addr-by-name mymaster
1) "<IP_ADDRESS>"
2) "6379"
rfs-redis:26379> rfs-redis:26379> exit
> redis-cli -h <IP_ADDRESS> -p 6379
<IP_ADDRESS>:6379> SET test "TEST"
OK

So I can connect using the redis-cli, but for some reason ioredis fails.

I have some problem，I connect redis with sentinel  ，why connect redis localhost


RESTClient: docs: add the troubleshooting section

Good point @sultaniman. Do you have good examples of rich formatting the exceptions? I think one thing that would be great is to see the actual response content in case of HTTP exceptions, as sometimes there's required context that can clarify the error.

@burnash I only know about rich traceback printer https://rich.readthedocs.io/en/stable/traceback.html


Add impact and urgency as optional params for the servicenow alerter

Extend the Servicenow alerter to support two new parameters: impact and urgency.
More info on these parameters can be found on the Servicenow doc page on defining priority lookup rules.
These are added as optional parameters to ensure backwards compatibility.

It's an additional impact and urgency, but I think it's better to prefix it with servicenow_.

Although it is an additional impact and urgency, since integer is specified in schema.yaml, in the actual rule, if you enclose it in single quotation marks or double quotation marks, an error will occur when executing the actual rule.
It doesn't cause an error in unit tests, but in the sense that it is close to the actual data, I think it is better not to enclose it in single quotes.

It's an additional impact and urgency, but I think it's better to prefix it with servicenow_.

Thank you. I'll add the servicenow prefix for the new parameters.

It's an additional impact and urgency, but I think it's better to prefix it with servicenow_.

Thank you. I'll add the servicenow prefix.

...I think it is beter not to enclose it in single quotes.

To make sure I understand, are you suggesting to omit the quotes in the unit test?

If schema.yaml is written as follows, the value will be checked and an error will occur when a rule other than 1,2,3 is specified and the rule is executed.
  servicenow_impact: {type: integer, minimum: 1, maximum: 3}
  servicenow_urgency: {type: integer, minimum: 1, maximum: 3}

...I think it is beter not to enclose it in single quotes.

To make sure I understand, are you suggesting to omit the quotes in the unit test?

If you don't agree, you can leave it as it is.

If schema.yaml is written as follows, the value will be checked and an error will occur when a rule other than 1,2,3 is specified and the rule is executed.
  servicenow_impact: {type: integer, minimum: 1, maximum: 3}
  servicenow_urgency: {type: integer, minimum: 1, maximum: 3}

Supplement. jsonschema checks the key and data type etc. listed in schema.yaml.

Great suggestion. Thank you.


Fix StochasticDepth layer error in training mixed_float16

Description
StochasticDepth layer throws error TypeError: Input 'y' of 'Mul' Op has type float16 that does not match type float32 of argument 'x'. if training in mixed_float16 condition.
I think we need a dtype=self._compute_dtype_object or dtype=self.compute_dtype in tf.keras.backend.random_bernoulli here.
Type of change

[x] Bug fix
[ ] New Tutorial
[ ] Updated or additional documentation
[ ] Additional Testing
[ ] New Activation and the changes conform to the activation contribution guidelines
[ ] New Callback and the changes conform to the callback contribution guidelines
[ ] New Image addition and the changes conform to the image op contribution guidelines
[ ] New Layer and the changes conform to the layer contribution guidelines
[ ] New Loss and the changes conform to the loss contribution guidelines
[ ] New Metric and the changes conform to the metric contribution guidelines
[ ] New Optimizer and the changes conform to the optimizer contribution guidelines
[ ] New RNN Cell and the changes conform to the rnn contribution guidelines
[ ] New Seq2seq addition and the changes conform to the seq2seq contribution guidelines
[ ] New Text addition and the changes conform to the text op contribution guidelines

Checklist:

[x] I've properly formatted my code according to the guidelines

[x] By running Black + Flake8
[ ] By running pre-commit hooks

[ ] This PR addresses an already submitted issue for TensorFlow Addons
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
Minimal reproducing code:
from tensorflow import keras
from tensorflow_addons.layers import StochasticDepth
import numpy as np

keras.mixed_precision.set_global_policy("mixed_float16")

inputs = keras.layers.Input([10, 10, 3])
nn = keras.layers.Conv2D(3, 1)(inputs)
sd = StochasticDepth(0.5)
nn = sd([inputs, nn])
nn = keras.layers.Flatten()(nn)
nn = keras.layers.Dense(2, activation="softmax", dtype="float32")(nn)
mm = keras.models.Model(inputs, nn)
mm.compile(loss="sparse_categorical_crossentropy", optimizer="adam")
xx = np.random.normal(size=[10, 10, 10, 3])
cc = np.mean(xx)
yy = np.array([ii.sum() > cc for ii in xx]).astype('int')
mm.fit(xx, yy, epochs=2)

Will throw error without dtype=self._compute_dtype_object or dtype=self.compute_dtype.
Or just use test_with_mixed_precision_policy with training=True.

@mhstadler
You are owner of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.

So...do anyone checking this? Every time I updated tensorflow-addons , had to change this manually when using StochasticDepth with mixed_float16.

The Codeowner Is not availabe Can you send a PR?

How to do that? By contacting the owner?
@WindQAQ Can you help review this?

@bhack @leondgarse changes look fine to me, I can't see an option to accept them though (I think someone else with higher access will have to do that)

Thanks! :)


[Mac] Support more identifiers for filtering devices

Mac: Support more identifiers for filtering devices
Mac: list-keyboards: Print all available identifiers
Mac: Change functions to accept DeviceIdentifiers record
Something's wrong. Add tracing function.
Mac: Fix for compilation errors.

For some reason, this is not working when a product name is given in iokit-name. Need help!
cc: @slotThe @thoelze1

@MuhammedZakir: https://github.com/kmonad/kmonad-dext
You should have an invite for the right perms

I didn't forget this. It will take a few more weeks before I am free to work on this. Sorry!
Before getting into this, I would like to know what you guys think about the question I posted above. Should we support capturing multiple devices using identifiers? I think it maybe useful for some split-keyboards.

I didn't forget this. It will take a few more weeks before I am free to work on this. Sorry!

No worries, I can relate... ;-) On the same note let me apologize for being gone for a while too. Life fluctuates and we do what we can, and no more.

Should we support capturing multiple devices using identifiers? I think it maybe useful for some split-keyboards.

I think this is a good idea. Ideally we would have a stream of uniquely identified keys. In the case of multiple keyboards, that means keyboard-id and keycode-id. I am again starting to think about how to merge ideas from keycode refactor into master, and I will be keeping this in mind as I work on the KeyEvent representations.

This may or may not be useful for your PR, but this comment seems to be affecting M2 Macs from identifying keyboards: https://github.com/kmonad/kmonad/issues/633#issuecomment-1326744388


Slashing import command

Fixes | Closes | Resolves #

Anything marked as optional that you didn't need to fill in, please remove it from the PR description. Choose one of the keywords above to refer to the issue this PR solves, followed by the issue number (e.g Fixes # 666). If there is no issue, remove the line. Remove this note after reading.

Changes:

Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (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 not work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[ ] Yes
[ ] No

In case you checked yes, did you write tests?

[ ] Yes
[ ] No

Comments about testing , should you have some (optional)
Further comments (optional)
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Codecov Report
Base: 56.92% // Head: 56.24% // Decreases project coverage by -0.67% :warning:

Coverage data is based on head (e93a0df) compared to base (8997e84).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #209      +/-   ##
===========================================
- Coverage    56.92%   56.24%   -0.68%
===========================================
  Files           58       59       +1
  Lines         3747     3785      +38
===========================================
- Hits          2133     2129       -4
- Misses        1527     1568      +41
- Partials        87       88       +1

Flag
Coverage Δ

unittests
56.24% <0.00%> (-0.68%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cli/slashingImport.go
0.00% <0.00%> (ø)

cli/cli_utils.go
91.39% <0.00%> (-2.65%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


AccessControl and CompositeAuth

Hi,
How can use CompositeAuth with AccessControl?
I want create Rest app, and set access rules for actions.
Example:
/api/users.get?id=112233 - allow for all
/api/users.set?access-token=[TOKEN]&status=online - allow for only auth users, and auth only access-token.
Thanks!

@qiangxue Can you tell me whether the following approach is acceptable or not:
Custom CompositeAuth class suppresses authentication errors, so that all the checks happen in AccessControl and the 401 Unauthorized will be thrown from there.
Custom CompositeAuth class:
namespace some\namespace;
class CompositeAuth extends \yii\filters\auth\CompositeAuth
{

    public function beforeAction($action){
        parent::beforeAction($action);
        return true; // always return true even if identity was not found
    }

    // probably challenge method should be stubbed as well

    public function handleFailure($response)
    {
        // do nothing
    }

}

I find this a bit hacky, but still better than to specify auth-free actions in two places: in except option of authenticator and then in access control.

@kirill-konshin what is the problem you are trying to solve? may be better to open a new issue and describe your problem in more detail. Solution does not look right to me but you have not given much info.

@cebe The problem was that CompositeAuth filter was interfering with AccessControl filter and if there were no access tokens supplied to CompositeAuth, then AccessControl becomes useless since CompositeAuth throws an exception.
In this issues there was a proposal to set up except option for authenticator, so that it does not run for auth-free actions. My proposal is to omit any errors in authenticator and let access control to do what it has to. So my question is whether this approach is legit?

Authentication and Authorization are two different things so I think it is okay if both have their own configuration...

@cebe Yes, Authentication and Authorization are two different things. But, from my experience, 2 configs often lead to errors that are hard to debug, plus, if Authorization (AccessControl) can take care of giving or not giving permission to execute an action, I really don't care what happened in Authentication (CompositeAuth). Plus it is a good practice to separate responsibilities. Authenticator only logs in the user, and Authorization takes care of all the access permissions.

The @qiangxue voice is the voice of God ;) you just have to obey XD

God created the framework. Must obey.

I was looking at this and use it the way described but would like it if in the ActionFilter where the except and only option check for unique ID insted of id as it will give me more control.

@kirill-konshin I think the 'optional' parameter will solve the issue you mentioned.
http://www.yiiframework.com/doc-2.0/yii-filters-auth-authmethod.html#$optional-detail


changes to MDE, AAD, and SOAR Essentials solutions

Required items, please complete
Change(s):
Update to
Azure Active Directory solution

Added Revoke-AADSignInSessions playbooks
Update to Block-AADUser playbooks

MicrosoftDefenderForEndpoint solution

added Unisolate-MDEMachine-entity-trigger playbook
added Isolate-MDEMachine-entity-trigger playbook

SentinelSOARessentials solution

added workbooks

AutomationHealth
IncidentOverview
IncidentTasksWorkbook
SecurityOperationsEfficiency

-added playbooks

CreateIncident-MicrosoftForms
CreateIncident-SharedMailbox
M365D_BEC_Playbook_for_SecOps-Tasks
M365D_Phishing_Playbook_for_SecOps-Tasks
M365D_Ransomware_Playbook_for_SecOps-Tasks
Send-Teams-adaptive-card-on-incident-creation

Deleted playbooks from Playbook folder (moved to Solutions, removing duplicates)
Unisolate-MDEMachine-entity-trigger - added to MDE solution
Isolate-MDEMachine-entity-trigger - added to MDE solution
Revoke-AADSignInSessions - added to AAD solution
Revoke-AADSignInSessions-entityTrigger - added to AAD solution
CreateIncident-MicrosoftForms - added to SOAR Essentials solution
CreateIncident-SharedMailbox - added to SOAR Essentials solution
M365D_BEC_Playbook_for_SecOps-Tasks - added to SOAR Essentials solution
Reason for Change(s):

Merging playbooks with solutions

Hello @BenjiSec looking into this

Hello @BenjiSec working on this

Hello @rahul0216 and @manishkumar1991 can you please review the playbooks and approve

Hi all, when we can expect that this is approved as we are waiting for this to publish blog about new Task support in Microsoft Sentinel?

Hi all, when we can expect that this is approved as we are waiting for this to publish blog about new Task support in Microsoft Sentinel?

Hello Benjamin ,
We are reviewing your PR .

Hello @manishkumar1991 benjisec has pushed the changes

Hello @manishkumar1991 witing for your feedback

Hi @manishkumar1991 - can we please merge this as we are waiting this to update official Microsoft Sentinel docs around Tasks and to publish blog about new possibilities.

Thank you @manishkumar1991,
Continuing to review this pr

Hello @BenjiSec please update the metadata of this playbook ' Solutions/Azure Active Directory/Playbooks/Revoke-AADSignInSessions/alert-trigger/azuredeploy.json '. Title & description is missing from the metadata.
You can refer to this playbook for metadata ' Solutions/Azure Active Directory/Playbooks/Revoke-AADSignInSessions/entity-trigger/azuredeploy.json'. If needed please add other parameters as well with reference to the Playbook metadata.
Title for this playbook ' Solutions/Azure Active Directory/Playbooks/Revoke-AADSignInSessions/incident-trigger/azuredeploy.json ' is " Revoke-AADSignInSessions " If its ok can we change it to " Revoke AAD Sign-in session using incident trigger "

And parameter of this two playbooks metadata is set to null " Solutions/MicrosoftDefenderForEndpoint/Playbooks/Isolate-MDEMachine/Isolate-MDE-Machine-entity-trigger/azuredeploy.json " & " Solutions/MicrosoftDefenderForEndpoint/Playbooks/Unisolate-MDEMachine/Unisolate-MDE-Machine-entity-trigger/azuredeploy.json "
Is this property null or can be filled with some instructions / Data

Hello @BenjiSec please add Workbook metadata of the Workbooks of ' SentinelSOARessentials ' in the given file path
" Tools/Create-Azure-Sentinel-Solution/V2/WorkbookMetadata/WorkbooksMetadata.json "

@v-prasadboke I deployed the workbook and confirmed that it works fine. Please make sure that there is an incident number entered within the workbook to load the widgets and content.

Regarding the addition of the workbooks to the workbooksmetadata file, the 4 workbooks in this solution are all existing workbooks in the gallery. Is it still required that they are added if they are in a solution?

Regarding the addition of the workbooks to the workbooksmetadata file, the 4 workbooks in this solution are all existing workbooks in the gallery. Is it still required that they are added if they are in a solution?

Hello @BenjiSec after repackaging the solution and doing arm-ttk on the maintemplate I came to know the the workbook metadata is not present in the workbookmetadata file. Which is causing an error.

Hello @BenjiSec any updates

Hello @malowe101 There are changes requested above, Thank you.

Regarding the addition of the workbooks to the workbooksmetadata file, the 4 workbooks in this solution are all existing workbooks in the gallery. Is it still required that they are added if they are in a solution?

Yes @malowe101, Even if the workbooks of ' Sentinel Soar Essentials ' already existed in the gallery but for AutomationHealth
& IncidentTasksWorkbook workbook metadata is missing in the   " Tools/Create-Azure-Sentinel-Solution/V2/WorkbookMetadata/WorkbooksMetadata.json " file.

Azure.Azure-Sentinel (JsonFileValidation)

Hi @v-prasadboke - I tried adding these 2 workbooks to Tools/Create-Azure-Sentinel-Solution/V2/WorkbookMetadata/WorkbooksMetadata.json " file, but I'm getting errors. Can you please help with this?

All looks good


Examples/Demos

I have seen this, and this looks very promising for our project, but I am a little lost with actually setting it up? There is no package.json file? So not sure if I need to additionally download any other packages? like ws for example...?
Also is there a basic demo, of this working? Anywhere? as I am not sure how the data-url link works?

You have to be a bit more specific. There are lot's of examples in the readme and you can see a basic working version on jsmpeg.com.


Update gshademac.js

Reworked gshade mac faq

@reiichi001 Can we get this merged when you get a chance?


Only start phantomjs when running acceptance tests

When running a test case that doesn't rely on wallaby, is it possible to not start PhantomJS in this case?

Right now I just set the pool_size to 0.

You can move the {:ok, _} = Application.ensure_all_started(:wallaby) from your test_helper.exs into a setup_all in your AcceptanceCase test case.

@keathley might that be a useful addition to the setup info in the README?

That seems reasonable for now. Long term we could configure poolboy to start with 0 phantoms started and spin them up as needed until we hit the max poolsize. That way the end user doesn't have to mess with this configuration. If someone wants to submit a PR for either fix that would be great. I'm going to close this for now.


EVRButtonId is always 0(k_EButton_System)

While I keep runningpollNextEvent(event), event.data.controller.button or EVRButtonId is always zero even though I press different buttons. Am I missing something?
Regarding EVREventType, event.eventType shows 200 or VREvent_ButtonPress, which seems fine.

Let me add the detail. Both pictures were taken when the Vive Controller's trigger is pulled. eventType shows the correct value, but the button id doesn't have the correct value.

I guess this is related to the issue https://github.com/cmbruns/pyopenvr/issues/27 and this hasn't been solved for the VREvent_t struct. Please let me know if anyone has similar issues.

https://github.com/cmbruns/pyopenvr/issues/27#issuecomment-313755407 temporary solves the problem.

That's good to hear!
For a permanent solution we may need to add _pack_ = 4 to the relevant structs when Linux or OSX. As opposed to the C# here in Python we can just to that and runtime when Linux or OSX, so we don't need such an elaborate solution using the union like in C#.
Judging from the "Terrible Hack" comments in https://github.com/ValveSoftware/openvr/blob/bcac1bfbe2be3c1ff37e4f62b8e412fcdf29248e/headers/openvr_api.cs I think these are VRControllerState_t and VREvent_t.
But looking at https://github.com/cmbruns/pyopenvr/blob/1c4fefe/src/translate/openvr_capi.h I see two more structs that are aligned on 4 bytes using pragma pack when Linux or OSX: RenderModel_TextureMap_t and RenderModel_t.
@cmbruns What do you think?

@daniel5gh thanks for the careful investigation. Do you have any ideas about how to design an automated unit test to determine whether these structures and methods are or are not working correctly? I won't be available to work on this at all for the next couple of weeks, unfortunately. Also I haven't got SteamVR working at all on linux nor OSX yet, so I am unable to test any of this myself.
I'm a bit confused about why openvr_api.cs contains hints about the packing of one set of structures (VREvent_t, VRControllerState_t), while openvr_capi.h hints about others (RenderModel_TextureMap_t and RenderModel_t).

I don't know how to automate a test for this, but maybe we can add fixes on a case by case basis so at least those cases will work on Linux and MacOS?
Here is one PR for the VRControllerState_t case.

Here is a patch to openvr/init.py (version 1.0.1301) to add pack = 4 to VREvent_t on Linux so that VR controller button press events have correct values for event.data.controller.button that identify which button is pressed.  The patch is the same as was done for VRControllerState_t in pyopenvr 1.0.1301.  I've tested this on CentOS 7 and Ubuntu 18.0.4.  I attach the patch as a zip file because GitHub will not accept attachments with suffix .patch.
openvr_linux_button.patch.zip

Thanks for patching openvr for Linux so quickly!  I will be trying openvr on macOS next and will report if anything goes wrong.


Redrive FIFO DLQ

We need a script/lamba for redriving FIFO DLQs, it should do something like

Get all messages from FIFO DLQ
Sort them by initial run date
Post back to initial queue

This has surely been done by someone before so might be worth checking for a free solution first

This script exists


Build Components

component to make:
Card Container
Card view (with card info)
Cheatsheet Card (with card summary/class count)
Card in play/out of play?
Private card alert

Made a quick cheat card json/component to plug into played hud

I like the cheat sheet. That's another thing we were talking about, should the descriptions be on the cards or not?

Reworked components/minimized css for reworking

So I'm making a main landing page 'lobby' after login, using purecss to keep the grid clean and uniform,
here is a first pass at some info that could be displayed/generated.

Thoughts, comments, concerns?

Made a seperate discard pile that could loop through what cards played/ or not played yet, if we wanted  to keep that information displayed?

Cleaned this up for legbility


Crash plz fix

Please fix
Download: https://www.dropbox.com/s/lupl1f3qsafa23g/76f1ee39-f42f-fc55-4e51fb62-193c82be.dmp?dl=0

Please upgrade to PhantomJS 2. 1.9.x is no longer supported, and known to have many crasher bugs that are fixed in 2 version.


Fix $volume normalization issue

Fix: https://github.com/microsoft/qlib/issues/765

Description
change $volume to $volume+1e-12 such that low volume stocks will not create infinity in features.
Motivation and Context
https://github.com/microsoft/qlib/issues/765
$volume creates infinity for low volume stocks.
How Has This Been Tested?

[ ] Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
[ ] If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

Pipeline test:
Your own tests:

Types of changes

[x] Fix bugs
[ ] Add new feature
[ ] Update documentation

@2young-2simple-sometimes-naive  Thanks for your contribution!
Could you help to format your code?
Thanks!
https://github.com/microsoft/qlib/blob/main/docs/developer/code_standard.rst#continuous-integration

@2young-2simple-sometimes-naive  Thanks for your contribution!
Could you help to format your code?
Thanks!
https://github.com/microsoft/qlib/blob/main/docs/developer/code_standard.rst#continuous-integration

Besides,  will your results change significantly if you add this patch?

Besides,  will your results change significantly if you add this patch?

The format errors are listed in the CI output.
https://github.com/microsoft/qlib/runs/4678866110?check_suite_focus=true#step:4:170

The format errors are listed in the CI output.
https://github.com/microsoft/qlib/runs/4678866110?check_suite_focus=true#step:4:170

Besides, will your results change significantly if you add this patch?

Depends on the selection of stocks. My strategy has a lot of low volumes stocks, and the fix does not change the IC/RIC significantly. It turns out the volume info is not that important in my stock pool.

Besides, will your results change significantly if you add this patch?

Depends on the selection of stocks. My strategy has a lot of low volumes stocks, and the fix does not change the IC/RIC significantly. It turns out the volume info is not that important in my stock pool.

It looks great to me.
Thanks so much!

It looks great to me.
Thanks so much!


Rename opentracing-examples to opentracing-testbed

Rename opentracing-examples to opentracing-testbed #185

Coverage remained the same at 81.515% when pulling 81d6d14dcb1012e2795a45b90f01e96bf23d7ebd on malafeev:testbed into 2691c703d3673322c2bcc1c733b3c26a0ef95ab1 on opentracing:master.

Merged this one ;)


runscript should exit with non-zero status code when no module found

This would be useful for running a script as part of a test suite, for example:
python manage.py runscript custom_lint_for_my_app

Would you accept a pull request for this?
Questions:

Could do this by raising CommandError, however this prints something. Is this ok when silent=True?
Half a dozen tests would need to change to acknowledge the raising of an error
Or could also do this by setting the exit code and calling sys.exit(code).

Yeah more then happy to accept a PR for this.
Probably sys.exit is the easiest way to go ?

I have a related issue; if my script fails with an ImportError, runscript exits with status 0 while it should fail with a proper error code. This is not expected behaviour and should be filed as a bug.

I had a look at the code and I'm proposing the following changes:

runscript should differentiate between not being able to import module because the script doesn't exist, not not being able to import because it contains an import error - this will help greatly when debugging scripts
runscript command should raise error instead of just printing messages to stdout. Current setup makes it impossible to use it inside a bash script because it does not communicate its exit status correctly. sys.exit() proposed above does not fully solve the problem because it interrupts tests (needs to be mocked). In my experience the best way to exit manage.py command is with an error.

I'd be happy enough to implement the above changes if they're approved.

@mick88 sounds good !

This appears to be resolved as of version 3.0.9. It implements CommandError() and returns non-zero status code for any import errors inside the script.

Thanks for the update !


The selected version (latest) cannot be installed because it does not exist

I have the latest version of the installer downloaded today.
[RuntimeException]
The selected version (3.3.12) cannot be installed because it does not exist.
Execute the following command to install the latest stable Symfony release:
symfony new apidemo
I used the same command to create new project. I tried creating new project with earlier versions too.
Its giving me same error with version changed.
I am using xampp with windows 10. It was working last week.

same for me... mac osx here.

I am facing the same issue when creating a new project with latest version.

Same issue, having on an Ubuntu installation.

Oh what a coincidence i have the same problem on :
Windows 10 Professional - 64bits
PHP 7.0.24
Symfony.phar 1.5.9
I already updated my cacert.pem and my symfony.phar.
At this time the only version that i found who work for me is 3.3.11 and 2.7.30.

I found a manner to get 3.3.12 :

install via composer : composer create-project symfony/framework-standard-edition urProjectName "3.3.*"
it install a 3.3.11
then move to the project and run composer update

Now you've got a fresh 3.3.12 (tx Sensio for the comment on FB ;) )

The easier solution though is to run symfony new directory-name 3.3.11 and afterwards run composer update symfony/symfony.

We are able to do a symfony install based on a version, but not an install based on a branch:
symfony new ./pathtoapp 2.8
fails. However
symfony new ./pathtoapp 2.8.30
is successful.

I am closing here as this is working again.


[masterchef-pool-balance] add uniPairToken0 options to select reserve index

Fixes # add uniPairToken0 options to select reserve index
Changes proposed in this pull request:

add uniPairToken0 options to select reserve index

if uniPairToken0 = true, used LP reserve token0 balance to calculate base token balance
if uniPairToken0 = false, used LP reserve token1 balance to calculate base token balance

Closing this PR for now, new strategies must be added into this repository now: https://github.com/snapshot-labs/snapshot-strategies, feel free to reopen this PR if needed.


chore: add a link to TF module

Add a link to the TF module incase users are interested in managing WIF infra via TF

Add a link to the TF module incase users are interested in managing WIF infra via TF

#86 #87 #77 #39 #85

#Afsoning-patch-1

#86 #87 #77 #85 #77 https://github.com/google-github-actions/get-secretmanager-secrets/issues/138#issue-1076183478

#87 #85 gh pr checkout 86 #77 #39


2023.4.2 HA breaks the integration.

After installing 2023.4.2:
Does not work and the error:
Logger: homeassistant.util.package
Source: util/package.py:107
First occurred: 09:21:06 (3 occurrences)
Last logged: 09:21:19

Unable to install package smartrent.py==0.4.0: ERROR: Cannot install smartrent-py==0.4.0 because these package versions have conflicting dependencies. ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts [notice] A new release of pip is available: 23.0 -> 23.0.1 [notice] To update, run: pip install --upgrade pip

Reverting back to the core 2023.4.1 backup restores the functionality

Consider this a "me too" I'm getting the exact same error message.

Alright ya'll, just pushed up a new release! Feel free to try it out and let me know if there are any issues w/ it

Working fine for me with new update. Thanks for the quick fix!

Working for me as well!

Resolved by @ZacheryThomas
Thanks!


Add new System.Diagnostics.UnreachableException polyfill

Closes #60
Additional context
The System.Diagnostics.UnreachableException.cs file was not actually auto-generated (see #88).
Also, only the English localization was considered for the exception message.

I just rebased on develop with a few improvements.

Addressed feedback from @cremor

Added the ExcludeFromCodeCoverage attribute
Made the UnreachableException class internal instead of public

Added mention of this new class in the README


Docsify Documentation

Following up on issue drogonframework/drogon#1460
This pull-request is ment to replace the abandoned Docsforge documentation.
This Docsify version can be hosted at Github Pages.
@an-tao mentioned that the pull-request must be compatible with the existing wiki.
If this is not the case, I would like some help, esspecially with the Chinese pages as I do not know Chinese.
Before this pull-request can be merged, I still need to remove the disclaimer which can be found on the home-page.
If possible, I would like to do that as late as possible because currently I also host it on my own Github-Pages site.
Some instructions for Github-Pages once the pull-request has been merged:

Open the repo-settings and go to Pages under the Code and automation section

Under Source, select Deploy from a branch
Under Branch select the master branch and the / (root) directory and press save
Optionally, you could use your own domain like docs.drogon.org

Feedback is welcome, hope that this pull-request can be merged asap :)

@rickiewars Thanks so much for your help.
I want to see the effect as soon as possible so I merged it in advance, I would be very grateful if you would do the fixes you mentioned.

@rickiewars In addition, I think pages can completely replace the wiki, you don't have to make your modifications compatible with the wiki, I can maintain a wiki branch myself. thanks so much.

I created a pull-request at #58

Glad I could make this contribution :)

How do you mean exactly?

Ah doxygen, i was confused due to the many "docs" names.
I searched a bit around in the Docsify repo's but couldn't find any reference to Doxygen unfortunately.
You could add URL references to the corosponding doxygen pages manualy and host the generated doxygen site seperately with e.g. github pages but that would take a bit more work.
You can however easily extend docsify using Javascript and many js extensions are already available so maybe if you can think of an easy relationship between doxygen and the markdown docs, you could write a plugin yourself.

@rickiewars
Hi, thanks so much for your help, I merged

Ah doxygen, i was confused due to the many "docs" names. I searched a bit around in the Docsify repo's but couldn't find any reference to Doxygen unfortunately.
You could add URL references to the corosponding doxygen pages manualy and host the generated doxygen site seperately with e.g. github pages but that would take a bit more work.
You can however easily extend docsify using Javascript and many js extensions are already available so maybe if you can think of an easy relationship between doxygen and the markdown docs, you could write a plugin yourself.

Thanks so much.

In this case I think we should generate a separate page with a HTML version of the Doxygen documentation in our project, and make a more prominent link in the updated documentation.
This would be a great first step, until we find a better way to integrate both parts of the documentation.
Thanks for your contribution.