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

Add workflow and API documentation #203

Open
smoia opened this issue Apr 5, 2020 · 2 comments
Open

Add workflow and API documentation #203

smoia opened this issue Apr 5, 2020 · 2 comments
Assignees
Labels
Documentation This issue or PR is about the documentation

Comments

@smoia
Copy link
Member

smoia commented Apr 5, 2020

Detailed Description

Our documentation lacks API and workflow documentation, i.e. the parts that document how phys2bids works. Adding it would make the life of new collaborators easier - helping them navigating through the code - and improve our transparency and support!

Possible Implementation

This is not an easy task for beginners. The people most familiar with the code should write this (those who wrote part of the code and tests), although all the help is welcome!
We need to think how to structure it, but as an example we could use tedana, bidscoin, fmriprep, etc. etc.

I will open a branch named doc/workflow in my fork (smoia/phys2bids). We can push documentation there, and once it's ready to be merged in the main program, someone else could review it and merge it.

@smoia smoia added the Documentation This issue or PR is about the documentation label Apr 5, 2020
@vinferrer
Copy link
Collaborator

So, You wants us to explain how each of the functions works?

@smoia
Copy link
Member Author

smoia commented Apr 9, 2020

Yep, more or less.
We could add (at least one) diagram to explain the workflow and write a couple of lines to explain better what each function does.
Generally, it means having another look at the docstrings and check that they are complete and make sense in order to prepare the API page (that, I guess, means just printing the numpydocs, like here and the raw code here ).

Maybe it's less of a big task than I imagine though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation This issue or PR is about the documentation
Projects
None yet
Development

No branches or pull requests

4 participants