-
Notifications
You must be signed in to change notification settings - Fork 58
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
Add pipeline run tests for DVC #867
Conversation
dvc-requirements.txt
Outdated
@@ -0,0 +1,4 @@ | |||
dvc==2.38.1 | |||
pandas |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we install the additional packages (panda, scikit-learn and SQLAlchemy) using the pipeline generated requirements file? As additional virtualenv.run() command after 'git init' and 'dvc init'?
@andycui97 I'm fine with the changes, I think the comment above is obsolete! |
Sg! Just waiting on @aayan636 to review internally as well. |
Description
Add tests for DVC pipelines to test that they actually run in tests/test_pipeline_run_dvc.py.
Aldo changes default flavor for DVC to be StagePerArtifact.
Fixes # (issue)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?