{"action":"created","issue":{"url":"https://api.github.com/repos/stepmania/stepmania/issues/539","labels_url":"https://api.github.com/repos/stepmania/stepmania/issues/539/labels{/name}","comments_url":"https://api.github.com/repos/stepmania/stepmania/issues/539/comments","events_url":"https://api.github.com/repos/stepmania/stepmania/issues/539/events","html_url":"https://github.com/stepmania/stepmania/issues/539","id":66025207,"number":539,"title":"Should radar values be uncapped?","user":{"login":"kyzentun","id":6657507,"avatar_url":"https://avatars.githubusercontent.com/u/6657507?v=3","gravatar_id":"","url":"https://api.github.com/users/kyzentun","html_url":"https://github.com/kyzentun","followers_url":"https://api.github.com/users/kyzentun/followers","following_url":"https://api.github.com/users/kyzentun/following{/other_user}","gists_url":"https://api.github.com/users/kyzentun/gists{/gist_id}","starred_url":"https://api.github.com/users/kyzentun/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kyzentun/subscriptions","organizations_url":"https://api.github.com/users/kyzentun/orgs","repos_url":"https://api.github.com/users/kyzentun/repos","events_url":"https://api.github.com/users/kyzentun/events{/privacy}","received_events_url":"https://api.github.com/users/kyzentun/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-04-02T22:40:31Z","updated_at":"2015-04-06T19:54:48Z","closed_at":null,"body":"Two things:  Capping, and inaccuracy of Stream and Voltage\r\n1.  Currently, Stream, Voltage, Air, Freeze, and Chaos radar values are capped at 1 by this line of code:\r\nhttps://github.com/stepmania/stepmania/blob/master/src/NoteDataUtil.cpp#L1120\r\n\r\nIf they were uncapped, they could be relevant at higher difficulties, instead of being discarded by practically all skilled players.\r\n\r\n\r\n2.  As mentioned in a comment, Stream and Voltage calculations count hold heads twice, and roll heads once.  This is because the old (slow) methods did it by accident, and rather than deal with any possible backlash, I duplicated that behavior in the new calculation method.\r\n\r\nI believe this should be fixed so that hold heads are only counted once, because counting something that only requires a single step twice is silly."},"comment":{"url":"https://api.github.com/repos/stepmania/stepmania/issues/comments/90222736","html_url":"https://github.com/stepmania/stepmania/issues/539#issuecomment-90222736","issue_url":"https://api.github.com/repos/stepmania/stepmania/issues/539","id":90222736,"user":{"login":"dguzek","id":1253483,"avatar_url":"https://avatars.githubusercontent.com/u/1253483?v=3","gravatar_id":"","url":"https://api.github.com/users/dguzek","html_url":"https://github.com/dguzek","followers_url":"https://api.github.com/users/dguzek/followers","following_url":"https://api.github.com/users/dguzek/following{/other_user}","gists_url":"https://api.github.com/users/dguzek/gists{/gist_id}","starred_url":"https://api.github.com/users/dguzek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dguzek/subscriptions","organizations_url":"https://api.github.com/users/dguzek/orgs","repos_url":"https://api.github.com/users/dguzek/repos","events_url":"https://api.github.com/users/dguzek/events{/privacy}","received_events_url":"https://api.github.com/users/dguzek/received_events","type":"User","site_admin":false},"created_at":"2015-04-06T19:54:48Z","updated_at":"2015-04-06T19:54:48Z","body":"I am in favor counting hold heads once.\r\n\r\nI cannot offer any useful feedback on capping/not-capping RadarValues.  I don't actually understand what RadarValues are, how they're used currently, or how they might benefit from the first proposed change."}}