Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Arm backend: Provide more debug info for numerical diff issues #3596

Closed
wants to merge 2 commits into from

Conversation

freddan80
Copy link
Collaborator

  • Save torch reference output
  • Print useful debug info
  • Print path to debug artifacts

* Save torch reference output
* Print useful debug info
* Print path to debug artifacts

Change-Id: I14a84a4b4bca5bef315685fdf9472051de2e5ddc
Signed-off-by: Fredrik Knutsson <fredrik.knutsson@arm.com>
Copy link

pytorch-bot bot commented May 14, 2024

🔗 Helpful Links

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

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

❌ 1 New Failure

As of commit f059e47 with merge base 4008600 (image):

NEW FAILURE - The following job has failed:

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 May 14, 2024
@freddan80 freddan80 added partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm ciflow/trunk labels May 14, 2024
@freddan80
Copy link
Collaborator Author

Failing tests not related to this PR. Ready to merge.

@facebook-github-bot
Copy link
Contributor

@digantdesai has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@mergennachin
Copy link
Contributor

@freddan80 have you seen this?

https://pytorch.org/executorch/main/sdk-bundled-io.html

Instead of saving the reference output as a ad-hoc file, we provide utility function to make the testing easier.

@facebook-github-bot
Copy link
Contributor

@digantdesai merged this pull request in bc5ba99.

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. Merged partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants