-
Notifications
You must be signed in to change notification settings - Fork 37
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
Release/0.25.1 #742
Merged
Merged
Release/0.25.1 #742
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
b-butler
requested review from
tcmoore3 and
Charlottez112
and removed request for
a team
April 21, 2023 20:07
bdice
approved these changes
Apr 21, 2023
Codecov Report
@@ Coverage Diff @@
## main #742 +/- ##
=======================================
Coverage 68.86% 68.86%
=======================================
Files 42 42
Lines 4194 4194
Branches 1032 1032
=======================================
Hits 2888 2888
Misses 1095 1095
Partials 211 211
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
3 tasks
b-butler
added a commit
that referenced
this pull request
Jul 27, 2023
* doc: Update changelog. * Bump up to version 0.25.1.
b-butler
added a commit
that referenced
this pull request
Oct 23, 2023
* refactor: Add _get_scheduler_values method to environments This moves much of the task and number of nodes logic to the environments where it is easier to manage the more complicated logic. * refactor: Port some templates over as examples of new style. * feat: Create way for environment classes to use partition config CPUS and GPUS per partition are in theory supported. * refactor: Update all environment which I have access to. * fix: Provide default node behavior for _get_scheduler_values Use sentinal of -1 to denote no node structure and always return 1 node requested for either CPU or GPU tasks. * fix: Fix typo in setting resources to Jinja2 context * fix: Delta template partition node submissions. * refactor: Provide infrastrucure to correctly warn on low-resources Add the ComputeEnvironment._shared_partitions attribute to check if less than single node submissions should be allowed in ComputeEnvironment._get_scheduler_values. * refactor (WIP): Add _shared_partitions to Delta env for testing. * fixup: removing operation from _get_scheduler_values * fix: Delta template The Delta template is now tested and works. * fix: Bridges2 template and environment. The environment is now tested. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * refactor: Reset Stampede2 template I do not have access to the cluster, so to prevent regressions, I am reseting this. * fix: Update shared partitions specifications in environments. * fix: Only specify node request on non-shared partitions (Expanse). * [pre-commit.ci] pre-commit autoupdate (#736) updates: - [github.com/psf/black: 23.1.0 → 23.3.0](psf/black@23.1.0...23.3.0) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> * Remove doc-filter after query switched to unified syntax (#738) * First pass at fix * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Add filter test * Update changelog --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> * fix: Delta hostname regex. (#740) Delta changed their compute node hostnames in their April 19th maintanance. This fixes the detection of the delta environment. * Release/0.25.1 (#742) * doc: Update changelog. * Bump up to version 0.25.1. * Add Frontier Environment (#743) * feat: Add the Frontier supercomputer to environments. * test: Add Frontier to template testing. * test: Update environment test template generation to signac 2.0 * doc: Update changelog * doc: Add Frontier documentation. * doc: Update incode comment clarity Co-authored-by: Bradley Dice <bdice@bradleydice.com> --------- Co-authored-by: Bradley Dice <bdice@bradleydice.com> * Feature/469 check status for specific operations (#725) Co-authored-by: Bradley Dice <bdice@bradleydice.com> * Build(deps-dev): Bump pre-commit from 3.2.1 to 3.2.2 (#746) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Build(deps): Bump pytest from 7.2.2 to 7.3.1 (#744) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Build(deps): Bump coverage from 7.2.2 to 7.2.5 (#745) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Bradley Dice <bdice@bradleydice.com> * Feat/template testing (#747) * feat (WIP): create flow CLI subcommand for testing templates * feat: Finish new CLI option. * test: flow test-workflow. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * doc: Add test-workflow to documentation. * doc: Add changes to changelog. --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> * Build(deps): Bump ruamel-yaml from 0.17.21 to 0.17.31 (#752) Bumps [ruamel-yaml](https://sourceforge.net/p/ruamel-yaml/code/ci/default/tree) from 0.17.21 to 0.17.31. --- updated-dependencies: - dependency-name: ruamel-yaml dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Build(deps): Bump pytest-cov from 4.0.0 to 4.1.0 (#751) Bumps [pytest-cov](https://github.com/pytest-dev/pytest-cov) from 4.0.0 to 4.1.0. - [Changelog](https://github.com/pytest-dev/pytest-cov/blob/master/CHANGELOG.rst) - [Commits](pytest-dev/pytest-cov@v4.0.0...v4.1.0) --- updated-dependencies: - dependency-name: pytest-cov dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Build(deps-dev): Bump pre-commit from 3.2.2 to 3.3.2 (#750) Bumps [pre-commit](https://github.com/pre-commit/pre-commit) from 3.2.2 to 3.3.2. - [Release notes](https://github.com/pre-commit/pre-commit/releases) - [Changelog](https://github.com/pre-commit/pre-commit/blob/main/CHANGELOG.md) - [Commits](pre-commit/pre-commit@v3.2.2...v3.3.2) --- updated-dependencies: - dependency-name: pre-commit dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Build(deps): Bump coverage from 7.2.5 to 7.2.7 (#749) Bumps [coverage](https://github.com/nedbat/coveragepy) from 7.2.5 to 7.2.7. - [Release notes](https://github.com/nedbat/coveragepy/releases) - [Changelog](https://github.com/nedbat/coveragepy/blob/master/CHANGES.rst) - [Commits](nedbat/coveragepy@7.2.5...7.2.7) --- updated-dependencies: - dependency-name: coverage dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix: UMich Greatlakes environment configuration * fix: Correctly GPU raise errors in Greatlakes template Error when GPU partition is requested without GPUs and when GPUs are requested without a GPU partition. * refactor: Small changes to new template environment code * refactor: Convert frontier template to new format * refactor: Frontier remove FrontierEnvironment.calc_num_nodes * feat: Finish conversion of Frontier environment. * refactor: Remove unnecessary empty shared_partition sets * fix: Frontier's allowable CPU use * test: Update submission scripts * fix: Stampede2Environment._get_scheduler_values * fix: remove Stampede2Environment._get_scheduler_values * test: Update template tests. * fix: typo in andes * test: Update tests to new code. * style: Remove incorrect comments. * Fix: Andes template typo * test: Update Andes template --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Corwin Kerr <cbkerr@umich.edu> Co-authored-by: Bradley Dice <bdice@bradleydice.com> Co-authored-by: rayasare <43545382+rayasare@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Patch version to fix two bugs.