Skip to content

Conversation

@zasdfgbnm
Copy link
Collaborator

@zasdfgbnm zasdfgbnm commented Mar 24, 2019

Fixes: #14093
cc: @ssnl

@ssnl
Copy link
Collaborator

ssnl commented Mar 24, 2019

Thanks for doing this! I think we should name it something other than numpy_repeat. @gchanan may have good suggestions!

@zasdfgbnm
Copy link
Collaborator Author

@ssnl I was thinking about repeat2 but feeling bad...

Copy link
Contributor

@apaszke apaszke left a comment

Choose a reason for hiding this comment

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

numpy_repeat is not a good name. Maybe repeat_interleave?

@zasdfgbnm
Copy link
Collaborator Author

@apaszke repeat_interleave sounds better, I have renamed it to repeat_interleave

@zasdfgbnm zasdfgbnm changed the title Add torch.numpy_repeat Add numpy like repeat as torch.repeat_interleave Mar 24, 2019
@zasdfgbnm
Copy link
Collaborator Author

@pytorchbot retest this please

@gchanan
Copy link
Contributor

gchanan commented Mar 25, 2019

CC @umanwizard.

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.

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

@ifedan ifedan self-requested a review March 26, 2019 16:25
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.

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

@zasdfgbnm
Copy link
Collaborator Author

Just resolved a merge conflict.

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.

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

@ezyang
Copy link
Contributor

ezyang commented Apr 2, 2019

It failed internal tests but it's not your fault.

@ezyang
Copy link
Contributor

ezyang commented Apr 2, 2019

@umanwizard can you please rebase the internal diff and reland it?

@zasdfgbnm
Copy link
Collaborator Author

@pytorchbot rebase this please

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.

@umanwizard is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

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.

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

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.

@umanwizard is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@umanwizard
Copy link
Contributor

Sorry, I have been trying to land this for the last few days and keep running into issues. Just commenting here @zasdfgbnm to let you know I haven't forgotten about it.

@zasdfgbnm
Copy link
Collaborator Author

@umanwizard Thanks for letting me know! Just take your time, and let me know if there is anything I can help.

@zasdfgbnm zasdfgbnm deleted the numpy_repeat branch April 6, 2019 02:31
zdevito pushed a commit to zdevito/ATen that referenced this pull request Apr 6, 2019
Summary:
Fixes: pytorch/pytorch#14093
cc: SsnL
Pull Request resolved: pytorch/pytorch#18395

Differential Revision: D14599509

Pulled By: umanwizard

fbshipit-source-id: 2391a1cc135fe5bab38475f1c8ed87c4a96222f3
@facebook-github-bot
Copy link
Contributor

@umanwizard merged this pull request in 8c9caf1.

zhangguanheng66 pushed a commit to zhangguanheng66/pytorch that referenced this pull request May 6, 2019
Summary:
Fixes: pytorch#14093
cc: SsnL
Pull Request resolved: pytorch#18395

Differential Revision: D14599509

Pulled By: umanwizard

fbshipit-source-id: 2391a1cc135fe5bab38475f1c8ed87c4a96222f3
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.

[feature request] np.repeat like function

8 participants