-
Notifications
You must be signed in to change notification settings - Fork 38
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
Added some rules for scaled airy and bessel functions and incomplete gamma #66
Open
amrods
wants to merge
9
commits into
JuliaDiff:master
Choose a base branch
from
amrods:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
406d579
added rule for gamma(a, x)
amrods 79ceb6f
added a few rules for airy and bessel functions
amrods af7dcfd
fixed missing $ in def of zeta
amrods e206f70
Fixed tests for 2-arg gamma
amrods b2957af
Merge branch 'JuliaDiff:master' into master
amrods 236ad7a
Merge branch 'master' into master
devmotion a5ee29d
Update test/runtests.jl
devmotion ce09b11
Update Project.toml
devmotion 813f4e8
Make rules consistent with ChainRules definitions in SpecialFunctions
devmotion File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I always assumed that DiffRules only deals with functions with real arguments and real outputs - but
hankelh1x
,hankelh2x
, and alsohankelh1
andhankelh2
for which a rule is already defined output complex numbers. So complex numbers are supported officially? It seems based on the derivatives forconj
andadjoint
that you added in #54 you also thought that DiffRules only deals with real numbers @mcabbott?