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 config error with 14.4V NiCad #65

Open
hamishcunningham opened this issue Oct 16, 2015 · 0 comments
Open

battery config error with 14.4V NiCad #65

hamishcunningham opened this issue Oct 16, 2015 · 0 comments

Comments

@hamishcunningham
Copy link
Owner

Philip Ingram says:

I'm trying to use my kickstarter Pi on a version 2 old Pi B (not a Pi 2B). After trying it as delivered with 8*AA cells [which is fine] I'm now trying to run it from a 14.4V NiCd drill battery. I run sudo mopi, specify 12 cells NiCd and get the response: "The good charge level (14.45V) is greater than the full level (14.4V). Please try again."
I guess I can fix this by using the low level commands but it looks as though a minor correction is needed to the mopi software. Clearly 14.4 NiCd has not proved a popular power source 8-}


The problem is a logical error in the simbamon.default code:
the final two elements of

BAT_CUTOFF_A=( # n=====Vhigh===Vlow 1 9.0 $OPERATING_FLOOR 2 1.4 1.0 3 1.6 0.8 4 2.0 1.58 5 3.7 3.0 6 4.2 3.0 7 1.2 1.1 )

and

BAT_LOW=$( echo "${BAT_FLOOR}*1.1" |bc )

conflict as

1.1(volts) * 1.1(magic number) is 1.21

hence exceeding the high point. Ouch!

The assumption of the same magic number for all classes of cell is clearly inappropriate. Clearly you have no other customers using NiCd so it's not a serious support issue but it's rather unsatisfying.

I suggest that the simplest and clearest resolution would be to make BAT_CUTOFF_A a three column array from which BAT_LOW is read directly rather than calculated.

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

1 participant