Skip to content

(A015) [ModelicaSystem] check model executable#425

Open
syntron wants to merge 1 commit intoOpenModelica:masterfrom
syntron:ModelicaSystem_check_model_executable
Open

(A015) [ModelicaSystem] check model executable#425
syntron wants to merge 1 commit 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 syntron force-pushed the ModelicaSystem_check_model_executable branch from 08056f6 to 6c794cb Compare March 9, 2026 19:27
@adeas31
Copy link
Member

adeas31 commented Mar 12, 2026

Rebase.

@syntron
Copy link
Contributor Author

syntron commented Mar 12, 2026

Rebase.

can but why is it needed? It is PR = one commit; should apply without problems?!

@adeas31
Copy link
Member

adeas31 commented Mar 13, 2026

Rebase.

can but why is it needed? It is PR = one commit; should apply without problems?!

It says,

This branch has conflicts that must be resolved
Use the web editor or the command line to resolve conflicts before continuing.

OMPython/ModelicaSystem.py

@syntron syntron force-pushed the ModelicaSystem_check_model_executable branch from 6c794cb to e5c8a41 Compare March 13, 2026 17:11
@syntron
Copy link
Contributor Author

syntron commented Mar 13, 2026

Rebase.

can but why is it needed? It is PR = one commit; should apply without problems?!

It says,

This branch has conflicts that must be resolved Use the web editor or the command line to resolve conflicts before continuing.

OMPython/ModelicaSystem.py

Interesting - I did the rebase and there is no diff between the two version:

There isn’t anything to compare.
6c794cba710abd55550d8c1311b45b2b7b5327bb and e5c8a4170b685b02fa45dbb323fc9271d605d879 are identical.

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