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

help output mixes MB and GB for memory argument #2440

Closed
tokeefe opened this issue Jul 7, 2021 · 3 comments
Closed

help output mixes MB and GB for memory argument #2440

tokeefe opened this issue Jul 7, 2021 · 3 comments
Labels
bug documentation UX
Milestone
21.0.0

Comments

@tokeefe
Copy link

tokeefe commented Jul 7, 2021

fmriprep --help mixes MB and GB for memory argument

--mem MEMORY_GB, --mem_mb MEMORY_GB, --mem-mb MEMORY_GB
                        upper bound memory limit for fMRIPrep processes
                        (default: None)
@oesteban
Copy link
Member

Thanks for reporting this. --mem_mb and --mem-mb are legacy flag names and we should definitely discontinue them (or at least, change them for corresponding gb versions.

@oesteban oesteban modified the milestones: 21.x, 21.0.0 Jul 19, 2021
@effigies
Copy link
Member

effigies commented Jul 27, 2021 •
edited
Loading

Should have been fixed in #2378.

@effigies
Copy link
Member

@oesteban I would suggest closing this and leaving it as --mem-mb.

@oesteban oesteban closed this as completed Aug 4, 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 documentation UX
Projects
None yet
Milestone
21.0.0
Development

No branches or pull requests
3 participants
@effigies @oesteban @tokeefe