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

Add AeroTherm2023 models #209

Merged
merged 179 commits into from
Oct 28, 2024
Merged

Add AeroTherm2023 models #209

merged 179 commits into from
Oct 28, 2024

Conversation

spahrenk
Copy link
Contributor

Bradford White Aerotherm2023 50-, 65-, and 80-gal models are incorporated into HPWHsim. The 50-gal model was refined to match reasonably the RE2H50_UEF50 and RE2H50_UEF67 test schedules extracted from the provided measured datasets. These schedules were translated into the standard HPWHsim, multifile, schedule format, using scripts added to scripts/conversion.

Additional python scripts and an html file were added to facilitate comparison of simulated and measured profiles. The comparison is done by 1) running poetry run python server.py from within the scripts/python folder and 2) opening the file index.html from a browser (e.g., Chrome). The user can interactively specify the HPWH model and specification, then apply test parameters or view the 24-hr test output. Presently, a measured test data file (for comparison to simulated test data) must be specified to generate plots.

@spahrenk spahrenk requested a review from nealkruis August 30, 2024 18:15
@spahrenk spahrenk changed the base branch from main to develop August 30, 2024 19:05
@spahrenk spahrenk marked this pull request as ready for review September 5, 2024 20:38
@nealkruis
Copy link
Member

@spahrenk is this ready to review again?

@spahrenk
Copy link
Contributor Author

Should be able to change the build folder now.

Copy link
Member

@nealkruis nealkruis left a comment

Choose a reason for hiding this comment

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

Looking good! The python server is working for me now.

I have a few minor comments, and would like to see the python modules documented with doc strings explaining what the module does and how/when to use them.

src/HPWH.hh Outdated
@@ -282,7 +282,10 @@ class HPWH : public Courier::Sender

MODELS_AquaThermAire = 400, // heat exchanger model

MODELS_GenericUEF217 = 410
MODELS_GenericUEF217 = 410,
Copy link
Member

Choose a reason for hiding this comment

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

This should probably be added to the other generic models (e.g., number 179).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

src/HPWH.hh Outdated
Comment on lines 286 to 288
MODELS_AeroTherm2023_50 = 411,
MODELS_AeroTherm2023_65 = 412,
MODELS_AeroTherm2023_80 = 413
Copy link
Member

Choose a reason for hiding this comment

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

Start a new series (500?) for Bradford-White AeroTherm. Follow same naming convention as other enumerations:

MODELS_BradfordWhiteAeroThermRE2H50

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@spahrenk
Copy link
Contributor Author

Ready to review.

Copy link
Member

Choose a reason for hiding this comment

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

I'm still a bit confused by this file. It seems hard-coded to villara. Do we anticipate needing this file for future models? We talked with Ben about setting a standardized template for manufacturers to supply test data. Would that obsolete the need for this file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Two formats encountered to date are both implemented. No information on other formats may emerge.

Copy link
Member

Choose a reason for hiding this comment

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

Similar with this file. Is there an example you can point to where this has been used?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Used for all Aerotherm2023 models provided. Not intended as an automated tool for arbitrary format.

@nealkruis nealkruis changed the base branch from develop to main October 28, 2024 15:54
@nealkruis nealkruis merged commit 83a2bae into main Oct 28, 2024
8 checks passed
@nealkruis nealkruis deleted the add-aerotherm branch October 28, 2024 15:55
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.

3 participants