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

LED meter model differ from measured one #294

Open
mmuman opened this issue Oct 19, 2024 · 7 comments
Open

LED meter model differ from measured one #294

mmuman opened this issue Oct 19, 2024 · 7 comments

Comments

@mmuman
Copy link
Contributor

mmuman commented Oct 19, 2024

when replacing my own model from the measurements I made with the one from the lib, I noticed mine was larger by quite a few mm… i measured 32.7mm total length, 10.5mm width though, so a little smaller.

Should we account for the larger one? But then the center distance is different as well… 🤔

Screenshot_20241019_233231

@nophead
Copy link
Owner

nophead commented Oct 20, 2024

It must be a slightly different version because I used them in a project and that is where I got the dimension from. So we would need a different model adding. Not sure what it would be called though!

@mmuman
Copy link
Contributor Author

mmuman commented Oct 20, 2024

That's the problem with these items, they all clone each other but with varying details… Maybe passing default dimensions and allowing to override them…

@nophead
Copy link
Owner

nophead commented Oct 20, 2024

It would be a bit odd to do that because it is a different object, so should have a new object definition as that is how everything else works in the library. Also it makes people aware there are different versions when they use it so they are more likely to not get s surprise when they don't fit.

@mmuman
Copy link
Contributor Author

mmuman commented Oct 20, 2024

How would we go about naming them? I just home we don't end up with 20 of them.

@nophead
Copy link
Owner

nophead commented Oct 20, 2024

You can name with the important dimensions that change. E.g. led_meter23x11x26

@mmuman
Copy link
Contributor Author

mmuman commented Oct 20, 2024

As I did with the rocker yeah… Should we keep the original for compatibility and add it with dimensions as an alias?

@nophead
Copy link
Owner

nophead commented Oct 20, 2024

With the rocker you just called it micro and put the dimensions in the description. I was suggesting making the name from the important dimensions as well. Yes you could add an alias for the original and rename it.

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