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

[Manual] Mention that Hall Effect PCB is for legacy support and not in any way an upgrade #388

Open
ChaosBlades opened this issue Sep 21, 2022 · 9 comments
Labels

Comments

@ChaosBlades
Copy link

In the manual the Hall Effect is presented as an upgrade. At least I interpreted it that way and it seems several people in the Amazon reviews for the Hall Effect PCB I bought thought this as well. Turns out the Hall Effect sensor is worse in basically every way from the micro switches as I found out asking about it in Discord. This should really be pointed out in the documentation or just removed from the manual all together.

@raymondh2
Copy link
Contributor

I'll bring it up to the team.

@nemgrea
Copy link
Contributor

nemgrea commented Sep 22, 2022

In the manual the Hall Effect is presented as an upgrade. At least I interpreted it that way and it seems several people in the Amazon reviews for the Hall Effect PCB I bought thought this as well.

unknown

its shown as an option, i dont think there is anything in here that expresses that one is better than the other or that one would be an upgrade over the other, simply two options are shown.

@ChaosBlades
Copy link
Author

I don't think that is good enough. According to my discussion on Discord the hall effect isn't any more reliable than micro switches, it is sensitive to chamber temp, switching to PLA when tuned for ABS it could go so out of spec enough to cause a crash, it drifts over time, and you can't even increase home speed. It is not an improvement in any way.

What I was told was...

StevenB V2.3370 — Yesterday at 2:55 AM
Because it was a test that failed and it's still kept as an option because of that.

When I go to some of these selling on Amazon I see people complaining that the Hall Effect PCBs are using a different connector (Molex instead of JST) so they need to re-cable when moving from Microswitch. Which is a downgrade. So clearly myself and other have fallen victim to this simply being show as an option. I am sure there is some kind of lesson here on assuming things.

@nemgrea
Copy link
Contributor

nemgrea commented Sep 22, 2022

i agree is is not any more reliable at all NOR is it ever claimed to be more reliable, im not sure why you are assuming that it would be more reliable when it is not presented as "better" or of higher quality in any of the documentation. it is simply showing steps for an OPTION that exists and happens to be popular. no claims of benefits or performance are shown, stated, or implied in the manual...

@subatomicglue
Copy link

subatomicglue commented Sep 23, 2022 via email

@raymondh2
Copy link
Contributor

Our BOM already lists the normal microswitch setup and not any of the PCBs. But these PCBs are included in the manual because they are so common and we get too many questions when we don't include the instructions for it.

@nemgrea nemgrea added the Manual label Dec 15, 2022
@asdasdwa-v02679
Copy link

Could we possibly solve this problem by adding text on page 191 below the images that has a warning specifying that although the pcbs can simplify wiring they tend to introduce their own issues with repeatability and aren't recommended.

or is it just the hall effect pcb? if so perhaps it would read more like "although the hall effect sensor pcbs have become super common and can simplify wiring, they tend to introduce their own issues with repeatability and aren't recommended."

does this seem like a good solution according to yall? if so and people approve I am happy to make the edit to the page :)

@majarspeed
Copy link

This is not the only place in the build where you can make a choice. These options exist because people have used them successfully. But to try to critique the benefits of one over the other is not really the purpose of an assembly manual.

@asdasdwa-v02679
Copy link

@majarspeed I understand that but It seems that Nemgrea marked it as manual so I was trying to find a solution. also It does seem a fair few people get confused by this page and so a edit might be in order ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants