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

Add map to input page #7

Open
julled opened this issue Dec 5, 2022 · 5 comments
Open

Add map to input page #7

julled opened this issue Dec 5, 2022 · 5 comments
Labels
enhancement New feature or request Future Feature Idea for future features prio:low question Further information is requested

Comments

@julled
Copy link
Collaborator

julled commented Dec 5, 2022

Map allows visualization of the input data ( specially the position).

Map should also allow setting a position by mouse.

@booo
Copy link
Collaborator

booo commented Dec 5, 2022

Hacked together a little map view with the possiblity to add the lat/lon by clicking.

Screenshot 2022-12-05 at 22-12-19 Leeway Simulation

It would be nice if the simulation would return a GeoJSON Feature Collection with the results for presentation. Unfortunately I could only find a ascii representation :)

@booo
Copy link
Collaborator

booo commented Dec 5, 2022

The diff: main...booo:leeway:main

@svenseeberg
Copy link
Member

svenseeberg commented Dec 6, 2022

First, thanks for your work. I thought about this feature for some time. I think we should not include a map into this tool for a couple of reasons:

  • This tool is intended for low bandwidth usage. The form should use as little data as possible.
  • Organizations tasked with Search and Rescue operations will be able to work with raw coordinates. We're not talking about the average Google Maps users here. They will have maps available somewhere else (on paper, digital, etc) and will be able to extract and input coordinates.
  • I think it would be more relevant to support the different notations for longitude and latitude (arc minutes and seconds instead of decimal system).

@julled
Copy link
Collaborator Author

julled commented Dec 6, 2022

Couldnt we make it optional and in the default case disabled?

I think there are usecases where it is useful.

Land crews are not limited by bandwidth.

also a map could be used by the user to see wether the input is plausible before starting up the simulation.

@svenseeberg
Copy link
Member

Couldnt we make it optional and in the default case disabled?

Agreed. If it is opt in, then yes.

@svenseeberg svenseeberg added the Future Feature Idea for future features label Dec 7, 2022
@svenseeberg svenseeberg added enhancement New feature or request question Further information is requested labels Dec 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Future Feature Idea for future features prio:low question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants