Skip to content

Conversation

@mergennachin
Copy link
Contributor

No description provided.

@pytorch-bot
Copy link

pytorch-bot bot commented Jun 28, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/12097

Note: Links to docs will display an error until the docs builds have been completed.

⏳ 28 Pending, 1 Unrelated Failure

As of commit 2195fd8 with merge base db8ef6b (image):

FLAKY - The following job failed but was likely due to flakiness present on trunk:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 28, 2025
@mergennachin mergennachin added the release notes: none Do not include this in the release notes label Jun 28, 2025
@mergennachin mergennachin requested a review from leafs1 June 28, 2025 16:53
@mergennachin mergennachin changed the title Fine tune atol and rtol in test_fp16_exp in backends/xnnpack/test/ops/test_exp.py Skip exp float16 test in XNNPACk Jun 28, 2025
@mergennachin mergennachin merged commit 268806c into main Jun 28, 2025
193 of 195 checks passed
@mergennachin mergennachin deleted the rtol_atol branch June 28, 2025 18:09
@mergennachin
Copy link
Contributor Author

@pytorchbot cherry-pick --onto release/0.7 -c fixnewfeature

pytorchbot pushed a commit that referenced this pull request Jun 30, 2025
@pytorchbot
Copy link
Collaborator

Cherry picking #12097

The cherry pick PR is at #12119 and it is recommended to link a fixnewfeature cherry pick PR with an issue. The following tracker issues are updated:

Details for Dev Infra team Raised by workflow job


# TODO (leafs1): Fix flaky tests. Land fix asap
# and cherry-pick onto release/0.7 branch
@unittest.skip(reason="For float16, numerical discepancies are too high")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mcr229 should we increase atol/rtol for this, or normalize/range-select inputs?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants