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.

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

Set up logger for Heudiconv related functions #36

Open
mirestrepo opened this issue Jul 2, 2020 · 5 comments
Open

Set up logger for Heudiconv related functions #36

mirestrepo opened this issue Jul 2, 2020 · 5 comments
Assignees

Comments

@mirestrepo
Copy link
Contributor

Should we be using the logger for this stuff? Why are we printing instead?

Originally posted by @broarr in https://github.com/brown-bnc/xnat-tools/diffs

@broarr broarr self-assigned this Jul 7, 2020
@broarr
Copy link
Contributor

broarr commented Jul 7, 2020

I want this one! I'm gonna take a crack at it this afternoon

@broarr
Copy link
Contributor

broarr commented Jul 7, 2020

#38 is my first attempt. The heudiconv functions look a little ugly, but it does work

@broarr
Copy link
Contributor

broarr commented Jul 7, 2020

It's double printing things from heudiconv. Not sure why. I'll take a peek at a later point

@mirestrepo
Copy link
Contributor Author

Also, I was running with Fae today and saw that logs are not printed until the processing has finished. I don't recall that behavior when I run the tests, but we should get them streaming... I wonder if there is anything different in the stdout in Oscar...

@broarr
Copy link
Contributor

broarr commented Jul 16, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants