Skip to content

Conversation

@agirodi
Copy link
Contributor

@agirodi agirodi commented Jun 20, 2023

This pull request adds docstrings to the methods of the FastSurferCNN files and converts recon_surf docs to Numpy
The conventions from the Numpy were mostly used for the docstrings.

The changes are not complete, as some descriptions are missing. The critical parts are marked with a "[MISSING]" to find them more easily.

TODOS:

  • Fill/check marked descriptions

@m-reuter m-reuter mentioned this pull request Jun 29, 2023
@agirodi agirodi force-pushed the development/docstrings branch from 3a8b8db to 06b6539 Compare July 11, 2023 09:13
@agirodi agirodi changed the title Add and edit docstrings of FastSurferCNN Add and edit docstrings Jul 11, 2023
@agirodi agirodi force-pushed the development/docstrings branch from 9647aa6 to b4176c2 Compare July 11, 2023 10:01
@agirodi
Copy link
Contributor Author

agirodi commented Jul 11, 2023

The docstrings has now been converted to Numpy using pyment and fixing any wrong changes. Changes from PR#340 has been merged. This branch should be ready for merging now

@agirodi agirodi marked this pull request as ready for review July 11, 2023 11:05
@agirodi agirodi force-pushed the development/docstrings branch from b4176c2 to e92aa01 Compare July 11, 2023 14:22
@m-reuter m-reuter changed the title Add and edit docstrings Add and edit Numpy docstrings Jul 12, 2023
@m-reuter m-reuter merged commit 44210b6 into Deep-MI:dev Jul 12, 2023
@agirodi agirodi deleted the development/docstrings branch September 28, 2023 09:03
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.

2 participants