Skip to content

Release 2.0.0#11

Merged
olivhoenen merged 87 commits into
mainfrom
release/2.0
Jul 21, 2026
Merged

Release 2.0.0#11
olivhoenen merged 87 commits into
mainfrom
release/2.0

Conversation

@olivhoenen

@olivhoenen olivhoenen commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Notes for release 2.0.0

Compatibility note

This release can re-use actor YAML descriptions developed for 1.x version, but the generated Python actors are not compatible with workflows developed for iWrap 1.x. This is due to the move from the legacy closed AL-Python access layer to the more modern, open-source and Data-Dictionary agnostic IMAS-Python. Related to the move to IMAS-Python, this release drops completely support for ancient Access-Layer 4 support.

Licensing

  • Released under LGPLv3

Breaking changes

  • Replace AL-Python by IMAS-Python for Python actors
  • Integrate MUSCLE3 plugin as part of the iWrap's core

Improvements

  • Documentation on readthedocs covers both iWrap core and MUSCLE3 "extension"
  • Build and packaging as wheel, available on PyPI
  • Internal code cleaning (deprecated AL4 API, replace use of legacy pulse/run with systematic use of URIs)

@olivhoenen

Copy link
Copy Markdown
Contributor Author

@prasad-sawantdesai @SimonPinches any last issues with this release candidate?
Otherwise I will merge and proceed with the release.

@prasad-sawantdesai prasad-sawantdesai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SimonPinches SimonPinches left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In addition to the comments in the files, the three m3_{cpp,fortran,python}/m3_*_actor.py generator classes are nearly identical (they differ only in a few language-string properties; Python just drops build()), and macros/flags.jinja2 is duplicated exactly across all three. They already share m3_utils.py so a parametrized base in muscle3_common would remove the triplication and the divergence risk (some of the comments I've flagged exist in one language's copy but not the others).

Comment thread iwrap/generators/actor_generators/muscle3_cpp/resources/src/standalone.cpp.jinja2 Outdated
Comment thread pyproject.toml Outdated
Comment thread iwrap/generators/actor_generators/muscle3_fortran/resources/Makefile.jinja2 Outdated
Comment thread iwrap/generators/actor_generators/muscle3_fortran/resources/Makefile.jinja2 Outdated
Comment thread pyproject.toml
Comment thread pyproject.toml Outdated
@olivhoenen
olivhoenen marked this pull request as draft June 26, 2026 11:30
Comment thread iwrap/common/utils.py
prasad-sawantdesai and others added 4 commits June 29, 2026 14:19
…cle3_tools.py.jinja2

Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
…scle3_tools.cpp.jinja2

Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
…kefile.jinja2

Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
@prasad-sawantdesai
prasad-sawantdesai marked this pull request as ready for review June 29, 2026 20:10
@olivhoenen olivhoenen mentioned this pull request Jun 30, 2026
dependabot Bot and others added 16 commits June 30, 2026 09:45
Bumps the actions-deps group with 1 update: [actions/checkout](https://github.com/actions/checkout).


Updates `actions/checkout` from 6 to 7
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v6...v7)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: actions-deps
...

Signed-off-by: dependabot[bot] <support@github.com>
…kend cleanup guards for C++/Java/Fortran wrappers
…ove-AL4-layout

Use uri format and remove al4 layout
@prasad-sawantdesai

Copy link
Copy Markdown
Contributor

In addition to the comments in the files, the three m3_{cpp,fortran,python}/m3_*_actor.py generator classes are nearly identical (they differ only in a few language-string properties; Python just drops build()), and macros/flags.jinja2 is duplicated exactly across all three. They already share m3_utils.py so a parametrized base in muscle3_common would remove the triplication and the divergence risk (some of the comments I've flagged exist in one language's copy but not the others).

These language-specific generators and templates are not updated frequently so the divergence risk is currently low. So I would prefer to keep the existing structure and avoid adding abstraction for now..

@olivhoenen
olivhoenen merged commit 2d737aa into main Jul 21, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants