Skip to content

Unnecessary print used in the FEAT class #3520

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
Closed
somso2e opened this issue Sep 26, 2022 · 1 comment · Fixed by #3521
Closed

Unnecessary print used in the FEAT class #3520

somso2e opened this issue Sep 26, 2022 · 1 comment · Fixed by #3521

Comments

@somso2e
Copy link
Contributor

somso2e commented Sep 26, 2022

There is an unnecessary print statement used in the _list_outputs method in the FEAT class.
This is especially annoying when running a lot of FEAT analysises since it litters the terminal and messes with other user-defined prints and or with progress bars such as tqdm.

nipype/nipype/interfaces/fsl/model.py

Line 487 in 72aac96

print("Outputs from FEATmodel:", outputs)

As a temporary workaround, you can suppress the print statement like below.

import contextlib
with contextlib.redirect_stdout(None):
        feat = fsl.FEAT()
        feat.inputs.fsf_file = fsf_file
        feat.terminal_output = "none"
        feat_output = feat.run()
@effigies
Copy link
Member

Looks like a debug statement added in #635. Please feel free to open a PR to remove it.

@somso2e somso2e mentioned this issue Sep 26, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Removed leftover debug print statement in FEAT class somso2e/nipype
2 participants
@effigies @somso2e