Skip to content

Apply custom options alongside defaults options #486

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.

Sign up for GitHub

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

Open
adeas31 wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Apply custom options alongside defaults options #486

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
47b6912
Apply custom options alongside defaults options
adeas31 Oct 2, 2026
048dd15
Remove test
adeas31 Oct 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 11 additions & 7 deletions OMPython/modelica_system_omc.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,13 +39,15 @@ def __init__(
work_directory: Optional[str | os.PathLike] = None,
omhome: Optional[str] = None,
session: Optional[OMSessionABC] = None,
use_default_command_line_options: bool = True,
) -> None:
"""Create a ModelicaSystem instance. To define the model use model() or convertFmu2Mo().

Args:
command_line_options: List with extra command line options as elements. The list elements are
provided to omc via setCommandLineOptions(). If set, the default values will be overridden.
To disable any command line options, use an empty list.
provided to omc via setCommandLineOptions() in addition to OMPython's default options.
use_default_command_line_options: Whether to set OMPython's default options for linearization.
Set to False to use only command_line_options, or no options if it is unspecified or empty.
work_directory: Path to a directory to be used for temporary
files like the model executable. If left unspecified, a tmp
directory will be created.
Expand All @@ -62,15 +64,17 @@ def __init__(
work_directory=work_directory,
)

# set commandLineOptions using default values or the user defined list
if command_line_options is None:
options: list[str] = []
if use_default_command_line_options:
# set default command line options to improve the performance of linearization and to avoid recompilation if
# the simulation executable is reused in linearize() via the runtime flag '-l'
command_line_options = [
options.extend([
"--linearizationDumpLanguage=python",
"--generateSymbolicLinearization",
]
for opt in command_line_options:
])
if command_line_options is not None:
options.extend(command_line_options)
for opt in options:
self.set_command_line_options(command_line_option=opt)

def model(
Expand Down
Loading