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

[typing] Tracer: add missing __round__ and __reversed__ methods #12428

Merged
merged 1 commit into from
Sep 20, 2022

Conversation

jakevdp
Copy link
Collaborator

@jakevdp jakevdp commented Sep 20, 2022

These are missing, which means that when the typing.Array abstract base class (#12049) is applied, then instantiating a Tracer leads to an error. This also meant that previously using round(x) within jit was unsupported (although reversed(x) was supported via the fallback to slicing).

@jakevdp jakevdp requested a review from yashk2810 September 20, 2022 16:11
@jakevdp jakevdp self-assigned this Sep 20, 2022
@google-ml-butler google-ml-butler bot added kokoro:force-run pull ready Ready for copybara import and testing labels Sep 20, 2022
@copybara-service copybara-service bot merged commit e855a9c into jax-ml:main Sep 20, 2022
@jakevdp jakevdp deleted the tracer-methods branch September 20, 2022 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kokoro:force-run pull ready Ready for copybara import and testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants