developmentseed / eoAPI

[Active Development] Earth Observation API (Metadata, Raster and Vector services)
https://eoapi.dev
MIT License
197 stars 21 forks source link

add mosaic search + other updates #36

Closed vincentsarago closed 2 years ago

vincentsarago commented 2 years ago

This PR adds a mosaic/list endpoint to list/search mosaic registered in PgSTAC database

ref:

Other fix/additions

breaking changes

bitner commented 2 years ago

not at my computer, there are some other small not critical changes in your query generation, but the big one is don't use "metadata::json" in your filters. the metadata column is jsonB and casting it to json will cause it to need to do the cast for every record and will prevent your index from getting used

On Fri, Mar 4, 2022, 5:36 AM Vincent Sarago @.***> wrote:

@.**** commented on this pull request.

In src/eoapi/raster/eoapi/raster/factory.py https://github.com/developmentseed/eoAPI/pull/36#discussion_r819497047:

  • ),
  • ),
  • model.Link(
  • rel="tilejson",
  • href=self.url_for(
  • request, "tilejson", searchid=search.id
  • ),
  • ),
  • ],
  • )
  • for search in searches_info
  • ],
  • links=links,
  • numberMatched=int(nb_items),
  • numberReturned=len(searches_info),
  • )

@bitner https://github.com/bitner can you check that I'm not doing any stupid SQL (or python, or general) stuff.

— Reply to this email directly, view it on GitHub https://github.com/developmentseed/eoAPI/pull/36#pullrequestreview-900121638, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABIHXDKJ2ZFFNRSFNTLJXTU6HYUVANCNFSM5P5JKURA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>