Skip to content

Let callers of the tabular validators supply their own location context instead of a forced FILE_NAME #1394

Description

@VisLab

SpreadsheetValidator.validate and SidecarValidator.validate always push an
ErrorContext.FILE_NAME context from their name argument, and ErrorHandler.push_error_context
turns a missing name into "", so every issue they produce carries an ec_filename key
whether or not the input is a file. For BIDS that is right: one tabular file is one input.
For NWB it is not: one NWB file contains many DynamicTable objects, each validated as
its own tabular input, and a table can exist only in memory before it is written. The
"file" is the NWB file, and the location inside it is the table.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    NWBNeeded for NWB compatibility

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions