Marlin: [BUG] [Marlin 2.0.x] LCD messages 3 Point bedleveling

Created on 13 Jul 2020 · 12Comments · Source: MarlinFirmware/Marlin

Graphic LCD Marlin version 20006

LCD messages 3 Point bedleveling.

Anet A6. No need to see the configfiles. It is not in there.
My screen is in Dutch. Meetpunt is Probing Point.

3 point bedleveling
LCD message:
Probing point 1: LCD message: Probing Point 0/3
Probing point 2: LCD message: Probing Point 1/3
Probing point 3: LCD message: Probing Point 2/3

it should be:
Probing Point 1/3
Probing Point 2/3
Probing Point 3/3
LCD

Source

roel8032

Most helpful comment

I've put up a pull request: https://github.com/MarlinFirmware/Marlin/pull/18639

Lord-Quake on 13 Jul 2020

👍2

All 12 comments

and what lcd do you have? we cant tell, because you didn't include your Configuration files. Could be stock, could be anything.
Please attach your configuration files.

ellensp on 13 Jul 2020

👍1

Your still missing the point, what if we need other settings? language? probe type? What version of marlin? and a million other questions.

We also want a configuration file that is full and complete so we can drop it in to marlin and replicate this issue.

Attach your config files! This is not optional.

ellensp on 13 Jul 2020

👍1

@roel8032
I just tested it on my Ender 3 in UBL configuration, latest bug fix and it shows it correctly (1/3, 2/3 and 3/3) on the screen.

So.... post the information as requested by @ellensp. It won't kill you if you want to be helped here.

If I were you I'd install the latest bug fix version before I'd do anything else.

Lord-Quake on 13 Jul 2020

OK.
But UBL (Unified Bed Leveling), is not AUTO_BED_LEVELING_3POINT
I just downloaded the latest. The same problem.

roel8032 on 13 Jul 2020

I took a look at the code:

We might have to take a closer look at the following in G29.ccp and add i+1 to the string output:

    if (verbose_level) SERIAL_ECHOLNPAIR("Probing point ", int(i), "/3.");
    TERN_(HAS_DISPLAY, ui.status_printf_P(0, PSTR(S_FMT " %i/3"), GET_TEXT(MSG_PROBING_MESH), int(i)));

Lord-Quake on 13 Jul 2020

Thanks. Now it is working OK

I changed this in:
MarlinsrcfeaturebedlevelablG29.cpp

line689

//TERN_(HAS_DISPLAY, ui.status_printf_P(0, PSTR(S_FMT " %i/3"), GET_TEXT(MSG_PROBING_MESH), int(i)));
TERN_(HAS_DISPLAY, ui.status_printf_P(0, PSTR(S_FMT " %i/3"), GET_TEXT(MSG_PROBING_MESH), int(i+1)));

The old code above i commented.

Thank you.

lcd meetpunt_ok

roel8032 on 13 Jul 2020

and what lcd do you have? we cant tell, because you didn't include your Configuration files. Could be stock, could be anything.
Please attach your configuration files.

roel8032 on 13 Jul 2020