Ryan Ly

Results 316 comments of Ryan Ly

I think I misunderstood the original issue. @jonahpearl are you creating / loading an NWB file and then adding to it / updating it for each step of processing? Or...

This is a good idea. Yes, `ImageSeries` will need to be handled specially.

This feature request also comes up from @lfrank where his lab found data where the length of `electrodes` of an `ElectricalSeries` did not match the second dimension of `data`. It...

The two test failures are fixed in https://github.com/NeurodataWithoutBorders/pynwb/pull/1325 . We will make a new release of PyNWB that includes these changes soon (target: end of January).

Related to https://github.com/NeurodataWithoutBorders/pynwb/pull/1050 and new issue https://github.com/NeurodataWithoutBorders/pynwb/issues/1255 . Initially, the problem with the linked approach of pickling the TypeMap is that it was not compatible with loading NWB extensions from...

Ah, looks like you are still working on this. Happy to help or review it when ready.

I fixed the tests. This looks good to me! Thanks, @CodyCBakerPhD ! Let me know if you think it is good to merge.

@CodyCBakerPhD I was writing a test for the cached extension condition but I was confused about the last commit. Currently with this branch, if I have a file with a...

It's a pretty major breaking change in the behavior of the validate function and CLI. For now, I suggest adding a new flag "--all" to validate against all cached namespaces,...

@kiroul We are doing some last minute preparations for the NWB workshop, so unfortunately, I do not have time right not to look at this issue in detail. I plan...