Skip to content

[Housekeeping] Cleanup Dead Code / References#199

Open
connorgallopo wants to merge 15 commits intomainfrom
cleanup/dead-code
Open

[Housekeeping] Cleanup Dead Code / References#199
connorgallopo wants to merge 15 commits intomainfrom
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.

Copy link
Copy Markdown
Collaborator

@jeshernandez jeshernandez left a comment

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
*.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
Copy link
Copy Markdown
Contributor

ronheywood commented Apr 27, 2026

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

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