Skip to content

Convert core/edge to plasma IDS #87

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.

Sign up for GitHub

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

Merged
olivhoenen merged 6 commits into from
Jan 16, 2026
Merged

Convert core/edge to plasma IDS #87

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
0aeeb22
Add conversion logic for core/edge -> plasma IDS
maarten-ic Dec 11, 2025
1df6e82
Add docstrings for convert to plasma functions
maarten-ic Dec 12, 2025
667dd62
Update __init__.py and docs
maarten-ic Dec 12, 2025
2329227
Add option to `imas convert` CLI to convert core/edge to plasma IDSs
maarten-ic Jan 13, 2026
2a3c973
Fix incorrect handling of core_instant_changes IDS in `imas convert -…
maarten-ic Jan 14, 2026
6bbf866
Merge branch 'develop' into feature/convert-core-edge-to-plasma-ids
maarten-ic Jan 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Merge branch 'develop' into feature/convert-core-edge-to-plasma-ids
  • Loading branch information
@maarten-ic
maarten-ic authored Jan 15, 2026
commit 6bbf8663be49cded3db6616455c14761b6d1334b
2 changes: 1 addition & 1 deletion imas/db_entry.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -561,7 +561,7 @@ def _get(
raise RuntimeError("Database entry is not open.")
if lazy and destination:
raise ValueError("Cannot supply a destination IDS when lazy loading.")
if not self._ids_factory.exists(ids_name) and autoconvert:
if autoconvert and not self._ids_factory.exists(ids_name):
raise IDSNameError(ids_name, self._ids_factory)

# Note: this will raise an exception when the ids/occurrence is not filled:
Expand Down
Loading
You are viewing a condensed version of this merge commit. You can view the full changes here.