search

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

[Tests] Added method to easily pull in data, #72 #84

Closed Rubix982 closed 3 years ago

Rubix982 commented 3 years ago

A nice to have and an enhancement (:trophy:), this PR works to close #72 and solve it by introducing a simple method of fetching data from the API without remembering the different defaults or going through the whole SDK just to retrieve a geojson.

You can test it by going into tests/helper/, and then running python client.py.

As an example, please see the below screenshot,

image

The above output is for guiding the user on how to use the written CLI program

Running the command python client.py 'MLY|XXX' fetch_map_feature_point, a pretty printed .geojson appears under data/,

image

Future contributors can now create examples they found or like by adding functions in fetch.py that has all the different functions listed out.

Signed-off-by: Saif Ul Islam saifulislam84210@gmail.com

OmarMuhammedAli commented 3 years ago

Related to testing on windows, the styling and formats don't work on windows cmd nor PowerShell. image image

Rubix982 commented 3 years ago

There's a reason I don't like Windows, @OmarMuhammedAli. It's not classy enough to handle the coolness of terminals. :(

OmarMuhammedAli commented 3 years ago

There's a reason I don't like Windows, @OmarMuhammedAli. It's not classy enough to handle the coolness of terminals. :(

Every day gets me closer to throwing Windows out of the window