Skip to content

Conversation

@vishwakftw
Copy link
Contributor

cc: @ssnl

@ssnl
Copy link
Collaborator

ssnl commented Jul 15, 2018

This breaks if we ever change <PowBackward>'s number of inputs, but I don't think that will happen. Thanks!

@vishwakftw
Copy link
Contributor Author

@ssnl I don't think there's a way to access the number of inputs via Python to a grad_fn, which is why I had to write the test case that way.

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.

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

facebook-github-bot pushed a commit that referenced this pull request Jul 16, 2018
Reviewed By: apaszke

Differential Revision: D8863572

Pulled By: ezyang

fbshipit-source-id: 13749f51320a4e403644674b0335aed4987fa887
petrex pushed a commit to petrex/pytorch that referenced this pull request Jul 16, 2018
* upstream/master: (24 commits)
  Implement tensor weak references (pytorch#9363)
  Nuke TestCollectEnv (pytorch#9459)
  Add test case for segmentation fault fix in grad_fn (pytorch#9457)
  Add peephole optimization for type_as operators. (pytorch#9316)
  Fix out-of-range error for test_neg (pytorch#9431)
  add depthwise conv support for mkldnn (pytorch#8782)
  Refactor `_log_sum_exp` (pytorch#9173)
  Add ModuleDict and ParameterDict containers (pytorch#8463)
  Introduce SupervisedPtr, delete THAllocator and THCDeviceAllocator (pytorch#9358)
  Introducing IsInf (pytorch#9169)
  add device to CUDAEvent (pytorch#9415)
  Make localScalar error message more intuitive (pytorch#9443)
  Only accept continguous tensors in TopK for cuda (pytorch#9441)
  Add support for .norm() pytorch onnx export and ReduceL1/ReduceL2 caffe2 operators (pytorch#9299)
  Only view() rhs of index_put if we need to (pytorch#9424)
  Add BatchBucketizeOp in caffe2 (pytorch#9385)
  Implementation of Wngrad optimizer caffe2 python wrapper and unit test on least square regression (pytorch#9001)
  Implementation and operator test for Wngrad optimizer (pytorch#8999)
  Fix segmentation fault in grad_fn (pytorch#9292)
  update docs (pytorch#9423)
  ...
@vishwakftw vishwakftw closed this Jul 16, 2018
@vishwakftw vishwakftw deleted the grad_fn-seg-fault-tests branch July 16, 2018 22:53
goldsborough pushed a commit to goldsborough/pytorch that referenced this pull request Jul 20, 2018
Reviewed By: apaszke

Differential Revision: D8863572

Pulled By: ezyang

fbshipit-source-id: 13749f51320a4e403644674b0335aed4987fa887
jramseyer pushed a commit to jramseyer/pytorch that referenced this pull request Jul 30, 2018
Reviewed By: apaszke

Differential Revision: D8863572

Pulled By: ezyang

fbshipit-source-id: 13749f51320a4e403644674b0335aed4987fa887
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Reviewed By: apaszke

Differential Revision: D8863572

Pulled By: ezyang

fbshipit-source-id: 13749f51320a4e403644674b0335aed4987fa887
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.

5 participants