Skip to content

Use AbstractPPL AD interface in tests.#460

Open
yebai wants to merge 10 commits intomainfrom
replace-di-with-abstractppl
Open

Use AbstractPPL AD interface in tests.#460
yebai wants to merge 10 commits intomainfrom
replace-di-with-abstractppl

Conversation

@yebai
Copy link
Copy Markdown
Member

@yebai yebai commented Apr 23, 2026

Replacing #458

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Update the docs example to avoid the removed DifferentiationInterface dependency and coerce vector test inputs to floats before AD prep so discrete vector cases work with AbstractPPL.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

Bijectors.jl documentation for PR #460 is available at:
https://TuringLang.github.io/Bijectors.jl/previews/PR460/

yebai and others added 8 commits April 26, 2026 00:43
Add `_ad_jacobian`/`_ad_gradient` helpers that collapse the
prepare+value_and_* two-step into a single expression, and remove the
unreachable `_to_ad_input(x) = x` catch-all overload.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
`value_and_gradient` / `value_and_jacobian` were renamed to `!!` variants,
the `mode=:jacobian` kwarg on `prepare` was dropped (the 3-arg form now
auto-detects scalar-vs-vector outputs), and `DifferentiationInterface`
must be loaded for AbstractPPL's DI extension to provide
`prepare(::AbstractADType, ...)`.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…o replace-di-with-abstractppl

# Conflicts:
#	src/vector/test_utils.jl
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant