Skip to content

Adding RandomShortestSize transform#5610

Merged
datumbox merged 2 commits intopytorch:mainfrom
datumbox:transforms/random_shortest_resize
Mar 14, 2022
Merged

Adding RandomShortestSize transform#5610
datumbox merged 2 commits intopytorch:mainfrom
datumbox:transforms/random_shortest_resize

Conversation

@datumbox
Copy link
Copy Markdown
Contributor

@datumbox datumbox commented Mar 14, 2022

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.

@datumbox datumbox requested a review from vfdev-5 March 14, 2022 14:48
@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented Mar 14, 2022 •
edited
Loading

💊 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.

@datumbox datumbox mentioned this pull request Mar 14, 2022
Closed
24 tasks
vfdev-5
vfdev-5 approved these changes Mar 14, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@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
@datumbox @facebook-github-bot
[fbsync] Adding RandomShortestSize transform (#5610)
89557a8 
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

Reviewers

1 more reviewer

@vfdev-5 vfdev-5 vfdev-5 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

ciflow/default cla signed module: reference scripts module: transforms new feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@datumbox @facebook-github-bot @vfdev-5