mapillary / mapillary-python-sdk

A Python 3 library built on the Mapillary API v4 to facilitate retrieving and working with Mapillary data.
MIT License
41 stars 16 forks source link

[Requirements] 8. Get All Map Features In A BBox #19

Closed Rubix982 closed 3 years ago

Rubix982 commented 3 years ago

Is your feature request related to a problem? Please describe. This issue deals with the 8th requirement from the PRD extracting all map features within a BBox

Describe the solution you'd like The base requirements are,

  1. Bbox coordinates as an argument
  2. Layer as argument (points or traffic signs)
  3. Filter argument 1: Value list as argument (only one value or multiple values or “all”)
  4. Filter argument 2: other -- date first seen, last seen, more?

Describe alternatives you've considered NA

Additional context Maybe split this into 2 functions, one for traffic signs and one for points

Output is a GeoJSON object

Rubix982 commented 3 years ago

@OmarMuhammedAli aren't first_seen and last_seen properties instead of fields?

OmarMuhammedAli commented 3 years ago

@Rubix982 Yes, they're properties. We generally have no fields associated with requests for tiles endpoints in general I think.

Rubix982 commented 3 years ago

So the filtering will be done after the data has been retrieved as a GeoJSON.

What,

Maybe I can create issue(s) for these and work on that.

OmarMuhammedAli commented 3 years ago

I will keep replying to this comment If I think of anything else