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 ThrM (modified throttle send to engine) to EFIS log #106

Merged

Conversation

Pradeep-Carbonix
Copy link
Contributor

ThrM = Modified Throttle (after linearization and offset) is part of the log

@Pradeep-Carbonix Pradeep-Carbonix requested review from robertlong13, Jono453 and a user February 15, 2024 22:01
@Pradeep-Carbonix Pradeep-Carbonix force-pushed the feature/SW-107-add-fc-throttle-request-in-logs branch from f8cb8a0 to 99ef9b2 Compare February 16, 2024 06:05
@Jono453
Copy link
Contributor

Jono453 commented Feb 19, 2024

Please update once the logging is fixed.

 - ThM added to help understand the engine logs better
 - Updated the field names to keep the total length 64-characters long.
@Pradeep-Carbonix Pradeep-Carbonix force-pushed the feature/SW-107-add-fc-throttle-request-in-logs branch from 99ef9b2 to 7c3886c Compare February 20, 2024 05:47
@Pradeep-Carbonix
Copy link
Contributor Author

Please update once the logging is fixed.

Logging has 64 characters limit. This is fixed now.

@Pradeep-Carbonix Pradeep-Carbonix merged commit 0c51200 into CxPilot Feb 20, 2024
62 checks passed
@robertlong13 robertlong13 deleted the feature/SW-107-add-fc-throttle-request-in-logs branch June 17, 2024 07:10
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