search

chembl / chembl_webresource_client

Official Python client for accessing ChEMBL API
https://www.ebi.ac.uk/chembl/api/data/docs
Other
377 stars 95 forks source link

Some print statements left in that print to console. #26

Closed Superraptor closed 7 years ago

Superraptor commented 7 years ago

I noticed that in the file url_query.py lines 290, 291, 292, 296, 297, and 298 print to the console by default. They don't appear to be necessary for function so perhaps adding a "verbose" option would be an interesting idea to suppress the print statements?

(Not necessarily an issue, just something I noticed)

mnowotka commented 7 years ago

Thanks, this actually is an issue since there is a logging mechanism in place so direct prints should be avaided. I will have a look.

mnowotka commented 7 years ago

I guess this is solved in the lastest version, I have a proof: https://github.com/chembl/chembl_webresource_client/search?utf8=%E2%9C%93&q=print&type=