Skip to content

Generalize update_resource#83

Open
iparask wants to merge 1 commit intomainfrom
feat/gen_resource
Open

Generalize update_resource#83
iparask wants to merge 1 commit intomainfrom
feat/gen_resource

Conversation

@iparask
Copy link
Member

@iparask iparask commented Mar 23, 2026

The purpose for this PR is to generalize the python file that sets up the SO configuration for the campaign manager backend.

It introduces a few more arguments:

  • env-path: The path or the name of the environment. This can be the same of CONDA-PREFIX for HPC created environments.
  • env-type: conda or venv. This allows the script to correctly setup the activation of the environment. The default value is conda
  • modules: If there are different modules to load, apart from the ones soconda expects, we can set this parameter. When set, the modules that are loaded before activating the environment change to what the user has defined.

@iparask iparask requested review from Bai-Chiang and tskisner March 23, 2026 21:16
Copy link
Contributor

@Bai-Chiang Bai-Chiang left a comment

Choose a reason for hiding this comment

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

lgtm

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