-
Notifications
You must be signed in to change notification settings - Fork 4
Feature/improved depth calculations #18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
phraktyl
wants to merge
15
commits into
spuder:main
Choose a base branch
from
phraktyl:feature/improved-depth-calculations
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Feature/improved depth calculations #18
phraktyl
wants to merge
15
commits into
spuder:main
from
phraktyl:feature/improved-depth-calculations
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
phraktyl
commented
Nov 18, 2025
- Correct chassis_depth_main calculation to include front_thickness
- Add full_chassis_depth variable for chassis body depth
- Update zip_tie_start_z to position at exact switch_depth
- Extend chassis to case_thickness past zip tie holes
- Update wire_channels to use full_chassis_depth
- Add +10 buffer to all cutout modules for clean boolean operations
- Ensures zip ties are fully enclosed with back wall
Replaced hardcoded values of 6 and 12 accordingly throughout
Add configurable case_thickness parameter
…meter option at the top of the file.
Changed the local hole_diameter to a more descriptive global wire_dia…
Moved the zip_tie_hole_width to the top for easier user configuration.
…ting Rotate air holes for easier 3D printing without bridges.
- Add switch_count parameter to specify number of switches - Add calc_required_height() helper function - Add adjust_rack_height() to auto-calculate rack height - Update main_body() to create continuous chassis - Update switch_cutout() to loop through each switch - Update power_wire_cutouts() for multiple switches - Update air_holes() to position per switch - Calculate wall and divider thickness from case_thickness
Add support for multiple stacked switches
- Create new wire_channels() module with curved channels - Calculate channel diameter based on wire_diameter + case_thickness - Position channels at switch edges for better wire management - Create curved hull between cylinders for smooth gradient - Only create channels when wire_diameter / 2 > case_thickness - Integrate channels into main_body() union - Extends full chassis depth to prevent wire slippage
Add enhanced wire channel system
- Replace horizontal indents with vertical slots only - Zip tie slots span entire continuous chassis height - Remove separate top/bottom indent grooves - Ensure case_thickness clearance from switch area - Update chassis_depth_main calculation - Add zip_tie_start_z positioning variable - Extend slots by tolerance for clean cuts - Increase default zip_tie_width to 3.0mm for doubled walls
Enhance zip tie system for multiple switches
- Correct chassis_depth_main calculation to include front_thickness - Add full_chassis_depth variable for chassis body depth - Update zip_tie_start_z to position at exact switch_depth - Extend chassis to case_thickness past zip tie holes - Update wire_channels to use full_chassis_depth - Add +10 buffer to all cutout modules for clean boolean operations - Ensures zip ties are fully enclosed with back wall
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.