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

display totals for mAh, KMs, mins #160

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Conversation

wowi69
Copy link

@wowi69 wowi69 commented Oct 15, 2018

added to display:
total mAh used, total distance flown (in km), total airborne time (mm:ss)

couldnt find osdconfig.txt, so these are the lines to be added:

#define TOTAL_AMPS true // set to false if you dont want to see this
#define TOTAL_AMPS_POS_X 19
#define TOTAL_AMPS_POS_Y 15
#define TOTAL_AMPS_SCALE 1
#define TOTAL_DIST true // set to false if you dont want to see this
#define TOTAL_DIST_POS_X 6
#define TOTAL_DIST_POS_Y 19
#define TOTAL_DIST_SCALE 0.8
#define TOTAL_TIME true // set to false if you dont want to see this
#define TOTAL_TIME_POS_X 19
#define TOTAL_TIME_POS_Y 19
#define TOTAL_TIME_SCALE 0.8

add to display:
total distance flown
total amps used
total flight time
add display of:
total distance, flighttime, mAh
these lines have to be added to osdconfig.txt, but i couldnt find the file here on github
@wowi69
Copy link
Author

wowi69 commented Oct 15, 2018

@bortek
Copy link
Collaborator

bortek commented Oct 15, 2018

This will have to wait until we update the repos with the files from the img file.

@Yes21
Copy link
Contributor

Yes21 commented Oct 18, 2018

This Pull Request will have to be merged with this one, before 1.7 integration.
Thanks

@Yes21 Yes21 mentioned this pull request Oct 18, 2018
@bortek
Copy link
Collaborator

bortek commented Oct 18, 2018

@RespawnDespair I merged your other PR #158 . This one now has a lot more changes. Do you think we need to merge this one cause most of the changes you are doing on your side will have the latest stuff already in it?

@Yes21
Copy link
Contributor

Yes21 commented Dec 5, 2018

@wowi69 Your PR has been transferred here. It's now ready to be merged.

@rodizio1
Copy link
Owner

rodizio1 commented Dec 5, 2018

There is one problem with this: It's not guaranteed that the OSD keeps running, whenever wbc_status is called to display a status message (for example when plugging a smartphone for USB tethering or connecting to the Wifi Hotspot) the OSD gets killed and then re-started after.

This would also reset those counters (and is the reason I haven't integrated a flight timer yet).

I haven't really thought it through yet, but I guess a better solution to the quick&dirty wbc_status would be to move this functionality into the OSD, i.e. implement some means to send messages to the OSD that the OSD then displays.

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

Successfully merging this pull request may close these issues.

4 participants