frameworks: Use `log_artifact`

[daavoo]

Once #378 is done, it would be nice to automatically use it under the hood in the frameworks where we currently support saving the model, i.e.:

https://github.com/iterative/dvclive/blob/41d12d7936c7a781a30da27e7fd12faa00b34a1e/src/dvclive/keras.py#L49-L53

To:

        if self.model_file:
            if self.save_weights_only:
                self.model.save_weights(self.model_file)
            else:
                self.model.save(self.model_file)
            self.live.log_artifact(self.model_file)

[dberenbaum]

Even better would be if DVCLive can inspect if other callbacks are being used to save the model and use log_artifact to track those paths.

[dberenbaum]

See also #472: if we have a log_model method, obviously let's use that here instead of the generic log_artifact.

[dberenbaum]

@daavoo Can you take this as a follow-up to #464?

[daavoo]

@daavoo Can you take this as a follow-up to #464?

Yep! was already planning on it 🚀
I was also counting on including the type=model meta field as it doesn't really break anything to include it

[dberenbaum]

I was also counting on including the type=model meta field as it doesn't really break anything to include it

Thanks! First need to settle the discussion on how to handle type=model from iterative/mlem.ai#323 (comment) / https://iterativeai.slack.com/archives/CB41NAL8H/p1678197700155189.