{"action":"created","issue":{"url":"https://api.github.com/repos/NREL/EnergyPlus/issues/4546","labels_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/4546/labels{/name}","comments_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/4546/comments","events_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/4546/events","html_url":"https://github.com/NREL/EnergyPlus/issues/4546","id":46158475,"number":4546,"title":"CurveFitTool.xlm is not calculating the function of flow fraction curves correctly","user":{"login":"mjwitte","id":4991075,"avatar_url":"https://avatars.githubusercontent.com/u/4991075?v=3","gravatar_id":"","url":"https://api.github.com/users/mjwitte","html_url":"https://github.com/mjwitte","followers_url":"https://api.github.com/users/mjwitte/followers","following_url":"https://api.github.com/users/mjwitte/following{/other_user}","gists_url":"https://api.github.com/users/mjwitte/gists{/gist_id}","starred_url":"https://api.github.com/users/mjwitte/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mjwitte/subscriptions","organizations_url":"https://api.github.com/users/mjwitte/orgs","repos_url":"https://api.github.com/users/mjwitte/repos","events_url":"https://api.github.com/users/mjwitte/events{/privacy}","received_events_url":"https://api.github.com/users/mjwitte/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/NREL/EnergyPlus/labels/EnergyPlus","name":"EnergyPlus","color":"c7def8"},{"url":"https://api.github.com/repos/NREL/EnergyPlus/labels/Priority","name":"Priority","color":"e11d21"},{"url":"https://api.github.com/repos/NREL/EnergyPlus/labels/S1%20-%20High","name":"S1 - High","color":"e11d21"}],"state":"closed","locked":false,"assignee":{"login":"Nigusse","id":5805126,"avatar_url":"https://avatars.githubusercontent.com/u/5805126?v=3","gravatar_id":"","url":"https://api.github.com/users/Nigusse","html_url":"https://github.com/Nigusse","followers_url":"https://api.github.com/users/Nigusse/followers","following_url":"https://api.github.com/users/Nigusse/following{/other_user}","gists_url":"https://api.github.com/users/Nigusse/gists{/gist_id}","starred_url":"https://api.github.com/users/Nigusse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Nigusse/subscriptions","organizations_url":"https://api.github.com/users/Nigusse/orgs","repos_url":"https://api.github.com/users/Nigusse/repos","events_url":"https://api.github.com/users/Nigusse/events{/privacy}","received_events_url":"https://api.github.com/users/Nigusse/received_events","type":"User","site_admin":false},"milestone":null,"comments":16,"created_at":"2014-10-18T01:06:01Z","updated_at":"2015-03-19T16:13:42Z","closed_at":"2015-03-19T15:43:11Z","body":"Helpdesk Ticket 9928\r\n\r\nFound bug in CurveFitTool.xlm\r\n\r\n From the source code (DXcoils.cc) it can be seen that the curves as a function of flow must be established using the fraction of flow relative to rated flow, i.e. \"AirMassFlowRatioRated\" and not the not the flowrate in m3/s. I think this is a carry over from the chiller curve setup. After amending this, I get the rating capacities of calculated by energyplus to match my spreadsheet calcs.\r\n\r\n TotCapFlowModFac = CurveValue( DXCoil( DXCoilNum ).CCapFFlow( 1 ), AirMassFlowRatioRated );\r\n TotCapTempModFac = CurveValue( DXCoil( DXCoilNum ).CCapFTemp( 1 ), CoolingCoilInletAirWetBulbTempRated, OutdoorUnitInletAirDryBulbTempRated );"},"comment":{"url":"https://api.github.com/repos/NREL/EnergyPlus/issues/comments/83646541","html_url":"https://github.com/NREL/EnergyPlus/issues/4546#issuecomment-83646541","issue_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/4546","id":83646541,"user":{"login":"Myoldmopar","id":849824,"avatar_url":"https://avatars.githubusercontent.com/u/849824?v=3","gravatar_id":"","url":"https://api.github.com/users/Myoldmopar","html_url":"https://github.com/Myoldmopar","followers_url":"https://api.github.com/users/Myoldmopar/followers","following_url":"https://api.github.com/users/Myoldmopar/following{/other_user}","gists_url":"https://api.github.com/users/Myoldmopar/gists{/gist_id}","starred_url":"https://api.github.com/users/Myoldmopar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Myoldmopar/subscriptions","organizations_url":"https://api.github.com/users/Myoldmopar/orgs","repos_url":"https://api.github.com/users/Myoldmopar/repos","events_url":"https://api.github.com/users/Myoldmopar/events{/privacy}","received_events_url":"https://api.github.com/users/Myoldmopar/received_events","type":"User","site_admin":false},"created_at":"2015-03-19T16:13:42Z","updated_at":"2015-03-19T16:13:42Z","body":"Good point, @mjwitte, I'll open a dummy branch/PR that points to this issue to get it included."}}