Skip to content

Conversation

@ifedan
Copy link
Contributor

@ifedan ifedan commented Jul 8, 2019

Added implementations: transpose2D transpose3D dilated2D and dilated3D for LongTensor

@pytorchbot pytorchbot added module: internals Related to internal abstractions in c10 and ATen module: nn Related to torch.nn module: operators labels Jul 8, 2019
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.

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

@ifedan ifedan assigned gchanan and unassigned gchanan Jul 8, 2019
@ifedan ifedan requested a review from gchanan July 8, 2019 22:15
} \
}()

#define AT_DISPATCH_FLOATING_TYPES_AND_LONG(TYPE, NAME, ...) \
Copy link
Contributor

Choose a reason for hiding this comment

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

It might be nicer to add a AT_DISPATCH_FLOATING_TYPES_AND(SCALARTYPE, TYPE, NAME, ...
because there is already AT_DISPATCH_FLOATING_TYPES_AND_HALF and we might want a AT_DISPATCH_FLOATING_TYPES_AND_HALF_AND_LONG in the future...

Copy link
Contributor

Choose a reason for hiding this comment

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

agreed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

Copy link
Contributor

@gchanan gchanan left a comment

Choose a reason for hiding this comment

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

looks good if you fix up the dispatch macro, as suggested by @zou3519.

} \
}()

#define AT_DISPATCH_FLOATING_TYPES_AND_LONG(TYPE, NAME, ...) \
Copy link
Contributor

Choose a reason for hiding this comment

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

agreed.

@gchanan
Copy link
Contributor

gchanan commented Jul 16, 2019

Also, please fix the title/commit message. This is clearly not implementing transpose.

@ifedan ifedan changed the title Added implementation of transpose and dilated 2D and 3D for LongTensor Added generation of transpose and dilated 2D and 3D for LongTensor Jul 16, 2019
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.

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

@facebook-github-bot
Copy link
Contributor

@ifedan merged this pull request in 7ed82ea.

zdevito pushed a commit to zdevito/ATen that referenced this pull request Jul 17, 2019
…22594)

Summary:
Added implementations: transpose2D transpose3D dilated2D and dilated3D for LongTensor
Pull Request resolved: pytorch/pytorch#22594

Differential Revision: D16155462

Pulled By: ifedan

fbshipit-source-id: af57330314bc2c3e0a38b9e75105b20030a1f9bb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: internals Related to internal abstractions in c10 and ATen module: nn Related to torch.nn

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants