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

WaterToAirHeatPumpEquationFit curves do not support Table:Lookup #4946

Closed
mleachNREL opened this issue Aug 25, 2023 · 3 comments · Fixed by #4950
Closed

WaterToAirHeatPumpEquationFit curves do not support Table:Lookup #4946

mleachNREL opened this issue Aug 25, 2023 · 3 comments · Fixed by #4950

Comments

@mleachNREL
Copy link
Contributor

Issue overview

CoilCoolingWaterToAirHeatPumpEquationFit and CoilHeatingWaterToAirHeatPumpEquationFit do not support specifying curve data using Table:Lookup.

They are supposed to accept \reference QuadvariateFunctions but the documentation shows only that Curve:QuadLinear and Curve:QuintLinear are allowed for their respective curve fields

Recreate the issue - see attached script

Context

This is needed for ComStock GSHP measure development
coil_bug.rb.txt

@mleachNREL mleachNREL added the Triage Issue needs to be assessed and labeled, further information on reported might be needed label Aug 25, 2023
@joseph-robertson
Copy link
Collaborator

Perhaps bool setTotalCoolingCapacityCurve(const CurveQuadLinear& totalCoolingCapacityCurve); needs to be relaxed to something like bool setTotalCoolingCapacityCurve(const Curve& totalCoolingCapacityCurve);?

Probably need to inventory all objects where curve type is too exclusive.

@joseph-robertson
Copy link
Collaborator

This issue is probably a lot like #4611.

@joseph-robertson joseph-robertson added Enhancement Request and removed Triage Issue needs to be assessed and labeled, further information on reported might be needed labels Aug 29, 2023
@jmarrec
Copy link
Collaborator

jmarrec commented Aug 30, 2023

I agree with you @joseph-robertson

  • Change the type to the more generic Curve. Important: This is an unavoidable API break and should be documented properly
  • Make sure the IDD references are up to date
  • Look at E+ source code to make sure that there aren't more restrictions on what's accepted, if so implement them in the setter

Edit: I see you already started a PR, leaving it up to you then.

@jmarrec jmarrec removed their assignment Aug 30, 2023
jmarrec added a commit that referenced this issue Sep 26, 2023
Addresses #4946, allow TableLookup curves for Coil:*:WaterToAirHeatPump:EquationFit objects
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants