WIP: Fix for #6: Incorrect bolt lengths #13
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "sw-tya/add_mesh_BoltFactory:swtya_BoltLength"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Fix for, Incorrect bolt sizes BoltFactory add-on #6
Since the first submission, the original code base always skipped 2 turns in the thread start, this is addressed.
The solution adds back a small bit of shank to account for up to 1 revolution of thread that is missed due to integer revolutions.
Note, minimum length is 5 turns, no change there.
I will review this when I can, but I still wanted to thank you for your contributions! 🎉
Minor wrinkle with further testing:
For head types that have either a bevel or radius on the underside their reported head height causes the washer face not to be at z=zero once complete. This in turn means that my calculated "thread" length is not correct due to already having some shank allocated in the head.
In def Bolt_Mesh() before Create_External_Thread() will be the place that a correction is required. I wonder if this is in part when there are global variables around for the Head_Height which are not used?
Personally I think what is here is much better than before - I'll have a few days of thinking...
da23330d34
to4718b93051
Second commit here makes modification to the head generation, by adding an additional return variable.
The washer face does not now move when changing between head types.
After editing the file I did:
I then messed around for a bit pulling and merging, but everything was up to date. so finally I did:
From the files changed view it seems to have only put in what I wanted to.
I'm unclear what I should have done to avoid the force???
Checkout
From your project repository, check out a new branch and test the changes.