Add Comprehensive Log Investigator plugin - #3754
Open
nadav-golan-yanay wants to merge 6 commits into
Open
nadav-golan-yanay wants to merge 6 commits into
nadav-golan-yanay wants to merge 6 commits into
Conversation
Added a more usefull log invesigator
Updated version comment in ComprehensiveLogInvestigator.cs
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Author
Example_Drone_Log_Investigation_20260913_0105.html Hi maintainers — friendly follow-up on this PR when you have time 🙏 I added a Comprehensive Log Investigator plugin and wanted to make review easier with concrete artifacts: UI preview attached (screenshot) |
Author
|
@copilot review this PR for correctness, potential bugs, edge cases, and maintainability. Please list findings by severity and include exact file paths. |
Co-authored-by: nadav-golan-yanay <78790309+nadav-golan-yanay@users.noreply.github.com>
…rehensive-log-investigator Harden Comprehensive Log Investigator for sparse/truncated telemetry and long-log sampling bias
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds a Comprehensive Log Investigator plugin for Mission Planner.
The plugin provides automated post-flight analysis of ArduPilot DataFlash .bin logs directly from Mission Planner.
It generates a structured investigation covering flight overview, mode transitions, navigation and GPS performance, attitude and control behavior, power system performance, significant events, warnings, anomalies, and pilot-oriented findings.
The goal is to make comprehensive flight-log investigation accessible to Mission Planner users without requiring external analysis tools.
The plugin is implemented as a single Mission Planner .cs plugin and is located under the Plugins directory.
I have tested the plugin with multiple ArduPilot flight logs.