brainvisa / aims-free

Analysis of Images and Signal for neuroimaging
Other
6 stars 3 forks source link

AimsDicomAnonymize.py is broken #79

Closed ylep closed 2 years ago

ylep commented 2 years ago
$ AimsDicomAnonymize.py
Traceback (most recent call last):
  File "/casa/install/bin/AimsDicomAnonymize.py", line 4, in <module>
    from soma import argparse
ImportError: cannot import name argparse

Environment:

Do we even want to fix that tool, or just throw it away and suggest a better-maintained external tool as an alternative?

denisri commented 2 years ago

No idea, I don't know if this command is used or not. It has been developed within the Cati project.

sapetnioc commented 2 years ago

The deidentification code written long ago by JBM had been rewritten by several people and is not used anymore as far as I know. The CATI now maintain these features in a separate project : https://github.com/cati-neuroimaging/deidentification

ylep commented 2 years ago

The broken code has been removed in https://github.com/brainvisa/aims-free/pull/80