Skip to content

[3D Printing] V3 Enclosure#187

Open
stefanferry wants to merge 95 commits intoPiTracLM:mainfrom
stefanferry:main
Open

[3D Printing] V3 Enclosure#187
stefanferry wants to merge 95 commits intoPiTracLM:mainfrom
stefanferry:main

Conversation

@stefanferry
Copy link

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


ISO 7380 to ISO 7380-2  

Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
increased nominal distance from 25 to 30 mm

Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
Signed-off-by: stefanferry <stefanfaehrmann@gmail.com>
@github-actions
Copy link
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
Author

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

@stefanferry
Copy link
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
Collaborator

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

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