-
Notifications
You must be signed in to change notification settings - Fork 21
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
OpenDACT 3.1.1A corrupts the value of "Z-probe height" #14
Comments
Was just checking through the GCode.cs and found the broken logic. Starts at line 143 and ends at 149. Been a while since I messed with this and see that the main reasoning for modifying the value was to determine the Z-probe height without asking the user (higher precision), assuming that the previous Z Max height was correct for printing. The only issue here is that the Z-Probe height value is never set based off of what the probe returns. If you want a temporary workaround, then reduce your max z height by 12.4mm before calibration and after calibration increase it by 12.4mm. Then of course the z probe height will need to be set to 0. I will work on a fix for this. |
Please consider making the tool more verbose (like writing g-codes it generates, to the log) - it could have made spotting this easier. I used http://www.serial-port-monitor.com/ - but since its freeware version is very limited, it was quite a challenge. |
Would this explain why after I got my Z-Probe dialed in and working with G30, running OpenDACT appears to be wiping EEPROM values and crashing into the plate? |
I couldn't help myself, I'm doing a bunch of rewriting. I'll see what I can track down. |
@outlookhazy , check that "steps per mm" in OpenDACT matches the actual setting. |
I have the exact same issue with it changing the z-probe height to -39 in the eeprom then homing after that it just crashes into the bed. I only spotted this when watching it in advance mode and saw the eeprom Z-Probe height change. Is they any way to stop it changing the z-probe height as it is correct. |
I ended up writing my own version of the program, using a different calibration algorithm |
Cool would you be willing to share? As I am not having very much luck with Auto Delta calibrations and Repetier. I have two different options left to me move to Marlin (that has it own problem to deal with) or manual calibrations and forget about automatic which just sucks. |
@outlookhazy , please share your work, it will be highly appreciated. |
@lincolnep @Bougakov I make no claims that it will work for other people - but if G30 already works for you and your printable radius is set properly - you can give it a shot. Firmware is selectable but I really only support Repetier. Check my fork, you'll find a v0.1 pre-release |
@outlookhazy Thanks Outlookhazy I gave your tool a go last night and found a few issues
Thank you for help if you would like me to test anything else I can do it tonight. |
@lincolnep for clarity sake, please leave future comments on my fork so as not to clutter up your feed with things not relevant. In reply - yes, I suppose I should have warned about that stuff - I really wrote it for myself and was tuning settings on-the-fly.
|
Hey, hello, i'am new here, I have a same problem that @outlookhazy, my z-probe height to 39 after 1 iteration. Have you resolve your problem, thank you a lot. |
I used serial port sniffer to locate this issue.
My z-probe is located 12.4mm below the nozzle on a hinge. The value is entered in EEPROM:
EPR:3 808 12.400 Z-probe height [mm]
For some reason OpenDACT corrupts it. When I open the program and use "check current heights", it probes once, then sends this:
M206 T3 P808 X5.280
and, almost immediately, this:
M206 T3 P808 X5.280
In the program the value gets displayed as 5.2799999
Why would it touch this value, it should be treated as a constant?
The text was updated successfully, but these errors were encountered: