Log Analysis Introduced with backend_log_extraction.py

[OmkarSarkar204]

Description

Pr Introducing the Log_Analysis module for Ardupilot log, which will be used by and with AMC

Checklist

• Run pre-commit checks locally
• Verified by a human programmer
• All commits are signed off (use git commit --signoff)
• Code follows our coding standards
• [] Documentation updated if needed
• No breaking changes or properly documented

Testing

Describe how you tested these changes:

• Unit tests pass( for existing files not real files)
• Integration tests pass
• Manual testing performed
• Tested on flight controller hardware

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Introduces a new log_analysis module to parse ArduPilot .bin logs, extracting parameter values/defaults, firmware version info, message counts, and frame type, and reuses that logic from extract_param_defaults.py.

Changes:

• Added log_analysis/backend_log_extraction.py with log parsing + extraction helpers (params, firmware version, frame type).
• Added package initializer log_analysis/__init__.py.
• Refactored extract_param_defaults.py to reuse shared validation/version-parsing helpers from the new module.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 10 comments.

File	Description
ardupilot_methodic_configurator/log_analysis/backend_log_extraction.py	New backend log extraction implementation (open/close log, parse messages, extract params/firmware/frame type).
ardupilot_methodic_configurator/log_analysis/init.py	Initializes the new log_analysis package with a module docstring.
ardupilot_methodic_configurator/extract_param_defaults.py	Replaces in-file regex/version parsing with imports from backend_log_extraction.

[OmkarSarkar204]

The codespell was failing

codespell................................................................Failed
- hook id: codespell
- exit code: 65

ardupilot_methodic_configurator/data_model_configuration_step.py:351: SER ==> SET

So i just supressed it for now

[coveralls]

Coverage Report for CI Build 28046795869

Coverage decreased (-0.7%) to 93.678%

Details

• Coverage decreased (-0.7%) from the base build.
• Patch coverage: No coverable lines changed in this PR.
• 6 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

6 previously-covered lines in 1 file lost coverage.

File	Lines Losing Coverage	Coverage
extract_param_defaults.py	6	95.88%

---

Coverage Stats

Relevant Lines:	13651
Covered Lines:	12788
Line Coverage:	93.68%
Relevant Branches:	2096
Covered Branches:	2014
Branch Coverage:	96.09%
Branches in Coverage %:	No
Coverage Strength:	2.8 hits per line

---

💛 - Coveralls

[amilcarlucas]

Thanks for this. I reviewed the code and did some improvements.

[OmkarSarkar204]

Thanks for this. I reviewed the code and did some improvements.

I was thinking of adding the test once ive finished the basic signals, but Thank You , Im working on the PM, IMU , VIBE and ERR messages and will try to do it by today midnight. :)

[OmkarSarkar204]

Extracted till now
PARM ,VER, MSG (version fallback), BAT, PM, IMU, VIBE, GPS, ERR, ARM, MODE

[amilcarlucas]

Looks good. By plugins I meant: ARCHITECTURE_battery_monitor.md and ARCHITECTURE_motor_test.md

[amilcarlucas]

This looks good, we can merge it in.

[OmkarSarkar204]

This looks good, we can merge it in.

Thank You! I updated the pr and did some minor commnets changes :)

[OmkarSarkar204]

Thank you for the tests i corrected the commit and fixed the ci failures should be passing now

[amilcarlucas]

Thanks. I requested a github copilot code review. It that has no findings, we can merge this.

[Copilot]

Pull request overview

Copilot reviewed 12 out of 13 changed files in this pull request and generated 5 comments.

[OmkarSarkar204]

@amilcarlucas , IS this ok, please let me know if any other changes required?