Skip to content

Conversation

NicolasHug
Copy link
Member

There's a bug in Resize which ignores max_size if size == smallest edge size.

Closes #5405

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Feb 11, 2022

💊 CI failures summary and remediations

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


💚 💚 Looks good so far! There are no failures yet. 💚 💚


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
Contributor

@prabhat00155 prabhat00155 left a comment

Choose a reason for hiding this comment

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

Thanks @NicolasHug!

@NicolasHug NicolasHug merged commit 6fcf0a2 into pytorch:main Feb 15, 2022
facebook-github-bot pushed a commit that referenced this pull request Feb 16, 2022
…one (#5409)

Summary:
* Fix resize when size == small_edge_size and max_size isn't None

* Better test name

Reviewed By: sallysyw

Differential Revision: D34265745

fbshipit-source-id: 8fd1e52581956b7f18c99a2da5c2b4f42d6d1954
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.

Resize ignores max_size when short side equals to target size
3 participants