Skip to content

Conversation

@chipbarnaby
Copy link
Contributor

Description

Generalized RSYS charge factor scheme.

  • Added rsFChgH = heating compressor charge factor. Implemented in both "old" and PM models.
  • Renamed rsFChg to rsFChgC = cooling compressor charge factor. (rsFChg is supported for backwards compatibility.)
  • Modified implementation of charge factors to increase COP (or input power) rather than decrease capacity.

These changes results in smaller autosized capacities, but that is expected and has only a minor impact on energy results.

ALSO merged branch "modernize" that allows mapping of old to new probe names.

Small test results changes (due to autosizing differences). Reference files are reviewed and updated.

Documentation updated.

@chipbarnaby chipbarnaby requested a review from nealkruis March 27, 2025 20:45
src/srd.h Outdated
// returns true iff word modernized (word updated)
// else false
{
bool bMatch = _strcmpi(word, mp_oldWord) == 0;
Copy link
Contributor

Choose a reason for hiding this comment

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

_strcmpi is not standard C++

@chipbarnaby
Copy link
Contributor Author

chipbarnaby commented Mar 28, 2025 via email

@nealkruis nealkruis merged commit 69b5b59 into main Mar 28, 2025
4 checks passed
@nealkruis nealkruis deleted the fChg branch March 28, 2025 16:10
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.

3 participants