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

Bug: Log Times Can Be Incorrect #87

Closed
keenanjohnson opened this issue Feb 22, 2018 · 6 comments
Closed

Bug: Log Times Can Be Incorrect #87

keenanjohnson opened this issue Feb 22, 2018 · 6 comments

Comments

@keenanjohnson
Copy link

I've noticed that on all of the flight logs I have submitted, the vehicle flight time and units of the plots seem to be incorrect.

See an example here: https://logs.px4.io/plot_app?log=7c57fa9a-0ee1-4916-80d7-fae18554a8fe

The tool says the flight time is over 3 hours.
image

This is confirmed by the timing in the plots:

image

As I was the pilot and confirmed by flightplotlib however, the flight time was actually much less than that:

image

It would appear the web tool is treating the minutes like hours.

@bkueng
Copy link
Member

bkueng commented Feb 22, 2018

The Vehicle Flight Time is the vehicle's total flight time.
The plots show the time since boot (which can be useful to know). Note that with FlightPlot you can switch the displayed time as well.
So there is no bug here, but if you have suggestions, I'm open for improvements.

@keenanjohnson
Copy link
Author

Oh I misunderstood what the total time label was implying.

Is that total flight time for the life of the vehicle or between firmware updates?

Perhaps we could improve the title slightly?

@bkueng
Copy link
Member

bkueng commented Feb 22, 2018

Is that total flight time for the life of the vehicle or between firmware updates?

It's for the life of the vehicle. Though you can reset it manually if you really wanted to, it's a parameter.

Perhaps we could improve the title slightly?

Sure, what do you suggest? It should be something short though. Maybe with a tooltip if needed.

@keenanjohnson
Copy link
Author

Maybe it could be "Vehicle Life Flight Time"?

@bkueng
Copy link
Member

bkueng commented Feb 23, 2018

Ok, updated in 6cc3d88

@bkueng bkueng closed this as completed Feb 23, 2018
@keenanjohnson
Copy link
Author

Thanks!

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

No branches or pull requests

2 participants