Closed daavoo closed 1 year ago
We should also add log_artifact to train.py, but it's not a blocker if we want to wait until we are ready to add type=model or other MR stuff since it should be a noop now. Do we have that tracked somewhere?
Um. So the model would not be a stage output?
No, the model can still be a stage output, that's why it would be a noop now and I think it's okay to not include it yet. Once log_artifact
also adds it to the artifacts
section, we will want it inside train.py
also. I think it also may confuse users if log_artifact
is used in the notebook but not train.py
.
Closes #171
It is currently deployed: