{"action":"created","issue":{"url":"https://api.github.com/repos/NREL/EnergyPlus/issues/6192","repository_url":"https://api.github.com/repos/NREL/EnergyPlus","labels_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/6192/labels{/name}","comments_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/6192/comments","events_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/6192/events","html_url":"https://github.com/NREL/EnergyPlus/pull/6192","id":239815922,"node_id":"MDExOlB1bGxSZXF1ZXN0MTI4Mzk2OTE4","number":6192,"title":"JSON (and CBOR/MessagePack) Tabular and Timeseries Outputs","user":{"login":"mbadams5","id":2001544,"node_id":"MDQ6VXNlcjIwMDE1NDQ=","avatar_url":"https://avatars1.githubusercontent.com/u/2001544?v=4","gravatar_id":"","url":"https://api.github.com/users/mbadams5","html_url":"https://github.com/mbadams5","followers_url":"https://api.github.com/users/mbadams5/followers","following_url":"https://api.github.com/users/mbadams5/following{/other_user}","gists_url":"https://api.github.com/users/mbadams5/gists{/gist_id}","starred_url":"https://api.github.com/users/mbadams5/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mbadams5/subscriptions","organizations_url":"https://api.github.com/users/mbadams5/orgs","repos_url":"https://api.github.com/users/mbadams5/repos","events_url":"https://api.github.com/users/mbadams5/events{/privacy}","received_events_url":"https://api.github.com/users/mbadams5/received_events","type":"User","site_admin":false},"labels":[{"id":74848839,"node_id":"MDU6TGFiZWw3NDg0ODgzOQ==","url":"https://api.github.com/repos/NREL/EnergyPlus/labels/EnergyPlus","name":"EnergyPlus","color":"c7def8","default":false},{"id":429517840,"node_id":"MDU6TGFiZWw0Mjk1MTc4NDA=","url":"https://api.github.com/repos/NREL/EnergyPlus/labels/IDDChange","name":"IDDChange","color":"006b75","default":false},{"id":179739851,"node_id":"MDU6TGFiZWwxNzk3Mzk4NTE=","url":"https://api.github.com/repos/NREL/EnergyPlus/labels/NewFeature","name":"NewFeature","color":"db5700","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/NREL/EnergyPlus/milestones/9","html_url":"https://github.com/NREL/EnergyPlus/milestone/9","labels_url":"https://api.github.com/repos/NREL/EnergyPlus/milestones/9/labels","id":3540652,"node_id":"MDk6TWlsZXN0b25lMzU0MDY1Mg==","number":9,"title":"EnergyPlus Next","description":"","creator":{"login":"Myoldmopar","id":849824,"node_id":"MDQ6VXNlcjg0OTgyNA==","avatar_url":"https://avatars0.githubusercontent.com/u/849824?v=4","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},"open_issues":44,"closed_issues":6,"state":"open","created_at":"2018-08-02T00:26:33Z","updated_at":"2018-11-09T11:42:47Z","due_on":"2019-03-31T07:00:00Z","closed_at":null},"comments":35,"created_at":"2017-06-30T15:38:03Z","updated_at":"2018-11-12T02:04:24Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/NREL/EnergyPlus/pulls/6192","html_url":"https://github.com/NREL/EnergyPlus/pull/6192","diff_url":"https://github.com/NREL/EnergyPlus/pull/6192.diff","patch_url":"https://github.com/NREL/EnergyPlus/pull/6192.patch"},"body":"Pull request overview\r\n---------------------\r\nThis PR is dependent on #5893 going in first since this is built off that branch. The following link provides a more accurate comparison of the features in this specific PR, [seen here.](https://github.com/NREL/EnergyPlus/compare/input_processor_refactor...output_processor_json)\r\n\r\nThis PR adds JSON outputs for Tabular and Timeseries outputs. Due to the JSON library we are using for the JSON input processing, we can also output [CBOR](http://cbor.io/) and [MessagePack](http://msgpack.org/) binary formats. Both of these formats provide faster serialization/deserialization and much smaller file sizes.\r\n\r\nThe JSON output format was built off of work completed a couple years ago and updated to current EnergyPlus source code and JSON library.\r\n\r\nRunning the Outpatient Reference Model for an annual simulation has the following file sizes:\r\n\r\neplusout.mtr - 2.1M\r\neplusmtr.csv - 1.6M\r\neplusout.eso - 31M\r\neplusout.csv - 24M\r\neplusout.sql - 31M\r\n\r\neplusout_hourly.json - 42M\r\neplusout_hourly.cbor - 12M (~71% reduction to JSON, 50% compared to CSV)\r\n\r\neplusout.json - 5.0M\r\neplusout.cbor - 1.2M (76% reduction)\r\n\r\n### Work Checklist\r\nAdd to this list or remove from it as applicable.  This is a simple templated set of guidelines.\r\n - [ ] Title of PR should be user-synopsis style (clearly understandable in a standalone changelog context)\r\n - [ ] 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 - C++ checks:\r\n   - [ ] Argument types\r\n   - [ ] If any virtual classes, ensure virtual destructor included, other things\r\n - IDD changes:\r\n   - [ ] Verify naming conventions and styles, memos and notes and defaults\r\n   - [ ] Open windows IDF Editor with modified IDD to check for errors\r\n   - [ ] If transition, add rules to spreadsheet\r\n   - [ ] If transition, add transition source\r\n   - [ ] If transition, update idfs\r\n - [ ] If new idf included, locally check the err file and other outputs\r\n - [ ] Documentation changes in place\r\n - [ ] Changed docs build successfully\r\n - [ ] ExpandObjects changes?\r\n - [ ] If output changes, including tabular output structure, add to output rules file for interfaces\r\n"},"comment":{"url":"https://api.github.com/repos/NREL/EnergyPlus/issues/comments/437729948","html_url":"https://github.com/NREL/EnergyPlus/pull/6192#issuecomment-437729948","issue_url":"https://api.github.com/repos/NREL/EnergyPlus/issues/6192","id":437729948,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNzcyOTk0OA==","user":{"login":"nrel-bot","id":7593710,"node_id":"MDQ6VXNlcjc1OTM3MTA=","avatar_url":"https://avatars3.githubusercontent.com/u/7593710?v=4","gravatar_id":"","url":"https://api.github.com/users/nrel-bot","html_url":"https://github.com/nrel-bot","followers_url":"https://api.github.com/users/nrel-bot/followers","following_url":"https://api.github.com/users/nrel-bot/following{/other_user}","gists_url":"https://api.github.com/users/nrel-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/nrel-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nrel-bot/subscriptions","organizations_url":"https://api.github.com/users/nrel-bot/orgs","repos_url":"https://api.github.com/users/nrel-bot/repos","events_url":"https://api.github.com/users/nrel-bot/events{/privacy}","received_events_url":"https://api.github.com/users/nrel-bot/received_events","type":"User","site_admin":false},"created_at":"2018-11-12T02:04:24Z","updated_at":"2018-11-12T02:04:24Z","author_association":"NONE","body":"@mbadams5 @lgentile it has been 25 days since this pull request was last updated."}}
{"actor":{"display_login":"nrel-bot"}}