-
Notifications
You must be signed in to change notification settings - Fork 69
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
DOC: Document steps where config options are used #862
Comments
Yeah we at least added a CI job (I think!) that checks to make sure all In principle instead we could think about trying to mark up the default
Yeah or in addition to or instead of a monolithic table we could figure out how to document it inline with the param itself, like for example Could have a list of all three decoding steps right at the end, since they are all directly affected by the choice (assuming that's the complete set anyway!). Complementary to a table I guess since when looking at a config option you probably wonder "what steps will this option affect?" in which case the inline list is useful; but when looking at a processing step you might wonder "what options are used in this step?" in which case a table is more useful. |
... for the table half, we already have a step table in https://mne.tools/mne-bids-pipeline/dev/features/steps.html , perhaps we could add config options to each table entry. The table would get a bit long but I think that's okay. |
I wasn't thinking of a table to include it in the docs, but what I had it mind was a look-up table (LUT) we could then use to enrich the docs; my idea would be that every parameter gets an Info button or something that, when clicked on, displays which steps will be affected |
Yes
We'd basically have to iterate over all steps and grep the arguments used in
get_config()
and then put that in a table or so…Originally posted by @hoechenberger in #860 (comment)
The text was updated successfully, but these errors were encountered: