Skip to content

Dropped testing of Buildings 10, added Buildings 12 #204

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.

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
merged 1 commit into from
Mar 28, 2025

Conversation

casella
Copy link
Collaborator

@casella casella commented Mar 28, 2025

No description provided.

@casella
Copy link
Collaborator Author

casella commented Mar 28, 2025

@adrpo, do we need to do something additional to get the Buildings 12 reference files in /mnt/ReferenceFiles/Buildings/csv/maint_12.x?

@casella casella merged commit 8364462 into master Mar 28, 2025
5 checks passed
@mwetter
Copy link

mwetter commented Mar 28, 2025

I added a link to
https://libraries.openmodelica.org/branches/master/Buildings_12/Buildings_12.html on https://simulationresearch.lbl.gov/modelica/environments.html, assuming this is where the results will be. The link is the same as for the 11 maintenance branch except for the version number.

@casella
Copy link
Collaborator Author

casella commented Mar 28, 2025

Will appear tomorrow

@perost
Copy link
Member

perost commented Mar 29, 2025

This seems to have broken the library testing because maint.12.x can't be loaded. I guess you also need to add it to the package manager index.

@casella
Copy link
Collaborator Author

casella commented Mar 29, 2025

Of course I do, thanks for the reminder!

@casella
Copy link
Collaborator Author

casella commented Mar 29, 2025

@adrpo's been there already #205

@adrpo
Copy link
Member

adrpo commented Mar 29, 2025

@adrpo, do we need to do something additional to get the Buildings 12 reference files in /mnt/ReferenceFiles/Buildings/csv/maint_12.x?

Yep, this one:
https://gitlab.liu.se/OpenModelica/apt-build/-/commit/f629dff0b08b8630c3f8afb9a97e89c14b4e7da0
and then run the Maintenance job on Jenkins.

@casella
Copy link
Collaborator Author

casella commented Apr 2, 2025

@adrpo, did everything go well with that? I checked https://libraries.openmodelica.org/branches/master/Buildings_12/Buildings_12.html but I still don't see the verification results. Thanks!

@adrpo
Copy link
Member

adrpo commented Apr 2, 2025

It should have worked. I will check what's going on.

@adrpo
Copy link
Member

adrpo commented Apr 2, 2025

Yeah, I forgot the most important place, in the script that downloads the result files from the web:
https://gitlab.liu.se/OpenModelica/apt-build/-/commit/4a04219045df2e23c05dfbee9c7515f86c8bcebe
hpefully now.

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.

4 participants