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

mbl with z_max endstops not usable #4088

Closed
chrisqwertz opened this issue Jun 19, 2016 · 26 comments
Closed

mbl with z_max endstops not usable #4088

chrisqwertz opened this issue Jun 19, 2016 · 26 comments
Labels
Bug: Confirmed ! Needs: Work More work is needed T: Feature Request Features requested by users.
Milestone

Comments

@chrisqwertz
Copy link
Contributor

When using mesh bed leveling thorugh the LCD menu on my machine with z-max endstops, the leveling prcess will start at z_max height thus making it very tedious to move all the way down with the encoder.
I tried using MESH_G28_REST_ORIGIN with no success.
Am I missing something?

@epatel
Copy link
Contributor

epatel commented Jun 19, 2016

@chrisqwertz No you are not missing anything. This is one of the things I have been wanting to add to MBL. If you want to be my tester I can see if I can add this.

@chrisqwertz
Copy link
Contributor Author

My spare time is a little rare, but yes sure, if I can help, I'm in.

@Roxy-3D
Copy link
Member

Roxy-3D commented Jun 19, 2016

Just as an FYI: Please consider doing your work and testing using the very latest RCBugFix. ThinkyHead has started cleaning up a lot of the probe stuff. It is very possible that it will be much easier to do the work there. (And even if it isn't easier... We ultimately need to get your work merged in with what will be the 'current code' anyway.)

@thinkyhead thinkyhead added T: Feature Request Features requested by users. Bug: Confirmed ! Needs: Work More work is needed labels Jun 20, 2016
@thinkyhead thinkyhead added this to the 1.1.0 milestone Jun 20, 2016
epatel added a commit to epatel/Marlin that referenced this issue Jun 25, 2016
@epatel
Copy link
Contributor

epatel commented Jun 25, 2016

@chrisqwertz Hi, I have made this commit https://github.com/epatel/Marlin/tree/epatel/max_z_endstop
Please test to see if it works for you.

Note about usage. Set Z_MAX_POS so that it is slightly less than the full length from the Z endstop to the bed. The bed should be within the MESH_HOME_SEARCH_Z distance, default is 4mm. When starting the probe sequence it should first make a HOME, going up to the Z endstop. Then go down the full Z_MAX_POS length, temporarily set position Z=4 (the search distance) and one should be able to use the "encoder" on the display to slowly lower the hotend to the bed (use a paper to feel when it is near). Press the "encoder" when you think its fine. Repeat until you have probed all mesh points.

@thinkyhead
Copy link
Member

Making a PR out of it…

@chrisqwertz
Copy link
Contributor Author

roger, will report back with results tonight!

@chrisqwertz
Copy link
Contributor Author

Looks like the logic works now, what I have observed although is that the extruder will move to 0 z-height, then scrape across the bed to rest at the first probing point and all subsequent ones. Could it be, that my "MESH_HOME_SEARCH_Z" is not respected? I used the LCD to level the bed and have a M206 Z-0.1 toolhead offset.

@epatel
Copy link
Contributor

epatel commented Jun 26, 2016

@chrisqwertz Sounds like Z_MAX_POS is the distance between the max endstop and the bed. Try making it 2mm shorter, i.e. if your Z_MAX_POS is 150 make it 148. With MESH_HOME_SEARCH_Z equals 4 (mm) you should be able to "probe" via the encoder about 2mm down with 2 mm to spare for lower other points (but that would be a crazy bad bed)

I made this pic :)

untitled - 1 kopia

@thinkyhead
Copy link
Member

thinkyhead commented Jun 26, 2016

Sounds like Z_MAX_POS is the distance between the max endstop and the bed

Not good. It should be the distance from the nozzle to the bed when Z is homed at the top.

@chrisqwertz Try setting a MIN_Z_HEIGHT_FOR_HOMING value also, as this will give you an additional lift in MBL. (Also, I would like to see how it behaves when homing to Z max.)

@epatel
Copy link
Contributor

epatel commented Jun 26, 2016

By making sure Z_MAX_POS is less than the distance to the first probe point one makes sure the software endstops triggers before Z reaches the endstop (and other stuff maybe) over the whole bed. I am not sure how or if MIN_Z_HEIGHT_FOR_HOMING helps here. But again, hopefully no-one is actually printing that tall stuff.

@thinkyhead
Copy link
Member

thinkyhead commented Jun 26, 2016

I am not sure how or if MIN_Z_HEIGHT_FOR_HOMING helps here.

I believe that without MIN_Z_HEIGHT_FOR_HOMING there is no lift between probe points in MBL, and in fact it moves back down to the bed, so the nozzle usually ends up dragging on the bed. I think we should probably use the Z_RAISE_BETWEEN_PROBINGS value for that instead.

@epatel
Copy link
Contributor

epatel commented Jun 26, 2016

I have never recommended to have the endstops so that the nozzle touches the bed, never! I have always recommended the opposite, set up the printer so it is safe to probe from a safe starting point.

@epatel
Copy link
Contributor

epatel commented Jun 26, 2016

And my hope is that @Roxy-3D will be able to make something useful of all the changes that has gone in. Please note that I was going to add a PR after @chrisqwertz verified this. But I see you have already merged this yesterday.

@thinkyhead
Copy link
Member

thinkyhead commented Jun 26, 2016

I have never recommended to have the endstops so that the nozzle touches the bed, never!

Nevertheless, we are talking about tilted and warped beds here, and users very, very often have their Z min endstops set slightly lower than the bed. This is certainly the case on my i3. Then there's tape on the bed, ABS slurry, leftover brims… It's just better, safer, and most prudent to raise the nozzle a little when moving between probe points.

@Roxy-3D will be able to make something useful of all the changes that has gone in

She will! She's pretty sharp. Should be no trouble for her.

@epatel
Copy link
Contributor

epatel commented Jun 26, 2016

Nevertheless, I believe it is bad to have the endstop lower than the bed period! as that will push up on the hotend and can cause various effects, moving bed, moving hotend/carriage movement etc and maybe even break it after some time. But just moving it slowly out of position I think is bad enough. Well, thats me. I would not set up my printers like that, ever.

@thinkyhead
Copy link
Member

I would not set up my printers like that, ever.

The thing is, on a Cartesian machine with a Z_MIN endstop, the endstop switch itself blocks Z from moving any lower. Thus, if you want to have the freedom to move lower (for whatever reason), you must move your endstop a little lower. It's also common on some setups for the Z endstop to slip a little over time. In any case, it isn't entirely uncommon, and we must accommodate a wide variety of "non-ideal" situations by providing some safety margins. A raise between probes in MBL is one such "safety margin."

@epatel
Copy link
Contributor

epatel commented Jun 27, 2016

The thing is that there are several kinds of switches. Switches with an arm and some without, and optos. Yes, for those without an arm that could be a problem. But, I'd suggest changing the endstop to one with an arm or best get an opto endstop. The price of the hotend is way larger than a good endstop so it should be a no-brainer. But, this issue is not about z min endstops, this is specifically for z max endstop. And I have explained even with a small picture how I recommend configuring for this, making it safe, because with a safer number for Z_MAX_POS it is less risk to run pass the z max endstop (with the software endstop). I will not explain this, I am tired.

This seem to be a good deal.
http://www.ebay.com/itm/3pcs-Opto-Endstop-Switch-Kit-for-CNC-3D-Printer-RepRap-Prusa-Mendel-RAMPS1-4-/161836552716

Oh yeah, then you might argue you need a small alu sheet or something. I have successfully used my favorite beer can for that :) https://www.brewdog.com/item/262/BrewDog/Dead-Pony-Club-Can.html

@epatel
Copy link
Contributor

epatel commented Jun 27, 2016

Not good. It should be the distance from the nozzle to the bed when Z is homed at the top.

Yeah, you are right. But I thought it was obvious that I meant nozzle-to-bed, because that is basically what Z_MAX_POS means. And really the distance between nozzle and z max endstop when endstop is triggered is not really interesting (can't recall seeing anyone ever mentioning it).

So sorry for not being precis enough.

@thinkyhead
Copy link
Member

I thought it was obvious that I meant nozzle-to-bed

I'm just thinking ahead, to the person who finds this thread months from now and gets out their tape measure, and…

@chrisqwertz
Copy link
Contributor Author

Is there anything more I can do to assist?

@epatel99
Copy link
Contributor

@chrisqwertz Uh, I think the fix is in RCBugFix now so test that maybe? Or that was what you did? Otherwise I think it is all fine and it's ok to close this issue if you think it's ok now.

@chrisqwertz
Copy link
Contributor Author

Allright, will try to check tonight
On Jun 29, 2016 09:52, "Edward Patel" [email protected] wrote:

@chrisqwertz https://github.com/chrisqwertz Uh, I think the fix is in
RCBugFix now so test that maybe? Or that was what you did? Otherwise I
think it is all fine and it's ok to close this issue if you think it's ok
now.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#4088 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AETW2nc6dVggtFzTJIsc1OYVfyzh4oznks5qQiRZgaJpZM4I5Ij4
.

@jbrazio
Copy link
Contributor

jbrazio commented Jul 11, 2016

@chrisqwertz did you had the opportunity to check ?

Offtopic:
@epatel something wrong with your Github account ? I now see a @epatel99 using your avatar.

@epatel
Copy link
Contributor

epatel commented Jul 12, 2016

@jbrazio No problem, I just sometime happen to use my "job" github account (when I am too lazy to logout).

@boelle
Copy link
Contributor

boelle commented Aug 11, 2016

has this one been reverified with RC7?

CONSULitAS pushed a commit to CONSULitAS/Marlin-K8200 that referenced this issue Aug 18, 2016
@github-actions
Copy link

github-actions bot commented Apr 4, 2022

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Apr 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug: Confirmed ! Needs: Work More work is needed T: Feature Request Features requested by users.
Projects
None yet
Development

No branches or pull requests

7 participants