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

import: same command creates .dvc file in different locations #4169

Closed
1 of 2 tasks
shcheklein opened this issue Jul 5, 2020 · 2 comments · Fixed by #4340
Closed
1 of 2 tasks

import: same command creates .dvc file in different locations #4169

shcheklein opened this issue Jul 5, 2020 · 2 comments · Fixed by #4340
Assignees
Labels
bug Did we break something? p1-important Important, aka current backlog of things to do

Comments

@shcheklein
Copy link
Member

shcheklein commented Jul 5, 2020

DVC Version

~/example-get-started$ dvc version
DVC version: 1.0.1
Python version: 3.7.1
Platform: Linux-4.15.0-88-generic-x86_64-with-debian-buster-sid
Binary: True
Package: deb
Supported remotes: azure, gdrive, gs, hdfs, http, https, s3, ssh, oss
Cache: reflink - not supported, hardlink - supported, symlink - supported
Filesystem type (cache directory): ('ext4', '/dev/mapper/host01--vg-root')
Repo: dvc, git
Filesystem type (workspace): ('ext4', '/dev/mapper/host01--vg-root')

Reproduce

Run the same command:

dvc import https://github.com/iterative/dataset-registry get-started/data.xml -o data/data.xml

when data directory exists and when it does not.

In the first case we get:

Importing 'get-started/data.xml (https://github.com/iterative/dataset-registry)' -> 'data/data.xml'

To track the changes with git, run:

        git add data/data.xml.dvc

In the second case:

Importing 'get-started/data.xml (https://github.com/iterative/dataset-registry)' -> 'data/data.xml'

To track the changes with git, run:

        git add data.xml.dvc data/.gitignore

There are two bugs:

  • It creates data.xml.dvc in different locations (data/data.xml.dvc vs data.xml.dvc)
  • It doesn't mention .gitignore in the first case in the git add hint.
@triage-new-issues triage-new-issues bot added the triage Needs to be triaged label Jul 5, 2020
@efiop
Copy link
Contributor

efiop commented Jul 5, 2020

It creates data.xml.dvc in different locations (data/data.xml.dvc vs data.xml.dvc)

Good catch!

It doesn't mention .gitignore in the first case in the git add hint.

Works for me, I suppose you just had gitignore entry before that (probably from the previous run).

@efiop efiop added bug Did we break something? p2-medium Medium priority, should be done, but less important p1-important Important, aka current backlog of things to do and removed triage Needs to be triaged p2-medium Medium priority, should be done, but less important labels Jul 5, 2020
@shcheklein
Copy link
Member Author

Works for me, I suppose you just had gitignore entry before that (probably from the previous run).

yep, confirmed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Did we break something? p1-important Important, aka current backlog of things to do
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants