{"action":"created","issue":{"url":"https://api.github.com/repos/RoaldFre/wacom_serial5/issues/3","labels_url":"https://api.github.com/repos/RoaldFre/wacom_serial5/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/RoaldFre/wacom_serial5/issues/3/comments","events_url":"https://api.github.com/repos/RoaldFre/wacom_serial5/issues/3/events","html_url":"https://github.com/RoaldFre/wacom_serial5/pull/3","id":97866475,"number":3,"title":"Modify wacom_serial5 Thumbwheel -> Rel Scrollwheel","user":{"login":"MackPI","id":13550094,"avatar_url":"https://avatars.githubusercontent.com/u/13550094?v=3","gravatar_id":"","url":"https://api.github.com/users/MackPI","html_url":"https://github.com/MackPI","followers_url":"https://api.github.com/users/MackPI/followers","following_url":"https://api.github.com/users/MackPI/following{/other_user}","gists_url":"https://api.github.com/users/MackPI/gists{/gist_id}","starred_url":"https://api.github.com/users/MackPI/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MackPI/subscriptions","organizations_url":"https://api.github.com/users/MackPI/orgs","repos_url":"https://api.github.com/users/MackPI/repos","events_url":"https://api.github.com/users/MackPI/events{/privacy}","received_events_url":"https://api.github.com/users/MackPI/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-29T05:56:48Z","updated_at":"2015-07-31T11:56:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RoaldFre/wacom_serial5/pulls/3","html_url":"https://github.com/RoaldFre/wacom_serial5/pull/3","diff_url":"https://github.com/RoaldFre/wacom_serial5/pull/3.diff","patch_url":"https://github.com/RoaldFre/wacom_serial5/pull/3.patch"},"body":"Added new code to take thumbwheel absolute value and translate that into scroll wheel actions with variable speeds."},"comment":{"url":"https://api.github.com/repos/RoaldFre/wacom_serial5/issues/comments/126667660","html_url":"https://github.com/RoaldFre/wacom_serial5/pull/3#issuecomment-126667660","issue_url":"https://api.github.com/repos/RoaldFre/wacom_serial5/issues/3","id":126667660,"user":{"login":"RoaldFre","id":251349,"avatar_url":"https://avatars.githubusercontent.com/u/251349?v=3","gravatar_id":"","url":"https://api.github.com/users/RoaldFre","html_url":"https://github.com/RoaldFre","followers_url":"https://api.github.com/users/RoaldFre/followers","following_url":"https://api.github.com/users/RoaldFre/following{/other_user}","gists_url":"https://api.github.com/users/RoaldFre/gists{/gist_id}","starred_url":"https://api.github.com/users/RoaldFre/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RoaldFre/subscriptions","organizations_url":"https://api.github.com/users/RoaldFre/orgs","repos_url":"https://api.github.com/users/RoaldFre/repos","events_url":"https://api.github.com/users/RoaldFre/events{/privacy}","received_events_url":"https://api.github.com/users/RoaldFre/received_events","type":"User","site_admin":false},"created_at":"2015-07-31T11:56:07Z","updated_at":"2015-07-31T11:56:07Z","body":"Hi MackPI,\r\n\r\nNice to see that people are using and contributing to the serial wacom driver! :-)\r\n\r\nI don't have a 4D mouse to test the code myself, but ideally switching between absolute and relative ('scrollwheel') mode should be something that can be done at runtime through some file in /sys or /proc. Likewise, the scroll sensitivity/delay of 800 should probably be configurable like that as well.\r\n\r\nI'm not too sure of the precision of those wheels, but if it springs back to the center and ends up at some small non-zero value (e.g stuck at +1 or -2 or something like that), then that can accumulate over time to give spurious/unexpected scroll inputs. If that's the case, you may need to introduce a deadzone where you don't take into account wheel deviations between -x and +x, for instance. You probably also want to reset the accumulated value when the wheel hits zero again (or enters this small deadzone), as well, so that the first subsequent scroll action behaves as expected.\r\n\r\nI probably won't have time to look into this myself the coming days (and I can't test it with a 4D mouse), but if you can find the time to do those adjustments, I'll gladly merge the changes ;-)\r\n\r\nCheers!\r\nRoald"}}