SpikeInterface / spikeinterface-gui

GUI for spikeinterface objects
MIT License
21 stars 7 forks source link

probegroup compatability #77

Open zm711 opened 1 month ago

zm711 commented 1 month ago

This is the reminder for @samuelgarcia to make sure we have probegroup compatibility (mostly in the probeview).

NathanielH-snek commented 1 month ago

Bumping this since I have probegroups (tetrodes) that block the gui from opening

zm711 commented 1 month ago

Hey @NathanielH-snek, just as an fyi this won't be worked on for a bit, which is why I put the reminder for when Sam is back to this project. If you're willing to a pop a reminder in a couple weeks that would be super helpful. If I have some more time I might try to implement it myself, but it won't get merged into the main for a few weeks either way :)

NathanielH-snek commented 1 month ago

Hey @NathanielH-snek, just as an fyi this won't be worked on for a bit, which is why I put the reminder for when Sam is back to this project. If you're willing to a pop a reminder in a couple weeks that would be super helpful. If I have some more time I might try to implement it myself, but it won't get merged into the main for a few weeks either way :)

Alright thanks for letting me know. If I have some time I might see if I can get it to work locally or just disable the probeview because currently some people in my lab are using phy and we've consistently run into some annoying bugs. Thanks for all your work!

zm711 commented 1 week ago

@NathanielH-snek,

Feel free to test #80! I just tested and it worked for me, but it is worth testing multiple people's set-ups.

NathanielH-snek commented 1 week ago

@NathanielH-snek,

Feel free to test #80! I just tested and it worked for me, but it is worth testing multiple people's set-ups.

Working for me on my data, nice!

samuelgarcia commented 1 week ago

cool