-
Notifications
You must be signed in to change notification settings - Fork 7.1k
Add typing annotations to detection/rpn #4619
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
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.
A minor type check error and few comments.
Thanks for the PR @oke-aditya. All changes look good to me. We just need to take care of the issue you marked here #4619 (comment) and we should be good to go :) |
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.
LGTM, thanks for the PR.
We can merge once the CI is green.
* Annotate rpn * Small fix * Small fix and ignore Co-authored-by: Vasilis Vryniotis <[email protected]>
Summary: * Annotate rpn * Small fix * Small fix and ignore Reviewed By: NicolasHug Differential Revision: D31758315 fbshipit-source-id: 4d16ddc96f26f4c01676cf2f9f6ab3208917ad8a Co-authored-by: Vasilis Vryniotis <[email protected]>
* Annotate rpn * Small fix * Small fix and ignore Co-authored-by: Vasilis Vryniotis <[email protected]>
Helps #4582
Sorry for the flood of PRs, I luckily have couple of days holiday 😅
cc @datumbox