Skip to content

nomnigraph - minor cleanup of Graph.h#9890

Closed
duc0 wants to merge 1 commit intopytorch:masterfrom
duc0:export-D9017108
Closed

nomnigraph - minor cleanup of Graph.h#9890
duc0 wants to merge 1 commit intopytorch:masterfrom
duc0:export-D9017108

Conversation

@duc0
Copy link
Contributor

@duc0 duc0 commented Jul 26, 2018

Summary: Minor cleanups for Graph.h to make it more consistent with our style guide

Differential Revision: D9017108

@duc0 duc0 force-pushed the export-D9017108 branch 4 times, most recently from 4b11835 to 380cf08 Compare July 26, 2018 20:32
Summary:
Pull Request resolved: pytorch#9890

Minor cleanups for Graph.h to make it more consistent with our style guide

Also fix opt/device.cc and binary_match_test.cc to not access subgraph.nodes_ which is now private

Differential Revision: D9017108

fbshipit-source-id: e12037ca36eb3d05f20aaa79e367c8a5f2557b30
@duc0 duc0 force-pushed the export-D9017108 branch from 380cf08 to 502ee0b Compare July 26, 2018 21:53
@ezyang
Copy link
Contributor

ezyang commented Jul 27, 2018

@pytorchbot retest this please

jramseyer pushed a commit to jramseyer/pytorch that referenced this pull request Jul 31, 2018
Summary:
Pull Request resolved: pytorch#9890

Minor cleanups for Graph.h to make it more consistent with our style guide

Also fix opt/device.cc and binary_match_test.cc to not access subgraph.nodes_ which is now private

Reviewed By: bwasti

Differential Revision: D9017108

fbshipit-source-id: 9f5cba4a2cd2a452a955005f4704f6c120bbc1d5
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
Pull Request resolved: pytorch#9890

Minor cleanups for Graph.h to make it more consistent with our style guide

Also fix opt/device.cc and binary_match_test.cc to not access subgraph.nodes_ which is now private

Reviewed By: bwasti

Differential Revision: D9017108

fbshipit-source-id: 9f5cba4a2cd2a452a955005f4704f6c120bbc1d5
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.

2 participants