Add an extension to support ForwardDiff#586
Add an extension to support ForwardDiff#586timholy wants to merge 2 commits intoJuliaIntervals:masterfrom
Conversation
Because of the extensive changes on `master`, ForwardDiff no longer works with this package. This add an extension (Julia 1.9+) providing at least basic support.
|
Thanks for opening this PR! |
|
That's actually where I started. But I grew to think it might be better this way: ForwardDiff is widely used by a lot of different things, it seems more scalable to have extensions in packages that may have fewer "interop" demands. As a practical matter, ForwardDiff's tests also take a long time to run! But in the end I'm not sure it matters a lot; the extension mechanism ensures the code will be loaded, one way or another. |
|
Good point about scalability. So the main concern here is the reliability. Do we have a guarantee, with your PR, that Unfortunately I am too unfamiliar with ForwardDiff.jl to know the answer. I see that it is used to define |
|
I tried the tests you included in your PR and they seem to pass using the PR #593; I will add them to the test suite to keep track of FowardDiff. |
|
@timholy how is the interaction between IntervalArithmetic (v0.22.14) and ForwardDiff for you? |
|
I haven't returned to this research topic in a while. I'll get back to it eventually, but for the time being...no idea! 🙂 |
|
Closing this for the time being 🙂 |
Because of the extensive changes on
master, ForwardDiff no longer works with this package. This add an extension (Julia 1.9+) providing at least basic support.Note that:
iszero_tupleJuliaDiff/ForwardDiff.jl#667. I am not sure when there will be a new release of ForwardDiff, but the master branch is now0.11-DEV.dev ForwardDiff in thetest` foldernightlyor we have to abandontest/Project.tomland copy everything to the[extras]section of the mainProject.toml.So obviously this can't be merged yet, but I thought it would be better to submit it now.