Kyu Hyun Lee

@alejoe91 @h-mayorquin Thanks so much! I think it's worth having an option to save all the waveforms and not just the mean. For example, many quality metrics require all the...

@zm711 @samuelgarcia sorry I forgot about this PR. In general I'm happy to let you guys make any changes to the PR as long as the functionality is preserved (which...

I think setting the channel names to `HWChan N` (where N is channel index) would be clearer.

@samuelgarcia @JuliaSprenger could you please review this? would like this functionality on spikeinterface asap

@zm711 it seems the test is failing to read analog signals because what is passed to `_get_analogsignal_chunk` is `block_index = 0, seg_index = 0, i_start = 0, i_stop = 1024,...

@zm711 I think I understand, but what exactly needs to be done to address this concern then? How does the reader automatically detect the possible streams? What is the appropriate...

@h-mayorquin thanks for the thoughtful response in the nwb files that our lab generates, the `rel_x`, `rel_y`, and `rel_z` of each electrode are unique _within the electrode group_ that it...

> Thanks for the explanation. All of this makes sense. First, to focus on your problem at hand. I think the NWBRecordingExtractor should take an optional list of the electrode...

@alejoe91 no the recording loads, just with non-unique channel locations. @h-mayorquin yeah we can sort per probe. I was just alerting you guys to this because it seems like an...

@h-mayorquin oh sorry, I take that back if it sounds harsh. I think what you said is reasonable and I dont have any better ideas. this isn't a big deal...