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

Metamodeling framework, Spawn data generator, reset function #24

Merged
merged 15 commits into from
Jan 4, 2022

Conversation

ThibaultMarzullo
Copy link
Contributor

This PR includes:

  • the new Spawn data extraction utility found in spawn-tools. It generates files for the Spawn FMU, later used by BOPTEST for forecasts and KPIs. The data is extracted from the epw and idf files used in the Spawn model.
  • the new metamodeling framework. This new version will run Spawn and collect model data, then identify a state-space model using N4SID. Implements best subset selection and forward step-wise selection. All the matrices are then saved where the MPC can find them.
  • a new reset function for the ACTB client, and helper functions
  • MPC (non-DR) example update: changed the TVP sources and minor cleanup
  • RBC (non-DR) update with the new stop() function
  • repo cleanup, all Spawn resources now go to testcases\SpawnResources\<testcase name>. This includes Spawn data to be included in FMUs, metamodels, etc.

@ThibaultMarzullo
Copy link
Contributor Author

It looks like there is a conflict due to the tick labels in the MPC example. I did not pull that commit before doing the changes. Can we resolve this without undoing all the changes I made in the new commit?

Copy link
Member

@nllong nllong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes look great. I didn't go through them 1-by-1, but they worked as expected on my computer.

@nllong
Copy link
Member

nllong commented Jan 3, 2022

@ThibaultMarzullo -- you'll need to resolve the conflicts before merging...

@ThibaultMarzullo
Copy link
Contributor Author

Thanks Nick, it has to do with the MPC plot ticks, I'll fix that.

@ThibaultMarzullo ThibaultMarzullo merged commit e635110 into actb Jan 4, 2022
@ThibaultMarzullo ThibaultMarzullo deleted the actb-metamodels branch April 23, 2022 13:22
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