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

Feedback on new Toolbar Dropdown expanded pages #12405

Open
DonLakeFlyer opened this issue Feb 6, 2025 · 14 comments
Open

Feedback on new Toolbar Dropdown expanded pages #12405

DonLakeFlyer opened this issue Feb 6, 2025 · 14 comments

Comments

@DonLakeFlyer
Copy link
Contributor

DonLakeFlyer commented Feb 6, 2025

A significant usability change in the 5.0 version QGC are the new toolbar dropdowns. They now have a section which can be expanded which can contain both application and vehicle settings associated with the indicator. The goal of this is to provide the user with access to settings which are of use to be changed possibly from one flight to the next or one mission to the next. As opposed to things like the full "Vehicle Setup" pages which mostly contains things which are of use when designing/creating a new vehicle. The expanded page also provides a link to the full "old school" vehicle setup page if needed.

What used to be called Vehicle Setup is now called "Vehicle Configuration". These are still the same as they were before but are not mainly meant to be used during vehicle build/design. And you can see from the example below the link back into the Power pages from Battery is done through the "Configure" button.

Also in this example the center "Low Battery Failsafe" section is firmware specific whereas the rest of the page is firmware agnostic.

Here is an example of the PX4 Battery Indicator with the pane expanded:

Image

In it you can see the "Value - Percentage" setting as well as the battery coloring which are Application Settings. And below it you can see failsafe settings which are Vehicle Settings. To a vehicle user they don't care about the differentiation it just works.

And here is the same indicator for ArduPilot firmwares:
Image

Keep in mind that altough the example above is battery all of the toolbar dropdowns have new expanded pages. I'm looking for feedback on all of them.

@DonLakeFlyer DonLakeFlyer changed the title Feedback on new Toolbar Dropdown expanded pages Draft: Feedback on new Toolbar Dropdown expanded pages Feb 6, 2025
@DonLakeFlyer
Copy link
Contributor Author

So the feedback I am looking for is what is missing from these expanded pages. As well as what is there that shouldn't. And most importantly why?

@DroneWuKong
Copy link

DroneWuKong commented Feb 6, 2025

Hello,

Taking a quick look at it, It would be nice to be able to display average single cell voltage as regardless of battery pack total voltage individual cell voltage stays the same. In addition to being able to display current draw.

With the ability to actively monitor current draw would allow a operator to have an understanding of how hard the motors/batteries are working in regards to load. Under varying weather conditions amp draw can be used to estimate if flight times are shortened or extended at a quick glance, in addition to a better ability to fine tune cruising speed in relation power draw to actively manage flight efficiency.

@AndKe
Copy link
Contributor

AndKe commented Feb 12, 2025

I like it, very nice and useful.
as @DroneWuKong mentions , cell voltage is a kind of "universal language" for those who switch between 4s,5s,6s,12s UAV's
Also seeing ThO plot (throttle-output) is a good visualisation of the actual power.
in planes ThO near 100% in cruise can tell you that the target groundspeed is too high for certain winds, or that the battery voltage is soon to low to maintain altitude, an elevated ThO can indicate icing too. In general, plotting/seeing ThO is a useful tool in terms of power.

@DonLakeFlyer
Copy link
Contributor Author

cell voltage is a kind of "universal language" for those who switch between

Yeah, that is very doable. I'll add it.

Where does throttle output come from mavlink. It's not available to be added to the instrument panel?

@DonLakeFlyer
Copy link
Contributor Author

Also keep in mind that all of the toolbar dropdowns have new expanded pages. Not just the battery example I posted above.

@DonLakeFlyer DonLakeFlyer changed the title Draft: Feedback on new Toolbar Dropdown expanded pages Feedback on new Toolbar Dropdown expanded pages Feb 12, 2025
@AndKe
Copy link
Contributor

AndKe commented Feb 12, 2025

Sorry the ThO is of course from datalogs.
Throttle demand in telemetry is logged in VRR_HUD.throttle for copters and planes (except plane in VTOL mode.)

@AndKe
Copy link
Contributor

AndKe commented Feb 12, 2025

As for the other expandable panes: I love them - especially the failsafe options.
Also: on Linux I see font size/alignment issue: (this issue is nowhere else in the app as I can see)

Image

Please consider making the "expansion arrow" more prominent - this deserves to be more accessible/easier to discover.

Image

@DonLakeFlyer
Copy link
Contributor Author

Please consider making the "expansion arrow" more prominent - this deserves to be more accessible/easier to discover.

Yeah, been meaning to do this forever. Royal PITA to hit it on my Herelink!

@DonLakeFlyer
Copy link
Contributor Author

On the cell voltage thing this is easily doable on PX4 since there is a standard vehicle param for cell count which is configured in Power setup. But on ArduPilot I can't seem to find the same sort of param to use.

@AndKe
Copy link
Contributor

AndKe commented Feb 13, 2025

Yes, Ardupilot does not show cell count, and unless it senses the cell voltages, there is no proper way to know.
I had another telemetry-based project that always got the detection right it worked like this:
on startup, if voltage is:
10 -12.7v = 3S
12.7 - 17v =4S
17--21.8v = 5s
21.8-25.5 =6s
38.4-51.0=12s

basically assuming that about 3.2v ...4.35v is the working rage of each cell. I did not care much for LIHV, sp the numbers may need to be adjusted a little for that.

@AndKe
Copy link
Contributor

AndKe commented Feb 13, 2025

BTW: Ninja builds work fine, but are slow, using only a few cores for most of the job.

Image

@DonLakeFlyer
Copy link
Contributor Author

BTW: Ninja builds work fine, but are slow, using only a few cores for most of the job.

@HTRamsey Normal or not?

@HTRamsey
Copy link
Collaborator

Yeah don't know what would cause that unless it's excessive linking or something. The linux builds on CI are slow because they have a very low cache hit rate, but the same build has very high cache hit rates on my desktop. Haven't figured that one out yet.

@DonLakeFlyer
Copy link
Contributor Author

I notice the same thing on my Mac. The CPUs are pegged only a minority of the time.

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

No branches or pull requests

4 participants