Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

provide guideline or mechanism to resolve conflicts in DVC files on merge #192

Closed
shcheklein opened this issue Aug 14, 2018 · 2 comments
Closed
Labels
A: docs Area: user documentation (gatsby-theme-iterative) type: enhancement Something is not clear, small updates, improvement suggestions

Comments

@shcheklein
Copy link
Member

When we change something (let's say a stage command) in one branch (experiment) and want to propagate this change to the derived branch it's easy to get a merge conflict in a DVC file.

Usually merge strategy theirs (git merge -X theirs) do the job.

We need to think if it's always the case. We should specify a guideline how to resolve these conflicts. If there are some ideas how to avoid them, let's discuss this. Right now it's frustrating and unclear how to manage these conflicts.

@efiop efiop transferred this issue from iterative/dvc Feb 25, 2019
@shcheklein shcheklein changed the title Provide guideline or mechanism to resolve conflicts in DVC files on merge provide guideline or mechanism to resolve conflicts in DVC files on merge Mar 25, 2019
@shcheklein shcheklein added type: enhancement Something is not clear, small updates, improvement suggestions question A: docs Area: user documentation (gatsby-theme-iterative) labels Mar 25, 2019
@dashohoxha
Copy link
Contributor

When we change something (let's say a stage command) in one branch (experiment) and want to propagate this change to the derived branch it's easy to get a merge conflict in a DVC file.

For small changes, I think that cherry-pick would be better and safer than a merge.

From what I have seen (and I have zero ML experience, I have just read some DVC tutorials), we merge a branch A onto a branch B when we want to completely overwrite B with A, for example when we merge the branch with the best metrics onto branch master. In this case it is safe to use the merge strategy theirs. Then, after making as well a dvc checkout, all the data and pipelines will be up to date (or at least most of them).

@dashohoxha dashohoxha mentioned this issue Oct 25, 2019
10 tasks
@jorgeorpinel
Copy link
Contributor

I think this is done! See https://dvc.org/doc/user-guide/merge-conflicts

Closing for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: docs Area: user documentation (gatsby-theme-iterative) type: enhancement Something is not clear, small updates, improvement suggestions
Projects
None yet
Development

No branches or pull requests

3 participants