This PR is some minor clean-ups to using helper utilities in sim/periodic. The goal is to more consistently use helper utilities to make sure we are doing the same thing in the same ways.
Updates:
update to use compute_nsamples more consistently
add a new helper function to compute the time length of a single cycle
update to use compute_cycle_nseconds in simulation functions
Note for review: this should be a pretty straight forward update, I think, but given it changes some basic underlying estimations when simulating signals, it's probably worth keeping an eye on if this changes any sims in any unexpected ways (I haven't noticed anything so far).
This PR is some minor clean-ups to using helper utilities in
sim/periodic
. The goal is to more consistently use helper utilities to make sure we are doing the same thing in the same ways.Updates:
compute_nsamples
more consistentlycompute_cycle_nseconds
in simulation functionsNote for review: this should be a pretty straight forward update, I think, but given it changes some basic underlying estimations when simulating signals, it's probably worth keeping an eye on if this changes any sims in any unexpected ways (I haven't noticed anything so far).