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

update pulling data from the cloud example #938

Merged
merged 1 commit into from
Jan 21, 2020

Conversation

Criztov
Copy link
Contributor

@Criztov Criztov commented Jan 21, 2020

update pulling data from the cloud example by setting new target repository

Disregard the recommendations below if you use Edit on GitHub button to improve the docs in place.

❗ Please read the guidelines in the Contributing to the Documentation list if you make any substantial changes to the documentation or JS engine.

🐛 Please make sure to mention Fix #issue (if applicable) in the description of the PR. This enables GitHub to link the PR to the corresponding bug and close it automatically when PR is merged.

Thank you for the contribution - we'll try to review and merge it as soon as possible. 🙏

update pulling data from the cloud example by setting new target repository
@pared
Copy link
Contributor

pared commented Jan 21, 2020

@jorgeorpinel jorgeorpinel merged commit a5387a5 into iterative:master Jan 21, 2020
@jorgeorpinel
Copy link
Contributor

Thanks!

@shcheklein
Copy link
Member

@Criztov @pared @jorgeorpinel this is a wrong merge. Ask @dmpetrov about the new_tag_classifier project. But we should be using that one instead of the example-get-started here.

@jorgeorpinel
Copy link
Contributor

Oops, sorry. Please approve/merge #941 @shcheklein

I'll check with Dmitry about the missing repo, and update in #931

jorgeorpinel added a commit that referenced this pull request Jan 24, 2020
review "Nothing to reproduce." output samples (after `dvc repro`)
rel: #938
@jorgeorpinel
Copy link
Contributor

See #946

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

Successfully merging this pull request may close these issues.

4 participants