-
Notifications
You must be signed in to change notification settings - Fork 17
Add events handling: trace/map + new view for aliggned/rasters + PSTHs #218
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Woah - one of the postdocs in my lab will love this! |
|
@alejoe91 I don't think you've commited the eventview |
|
done! |
|
@chrishalcrow ready to test for real! :) |
| self.sampling_frequency = self.analyzer.sampling_frequency | ||
|
|
||
| self.events = None | ||
| if events is not None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets move this events handling in a separate function no ?
| import numpy as np | ||
| from .view_base import ViewBase | ||
|
|
||
| class EventView(ViewBase): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need to discuss the behavior we want.
If I understand correctly this is a psth segment by segment only.
Maybe we should also have a cumulate across segmetn psth no ?
Also the name eventview is to explore event I would call this veiw spikeonevent no ?
|
Cool.
|
Events can be passed to the
run_mainwindoweither as a dict:In case of multi segment, the events must be list of lists/arrays.
Alternatively, one can use a
spikeinterface.BaseEventobject.The Trace/TraceMap views allow to go to previous/next event (and select event type in case of multiple).
A new
EventViewshows aligned rasters or PSTHs.Here's how it looks:
event-sigui.webm