Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

erode module produces TypeError: convert_list_str() missing 1 required positional argument: 'type' #2612

Closed
jcohenadad opened this issue Feb 27, 2020 · 0 comments · Fixed by #2613
Closed

erode module produces TypeError: convert_list_str() missing 1 required positional argument: 'type' #2612

jcohenadad opened this issue Feb 27, 2020 · 0 comments · Fixed by #2613
Labels
bug category: fixes an error in the code priority:HIGH sct_maths context: user requested Raised by user on the SCT forum/email/GitHub. Be sure to notify them when fixed in a release.

Comments

@jcohenadad
Copy link
Member

jcohenadad commented Feb 27, 2020 •
edited

This bug appears when using the -erode feature of sct_maths.

To reproduce:
SCT commit efcb674

cd sct_testing_data/t2
sct_propseg -i t2.nii.gz -c t2
sct_maths -i t2_seg.nii.gz -erode 3,3,3 -o t2_seg_erode.nii.gz

--
Spinal Cord Toolbox (git-master-efcb6746a1a67ae0eee4071a9b9b7ef825c03d84)

Traceback (most recent call last):
  File "/Users/julien/code/sct/scripts/sct_maths.py", line 774, in <module>
    main()
  File "/Users/julien/code/sct/scripts/sct_maths.py", line 342, in main
    data_out = erode(data, convert_list_str(arguments.erode))
TypeError: convert_list_str() missing 1 required positional argument: 'type'

User report:
http://forum.spinalcordmri.org/t/problem-with-latest-version-installation/311/25?u=jcohenadad
@jcohenadad jcohenadad added bug category: fixes an error in the code priority:HIGH sct_maths context: labels Feb 27, 2020
@jcohenadad jcohenadad changed the title TypeError: convert_list_str() missing 1 required positional argument: 'type' erode module produces TypeError: convert_list_str() missing 1 required positional argument: 'type' Feb 27, 2020
@jcohenadad jcohenadad mentioned this issue Feb 27, 2020
Merged
@joshuacwnewton joshuacwnewton added the user requested Raised by user on the SCT forum/email/GitHub. Be sure to notify them when fixed in a release. label Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug category: fixes an error in the code priority:HIGH sct_maths context: user requested Raised by user on the SCT forum/email/GitHub. Be sure to notify them when fixed in a release.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed missing type when using erode feature spinalcordtoolbox/spinalcordtoolbox
2 participants
@jcohenadad @joshuacwnewton