search

GSTT-CSC / MLOps

Framework for building ML apps
GNU General Public License v3.0
9 stars 5 forks source link

replace print statements with logger #30

Closed laurencejackson closed 2 years ago

saurabh042 commented 2 years ago

Hi @laurencejackson Can you please assign this to me?

laurencejackson commented 2 years ago

Hi saurabh042, thanks for the offer! I'm happy to assign this to you. Please could you fork your branch off develop?

Ideally, we want to see all the existing print statements in mlops.Experiment and mlops.ProjectFile replaced with logging statements using the python logging package. One expectation to this is the Experiment.print_experiment_info method which needs to be printed to stdout - although it would be great if this was logged too.

saurabh042 commented 2 years ago

HI @laurencejackson could you please review the MR #31

saurabh042 commented 2 years ago

Hi @laurencejackson could you please review this PR #32 and let me know if this needs any changes.