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

Mounting hole options #16

Open
dtex opened this issue Jun 2, 2016 · 4 comments
Open

Mounting hole options #16

dtex opened this issue Jun 2, 2016 · 4 comments

Comments

@dtex
Copy link

dtex commented Jun 2, 2016

I'd like to add a parameter to the SCAD file for choosing different hole patterns for the top plate (Tessel 2 is my use case). Does that sound like a worthwhile PR?

@makenai
Copy link
Owner

makenai commented Jun 2, 2016

Yup - most certainly. Thanks for looking at the SCAD files instead of the vectors. :)

I think you should be able to add an parameter / method to it similar to the arduino and pinoccio ones here: https://github.com/makenai/sumobot-jr/blob/master/3d_print/sumobot.scad#L186

@dtex
Copy link
Author

dtex commented Jun 2, 2016

Cool. I was thinking of adding a parameter like this:

// Mounting hole pattern
// valid values are "Arduino Uno", "Tessel 2"
hole_pattern = "Arduino Uno";

And then just add a conditional in top to call the right module.

@dtex
Copy link
Author

dtex commented Jun 5, 2016

Hmmm, like most things it's never as easy as it seems. I had to make the sled wider to fit the mounting holes, but the GPIO ports are still kind of exposed (anything plugged in could be easily ripped out by another sumo).

Sigh...

I'll keep playing with it.

Sumobot with Tessel 2

@dtex
Copy link
Author

dtex commented Jun 26, 2016

I'm thinking it makes the most sense to have an option for Tessel mounting holes like we discussed, but then just have a note somewhere (the readme?) that gives the minimum sled width and a suggested sled length... Well, maybe not. I don't see any reason we couldn't set the default width and height based on that same hole_pattern variable.

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