Skip to content

[pull] main from pytorch:main#586

Merged
pull[bot] merged 8 commits intomcx:mainfrom
pytorch:main
Apr 21, 2025
Merged

[pull] main from pytorch:main#586
pull[bot] merged 8 commits intomcx:mainfrom
pytorch:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Apr 21, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

zxiiro and others added 8 commits April 21, 2025 14:39
The reusable workflow requires OIDC authentication to work and is configured via it's only caller xpu.yml however setting it here too to clarify that it is required. This setting also flags jobs that call this workflow without the required permissions set to remind them it need to be set.

JWT ID token requires `id-token: write` permissions as documented here https://docs.github.com/en/actions/security-for-github-actions/security-hardening-your-deployments/configuring-openid-connect-in-cloud-providers#adding-permissions-settings

Ref: pytorch-fdn/multicloud-ci-infra#3

Pull Request resolved: #151455
Approved by: https://github.com/chuanqi129, https://github.com/atalman
…lways is 0 (#151226)"

This reverts commit 8e5fefe.

Reverted #151226 on behalf of https://github.com/malfet due to Reverting to unblock revert of #151404 ([comment](#151226 (comment)))
…torch.Event (#151404)"

This reverts commit 783be8f.

Reverted #151404 on behalf of https://github.com/malfet due to suspected of breaking linux builds and breaks internal tests as well ([comment](#151404 (comment)))
But don't test them for unsupported dtypes (which is float64 for MPS)
- Skip int64 for log1p for now (next PR will fix that)

Pull Request resolved: #151790
Approved by: https://github.com/Skylion007
@pull pull bot added the ⤵️ pull label Apr 21, 2025
@pull pull bot merged commit 0f8613b into mcx:main Apr 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants