mavlink / qgroundcontrol

Cross-platform ground control station for drones (Android, iOS, Mac OS, Linux, Windows)
http://qgroundcontrol.io
3.18k stars 3.53k forks source link

Change compass heading display to 3 digits #4691

Closed marinesimulation closed 7 years ago

marinesimulation commented 7 years ago

The heading indicator numbers from 1-99 degrees are in a two digit format, can we chance these to three digits where we have a zero in front? ("045") This would bring it inline with aircraft and vessel displays.

qgc-heading

Antiheavy commented 7 years ago

I will second this request!

DonLakeFlyer commented 7 years ago

Is such a big compass display critical? We were thinking of moving to something more like this: screen shot 2017-03-02 at 3 08 45 pm Which has the compass and hud combined.

jaxxzer commented 7 years ago

I am in favor of reducing the overall size of the display.

DonLakeFlyer commented 7 years ago

Not that specific image necessarily. Just the concept of compass around attitude.

KevinKW87 commented 7 years ago

Combining the compass and the attitude I think would work out well. If that's the case, can we still get a three digit compass reading in the Widget Value area? It makes navigation easier for me when I do quick glances at the widget values. I have a rough idea what my heading is just by glancing at the first digit (0-1-2-3). qgc heading

Antiheavy commented 7 years ago

I like the idea of combined compass and horizon as long as the pitch/roll and heading are clearly visible from a couple/few meters away from the screen. We don't have to read the numbers from that far, but need to see the orientation by tilting and turning your aircraft during pre-flight checks. For UAVs I like circular compasses compared tape style.

I also support the original request to display heading as a 3 digit number where shown.

DonLakeFlyer commented 7 years ago

If that's the case, can we still get a three digit compass reading in the Widget Value area?

Unfortunately that would be way more difficult since that is all generic code which at the point it's being displayed doesn't know it's a heading. It would require formatting meta data support in the internal FactSystem. Doable but non-trivial.

DonLakeFlyer commented 7 years ago

Maybe if the compass goes around this:

screen shot 2017-03-02 at 8 17 58 pm

We could put three digit heading in big text towards the bottom center part of the attitude indicator.

DonLakeFlyer commented 7 years ago

Quick Qml hacking:

screen shot 2017-03-02 at 8 48 49 pm
jaxxzer commented 7 years ago

@DonLakeFlyer In the intermediary term (at least), can we make this new style an option?

DonLakeFlyer commented 7 years ago

I tend to think options are evil. 5% of the user base ever finds anything beyond the default setting.

DonLakeFlyer commented 7 years ago

An image of it working for real:

screen shot 2017-03-02 at 9 03 48 pm
jaxxzer commented 7 years ago

@rjehangir Has reservations about this. Almost all of Sub navigation is done by heading, and it may be unclear/less wieldy for operators. Even if the option is not exposed, something that we could default to with firmware?

DonLakeFlyer commented 7 years ago

That's a good argument.

You can't default with firmware. But you can default with your own custom version of QGC using your own core plugin. This allows you to change the default on settings as well as hide settings among a whole pile of other things which allow you to create a third party branded version of QGC targeted to a specific vehicle.

DonLakeFlyer commented 7 years ago

With your own QGCCorePlugin you can even go so far as replace the instrument panel entirely with something custom created by you.

DonLakeFlyer commented 7 years ago

Ok, one more. You can even change the vehicle icon throughout QGC from the core plugin to be a sub looking thingy instead of a plane looking thingy.

DonLakeFlyer commented 7 years ago

Here is a full screen shot. It leaves a lot of room for lots of values to show in the page control below. Implementing as a second variant. So old one is still usable if you want.

screen shot 2017-03-02 at 9 35 02 pm
marinesimulation commented 7 years ago

Here's one vote against combining the attitude and compass displays. When operating an ROV we really don't use the attitude display all that much. In fact, the BlueROV2 is the only mini sized ROV that includes an attitude display. It may help in some rare situations but our primary focus, especially when operating in poor visibility, is the compass heading. Unlike aerial drones, ROVs frequently operate in water that may have almost zero visibility. Having a large, easily read compass display is critical for this type of operation. The current compass heading display is just about perfect. It's a good size and it's easily read. I think removing it and combining it with another, less used display would be a big mistake and would make QGC much less functional for ROV operators.

DonLakeFlyer commented 7 years ago

I'm putting it on an option. But I'm defaulting to combined. We'll see how it goes. Easy to back off after it's through.

DonLakeFlyer commented 7 years ago

Fixed