{"action":"created","issue":{"url":"https://api.github.com/repos/NREL/EnergyPlus/issues/5669","repository_url":"https://api.github.com/repos/NREL/EnergyPlus","labels_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/5669/labels{/name}","comments_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/5669/comments","events_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/5669/events","html_url":"https://github.com/NREL/EnergyPlus/pull/5669","id":155610254,"number":5669,"title":"Fix conversion of unitless variables to IP in tabular output","user":{"login":"nealkruis","id":1964570,"avatar_url":"https://avatars.githubusercontent.com/u/1964570?v=3","gravatar_id":"","url":"https://api.github.com/users/nealkruis","html_url":"https://github.com/nealkruis","followers_url":"https://api.github.com/users/nealkruis/followers","following_url":"https://api.github.com/users/nealkruis/following{/other_user}","gists_url":"https://api.github.com/users/nealkruis/gists{/gist_id}","starred_url":"https://api.github.com/users/nealkruis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nealkruis/subscriptions","organizations_url":"https://api.github.com/users/nealkruis/orgs","repos_url":"https://api.github.com/users/nealkruis/repos","events_url":"https://api.github.com/users/nealkruis/events{/privacy}","received_events_url":"https://api.github.com/users/nealkruis/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/NREL/EnergyPlus/labels/Defect","name":"Defect","color":"e11d21"},{"url":"https://api.github.com/repos/NREL/EnergyPlus/labels/Priority","name":"Priority","color":"e11d21"}],"state":"open","locked":false,"assignee":{"login":"JasonGlazer","id":2582150,"avatar_url":"https://avatars.githubusercontent.com/u/2582150?v=3","gravatar_id":"","url":"https://api.github.com/users/JasonGlazer","html_url":"https://github.com/JasonGlazer","followers_url":"https://api.github.com/users/JasonGlazer/followers","following_url":"https://api.github.com/users/JasonGlazer/following{/other_user}","gists_url":"https://api.github.com/users/JasonGlazer/gists{/gist_id}","starred_url":"https://api.github.com/users/JasonGlazer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JasonGlazer/subscriptions","organizations_url":"https://api.github.com/users/JasonGlazer/orgs","repos_url":"https://api.github.com/users/JasonGlazer/repos","events_url":"https://api.github.com/users/JasonGlazer/events{/privacy}","received_events_url":"https://api.github.com/users/JasonGlazer/received_events","type":"User","site_admin":false},"milestone":null,"comments":20,"created_at":"2016-05-18T21:52:40Z","updated_at":"2016-06-16T17:21:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/NREL/EnergyPlus/pulls/5669","html_url":"https://github.com/NREL/EnergyPlus/pull/5669","diff_url":"https://github.com/NREL/EnergyPlus/pull/5669.diff","patch_url":"https://github.com/NREL/EnergyPlus/pull/5669.patch"},"body":"Pull request overview\r\n---------------------\r\nFixes #5668.\r\n\r\n### Work Checklist\r\n - [x] Title of PR should be user-synopsis style (clearly understandable in a standalone changelog context)\r\n - [x] At least one of the following appropriate labels must be added to this PR to be consumed into the changelog:\r\n   - Defect: This pull request repairs a github defect issue.  The github issue should be referenced in the PR description\r\n   - Refactoring: This pull request includes code changes that don't change the functionality of the program, just perform refactoring\r\n   - NewFeature: This pull request includes code to add a new feature to EnergyPlus\r\n   - Performance: This pull request includes code changes that are directed at improving the runtime performance of EnergyPlus\r\n   - DoNoPublish: This pull request includes changes that shouldn't be included in the changelog\r\n\r\n### Review Checklist\r\nThis will not be exhaustively relevant to every PR.\r\n - [ ] Code style (parentheses padding, variable names)\r\n - [ ] Functional code review (it has to work!)\r\n - [ ] If defect, results of running current develop vs this branch should exhibit the fix\r\n - [ ] CI status: all green or justified\r\n - [ ] Performance: CI Linux results include performance check -- verify this\r\n - [ ] Unit Test(s)\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/NREL/EnergyPlus/issues/comments/226553556","html_url":"https://github.com/NREL/EnergyPlus/pull/5669#issuecomment-226553556","issue_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/5669","id":226553556,"user":{"login":"nealkruis","id":1964570,"avatar_url":"https://avatars.githubusercontent.com/u/1964570?v=3","gravatar_id":"","url":"https://api.github.com/users/nealkruis","html_url":"https://github.com/nealkruis","followers_url":"https://api.github.com/users/nealkruis/followers","following_url":"https://api.github.com/users/nealkruis/following{/other_user}","gists_url":"https://api.github.com/users/nealkruis/gists{/gist_id}","starred_url":"https://api.github.com/users/nealkruis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nealkruis/subscriptions","organizations_url":"https://api.github.com/users/nealkruis/orgs","repos_url":"https://api.github.com/users/nealkruis/repos","events_url":"https://api.github.com/users/nealkruis/events{/privacy}","received_events_url":"https://api.github.com/users/nealkruis/received_events","type":"User","site_admin":false},"created_at":"2016-06-16T17:21:45Z","updated_at":"2016-06-16T17:21:45Z","body":"@mjwitte I 100% agree. Maybe we can define a small task to do this at the planning meeting? I think a regex script should be able to handle 95% of the SetupOutputVariable instances. Most of the work would be in designing the table and refactoring the current functions that interpret units."}}
{"actor":{"display_login":"nealkruis"}}