Skip to content

new class ModelicaSystemDoE #312

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

Draft
wants to merge 34 commits into
base: master
Choose a base branch
from

Conversation

syntron
Copy link
Contributor

@syntron syntron commented Jun 25, 2025

New class to run DoEs based on the available functionality

Usage (see test_ModelicaSystemDoE):

  • define the model
  • define all parameters with the values to iterate
  • run DoE (simulation of all parameter combinations)

based on PR #310, #311, #313 and #314; see also #230

syntron added 19 commits June 25, 2025 20:26
(1) create ModelicasystemCmd instance - simulate_cmd()
(2) run it - simulate()
…ents

* fix some type hint issues in setInput()
* prepare for definition via dictionary replacing 'a=b' and '[a=b, c=d]' style
* rename from setMethodHelper()
* use _prepare_inputdata()
* cleanup code to align with new input as dict[str, str]

* setInput() is a special case
* replace eval() with ast.literal_eval() as a saver version
* use _prepare_input_data()
* simplify code
This method is used to set input values. It can be called with a sequence of input name and assigning
corresponding values as arguments as show in the example below. Compared to other set*() methods this is a
special case as value could be a list of tuples - these are converted to a string in _prepare_input_data()
and restored here via ast.literal_eval().
…resultfile in simulate()

reason:
* variable not needed / used as class variable
* using name based on resultfile allows to run the same model executable several times
@syntron syntron force-pushed the ModelicaSystemDoE branch 3 times, most recently from 1d86c64 to 2c58c48 Compare June 25, 2025 20:01
@syntron syntron force-pushed the ModelicaSystemDoE branch from fa8ed28 to 787a78a Compare June 25, 2025 20:20
@syntron
Copy link
Contributor Author

syntron commented Jun 26, 2025

converted to draft - It is running but there is still some work need to get it to a working state for all corner cases

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