Skip to content

dvc get from dvc-branch in not dvc-repo #2848

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

Closed
dmpetrov opened this issue Nov 26, 2019 · 1 comment · Fixed by #2852
Closed

dvc get from dvc-branch in not dvc-repo #2848

dmpetrov opened this issue Nov 26, 2019 · 1 comment · Fixed by #2852
Assignees
Labels
bug Did we break something?

Comments

@dmpetrov
Copy link
Member

$ dvc -V
0.70.0+38be14

master branch in https://github.com/dmpetrov/test_repo is NOT a DVC-repo. However, the extracted branch b-test IS DVC-repo and should work:

$ dvc get https://github.com/dmpetrov/test_repo --rev  b-test b-file
ERROR: failed to get 'b-file' from 'https://github.com/dmpetrov/test_repo' - URL 'https://github.com/dmpetrov/test_repo' is not a dvc repository.

This a regression since this command worked before: dvc get https://github.com/topusOctopus/mmdetection/ --rev fasterRCNN_resnet50_support model_weights/faster_rcnn_r50_c4_2x-6e4fdf4f.pth

@dmpetrov dmpetrov added the bug Did we break something? label Nov 26, 2019
@efiop efiop added p1-important Important, aka current backlog of things to do p0-critical and removed p1-important Important, aka current backlog of things to do labels Nov 26, 2019
@pared pared self-assigned this Nov 26, 2019
@pared
Copy link
Contributor

pared commented Nov 26, 2019

Reproduction script:

#!/bin/bash

rm -rf repo remote_repo storage git_repo
mkdir repo remote_repo storage git_repo

maindir=$(pwd)
pushd remote_repo
git init >> /dev/null 

echo README >> README.md
git add README.md
git commit -m "add readme"

git checkout -b dvc_branch
dvc init -q
echo data >> data
dvc add -q data
dvc remote add -d str $maindir/storage
git add .dvc/config data.dvc .gitignore
dvc push -q

popd
pushd repo

git init >> /dev/null && dvc init -q

set -x
set -e

dvc get $maindir/remote_repo --rev dvc_branch data

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Did we break something?
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants