search

argoverse / argoverse-api

Official GitHub repository for Argoverse dataset
https://www.argoverse.org
Other
833 stars 239 forks source link

better use np.arctan2 instead of np.arctan #265

Closed jr0jr closed 2 years ago

jr0jr commented 2 years ago

In file centerline_utils.py, line 73... in case infinity coming up

johnwlambert commented 2 years ago

Hi @jr0jr, thanks for the feedback. Feel free to add a PR with your recommended change. If you could add a tiny reproducible example via a unit test that fails before the fix, and succeeds after, that would be great too.

benjaminrwilson commented 2 years ago

Closed due to inactivity. @jr0jr, feel free to submit a PR as @johnwlambert suggested.