Skip to content
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

lathe change gears #90

Open
skeptonomicon opened this issue Jul 4, 2021 · 6 comments
Open

lathe change gears #90

skeptonomicon opened this issue Jul 4, 2021 · 6 comments

Comments

@skeptonomicon
Copy link

skeptonomicon commented Jul 4, 2021

I would like to add a feature for making involute change gears which are used on lathes, which are similar to the gear function in this file, but generally are symmetric about the plane of rotation instead of having the hub offset, and generally have spokes rather than a flat disk or disk with circular holes. I think many will find this useful. I have the code working, and could simply add the features into the existing gear function with the addition of some spoke parameters and a symmetric flag. Otherwise I could have a separate function altogether. I am not sure who is maintaining the involute gear library and want to tread lightly as I go since I am new to this environment. If anyone has advise, please respond.


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@hyperair
Copy link

hyperair commented Jul 5, 2021

Integrating those features via new opt-in parameters that sounds like a good approach. Could you submit the PR against the dev branch please? The gear module has had some changes since the master branch and that's where development happens these days.

@skeptonomicon
Copy link
Author

the dev branch involute_gears seams quite broken, especially the internal gears. do I fix the involute problems first on a separate branch? I am comfortable with any changes in openscad, but less familiar with this branch/branch/pullrequest github process.

@hyperair
Copy link

hyperair commented Jul 7, 2021

Submitting a pull request against the dev branch is similar to submitting against the master branch, just that you need to change the base branch to dev like so:
image

@hyperair
Copy link

hyperair commented Jul 7, 2021

If it's too much trouble, you can also submit your PR to master as-is, and I'll just merge it into master and port the changes over to dev at a later date

@skeptonomicon
Copy link
Author

Let me look at the state of the main branch. I will get this in soon, but I am busy at work this week.

@skeptonomicon
Copy link
Author

skeptonomicon commented Jul 19, 2021

I fixed several issues including fixing the test case, fixing the rack, and adding the spoked gears, all based on the Master. I created a pull request.

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

No branches or pull requests

2 participants