Eric Denovellis comments

Results 118 comments of


                                            Eric Denovellis

PositionGroup.fetch_position_info() breaks if merge ids order does not match chronological order of the positions

That makes sense. Thank for filing the issue!

The parameter `local_radius_um` in `SpikeSorterParameters` table for the row of `clusterless_thresholder` has been renamed to `raidus_um`

We should be able to update the spyglass image. It's just going to take time to make sure everything works.

WIP: Napari playback of spyglass entries

Let's make sure to make napari and the viewer optional.

Nwb-object oriented ingestion

I thought about this some more and I think we should make this a priority in the medium term if you have the time. There's probably other things that could...

MoSeq import issue

Just to note here that one potential solution is to have different dependencies per pipeline. Would have to change ci/cd to test for this per pipeline dependency install.

Decoding cleanup issues

Happy to have it removed if it is redundant

Decoding cleanup issues

I don't know if we have to worry about it yet. Ultimately, it would be nice to shift the decoding results to use the nwb extension for xarray so that...

Add AGENTS.md for LLM guidance on Spyglass standards and conventions

Possibly should wait to merge until https://github.com/LorenFrankLab/spyglass/pull/1377 is merged