Skip to content

Load the OpenSCALING branches for LargeTestSuite - #412

Merged
sjoelund merged 1 commit into
masterfrom
largetestsuite-openscaling-branches
Nov 10, 2026
Merged

sjoelund merged 1 commit into
masterfrom
largetestsuite-openscaling-branches

Conversation

@sjoelund

Copy link
Copy Markdown
Member

LargeTestSuite uses LSGreenH2Production, IDADistricts and Battery without a version. loadModel(LargeTestSuite) does not load them. omc loads them only when a model needs them, with the default version.

The package manager in omc now also installs the newest release of a library next to the installed branch. Thus LSGreenH2Production 1.0.0 and IDADistricts 1.0.0 are installed next to main. These are tags of the whole OpenSCALING repository from before the models that LargeTestSuite uses. omc prefers a release to a branch, so it loaded 1.0.0, and all LSGreenH2Production models and the IDADistricts models failed (OpenModelica/OpenModelica#17990).

Load the main branches explicitly. heavy_tests also tests these libraries at main.

Assisted-by: Claude Opus 5.5

LargeTestSuite uses LSGreenH2Production, IDADistricts and Battery
without a version. loadModel(LargeTestSuite) does not load them. omc
loads them only when a model needs them, with the default version.

The package manager in omc now also installs the newest release of a
library next to the installed branch. Thus LSGreenH2Production 1.0.0
and IDADistricts 1.0.0 are installed next to main. These are tags of
the whole OpenSCALING repository from before the models that
LargeTestSuite uses. omc prefers a release to a branch, so it loaded
1.0.0, and all LSGreenH2Production models and the IDADistricts models
failed (OpenModelica/OpenModelica#17990).

Load the main branches explicitly. heavy_tests also tests these
libraries at main.

Assisted-by: Claude Opus 5.5
@sjoelund
sjoelund merged commit 8314173 into master Nov 10, 2026
7 checks passed
@sjoelund
sjoelund deleted the largetestsuite-openscaling-branches branch November 10, 2026 19:54
@casella

casella commented Nov 10, 2026

Copy link
Copy Markdown
Collaborator

OK, thanks!

I also added the version to the uses annotation, I missed that it is compulsory.

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