-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Add __torch_function__ benchmarks. #35530
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
Conversation
💊 CircleCI build failures summary and remediationsAs of commit 369c0ce (more details on the Dr. CI page):
🕵️ 4 new failures recognized by patternsThe following build failures do not appear to be due to upstream breakages:
|
05049cb
to
09ae99a
Compare
24b004e
to
7a84697
Compare
7a84697
to
4b34fb7
Compare
Last time #34645 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You haven't fixed the issue that caused CI to fail?
Mar 26 20:13:22 + python bench.py -n 1 -m 1
Mar 26 20:13:22 ~/workspace/benchmarks/overrides_benchmark ~/workspace
Mar 26 20:13:22 Traceback (most recent call last):
Mar 26 20:13:22 File "bench.py", line 67, in <module>
Mar 26 20:13:22 main()
Mar 26 20:13:22 File "bench.py", line 61, in main
Mar 26 20:13:22 t.__name__, (10 ** 6) * bench_min, (10 ** 6) * bench_std,
Mar 26 20:13:22 UnicodeEncodeError: 'ascii' codec can't encode character '\u03bc' in position 54: ordinal not in range(128)
You may have to push to pytorch/pytorch on a branch named ci-all/your-branch-name
to trigger the relevant ci job
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I assumed it was hitting that error when parsing the file, not when printing the char.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may have to push to pytorch/pytorch on a branch named ci-all/your-branch-name to trigger the relevant ci job
Pushed to ci-all/torch-function-benchmarks
fa968a5
to
369c0ce
Compare
This time, the failures really are unrelated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ezyang is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
I think this broke master:
example build: https://app.circleci.com/pipelines/github/pytorch/pytorch/149594/workflows/e8dd000c-8a01-4938-9fa8-ab9598b46e65/jobs/5020601 |
I apologize, I wonder why the CI didn't catch that. |
Summary: Re-land of pytorch#35530 and pytorch#34645 Pull Request resolved: pytorch#36138 Differential Revision: D20893770 Pulled By: ezyang fbshipit-source-id: 75ab688a086f5fb87412a853df5246c0c39704ca
Since the last one was apparently reverted.