Skip to content

Reorganize parallel module and add PBS support #316

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

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented May 16, 2025

This will be needed for Aurora support.

This merge reorganizes the polaris.parallel module into classes to avoid a growing chain of if/elif/else statements that were arising.

Checklist

  • Developer's Guide has been updated
  • API documentation in the Developer's Guide (api.md) has any new or modified class, method and/or functions listed
  • Documentation has been built locally and changes look as expected
  • Testing comment in the PR documents testing used to verify the changes

@xylar xylar self-assigned this May 16, 2025
@xylar xylar added enhancement New feature or request clean-up labels May 16, 2025
@xylar
Copy link
Collaborator Author

xylar commented May 16, 2025

I need to test both that this doesn't break things on Chrysalis and that it works on Aurora before we proceed...

@xylar xylar added the framework Changes relating to the polaris framework as opposed to individual tests or analysis label May 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clean-up enhancement New feature or request framework Changes relating to the polaris framework as opposed to individual tests or analysis
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant