search

paranoya / population-synthesis-toolkit

Spetral synthesis, photometry, SED fitting, etc.
BSD 3-Clause "New" or "Revised" License
3 stars 1 forks source link

Logging #45

Closed paranoya closed 3 weeks ago

paranoya commented 1 month ago

Right now, there are quite a few print statements here and there. We should definitely move to logging; the question is whether this is something for version 1 or version 2

PabloCorcho commented 1 month ago

VERSION 2

PabloCorcho commented 1 month ago

Besides, I don't think a logging facility is really necessary for this library...

paranoya commented 1 month ago

What to do with the print statements? Comment those at "INFO" level, and keep "WARNING"s?

paranoya commented 1 month ago

I actually find many of the "INFO"s useful (e.g. those that initialise the SSP classes). Maybe add a verbose=True parameter to all functions that currently print anything? I think this is the option I like the most...

PabloCorcho commented 1 month ago

Then you have to update all the documentation