Release v0.8.0 #216
Merged
Release v0.8.0 #216
Conversation
This file contains hidden or 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
Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 4 to 5. - [Release notes](https://github.com/actions/download-artifact/releases) - [Commits](actions/download-artifact@v4...v5) --- updated-dependencies: - dependency-name: actions/download-artifact dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
…s/download-artifact-5 Bump actions/download-artifact from 4 to 5
* Rework parts of snapshot computation * Add config options to set snapshot output directory and snapshot output file name * Remove unnecessary code in snapshot.py * Further improvement to streamline code, and add a CHANGELOG entry * Fix tests * Use bool in snapshot-config.json in tests
Bumps [actions/checkout](https://github.com/actions/checkout) from 4 to 5. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v4...v5) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/setup-python](https://github.com/actions/setup-python) from 5 to 6. - [Release notes](https://github.com/actions/setup-python/releases) - [Commits](actions/setup-python@v5...v6) --- updated-dependencies: - dependency-name: actions/setup-python dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Use openmpi in the mpi4py setup action * Add changelog entry
* Use booleans in JSON config files of integration tests * Add changelog entry
* Refactor adaptivity to have accurate function names * Port old active simulation ids call from adaptivity * Shorten iterator variable names * Shorten iterator variable names in global adaptivity computations * Add changelog entry
…#187) * Revert using openmpi as mpich is fixed in setup-mpi action * Add changelog entry
* Add load imbalance data gathering * Remove rank from filename * Add new class for global adaptivity with dynamic load balancing with a config option * Add tools/misc.py and some initial class structure * Making the Micro Manager interface compliant to dynamic load balancing * First part of the implementation of redistribution of active and inactive simulations in load balancing * Add first tests for dynamic load balancing * Remove incorrect underscore from Config class function call * Add unit tests for dynamic load balancing * Add code removed by mistake from GlobalAdaptivityCalculator * New algorithm for load balancing based targetted moving of simulations. NOT FULLY WORKING * Add code to implement two-step load balancing * Add two-step load balancing and clean up code * Solve several bugs to get the two step load balancing functionality to work * Fix get_ranks_of_sims test and add load balancing threshold as config parameter * Add load balancing redistribute function call to the solve loop in micro_manager.py * Fix test * Set threshold to a default of 0 * Several fixes to make the unit cube integration test work * Modify functionality in GlobalAdaptivityCalculator to work with a varying number of simulations per rank * Stop load balancing if either the send or receive requests are zero * Greatly simplify reading and writing data to preCICE by using global ids as coupling mesh vertices * Fix bux in two-step load balancing * Fix unit tests * Stop using coupling mesh vertex ids as global ids of micro simulations * Fix read write unit test * Fix update_inactive_sims function * Add timeout to parallel tests workflow and fix load balancing unit tests * Fix parallel load balancing tests * Add logging to the load balancing functions * Add load balancing cpu time to the metrics logging * Add load balancing CPU time to metrics logging * Add boolean to global adaptivity class constructor to indicate if load balancing is on * Fix unit test and remove print statement * Fix crash handling unit test and add debugging print statements * Use warning logging of the logging wrapper * Fix load balancing for implicit coupling * Change order of operations in the coupling loop to ensure consistent logging * Put adaptivity checkpointing in the adaptivity conditional * Add experimental profiling API of preCICE to profile global adaptivity and dynamic load balancing * Reworking metrics logging to have local and global logging separately * Add more granularity to profiling sections of the load balancing * Fix tests by using the correct constructors * Add profiling API to the mocked preCICE * Use correct constructors in tests * Fix logging in snapshot functionality * Fix irregular defintion from merge conflict resolution * Use appropriate adaptivity function to get global ids of active sims * Remove requirement to point to a specific branch of the Python bindings * Remove duplicate logging of headers * Remove excess stop_last_profiling_section() call * Remove excess API call and fix indexing in updating inactive simulations * Fix case with adaptivity in every implicit iteration * Fix parallel test * Use COMM_WORLD in load balancing * Fix parallel test and add profiling section for redistribute_sims * Add changelog entry in the correct place * Add first documentation about load balancing * Add documentation about load balancing * Add further documentation related to load balancing * Fix formulas for balancing range in load balancing documentation * Use openmpi in setup mpi4py action in load balancing unit tests action * Move a check from load balancing class to config * Remove input arg from docstring because it is no longer implemented * Use newly created shorter interator names * Fix bugs in solve time loop. Not a working state yet. * Working state - two-scale heat conduction * Remove logger from global adaptivity, and add a global IDs function for local adaptivity * Add documentation and remove incorrect passing in of the logger object * Add logic to balance inactive simulations * Fix load balancing tests
* Remove diagnostics output of simulation solve time * Remove last traces of solve_cpu_time * Remove last remnant of sim solve time and add changelog entry * Fix typo in table
Bumps [actions/checkout](https://github.com/actions/checkout) from 4 to 5. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v4...v5) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Remove two-step-balancing option from load balancing * Remove test one-step load balancing with 4 ranks * Add changelog entry
…ions as the balancing bound (#192) * Fix bug in load balancing when ranks have exactly as many simulations as bounds * Fix the error correctly * Completely remove two-step balancing procedure. Balance in one sweep * Add one simulation to send and recv counts in balancing when the rank has exactly as many as the balancing bounds * Fix balancing bound issue and also the test * Add changelog entry
* Log adaptivity metrics at t = 0 * Add changelog entry
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 5. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@v4...v5) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 5 to 6. - [Release notes](https://github.com/actions/download-artifact/releases) - [Commits](actions/download-artifact@v5...v6) --- updated-dependencies: - dependency-name: actions/download-artifact dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Use global maximum similarity distance in local adaptivity * Add changelog entry
…lation as the global average (#200) * Handle case where ranks have exactly as many active simulation as the global average * Add changelog entry
* Add more metrics to adaptivity logging * General refactoring * Fix tests * Add changelog entry * Remove logging for debugging * Remove debugging statement
* Add markdownlint to pre-commit * Exempt rules from markdown linting * Exempt some markdown linting rules in the Ruby format * Put markdown lint exempted rules in quotes to not confuse them with a file path * Markdown linting fixes * Exempt rule MD029 for ordered lists * More markdown fixes * Remove markdown linting check Action because it is now tested via pre-commit
…l adaptivity (#202) * Implement a rank-based round-robin scheme for deactivating simulations * Further simplify activation of deactivated simulations * Add changelog entry and try to fix tests * Fix tests * Revert to comparing inactive sims with freshly activated sims
Bumps [actions/checkout](https://github.com/actions/checkout) from 5 to 6. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v5...v6) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…erations being profiled correctly (#205) * Place profiling sections at more appropriate places in the code base * Fix tests * Add changelog entry
* first basic model switching impl * added file based switching and full docstrings * fix style issues * add missing dep in setup description * add input to switching fun * integrate adaptivity with new micro sim loading * use pre-commit fix * fix tests adapted to new AdaptivityCalculator signature * use pre-commit fix * clean up and documentation * fix linting * Update docs/configuration.md Co-authored-by: Ishaan Desai <ishaandesai@gmail.com> * Update docs/configuration.md Co-authored-by: Ishaan Desai <ishaandesai@gmail.com> * Update docs/model-adaptivity.md Co-authored-by: Ishaan Desai <ishaandesai@gmail.com> * Update docs/model-adaptivity.md Co-authored-by: Ishaan Desai <ishaandesai@gmail.com> * Update docs/model-adaptivity.md Co-authored-by: Ishaan Desai <ishaandesai@gmail.com> * Update model_adaptivity.py Co-authored-by: Ishaan Desai <ishaandesai@gmail.com> * switch to per micro-sim switching func evaluation * Apply documentation changes * Apply suggested changes Co-authored-by: Ishaan Desai <ishaandesai@gmail.com> * apply linting fixes * Remove package name from proj toml * Make changelog entry more descriptive --------- Co-authored-by: Ishaan Desai <ishaandesai@gmail.com>
* Change load balancing setting at the start and some cleanup * Add changelog entry
Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 6 to 7. - [Release notes](https://github.com/actions/download-artifact/releases) - [Commits](actions/download-artifact@v6...v7) --- updated-dependencies: - dependency-name: actions/download-artifact dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 5 to 6. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@v5...v6) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ensure consistency (#210) * Fix coordinate ordering when accessing global mesh on every rank for load balancing * Rename DomainDecomposer methods to reflect their functionalities * Make the local mesh bounds function as before * Add changelog entry
… documentation (#213) * Set adaptivity metrics output default to having no output * Add defaults to configuration documentation tables * Simplify string check for adaptivity metrics and memory output * Add changelog entry * Simplify configuration documentation * Fix link in snapshot documentation * Add defaults to snapshot parameters documentation
…214) The rest of the page does not refer to that option anyway. Follow-up of precice/precice.github.io#677 Related to precice/precice.github.io#398
* Conform to naming standard in precice/tutorials for quantities relevant to adaptivity * Conform to naming standard in the dummy * Conform to data naming nomenclature in all configs and code * Try to fix HDF5 test * Fix tests * Fix snapshot tests
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.
This release contains:
|delimiter in CSV files of adaptivity metrics data #193