add HED as optional column for other TSV files

[sappelhoff]

• some steps towards: https://github.com/bids-standard/bids-validator/issues/1875

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7108f45) 85.43% compared to head (ad9e821) 85.43%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1876   +/-   ##
=======================================
  Coverage   85.43%   85.43%           
=======================================
  Files         131      131           
  Lines        6269     6269           
  Branches     1524     1524           
=======================================
  Hits         5356     5356           
  Misses        806      806           
  Partials      107      107           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[VisLab]

This looks good --- except I don't see samples.tsv and samples.json in this list. Thx!

[VisLab]

Also, I am wondering about the phenotype entry. The phenotype is a directory and the requirement is that phenotype/*.tsv should have a participant_id. Thx

…

On Thu, Jan 4, 2024 at 9:02 AM Kay Robbins ***@***.***> wrote: ***@***.**** commented on this pull request. This looks good --- except I don't see samples.tsv and samples.json in this list. Thx! — Reply to this email directly, view it on GitHub <https://github.com/bids-standard/bids-validator/pull/1876#pullrequestreview-1804343688>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAJCJOQRYI7EPFHLKGOMP23YM3ABPAVCNFSM6AAAAABBMWOBASVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQMBUGM2DGNRYHA> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[sappelhoff]

Also, I am wondering about the phenotype entry. The phenotype is a directory and the requirement is that phenotype/*.tsv should have a participant_id.

You are right about this. There is no phenotype.tsv ... only a phenotype/<whatever-name>.tsv.

I will merge this here and open a new issue.

[sappelhoff]

This looks good --- except I don't see samples.tsv and samples.json in this list. Thx!

• fixes here: https://github.com/bids-standard/bids-validator/pull/1896