Skip to content

(A015) [ModelicaSystem] check model executable#425

Open
syntron wants to merge 2 commits intoOpenModelica:masterfrom
syntron:ModelicaSystem_check_model_executable
Open

(A015) [ModelicaSystem] check model executable#425
syntron wants to merge 2 commits intoOpenModelica:masterfrom
syntron:ModelicaSystem_check_model_executable

Conversation

@syntron
Copy link
Contributor

@syntron syntron commented Feb 18, 2026

define one fucntion to check if the model executable exists and is working

see PR #404

@syntron syntron force-pushed the ModelicaSystem_check_model_executable branch 4 times, most recently from 8b40c54 to 08056f6 Compare February 27, 2026 18:09
syntron added 2 commits March 9, 2026 20:15
[ModelExecution*] create classes to handle model execution

* rename ModelicaSystemCmd => ModelExecutionCmd
* rename OMCSessionRunData => ModelExecutionData
* create class ModelExecutionException
* move some code:
  * OMCSession.omc_run_data_update() => merge into ModelExecutionCmd.define()
  * OMCSession.run_model_executable() => ModelExecutionData.run()

[test_ModelicaSystemCmd] update unittest

[ModelExecutionData] include the original exception if reraised as ModelExecutionException

[ModelicaSystem] fix usage of ModelicaSystemCmd
@syntron syntron force-pushed the ModelicaSystem_check_model_executable branch from 08056f6 to 6c794cb Compare March 9, 2026 19: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.

1 participant