Oliver Ruebel

Results 180 comments of Oliver Ruebel

@ajtritt is this still an issue or can this be closed?

I agree that having testing infrastructure for extensions will be useful. I'm not sure, whether this will need to be part of the extension catalog, individual extensions, or PyNWB (probably...

Ok, that should not happen. I'll take a look. I opened an icephys files from dandiset 20 yesterday without an issue. I'm wondering whether something strange is happening where things...

@t-b thanks for the additional case. Could you provide a link to the file. As far as I can tell the issue is somehow related to the use of extensions...

@t-b thanks for the clarification. Interesting that the validator fails differently than the normal read here. In particular, it seems that in the log output you provided for the validator,...

> As for why it's only a warning instead of an error, I believe the reasoning was to not break back-compatibility (is that right Yes, that is correct. If we...

@ajtritt what are your thoughts on this? I’d be Ok with changing things to __fields__ in PyNWB if that doesn’t break extensions. @rly I assume this would then also change...

@luiztauffer sounds reasonable. I would make this a note box (possibly at the end of the S3 tutorials, depending on length) to describe the issue, how to check for it,...

@rly this issue would be good to address in the next minor release

> what do you think about also extracting most of the code in `__main__` in `src/pynwb/validate.py` to a function that makes sense