Now that JModelica isn't a thing, and we aren't using a branch of the MBL, we can have a cleaner simulation workflow. As a "side benefit" we also support Windows computers now!
Huge thanks to @chriswmackey for making this happen
What does this PR accomplish?
Remove om_docker.sh and om.py
These spun up a docker container, connected it to the user's local MBL, and provided a cli inside the container to run JModelica
We used the local MBL because we had to use a branch at the time. Now that we use released versions of the MBL, there is no need for that step
Installs MBL right into the docker container
Runs the container and sends commands straight to OM from modelica_runner.py
This is suuuuuuuch a nice improvement!
How should this be manually tested?
Build the container locally
Update this line to point to your newly-built container
Run pytest with all the (non-dymola) simulations, especially if you have a Windows computer!
Any background context you want to provide?
Now that JModelica isn't a thing, and we aren't using a branch of the MBL, we can have a cleaner simulation workflow. As a "side benefit" we also support Windows computers now!
Huge thanks to @chriswmackey for making this happen
What does this PR accomplish?
How should this be manually tested?
What are the relevant tickets?
Resolves #598
Screenshots (if appropriate)