Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add input argument to autograd.backward() cpp api #47214

Closed
wants to merge 25 commits into from

Conversation

soulitzer
Copy link
Contributor

@soulitzer soulitzer commented Nov 2, 2020

Helps fix #46373 for the cpp api.

Follow up to #46855 which only changed the api for python only

@facebook-github-bot facebook-github-bot added cla signed oncall: jit Add this issue/PR to JIT oncall triage queue oncall: distributed Add this issue/PR to distributed oncall triage queue labels Nov 2, 2020
@dr-ci
Copy link

dr-ci bot commented Nov 2, 2020

💊 CI failures summary and remediations

As of commit b8abed0 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 18 times.

@codecov
Copy link

codecov bot commented Nov 3, 2020

Codecov Report

Merging #47214 into master will increase coverage by 0.00%.
The diff coverage is 96.87%.

@@           Coverage Diff           @@
##           master   #47214   +/-   ##
=======================================
  Coverage   60.82%   60.82%           
=======================================
  Files        2748     2748           
  Lines      254055   254079   +24     
=======================================
+ Hits       154519   154538   +19     
- Misses      99536    99541    +5     

aten/src/ATen/templates/TensorBody.h Show resolved Hide resolved
aten/src/ATen/templates/TensorBody.h Show resolved Hide resolved
torch/csrc/autograd/VariableTypeManual.cpp Outdated Show resolved Hide resolved
torch/csrc/autograd/autograd.cpp Show resolved Hide resolved
@soulitzer soulitzer changed the title draft Change backward api cpp 2 Add input argument to autograd.backward() cpp api Nov 3, 2020
@soulitzer soulitzer marked this pull request as ready for review November 3, 2020 17:06
@soulitzer soulitzer requested a review from albanD November 3, 2020 22:21
Copy link
Collaborator

@albanD albanD left a comment

Choose a reason for hiding this comment

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

lgtm!

torch/csrc/autograd/VariableTypeManual.cpp Show resolved Hide resolved
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.

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

@facebook-github-bot
Copy link
Contributor

@soulitzer merged this pull request in 2e5bfa9.

@soulitzer soulitzer deleted the change_backward_api_cpp2 branch November 5, 2020 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla signed Merged oncall: distributed Add this issue/PR to distributed oncall triage queue oncall: jit Add this issue/PR to JIT oncall triage queue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add inputs argument to autograd.backward()
3 participants