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

Global sim manager sim air #8388

Merged
merged 18 commits into from
Nov 21, 2020
Merged

Global sim manager sim air #8388

merged 18 commits into from
Nov 21, 2020

Conversation

Myoldmopar
Copy link
Member

Continued refactoring state. Captured the following namespaces:

  • EIRPlantLoopHeatPump
  • PlantValves
  • SetpointManager
  • SimulationManager
  • SingleDuct
  • SizingManager
  • SolarReflectionManager

In addition, started making better use of c++17 by using make_unique, and also added the start of a diagnostic script that will verify we are properly handling state clearing and instantiation. The new script should run on Github Actions as part of the custom check gauntlet.

This is currently up to date with develop and could go in if clean.

@Myoldmopar Myoldmopar added the Refactoring Includes code changes that don't change the functionality of the program, just perform refactoring label Nov 21, 2020
@Myoldmopar
Copy link
Member Author

That is a whole lotta green. OK, merging this in, then pushing up an updated version of my unit test build speed up branch.

@Myoldmopar Myoldmopar merged commit 52873dc into develop Nov 21, 2020
@Myoldmopar Myoldmopar deleted the global_SimManager_SimAir branch November 21, 2020 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Refactoring Includes code changes that don't change the functionality of the program, just perform refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants