weasel icon indicating copy to clipboard operation
weasel copied to clipboard

BUG: Weasel DVC command error "run"

Open pcschreiber1 opened this issue 1 year ago • 1 comments

Hi, I'm quite new to weasel and ran into this basic problem and haven't been able to figure out a way to solve it.

Expected Behaviour

With running weasel dvc I can automatically create and maintain a dvc.yml from my project.yml.

Current Behaviour

When running weasel dvc I get an error message that "run" is an invalid command choice and no dvc.yml is created.

$ weasel dvc
âš  No workflow specified for DVC pipeline. Using the first workflow
defined in project.yml: 'all'
ERROR: argument command: invalid choice: 'run' (choose from 'add', 'artifacts', 'cache', 'check-ignore', 'checkout', 'commit', 'completion', 'config', 'daemon', 'dag', 'data',

Steps to reproduce

I ran into this issue with both the templates tagger_parser_ud and textcat_demo as well as the tutorial ner_fashion_brands.

Set-up new project

  1. mkdir new_folder && cd new_folder
  2. python -m venv .venv
  3. source .venv/bin/activate
  4. pip install weasel && pip install dvc
  5. git init && dvc init

Download and set-up template 6. weasel clone pipelines/tagger_parser_ud 7. cd tagger_parser_ud 8. weasel assets

Set-up weasel from project.yml 9. weasel dvc

Environment

  • WSL 2 Ubuntu
  • Python 3.10.12
  • weasel 0.3.4
  • dvc 3.48.1
  • git version 2.34.1

Detailed Description

It seems to me like under the hood a dvc run command is triggered, which causes an issue. It already tried to check the code weasel/cli/dvc.py, but couldn't find a run command there or in utils.

pcschreiber1 avatar Mar 03 '24 12:03 pcschreiber1

Hi! Thanks for the extensive report and way of reproducing. I can reproduce this on my own system as well.

The DVC integration is in a dire need of updating, as this was written before dvc run was deprecated for its 3.0.0 release. Weasel still uses this here.

If I downgrade dvc to version 2.58.2 this error goes away, though I'm running into other issues. We'll have to spend some time looking into this properly.

svlandeg avatar May 14 '24 20:05 svlandeg