move model logging (#676)

facebookresearch / ClassyVision

An end-to-end PyTorch framework for image and video classification

https://classyvision.ai

MIT License

1.59k stars 277 forks source link

move model logging (#676) #677

Closed scottyshi closed 3 years ago

scottyshi commented 3 years ago

Summary: As mentioned in the task, logging the model usage within the build_model function in __init__.py does not capture all use cases. This diff moves it to the init function of ClassyModel instead, where the logging would capture all use cases.

Pull Request resolved: https://github.com/facebookresearch/ClassyVision/pull/676

Reviewed By: kazhang

Differential Revision: D25728909

Pulled By: scottyshi

facebook-github-bot commented 3 years ago

This pull request was exported from Phabricator. Differential Revision: D25728909