Results 17 comments of Alexander Jones

> I think we should wait on the validation of the `HEDVersion` using a regular expression unless this is standard for BIDS and we have to do it. I would...

We have determined that this issue is *not* related to #967 after all. I do not currently have plans to tackle this issue.

Bumping this. Creating a new map with the same file (or at least the same filename) as an existing map is not allowed by the software, and in any case...

I'm not sure what's going on with the Deno build. The lines in the errors do not appear in the version of that file in our code base, and we...

@rwblair Has there been any further progress on this?

> Thx! In the longer term, we'll probably want to build the HED schema object from the dataset description before validation and pass it in as it doesn't change. We'll...

I believe the failing test is because of an off-by-one error (you're converting the TSV data to the old format, which you shouldn't (`BidsTsvFile`'s constructor now expects either a string...

@rwblair Please merge this. Several real-world datasets are currently failing validation because of issues fixed in this release.

The assignment in https://github.com/bids-standard/bids-validator/blob/6fc6d152b52266934575442e61f1477ba18f42ec/bids-validator/validators/tsv/validate.js#L40 is not showing up upon returning to `fullTest`. `null` is passed as the value of `participants` into the function, so perhaps the assignment is being ignored?...

It appears to be attempting to create a sidecar object with non-existent data. I'm wondering why https://github.com/bids-standard/bids-validator/blob/40206be37d9db2114ed16e8891625f65a1aab449/bids-validator/validators/events/hed.js#L75 is using a `union` rather than an `intersection` - maybe that'll fix it?