adds quiescence_period_refractory_duration generator and curriculum - #155
Conversation
…pec and curriculum
bruno-f-cruz
left a comment
bruno-f-cruz
left a comment
There was a problem hiding this comment.
Probably a stupid question, but cant you just abstract this out since from what @alexpiet said this should be the same as the ITI (which is already being defined in the generator?)?
|
@bruno-f-cruz I could. I thought it would be better to expose it directly in case any one wants the option for it to be different than the iti in the future. Unlikely but now a possibility without too much overhead |
|
Given this is how it was explained by Alex I would hide that implementation detail to make it easier for people to reason about it. But I don't disagree w you about future proofing :p |
|
Switched to using iti distribution for refractory period and also fixed some discrepant values between v1 and v2 curriculums |
|
When quiescence is defined, it doesn't look like the distribution is being resampled. Here are the values with the given distribution. No error now tho :) |
|
Sorry, I was using the wrong type overload for the RNG and always using the same seed. Try now. |
3 participants
Adds quiescence_period_refractory_duration to block based spec and adds to curriculum. Addresses #165