Skip to content

Support append to ElectrodeTable #210

@oruebel

Description

@oruebel

Aappending new rows to the ElectrodesTable after it has been finalized does currently not work correctly. I.e., currently the ElectrodesTable still has to be created in one shot. #209 addressed finalization of ElectrodeTable to support multiple calls to ElectrodeTable.finalize. Appending to location and group_name should in principal already work, as DynamicTable.addColumn was updated in #209 to append values to a column if it already exists. However, the following issues remain:

  • DynamicTable.addReferenceColumn does currently not support appending of values, i.e., appending values after finalize will result in invalid behavior
  • When an ElectrodeTable (or any DynamicTable) is being read from file, then the VectorData should be read from file to make sure we append to existing data. We should check that this actually happens.
  • finalize should be save to call multiple times on both DynamicTable and ElectrodeTable. Add test to confirm that ElectrodeTable::finalize() can be called again when appending to an existing table.

Metadata

Metadata

Assignees

No one assigned

    Labels

    category: bugerrors in the code or code behaviorpriority: highimpacts proper operation or use of a feature important to most users

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions