search

gchoumos / mapillary

Tools to interact with the Mapillary API (v4)
MIT License
8 stars 0 forks source link

missing "z" declaration at "get_sequences.py" #2

Open kauevestena opened 2 years ago

kauevestena commented 2 years ago

When one tries to call it at line 106, raises the error:

NameError: name 'z' is not defined,

Thank you for the repo!!

Best Regards, Kauê

gchoumos commented 2 years ago

Hello Kauê!

Thanks for sharing this issue. Sorry for this error, it looks like some changes I have done locally were not pushed to the actual repo. I have just pushed a couple of new commits. Can you try pulling the changes and try running the get_sequences.py script again?

I hope it will resolve your issue. In case you encounter more, let me know! Assigning to you for now to make sure you will get a notification!

Cheers, George

@kauevestena

gchoumos commented 2 years ago

Just an FYI, the execution of get_sequences.py for my use case takes some time (for me it was around 16 minutes). So you may have to be a bit patient with that particular script.

I believe that there is some kind of issue or just room for improvement with the filtering request, and I plan to investigate on it in the near future to see how it can run faster by reducing the number of requests made to the graph API.