Chris Markiewicz comments

Results 2126 comments of


                                            Chris Markiewicz

[SCHEMA] harmonize into dirs, ids, ids_phenotype for "subjects" and "sessions"

Blocked on bids-standard/bids-validator#102.

[SCHEMA] harmonize into dirs, ids, ids_phenotype for "subjects" and "sessions"

This will be impacted by #2107, which removes the phenotype summary fields.

[INFRA] Do check for being a pull request not at job level

I've approved this on the grounds that it's not a big deal and it doesn't cause problems, but I also have not experienced the confusion that this addresses. I would...

[INFRA] Do check for being a pull request not at job level

@yarikoptic This is what I'm seeing: ![image](https://github.com/user-attachments/assets/cc14b963-8ea5-47df-bb3a-f032cf79ae38)

[INFRA] Do check for being a pull request not at job level

I still don't see that in either Firefox or Chromium. I'm going to interpret the lack of additional reviews as a sign others aren't having this problem and close. (I...

false positive 'dead links'

The general problem is an expired SSL certificate. I am surprised that some are undefined, but I guess we will need to do some reconstruction to figure out where these...

reorganize schema section

The README in the spec website is just an accident of `mkdocs` rendering any markdown it finds. It is intended to be descriptive and not at all make statements that...

reorganize schema section

Sorry, just realized this is the website. Yes, if we're linking to the README, we should switch to the bidsschematools RTD.

[STY] add some admonitions to common principles

Okay, if we want to do this, we'll need to switch our macro preprocessing for the PDF to use jinja2, so that the same tools are available to us in...