Skip to content

Conversation

@Didou09
Copy link
Member

@Didou09 Didou09 commented Dec 5, 2025

Main changes:

There were 2 main issues:

  1. IMAS does not have imasdef anymore, and instead uses imas.ids_defs to get the backend
  2. IMAS has new built-in classes for arrays, scalar, str etc that tofu was not identifying

To address 1, the main useful changes were in tofu.imas2tofu._core.py

    from imas.ids_defs import HDF5_BACKEND
    try:
        from imas import imasdef
    except Exception:
        imasdef = None

To address 2: the main change happened in tofu.imas2tofu._com.py
tofu now reckognizes them and converts to the standard numpy, float, int, str

image

Issues:

Fixes, in devel, issue #1158

@Didou09 Didou09 self-assigned this Dec 5, 2025
@Didou09 Didou09 mentioned this pull request Dec 11, 2025
@Didou09 Didou09 merged commit 26cbebd into devel Dec 12, 2025
4 checks passed
@Didou09 Didou09 deleted the Issue1158_IMASbugs branch December 12, 2025 13:47
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.

3 participants