Skip to content

C++ parity, grid_sample functional #28354

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

Closed
wants to merge 5 commits into from
Closed

C++ parity, grid_sample functional #28354

wants to merge 5 commits into from

Conversation

lsrock1
Copy link
Contributor

@lsrock1 lsrock1 commented Oct 21, 2019

#25883
I put grid_sample in vision.h with affine grid.

I have a question in string argument(interpolation mode, padding mode)
I reuse torch::native::detail::GridSamplerInterpolation in GridSampler.h instead of using string.
It follows the way that uses reduction enum in loss functions.
I am not sure this is right.

@yf225

Copy link
Contributor

@yf225 yf225 left a comment

Choose a reason for hiding this comment

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

@lsrock1 Thanks so much for the awesome work! I left some comments :D

@lsrock1
Copy link
Contributor Author

lsrock1 commented Oct 22, 2019

@yf225 updated!

@lsrock1 lsrock1 requested a review from yf225 October 22, 2019 06:27
Copy link
Contributor

@yf225 yf225 left a comment

Choose a reason for hiding this comment

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

@lsrock1 Thanks a lot for the update! I left some minor comments.

@lsrock1 lsrock1 requested a review from yf225 October 23, 2019 04:10
@lsrock1
Copy link
Contributor Author

lsrock1 commented Oct 23, 2019

@yf225 check please!

Copy link
Contributor

@yf225 yf225 left a comment

Choose a reason for hiding this comment

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

@lsrock1 Thanks so much for the awesome work! My apologies that I just realized we do need torch/csrc/api/src/nn/options/vision.cpp for Windows builds, and I added it back :D

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@yf225 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@yf225 merged this pull request in e885ce6.

@yf225 yf225 added the module: cpp Related to C++ API label Oct 28, 2019
thiagocrepaldi pushed a commit to thiagocrepaldi/pytorch that referenced this pull request Feb 4, 2020
Summary:
pytorch#25883
I put grid_sample in vision.h with affine grid.

I have a question in string argument(interpolation mode, padding mode)
I reuse torch::native::detail::GridSamplerInterpolation in GridSampler.h instead of using string.
It follows the way that uses reduction enum in loss functions.
I am not sure this is right.

yf225
Pull Request resolved: pytorch#28354

Differential Revision: D18109333

Pulled By: yf225

fbshipit-source-id: 1bf972b671b107464f73b937bbe0de76fb259fbf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged module: cpp Related to C++ API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants