AliceJoubert
AliceJoubert
Close #1327
The function `create_scans_dict` used for AIBL and OASIS converters : https://github.com/aramis-lab/clinica/blob/6f81685bfdb19e455a8f8d1020904fa7beede9f0/clinica/iotools/bids_utils.py#L468-L476 can be very much improved. - It uses inefficient arguments. For example, `ses_dict` is the whole sessions dict used...
As noticed in issue #1299, using the converter with the option `--xml_path` results in a KeyError "image_proc_id" when executing code specific to that option.
Currently, writing sessions.tsv for converter OASIS-to-BIDS takes two steps : - creating a dictionary : https://github.com/aramis-lab/clinica/blob/6f81685bfdb19e455a8f8d1020904fa7beede9f0/clinica/iotools/converters/oasis_to_bids/oasis_to_bids_utils.py#L14-L19 - modifying it to be a dataframe and writing it : https://github.com/aramis-lab/clinica/blob/6f81685bfdb19e455a8f8d1020904fa7beede9f0/clinica/iotools/converters/oasis_to_bids/oasis_to_bids_utils.py#L103 Following #1326,...
Currently, the converter only indicates : - Clinical data conversion - The difference between the result and clinical data (see issue #249) -> too much noise ! - Missing metadata...
As noticed in PR #1285, the function `find_sub_ses_pattern_path` (renamed as `find_images_path`) from `utils/inputs.py` currently returns nothing because it modifies its input : https://github.com/aramis-lab/clinica/blob/954d4a950aa341acf54059c1e93b27c9de18fbc2/clinica/utils/inputs.py#L273-L281 The lists `results` and `error_encountered` are incremented...
Make important information stand out vs details : - required files - how to use / possible options - output to expect Use relevant examples Make sure the info is...
In an effort to harmonize the options of the command lines available in clinica, the option --subjects-lists (-sl) for the converter adni-to-bids should be progressively replaced by the option -tsv...
## Description Improve Clinica test coverage. For non-regression tests : add comparison of images and produced side files. ## Objectives ### Non-regression in converters : _(should we add Bids Validator...