Luigi Petrucco comments

Results 90 comments of


                                            Luigi Petrucco

Option to have a Tsd as an index

Or you mean I should exclude tensors?

Option to have a Tsd as an index

I'll come back to this soon if still unaddressed! Since I am at it, I'll ask a slightly related but separate question: why is `TsIndex` not a `Ts` but a...

Option to have a Tsd as an index

This should now good to go! I did a minor refactoring of test classes for readability, hopefully that's fine. Also, I'd love to do a separate PR for tests cleanup...

Option to have a Tsd as an index

Let mw know if anything else is needed here! (In the test refactoring, move test generated files to temporary folders would be very useful to avoid spurious commits)

Option to have a Tsd as an index

Ok, done it! I also change a bit the logic to bit more interpretable, and fixed the tests accordingly

Option to have a Tsd as an index

Should have fixed stuff and rebased on dev head. Would be great if we can merge so that I won't have to resolve more conflicts in the future!

Added Support for loading time co-ordinates

Writing here to express a +1 for the need for this and to follow further conversations on timestamp loading

Add the ability to import skeletons

Out of curiosity, how come you don't consider anywhere in the current implementation of the `movement` data the skeleton? For me it is important enough to deserve a dedicated entry...

Add the ability to import skeletons

I think 0 sofistication is required here, and the minimal representation of list of edges is by far the most convenient one. Would not add the burden of dealing with...

How to deal with high baseline firing rates

Awesome, thanks! I will try out the branch.