search

daducci / COMMIT

Linear framework to combine tractography and tissue micro-structure estimation with diffusion MRI
Other
45 stars 33 forks source link

Tck reader #74

Closed FrancescoGobbi closed 5 years ago

FrancescoGobbi commented 5 years ago

Added features to read .tck files. In the fiber reading the affine matrix is used to allow the correct overlap between the COMMIT outputs between .trk and .tck

FrancescoGobbi commented 5 years ago

Ok, I just committed to edit. Is the rest okay? Do we keep "filename_trk" and "filename_tractogram" to None?

Francesco

Il giorno mar 26 nov 2019 alle ore 14:24 Alessandro Daducci < notifications@github.com> ha scritto:

@daducci commented on this pull request.

In commit/trk2dictionary/trk2dictionary.pyx https://github.com/daducci/COMMIT/pull/74#discussion_r350738220:

@@ -35,6 +35,8 @@ cpdef run( filename_tractogram, path_out, TCK_ref_image = None, filename_peaks =

filename_tractogram : string Path to the .trk or .tck file containing the tractogram to convert. +

  • filename_trk : DEPRECATED

Should be: "filename_trk : DEPRECATED. Use filename_tractogram instead."

(or something similar)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/daducci/COMMIT/pull/74?email_source=notifications&email_token=AKOM274T5AMOIQGQURNZ5GDQVUPRXA5CNFSM4JOZURTKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCNADK6I#pullrequestreview-322975097, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKOM274CXLSVBJ7QR4KWGNDQVUPRXANCNFSM4JOZURTA .