-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Add compute_deterministics
helper
#7238
Add compute_deterministics
helper
#7238
Conversation
pm.compute_deterministics
helper
9dbad38
to
ff53eed
Compare
ff53eed
to
2d29175
Compare
pm.compute_deterministics
helpercompute_deterministics
helper
Tests are failing because it's picking up the new scipy but the old arviz. Should work if we rerun again in a couple of hours/days |
@OriolAbril the RTD preview is failing because it installs old arviz. Should we rerun or you are sure the docs are building correctly? |
JAX test is failing with new scipy, should be fixed soon. Since these changes don't touch any related code I'm okay with merging despite them. |
Go ahead, there are still things to fix with step methods so if something were to break I'll open a PR |
Description
After #7206 this utility becomes more relevant.
Also refactored the base logic to apply a Dataset to a PointFunc which was needed in a couple of places (here and in pymc-experimental).
This function could plausible be used by
sample_prior_predictive
andsample_posterior_predictive
, but it seems that for backward compatibility these return and accept more diverse kinds of inputs. Let me know if I am wrong.I am not so sure where to place the
compute_deterministics
. I put it in thesampling
module which is a bit odd perhaps.Related Issue
pm.compute_deterministics
to compute any deterministics on demand #7221Checklist
Type of change
📚 Documentation preview 📚: https://pymc--7238.org.readthedocs.build/en/7238/