search

MarlinFirmware / Marlin

Marlin is an optimized firmware for RepRap 3D printers based on the Arduino platform. Many commercial 3D printers come with Marlin installed. Check with your vendor if you need source code for your specific machine.
https://marlinfw.org
GNU General Public License v3.0
16.27k stars 19.23k forks source link

[BUG] DWIN_CREALITY_LCD Current Position. #22831

Closed tome9111991 closed 3 years ago

tome9111991 commented 3 years ago

Did you test the latest bugfix-2.0.x code?

Yes, and the problem still exists.

Bug Description

Hello With latest version. I have strange coordinate after auto homing. also i cant set Z offset for probe it jumps in 1mm steps.

My bed Size is 230x230x250

I have a ender 3v2 with e3 turbo and stock display. Tested with DWIN_CREALITY_LCD and DWIN_CREALITY_LCD_JYERSUI

Edit:

JyersUI Allready fixed with https://github.com/MarlinFirmware/Marlin/commit/f6c7fbd1ab62b0c3aa70c971ad6c30d062d3252a https://github.com/MarlinFirmware/Marlin/commit/b93c3301f2146bed7287ad097b95b53507c35420

Only have issue with DWIN_CREALITY_LCD current position and Zoffset show wrong value.

20210924_164737

~~Btw. Durring compiling i got #warning "Auto-assigned Z2 STEP/DIR/ENABLE_PINs to unused En_STEP/DIR/ENABLE_PINs." I use dual Z~~

Config config.zip

tome9111991 commented 3 years ago

@thinkyhead i guess the bug start at https://github.com/MarlinFirmware/Marlin/commit/5b5a8798f8fe202e86c008088f847a3edb51cc71

i have tried https://github.com/MarlinFirmware/Marlin/commit/cca3250c3ff8e8ed4277f4918f507ab13913d3b5 this works.

with latest commit from now still same issue.

it looks the . (point) is on wrong place i have a Probe Z Ofsset by -2.74 LCD show me -27.4

For exemple in DWIN_CREALITY_LCD 20210925_150059

tome9111991 commented 3 years ago

With last bugfix.

only have the issue on DWIN_CREALITY_LCD

working is:

DWIN_CREALITY_LCD_ENHANCED
DWIN_CREALITY_LCD_JYERSUI
DWIN_MARLINUI_PORTRAIT

Edit: In JyersUI Looks only zOffset is fixed maybe with https://github.com/MarlinFirmware/Marlin/commit/b93c3301f2146bed7287ad097b95b53507c35420 But the current Position (bottom from display) still wrong.

The-EG commented 3 years ago

Btw. Durring compiling i got

warning "Auto-assigned Z2 STEP/DIR/ENABLE_PINs to unused En_STEP/DIR/ENABLE_PINs."

I use dual Z

This is just an informational "warning," letting you know that it's assuming you want to use the next unused E driver for Z2. If that's the case you can just ignore.

thisiskeithb commented 3 years ago

23005 was merged. Closing.

tome9111991 commented 3 years ago

Merge will not fix DWIN_CREALITY_LCD Still wrong value for current Position and Zoffset

Is only for JyersUI fixed

thisiskeithb commented 3 years ago

Hopefully more of this UI code is shared in the future so we don’t have to keep fixing the same bug in multiple places.

github-actions[bot] commented 2 years ago

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.