-
-
Notifications
You must be signed in to change notification settings - Fork 11
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 v0.5.0 #126
Merged
Merged
Release v0.5.0 #126
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
…. Also fix formatter
* Fix image prefix * Add preCICE pre-commits
* Follow time stepping proposed in documentation In the step-by-step guide on precice.org it is suggested to get the maximum allowed time step size from preCICE before calling advance. * Formatting --------- Co-authored-by: Ishaan Desai <[email protected]>
* Add first draft of simulation crash handling * Extend handling of crashing simulations to adaptivity and corner cases * Adapt format in crash handling to pep8 * Add tests for simulation crashes * Adapt formatting of tests * Update logger message for crashing simulation in the first run Co-authored-by: Ishaan Desai <[email protected]> * Restructure simulation crash handling and add global condition for exit * Add first draft of simulation crash handling * Extend handling of crashing simulations to adaptivity and corner cases * Adapt format in crash handling to pep8 * Add tests for simulation crashes * Adapt formatting of tests * Add interpolation to crash handling * Add Inverse Distance Weighting and improve crash handling * Format with pre-commit * Apply suggestions from code review Co-authored-by: Ishaan Desai <[email protected]> * Incoporate review into crash handling * Base crash interpolation on macro parameter * Format simulation crash handling * Adapt error for no available neighbor * Apply text improvement suggestions from code review Co-authored-by: Ishaan Desai <[email protected]> * Apply suggestions from crash handling review * Adapt interpolation test --------- Co-authored-by: Torben Schiz <[email protected]> Co-authored-by: Ishaan Desai <[email protected]>
* Add pyproject.toml * Further changes to the pyproject.toml * Add adaptivity module explicitly to the pyproject.toml * Adding package information to pyproject.toml * Reintroduce micro_manager.adaptivity to packaged in pyproject.toml * Add scikit-learn as a dependency in pyproject.toml * Change package name to micro_manager_precice * Go back to using micro_manager in the unit tests * Upgrade pip before installing the Micro Manager in the unit tests CI * Make executable name the same as the package name
* Use initial macro data to initialize micro simulations * Explicitly define the local variable is_initial_data_required * Correctly check number of input arguments of the initialiez() function of the micro simulation library * Add comment on the boolean which states the requirement of initial data in the micro simulation class
* Make initialize() a public method * Add pull request template * Modify tests to use public function initialize(), and add github PR template * Use version compatible format to state dependency version in pyroject.toml * Use original way of defining dependency version * Remove run_micro_manager.py from examples and tests * Upgrade pip3 in the adaptivity tests workflow * Remove LD_LIBRARY_PATH setting from adaptivity test workflow to check if it works * Change order of execution in local adaptivity test workflow * Use micro-manager-precice in running scripts of tests * Revert to old way of updating pip * Make the steps about upgrading pip and installing dependencies for the Micro Manager consistent in all workflows * Try to run dummy without updating pip * Explicitly install setuptools and wheel before installing the Micro Manager * Add local bin folder to path to be able to execute the micro-manager-precice executable * Don't set the PATH and install the Micro Manager without the --user flag * Restructure Micro Manager config files in the examples/ directly so that the executable can be used to run the solver dummy * Add CHANGELOG entries * Remove ./ from precice-config file path in one of the Micro Manager configs
* Add documentation for handling of crashing micro simulations * Rewording --------- Co-authored-by: Ishaan Desai <[email protected]>
…ut action of the publishing workflow
…that the adaptivity computation requires (#109) * Check if initial data returned from the micro simulation is the data that the adaptivity computation requires * Add CHANGELOG entry * Rephrase exception * Add documentation about initializing micro simulations
…ialize()` function (#110) * Handle exceptions from the initialize() function call from wrapped functions * Rework comments and add CHANGELOG entry
* Restructure execution with main * Add base class and abstract base class to unify common properties * Add expected return-type to main * Adapt class descriptions * Change class descriptions and imports * Update Changelog * Update CHANGELOG.md Co-authored-by: Ishaan Desai <[email protected]> --------- Co-authored-by: Ishaan Desai <[email protected]>
* Add option for the user to provide initial time window size of the micro simulation * Put config parameter micro_time_window_size in coupling_params group * Paasing dt correctly * Fix unit tests * Fix crash handling tests * Add CHANGELOG entry * Use micro_dt instead of micro_time_window_size * Use dt=0 when reading initial data
…e configuration (#114) * Move micro_dt from coupling_params to simulation_params in the JSON config * Update CHANGELOG
* Make sklearn an optional dependency * Fix optional crash handling * Fix trailing white space in config * Make sklearn an optional dependency * Fix optional crash handling * Fix trailing white space in config * Apply suggestions from code review Co-authored-by: Ishaan Desai <[email protected]> * Add comment regarding parameters potentially configurable by the user * Resolve conflict in CHANGELOG --------- Co-authored-by: Ishaan Desai <[email protected]>
* Initial commit for snapshot commputation * Snapshot: change import of module * Adapt SnapshotComputation further * Adapt Snapshot test * Snapshot: Formatting * Prepare Snapshot class further * First working version of snapshot computation * Add snapshot test, improve snapshot and restructure * Add snapshot test, improve snapshot and restructure part 2 * Improve logging, comments and minor structural improvements * Snapshot: extend setup and github actions * Move Snapshot into Micro Manager and update execution * Add post-processing script option and restructure snapshot example * Add post-processing to snapshot config file * Make snapshot executable and rename snapshot files * Restructure snapshot computation * Restructure config * Update snapshot computation still without chunking * Remove unintentional print statement * Update comments in tests * Read only the necessary input on each process * Clarify text and names * Add HDF5 dependency * Make h5py optional dependency * Fix h5py optional * Initial commit for snapshot commputation * Snapshot: change import of module * Adapt SnapshotComputation further * Snapshot: Formatting * Prepare Snapshot class further * First working version of snapshot computation * Add snapshot test, improve snapshot and restructure * Add snapshot test, improve snapshot and restructure part 2 * Improve logging, comments and minor structural improvements * Move Snapshot into Micro Manager and update execution * Add post-processing script option and restructure snapshot example * Make snapshot executable and rename snapshot files * Restructure snapshot computation * Restructure config * Update snapshot computation still without chunking * Remove unintentional print statement * Update comments in tests * Make h5py optional dependency * Fix h5py optional * Change snapshot case in main * Move dt to simulation parameters in Snapshot and snapshot specific dt * Fix init * Fix issue of using member variable in Config class * Fix tests * Apply suggestions from code review Co-authored-by: Ishaan Desai <[email protected]> * Apply review suggestions * Remove output dir from gitignore * Extend CHANGELOG.md * Update CHANGELOG.md --------- Co-authored-by: Ishaan Desai <[email protected]>
* Explicitly check if the time window has completed * Add CHANGELOG entry
* Add snapshot docs and update micro manager docs * Add snapshot configuration docs * Adapt documentation * Update postprocessing description * Remove next step * Apply suggestions from code review Co-authored-by: Ishaan Desai <[email protected]> * Move snapshot mention to -what can it do?- --------- Co-authored-by: Ishaan Desai <[email protected]>
) * Allow to compute snapshots on single object * Update snapshot documentation and CHANGELOG * Apply suggestions from code review Co-authored-by: Ishaan Desai <[email protected]> * Add note when to use a single object --------- Co-authored-by: Ishaan Desai <[email protected]>
…s in adaptivity (#125) * When taking relative norm differences in similarity calculation, use absolute values to get the normalizing factor * Fix comments with respect to earlier changes * Add CHANGELOG entry
tjwsch
approved these changes
Aug 16, 2024
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 of the Micro Manager consists of the following changes/additions:
is_time_window_complete()
Explicitly check if the time window has completed #118MicroManagerSnapshot
enabling snapshot computation and storage of microdata in HDF5 format Snapshot computation #101sklearn
an optional dependencymicro_dt
from the coupling parameters section to the simulation parameters section Move micro_dt from coupling parameters to simulation parameters in the configuration #114MicroManager
with minimal API and member function definitions, rename the existingMicroManager
class toMicroManagerCoupling
Add base class for Micro Manager and restructure main #111initialize()
function of micro simulations written in languages other than Python Handle the case where micro simulations written in C++ have theinitialize()
function #110micro-manager-precice
by default, and stop using the scriptrun_micro_manager.py
A collection of restructuring and minor updates #105initialize()
method of the MicroManager class public A collection of restructuring and minor updates #105pyproject.toml
instead ofsetup.py
to configure the build. Package name is nowmicro_manager_precice
Add pyproject.toml #84