Open reubenharry opened 4 months ago
Attention: Patch coverage is 96.15385%
with 5 lines
in your changes missing coverage. Please review.
Project coverage is 98.60%. Comparing base (
20666de
) to head (f4e8064
). Report is 7 commits behind head on main.:exclamation: Current head f4e8064 differs from pull request most recent head 477b11a
Please upload reports for the commit 477b11a to get more accurate results.
Files | Patch % | Lines |
---|---|---|
blackjax/mcmc/adjusted_mclmc.py | 94.54% | 3 Missing :warning: |
blackjax/adaptation/mclmc_adaptation.py | 97.10% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Any advice for how to improve the code coverage here? I'm not entirely sure how to interpret the report.
I think this is ready to review again @junpenglao
EDIT: actually, let me first update the tuning algorithm.
A change is also made to
partially_refresh_momentum
, which allows (adjusted) MCHMC to be the limit of MCLMC with infinite $L$. This is convenient since the user may or not want partial refreshment (in the unadjusted case, we find that it is always beneficial, but don't know yet in the adjusted case)Addresses #676.
Checklist:
main
commit;pre-commit
is installed and configured on your machine, and you ran it before opening the PR;