Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Release 0.10.0: migration to rio-tiler #183

Merged
merged 38 commits into from
Jan 21, 2024
Merged

Conversation

banesullivan
Copy link
Owner

@banesullivan banesullivan commented Jan 21, 2024

This is a rather massive reimplementation of localtileserver that transitions to using rio-tiler entirely for all tiling needs and drops all usage of large-image.

Been working on this for a long time...

This release unfortunately has many breaking changes and some loss of features. The breaking changes are mostly around API renaming and some methods changing to properties. Otherwise, I stripped out features that are just better handled upstream or not needed here.

cc @giswqs, this may be a painful upgrade considering how much has changed. Highlights:

  • band is now indexes to follow rio-tiler's implementation
  • cmap is now colormap
  • palette is now colormap
  • Custom colormaps are no longer supported
  • No more region of interest extraction
  • ...

@codecov-commenter
Copy link

codecov-commenter commented Jan 21, 2024

Codecov Report

Attention: 72 lines in your changes are missing coverage. Please review.

Comparison is base (41aed45) 83.14% compared to head (1b56028) 83.36%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #183      +/-   ##
==========================================
+ Coverage   83.14%   83.36%   +0.22%     
==========================================
  Files          26       26              
  Lines        1424      992     -432     
==========================================
- Hits         1184      827     -357     
+ Misses        240      165      -75     

@banesullivan banesullivan marked this pull request as ready for review January 21, 2024 23:53
@banesullivan
Copy link
Owner Author

@giswqs, I'm going to merge this and issue pre-releases for testing. Do you have the bandwidth to validate this new version against leafmap?

@banesullivan banesullivan merged commit fe7c7cf into main Jan 21, 2024
8 checks passed
@banesullivan banesullivan deleted the refactor/rio-tiler branch January 21, 2024 23:54
@banesullivan
Copy link
Owner Author

Installation can be done with the --pre option for pip:

pip install --pre --no-cache localtileserver

@giswqs
Copy link
Contributor

giswqs commented Jan 22, 2024

@banesullivan Excellent work! Thank you very much for your work on this. I will test it and update leafmap accordingly later this week. The new semester starts tomorrow, and I won't be able to follow up on this until later this week.

@banesullivan
Copy link
Owner Author

banesullivan commented Jan 22, 2024

No worries! I'm only issuing a pre-release and want to make sure that downstream packages like leafmap are still going to be well supported here. No rush but when you get around to it, don't hesitate to report what's broken

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants