-
Notifications
You must be signed in to change notification settings - Fork 111
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
Pickup config from a dataset at some standard location #741
Comments
ha ha - was looking for something like that and found that me-in-the-past already expressed himself. I wish I knew JS enough to not break things, then I would have submitted a PR ;-) |
I found myself again here! amazing! So is there a perspective to have this issue addressed (or at least commented on), or I will just keep talking to the past myself here? ;) |
This is definitely beyond my JS knowledge, so I don't even have a sense of how great an effort this would be. @62442katieb Do you happen to have some time to prod at this? @yarikoptic Do you know any JS developers? |
THANK YOU @nellh -- skimmed through the code (I have little to no JS knowledge, so it was very superficial) -- looks great to me ;) FWIW, in the use case where I was providing config file, I was using absolute path. |
ATM config should be specified in the command line. Since config quite often dataset specific, such configuration would be stored within the dataset. To not breed various user-specific paths, would be nice to standardize location of such a config (e.g.
.bids-validator.cfg
) and pick it up by default. It would then require an option to avoid loading it though (e.g. an empty config path in-c
?)Might interact with #279
The text was updated successfully, but these errors were encountered: