-
Notifications
You must be signed in to change notification settings - Fork 432
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
Use libdeflate instead of zlib #696
base: master
Are you sure you want to change the base?
Conversation
Thanks for the contribution. As you can see, the build is breaking due to the missing dependency, which will need to be installed in |
…sored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
… | This work was sponsored by SkySight (@Plantain)
I made many commits, there is still an error during test phase, I think this is not an error of my code. |
Thanks. I'll take a look and see if I can understand what is going wrong with that tile. |
This won't build on OSX/Clang/llvm currently: |
… | This work was sponsored by SkySight (@Plantain)
Interestingly the file sizes are coming out considerably larger with libdeflate. I am investigating why that might be. |
… | This work was sponsored by SkySight (@Plantain)
Sorry for these commits, maybe this last commit will fix the compression error |
The new commits have the void pointer arithmetic issue again: |
Sorry for these commits, maybe this last commit will fix the compression error |
This is not generating valid mbtiles in the current state - tiles generated are just hundreds of kb of null bytes. |
No description provided.