Skip to content

Follow IS's move to EnumX - #169

Merged
jd-lara merged 2 commits into
mainfrom
lk/enumx
Sep 21, 2026
Merged

jd-lara merged 2 commits into
mainfrom
lk/enumx

Conversation

@luke-kiernan

Copy link
Copy Markdown
Collaborator

IS.@scoped_enum now wraps EnumX.@enumx (IS #643, merged into IS4), so a scoped enum's type is Name.Value and the bare name is a module. Value uses (TableFormat.LONG) are unchanged; what moves is type positions.

  • IS's ModelBuildStatus / SimulationBuildStatus / RunStatus / TableFormat annotations, and IOM's own SOSStatusVariable and OptimizationModelExportFormat.
  • ENUMS and the ENUM_MAPPINGS built from it key on the enum type, so they take .Value too — the load-time instances loop over them is what would otherwise stop the package precompiling.

The IS pin already points at IS4, which now carries EnumX; no pin changes needed.

🤖 Generated with Claude Code

`IS.@scoped_enum` now wraps `EnumX.@enumx`, so a scoped enum's type is
`Name.Value` and the bare name is a module. Value uses are unchanged; type
positions move, as do the `ENUMS` table and its lookups, which key on the type.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Performance Results
Main


This branch


@codecov

codecov Bot commented Sep 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 40.00000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/core/definitions.jl 0.00% 1 Missing ⚠️
src/core/settings.jl 50.00% 1 Missing ⚠️
src/operation/optimization_model_interface.jl 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@jd-lara
jd-lara merged commit 722757f into main Sep 21, 2026
5 of 7 checks passed
@jd-lara
jd-lara deleted the lk/enumx branch September 21, 2026 23:27
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.

2 participants