search

aesara-devs / aehmc

An HMC/NUTS implementation in Aesara
MIT License
33 stars 6 forks source link

Fix schedule length for small number of adaptation steps #77

Closed rlouf closed 2 years ago

rlouf commented 2 years ago

The build_schedule function returns a schedule of length num_steps when the number of specified steps is >= 20 and num_steps-1 otherwise. In this PR we fix the latter case.

codecov[bot] commented 2 years ago

Codecov Report

Merging #77 (af0a0ba) into main (28444e7) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #77   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           13        13           
  Lines          545       545           
  Branches        32        32           
=========================================
  Hits           545       545
Impacted Files Coverage Δ
aehmc/window_adaptation.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 28444e7...af0a0ba. Read the comment docs.