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

New version: ChainRules v0.7.2 #17266

Merged
merged 1 commit into from
Jul 1, 2020

Conversation

JuliaRegistrator
Copy link
Contributor

UUID: 082447d4-558c-5d27-93f4-14fc19e9eca2
Repo: https://github.com/JuliaDiff/ChainRules.jl.git
Tree: 9d1951a1686e4849bda5469caadf7de593d56dcd

Registrator tree SHA: 58e84c9733c0606ca53349dbce519209d8a8402b
JuliaRegistrator referenced this pull request in JuliaDiff/ChainRules.jl Jun 30, 2020
* Add rules for evalpoly

* Rename pullback

* Apply suggestions from code review

Co-authored-by: Lyndon White <[email protected]>

* Make backx its own method for the future

* Update UniformScaling to-dos

* Add matrix poly tests

* Use correct indices

* Reorganize

* Use generated functions with fallbacks

* Reimplement rules for matrices

* Deactivate complex tests

Until complex conventions are clarified and FiniteDifferences v0.10.0 is supported.

* Add generated functions for tuple case

* Add comment

* Rename defs to exs

* Refactor and test fallbacks

* Simplify indexing

* Don't store output as an intermediate

* Support scalar x with matrix pi

* Make extensible for other ps

* Move fallback tests under rrule tests

* Reorder args and remove unnecessary product

* Eliminate unneeded mul and reorganize

* Remove unnecessary product

* Fix length of ys and wrap lines

* Place final ∂yi to in loop

This for some reason profiles much faster

* Keep other rules consistent with vector

* Unify tests

* Increment version number

* Try equality check outside of tuple

* Approximate check due to muladd

* Approximate check scalar output too

* Decrement version number

Co-authored-by: Lyndon White <[email protected]>
@github-actions
Copy link
Contributor

Your new version pull request met all of the guidelines for auto-merging and is scheduled to be merged in the next round.


If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.

@github-actions github-actions bot merged commit a4b39e2 into master Jul 1, 2020
@github-actions github-actions bot deleted the registrator/chainrules/082447d4/v0.7.2 branch July 1, 2020 00:39
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.

1 participant