Skip to content
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

Adding ThM (modified throttle send to engine) to EFIS log #27077

Merged
merged 1 commit into from
Aug 18, 2024

Conversation

loki077
Copy link
Contributor

@loki077 loki077 commented May 16, 2024

  • ThM added to help understand the engine logs better
  • Updated the field names to keep the total length 64-characters long.

Copy link
Contributor

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

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

Commit message needs fixing.

libraries/AP_EFI/AP_EFI_Serial_Hirth.cpp Outdated Show resolved Hide resolved
libraries/AP_EFI/AP_EFI_Serial_Hirth.cpp Outdated Show resolved Hide resolved
@loki077 loki077 force-pushed the pr_modif_hirth_logging branch from 3cc5e0f to 76a69cd Compare July 30, 2024 00:40
@loki077
Copy link
Contributor Author

loki077 commented Jul 30, 2024

Commit message needs fixing.

I will fix the commit message once the code is all good.

@loki077
Copy link
Contributor Author

loki077 commented Jul 30, 2024

@Pradeep-Carbonix @robertlong13 FYI

@loki077 loki077 force-pushed the pr_modif_hirth_logging branch 3 times, most recently from b70123f to 8a31c57 Compare August 11, 2024 23:58
- Added EFIS log for Hirth engine
- Added ThM to help understand the engine logs better
- Updated the field names to keep the total length 64-characters long.
- Added Bitfield for the EFIS log to log all errors.
@tridge tridge force-pushed the pr_modif_hirth_logging branch from 8a31c57 to 488de85 Compare August 15, 2024 22:57
@peterbarker peterbarker merged commit 54ec26a into ArduPilot:master Aug 18, 2024
93 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants