Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Custom prefix (X-) for arbitrary columns in .tsv files #86

Open
yarikoptic opened this issue Oct 25, 2024 · 3 comments
Open

Custom prefix (X-) for arbitrary columns in .tsv files #86

yarikoptic opened this issue Oct 25, 2024 · 3 comments
Labels
consistency Aspect requiring special treatment/logic outside of generic common principles metadata Changes to metadata fields/files.

Comments

@yarikoptic
Copy link
Contributor

yarikoptic commented Oct 25, 2024

This is a twin for

which is IMHO more obvious/easier to address. Relates to

in a step towards unification.

Should be easy to automatically migrate on BIDS 1 -to- 2.

@yarikoptic yarikoptic added the consistency Aspect requiring special treatment/logic outside of generic common principles label Oct 25, 2024
@yarikoptic yarikoptic added the metadata Changes to metadata fields/files. label Oct 25, 2024
@effigies
Copy link

Just a note that it is often convenient for fields in JSON and columns in TSV to have names that are valid identifiers, which are typically /[A-Za-z0-9_]+/.

@yarikoptic
Copy link
Contributor Author

so ___ or 0 is valid too? Should we aim to constrain more, e.g. /[A-Za-z][A-Za-z0-9_]*/ if we are to not enforce camel-vs-snake but at least for it to start with a letter?

@effigies
Copy link

At least javascript is happy to have "0" or "__" as keys, but sure, there is a safer regex.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consistency Aspect requiring special treatment/logic outside of generic common principles metadata Changes to metadata fields/files.
Projects
Status: No status
Development

No branches or pull requests

2 participants