jrossignol / WaypointManager

MIT License
38 stars 14 forks source link

Show altitude in Waypoint Manager #49

Open Svish opened 8 years ago

Svish commented 8 years ago

I find using the Waypoint Manager super helpful, especially when trying to do science contracts and such where I'm supposed to do a reading in a certain location.

These contracts are usually also linked to an altitude requirement. Think there are 3 types:

Would it be possible to display the altitude of the waypoint in the Waypoint Manager? And also, if possible, below or above? Could be displayed like this:

Would like it both in the Manager UI, and in the optionally below the altimeter for a selected waypoint.

In the Manager UI, I'd actually prefer to have the Altitude displayed rather than Lat and Long, since those values are kind of cryptic to me anyways :stuck_out_tongue: Distance and altitude would be much more useful for me to see in that list :smile:

jrossignol commented 8 years ago

This is good feedback, I'll look at adding in altitude. Note that I can't easily get the "above/below" part, because that's part of the contract definition, not the waypoint definition.

Svish commented 8 years ago

Cool!

Yeah, I kind of suspected the above/below part being more tricky. Not as important though, as it's easier to remember if it was above or below, than the exact altitude. Especially since they are often the same for a contract :smile:

Svish commented 8 years ago

Out of curiosity, could the ContractParser mod be used to extract the landed/above/below part from the contract? Either as a dependency, or just as a code example on how to extract the info you need...

I see it has a checkForWaypoint method in there, maybe the information is in that or somewhere around it in a parameter or something...?