Two reasons for this PR:
(i) The current implementation of diagnostic recording fails when the user
specifies a max_time with diagnostics_every_N. This is beacuse _each invocation
of p_State.transition uses the same max_time inside the for loop over
child_n_steps, whereas we really need max_time to apply to the union of
calls to p_State.transition.
(ii) The progress bar is now compatible with diagnostic recording, which
occurs in the inner loop of State.transition.
Parent PR: https://github.com/probcomp/crosscat/pull/107
Two reasons for this PR: (i) The current implementation of diagnostic recording fails when the user specifies a
max_time
withdiagnostics_every_N
. This is beacuse _each invocation of p_State.transition uses the samemax_time
inside the for loop overchild_n_steps
, whereas we really needmax_time
to apply to the union of calls to p_State.transition.(ii) The progress bar is now compatible with diagnostic recording, which occurs in the inner loop of State.transition.