hed-standard / hed-specification

Specification documents for HED (Hierarchical Event Descriptors)
https://hed-specification.readthedocs.io/en/latest/index.html
Creative Commons Attribution 4.0 International
8 stars 11 forks source link

PLACEHOLDER_INVALID for `Duration/#` #610

Closed neuromechanist closed 3 months ago

neuromechanist commented 3 months ago

I am trying to get the task duration from the Duration column:

  "duration": {
    "Description": "Duration of the event, set typically to n/a. Instead, the duration of the event is determined by the onset/offset flags or the next event.",
    "Units": "seconds",
    "HED": {
      "duration": "Duration/#"
    }
  }

, but the HED description throws a placeholder error:

Column 'duration':
    hed string: Duration/#
        PLACEHOLDER_INVALID: There should be no # characters in a category sidecar string. Found 1

This seems to be at odds with the Duration and Delay definitions in the specifications

VisLab commented 3 months ago

You have the wrong level for the sidecar --- you are annotating duration as a category column.

The correct annotation is:

 "duration": {
    "Description": "Duration of the event, set typically to n/a....",
    "Units": "seconds",
    "HED": "Duration/#"
 }
neuromechanist commented 3 months ago

Thanks, it makes sense now 🙌🏼