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

Make Simulation's PIMixin/CSVMixin set_timeseries and write similar to optimization #1127

Open
SGeeversAtVortech opened this issue Oct 15, 2019 · 0 comments

Comments

@SGeeversAtVortech
Copy link
Contributor

In GitLab by @vreeken on Oct 15, 2019, 15:48

Currently (in simulation)

  • set_timeseries only works for PIMixin
  • extract_results only works for PIMixin
  • set_timeseries with output=True stores directly in timeseries_export, instead of relying on self.io (and getting it again later). The self.io call is superfluous (but shouldn't be).
  • write of PIMixin's write loops only over values of get_output_variables, not those added with set_timeseries and output=True.
  • Basically, it should be possible to use both CSVMixin and PIMixin together, that can read from each other's values (e.g. with extract_results)
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

No branches or pull requests

1 participant