-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ReadME #18
Comments
Yes, I would be interested too. I was not sure how all this works. @ellenhp, you mentioned you run both this engine and then object storage on Fly.io. How exactly do they communicate? How to set it up? Such a great project to reduce costs on geocoding. Please, provide some sort of documentation/tutorial, sort of Getting Started. Thank you! |
Thanks for the interest! Yes, I'm hoping to tackle documentation very soon. My plan is to
My hope was to be able to claim that Airmail, despite its tiny footprint, is only X% behind Pelias in geocoding performance. for the initial release, but if people want to test now I can prioritize documentation. There's not much to do while the model is training and I have a bunch of free time this weekend so I'll probably start on that. :) |
These improvements are also important . Thank you. If you anticipate significant changes to the documentation following these improvements, I can wait documentation. However, if the changes will be minimal, it would be preferable to have the documentation first. Regardless, we'll be pleased to have the documentation whenever it's ready. |
Hello @ellenhp, could you please direct me on where to find the 'spatial_custom' docker image? After hours, i managed to install and compile the code successful, but the program exits with an error indicating it cannot locate the docker image. If everything works, i will submit a pull for the documenation. By the way i searched 'spatial_custom' on public docker images but i can't find it yet.
|
That's here: https://github.com/ellenhp/spatial I had to fork it from upstream due to a bug/missing feature to return all names in all languages for a whosonfirst administrative area. PR here, but I'm not certain if it'll be accepted so I'll probably set up some CI and push my forked container to ghcr. pelias/spatial#84 Just clone my fork and |
There may be some rough edges relating to the indexing process but from memory I think you're pretty far along if you're running into that failure. If you do manage to build an index and stuff please let me know what your impression is of the process, which parts were hardest, and what your impression of the API is. Everything about the project is subject to change because I haven't attached a version number to a release yet, but the upside of that is we can reshape it to be more ergonomic pretty easily, so let me know :) |
Another thing to be aware of is that you should enable the |
@ellenhp, is this going to work only for geocoding and not reverse geocoding (find approximate place/city name from lat/lng coordinates)? Thanks. |
I may add reverse geocoding later but for the time being it's forward only. edit: Sorry I mixed up placeholder and the spatial server. This is the one that I'd recommend for coarse reverse geocoding. https://github.com/pelias/spatial Airmail uses it internally to get high-quality information about which administrative areas a POI is in, since that information is often missing from OSM. |
@ellenhp Thank you for the Info. i managed to build your custom spatial image. Now when i run the program it seem to run without errors but theres this warning that makes indexing not to work Here is the full log
After the run, I started the API like this |
So far, my main challenges revolve around the operating system and third-party programs. For example, the build process (e.g cargo build) couldn't complete on Ubuntu 22.04; I had to switch to Debian, where the build was successful. Additionally, I encountered difficulties with using osmx as the binary didn't function for me, and i had to building it from source (e.g., using 'make'). Other issues stemmed from my limited experience with Rust and Docker. I'll share my API experience with you once I manage to run it. This is the error i get when i try to build it on Ubuntu 22.04. (it works fine on Debian GNU/Linux 12 (bookworm))
|
That's really frustrating that you had to switch operating systems to get it to build. ZSTD is one of only a couple non-rust dependencies of this project, which is usually where build issues show up. I put some time on my calendar after work to look into that. |
Hi, @ellenhp, any news on the documentation and how to get started? I'm really looking forward to this. Cheers. |
Hi, it makes me really happy to hear people are enthusiastic about this, but I just want to be transparent that I'm pretty low on capacity lately. Another project has captured my interest but as that excitement wanes I'll try to come back to this! I'm pretty happy with where Airmail is at as a project. It's usable for a lot of use-cases and way easier on the wallet than anything else out there, even if you run it against a local index for <100ms response times. It definitely needs some docs though. |
I've made a little progress as of last night. I wrote up all the prerequisites and now I need to do the indexing commands and the serving commands, then go through the docs myself and do a build, then push and PR. Also need to look into what's going on with the zstd dependency. I think it only gets pulled in because of tantivy, so you might be able to find a solution by looking through the tantivy github issues in the meantime. |
I was able to get it to build in docker, but I needed to tweak the
After looking at all of the options out there, this project is really making me excited! We have need for an offline geocoder to run along side an full offline version of our entire platform. So having something that is compact like this perfectly fits our use case! Happy to provide feedback and help debug 🎉 |
@ellenhp, sorry for pinging again, I just really wanted to see the progress, maybe we could get at least what is done? I believe this basic 'how to get started' shouldn't be that extensive? Just a minimal how to launch is fine, with time and others help this can be improved. Thanks and sorry again for pinging, just really excited for this. |
Let me know if you have any issues with the docs. For Ahmed specifically, I think I fixed the zstd issue you had by bumping the version of tantivy we're using. If you have any more problems like that feel free to open an issue. If any of those issues come up in the future you should be able to use the Option 2 I listed in the build docs to hopefully bypass them. |
@mckelveygreg Let me know if you have any feedback as well. You mentioned you got it building but this should give you everything you need to create an index and serve requests. Hopefully. :) |
Thank you @ellenhp, i quickly tested and not seeing the errors i was facing before |
Thank you for this project. It's remarkable for its affordability.
Could you please provide documentation on how to install locally or on Fly.io, which you mentioned in your blog post
The text was updated successfully, but these errors were encountered: