Skip to content

[3D Printing] V3 Enclosure#187

Merged
jamespilgrim merged 1 commit intoPiTracLM:mainfrom
stefanferry:main
Mar 25, 2026
Merged

[3D Printing] V3 Enclosure#187
jamespilgrim merged 1 commit intoPiTracLM:mainfrom
stefanferry:main

Conversation

@stefanferry
Copy link
Copy Markdown
Contributor

Description

Pull request for the Enclosure Version 3

What does this PR do?

Adds all files from the Enclosure Version 3 design

Why is this change needed?

Improved enclosure design to ensure long term compatibility with the projects developments.

Related Issue(s)

not applicable

Changes Made

-Created a new directory called "Enclosure Version 3" in 3D Printed Parts containing all files for the V3 desing.
-Changed .gitignore to ignore temporary FreeCAD files (.FCbak)
-Updated /docs

Testing Performed

  • multiple fully functional builds by the community and me

Test Environment

  • Pi Model:
  • Camera Type:
  • OS Version:
  • Installation Method:

Test Results

  • pitrac test hardware passes
  • pitrac test camera passes (if camera-related)
  • pitrac test pulse passes (if strobe-related)
  • Simulator integration tested with:
  • Performance metrics:

Test Commands Run

# Paste the actual commands and output

Performance Impact

  • Before:
  • After:
  • Impact:

Breaking Changes

  • This PR includes breaking changes
    • What breaks:
    • Migration steps:

Dependencies

  • No new dependencies
  • [ ] New dependencies added:

  • [ ] Updated dependencies:

Hardware Compatibility

  • Tested on Pi 5
  • Tested on Pi 4
  • Tested with single Pi setup
  • Tested with dual Pi setup

Documentation

  • No documentation needed
  • Documentation updated in this PR
  • Documentation PR to follow
  • Updated relevant sections:
    • README
    • Hardware guide
    • Software guide
    • Troubleshooting guide

Screenshots/Videos

image

AI (Vibe-Coded) Content Description

Copilot was used for grammar/spell checks on the docs/ .md files

Checklist

Code Quality

  • Code follows existing patterns and conventions
  • No unnecessary comments added
  • Error handling implemented appropriately

Build & Test

  • Successfully builds with ./packaging/build.sh build
  • All existing tests pass
  • New tests added for new functionality
  • Tested on actual Raspberry Pi hardware (not just CI)

Submission Requirements

  • Commits squashed if needed (git rebase -i HEAD~n)
  • CLA signed
  • PR title follows format: [PR TYPE] Brief description
  • Branch is up-to-date with main

Additional Context


@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 22, 2026

All contributors have signed the CLA. Thanks!
Posted by the CLA Assistant Lite bot.

@stefanferry
Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@stefanferry
Copy link
Copy Markdown
Contributor Author

I think i messed up with the verified signatures and could not resolve it as I tought (see force push). Would appreciate input.

@connorgallopo
Copy link
Copy Markdown
Collaborator

@stefanferry if you squash into one signed commit you should be good

@jamespilgrim jamespilgrim merged commit 92f9c2b into PiTracLM:main Mar 25, 2026
2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Mar 25, 2026
@jamespilgrim
Copy link
Copy Markdown
Collaborator

I just bypassed and merged. Sorry I didn't see this earlier - I'm on travel and will be kinda pokey responding.

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