Skip to content
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

Alternative simulatiom method: Independent translation from simulation #69

Merged
merged 5 commits into from
Oct 29, 2015

Conversation

mrosemeier
Copy link
Contributor

The translation time of relatively large models can be significant. If lots of cases of such large models are simulated it makes sense to translate the model only once and simulate it with varied parameters (if the model allows to vary parameters after translation).
For this purpose I have added two methods:

  • translate()
  • simulate_translated()

which can be executed consecutively. Each method executes Dymola once. An example (runSimulationTranslated.py) is also added showing how to benefit from this alternative method.

Basically the workflow looks as follows:

  • Create a Simulate object.
  • Set arbitrary model parameters, modifiers, and simulation settings.
  • Translate the model (while keeping the temporary translate folder in mind).
  • Deepcopy the Simulate object as many times as wished.
  • Set arbitrary model parameters and simulation settings for each copy.
  • For each simulation the temporary translate folder is copied.
  • Run simulations in parallel (note that for the simulations a Dymola license is not required).
  • Delete the temporary translate folder.

Some notes on fixes:

Feel free to comment on my request!

@mwetter
Copy link
Member

mwetter commented Oct 6, 2015

@mrosemeier : Thanks for the pull request. I merged it to branch issue70_simulation for further development and added the ticket #70 to track further changes and discussions that are needed before putting on the master.

I also changed back the code from BuildingsPy/src/buildingspy to BuildingsPy/buildingspy. It seems not necessary to put it in src (for me PyDev works as it is).

The regression and doctests have been fixed, now make doctest unittest works again.

@mwetter mwetter merged commit 58dd0d3 into lbl-srg:master Oct 29, 2015
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