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

Starting doc overhaul #948

Merged
merged 44 commits into from
Apr 26, 2023
Merged

Starting doc overhaul #948

merged 44 commits into from
Apr 26, 2023

Conversation

qiqiliang
Copy link
Contributor

No description provided.

docs/source/conf.py Outdated Show resolved Hide resolved
@@ -1,3 +1,5 @@
Move to reference
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can delete this now :-)

Suggested change
Move to reference

developed called `Cloudknot <https://nrdg.github.io/cloudknot/>`_.
"""

# ##########################################################################
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can all be uncommented.

`80 bundle atlas <https://figshare.com/articles/Advanced_Atlas_of_80_Bundles_in_MNI_space/7375883>`_.

"""
# import os.path as op
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can all be uncommented

described in [Yeatman2012]_ and further elaborated in [Yeatman2014]_.

"""
# import os.path as op
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can all be uncommented


"""

# import os
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here


"""

# import os.path as op
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here. Etc.

examples.

"""
# import os.path as op
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK - last one.

@36000
Copy link
Collaborator

36000 commented Apr 26, 2023

@arokem I think this is ready to merge?

@arokem arokem changed the title WIP: Starting doc overhaul Starting doc overhaul Apr 26, 2023
@arokem
Copy link
Collaborator

arokem commented Apr 26, 2023

OK - let's merge and then go through and clean up / fill in as needed.

@arokem arokem merged commit 029b496 into yeatmanlab:master Apr 26, 2023
@arokem
Copy link
Collaborator

arokem commented Apr 26, 2023

Congratulations @qiqiliang on all your great work on this pull request! 🎉

@qiqiliang
Copy link
Contributor Author

Awesome! Thank you!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants