Testing on ftp files #10
Conversation
| yield p | ||
|
|
||
| def add_to_nii_result(folder: str): | ||
| nii_file_list.write(os.path.join(os.path.abspath(folder), 'result.nii') + '\n') |
There was a problem hiding this comment.
'result.nii', 'list.txt' и 'converted' — мейби в константы вынести, раз повторяются? Ну и перенос строки наверняка в Питоне же есть менее платформозасивимый, раз ты в файл записываешь.
| run_on_nii(args.file_list, getattr(mod, args.function_name)) | ||
|
|
||
| elif args.mode == 'dicom': | ||
| pass # TODO |
There was a problem hiding this comment.
Я так понял, что этот TODO надо когда-то доделать? Тогда, хотя бы, надо выводить, что пока что dicom не поддерживается в виде аля предупреждения, а то так будет не очень понятно.
There was a problem hiding this comment.
У меня несколько глобальный вопрос: удобно ли нам этим делом будет пользоваться? В плане того, что в чате я писал, что для CI есть каталог, откуда можно с файловой системы считывать все данные. А локально если нужно проверить - то кажется проще скопировать с FTP и прогнать тоже на локальном каталоге. То есть не очень понятно зачем может понадобиться работа с FTP
Остальное вроде как концептуально удобно - говорим где каталог, проходим по нему и выполняем действия с КТ
No description provided.