Skip to content

Decoupling the roll and yaw axis by modifying the mixer#41

Draft
Manuraj-PM wants to merge 55 commits intoairbound-developfrom
pr-add_non_linear_mixer_v1
Draft

Decoupling the roll and yaw axis by modifying the mixer#41
Manuraj-PM wants to merge 55 commits intoairbound-developfrom
pr-add_non_linear_mixer_v1

Conversation

@Manuraj-PM
Copy link
Copy Markdown

Please find the notion page here for the non linear mixer implementation:
https://www.notion.so/airbound/Mixer-for-the-tailsitter-2ea21adf4be980b4bb11f953aeb3d857?source=copy_link

…scaling) (#28)

* Initial setup for the vertical thrust maping to motor frame is done. Phi and drone's attitude based calculation

* Included loging and the implementation of throttle boosting based on TRT's orientation and TV orientation.

* Get the Average RPM data to the Tailsitter module.

* Did pitch scaling but commented due to throttle scaling test.

* Implementation for real flight is completed.

* RPM based implementation for Realflight..

* Made changes for the real flight and simulation tests..

* Changed the dumstack pop up

* Minor bug fix and log improvement

* I was very confident of my code but one comma made an issue comma issue fixed

* Some update in the estimator to fix the Kalman estimation

* Exposed Hover RPM as a parameter

* Some unit changes

* Attitude_Control: Updated the compensation gain

* Tailsitter:Did the clean SITL DEBUG

* Tailsitter:Commented SITL Debug

* AttitudeControl:Modified the boost factor

* ESCTelem_SIM:Removed Debugg comment

* Tailsitter:Correct spelling mistake and Do time based msg print

* Taisitter:Correct the coefficients

* AttitudeControl:Remove dead code

* Tailsitter:Change the RPM Debug msg

* Tailsitter:Change the RPM Debug msg add index

* Tailsitter:Add bias state in Kalman to remove the bias in the estimation

* tailsitter:Apply existing throttle scaling in absence of RPM data

* tailsitter:Apply existing throttle scaling in absence of RPM data-1

* TailSitter:Disable the throttle boost during transition

* tailsitter:Change the lower value of RPM to 2500 to remove servo jitter during arming

* Pixhaw6C-bdshot default_param: Change the rate parameters to quick tune value

* Version:Update version change

* Remove whitespace for esc telem get_average_motor_rpm method

* Add hal check for esc telem

* Add #if guards for esc telem in tailsitter.cpp

---------

Co-authored-by: Manuraj P M <manurajp@airbound.co>
Co-authored-by: Mayank Joneja <mayankj@airbound.co>
@Manuraj-PM Manuraj-PM requested a review from botmayank February 9, 2026 09:28
@Manuraj-PM Manuraj-PM self-assigned this Feb 9, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 9, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b648cbfc-4cf0-4185-949a-bd3fa6c1d45a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch pr-add_non_linear_mixer_v1

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Manuraj-PM and others added 4 commits February 11, 2026 17:10
…atti decouple control. (#42)

* Deafult_params: Change the pitch rate tuning parameters to new quick tune values

* Default_params:Change the vertical tuning parameters

---------

Co-authored-by: Manuraj P M <manurajp@airbound.co>
* Added custom storage feature

* Removed dead code, added definition for max parameter name size.

* Use MAVLink generated constant for parameter size
- Replace hardcoded MAX_AB_PARAM_SIZE value with MAVLINK_MSG_AIRBOUND_PARAMETER_GETSET_FIELD_DATA_VALUE_LEN from the generated header to ensure consistency with the MAVLink message definition

* AP_CustomMavlinkHandler.cpp: modified AIRBOUND_PARAMETER_STATUS message argument to reflect the result when airbound parameter get/set action fails

* Fixed styling in AP_CustomMavlinkHandler
Replaced print statement with GCS statements

* Removed gcs print from custom_storage init function.

* Fixed formatting

* Fixed formatting

---------

Co-authored-by: Atharva Sawant <atharvas@airbound.co>
* defaults_param:Change Angle and rate PID gain to manual tuned PID gains

* version:Change RC version from 2 to 3

---------

Co-authored-by: Manuraj P M <manurajp@airbound.co>
@Manuraj-PM Manuraj-PM force-pushed the pr-add_non_linear_mixer_v1 branch from 316f779 to e39bbdd Compare February 26, 2026 04:37
akshar-airbound and others added 2 commits March 10, 2026 15:20
#44)

* Add lua script to autobailout when there is pitch instability during vtol descend.

* autobail.lua:
	- Changed parameter defaults
	- modified landing detection function logic to accomodate NAV_VTOL_LAND

* Changed autobailout script parameter prefix from QREC_* -> AUTOB_*

* autobailout.lua: Replace bad count to autobailout with time based autobailout

* Moved autobail file location

* autobailout.lua: Modified parameter table key and added parameter asserts

* Renamed autobailout.lua to ab_autobailout.lua

* rename ab_autobailout.lua to autobailout.lua
autobailout.lua: Ensure type consistency on variables holding time

* default.parm: add Q_M_SPOOL_TIME to 1

* Fix for test chibios failure due to custom log field
@Manuraj-PM Manuraj-PM force-pushed the pr-add_non_linear_mixer_v1 branch from 94ea5c7 to 3b5d02b Compare March 11, 2026 05:20
Manuraj P M added 25 commits March 30, 2026 13:03
@Manuraj-PM Manuraj-PM force-pushed the pr-add_non_linear_mixer_v1 branch from 3b5d02b to 7acfd6d Compare April 7, 2026 06:02
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

Successfully merging this pull request may close these issues.

3 participants