Skip to content

Adding RandomShortestSize transform #5610

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 2 commits into from
Mar 14, 2022

Conversation

datumbox
Copy link
Contributor

Cherrypicking from #5444 the RandomShortestSize.

The specific transform is used to improve the accuracy of the RetinaNet model. It is a porting of the existing GeneralizedRCNNTransform.resize() method, to make it available outside of the model.

@facebook-github-bot
Copy link

facebook-github-bot commented Mar 14, 2022

💊 CI failures summary and remediations

As of commit 0fcc09e (more details on the Dr. CI page):


  • 1/1 failures introduced in this PR

1 failure not recognized by patterns:

Job Step Action
CircleCI cmake_macos_cpu curl -o conda.sh https://repo.anaconda.com/miniconda/Miniconda3-latest-MacOSX-x86_64.sh
sh conda.sh -b
source $HOME/miniconda3/bin/activate
conda install -yq conda-build cmake
packaging/build_cmake.sh
🔁 rerun

This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

Copy link
Collaborator

@vfdev-5 vfdev-5 left a comment

Choose a reason for hiding this comment

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

Thanks @datumbox

@datumbox datumbox merged commit a0867fa into pytorch:main Mar 14, 2022
@datumbox datumbox deleted the transforms/random_shortest_resize branch March 14, 2022 14:56
facebook-github-bot pushed a commit that referenced this pull request Mar 15, 2022
Reviewed By: vmoens

Differential Revision: D34878987

fbshipit-source-id: 0553419e2fdaa251590fbdeddae1cba88c19d023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants