{"action":"closed","issue":{"url":"https://api.github.com/repos/openhab/openhab/issues/2174","labels_url":"https://api.github.com/repos/openhab/openhab/issues/2174/labels{/name}","comments_url":"https://api.github.com/repos/openhab/openhab/issues/2174/comments","events_url":"https://api.github.com/repos/openhab/openhab/issues/2174/events","html_url":"https://github.com/openhab/openhab/issues/2174","id":58511000,"number":2174,"title":"ZWave Thermostat mode classes use BigDecimal","user":{"login":"cdjackson","id":1041475,"avatar_url":"https://avatars.githubusercontent.com/u/1041475?v=3","gravatar_id":"","url":"https://api.github.com/users/cdjackson","html_url":"https://github.com/cdjackson","followers_url":"https://api.github.com/users/cdjackson/followers","following_url":"https://api.github.com/users/cdjackson/following{/other_user}","gists_url":"https://api.github.com/users/cdjackson/gists{/gist_id}","starred_url":"https://api.github.com/users/cdjackson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cdjackson/subscriptions","organizations_url":"https://api.github.com/users/cdjackson/orgs","repos_url":"https://api.github.com/users/cdjackson/repos","events_url":"https://api.github.com/users/cdjackson/events{/privacy}","received_events_url":"https://api.github.com/users/cdjackson/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"cdjackson","id":1041475,"avatar_url":"https://avatars.githubusercontent.com/u/1041475?v=3","gravatar_id":"","url":"https://api.github.com/users/cdjackson","html_url":"https://github.com/cdjackson","followers_url":"https://api.github.com/users/cdjackson/followers","following_url":"https://api.github.com/users/cdjackson/following{/other_user}","gists_url":"https://api.github.com/users/cdjackson/gists{/gist_id}","starred_url":"https://api.github.com/users/cdjackson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cdjackson/subscriptions","organizations_url":"https://api.github.com/users/cdjackson/orgs","repos_url":"https://api.github.com/users/cdjackson/repos","events_url":"https://api.github.com/users/cdjackson/events{/privacy}","received_events_url":"https://api.github.com/users/cdjackson/received_events","type":"User","site_admin":false},"milestone":null,"comments":11,"created_at":"2015-02-22T17:44:59Z","updated_at":"2015-03-06T18:53:48Z","closed_at":"2015-03-06T18:53:48Z","body":"@digitaldan - do you know if there is there a reason that the thermostat mode classes use BigDecimal? I suspect it's because when they were written, the setpoint class used BigDecimal, so this was also used for the other classes.\r\nI came across this when adding a converter elsewhere, but I don't think it's needed (8 bytes to represent 8 bits!) and have removed it (using Integer instead) as BigDecimal is expensive.\r\nI'm going to push this into my \"configuration item\" branch since that's where I came across it - I was wondering if you could grab a copy and give it a quick test? (or if you think it's needed, please tell me I'm stupid! :) ).\r\nI'll reference the commit here...\r\n"}}