Schedule Order with Blank Options showing if Preorder is disabled

Expected behavior:
If a location has Preorder turned off for either pickup or delivery, the only option should be ASAP for delivery time (should not show Scheduled Delivery as an option)
Actual behavior:
When preorder is disabled, the ASAP button still allows you to select Scheduled Delivery, but the delivery windows are blank.

Reproduce steps:
Set the preorder to disabled for either (or both) Pickup and Delivery. The scheduled delivery shows up as an option, but with blank drop downs.
Version:
Beta.26
Additional Information:

Scheduled should still appear so you can choose a timeslot in the current opening hours.
The blank drop downs are obviously a problem - what are your opening hour settings and does the same problem exist on beta 27?

On 6 Mar 2021, at 02:30, djtimca<EMAIL_ADDRESS>wrote:

Expected behavior:
If a location has Preorder turned off for either pickup or delivery, the only option should be ASAP for delivery time (should not show Scheduled Delivery as an option)
Actual behavior:
When preorder is disabled, the ASAP button still allows you to select Scheduled Delivery, but the delivery windows are blank.
Reproduce steps:
Set the preorder to disabled for either (or both) Pickup and Delivery. The scheduled delivery shows up as an option, but with blank drop downs.
Version:
Beta.26
Additional Information:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I haven't gone to Beta 27 yet - I will try a clean install in a dev env to test.
I would think that if preorder is off you can't preorder at all. Preorder shouldn't matter if you are open or not - at least the way I would read that feature / capability.
Otherwise would think preorder should have three settings: On, Off, When Open.

Pre order means outside of the current days hours.
If you want to limit during the opening hours that’s something an extension could do.

On 6 Mar 2021, at 11:12, djtimca<EMAIL_ADDRESS>wrote:

I haven't gone to Beta 27 yet - I will try a clean install in a dev env to test.
I would think that if preorder is off you can't preorder at all. Preorder shouldn't matter if you are open or not - at least the way I would read that feature / capability.
Otherwise would think preorder should have three settings: On, Off, When Open.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Ok - again seems like a different interpretation of what Preorder means. Current days hours would seem like preorder on / days in advance 0.
Preorder off would still mean to me that I only take orders that people want ASAP - I don't have to think about orders for the future.
If that isn't how the core team interprets "off" then I am sure I can figure something out locally.

Yep it is :)
Pre order 0 is same day ordering - all time slots that day.

On 6 Mar 2021, at 11:33, djtimca<EMAIL_ADDRESS>wrote:

Ok - again seems like a different interpretation of what Preorder means. Current days hours would seem like preorder on / days in advance 0.
Preorder off would still mean to me that I only take orders that people want ASAP - I don't have to think about orders for the future.
If that isn't how the core team interprets "off" then I am sure I can figure something out locally.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I'm confused then. Why have a preorder off if preorder 0 is the same behaviour?

pre order should only allow a min of 1 to avoid confusion.

On 6 Mar 2021, at 11:43, djtimca<EMAIL_ADDRESS>wrote:

I'm confused then. Why have a preorder off if preorder 0 is the same behaviour?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Or it should allow 0 for the behaviour you describe and actually turn off preorder if it is turned off. :)

Nope :)

On 6 Mar 2021, at 11:53, djtimca<EMAIL_ADDRESS>wrote:

Or it should allow 0 for the behaviour you describe and actually turn off preorder if it is turned off. :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Ok - agree to disagree on this one. I guess I will have to solve locally. Feels like a lost opportunity to have flexibility in how a restaurant wants to operate with minimal effort given the settings are already there to allow for both.

We can’t cover every use case. Everything we add means more support and more bugs to fix. That needs to be considered.
To me this would be big functional change when we are really just fixing bugs before RC.
It’s really very easy to do this with an extension - the flexibility is there. Alternatively just hide the schedule drop down :)

On 6 Mar 2021, at 11:59, djtimca<EMAIL_ADDRESS>wrote:

Ok - agree to disagree on this one. I guess I will have to solve locally. Feels like a lost opportunity to have flexibility in how a restaurant wants to operate with minimal effort given the settings are already there to allow for both.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Completely understand not wanting to add features. This feels like a confusing use case in the core given you have an on / off setting but even off allows the functionality to work in a limited use case. Doesn't feel like a new feature to me. Not my call - I will solve locally.

As I said making it a minimum of 1 will remove that confusion.
@sampoyigi ?

I'm not sure it does remove the confusion.

If this this important to you work on a PR for the change you want to see. It will also need carried through on local extension, orange them and probably flame.


Short install command

changes their "entry" file from one version to the next.

Quite right, for example the install command assumes that the authors do not change the entry point.
To remain backwards compatible and not break anything while gaining this advantage, here is what I propose:
eggs.ts
import "./mod.ts";

It may not be ideal, but I think it's a good compromise.


Trigger lockout actions

The control actually never entered the lockout actions code snippet.
Watch out for the logout(request) will trigger an empty logout signal.

@zoten I just check the tests again and there are still failing.

Aye, sorry, didn't had the chance to get more into the testing environment.
Will try to fix them asap, if I manage to.

I just check it and seems is working fine, if you still have issues please also take a look at the tests.


Implement merge method for GroupedGemmLoraLayer to merge LoRA weights into base model

This is a solution for #57

Hi @xffxff , I have tested this PR in my project and it works as expected. Would you please review and let me know if there's anything else I need to do?


StreamParse and Apache Fink

It looks like Apache Fink may be the successor to Storm, so what does this mean for the future of StreamParse?

Don't think Flink is really the "successor" to Storm. In fact, Storm is actively developed with a new 1.0 release coming soon. See these slides for what's coming soon:
https://www.slideshare.net/mobile/ptgoetz/the-future-of-apache-storm
I think streamparse will likely stay focused on Storm support for time being.

:+1: Thanks @amontalenti, good to hear


Add Changeset Group operations

Added Changeset Group operations.
Using TEST_BEHAVIOR_OPTIONS_RECREATE_IMODEL value from .env file instead of hardcoded false for tests.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Arminas Brazenas seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

/azp run


Adding a user setting so we can still have snippet popups

@jbrooksuk I took the liberty of merging it myself, since it's nothing that could break anything ;-)

No problem! :)


IAM controller supports AttachRolePolicy

Is your feature request related to a problem?
we have some cases to create roles before creating policies, then we need a proper/flexible way to attach policies to roles instead of updating role when new policy created.
Describe the solution you'd like
support k8s obj like

IAM API - AttachRolePolicy
AWS::IAM::ManagedPolicy.Properties.Roles

Type: AWS::IAM::ManagedPolicy
Properties:
  Description: String
  Groups:
    - String
  ManagedPolicyName: String
  Path: String
  PolicyDocument: Json
  Roles:   # this
    - String
  Users:
    - String

crossplane IAMRolePolicyAttachment

example:
kind: IAMRolePolicyAttachment
metadata:
  name: examplepolicyattach
spec:
    roles:
        - examplerole
    policy: examplepolicy

@Julian-Chu we already support this. A Role resource has a Spec.Policies field that contains the Policy ARNs to associate with the role. You can also use the Spec.PolicyRefs field to refer to one or more Policy custom resources that you may have created (or intend to create).

@jaypipes  Thanks for hints.  it will be good alternative solution if we can't have policy attachment API.
Let me explain more detail. We are looking for loose coupling solution no need to update Role yaml when we deploy new resource with  policy.
for example:
kind: IAMRolePolicyAttachment
metadata:
  name: examplepolicyattach
spec:
    roles:
        - examplerole
    policy: examplepolicy

We are investigating ack to replace Crossplane.  Crossplane provides this similar policy attachment solution,  and we have something reply on this concept, and think it's not bad to have this flexibility. 😄

@jaypipes   updating the Policy  is also a good solution for us 🍺

@Julian-Chu OK, an update on this...
I spent a few hours yesterday coding up support for Policy.Roles, Policy.Users and Policy.Groups. I got most of the way through implementing it and then realized there was a big problem with implementing these many-to-many relationships from BOTH sides of the relation, as opposed to only a single side of the relation as is currently implemented (with Role.Policies, User.Policies and Group.Policies).
If I implemented support for Policy.Roles, I would need to look up all of the referenced Role CRs and ensure that each Role.Spec.Policies field contained matching Policy ARNs. Likewise, if a user modified Role.Policies to include a customer-defined policy (as opposed to an AWS-managed policy), I would need to look up the Policy CR and ensure that its Policy.Spec.Roles field contained that matching Policy ARN. Keeping these things in sync would be a bit of a nightmare and would go against GitOps best practices of only having the Kubernetes user define desired state and not a controller (controllers should merely reconcile changes in desired state, not change the desired state itself).
All that said, I'm afraid I'm going to conclude that we will not support a Policy.Spec.Roles field and instead only support the existing mechanism of using Role.Spec.Policies to assign policies to a role. Same goes for User and Group policies. I think that this user experience (of ensuring that the Role, User and Group's Policies field is the single source of desired state truth for policy-to-entity relationships is actually the most user-friendly way of describing these relationships and since the whole Policy.Spec.Roles field was just an additional way to express the same relationship, it's OK to not add this functionality into the IAM controller.
Hope the above explanation makes sense!
-jay


Leaving remote session unlocked

I am trying to use my windows 10 Home PC for inhome streaming (steam). For that I have to be able to disconnect from a remote session, without locking the session. The solution I've found on the web suggests using tscon.exe %sessionname% /dest:console, but tscon.exe doesn't exist. I have read in another issue that including tscon.exe in the installer is not possible, so I am asking for any workarounds.

If you need to leave session active (e.g. connected to some display — Console or RDP-Tcp#xx), you may create an .rdp file with Remote Desktop Client to connect to your session (or create if not exist).
Then add this file to scheduled tasks, and start the task. Remote Desktop Client will run in background, connected to your destination session.

@PsychoTheHedgehog I'm not quite shure why you'd need to remote connect to the W10 PC when all you want is to stream Steam to another computer (or Steam Link), but maybe describing my setup will help you.
I've got a game PC with one local account set to automatically login which runs Steam (also auto-login) so that basically when I boot that PC I've got Steam available on my network within a few minutes.
If I need to make changes using that account I remote connect, make my changes and then hit the Reboot shortcut on my desktop so that after a while I've got Steam available again ready for any streaming. No need to keep remote sessions unlocked et al.
The Reboot shortcut is basically:

right-click desktop
click new -> shortcut
enter the text "%windir%\system32\shutdown.exe -r -t 0" (without quotes) and hit next
give it a nice name and hit enter
optionally right-click shortcut, properties, and select a different icon

I've got a similar one for Shutdown so that I can power off that PC remotely (shutdown.exe -s -t 0)
Hope this helps.

Rebooting the entire machine won't work for everyone.  For mine, the bootloader will go to another operating system by default, so rebooting remotely would leave me worse off.

tscon.exe doesn't come with Windows Home Edition but the solution is easy just copy anywhere from Windows pro and paste it in %windir%system32 Work like charm

Closed as solved. If not - reopen it.


[Enhancement] Use the static final modifier to decorate Logger

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request
In Java, it is a best practice to declare a logger static final.

private - so that no other class can hijack your logger
static - so there is only one logger instance per class, also avoiding attempts to serialize loggers
final - no need to change the logger over the lifetime of the class

Location:
Search for private final Logger in project source code.
Describe the solution you'd like
Add static modifier for Logger, rename foobarLogger to FOOBAR_LOGGER, and substitute logger name with constants.
Take org.apache.eventmesh.runtime.metrics.tcp.EventMeshTcpMonitor as an example:

You can add more constants in org.apache.eventmesh.runtime.constants.EventMeshConstants.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct *

Hey @Pil0tXia, would be glad to be assigned to fix this issue!

@MovieTone Welcome to EventMesh community!

@MovieTone If you wish to start coding immediately, you may proceed with further modifications based on https://github.com/apache/eventmesh/pull/4698 to avoid potential conflicts. Alternatively, you can wait until this pull request is merged into the master branch and then code on the master branch.

@Pil0tXia Thank you! I'll probably wait for the merge then

@MovieTone Hi! #4698 is merged and it's clear for you now.

@Pil0tXia Hey, thank you for mentioning! I've created a PR to this


fix: fix bin commands

:tada: This PR is included in version 0.1.1-next.3 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Auto center and scale

Great performance!
It would be very cool to make it possible to automatically center and scale, relative to the parent, as it is done at https://github.com/ariutta/svg-pan-zoom
Are you planning to create such functionality?

I don't currently have plans to do this, but I'm not opposed given enough interest (https://github.com/timmywil/panzoom/blob/master/.github/CONTRIBUTING.md#feature-requests). It can be done now with the pan and zoom methods after programmatically determining the center point. One potential issue I foresee would be that the center of the parent element may not necessarily be the center of the screen. In the first demo, I start with the element in the center and pressing the reset button essentially returns the element to the center. I personally haven't had a need for more than that.


⚠ Bofrost.at has degraded performance

In 3284834, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2439 ms

Resolved: Bofrost.at performance has improved in bcd96eb.


New version: Mega.MEGASync version <IP_ADDRESS>

Pull request has been created with Komac v1.11.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

2024-01-27T04:31:44.5557809Z ##[error] Manifest Error: Duplicate installer entry found.

(Automated response - build 669.)


python312Packages.cohere: 5.9.0 -> 5.9.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.cohere is: Simplify interfacing with the Cohere API
meta.homepage for python312Packages.cohere is: https://docs.cohere.com/docs
meta.changelog for python312Packages.cohere is: https://github.com/cohere-ai/cohere-python/releases/tag/5.9.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.9.1 with grep in /nix/store/w29rc5z46yj6hgcpz8xjpnvln2fal2v1-python3.12-cohere-5.9.1
found 5.9.1 in filename of file in /nix/store/w29rc5z46yj6hgcpz8xjpnvln2fal2v1-python3.12-cohere-5.9.1

Rebuild report (if merged into master) (click to expand)

5 total rebuild path(s)

5 package rebuild(s)

First fifty rebuilds by attrpath

python311Packages.cohere
python311Packages.llmx
python312Packages.cohere
python312Packages.llmx

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/w29rc5z46yj6hgcpz8xjpnvln2fal2v1-python3.12-cohere-5.9.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.cohere https://github.com/r-ryantm/nixpkgs/archive/ce6176e9c422b86bc6aeac02c7c33295a9dfd014.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ce6176e9c422b86bc6aeac02c7c33295a9dfd014#python312Packages.cohere

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/w29rc5z46yj6hgcpz8xjpnvln2fal2v1-python3.12-cohere-5.9.1
ls -la /nix/store/w29rc5z46yj6hgcpz8xjpnvln2fal2v1-python3.12-cohere-5.9.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

8 packages built:

python311Packages.cohere
python311Packages.cohere.dist
python311Packages.llmx
python311Packages.llmx.dist
python312Packages.cohere
python312Packages.cohere.dist
python312Packages.llmx
python312Packages.llmx.dist

Maintainer pings
cc @natsukium for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 339986 at ce6176e9 run on aarch64-darwin 1

4 packages built successfully:

python311Packages.cohere
python311Packages.llmx
python312Packages.cohere
python312Packages.llmx


feat: adds interaction and dialog node types

I agree with @StraToN . Of the two proposals he put forth, I'd tend to favour the first one as it promotes composition and avoids adding another property.

@StraToN / @BHSDuncan - Based on your request, I've created a new node types for interaction locations, inheriting from ESCLocation. It didn't make sense to leave the existing logic in that looks for a node specifically called "dialog_location" if the plan was not to use a named node for interaction positions, so I've created a dialog_location node type as well. The Mark and Worker characters have been updated to use the new dialog location node type.
For ESCItems, if an "ESCInteractionLocation" node isn't found, it falls back to an "ESCLocation" node, which falls back to the middle of the collision shape.  If it doesn't have a collision shape, the click on the item is never registered, so the get_interact_position() function is never called.

Good work!

Great work, thank you! 🥳


⚠ Spryr has degraded performance

In fa60a19, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 4799 ms

Resolved: Spryr performance has improved in 8263d9f after 12 minutes.


Update app-service-move-limitations.md

I'm afraid  this page is a bit confusing: in the title, it tells about moving to a resource group and subscription but down in the page, it has section only for, and mainly / only describes moving  to a subscription (not resource group).
AFAIK, the way it describes restrictions about moving TLS, isn't technically correct. Also, that applies to both subscription and resource group. So, I've modified and re-arranged that paragraph:
"You can move App Service resources to a new resource group or subscription but you need to delete and upload its TLS/SSL certificates to the new resource group or subscription. Also, you can't move a free App Service managed certificate. For that scenario, see Move with free managed certificates. "

@mumian
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@mumian
Can you review this old PR and determine whether it needs to be closed or merged?
@MicrosoftDocs/public-repo-pr-review-team

#assign-reviewer:msangapu-msft

@msangapu-msft can you please review?

I sent an email to the content owner today.

Waiting to hear back from product.
From: Courtney Wales @.>
Date: Thursday, August 29, 2024 at 10:51 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Mangesh Sangapu @.>, Assign @.>
Subject: Re: [MicrosoftDocs/azure-docs] Update app-service-move-limitations.md (PR #123348)
I sent an email to the content owner today.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/123348#issuecomment-2318207236, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AL3FF2CY2IWYBWUPYTQABFDZT47PZAVCNFSM6AAAAABJJTEBQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJYGIYDOMRTGY.
You are receiving this because you were assigned.Message ID: @.***>

Verified with product team. #sign-off

@mumian please sign-off here. Thanks!

@msangapu-msft - thank you for verifying.
#sign-off


Examples

It would be EXTREMELY helpful to have some examples that showed how to use things like a todo example.
The docs are GREAT but nothing like some good examples.

+1 and a bump


Enable error handling for volume operation errors

What type of PR is this?
/kind feature

What this PR does / why we need it:
This change will propagate the errors occurred in cloudProvisioner to crdProvisioner so that there aren't any unintended retries and the driver can fail fast.
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build 837344436

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

Totals

Change from base Build 833538260:
0.0%

Covered Lines:
1101

Relevant Lines:
2980

💛  - Coveralls

Can we add a check statement for Status.Error in crd_provisioner.PublishVolume and return error if found? I think it would be useful for controllerserver's error handling.
https://github.com/abhisheksinghbaghel/azuredisk-csi-driver/blob/8fb38083d86dc7964b481532bfa19739299ee9d0/pkg/provisioner/crdprovisioner.go#L281-L282


Feature Request: Add support for deprecated on a definitions property

I noticed operations can be marked deprecated. Do you guys think it would make sense to also allow for deprecating fields on a property.
A specific use case might be when generating servers and clients, it would be nice to inform users that a property is deprecated and will be removed in the next major version of the api.
I can work on this if we feel there is value?

Hi @roykachouh, this is something that's can't be done because it isn't supported by the spec itself.
You're welcome to look in swagger-spec for an existing issue, and if you don't fine one, please open a new one to be considered for the next version of the spec.


Neutron event errors with a payload as a string instead of dict

ERROR:
An exception occurred processing the API call with args: (<auditmiddleware.AuditMiddleware object at 0x7f4ebd86d430>, <Request at 0x7f4eaa3d2af0 POST https:///v2.0/nsxt-ops>, <Response at 0x7f4eaa13e730 201 Created>){}
Exception(most recent call first)
AttributeError: 'str' object has no attribute 'get'
File "auditmiddleware/init.py", line 84, in wrapper
return fn(*args, **kwargs)
File "auditmiddleware/init.py", line 136, in _process_request
events = self._cadf_audit.create_events(request, response)
File "auditmiddleware/_api.py", line 255, in create_events
return self._build_events(target_project, self._resource_specs,
File "auditmiddleware/_api.py", line 306, in _build_events
return self._build_events(target_project, sub_res_spec, None, None,
File "auditmiddleware/_api.py", line 281, in _build_events
return self._create_events(target_project, res_id,
File "auditmiddleware/_api.py", line 382, in _create_events
isinstance(res_payload.get(res_spec.type_name), list):
Looks to be from nsxt-ops which I'm unfamiliar with. Will dig further.

https://github.com/sapcc/networking-nsx-t
It looks to be an issue with their payload. I see in the changelog some fix for payloads. I don't see this error reoccuring.

Closing this issue as it seems to have been specific to an endpoint and resolved


Remove block and line comments when detecting '.flutter-plugins' in settings.gradle(.kts)

PR Title:
Remove block and line comments when detecting '.flutter-plugins' in settings.gradle

Description:
This PR modifies the configureLegacyPluginEachProjects function to remove block (/* ... */) and line (// ...) comments from the settings.gradle or settings.gradle.kts file content before checking for the presence of the '.flutter-plugins' string. This ensures that only uncommented, meaningful code is considered during the detection, preventing false positives when the string appears within comments.
Why is this change necessary?
In some cases, the '.flutter-plugins' string may be present inside comments in the settings.gradle file. The existing implementation does not account for this and may incorrectly detect the string even when it's commented out. This can lead to unintended behavior, such as configuring plugin projects when it is not necessary.
By removing comments before performing the check, we prevent false positives and ensure that the detection logic is accurate, only acting when the '.flutter-plugins' string is present in active code.
Changes Made:

Added comment removal logic:

Removed block comments (/* ... */) using the regular expression /(?s)\/\*.*?\*\//.

The (?s) flag enables dot-all mode, allowing . to match newline characters.

Removed line comments (// ...) using the regular expression /(?m)\/\/.*$.

The (?m) flag enables multi-line mode, so ^ and $ match the start and end of each line.

Combined both comment removal steps into a single chain for efficiency.

Updated the string detection:

The check for '.flutter-plugins' is now performed on the uncommented content of the settings.gradle file.
This ensures that only meaningful, uncommented code is considered during detection.

Issue Fixed:

Fixes #155484

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need any further assistance or have questions, feel free to reach out!

Links:

Contributor Guide
Tree Hygiene
Flutter Style Guide
Features we expect every widget to implement
CLA
flutter/tests
breaking change policy
Discord
Data Driven Fixes

To pass the PR checks, I have made several commits, resulting in a lengthy commit history. Therefore, I have performed a rebase and force-pushed the final code submission.

Thanks @goosetapa! Looks good to me!


Allow a few more special chars in username regex

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes a bug found in #915

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Documentation has been added / updated (for bug fixes / features)
[x] HISTORY.rst has been updated (with summary of main changes)

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added.

[ ] bumpversion patch has been called on this branch
[ ] The relevant author information has been added to .zenodo.json

What kind of change does this PR introduce?

Allows for two special characters found in GitHub usernames to be parsed when generating the docs

Does this PR introduce a breaking change?
Nope.
Other information:
This regex is not as in-depth as it could be (see: https://github.com/shinnn/github-username-regex), as starting/ending with a dash isn't allowed and some names are reserved, but we don't need this level of complexity here.

Coverage remained the same at 90.721% when pulling 931e2f2927bfcaad54db285d0e995181694dcd47 on fix_regex into b30fd7e42e70a88a64d0477b87297f617cebb660 on master.


Adding E2E test for VM Put bug for protected members

Adding E2E test for VM Creation and Update.
Testing PUT request for VM succeeds even if Protected Properties are changed.

using System.Reflection;

Something is missing at the top of the file :)

Refers to: src/ClientRuntime/Microsoft.Rest.ClientRuntime.E2E.Tests/Properties/AssemblyInfo.cs:1 in 3179142. [](commit_id =<PHONE_NUMBER>527e3904729cae26002231fb871773, deletion_comment = False)

namespace Microsoft.Rest.ClientRuntime.E2E.Tests.ScenarioTests

Missing license

Refers to: src/ClientRuntime/Microsoft.Rest.ClientRuntime.E2E.Tests/ScenarioTests/E2ETestBase.cs:1 in 3179142. [](commit_id =<PHONE_NUMBER>527e3904729cae26002231fb871773, deletion_comment = False)

namespace Microsoft.Rest.ClientRuntime.E2E.Tests.ScenarioTests

Missing license

Refers to: src/ClientRuntime/Microsoft.Rest.ClientRuntime.E2E.Tests/ScenarioTests/VMTests.cs:1 in 3179142. [](commit_id =<PHONE_NUMBER>527e3904729cae26002231fb871773, deletion_comment = False)

using Microsoft.Azure.Management.Compute;

Missing license

Refers to: src/ClientRuntime/Microsoft.Rest.ClientRuntime.E2E.Tests/TestAssets/ExtendingTypes.cs:1 in 3179142. [](commit_id =<PHONE_NUMBER>527e3904729cae26002231fb871773, deletion_comment = False)

@azuresdkci retest this please


Create a Network Stack

Create a Network Stack to deploy a VPC (Virtual Private Network) with a public subnet and an isolated (private) subnet.
The public subnet will  contain an Application Load Balancer (ALB) that forwards incoming traffic to an ECS (Elastic Container Service) Cluster - the runtime of our application.
This should (re)use some cdk constructs to get a cleaner code.

Implemented after 2c0706cffcedcce5d62a8722e8291047d5ce4287


Update bandit-scanner-reference.md

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

https://harness.atlassian.net/browse/DOC-3000

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Flickering on MTA (Multi Theft Auto) Client

in Fullscreen the game is practically unplayable with low fps and constant flickering to work around this it is necessary to leave the client in windowed mode
MTA in Fullscreen:
Kooha-2023-04-29-23-20-28.webm
Software information
MTA (Multi Theft Auto) is a client for GTA SA to play online
System information

GPU: GTX 1060 6GB
Driver: 530.41.03
Wine version: Wine 8.0
DXVK version: 2.1

Apitrace file(s)
Multi Theft Auto-apitrace.zip
sorry if i didn't use the apitrace correctly
Log files
Multi Theft Auto-logs.zip
getting MTA running is a bit tricky but here's how to run it:

MTA:SA will only work in GTA:SA v1.0
there is an error when trying to run the client on linux to resolve this it is necessary to run the server for a few seconds and close so that it can validate your computer as a user perhaps
here is the tutorial to run the server: https://wiki.multitheftauto.com/wiki/Installing_and_Running_MTASA_Server_on_GNU_Linux

Try with current master as this might already have been fixed. You can download here https://github.com/doitsujin/dxvk/actions/runs/4949140079
The apitrace doesn't contain anything sadly. If it's just a few KB that means it didnt work

Try with current master as this might already have been fixed. You can download here https://github.com/doitsujin/dxvk/actions/runs/4949140079 The apitrace doesn't contain anything sadly. If it's just a few KB that means it didnt work

ok i will test

The newly released 2.2 is also fine to test with

That apitrace is useless unfortunately. It's practically empty.

See if "https://github.com/SirSAC/dxvk/actions/runs/5000894491" have the same effect on the client.

@SirSAC There's no way that altering a couple of build flags will make any difference here and I would appreciate it if you didn't spam that under any open issues you come across.

ok

d3d9.deferSurfaceCreation = 1 as a workaround fixes the issue, at least on borderless windowed mode (which is enabled by default). Standard fullscreen crashes for some reason.

Log:
gta_sa_log.txt
Log with standard fullscreen:
gta_sa_defer_fs_log.txt
I even tried ANV driver just in case if it's a driver bug. Same thing.
Fortunately it will show dialog about switching back to borderless windowed after several crashes (like if it was mod issue and not DXVK itself)
I'm also missing some patches like child window or shared resources, which maybe can cause the crash.
I can't manage to make apitrace at all. Mod has weird anti-cheat which blocks everything in game path and just loads d3d9.dll from SysWOW64.
Maybe someone with Windows rig will help here.

Okey so I managed to get traces with wined3d and dxvk (crashing).
Sadly the crash doesnt reproduce with dxvk on replay of the wined3d trace.
mta_dxvk_trace.tar.gz
mta_wined3d_trace.tar.gz

Maybe the flickering was resolved. I tested it on wine 8.14 with the latest version of dxvk and I was able to navigate the menus without the screen constantly blinking. If anyone can test, confirm if the bug has been resolved as well.


Improve "Auto-Allocation" process

Allow user to "adjust" the automated allocation by adjusting item quantity, and removing rows via a checkbox

Add similar functionality for automatically allocating lines against a Purchase Order

For each line which cannot be "automatically" allocated, instead of omitting, provide a drop-down menu populated with possible options.

The form then becomes "Allocate" not "Auto Allocate" and allows the user to allocate items in a single step

This is the screen that should be cleaned up!

I don't know if this is related but since the linked issue was closed I thought I would re-reference it. https://github.com/inventree/InvenTree/issues/761#issuecomment-620335675
Tell me if I should create a dedicated issue for it since it is in the BOM and not build.

@SchrodingersGat In the case that part stock is spread among different location, the "Auto Allocate" method fails to propose at least one of them. At a minimum requirement, it should propose the stock for the default_location set for that part if that stock exists.


[Fix #380] Change item API

Many thanks for submitting your Pull Request :heart:!
What this PR does / why we need it:
Fix #380
Special notes for reviewers:
Additional information (if needed):

Please add the Fix #NUMBER in the issue body next time, otherwise, GH won't link both. 👍

Please add the Fix #NUMBER in the issue body next time, otherwise, GH won't link both. 👍

I know, but this time it was late when I opened the PR and I forgot.


npm repo not found

Not Found - GET https://registry.npmjs.org/ngIfResponsive - Not found

@Fenki404 thx a lot for reporting this. This is wrong in the README. The correct install command is ng-if-responsive. I just updated the README. Thx a lot for reporting this.


Heron's Formula #1155 added in all 4 languages

🛠 Issue (#1155)
👨💻 Changes proposed

[x] Added Heron's formula code in C, C++, Java, Python.

📝Note

My code follows the code style of this project.
This PR does not contain plagiarized content.
The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
I have myself reviewed all 4 files and all of them are working correctly.
📷 Screenshots

@Kumar-laxmi I have completed this issue and hence I would like to draw your attention towards it.

I have done the changes, kindly check @Kumar-laxmi @zugzwang03

Place java program in Java/Maths folder

Resolved


Use absolute path for nebula binary and config file in docker image

as title.

If I specify that the installation directory will be affected?

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: yixinglu:x: jievinceYou have signed the CLA already but the status is still pending? Let us recheck it.


Hacktoberfest Changes

The hacktoberfest is coming to a close very soon, this Pull Request will be reviewed and merged once it concluded.
We are still accepting Pull Requests to this branch until it has been removed
This closes #486

This needs to deal with the grammar from #505


knife vra  fails

Versions

Version of knife-vrealize: 6.0.0
Version of chef:
Chef Workstation version: 20.9.136
Chef Infra Client version: 16.4.41
Chef InSpec version: 4.22.22
Chef CLI version: 3.0.27
Chef Habitat version: 1.6.56
Test Kitchen version: 2.6.0
Cookstyle version: 6.16.10

Platform Details

Version of vRA: Version: 7.5.0 (Build: 10053500)
Version of vRO: n/a
Version of ESXi: n/a

Scenario
Get info from the vRA API using knife
Steps to Reproduce
Run chef shell-init powershell | invoke-expression
Install the gem with chef gem install knife-vrealize
Run any knife vra command: knife vra server create, knife vra catalog list etc
Expected Result
get the expected result of the commands - a server is spun up, the list is returned, etc
Actual Result
Fatal errors.
One thing i notice is that the gem itself appears to be coming from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/. Is that correct? I thought that by using chef gem install and/or chef shell-init powershell they'd be put in the right place.
Fatal errors:
~\Documents\github> knife vra catalog list --entitled
Traceback (most recent call last):
        16: from C:/opscode/chef-workstation/bin/knife:370:in `<main>'
        15: from C:/opscode/chef-workstation/bin/knife:370:in `load'
        14: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/bin/knife:24:in `<top (required)>'
        13: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/application/knife.rb:163:in `run'
        12: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife.rb:224:in `run'
        11: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife.rb:161:in `subcommand_class_from'
        10: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:129:in `command_class_from'
         9: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:115:in `load_command'
         8: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `load_commands'
         7: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `each'
         6: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `block in load_commands'
         5: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `load'
         4: from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:26:in `<top (required)>'
         3: from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:27:in `<class:Chef>'
         2: from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:28:in `<class:Knife>'
         1: from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:29:in `<class:Cloud>'
C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:30:in `<class:VraServerCreate>': uninitialized constant Chef::Knife::Cloud::VraServerCreate::VraServiceHelpers (NameError)
Did you mean?  Chef::Knife::Cloud::VraServiceOptions

~\Documents\github> knife vra server create $catitem
Traceback (most recent call last):
        16: from C:/opscode/chef-workstation/bin/knife:370:in `<main>'
        15: from C:/opscode/chef-workstation/bin/knife:370:in `load'
        14: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/bin/knife:24:in `<top (required)>'
        13: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/application/knife.rb:163:in `run'
        12: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife.rb:224:in `run'
        11: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife.rb:161:in `subcommand_class_from'
        10: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:129:in `command_class_from'
         9: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:115:in `load_command'
         8: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `load_commands'
         7: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `each'
         6: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `block in load_commands'
         5: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `load'
         4: from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:26:in `<top (required)>'
         3: from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:27:in `<class:Chef>'
         2: from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:28:in `<class:Knife>'
         1: from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:29:in `<class:Cloud>'
C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:30:in `<class:VraServerCreate>': uninitialized constant Chef::Knife::Cloud::VraServerCreate::VraServiceHelpers (NameError)
Did you mean?  Chef::Knife::Cloud::VraServiceOptions

I've just noticed that running a non-vra basic knife command fails too, with the same error:
> knife cookbook list
Traceback (most recent call last):
        16: from C:/opscode/chef-workstation/bin/knife:370:in `<main>'
        15: from C:/opscode/chef-workstation/bin/knife:370:in `load'
        14: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/bin/knife:24:in `<top (required)>'
        13: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/application/knife.rb:163:in `run'
        12: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife.rb:224:in `run'
        11: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife.rb:161:in `subcommand_class_from'
        10: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:129:in `command_class_from'
         9: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:115:in `load_command'
         8: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `load_commands'
         7: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `each'
         6: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `block in load_commands'
         5: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `load'
         4: from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:26:in `<top (required)>'
         3: from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:27:in `<class:Chef>'
         2: from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:28:in `<class:Knife>'
         1: from C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:29:in `<class:Cloud>'
C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:30:in `<class:VraServerCreate>': uninitialized constant Chef::Knife::Cloud::VraServerCreate::VraServiceHelpers (NameError)
Did you mean?  Chef::Knife::Cloud::VraServiceOptions

@mcascone Please check if updating your GEM_PATH & GEM_HOME to C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0 resolves this issue.

Interesting; I thought that's what chef shell-init powershell does? What might cause mine to be set to my user appdata? I'll change and test and report.

I thought at first it was an issue between Powershell 5.1 and 7, but i've done everything i can think to do regarding those $env:gem_x vars, and the gem is still getting installed to C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/, and the knife call fails. I'm out of ideas. Help?

Can you check removing this from PATH C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/bin and keep GEM_PATH & GEM_HOME to C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0
Errors you are getting is only due to incorrect  path vars.

It's gotta be. So, i checked my PATH in the windows system properties control panel, and that path isn't in it.
I remembered that my $PROFILE runs chef shell-init powershell | iex every time. So in a fresh shell, my path is:
~\Documents\github> path
C:/opscode/chef-workstation/bin
C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/bin
C:/opscode/chef-workstation/embedded/bin
C:\Program Files\PowerShell\7
C:\Program Files\ConEmu\ConEmu\Scripts
C:\Program Files\ConEmu
C:\Program Files\ConEmu\ConEmu
C:\Users\mcascone1\AppData\Local\Microsoft\WindowsApps
C:\Users\mcascone1\AppData\Roaming\npm
C:\Users\mcascone1\.dotnet\tools

C:\opscode\chef-workstation\bin\
C:\Program Files\Microsoft VS Code\bin
C:\ProgramData\chocolatey\bin
C:\Program Files\PowerShell\7\
C:\Program Files\Git\cmd
C:\Program Files (x86)\Common Files\Oracle\Java\javapath
C:\windows\system32
C:\windows
C:\windows\System32\Wbem
C:\windows\System32\WindowsPowerShell\v1.0\
C:\windows\System32\OpenSSH\
C:\Program Files\Intel\WiFi\bin\
C:\Program Files\Common Files\Intel\WirelessCommon\
C:\Program Files\Docker\Docker\resources\bin
C:\ProgramData\DockerDesktop\version-bin
C:\HashiCorp\Vagrant\bin
C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\MSBuild\Current\Bin\
C:\Program Files (x86)\Microsoft SQL Server\150\DTS\Binn\
C:\Program Files\Microsoft SQL Server\Client SDK\ODBC\130\Tools\Binn\
C:\Program Files (x86)\Microsoft SQL Server\130\Tools\Binn\
C:\Program Files\Microsoft SQL Server\130\Tools\Binn\
C:\Program Files\Microsoft SQL Server\130\DTS\Binn\
C:\Program Files (x86)\Google\Chrome\Application
C:\Program Files\nodejs\
C:\Program Files\dotnet\
C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\Common7\IDE\CommonExtensions\Microsoft\TestWindow

C:/opscode/chef-workstation/embedded/git/usr/bin

This is the result of the shell-init at the launching of the shell. When i run refreshenv, i get the path i would expect from my system settings:
~\Documents\github> refreshenv
Refreshing environment variables from the registry for powershell.exe. Please wait...
Finished
~\Documents\github> path
C:\opscode\chef-workstation\bin\
C:\Program Files\Microsoft VS Code\bin
C:\ProgramData\chocolatey\bin
C:\Program Files\PowerShell\7\
C:\Program Files\Git\cmd
C:\Program Files (x86)\Common Files\Oracle\Java\javapath
C:\windows\system32
C:\windows
C:\windows\System32\Wbem
C:\windows\System32\WindowsPowerShell\v1.0\
C:\windows\System32\OpenSSH\
C:\Program Files\Intel\WiFi\bin\
C:\Program Files\Common Files\Intel\WirelessCommon\
C:\Program Files\Docker\Docker\resources\bin
C:\ProgramData\DockerDesktop\version-bin
C:\HashiCorp\Vagrant\bin
C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\MSBuild\Current\Bin\
C:\Program Files (x86)\Microsoft SQL Server\150\DTS\Binn\
C:\Program Files\Microsoft SQL Server\Client SDK\ODBC\130\Tools\Binn\
C:\Program Files (x86)\Microsoft SQL Server\130\Tools\Binn\
C:\Program Files\Microsoft SQL Server\130\Tools\Binn\
C:\Program Files\Microsoft SQL Server\130\DTS\Binn\
C:\Program Files (x86)\Google\Chrome\Application
C:\Program Files\nodejs\
C:\Program Files\dotnet\
C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\Common7\IDE\CommonExtensions\Microsoft\TestWindow

C:\Users\mcascone1\AppData\Local\Microsoft\WindowsApps
C:\Users\mcascone1\AppData\Roaming\npm
C:\Users\mcascone1\.dotnet\tools

Why would the shell-init be putting my user path in there? Especially with a chefdk component in the path?
Anyway, i'll try installing with the refreshed path, and report.

Alright, now i really don't understand.
I reset my path, set $env:gem_home and $env:gem_path to $env:gem_root which is C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0.
I even renamed C:\Users\mcascone1\AppData\Local\chefdk to ...chefdk-backup.
I put the gem in my gemfile and ran chef exec bundle install.
It still installs into my chefdk folder! It creates a new one and installs to it!
And i get the same ol' error when running knife vra.
Where is that path configured?

this is still an issue, on multiple machines; any ideas?

i tried installing on a completely different machine and got exactly the same result. I can't even knife rehash with this gem installed.
@dheerajd-msys this is going to become a big embarrassing issue for me in the very near future, how can i get this gem to work?
PS C:\Users\hl_tfsbuild> chef gem info knife-windows

*** LOCAL GEMS ***

knife-windows (4.0.5, 1.9.6)
    Author: Seth Chisamore
    Homepage: https://github.com/chef/knife-windows
    License: Apache-2.0
    Installed at (4.0.5): C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0
                 (1.9.6): C:/Users/hl_tfsbuild/AppData/Local/chefdk/gem/ruby/2.7.0

    Plugin that adds functionality to Chef's Knife CLI for
    configuring/interacting with nodes running Microsoft Windows
PS C:\Users\hl_tfsbuild> chef gem install knife-vrealize
Fetching knife-vrealize-6.0.0.gem
WARNING:  You don't have c:\users\hl_tfsbuild\appdata\local\chefdk\gem\ruby\2.7.0\bin in your PATH,
          gem executables will not run.
Successfully installed knife-vrealize-6.0.0
Parsing documentation for knife-vrealize-6.0.0
Installing ri documentation for knife-vrealize-6.0.0
Done installing documentation for knife-vrealize after 14 seconds
1 gem installed
PS C:\Users\hl_tfsbuild> chef gem info knife-vrealize

*** LOCAL GEMS ***

knife-vrealize (6.0.0)
    Author: Chef Software
    Homepage: https://github.com/chef/knife-vrealize
    License: Apache-2.0
    Installed at: C:/Users/hl_tfsbuild/AppData/Local/chefdk/gem/ruby/2.7.0

    Chef Infra Knife plugin to interact with VMware vRealize.

PS C:\Users\hl_tfsbuild> knife vra catalog list
FATAL: Cannot find subcommand for: 'vra catalog list'
If this is a recently installed plugin, please run 'knife rehash' to update the subcommands cache.

PS C:\Users\hl_tfsbuild> knife rehash
Traceback (most recent call last):
        16: from C:/opscode/chef-workstation/bin/knife:370:in `<main>'
        15: from C:/opscode/chef-workstation/bin/knife:370:in `load'
        14: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/bin/knife:24:in `<top (required)>'
        13: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/application/knife.rb:163:in `run'
        12: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife.rb:224:in `run'
        11: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife.rb:161:in `subcommand_class_from'
        10: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:129:in `command_class_from'
         9: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:115:in `load_command'
         8: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `load_commands'
         7: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `each'
         6: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `block in load_commands'
         5: from C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0/gems/chef-16.4.41-universal-mingw32/lib/chef/knife/core/subcommand_loader.rb:105:in `load'
         4: from C:/Users/hl_tfsbuild/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:26:in `<top (required)>'
         3: from C:/Users/hl_tfsbuild/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:27:in `<class:Chef>'
         2: from C:/Users/hl_tfsbuild/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:28:in `<class:Knife>'
         1: from C:/Users/hl_tfsbuild/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:29:in `<class:Cloud>'
C:/Users/hl_tfsbuild/AppData/Local/chefdk/gem/ruby/2.7.0/gems/knife-vrealize-6.0.0/lib/chef/knife/vra_server_create.rb:30:in `<class:VraServerCreate>': uninitialized constant Chef::Knife::Cloud::VraServerCreate::VraServiceHelpers (NameError)
Did you mean?  Chef::Knife::Cloud::VraServiceOptions
PS C:\Users\hl_tfsbuild> chef gem uninstall knife-vrealize
Successfully uninstalled knife-vrealize-6.0.0

Based on this github issue, i got some more insight but i still don't know how to fix it.
prior to running chef shell-init powershell | iex, I don't have $env:gem_home/path set. After running that, they are set to the "bad" path:
> $env:gem_home
> chef shell-init powershell | iex
> $env:gem_home
C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0
> $env:gem_path
C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0;C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0

But $env:gem_root is still set to the 'good' path:
> $env:gem_root
C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0

The 'bad' path is visible in my Ruby env vars:
> chef gem env | select-string -Context 2 chefdk

    - RUBY VERSION: 2.7.1 (2020-03-31 patchlevel 83) [x64-mingw32]
    - INSTALLATION DIRECTORY: C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0
>   - USER INSTALLATION DIRECTORY: C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0
    - RUBY EXECUTABLE: C:/opscode/chef-workstation/embedded/bin/ruby.exe
    - GIT EXECUTABLE: C:/opscode/chef-workstation/embedded/bin/git.BAT
    - GEM PATHS:
       - C:/opscode/chef-workstation/embedded/lib/ruby/gems/2.7.0
>      - C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0
    - GEM CONFIGURATION:
       - :update_sources => true
    - SHELL PATH:
       - C:/opscode/chef-workstation/bin
>      - C:/Users/mcascone1/AppData/Local/chefdk/gem/ruby/2.7.0/bin
       - C:/opscode/chef-workstation/embedded/bin
       - C:\Program Files\PowerShell\7

I tried setting all the $env:gem_x paths in my windows env vars, and opening a new shell, and those chefdk paths just won't go away.
How can I get this chefdk path out of here, permanently?

@mwrock, you're my hero. Any idea when this will be released?

Should be soon. We are currently prepping a release.

Thank you so much. Do you have any insight on what's going on with the zombie chefdk path, described in detail in the thread? Is it all related to the bug you fixed, or is something hinky with my environment (and on the other env I tested on and had the same problem)?

I think workstation adds that path but if it does not exist it should not have anything to do with the bug.

Just coming back to confirm that this appears to be fixed in v6.0.2. THANK YOU!!

Just coming back to confirm that this appears to be fixed in v6.0.2. THANK YOU!!


Add danger plugin danger-undercover

What happened
Added danger-undercover plugin to show missing test coverage in new changes in PRs with undercover
Insight
As mentioned in this comment in the PR: https://github.com/nimblehq/rails-templates/issues/221 we want to show a report about missing test coverages in new changes with undercover in PRs using Danger.
Proof Of Work
Used in our internal project -

No Changes

Changes in new commits

@longnd @ankitkalia1195 could you have a look on this PR


Suggestion: Work with open files

Hi,
Is there a way to use SharpCompress and have it add files that are currently open by another application?  I'm trying to create a log uploader, but it will fail if a file is currently open by another process.  These are text files.
Thanks
chris

use a Stream and File.Open with different FileAccess settings.  This isn't unique to sharpcompress


Limit page size

Details
Hello Alireza.
I was wondering is there any way to limit page size? for example page size can't be more than 2000 records. for performance and security reasons.

For anyone who had same problem. I implemented pagination my self and used this wonderful package for sorting and filtering using ApplyFilteringAndOrdering() method.

Hi @dev-sajad,
We don't have any limitation for the PageSize, although by default if you don't specify any value it will be set to 20 for performance reasons but you can change it from both client side with passing the desired PageSize or in the backEnd by changing the GlobalConfiguration GridifyGlobalConfiguration.DefaultPageSize = 500000


fix: add type check for packages/remix-netlify-edge

Adds type checking for packages/remix-netlify-edge
Relates to #3104

[ ] Docs
[ ] Tests

Testing Strategy:

Clone this repository
Checkout the branch, e.g. gh co 2
Install all dependencies by running yarn
Run node scripts/build.mjs --deno from the root folder of the project in your shell.
All type checks should pass.

Currently I'm sorting our why it can't find createRequestHandler @ascorbic as it exist from the * export from deno in that file.
➜ node scripts/build.mjs --deno
yarn run v1.22.19
$ /Users/nicktaylor/dev/work/remix/node_modules/.bin/rollup -c

packages/create-remix/cli.ts → build/node_modules/create-remix/dist...
created build/node_modules/create-remix/dist in 315ms

packages/remix/index.ts → build/node_modules/remix/dist...
created build/node_modules/remix/dist in 16ms

packages/remix/index.ts → build/node_modules/remix/dist/esm...
created build/node_modules/remix/dist/esm in 13ms

packages/remix-architect/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/architect/dist in 77ms

packages/remix-cloudflare/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/cloudflare/dist in 50ms

packages/remix-cloudflare-pages/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/cloudflare-pages/dist/esm in 20ms

packages/remix-cloudflare-pages/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/cloudflare-pages/dist in 22ms

packages/remix-cloudflare-workers/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/cloudflare-workers/dist/esm in 22ms

packages/remix-cloudflare-workers/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/cloudflare-workers/dist in 21ms

packages/remix-deno/.empty.js → stdout...

(!) Generated an empty chunk
.empty

packages/remix-dev/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/dev/dist in 568ms

packages/remix-dev/cli.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/dev/dist in 23ms

packages/remix-dev/server-build.ts → build/node_modules/@remix-run/dev, <EMAIL_ADDRESS>created build/node_modules/@remix-run/dev, build/node_modules/@remix-run/dev/dist in 31ms

packages/remix-express/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/express/dist in 21ms

packages/remix-netlify/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/netlify/dist in 21ms

packages/remix-netlify-edge/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/netlify-edge/dist in 16ms

packages/remix-node/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/node/dist in 61ms

packages/remix-react/index.tsx →<EMAIL_ADDRESS>created build/node_modules/@remix-run/react/dist in 418ms

packages/remix-react/index.tsx →<EMAIL_ADDRESS>created build/node_modules/@remix-run/react/dist/esm in 284ms

packages/remix-serve/index.ts, packages/remix-serve/env.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/serve/dist in 11ms

packages/remix-serve/cli.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/serve/dist in 11ms

packages/remix-server-runtime/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/server-runtime/dist in 263ms

packages/remix-server-runtime/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/server-runtime/dist/esm in 241ms

packages/remix-vercel/index.ts →<EMAIL_ADDRESS>created build/node_modules/@remix-run/vercel/dist in 16ms
Done in 3.02s.
Import map diagnostics:
  - Invalid top-level key "comment". Only "imports" and "scopes" can be present.
Download https://esm.sh/v99/<EMAIL_ADDRESS>Check file:///Users/nicktaylor/dev/work/remix/packages/remix-netlify-edge/index.ts
error: TS2614 [ERROR]: Module '"https://esm.sh/<EMAIL_ADDRESS>has no exported member 'createRequestHandler'. Did you mean to use 'import createRequestHandler from "https://esm.sh/<EMAIL_ADDRESS>instead?
import { createRequestHandler as createRemixRequestHandler } from "@remix-run/deno";
         ~~~~~~~~~~~~~~~~~~~~
    at file:///Users/nicktaylor/dev/work/remix/packages/remix-netlify-edge/server.ts:1:10

TS2614 [ERROR]: Module '"https://esm.sh/<EMAIL_ADDRESS>has no exported member 'ServerBuild'. Did you mean to use 'import ServerBuild from "https://esm.sh/<EMAIL_ADDRESS>instead?
import type { ServerBuild } from "@remix-run/deno";
              ~~~~~~~~~~~
    at file:///Users/nicktaylor/dev/work/remix/packages/remix-netlify-edge/server.ts:2:15

Found 2 errors.
Error: deno exited with code 1
    at ChildProcess.<anonymous> (file:///Users/nicktaylor/dev/work/remix/scripts/build.mjs:48:16)
    at ChildProcess.emit (node:events:513:28)
    at maybeClose (node:internal/child_process:1100:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:304:5)

Looking again at the Deno handler, it's getting us in a bit of a mess. The signature doesn't allow you to pass in the context, and once it calls getLoadContext the context object isn't in scope. I think the simplest solution is to call the version of createRequestHandler from server-runtime directly. Rather than importing it from there, we could re-export it as createRemixRequestHandler from @remix-run/deno. That then allows us to pass in the context directly.

Thanks @ascorbic, I was toying with using it from server-runtime directly yesterday for the reasons you mentioned, so sounds like I'm on the right track.

Still sorting out some type issues:
error: Module not found "file:///Users/nicktaylor/dev/work/remix/packages/remix-netlify-edge/node_modules/@remix-run/server-runtime/dist/routes".
    at file:///Users/nicktaylor/dev/work/remix/packages/remix-netlify-edge/node_modules/@remix-run/server-runtime/dist/build.d.ts:3:42
Error: deno exited with code 1
    at ChildProcess.<anonymous> (file:///Users/nicktaylor/dev/work/remix/scripts/build.mjs:47:16)
    at ChildProcess.emit (node:events:513:28)
    at maybeClose (node:internal/child_process:1100:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:304:5)

The error is because you need to update the versions in the package.json. Right now because the versions don't match it's installing them from npm instead of the version from the monorepo.


Sql generation and combining labmda expressions

My goal is to apply multiple conditions via OR. The actual number of conditions is defined dynamicaly and i would really like to avoid constructing them as expressions from scratch. My actual code is here:
                Expression<Func<Shedule, bool>> ex1 = s => s.Id == 1;
                Expression<Func<Shedule, bool>> ex2 = s => s.Id == 2;

                var ex3 = ex1.Or(ex2);

                var q = _context.DiosShedule.Where(ex3);

                var list = await q.ToListAsync();

I also had to add following codes to substitute parameter expression in the expressions so it would be the same one everywhere:
public class ParameterRebinder : ExpressionVisitor
    {
        private readonly Dictionary<ParameterExpression, ParameterExpression> map;

        public ParameterRebinder(Dictionary<ParameterExpression, ParameterExpression> map)
        {
            this.map = map ?? new Dictionary<ParameterExpression, ParameterExpression>();
        }

        public static Expression ReplaceParameters(Dictionary<ParameterExpression, ParameterExpression> map, Expression exp)
        {
            return new ParameterRebinder(map).Visit(exp);
        }

        protected override Expression VisitParameter(ParameterExpression p)
        {
            ParameterExpression replacement;

            if (map.TryGetValue(p, out replacement))
            {
                p = replacement;
            }

            return base.VisitParameter(p);
        }
    }

    public static class Utility
    {

        public static Expression<T> Compose<T>(this Expression<T> first, Expression<T> second,
            Func<Expression, Expression, Expression> merge)
        {

            // build parameter map (from parameters of second to parameters of first)
            var map = first.Parameters.Select((f, i) => new {f, s = second.Parameters[i]})
                .ToDictionary(p => p.s, p => p.f);

            // replace parameters in the second lambda expression with parameters from the first
            var secondBody = ParameterRebinder.ReplaceParameters(map, second.Body);

            // apply composition of lambda expression bodies to parameters from the first expression
            return Expression.Lambda<T>(merge(first.Body, secondBody), first.Parameters);
        }

        public static Expression<Func<T, bool>> And<T>(this Expression<Func<T, bool>> first, Expression<Func<T, bool>> second)
        {
            return first.Compose(second, Expression.And);
        }

        public static Expression<Func<T, bool>> Or<T>(this Expression<Func<T, bool>> first, Expression<Func<T, bool>> second)
        {
            return first.Compose(second, Expression.Or);
        }

    }

Well, this actually works but the resulting sql is sub-obtimal:
SELECT s.id, s.classroomid, s.comment, s.finish, s.groupid, s.groupid_old, s.planid, s.start, s.teacherid, s.teacherid_old
FROM dios.shedule AS s
WHERE (CASE
    WHEN s.id = 1
    THEN TRUE::bool ELSE FALSE::bool
END OR CASE
    WHEN s.id = 2
    THEN TRUE::bool ELSE FALSE::bool
END) = TRUE

It doesn't use the PK index on "id" column and performs full scan of the table.
How can i optimize the code to pass existing lambdas to EF Core and get proper SQL?
Further technical details
EF Core version: 2.2.0
Database Provider: Npgsql.EntityFrameworkCore.PostgreSQL 2.2.0
Operating system: Windows 10 Professional
IDE: Visual Studio 2017 Community 15.9.8

Expression.And & Expression.Or are functions to create expression which represent bitwise AND & OR respectively. Since bitwise operator works on bit values, the condition was changed into case block and  you are seeing such SQL.
What you need is Expression.AndAlso & Expression.OrElse which represent logical AND & OR.


weights in Exp4.P are incorrect

https://github.com/ntucllab/striatum/blob/master/striatum/bandit/exp4p.py#L90
I found that in the second round (the model has been rewarded), w will be something like this:
{0: array([ 2.36154256,  2.03240672,  2.00984322,  2.01720784,  2.220576  ]), 1: array([ 2.36064009,  2.03172242,  2.00917387,  2.01653361,  2.21976697])}
which I think it's not the same as the original algorithm (every expert has only one weight value).
This doesn't generate error because the np.sum here:
https://github.com/ntucllab/striatum/blob/master/striatum/bandit/exp4p.py#L98
incidentally sum all the value, so there won't be any array with wrong dimension.
Is this a bug? I will update this issue after reading the reward part. Maybe I can find the answer myself.

I think the problem is here:
https://github.com/ntucllab/striatum/blob/master/striatum/bandit/exp4p.py#L191
should be context[i][k] / query_vector[k]

https://github.com/ntucllab/striatum/blob/master/striatum/bandit/exp4p.py#L193
this line is also incorrect...

https://github.com/ntucllab/striatum/blob/master/striatum/bandit/exp4p.py#L195
this line is also incorrect (wrong T and K)

Yeah, you are correct. It's a bug giving wrong query_vector.


White spaces in role names are not supported in Admin portal

Description
When trying to add scope mappings to roles that contain whitespace in their name, the whitespace(s) are not supported at the Admin portal.
Steps to Reproduce

Create a role name with the white spaces from the carbon Console.
Try to add it as the role name from the Admin portal.

Affected Component
APIM
Version
3.2.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

This issue is not reproducible in the master branch since it has been fixed in the latest releases.


Add repl.it to online compiler

Added repl.it to online compiler

The site under the link does not work.
Also, a strange description for a online compiler.

I am surprised that the link didn't work for you. I just tried it without logging in and it worked for me. I could start coding there as an anonymous user. The description was copied from their website and they certainly are working towards becoming more than an online compiler overall.

Not the link, there was cloudflare dummy page with message that the actual site is down. Now it is working.


Support Subpasses for Vulkan

Description
We need to implement support for subpasses for vulkan/gl/gles

WAiting till WebGPU

done


cmake error：install TARGETS given target "scap" which does not exist in this directory.

cmake -DUSE_BUNDLED_DEPS=ON ../;
-- Libs version: 0.14.0-409+c2fd308
-- Driver version: 6.1.0-409+c2fd308-driver
-- Kernel version: 4.19.90-17.ky10.x86_64
-- Driver API version 8.0.0
-- Driver schema version 2.14.0
-- Configuring done
-- Generating done
-- Build files have been written to: /root/libs/build/googletest-download
[100%] Built target googletest
-- Using bundled uthash in '/root/libs/build/uthash-prefix/src/uthash/src'
-- No strlcpy found, will use local definition
-- No strlcat found, will use local definition
-- Using bundled zlib in '/root/libs/build/zlib-prefix/src/zlib'
-- Using bundled libelf: include'/root/libs/build/libelf-prefix/src/libelf/libelf', lib: /root/libs/build/libelf-prefix/src/libelf/libelf/libelf.a
-- Using bundled protobuf in '/root/libs/build/protobuf-prefix/src/protobuf'
-- Using bundled jsoncpp in '/root/libs/build/jsoncpp-prefix/src'
CMake Error at cmake/modules/libscap.cmake:124 (install):
install TARGETS given target "scap" which does not exist in this directory.
Call Stack (most recent call first):
CMakeLists.txt:109 (include)
-- Using bundled tbb in '/root/libs/build/tbb-prefix/src/tbb'
-- Using bundled c-ares in '/root/libs/build/c-ares-prefix/src/c-ares'
-- Using bundled openssl in '/root/libs/build/openssl-prefix/src/openssl'
-- Using SSL for curl in '/root/libs/build/openssl-prefix/src/openssl/target'
-- Using zlib for curl in '/root/libs/build/zlib-prefix/src/zlib'
-- Using bundled curl in '/root/libs/build/curl-prefix/src/curl'
-- Using bundled valijson in '/root/libs/build/valijson-prefix/src/valijson'
-- Using bundled re2 in '/root/libs/build/re2-prefix/src/re2'
-- Using bundled tinydir in '/root/libs/build/tinydir-prefix/src/tinydir'
-- Using jsoncpp: include: /root/libs/build/jsoncpp-prefix/src/include/, lib: /root/libs/build/jsoncpp-prefix/src/lib/libjsoncpp.a
-- Using re2: include: /root/libs/build/re2-prefix/src/re2/include, lib: /root/libs/build/re2-prefix/src/re2/lib/libre2.a
-- Using bundled grpc in '/root/libs/build/grpc-prefix/src/grpc'
-- Using jsoncpp: include: /root/libs/build/jsoncpp-prefix/src/include/, lib: /root/libs/build/jsoncpp-prefix/src/lib/libjsoncpp.a
-- Download all scap-files from: https://download.falco.org/fixtures/libs/scap_files
-- Downloading scap-file: https://download.falco.org/fixtures/libs/scap_files/kexec_arm64.scap
-- Skipping download, file already present
-- Downloading scap-file: https://download.falco.org/fixtures/libs/scap_files/kexec_x86.scap
-- Skipping download, file already present
CMake Error at cmake/modules/libsinsp.cmake:89 (install):
install TARGETS given target "sinsp" which does not exist in this
directory.
Call Stack (most recent call first):
CMakeLists.txt:110 (include)
CMake Warning at test/e2e/CMakeLists.txt:7 (message):
e2etestscanonlyberunwiththeeBPFprobe
-- Configuring incomplete, errors occurred!
See also "/root/libs/build/CMakeFiles/CMakeOutput.log".
See also "/root/libs/build/CMakeFiles/CMakeError.log".

ei @YanjiangChen could you provide more info about your system? architecture,  the cmake version used, ...
Moreover, could you try a clean build with latest libs master https://github.com/falcosecurity/libs/commit/02877435e21afd7bdd6715ffd7c865a2f01ae682:
git clone https://github.com/falcosecurity/libs.git
cd libs
mkdir build && cd build
cmake -DUSE_BUNDLED_DEPS=On ..

The previous problem is solved, but there is a new problem， CMake Error at cmake/modules/libsinsp.cmake:88 (install):
install TARGETS given target "sinsp" which does not exist in this
directory.
cmake version==3.12.1, architecture is x86_64, system is kylin，maybe you haven't heard of it, similar to centos, rhel..

The problem is still not solved after using the latest version。
I also encountered another problem when executing cmake
install TARGETS given target "sinsp" which does not exist in this
directory.
The following is the prompt in CMakeError：
Determining if the pthread_create exist failed with the following output:
Change Dir: /root/libs/build/CMakeFiles/CMakeTmp
Run Build Command:"/usr/bin/gmake" "cmTC_97567/fast"
/usr/bin/gmake -f CMakeFiles/cmTC_97567.dir/build.make CMakeFiles/cmTC_97567.dir/build
gmake[1]: 进入目录“/root/libs/build/CMakeFiles/CMakeTmp”
Building C object CMakeFiles/cmTC_97567.dir/CheckSymbolExists.c.o
/usr/bin/cc   -Wall -ggdb    -o CMakeFiles/cmTC_97567.dir/CheckSymbolExists.c.o   -c /root/libs/build/CMakeFiles/CMakeTmp/CheckSymbolExists.c
Linking C executable cmTC_97567
/usr/bin/cmake -E cmake_link_script CMakeFiles/cmTC_97567.dir/link.txt --verbose=1
/usr/bin/cc  -Wall -ggdb     CMakeFiles/cmTC_97567.dir/CheckSymbolExists.c.o  -o cmTC_97567
/usr/bin/ld: CMakeFiles/cmTC_97567.dir/CheckSymbolExists.c.o: in function main': /root/libs/build/CMakeFiles/CMakeTmp/CheckSymbolExists.c:8: undefined reference to pthread_create'
collect2: 错误：ld 返回 1
gmake[1]: *** [CMakeFiles/cmTC_97567.dir/build.make:87：cmTC_97567] 错误 1
gmake[1]: 离开目录“/root/libs/build/CMakeFiles/CMakeTmp”
gmake: *** [Makefile:121：cmTC_97567/fast] 错误 2
Determining if the include file sys/mkdev.h exists failed with the following output:
Change Dir: /root/libs/build/CMakeFiles/CMakeTmp
Run Build Command:"/usr/bin/gmake" "cmTC_8b47c/fast"
/usr/bin/gmake -f CMakeFiles/cmTC_8b47c.dir/build.make CMakeFiles/cmTC_8b47c.dir/build
gmake[1]: 进入目录“/root/libs/build/CMakeFiles/CMakeTmp”
Building C object CMakeFiles/cmTC_8b47c.dir/CheckIncludeFile.c.o
/usr/bin/cc   -Wall -ggdb    -o CMakeFiles/cmTC_8b47c.dir/CheckIncludeFile.c.o   -c /root/libs/build/CMakeFiles/CMakeTmp/CheckIncludeFile.c
/root/libs/build/CMakeFiles/CMakeTmp/CheckIncludeFile.c:1:10: 致命错误：sys/mkdev.h：没有那个文件或目录
#include <sys/mkdev.h>
^~~~~~~~~~~~~
编译中断。
gmake[1]: *** [CMakeFiles/cmTC_8b47c.dir/build.make:66：CMakeFiles/cmTC_8b47c.dir/CheckIncludeFile.c.o] 错误 1
gmake[1]: 离开目录“/root/libs/build/CMakeFiles/CMakeTmp”
gmake: *** [Makefile:121：cmTC_8b47c/fast] 错误 2
The following is the system information：
cmake version==3.12.1, architecture is x86_64, system is kylin，maybe you haven't heard of it, similar to centos, rhel..

The problem is solved. The cmake version is too low.

ok thank you! just out of curiosity which cmake version did you use before?

cmake version is 3.12.1

The result of make driver is scap.ko, how to convert it to falco.ko and load it into it?

make drvier errr：
/root/libs/build/driver/src/ppm_events.c: 在函数‘ppm_copy_from_user’中:
/root/libs/build/driver/src/ppm_events.c:94:48: 错误：宏“access_ok”传递了 3 个参数，但只需要 2 个
if (likely(ppm_access_ok(VERIFY_READ, from, n)))
^
In file included from ./include/linux/export.h:45:0,
from ./include/linux/linkage.h:7,
from ./arch/x86/include/asm/cache.h:5,
from ./include/linux/cache.h:6,
from ./include/linux/time.h:5,
from ./include/linux/compat.h:10,
from /root/libs/build/driver/src/ppm_events.c:13:
/root/libs/build/driver/src/ppm_events.c:50:41: 错误：‘access_ok’ undeclared (first use in this function); did you mean ‘access_flags’?
#define ppm_access_ok(type, addr, size) access_ok(type, addr, size)
^
./include/linux/compiler.h:76:40: 附注：in definition of macro ‘likely’
define likely(x) __builtin_expect(!!(x), 1)
                                    ^

/root/libs/build/driver/src/ppm_events.c:94:13: 附注：in expansion of macro ‘ppm_access_ok’
if (likely(ppm_access_ok(VERIFY_READ, from, n)))
^~~~~~~~~~~~~
/root/libs/build/driver/src/ppm_events.c:50:41: 附注：每个未声明的标识符在其出现的函数内只报告一次
#define ppm_access_ok(type, addr, size) access_ok(type, addr, size)
^
./include/linux/compiler.h:76:40: 附注：in definition of macro ‘likely’
define likely(x) __builtin_expect(!!(x), 1)
                                    ^

/root/libs/build/driver/src/ppm_events.c:94:13: 附注：in expansion of macro ‘ppm_access_ok’
if (likely(ppm_access_ok(VERIFY_READ, from, n)))
^~~~~~~~~~~~~
/root/libs/build/driver/src/ppm_events.c: 在函数‘ppm_strncpy_from_user’中:
/root/libs/build/driver/src/ppm_events.c:129:54: 错误：宏“access_ok”传递了 3 个参数，但只需要 2 个
if (!ppm_access_ok(VERIFY_READ, from, bytes_to_read)) {
^
/root/libs/build/driver/src/ppm_events.c:50:41: 错误：‘access_ok’ undeclared (first use in this function); did you mean ‘access_flags’?
#define ppm_access_ok(type, addr, size) access_ok(type, addr, size)
^
/root/libs/build/driver/src/ppm_events.c:129:8: 附注：in expansion of macro ‘ppm_access_ok’
if (!ppm_access_ok(VERIFY_READ, from, bytes_to_read)) {
^~~~~~~~~~~~~
make[6]: *** [scripts/Makefile.build:304：/root/libs/build/driver/src/ppm_events.o] 错误 1
make[5]: *** [Makefile:1598：module/root/libs/build/driver/src] 错误 2
make[5]: 离开目录“/usr/src/kernels/4.19.90-52.22.v2207.ky10.x86_64”
make[4]: *** [Makefile:18：all] 错误 2
make[4]: 离开目录“/root/libs/build/driver/src”
make[3]: *** [driver/CMakeFiles/driver.dir/build.make:57：driver/CMakeFiles/driver] 错误 2
make[3]: 离开目录“/root/libs/build”
make[2]: *** [CMakeFiles/Makefile2:793：driver/CMakeFiles/driver.dir/all] 错误 2
make[2]: 离开目录“/root/libs/build”
make[1]: *** [CMakeFiles/Makefile2:800：driver/CMakeFiles/driver.dir/rule] 错误 2
make[1]: 离开目录“/root/libs/build”
make: *** [Makefile:268：driver] 错误 2
kernel  version is 4.19.90-52.22

it seems like you are compiling the libraries using an old version different from master(5.0.0 means this is an old code version, actual API version is 8.0.0).
If you compile everything using the master commit all should work out of the box like i wrote here
https://github.com/falcosecurity/libs/issues/1513#issuecomment-1829615843

What I pulled is the code of the master branch. The ko generated by make driver is indeed unusable, but the falco code is not the latest.

falco version is 0.36.2, libs version is matser lastest

I modified this paragraph(in this libs/build/driver/src/ppm_events.c). It originally looked like this.
#define ppm_access_ok(type, addr, size) access_ok(type, addr, size)
Because it couldn't be compiled, this error was reported.:
/root/libs/build/driver/src/ppm_events.c:94:13: 附注：in expansion of macro ‘ppm_access_ok’
if (likely(ppm_access_ok(VERIFY_READ, from, n)))
^~~~~~~~~~~~~
/root/libs/build/driver/src/ppm_events.c: 在函数‘ppm_strncpy_from_user’中:
/root/libs/build/driver/src/ppm_events.c:129:54: 错误：宏“access_ok”传递了 3 个参数，但只需要 2 个
if (!ppm_access_ok(VERIFY_READ, from, bytes_to_read)) {
After the modification, it looked like this.
#define ppm_access_ok(type, addr, size) access_ok(addr, size)
Then the compiled version could not be loaded and the following error was reported:
Error: Driver supports API version 8.0.0, but running version needs 5.0.0

make sinsp err:
/root/falco/build/libsinsp/cri-v1alpha2.pb.cc: 在构造函数‘constexpr runtime::v1alpha2::VersionRequestDefaultTypeInternal::VersionRequestDefaultTypeInternal(
’中:
/root/falco/build/libsinsp/cri-v1alpha2.pb.cc:26:76: 错误：temporary of non-literal type ‘runtime::v1alpha2::VersionRequest’ in a constant expression
: _instance(::PROTOBUF_NAMESPACE_ID::internal::ConstantInitialized{}) {}
^

I have compiled falco and it runs fine, thanks for your help


Portal Footer Cant remove

Category

[ ] Question
[x] Bug
[ ] Enhancement

Portal footer still appears after i have removed the custom action from the site collection.

Prior to this happening i removed the footer to edit and then readded footer.  Was working fine until i added a new site page.  After adding site page footer appeared with two lines in it.
The default line on top and my custom footer below it with a single entry for the footer in custom actions.  I when through and removed custom action but the default footer still remains and i cant find an entry for it anywhere.

I am receiving this error as well. In addition, One of my associated hub sites is displaying the collab Footer in addition to the portal Footer.  This occurred after switching the Site Design from Collaboration to Teams in an attempt to clear the duplicate portal footers.

After I uploaded the sharepoint-starter-kit.sppkg version 1.5.3.  the footer is duplicate.  I removed the footer from the hub communication site using the command Remove-PnPCustomAction -Identity "PortalFooter" the footer is not removed from other associated communication sites.

@pik10 You would need to execute the command on all sites where the footer was loaded. Removing the footer from the hub site, would not remove it from the associated sites.


[BUG]

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior (if applicable):
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

What is your domain name??? I am not psychic.


Create Project Plan

Get information of project and Create project plan document.

Kick-off meeting with client.
Get Draft requirement.
Create project plan document.

Kick-off meeting with client - Meeting note

Requirements

Build a online e-commerce for selling toys and teddy bears for children.
The system can support online payment for Vietnamese Market.
The client can monitor real-time sale statistics and get report for sales.

Time-line for project is 1 month.

Technical request is NONE.

@NguyenThienBao Create Project Plan

Using document template from https://sceweb.sce.uhcl.edu/helm/RUP_Folder/RationalUnifiedProcess/process/templates.htm


fix(curriculum): missing back-tick for telephone-validator

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #55975

Hi @akhilk2802. Congrats on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We are happy to accept these changes and look forward to future contributions. 📝

Thank you, will contribute more


Release updates to the online version from time to time

I'm having some troubles logging in/deplying to firebase from the CLI, but we should learn that from @rhyolight to update the WWW version from time to time.

I believe I created an app through the web ui and then linked it from the
cli.

Matt Taylor
OS Community Flag-Bearer
Numenta
On Mon, Dec 14, 2015 at 11:40 AM, breznak<EMAIL_ADDRESS>wrote:

I'm having some troubles logging in/deplying to firebase from the CLI, but
we should learn that from @rhyolight https://github.com/rhyolight to
update the WWW version from time to time.
—
Reply to this email directly or view it on GitHub
https://github.com/nupic-community/nupic.visualizations/issues/59.

https://github.com/nupic-community/nupic.visualizations/issues/22#issuecomment-170673376

Just redeployed latest code to https://nupic-visualizations.firebaseapp.com/


deps: updates to latest wapc-go

Description
This updates wazero to 1.0.0-pre.2 which improves WebAssembly performance.
Issue reference
n/a
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests

/ok-to-test


Add Pixelmator.app

trial version
:latest version, as only unversioned download is available

Hi!  Thanks!
I think, since Pixelmator.app can only be purchased via the App Store, that we have decided against including it in the past.  Is that still the case?

Hey @rolandwalker, I believe that is still the case, yes.
And with this in mind, decided against including it does make sense.

Thanks.  It's mostly the confusion that would arise from the need to uninstall in order to take the upgrade path.


Tests failing when checking out from master

I just checked out from master and got these errors when running the test suite:
Failures:

  1) Users With default settings change a LDAP password should change password
     Failure/Error: @user.change_password!("secret")
     NoMethodError:
       undefined method `ldap_auth_password_builder' for Devise:Module
     # ./lib/devise_ldap_authenticatable/ldap/adapter.rb:29:in `update_own_password'
     # ./lib/devise_ldap_authenticatable/model.rb:28:in `change_password!'
     # ./spec/unit/user_spec.rb:50:in `block (4 levels) in <top (required)>'

  2) Users With default settings check group membership w/out admin bind should return true for user being in the users group
     Failure/Error: ::Devise.ldap_check_group_membership_without_admin = true
     NoMethodError:
       undefined method `ldap_check_group_membership_without_admin=' for Devise:Module
     # ./spec/unit/user_spec.rb:176:in `block (4 levels) in <top (required)>'

  3) Users With default settings check group membership w/out admin bind should return false for user being in the admins group
     Failure/Error: ::Devise.ldap_check_group_membership_without_admin = true
     NoMethodError:
       undefined method `ldap_check_group_membership_without_admin=' for Devise:Module
     # ./spec/unit/user_spec.rb:176:in `block (4 levels) in <top (required)>'

  4) Users With default settings check group membership w/out admin bind should return false for a user being in a nonexistent group
     Failure/Error: ::Devise.ldap_check_group_membership_without_admin = true
     NoMethodError:
       undefined method `ldap_check_group_membership_without_admin=' for Devise:Module
     # ./spec/unit/user_spec.rb:176:in `block (4 levels) in <top (required)>'

I went through and it seems all of this goes back to PR #165 where ldap_check_group_membership_without_admin and ldap_auth_password_builder were removed.
From what I gather both of these attributes does become obsolete given that commit, but there are still tests around that depends on this functionality. And the documentation refers to these attributes.
Is the functionality that were backed by these options no longer valid? In which case the backing tests ought to be removed together with the documentation mentioning them.
I can take it up on myself to deal with this if I just know how it's supposed to be handled. :)

:+1: Getting this as well

Any update to this? Tests are still failing


Support easy manipulation of size of a blip - for example shift+drag left or right or shift and + or - when hovering

It would be convenient to increase and decrease the size of blips in the radar.
Options for doing so:

hover over element, the press shift together with + or - for inc or dec
use shift + drag left (decrease) or right (increase) (or up or down)

Resolved using context menu - open panel, click on desired size


texlab: 5.2.0 -> 5.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for texlab is: An implementation of the Language Server Protocol for LaTeX
meta.homepage for texlab is: https://texlab.netlify.app
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.3.0 with grep in /nix/store/5a8a6ai7mxi2dfld2pnck2p3bbxr8n6z-texlab-5.3.0
found 5.3.0 in filename of file in /nix/store/5a8a6ai7mxi2dfld2pnck2p3bbxr8n6z-texlab-5.3.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
texlab

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5a8a6ai7mxi2dfld2pnck2p3bbxr8n6z-texlab-5.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A texlab https://github.com/r-ryantm/nixpkgs/archive/f1698cbb38f9c5c4c13284e1e48d808868089618.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5a8a6ai7mxi2dfld2pnck2p3bbxr8n6z-texlab-5.3.0
ls -la /nix/store/5a8a6ai7mxi2dfld2pnck2p3bbxr8n6z-texlab-5.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

texlab

Maintainer pings
cc @doronbehar @kira-bruneau for testing.

Result of nixpkgs-review pr 218573 run on x86_64-linux 1

1 package built:

texlab


Fix typo in documentation example

Aww that's unfortunate! Thanks for submitting the pull request!

This will go into https://github.com/vinniefalco/Beast/commits/b37

Merged to master, thanks for the submission!


linux: add more constants and FUTEX_OP for futex

Add FUTEX_BITSET_MATCH_ANY, FUTEX_OP_* constant and a const fonction, FUTEX_OP() to replace the macro in C.

@bors r+

:pushpin: Commit 7f9965ce609fa0734100437cb3b447ecc8344d7c has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 7f9965ce609fa0734100437cb3b447ecc8344d7c with merge c5c6b5173c6db286b7496b24840bc2322ac11afe...

:broken_heart: Test failed - checks-actions

Hi, I've removed the 'const' which was causing some issue with the previous commit. Also, I think making this function was anyway incorrect :)

@bors r+

:pushpin: Commit c40f8e43480ef37f65235fdcb1465adeb9685b9f has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit c40f8e43480ef37f65235fdcb1465adeb9685b9f with merge 4dc004192b54f93871418a9e17a7d30554c601e3...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 4dc004192b54f93871418a9e17a7d30554c601e3 to master...


[NFC] Stored property has non-sendable type 'FileRuleDescription.Rule'

Explanation: TargetSourcesBuilder.swift: Stored property 'rule' of 'Sendable'-conforming struct 'FileRuleDescription' has non-sendable type 'FileRuleDescription.Rule'; this is an error in the Swift 6 language mode.
Scope: Non-functional change.
Risk: Very low as it's an NFC change removing a compiler warning.
Testing: Manually verified that the warning is resolved.
Issue: rdar://131439553
Reviewer: @bnbarham

@swift-ci test


File transfers from agent return random bytes

This recent change introduced a serious regression where sometimes the agent-streamed job files consist of random bytes.
Also improve gRPC random stream cancellation we sometime observe.

Coverage increased (+0.001%) to 92.928% when pulling<PHONE_NUMBER>ab9a1d7b6d73c66ec4e45a80df7730 on mprimi:GENIE-639-agent-file-garbage into 8e382d00f7260ebee103cbec4bec2ac03c262d71 on Netflix:master.


docs: add Astro integration page

Adds an Astro page per discussion in https://github.com/vanilla-extract-css/vanilla-extract/discussions/678#discussioncomment-7496065 -> #1266.
@bejzik8 answered flawlessly so this shamelessly steals that explanation 😄. Hence the co-author attribute below.
Co-authored-by: Mirko Basic<EMAIL_ADDRESS>

Thank you @JoshuaKGoldberg!