-
Notifications
You must be signed in to change notification settings - Fork 18
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
High Gamma Button #19
Comments
For some reason we have assumed before that the default was 70-150 Hz. |
@luiztauffer Indeed the default should be 70-150Hz (this is the classic definition of high gamma). All I was saying is to have the option to add bands that are already defined. Currently, if you want to add a new band to the high gamma calculation, you have to enter the center and sigma frequencies yourself. So I'd like there to be the other pre-defined bands (as defined in https://github.com/luiztauffer/process_ecog/blob/docker/ecog/utils/bands.py), available. As a specific example, I'd imagine you click the High Gamma button and you want to create a new high gamma file. The default bands 70-150 Hz are already selected, and greyed out are the other Chang Lab frequency bands. Perhaps each band has a checkbox next to them? So if I wanted to then include up to 200Hz, I could just select those extra few bands. Let me know if that make sense! |
The changes are in place, let me know if that's what you had in mind! |
The dialog box looks great, this is exactly what I had in mind! However, when I try to make the new file, I get this error: We previously discussed that invalid times have to be stored as a separate .csv file. Do you know why it needs to reference this table? In the new nwb file that I'm working off of, there is no invalid times table. I could make an empty one if need be. |
Oops, I should be testing for the existence of this group (and other things, I just came to realize). Thanks for pointing that out! |
Ok, I corrected for this and some other fields present in the files @bendichter provided me with. |
Ah, okay. The high gamma button works properly now for me. So is it linking the data right now? That is, the relative paths must be maintained between the new high gamma file and the original raw file? |
No, the way it's done right now there's no link. You can move your files around and test it! Let me know if anything is failing |
Okay, so right now, everything gets copied into the new files? Having the raw+preprocessed files and the high gamma files be independent is, I think, our goal? Do you think there's a reason for it not to be? Perhaps I'm not grasping what you mean exactly, by how this isn't the best way to do this. |
The way it is implemented right now, you have two options: |
Right, and option 2 is what I envision most people doing. Thanks for clarifying! |
Now the high gamma button works for me.
Can you make the default name option for saving to a separate file be 'ECX_BY_hg.nwb'?
I also like how there's the option to add more bands. Can you make it the option to add both a custom band (which is what there is now), but also to add a band from the typical Chang Lab bands? Like, the preprocessing should be done on bands up to 200Hz. Some people in the lab use 70-200 Hz for high gamma so it'd be super nice to be able to just select (maybe like checkboxes) those extra couple bands above 150Hz.
The text was updated successfully, but these errors were encountered: