Skip to content

Tech Debt Fix: FBI converted to Python (Incomplete)#84

Merged
AndrewQuijano merged 14 commits intomasterfrom
fbi-refactor
Dec 21, 2025
Merged

Tech Debt Fix: FBI converted to Python (Incomplete)#84
AndrewQuijano merged 14 commits intomasterfrom
fbi-refactor

Conversation

@AndrewQuijano
Copy link
Copy Markdown
Collaborator

@AndrewQuijano AndrewQuijano commented Nov 30, 2025

Your checklist for this pull request

  • I've documented or updated the documentation of every function and struct this PR changes.
  • I've added tests that prove my fix is effective or that my feature works (if possible)

Detailed description

Rewrite FIB to use Python instead of C++. This minimizes tech debt, plus now that PANDA logs are in JSON, there is no need for C++

This allows for the deletion of the FIB C++ version in a later PR.

...

Test plan

  1. Confirm that this passes the same CI/CD
  2. I can confirm that the number of bugs is the same, although the Python version has 10 more DUAs than the C++ version. This bug will be investigated later, but given that the number of bugs hasn't changed, it is likely not critical.

...

Closing issues

...

@AndrewQuijano AndrewQuijano mentioned this pull request Nov 30, 2025
5 tasks
@github-actions github-actions bot added the Lava-Core LAVA C/C++ code label Dec 19, 2025
@AndrewQuijano AndrewQuijano force-pushed the fbi-refactor branch 6 times, most recently from 08f3f83 to a40e215 Compare December 20, 2025 21:57
@AndrewQuijano AndrewQuijano changed the title Tech Debt Fix: FBI refactor and cleaned up Python code Tech Debt Fix: FBI converted to Python Dec 21, 2025
@AndrewQuijano AndrewQuijano merged commit a0be150 into master Dec 21, 2025
4 checks passed
@AndrewQuijano AndrewQuijano deleted the fbi-refactor branch December 21, 2025 02:29
@AndrewQuijano AndrewQuijano linked an issue Dec 24, 2025 that may be closed by this pull request
5 tasks
@AndrewQuijano AndrewQuijano changed the title Tech Debt Fix: FBI converted to Python Tech Debt Fix: FBI converted to Python (Incomplete) Jan 6, 2026
@AndrewQuijano AndrewQuijano added this to the LAVA Modernization milestone Jan 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Lava-Core LAVA C/C++ code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remaining Technical Debt

1 participant