Ryan Ly
Ryan Ly
I like `IncludeDatasetSpec` or maybe `InnerDatasetSpec`. Good point. I forgot that use case. No, those should be allowed in this new spec type.
Just noting another instance where this `IncludeDatasetSpec` (or whatever we want to call it) class, automatic checks for nested type definitions, and appropriate documentation around these would have prevented copied...
Tests are failing because the base PR is too out of date. I will merge and we can review in the base PR.
We should do this for all DynamicTableRegions that are small enough (e.g.,
Looks good, thanks!
I suggested a few corrections to comments, but otherwise this looks good to me! Sorry for the delay in review.
Thanks @mavaylon1 !
In PyNWB, consider allowing the user to pass all devicemodel arguments to device constructor which creates a devicemodel behind the scenes with the same name + " Model" and links...
> > In PyNWB, consider allowing the user to pass all devicemodel arguments to device constructor which creates a devicemodel behind the scenes with the same name + " Model"...