Skip to content

[AutoDiff] Gardening. #29038

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

Merged
merged 1 commit into from
Jan 7, 2020
Merged

Conversation

dan-zheng
Copy link
Contributor

  • Clean up AutoDiff attribute parsing, printing, type-checking.
  • Rename "wrt parameters" to "differentiability/linearity parameters".
  • Improve @transpose attribute diagnostics.
    • Generalize diagnostic for @transpose attribute invalid original result.
  • Uncomment tests in test/AutoDiff/transpose_attr_type_checking.swift.

NFC except @transpose attribute diagnostic message changes.

Todo: upstream changes to master.

- Clean up AutoDiff attribute parsing, printing, type-checking.
- Rename "wrt parameters" to "differentiability/linearity parameters".
- Improve `@transpose` attribute diagnostics.
  - Generalize diagnostic for `@transpose` attribute invalid original result.
- Uncomment tests in test/AutoDiff/transpose_attr_type_checking.swift.

NFC except `@transpose` attribute diagnostic message changes.

Todo: upstream changes to master.
@dan-zheng dan-zheng added the tensorflow This is for "tensorflow" branch PRs. label Jan 7, 2020
@dan-zheng
Copy link
Contributor Author

@swift-ci Please test tensorflow

@dan-zheng
Copy link
Contributor Author

This may cause a bunch of minor merge conflicts for the next master -> tensorflow merge, sorry about that. I'll upstream changes to master ASAP.

@dan-zheng dan-zheng merged commit 239dd3b into swiftlang:tensorflow Jan 7, 2020
@dan-zheng dan-zheng deleted the autodiff-gardening branch January 7, 2020 16:59
dan-zheng added a commit to dan-zheng/swift that referenced this pull request Jan 7, 2020
Upstream changes from `tensorflow` branch:

- swiftlang#28932:
  deprecate `@differentiable(jvp:vjp)` arguments.

- swiftlang#29038: gardening.

Additional gardening included.
dan-zheng added a commit that referenced this pull request Jan 8, 2020
Upstream changes from `tensorflow` branch:
- #28932: deprecate `@differentiable(jvp:vjp)` arguments.
- #29038: gardening.

Additional gardening included.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tensorflow This is for "tensorflow" branch PRs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants