-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
Fixedwing: implement TAS output scaling for FF #11975
Comments
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Agree, it makes sense to use TAS here. Although it would be nice to have better ways to estimate outside air temperature (OAT) in PX4. For example, our airspeed sensor heats up significantly in the cavity it is mounted in causing PX4 TAS estimate to be wrong. I've seen this handled in other systems by allowing a OAT offset parameter or also allowing different sources of OAT measurement, not just from the Diff Press sensor. |
This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions. |
@Antiheavy Are you sure that the temperature is affecting your TAS measurement? Last time I checked the code, it seemed that PX4 just assumes the OAT is 20C and doesn't trust the temperature at all. I agree with the need to compute TAS more accurately, but I don't think I've seen fluctuations in TAS even though the temperature measurements were wildly wrong. |
This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions. |
In the devguide (here) I demonstrated that the FF part of the rate controller has to be scaled linearly with true airspeed (TAS). At the moment, both PI quadratic scaling and FF linear scaling are done with indicated airspeed (IAS):
https://github.com/PX4/ecl/blob/c4492b17c1d7bb9b74011efa9890452b091421e8/attitude_fw/ecl_roll_controller.cpp#L111-L113
Since IAS depends on air density, this makes a real difference when flying at high altitudes, cold or warm places.
Note: I open this issue to remember that I still have to do that.
The text was updated successfully, but these errors were encountered: