Skip to content

PSROIPool + Dispatcher + Autocast + Code Cleanup #2926

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

Merged
merged 9 commits into from
Oct 30, 2020

Conversation

datumbox
Copy link
Contributor

@datumbox datumbox commented Oct 28, 2020

part of #2797 and #2796

Implements Dispatcher + Autocast for PSROIPool.

I'm also doing some clean up on the original code such as:

  • Remove primitive const declaration from method names.
  • Using references when possible.
  • Fix variable naming.

- Remove primitive const declaration from method names.
- Using references when possible.
- Fix variable naming.
@codecov
Copy link

codecov bot commented Oct 28, 2020

Codecov Report

Merging #2926 into master will increase coverage by 1.07%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2926      +/-   ##
==========================================
+ Coverage   72.33%   73.41%   +1.07%     
==========================================
  Files          99       99              
  Lines        8801     8801              
  Branches     1389     1389              
==========================================
+ Hits         6366     6461      +95     
+ Misses       1997     1915      -82     
+ Partials      438      425      -13     
Impacted Files Coverage Δ
torchvision/models/detection/rpn.py 93.86% <0.00%> (+3.68%) ⬆️
torchvision/models/detection/roi_heads.py 82.34% <0.00%> (+5.10%) ⬆️
torchvision/ops/boxes.py 95.40% <0.00%> (+8.04%) ⬆️
torchvision/ops/poolers.py 98.07% <0.00%> (+11.53%) ⬆️
torchvision/models/detection/transform.py 95.55% <0.00%> (+17.22%) ⬆️
torchvision/ops/_register_onnx_ops.py 85.36% <0.00%> (+36.58%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0125a7d...ff06763. Read the comment docs.

@datumbox datumbox changed the title [WIP] PSROIPool code cleanup PSROIPool + Dispatcher + Autocast + Code Cleanup Oct 30, 2020
Copy link
Member

@fmassa fmassa left a comment

Choose a reason for hiding this comment

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

Looks great, thanks a lot!

@datumbox datumbox requested a review from fmassa October 30, 2020 12:22
Copy link
Member

@fmassa fmassa left a comment

Choose a reason for hiding this comment

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

Merging, thanks a lot!

For the future, let's use ghstack to avoid adding unrelated changes to the PR in a way that helps with separate feature development

@fmassa fmassa merged commit b93d5ee into pytorch:master Oct 30, 2020
@datumbox datumbox deleted the refactor/psroipool_cleanup branch October 30, 2020 13:42
bryant1410 pushed a commit to bryant1410/vision-1 that referenced this pull request Nov 22, 2020
* Fixing types.

* Dispatcher + Autocast.

* + Autograd.

* Formating.

* Clean up and refactor PSROIPool implementation:
- Remove primitive const declaration from method names.
- Using references when possible.
- Fix variable naming.

* Restore include headers.

* New line at end of file.

* Resolving conflict, final cleanup, ordering method consistently across files.
vfdev-5 pushed a commit to Quansight/vision that referenced this pull request Dec 4, 2020
* Fixing types.

* Dispatcher + Autocast.

* + Autograd.

* Formating.

* Clean up and refactor PSROIPool implementation:
- Remove primitive const declaration from method names.
- Using references when possible.
- Fix variable naming.

* Restore include headers.

* New line at end of file.

* Resolving conflict, final cleanup, ordering method consistently across files.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants