-
Notifications
You must be signed in to change notification settings - Fork 1.8k
CA_ROTOR{i}_CT Documentation is unhelpful and maybe incorrect #3593
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
Comments
@udalmia94 The docs are generated from the source here: https://github.com/PX4/PX4-Autopilot/blob/main/src/modules/control_allocator/module.yaml#L200-L202 The result is a normalized value between 0 and 100. Suggest you post an issue in that repo, since that is where code issues go and cross link to here. Even better, perhaps a PR for a proposal for a fix there. If you cross link here I will see if I can get some eyes on it. |
Cross post Once I get a proper answer to what exactly the parameter should be, I will be happy to make a PR. Thank you. |
Sorry accidentally marked as closed. @hamishwillee can you please re-open? |
In the parameter reference the documentation for the
CA_ROTOR{i}_CT
orCA_ROTOR0_CT
,CA_ROTOR1_CT
etc. is as follows:This is unhelpful for the following reasons:
N/"PWM"^2
.u
should be - I think I understand, but an example would go a long way here to clarify.Also, I found evidence in this PX4 issue report and in the Advanced Tuning Guide that this description is incorrect.
Would be super helpful if someone in the community could clarify how to correctly calculate the
CA_ROTOR{i}_CT
parameter and update the documentation at the same time.The text was updated successfully, but these errors were encountered: