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

Specify conda env for Python Operators #468

Merged
merged 4 commits into from
Apr 10, 2024
Merged

Specify conda env for Python Operators #468

merged 4 commits into from
Apr 10, 2024

Conversation

haixuanTao
Copy link
Collaborator

This PR makes it possible to specify the conda env that we want to use in
a specific operator.

  - id: robot
    operator:
      python:
        source: ../operators/robot.py
        conda_env: robomaster

This will call:

conda run -n robomaster python -c "import dora; dora.start_runtime()"

@haixuanTao haixuanTao requested a review from phil-opp April 10, 2024 11:56
@haixuanTao haixuanTao force-pushed the specify-conda-env branch 2 times, most recently from 262a5e3 to 86686fc Compare April 10, 2024 12:12
This commit makes it possible to specify the conda env that we want to use in
a specific operator.

```yaml
  - id: robot
    operator:
      python:
        source: ../operators/robot.py
        conda_env: robomaster
```

This will call:
```bash
conda run -n robomaster python -c "import dora; dora.start_runtime()"
```
Copy link
Collaborator

@phil-opp phil-opp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@haixuanTao haixuanTao merged commit 7126454 into main Apr 10, 2024
17 checks passed
@haixuanTao haixuanTao deleted the specify-conda-env branch April 10, 2024 20:13
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.

2 participants