Skip to content

[Housekeeping] Cleanup Dead Code / References#199

Merged
connorgallopo merged 18 commits into
mainfrom
cleanup/dead-code
May 31, 2026
Merged

[Housekeeping] Cleanup Dead Code / References#199
connorgallopo merged 18 commits into
mainfrom
cleanup/dead-code

Conversation

@connorgallopo

Copy link
Copy Markdown
Collaborator

This is just a massive cleanup.

This is also pr 1/3. The other two will deal with code formatting.

@jeshernandez jeshernandez left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice cleanup. Left a couple comments.

Comment thread .github/ISSUE_TEMPLATE/bug_report.yml
Comment thread .prettierignore Outdated
*.FCBak

# Jekyll / docs build
_site/

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can get rid of this I suppose since we wont have it. Doesn't hurt if not.

@ronheywood

ronheywood commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

I was thinking of removing
https://github.com/PiTracLM/PiTrac/tree/58e0a1c4304763b6f02f9a894de6154f24e5e363/Software/LMSourceCode/ImageProcessing/Camera
and
https://github.com/PiTracLM/PiTrac/tree/58e0a1c4304763b6f02f9a894de6154f24e5e363/Software/LMSourceCode/ImageProcessing/ImageAnalysis

They were staging spaces for a potential refactor into a more Clean design, but I got side tracked - and you've been doing good work, so I don't think we'll need this now

Looks like maybe ImageAnalysis.infrastructure picked up some of the yolo capability, but I think the rest of that BC is currently unused

@connorgallopo connorgallopo merged commit 75f5932 into main May 31, 2026
8 checks passed
@connorgallopo connorgallopo deleted the cleanup/dead-code branch May 31, 2026 17:32
@github-actions github-actions Bot locked and limited conversation to collaborators May 31, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants