{"action":"created","issue":{"url":"https://api.github.com/repos/mapsme/omim/issues/3922","repository_url":"https://api.github.com/repos/mapsme/omim","labels_url":"https://api.github.com/repos/mapsme/omim/issues/3922/labels{/name}","comments_url":"https://api.github.com/repos/mapsme/omim/issues/3922/comments","events_url":"https://api.github.com/repos/mapsme/omim/issues/3922/events","html_url":"https://github.com/mapsme/omim/pull/3922","id":168871273,"number":3922,"title":"Using DeltaCoder bit writer to save altitude.","user":{"login":"bykoianko","id":1768114,"avatar_url":"https://avatars.githubusercontent.com/u/1768114?v=3","gravatar_id":"","url":"https://api.github.com/users/bykoianko","html_url":"https://github.com/bykoianko","followers_url":"https://api.github.com/users/bykoianko/followers","following_url":"https://api.github.com/users/bykoianko/following{/other_user}","gists_url":"https://api.github.com/users/bykoianko/gists{/gist_id}","starred_url":"https://api.github.com/users/bykoianko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bykoianko/subscriptions","organizations_url":"https://api.github.com/users/bykoianko/orgs","repos_url":"https://api.github.com/users/bykoianko/repos","events_url":"https://api.github.com/users/bykoianko/events{/privacy}","received_events_url":"https://api.github.com/users/bykoianko/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":18,"created_at":"2016-08-02T12:19:48Z","updated_at":"2016-08-04T08:28:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mapsme/omim/pulls/3922","html_url":"https://github.com/mapsme/omim/pull/3922","diff_url":"https://github.com/mapsme/omim/pull/3922.diff","patch_url":"https://github.com/mapsme/omim/pull/3922.patch"},"body":"При подготовке данного PR я сравнил ряд способов хранения высот точек фич на картах: Амстердама, Москвы, Катмаду и Парижа. Были проверены следующие способы:\r\n1. хранение разницы между минимальной высотой mwm и первой точной фичи, а далее разницу между предыдущей высотой и нынешней высотой в VarInt;\r\n2. тоже самое в GammaCoder;\r\n3. тоже самое в DeltaCoder;\r\n4. хранение отклонений от линейной модели в GammaCoder;\r\n5. хранение отклонений от линейной модели в DeltaCoder;\r\n\r\nЛидером оказался вариант 3. Он и реализован в данном PR.\r\n\r\nПодробно результаты исследований описаны в https://docs.google.com/document/d/1mZJ5SsaHZCYTLgiXPGznv1OVyvFWYJyNENnlp8H4jXQ/edit (глава Результаты исследования хранения информации о высоте разными способами)\r\n\r\nКод, которым я тестировал хранение отклонения от линейной модели находится здесь: https://github.com/mapsme/omim/pull/3921\r\n\r\nhttps://jira.mail.ru/browse/MAPSME-2051\r\n\r\n@ygorshenin @mpimenov @Zverik PTAL"},"comment":{"url":"https://api.github.com/repos/mapsme/omim/issues/comments/237487503","html_url":"https://github.com/mapsme/omim/pull/3922#issuecomment-237487503","issue_url":"https://api.github.com/repos/mapsme/omim/issues/3922","id":237487503,"user":{"login":"ygorshenin","id":454567,"avatar_url":"https://avatars.githubusercontent.com/u/454567?v=3","gravatar_id":"","url":"https://api.github.com/users/ygorshenin","html_url":"https://github.com/ygorshenin","followers_url":"https://api.github.com/users/ygorshenin/followers","following_url":"https://api.github.com/users/ygorshenin/following{/other_user}","gists_url":"https://api.github.com/users/ygorshenin/gists{/gist_id}","starred_url":"https://api.github.com/users/ygorshenin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ygorshenin/subscriptions","organizations_url":"https://api.github.com/users/ygorshenin/orgs","repos_url":"https://api.github.com/users/ygorshenin/repos","events_url":"https://api.github.com/users/ygorshenin/events{/privacy}","received_events_url":"https://api.github.com/users/ygorshenin/received_events","type":"User","site_admin":false},"created_at":"2016-08-04T08:28:53Z","updated_at":"2016-08-04T08:28:53Z","body":"LGTM"}}
{"actor":{"display_login":"ygorshenin"}}