Skip to content
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

Contiuous Integration using OpenModelica/openmodelica-library-testing-action #83

Merged
merged 12 commits into from
Feb 2, 2024

Conversation

AnHeuermann
Copy link
Contributor

@AnHeuermann AnHeuermann commented Feb 1, 2024

Related Issues

Changes

  • Addes 3-Clause BSD license.
  • Refactored README.md.
  • Added Continuous Integration GitHub workflow.
    • Using OpenModelica/setup-openmodelica@v1 and OpenModelica/[email protected].
    • Removing mos scripts for outdated tests.
  • Adding CODEOWNERS, so that @bernhardbachmann needs to review all license changes.
  • Updating reference file for PNlib.Examples.ExtTest.ArcswithPCtoTC3.

@AnHeuermann
Copy link
Contributor Author

AnHeuermann commented Feb 1, 2024

PR's from forks don't have the rights to push to gh-pages. I need to find a better solution.

@AnHeuermann AnHeuermann marked this pull request as draft February 1, 2024 17:59
@AnHeuermann
Copy link
Contributor Author

I'm updating the reference file for PNlib.Examples.ExtTest.ArcswithPCtoTC3. OpenModelica and Dymola agree on a solution so without any clue what the model should be doing it blindly trust the algorithms.

@AnHeuermann AnHeuermann marked this pull request as ready for review February 2, 2024 08:05
@AnHeuermann
Copy link
Contributor Author

I removed ResultFiles/OpenModelica/rtest. The Dymola test script needs it, but that one is failing anyway, see #84.

@AMIT-HSBI AMIT-HSBI merged commit 748df49 into AMIT-HSBI:master Feb 2, 2024
1 check passed
@phannebohm phannebohm mentioned this pull request Feb 3, 2024
5 tasks
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.

Add Modelica License 2
2 participants