looooo / freecad.gears

a gear module for freecad
GNU General Public License v3.0
252 stars 95 forks source link

Fix head and clearance for the cycloid gear #93

Closed jbaehr closed 2 years ago

jbaehr commented 2 years ago

Head and clearance values have been applied only once, but as we're dealing with a diameter, not a radius, they are needed twice. (The involute gear does not suffer from this bug)

Note that master suffers from this bug, too, but as this change is breaking, backporting needs careful consideration.

jbaehr commented 2 years ago

@looooo what do you think, should we backport this fix to master, too?

Both cases are quite easy to fix, though: just divide the clearance value by two. But: you have to know!

@jacprog, you did a lot of work using the cycloid gears for your 3D printed clocks. What do you think, as a user?

looooo commented 2 years ago

@jbaehr I guess we can also apply the fix on master. But I am not sure how much sense this makes. In my mind it is ok to keep the commit in the develop branch only. I am not yet sure when to make a new release, but I guess this will happen in the spring. So best to concentrate on the develop branch. Btw. thanks for your work!!!