microsoft / aiida-dynamic-workflows

An AiiDA plugin for dynamically defining workflows using Python functions
MIT License
17 stars 4 forks source link

Reorder parameters to `execution_environment` #17

Open jbweston opened 2 years ago

jbweston commented 2 years ago

Currently the first parameter is Optional[str] naming a conda environment, or None for "the conda environment with the same name as the currently active one". The second parameter is a str, naming the computer.

Instead, we should take the computer name first and have the conda environment name second, with None as the default.

This will be a breaking change, but is trivial to fix.