Skip to content

Release/2.5 - #39

Merged
olivhoenen merged 85 commits into
mainfrom
release/2.5
Sep 14, 2026
Merged

olivhoenen merged 85 commits into
mainfrom
release/2.5

Conversation

@olivhoenen

@olivhoenen olivhoenen commented Jul 30, 2026 •

Copy link
Copy Markdown
Contributor

Notes for release 2.5.0

New features

Improvements

Bug fixes

prasad-sawantdesai and others added 30 commits March 17, 2026 09:29
fix/using np.nan as np.NaN was removed in the NumPy 2.0 release
…s_with_mastu_data

Improve failsafe for plotequicomp when data is missing
Bumps the actions-deps group with 4 updates: [actions/checkout](https://github.com/actions/checkout), [actions/setup-python](https://github.com/actions/setup-python), [actions/upload-artifact](https://github.com/actions/upload-artifact) and [actions/download-artifact](https://github.com/actions/download-artifact).


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

Updates `actions/setup-python` from 4 to 6
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](actions/setup-python@v4...v6)

Updates `actions/upload-artifact` from 4 to 7
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v4...v7)

Updates `actions/download-artifact` from 4 to 8
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v4...v8)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: actions-deps
- dependency-name: actions/setup-python
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: actions-deps
- dependency-name: actions/upload-artifact
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: actions-deps
- dependency-name: actions/download-artifact
  dependency-version: '8'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: actions-deps
...

Signed-off-by: dependabot[bot] <support@github.com>
* use polycollection from matplotlib and use cells instead of nodes

* Use Outer midplane, Outer divertor target, Inner divertor target values and plot the same
…uda uri (#38)

* show warning/exception from IMAS-Python when time is not readble when using UDA uri

* add cache_mode=none when not used in UDA URI

* Moved URI handling into utils, simplified it using urllib.parse, added a reusable backend-specific query helper, automatically appended cache_mode=none to UDA URIs

* removed add_default_uda_cache_mode function, simplified _replace and updated documentation
@olivhoenen

Copy link
Copy Markdown
Contributor Author

@SimonPinches in the EB script associated with this release, we need in the module to either unset MPLBACKEND or setenv MPLBACKEND=TkAgg (to counterbalance the MPLBACKEND=Agg which is in matplotlib module and keep more user friendly default behaviour).

@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.

I found three user-facing crashes in code paths this release ships (empty scalar leaf in idsprint, GGD data without a "cells" subset in plotedgeprofiles, and dbperf on entries with no readable IDS), plus a few behavioural changes worth either fixing or calling out in the release notes (forced cache_mode=none on UDA and the silent removal of --show-labels). Details in the line comments.

Comment thread idstools/scripts/bin/idsprint
Comment thread idstools/view/edge_profiles.py
Comment thread idstools/scripts/bin/dbperf
Comment thread idstools/compute/equilibrium.py Outdated
Comment thread idstools/compute/equilibrium.py Outdated
Comment thread idstools/scripts/bin/plotequicomp Outdated
Comment thread idstools/scripts/bin/plotequilibrium
Comment thread idstools/view/equilibrium.py Outdated
Comment thread idstools/domain/kineticprofiles.py Outdated
Comment thread idstools/compute/edge_profiles.py Outdated
olivhoenen and others added 19 commits August 12, 2026 15:34
Assisted-by: Copilot:claude-5-sonnet
Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
@prasad-sawantdesai

Copy link
Copy Markdown
Contributor

@olivhoenen For the CI to pass at ITER, we also need this PR to be merged: #46

@olivhoenen
olivhoenen merged commit 66a0188 into main Sep 14, 2026
12 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.

6 participants