Skip to content

ATen layer norm symbolic#10513

Closed
jamesr66a wants to merge 2 commits intopytorch:masterfrom
jamesr66a:layer_norm_symbolic
Closed

ATen layer norm symbolic#10513
jamesr66a wants to merge 2 commits intopytorch:masterfrom
jamesr66a:layer_norm_symbolic

Conversation

@jamesr66a
Copy link
Collaborator

We can't rely on the ATen fallback pathway here because we need to parse out the constant attributes explicitly

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.

jamesr66a 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.

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

@jamesr66a jamesr66a force-pushed the layer_norm_symbolic branch from 5bd7154 to e48859e Compare August 15, 2018 02:51
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.

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

goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
We can't rely on the ATen fallback pathway here because we need to parse out the constant attributes explicitly
Pull Request resolved: pytorch#10513

Reviewed By: dzhulgakov

Differential Revision: D9322133

Pulled By: jamesr66a

fbshipit-source-id: 52af947e6c44532ef220cb4b94838ca838b5df06
@ezyang ezyang added the merged label Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants