Skip to content

Rework ModelicaSystem.simulate #217

Closed

Description

@lafrech

Follow-up to #214. See comments.

Popen should be fed a list of arguments, not a single string. The fix in #214 splits the string but

  • arguments should be split one step further ("-r path" -> ["-r", "path"]
  • it makes no sense to concatenate strings then split them, the arguments should be built as a list in the first place

Activity

  1. syntron commented on Jun 14, 2025

    @syntron
    Contributor

    @arun3688 I think this is long solved in master; see PR #273 and #257

  2. added this to the 4.0.0 milestone on Jun 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions