Skip to content

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