{"action":"created","issue":{"url":"https://api.github.com/repos/crflynn/chicken-dinner/issues/5","repository_url":"https://api.github.com/repos/crflynn/chicken-dinner","labels_url":"https://api.github.com/repos/crflynn/chicken-dinner/issues/5/labels{/name}","comments_url":"https://api.github.com/repos/crflynn/chicken-dinner/issues/5/comments","events_url":"https://api.github.com/repos/crflynn/chicken-dinner/issues/5/events","html_url":"https://github.com/crflynn/chicken-dinner/issues/5","id":344218929,"node_id":"MDU6SXNzdWUzNDQyMTg5Mjk=","number":5,"title":"Telemetry xbox vs pc ( specifically 'blueZoneCustomOptions' )","user":{"login":"CemuUser8","id":29107989,"node_id":"MDQ6VXNlcjI5MTA3OTg5","avatar_url":"https://avatars2.githubusercontent.com/u/29107989?v=4","gravatar_id":"","url":"https://api.github.com/users/CemuUser8","html_url":"https://github.com/CemuUser8","followers_url":"https://api.github.com/users/CemuUser8/followers","following_url":"https://api.github.com/users/CemuUser8/following{/other_user}","gists_url":"https://api.github.com/users/CemuUser8/gists{/gist_id}","starred_url":"https://api.github.com/users/CemuUser8/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/CemuUser8/subscriptions","organizations_url":"https://api.github.com/users/CemuUser8/orgs","repos_url":"https://api.github.com/users/CemuUser8/repos","events_url":"https://api.github.com/users/CemuUser8/events{/privacy}","received_events_url":"https://api.github.com/users/CemuUser8/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-07-24T21:58:09Z","updated_at":"2018-07-24T22:52:26Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"Hey, sorry, me again.\r\n\r\nI got around to playing with telemetry data, and it always fails for me with xbox matches. This time I'm not sure what the fix could possibly be, but like my last issue I did my best to trace down the problem.\r\n\r\nIt appears to be within Telemetry/events.py\r\n\r\non `__init__`  at this part:\r\n```Python\r\n    elif k == \"blueZoneCustomOptions\":  # serialized json\r\n        if len(v) > 2:  # more than just '[]'\r\n            setattr(self, camel_to_snake(k), TelemetryObject(json.loads(v), k))\r\n         else:\r\n             setattr(self, camel_to_snake(k), json.loads(v))\r\n    else:\r\n        setattr(self, camel_to_snake(k), v)\r\n```\r\n\r\nSpecifically with `TelemetryObject(json.loads(v), k)`  on the condition when `if len(v) > 2` ( On Xbox it appears to always be filled, with every attribute set to 0? )\r\n\r\nthe `json.loads(v)` returns a list of dicts in this case instead of just a dict. Which then fails on the `__init__` of the TelemetryObject since it calls `items()` which doesn't exist on a list.\r\n\r\n\r\nUnfortunately every test I do using PC  `v` is always equal to `[]` so it never gets to this point, so I am unsure if it is just an Xbox issue, or if a PC match with those values set would also cause problems as well.\r\n\r\n\r\n&nbsp;\r\n\r\nAs an unrelated find I made, which I can just create a pull request for if you'd like (now that I know how) is in:\r\n\r\nTelemetry\\telemetry.py\r\n\r\nI added `from chicken_dinner.constants import map_to_map_name` since you use that mapping but it fails if I don't have this included."},"comment":{"url":"https://api.github.com/repos/crflynn/chicken-dinner/issues/comments/407577454","html_url":"https://github.com/crflynn/chicken-dinner/issues/5#issuecomment-407577454","issue_url":"https://api.github.com/repos/crflynn/chicken-dinner/issues/5","id":407577454,"node_id":"MDEyOklzc3VlQ29tbWVudDQwNzU3NzQ1NA==","user":{"login":"CemuUser8","id":29107989,"node_id":"MDQ6VXNlcjI5MTA3OTg5","avatar_url":"https://avatars2.githubusercontent.com/u/29107989?v=4","gravatar_id":"","url":"https://api.github.com/users/CemuUser8","html_url":"https://github.com/CemuUser8","followers_url":"https://api.github.com/users/CemuUser8/followers","following_url":"https://api.github.com/users/CemuUser8/following{/other_user}","gists_url":"https://api.github.com/users/CemuUser8/gists{/gist_id}","starred_url":"https://api.github.com/users/CemuUser8/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/CemuUser8/subscriptions","organizations_url":"https://api.github.com/users/CemuUser8/orgs","repos_url":"https://api.github.com/users/CemuUser8/repos","events_url":"https://api.github.com/users/CemuUser8/events{/privacy}","received_events_url":"https://api.github.com/users/CemuUser8/received_events","type":"User","site_admin":false},"created_at":"2018-07-24T22:52:26Z","updated_at":"2018-07-24T22:52:26Z","author_association":"CONTRIBUTOR","body":"That certainly looks much cleaner than what I had. Thank you again for taking the time to respond, and so quickly. I am still learning, and also for some reason am addicted to pulling stats for PUBG for myself and my friends. I wouldn't be able to do anything without a wrapper like yours, and the ability to create a visual so easily on my end is amazing.\r\n\r\nI'll let you make the changes since its a single import, and you already have a new cleaner idea it seems like."}}
{"actor":{"display_login":"CemuUser8"}}