-
Notifications
You must be signed in to change notification settings - Fork 52
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
[WIP] Flip specified drives in input histogram #905
base: master
Are you sure you want to change the base?
[WIP] Flip specified drives in input histogram #905
Conversation
…ts to invert about the y axis when plotting the input histogram
…in plot_spikes_hist in viz.py
…ts to invert about the y axis when plotting the input histogram
…in plot_spikes_hist in viz.py
can you add a tiny unit test to check that the inverted axis is indeed added? |
if invert_spike_types is None: | ||
invert_spike_types = list() | ||
else: | ||
if isinstance(invert_spike_types, dict): |
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.
if isinstance(invert_spike_types, dict): | |
if not isinstance(invert_spike_types, (str, list)): |
Hi Dylan, Great PR.. Just a small suggestion: in the type check for invert_spike_types
, instead of raising a TypeError only when it's a dict
, it would be clearer to ensure it's either a string
or a list
Adding functionality to specify spike types to invert when creating the input histogram.
So far:
Still need to test, etc. More coming soon :)