it0uchpods wrote in Fri Nov 23, 2018 4:24 am:Ja, please IGNORE everything i wrote before about my changes to the AP here... When I look back at what I did, I am thinking, I am not known what I am doing! Sorry for the confusion. :D
Flying wise, it is a huge improvement already, well done. I did fix a few issues, duplicating and therefore conflicting (causing oscillation) controllers and some oscillation, do you mind if I submit a pull request? (to tdammers)
I also made the Yaw Damper work, made the elevator controlled by autopilot (correctly thru FCS) added AP autotrim, and AP yoke backdrive.
I also fixed incorrect specification of the min/max, should be u_min and u_max for PID, and inside <config>
Kind Regards,
Josh
<Kp>
<expression>
<table>
<property>velocities/airspeed-kt</property>
<entry><ind>140</ind><dep>0.0029</dep></entry>
<entry><ind>330</ind><dep>0.0012</dep></entry>
</table>
</expression>
</Kp>
<Ti>2.5</Ti>
<Td>0.0001</Td>
it0uchpods wrote in Mon Nov 26, 2018 12:13 am:Hi tdammers,
make sure you put the new commit in master also (merge nl256 or cherry pick, or such, both would work) ;)
Do you want any further things to be done in master instead?
Kind Regards,
Josh
Users browsing this forum: No registered users and 9 guests