search

mc2-project / mc2

A Platform for Secure Analytics and Machine Learning
Apache License 2.0
297 stars 43 forks source link

Remove print statements before releasing on PyPi #179

Open jochasinga opened 3 years ago

jochasinga commented 3 years ago

There is current a few places where there are still print statements. Can we switch to logger instead?

https://github.com/mc2-project/mc2/blob/bfbec919c0e10a48b3655a4cde5b5a5640f9ef58/python-package/tests/azure/delete_azure_resources.py#L6

jochasinga commented 3 years ago

@chester-leung are these print statements save to remove and use logger instead?

chester-leung commented 3 years ago

Yes, we can replace these print statements with log messages. Currently I don't believe there's a logger configured for the tests as a whole though.