{"action":"created","issue":{"url":"https://api.github.com/repos/mapbox/carmen/issues/341","labels_url":"https://api.github.com/repos/mapbox/carmen/issues/341/labels{/name}","comments_url":"https://api.github.com/repos/mapbox/carmen/issues/341/comments","events_url":"https://api.github.com/repos/mapbox/carmen/issues/341/events","html_url":"https://github.com/mapbox/carmen/pull/341","id":109420598,"number":341,"title":"Versions Are Cheap geocoder_format (WIP)","user":{"login":"ingalls","id":1297009,"avatar_url":"https://avatars.githubusercontent.com/u/1297009?v=3","gravatar_id":"","url":"https://api.github.com/users/ingalls","html_url":"https://github.com/ingalls","followers_url":"https://api.github.com/users/ingalls/followers","following_url":"https://api.github.com/users/ingalls/following{/other_user}","gists_url":"https://api.github.com/users/ingalls/gists{/gist_id}","starred_url":"https://api.github.com/users/ingalls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ingalls/subscriptions","organizations_url":"https://api.github.com/users/ingalls/orgs","repos_url":"https://api.github.com/users/ingalls/repos","events_url":"https://api.github.com/users/ingalls/events{/privacy}","received_events_url":"https://api.github.com/users/ingalls/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-10-02T02:42:28Z","updated_at":"2015-10-02T12:21:09Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mapbox/carmen/pulls/341","html_url":"https://github.com/mapbox/carmen/pull/341","diff_url":"https://github.com/mapbox/carmen/pull/341.diff","patch_url":"https://github.com/mapbox/carmen/pull/341.patch"},"body":"Captured from chat\r\n\r\n```\r\ningalls [22:09]\r\nunintended side affects\r\nbasically really really early on the geocoder_address flag is used to determine if phrasematch should generate address permulations for a given source\r\nfor most layers this doesn’t really matter\r\nas places/regions etc\r\ndon’t have numbers in them\r\n\r\nheyitsgarrett [22:09] \r\nright\r\n\r\ningalls [22:09] \r\nso the address permutations fail and are then ignored\r\nfor postcode however\r\nthe address permutations ​_can_​ be generated\r\nas a number is found\r\nthat is a potential housenumbers\r\nbut when it goes back to actually look up this permutation in the index\r\nit can’t find a housenumber that matches\r\nbecause the postcode indexes don’t have the additional housenumber info\r\nwhich results in the postcode index hard failing and returning no results\r\n```\r\n\r\ncc/ @heyitsgarrett @sbma44 "},"comment":{"url":"https://api.github.com/repos/mapbox/carmen/issues/comments/145001609","html_url":"https://github.com/mapbox/carmen/pull/341#issuecomment-145001609","issue_url":"https://api.github.com/repos/mapbox/carmen/issues/341","id":145001609,"user":{"login":"ingalls","id":1297009,"avatar_url":"https://avatars.githubusercontent.com/u/1297009?v=3","gravatar_id":"","url":"https://api.github.com/users/ingalls","html_url":"https://github.com/ingalls","followers_url":"https://api.github.com/users/ingalls/followers","following_url":"https://api.github.com/users/ingalls/following{/other_user}","gists_url":"https://api.github.com/users/ingalls/gists{/gist_id}","starred_url":"https://api.github.com/users/ingalls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ingalls/subscriptions","organizations_url":"https://api.github.com/users/ingalls/orgs","repos_url":"https://api.github.com/users/ingalls/repos","events_url":"https://api.github.com/users/ingalls/events{/privacy}","received_events_url":"https://api.github.com/users/ingalls/received_events","type":"User","site_admin":false},"created_at":"2015-10-02T12:21:09Z","updated_at":"2015-10-02T12:21:09Z","body":"No this just means the build script in mapbox places will need to set\ngeocoder_asdress:1 when the extid is address. The formatting of addresses\nwill now be in geocoder_format\n\nOn Friday, October 2, 2015, Ian Villeda <notifications@github.com> wrote:\n\n> ooh @incanus <https://github.com/incanus> @heyitsgarrett\n> <https://github.com/heyitsgarrett> this means we need to preserve the\n> info in addrType\n> <https://github.com/mapbox/mapbox-places/blob/master/scripts/addrType>,\n> right?\n>\n> i wonder if carmen can parse the whole geocoder_format template to get\n> the right address name/num order so we dont have to store address format\n> twice (once in geocoder_address and once in geocoder_format)\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/mapbox/carmen/pull/341#issuecomment-145001157>.\n>\n"}}