org-arl / arlpy

ARL Python Tools
BSD 3-Clause "New" or "Revised" License
118 stars 37 forks source link

arlpy outputs debug messages to stdout #79

Closed vlaskine closed 1 year ago

vlaskine commented 1 year ago

Various arlpy methods output debug messages to stdout. It makes it impossible to use arlpy in the standard Linux pipeline mode like cat some-input.csv | some-script-using-arlpy | process-output ... It is quite a limitation and inconvenience. The fix is really straightforward.

The places I tripped over are (there may be other cases):

mchitre commented 1 year ago

Fair enough. Would you be willing to PR the changes needed @vlaskine?

vlaskine commented 1 year ago

Thanks, I just created a pull request: https://github.com/org-arl/arlpy/pull/80

It fixes only uwapm, because that's the only part of uwapy that we are currently using and thus that's the only one that I could test.

mchitre commented 1 year ago

Fixed by #80