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

Add inherit option to tasks to inherit from other tasks (e.g., root) #7

Open
GeigerJ2 opened this issue Jun 6, 2024 · 2 comments
Open
Labels
enhancement New feature or request
Milestone

Comments

@GeigerJ2
Copy link
Collaborator

GeigerJ2 commented Jun 6, 2024

Currently, the root task that is supposed to contain runtime information to be inherited by all other tasks (e.g. HPC host and account) is defined in the same way the other tasks. This might lead to some complications down the road, as it should actually be treated entirely differently from the other tasks. Therefore, I propose to instead add it as a separate section under runtime, for instance called runtime_defaults. This allows for it to be parsed separately from the other tasks (which are actual tasks to be executed, while root is not), and populate the other tasks accordingly.

To keep things simple while developing, I just commented it out for now.

@leclairm
Copy link
Contributor

leclairm commented Jun 6, 2024

There are pros and cons but it's not urgent anyhow. Having a special rule for root doesn't seem too cumbersome to me but it's fine to keep it commented out for now.

@leclairm leclairm added this to the v0.0.1 milestone Nov 27, 2024
@GeigerJ2
Copy link
Collaborator Author

Possibly, instead, add an inherit option in the YAML file for tasks, to inherit from arbitrary tasks, not necessarily root.

@GeigerJ2 GeigerJ2 changed the title Treat root task differently than other tasks Add inherit option to tasks to inherit from other tasks (e.g., root) Nov 27, 2024
@leclairm leclairm added the enhancement New feature or request label Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants