Feat: Introduce abstract RateMonitor class for unified rate analysis #1657
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #1368
Based on the great discussions and instructions in the issue (thanks, they were really helpful!), I've started the implementation for a unified rate-handling mechanism.
So far, I've created the abstract
RateMonitor
base class and refactoredPopulationRateMonitor
to inherit from it. I've also added the newbinned()
method toPopulationRateMonitor
to allow for re-binning of the recorded population rate.So I guess ,
EventMonitor
is a bit more complex since it handles generic events, not just spikes. I'll need to look more closely at how to pass the correct variable information to the template for that implementation.Did:
RateMonitor
class.PopulationRateMonitor
to inherit fromRateMonitor
.binned()
method forPopulationRateMonitor
.ToDo:
binned()
method forEventMonitor
to calculate binned rates from event times.