-
Notifications
You must be signed in to change notification settings - Fork 14
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
add an easy way for user to add a README and dataset_description to their output #221
Comments
related to #154 |
#154 stalled because it wasn't clear what's the best way to approach this. Thinking about it now, my preferred approach would be to make
Currently, my I like the idea of templating, but I think it's ultimately beyond the scope of @tkkuehn, not sure if you have thoughts on this? |
so more something that should go in the cookiecutter: like a rule that people can opt in ? |
I haven't tried it out, but there is now built-in jinja templating with snakemake: https://snakemake.readthedocs.io/en/stable/snakefiles/rules.html#template-rendering-integration So the rule-based option would be pretty trivial. I agree having this in the cookiecutter and/or example-snakebids app would make sense. |
Have a very basic implementation for this now using cookiecutter and the template integration. Some of the implementation details should be discussed. Briefly:
|
During the dev lunch, we discussed moving this away from the cookiecutter / snakemake integration and generating these files with pure Python and as a Snakebids plugin that is run (potentially as a post-workflow hook). |
as the dataset_description and the README.md are required by the specification
https://bids-specification.readthedocs.io/en/latest/modality-agnostic-files.html
I think there should be a way to automatically add them in the output dataset, possibly by using some of the config to populate them.
Another good plus would be to have an easy way to generate a method section + list of references a la fmriprep, possibly by relying on jinja templates or similar for more flexibility.
The text was updated successfully, but these errors were encountered: