Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Update power docs with info on how to calibrate #120

Merged
merged 3 commits into from
Dec 12, 2017
Merged

Update power docs with info on how to calibrate #120

merged 3 commits into from
Dec 12, 2017

Conversation

hamishwillee
Copy link
Collaborator

This is supposed to address #119.

@DonLakeFlyer @sanderux I have not actually done battery setup before so this needs a review. As I understand it (and what this says) is that you enter battery cells, and high/low voltage values and the other values are auto-generated. However you can do further calibration by measuring voltage/current and entering these values to recalculate the voltage divider and volts/amp.

Is this correct?

@sanderux
Copy link

The other values are not auto generated. You always need to do a calibration of both the voltage and the current divider. Sometimes these are provided with the power module or vehicle.

@hamishwillee
Copy link
Collaborator Author

Sometimes these are provided with the power module or vehicle.

Thanks @sanderux -what do you mean "provided by the vehicle"? Do you mean that the power module will supply all the values?

@sanderux
Copy link

I mean the supplier sometimes provides the numbers. Or if you buy a vehicle that is running something like APM you could get the values from that system.

@hamishwillee
Copy link
Collaborator Author

Thanks. I've updated accordingly. It does not talk about the ardupilot "automated" update of values, but could do so if @DonLakeFlyer wants to comment on how "exactly" that works.

OK for this to be merged?


1. Measure the voltage from the battery using a multimeter.
1. Click **Calibrate** next to the *Voltage divider* field. On the prompt that appears:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's Calculate not Calibrate. Same below

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Fixed

@DonLakeFlyer
Copy link
Collaborator

Or if you buy a vehicle that is running something like APM you could get the values from that system.

Can someone point me to how that works? Wondering if there is some missing QGC feature there.

@sanderux
Copy link

Suppose you buy a Solo, it comes with APM and already has the calibration values. no need to get fancy current measurement tools to calibrate if you want to switch to px4, just copy the dividers.

@DonLakeFlyer
Copy link
Collaborator

Oh. I thought you were saying that there was some ArduPilot coded feature which you could call to automatically set these or something.

@sanderux
Copy link

No, but i do have a much better way to calibrate the current sensor.
It uses 2 readings, a low and a high current reading.
Based on that it can determine not only the scaling but also the offset.

cscalib.py.zip

@hamishwillee
Copy link
Collaborator Author

Oh. I thought you were saying that there was some ArduPilot coded feature which you could call to automatically set these or something.

Me too!

No, but i do have a much better way to calibrate the current sensor. It uses 2 readings, a low and a high current reading. Based on that it can determine not only the scaling but also the offset.

I looked at that and it made sense. Worth getting into QGC @DonLakeFlyer

@hamishwillee
Copy link
Collaborator Author

@DonLakeFlyer I fixed that calibrate/calculate typo. Can this be merged now?

@DonLakeFlyer
Copy link
Collaborator

yup

@hamishwillee
Copy link
Collaborator Author

Thanks, merged.
FYI, I also created mavlink/qgroundcontrol#5877 to track @sanderux current sensor calibration approach.

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

Successfully merging this pull request may close these issues.

3 participants