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

Battery warning comes too late (blinking in OSD) #4903

Open
FinalFrag opened this issue Jul 5, 2019 · 5 comments

Comments

Projects
None yet
2 participants
@FinalFrag
Copy link

commented Jul 5, 2019

Current Behavior

I have set up the battery warnings:

vbat_cell_detect_voltage = 450
vbat_max_cell_voltage = 420
vbat_min_cell_voltage = 330
vbat_warning_cell_voltage = 350

But I haven't set any capacity warnings:

battery_capacity = 0
battery_capacity_warning = 0
battery_capacity_critical = 0

Video: https://youtu.be/kiRcwrcCEo4

Steps to Reproduce

  1. Fly
  2. Get the blinking warning too late (at 12.0v)
  3. Almost kill your battery

Expected behavior

The battery voltage and capacity to start blinking at 14.0v (3.5v per cell)

Suggested solution(s)

Fix? :)

Additional context

Notice that I am running "bat_voltage_src = SAG_COMP"
Full diff: https://pastebin.com/MVp3R2Hi
Possibly related to #4902


  • FC Board name and vendor: Matek F405 STD
  • INAV version string:INAV/MATEKF405_SERVOS6 2.2.1 Jul 3 2019 / 22:32:52 (a6d8474)
    GCC-8.2.1 20181213 (release) [gcc-8-branch revision 267074]
@issue-label-bot

This comment has been minimized.

Copy link

commented Jul 5, 2019

Issue-Label Bot is automatically applying the label BUG to this issue, with a confidence of 0.86. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@issue-label-bot issue-label-bot bot added the BUG label Jul 5, 2019

@teckel12

This comment has been minimized.

Copy link
Contributor

commented Jul 6, 2019

@FinalFrag The cell detect voltage seems too high, probably should be 4.3? Also, you can specify the cell count to avoid any problems.

@FinalFrag

This comment has been minimized.

Copy link
Author

commented Jul 6, 2019

Since the voltage starts blinking at 12.0v and the setting is at 3.5v, that means the system detected a 12/3,5 = 3,4 cell battery?

I'll try setting the cell count manually and test again, but it seems unlikely that the system would detect non-integer cell counts. I'll also add the cell voltage osd element in the next test.

@teckel12

This comment has been minimized.

Copy link
Contributor

commented Jul 7, 2019

@FinalFrag What was the charged voltage of your battery? 16.8v or 17.6?

Setting the cell count can just avoid any strange conditions, like installing a partially charged battery and it believes you have a different cell count.

@FinalFrag

This comment has been minimized.

Copy link
Author

commented Jul 8, 2019

The battery was plugged in at 16.6v, so that is without a doubt a 4S pack. I have set the cell count to 4 in the cli now, I will retest soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.