agencyenterprise / ndx-nirs

An NWB extension for storing Near-Infrared Spectroscopy (NIRS) data
BSD 3-Clause "New" or "Revised" License
5 stars 2 forks source link

Fix inconsistency in name/default_name for DynamicTables #11

Closed dsleiter closed 3 years ago

dsleiter commented 3 years ago

The three data types which inherit from DynamicTables (NIRSSourcesTable, NIRSDetectorsTable, and NIRSChannelsTable) have a defined name in the specification, but have a different default name used for class initialization.

Either of the following should be implemented:

  1. the spec should define default_name instead of name and update the default in the class initializer to have the same default value.
  2. the table constructors should not accept a name parameter and instead directly assign the name from the spec

The tentative approach we've discussed is to: