Skip to content

Shim PR to handle change to is_autodiff#1554

Closed
SteveBronder wants to merge 2 commits intomasterfrom
shim/if-constexpr-requires
Closed

Shim PR to handle change to is_autodiff#1554
SteveBronder wants to merge 2 commits intomasterfrom
shim/if-constexpr-requires

Conversation

@SteveBronder
Copy link
Copy Markdown
Contributor

Submission Checklist

  • Run unit tests
  • Documentation
    • If a user-facing facing change was made, the documentation PR is here:
    • OR, no user-facing changes were made

Release notes

This changes from using is_autodiff to is_fvar<T> | is_var<T> in the requires for user defined functions. Once stan-dev/math#3240 is merged we can just change to is_autodiff_scalar.

Copyright and Licensing

By submitting this pull request, the copyright holder is agreeing to
license the submitted work under the BSD 3-clause license (https://opensource.org/licenses/BSD-3-Clause)

@SteveBronder
Copy link
Copy Markdown
Contributor Author

Srry need to close this. My local copy of stanc3 is behind in some odd ways and it's easier to make a branch

@WardBrian
Copy link
Copy Markdown
Member

@SteveBronder let me know if it’s easier for me to just make the change, looks straightforward enough

@SteveBronder
Copy link
Copy Markdown
Contributor Author

Sure if you don't mind! It looks like my local opam is bonked in some weird way and I probably have to delete and reinstall the whole thing

@WardBrian
Copy link
Copy Markdown
Member

@SteveBronder -- just a guess, but try opam update && scripts/install_build_deps.sh

@SteveBronder SteveBronder reopened this Sep 26, 2025
@WardBrian WardBrian deleted the shim/if-constexpr-requires branch October 16, 2025 02:48
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.

2 participants