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

Limit flask and bug fixes #178

Merged
merged 4 commits into from
Oct 9, 2023
Merged

Limit flask and bug fixes #178

merged 4 commits into from
Oct 9, 2023

Conversation

banesullivan
Copy link
Owner

@banesullivan banesullivan commented Oct 9, 2023

Limit's flask<3 until flask_restx's python-restx/flask-restx#566 is handled.

Resolves #177

Also:

  • Fixes an issue from large-image where the rasterio source is passing a CRS object to getMetdata() which isn't JSON serializable
  • Drops support for extracting ROIs with the REST client as this has been broken and isn't a "rest-ful" operation.
  • Drops support for histograms as it has been broken and not tested for a while

cc @dfguerrerom

@codecov-commenter
Copy link

Codecov Report

Merging #178 (1b17ede) into main (70e1a22) will increase coverage by 0.70%.
The diff coverage is 100.00%.

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

@@            Coverage Diff             @@
##             main     #178      +/-   ##
==========================================
+ Coverage   82.43%   83.14%   +0.70%     
==========================================
  Files          26       26              
  Lines        1469     1424      -45     
==========================================
- Hits         1211     1184      -27     
+ Misses        258      240      -18     

@banesullivan banesullivan changed the title Limit flask Limit flask and bug fixes Oct 9, 2023
@banesullivan banesullivan merged commit 2753554 into main Oct 9, 2023
@banesullivan banesullivan deleted the maint/limit-flask branch October 9, 2023 04:40
# 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.

localtileserver fails with flask==3
2 participants