Closed nickjalbert closed 2 years ago
@andyk this is ready for you to take a look at when you have time!
This looks great! Thanks for getting this working again. I'm in favor of having this in place as a demo solution and potentially re-thinking in the future.
I tried this out per your instructions on my Macbook Pro M1 and it worked great! Merging.
This PR gets artifact upload working again.
I think this is the shortest path to getting us back to sharing models; this is basically just porting forward the upload system to the new spec revamp.
Longer term, we should think more about artifacts being explicitly represented in the DAG (maybe something like a file-in and file-out component?) rather than doing this via a "side-channel" as we do here.
Demo:
If you follow the demo, all the AgentRuns should have tarballs associated with them that contain the model that was trained/evaluated. For example: