Skip to content

Support complex pullbacks #23

New issue

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

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

Already on GitHub? # to your account

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

sethaxen
Copy link
Member

Replaces _dot with _realdot, which allows for pulling back through functions with complex outputs. Resolves #22. This is currently untested.

@codecov-commenter
Copy link

Codecov Report

Merging #23 (9bab121) into master (b4343f5) will decrease coverage by 1.15%.
The diff coverage is 16.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #23      +/-   ##
==========================================
- Coverage   78.26%   77.10%   -1.16%     
==========================================
  Files           1        1              
  Lines         345      345              
==========================================
- Hits          270      266       -4     
- Misses         75       79       +4     
Impacted Files Coverage Δ
src/AbstractDifferentiation.jl 77.10% <16.66%> (-1.16%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b4343f5...9bab121. Read the comment docs.

@mohamed82008
Copy link
Member

Could you please add a test here for the intended use case?

@sethaxen
Copy link
Member Author

sethaxen commented Jan 9, 2022

Certainly, though the tests need a few refactors first to make testing these cases more straightforward. Some changes made in #27. More will be necessary.

@sethaxen
Copy link
Member Author

sethaxen commented Jan 9, 2022

Also, this PR should use RealDot.jl now that it exists.

@gdalle gdalle marked this pull request as draft September 20, 2023 11:11
@gdalle gdalle added the feature New feature or request label Oct 5, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Complex pullback support
4 participants