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

Feature/radiant controls update #1534

Merged
merged 8 commits into from
Jul 21, 2023
Merged

Conversation

carlosduarteroa
Copy link
Collaborator

Initial refactoring of CBE radiant system controls incorporating the latest EnergyPlus objects for radiant system modeling.

Copy link
Collaborator

@mdahlhausen mdahlhausen left a comment

Choose a reason for hiding this comment

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

This looks great! Thanks for making the adjustments.

I'll add a short test script for the method to the testing framework before merging.

@mdahlhausen mdahlhausen self-requested a review July 13, 2023 22:40
@mdahlhausen mdahlhausen merged commit 19db3eb into master Jul 21, 2023
@mdahlhausen
Copy link
Collaborator

@carlosduarteroa added test placeholder and merged. Feel free to update your other branches from the main branch now.

@mdahlhausen mdahlhausen deleted the feature/radiant_controls_update branch August 9, 2023 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants