{"action":"created","issue":{"url":"https://api.github.com/repos/NREL/EnergyPlus/issues/5704","repository_url":"https://api.github.com/repos/NREL/EnergyPlus","labels_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/5704/labels{/name}","comments_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/5704/comments","events_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/5704/events","html_url":"https://github.com/NREL/EnergyPlus/issues/5704","id":158240294,"number":5704,"title":"Table:OneIndependentVariable documentation doesn't agree with behavior","user":{"login":"jasondegraw","id":4623433,"avatar_url":"https://avatars.githubusercontent.com/u/4623433?v=3","gravatar_id":"","url":"https://api.github.com/users/jasondegraw","html_url":"https://github.com/jasondegraw","followers_url":"https://api.github.com/users/jasondegraw/followers","following_url":"https://api.github.com/users/jasondegraw/following{/other_user}","gists_url":"https://api.github.com/users/jasondegraw/gists{/gist_id}","starred_url":"https://api.github.com/users/jasondegraw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jasondegraw/subscriptions","organizations_url":"https://api.github.com/users/jasondegraw/orgs","repos_url":"https://api.github.com/users/jasondegraw/repos","events_url":"https://api.github.com/users/jasondegraw/events{/privacy}","received_events_url":"https://api.github.com/users/jasondegraw/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"rraustad","id":5778156,"avatar_url":"https://avatars.githubusercontent.com/u/5778156?v=3","gravatar_id":"","url":"https://api.github.com/users/rraustad","html_url":"https://github.com/rraustad","followers_url":"https://api.github.com/users/rraustad/followers","following_url":"https://api.github.com/users/rraustad/following{/other_user}","gists_url":"https://api.github.com/users/rraustad/gists{/gist_id}","starred_url":"https://api.github.com/users/rraustad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rraustad/subscriptions","organizations_url":"https://api.github.com/users/rraustad/orgs","repos_url":"https://api.github.com/users/rraustad/repos","events_url":"https://api.github.com/users/rraustad/events{/privacy}","received_events_url":"https://api.github.com/users/rraustad/received_events","type":"User","site_admin":false},"milestone":null,"comments":13,"created_at":"2016-06-02T21:08:37Z","updated_at":"2016-06-08T01:07:05Z","closed_at":null,"body":"Issue overview\r\n--------------\r\nThe documentation for Table:OneIndependentVariable has this for the minimum value independent variable field:\r\n\r\n> Field: Minimum Value of X\r\n>\r\n> The minimum allowable value of X. Values of X less than the minimum will be replaced by the\r\n> minimum. This field can only limit extrapolation when the Interpolation Method is selected as\r\n> EvaluateCurveToLimits or LagrangeInterpolationLinearExtrapolation. This filed can be used to\r\n> limit interpolation using any Interpolation Method. If this field is left blank, the data set determines\r\n> the limit.\r\n\r\nThere documentation for the maximum value is similar. If the documentation is correct, then neither value should be required. But that doesn't seem to be the case. Here's a unit test that fails (as long as I cut-and-pasted correctly):\r\n\r\n    TEST_F(EnergyPlusFixture, Tables_OneIndependentVariable_MinMaxFailure) {\r\n        std::string const idf_objects = delimited_string({\r\n            \"Version,8.5;\",\r\n            \"Table:OneIndependentVariable,\",\r\n            \"TestTableMinMax,         !- Name\",\r\n            \"Linear,                  !- Curve Type\",\r\n            \"LinearInterpolationOfTable,  !- Interpolation Method\",\r\n            \",                        !- Minimum Value of X\",\r\n            \",                        !- Maximum Value of X\",\r\n            \",                        !- Minimum Table Output\",\r\n            \",                        !- Maximum Table Output\",\r\n            \"Dimensionless,           !- Input Unit Type for X\",\r\n            \"Dimensionless,           !- Output Unit Type\",\r\n            \",                        !- Normalization Reference\",\r\n            \"0,                       !- X Value #1\",\r\n            \"0,                       !- Output Value #1\",\r\n            \"1,                       !- X Value #2\",\r\n            \"1;                       !- Output Value #2\" });\r\n        ASSERT_FALSE(process_idf(idf_objects));\r\n        EXPECT_EQ(0, CurveManager::NumCurves);\r\n        CurveManager::GetCurveInput();\r\n        CurveManager::GetCurvesInputFlag = false;\r\n        ASSERT_EQ(1, CurveManager::NumCurves);\r\n        EXPECT_EQ(\"LINEAR\", CurveManager::GetCurveType(1));\r\n        EXPECT_EQ(\"TESTTABLEMINMAX\", CurveManager::GetCurveName(1));\r\n        EXPECT_EQ(1, CurveManager::GetCurveIndex(\"TESTTABLEMINMAX\"));\r\n        bool error = false;\r\n        int index = CurveManager::GetCurveCheck(\"TESTTABLEMINMAX\", error, \"TEST\");\r\n        EXPECT_FALSE(error);\r\n        EXPECT_EQ(1, index);\r\n        EXPECT_EQ(CurveManager::CurveType_TableOneIV, CurveManager::GetCurveObjectTypeNum(1));\r\n        EXPECT_DOUBLE_EQ(0.0, CurveManager::CurveValue(1, 0)); // In-range value\r\n        EXPECT_DOUBLE_EQ(0.75, CurveManager::CurveValue(1, 0.75)); // In-range value\r\n        EXPECT_DOUBLE_EQ(0.0, CurveManager::CurveValue(1, -10.0)); // Minimum x\r\n        EXPECT_DOUBLE_EQ(2.0, CurveManager::CurveValue(1, 5000)); // Maximum x\r\n        EXPECT_FALSE(has_err_output());\r\n    }\r\n\r\nAll of the value checks fail, it always returns 1. Here is my defect file:\r\n[AirflowNetwork_Simple_House_Linear_NoMinMax.idf.txt](https://github.com/NREL/EnergyPlus/files/296514/AirflowNetwork_Simple_House_Linear_NoMinMax.idf.txt)\r\n\r\n### Details\r\nSome additional details for this issue (if relevant):\r\n - Platform: Windows 7, 64 bit\r\n - Version of EnergyPlus: 8.5\r\n\r\n### Checklist\r\nAdd to this list or remove from it as applicable.  This is a simple templated set of guidelines.\r\n - [x] Defect file added\r\n - [ ] Ticket added to Pivotal for defect (development team task)\r\n - [ ] Pull request created (the pull request will have additional tasks related to reviewing changes that fix this defect)\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/NREL/EnergyPlus/issues/comments/224459361","html_url":"https://github.com/NREL/EnergyPlus/issues/5704#issuecomment-224459361","issue_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/5704","id":224459361,"user":{"login":"jasondegraw","id":4623433,"avatar_url":"https://avatars.githubusercontent.com/u/4623433?v=3","gravatar_id":"","url":"https://api.github.com/users/jasondegraw","html_url":"https://github.com/jasondegraw","followers_url":"https://api.github.com/users/jasondegraw/followers","following_url":"https://api.github.com/users/jasondegraw/following{/other_user}","gists_url":"https://api.github.com/users/jasondegraw/gists{/gist_id}","starred_url":"https://api.github.com/users/jasondegraw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jasondegraw/subscriptions","organizations_url":"https://api.github.com/users/jasondegraw/orgs","repos_url":"https://api.github.com/users/jasondegraw/repos","events_url":"https://api.github.com/users/jasondegraw/events{/privacy}","received_events_url":"https://api.github.com/users/jasondegraw/received_events","type":"User","site_admin":false},"created_at":"2016-06-08T01:07:05Z","updated_at":"2016-06-08T01:07:05Z","body":"@rraustad I agree 100% with @mjwitte"}}
{"actor":{"display_login":"jasondegraw"}}