Skip to content

Add padding parameter explanation to manual for filament winding#776

Merged
kraemr merged 3 commits intomasterfrom
775-add-explanation-of-padding-to-the-winder-manual
Oct 13, 2025
Merged

Add padding parameter explanation to manual for filament winding#776
kraemr merged 3 commits intomasterfrom
775-add-explanation-of-padding-to-the-winder-manual

Conversation

@TheBest6337
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings October 3, 2025 17:23
@TheBest6337 TheBest6337 linked an issue Oct 3, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds documentation for the padding parameter in the filament winding manual, explaining how it creates safety margins to prevent filament from winding too close to spool edges.

Key Changes

  • Added a new "Understanding Padding" section explaining the padding parameter functionality
  • Included practical examples showing how padding affects winding area calculations
  • Provided guidance on adjusting padding values based on filament requirements

@TheBest6337 TheBest6337 requested a review from kraemr October 6, 2025 09:47
@TheBest6337 TheBest6337 marked this pull request as draft October 6, 2025 09:50
@TheBest6337
Copy link
Copy Markdown
Member Author

Milan wants me to add an illustration to it, so im marking it as a draft.

@TheBest6337 TheBest6337 force-pushed the 775-add-explanation-of-padding-to-the-winder-manual branch from eec90f8 to 93d1db2 Compare October 8, 2025 18:57
@TheBest6337 TheBest6337 marked this pull request as ready for review October 8, 2025 18:57
@kraemr kraemr force-pushed the 775-add-explanation-of-padding-to-the-winder-manual branch from 93d1db2 to 97cb143 Compare October 13, 2025 10:06
@kraemr kraemr force-pushed the 775-add-explanation-of-padding-to-the-winder-manual branch from 97cb143 to 5ebb978 Compare October 13, 2025 10:09
@kraemr kraemr merged commit 57a0cc8 into master Oct 13, 2025
4 checks passed
@kraemr kraemr deleted the 775-add-explanation-of-padding-to-the-winder-manual branch October 13, 2025 10:13
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.

Add explanation of 'Padding' to the Winder manual

3 participants