Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

new world map visualiser #9

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Nkluge-correa
Copy link

I replaced the world map plot made using matplotlib with a geoplot using plotly. The end result is an image (which I believe to be...) that is visually more satisfactory. The image is saved as an html plot, and can be viewed directly in the browser.

@richardclegg
Copy link
Owner

Sorry for the huge amount of time it took me to look at this -- teaching is such a nightmare. The map looks nicer but for me the request gives inaccurate results.
Eg try:
./vis_route.py 210.152.243.234
On both versions

At least for me (from UK) in the new version this is broken - it sends two straight lines from the UK to NA then another back to Japan. In the older version these are curved lines (great circles) from UK to NA then on to Japan. So the big issues (for me) are

  1. Lines are straight, should be great circle curves
  2. Lines do not "loop" if the path would circle the globe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants