mlem icon indicating copy to clipboard operation
mlem copied to clipboard

`init`: stage .mlem/ (like DVC)

Open jorgeorpinel opened this issue 2 years ago • 2 comments

Similar to how dvc init automatically git adds .dvc/

jorgeorpinel avatar May 12 '22 16:05 jorgeorpinel

Maybe, if there is a git repo. But won't it be unintuitive? :

  1. If you have git repo, mlem init add .mlem to stage
  2. If you don't have it, mlem init doesn't do that

And what about model saving? When you do mlem.api.save, should we also add all new files and all changes to stage? 🤔

aguschin avatar May 13 '22 13:05 aguschin

IMO it makes sense to stage it if there's a Git repo (not unintuitive since you can't git add without a repo 😋). And most importantly it'd be consistent with dvc with which some MLEM users may be familiar and also use. Alternatively, remove that action from DVC? Cc @omesser @dberenbaum

When you do mlem.api.save, should we also add all new files

If we're going for consistency with DVC, no.

jorgeorpinel avatar May 18 '22 21:05 jorgeorpinel