Feature/add point endpoint #150
Conversation
| "values": [ | ||
| (src, pts.data.tolist(), pts.band_names) for src, pts in values | ||
| ], | ||
| } |
There was a problem hiding this comment.
This is now equivalent to https://github.com/developmentseed/titiler/blob/main/src/titiler/mosaic/titiler/mosaic/factory.py#L672-L677 and remove the need of custom model
|
|
||
| out: Dict[str, Any] = {} | ||
| for val, asset in filter_tasks(tasks, allowed_exceptions=allowed_exceptions): | ||
| item_id = f"{asset['collection']}/{asset['id']}" |
There was a problem hiding this comment.
collection and id are mandatory
There was a problem hiding this comment.
@vincentsarago that looks like a cleaner version of what I attempted in #146 - thank you for fixing it up!
|
@hrodmn I'm having second though on the endpoint path I know we already have would you be ok if we change this to |
That's fine with me. When I started working on it I went with |
continues #146
The mai change is that I'm introducing
multi_points_pgstacwhich replace rio-tiler'smulti_valuesmethod (because we use item dict instead of url, so it's not hashable as dict key).In
multi_points_pgstacI'm constructing an Item ID using the id and collection which we says aremandatory(this is not asserted but should)@hrodmn could you have a look and tell me your tough