Ryan Ly

Results 297 comments of Ryan Ly

Hi @Christonikos , thanks for the bug report. Could you share the file that produces the error? If it is too big or sensitive to share, I can try to...

> Check out component libraries to see if we can find one that optionally allows time Since most use cases do not involve setting time, I would say the default...

Good questions! > Can you add instructions for this to the dev guide Done > I'd go with Pydantic personally I was mistaken. Pydantic doesn't use the Pydata Sphinx theme....

Currently RTD is configured to display `latest` by default when users go to `nwb-guide.readthedocs.io`. We could configure it to go `stable` instead (I believe `stable` automatically maps to the latest...

> What particular styling / behavior are we interested in replicating? Repositioning to cover the current selection is fairly simple, as is swapping the hover behavior and decoration. On the...

> Something to try in a follow up after this is merged is to rebuild recent release versions (we have linux branches pinned to the commit points to leverage for...

If it's a hassle, I suggest we merge this PR sooner and optimize the styling later.

Yes. Thanks for going through these! 2.2.0 has been released with the fixes from 2.1.1.

@bendichter Can you add 1. a roundtrip test and 2. a backwards compatibility check that checks that age__reference = None if age__reference is not found in the file?

tl;dr I think we should not allow the case where `Subject(age__reference=None)` sets the age_reference to None which means no value is written, and then on read, the value is set...