Marlin 2.1 Sensorless probing] (While building task, Errors in probe.cpp)

elongmugs commented 2 years ago

Did you test the latest `bugfix-2.1.x` code?

Yes, and the problem still exists.

Bug Description

I went to #define SENSORLESS_PROBING and this error comes while bulding task. error

Bug Timeline

new bug 03-07-2022

Expected behavior

I expected to be able to build task and use the sensorless probing just as in marlin 2.0

Actual behavior

It makes error about probe.cpp

Steps to Reproduce

remove the "//" from #define SENSORLESS_PROBING
remove the "//" from #define X_DRIVER_TYPE TMC2209
define Y_DRIVER_TYPE TMC2209

define Z_DRIVER_TYPE TMC2209

//#define X2_DRIVER_TYPE A4988 //#define Y2_DRIVER_TYPE A4988 //#define Z2_DRIVER_TYPE A4988 //#define Z3_DRIVER_TYPE A4988 //#define Z4_DRIVER_TYPE A4988

define E0_DRIVER_TYPE TMC2209

Configuration.zip

Version of Marlin Firmware

2.1

Printer model

FLsun G

Electronics

BIGTREE_SKR_PRO v1.1, TMC 2209

Add-ons

No response

Bed Leveling

MBL Manual Bed Leveling

Your Slicer

Cura

Host Software

Pronterface

Don't forget to include

[X] A ZIP file containing your Configuration.h and Configuration_adv.h.

Additional information & file uploads

Configuration.zip error

Bob-the-Kuhn commented 2 years ago

Confirmed (with a wrinkle)

As best I can tell there was an attempt to add SENSORLESS_PROBING to Delta printers but a couple of the variables/structures are no longer properly defined for non-Delta printers. test_sensitivity and offset_sensorless_adj are two I've run into so far.

I used Auto Build Marlin within VSCode to build using the last set of configuration files attached to the original post.

The first error I get is:

Marlin\src\module\probe.cpp:106:68: error: too many initializers for 'Probe::sense_bool_t'
  106 |   Probe::sense_bool_t Probe::test_sensitivity = { true, true, true };
      |                                                                    ^

This is immediately followed by the one shown in the author's screenshot.

Bob-the-Kuhn commented 2 years ago

Looks like the changes are part of PNR #24220.

Bob-the-Kuhn commented 2 years ago

Try using these files. They at least compile.

deleted: ZIP file

I have not tried compiling for Delta sensorless homing .

Bob-the-Kuhn commented 2 years ago

Updated ZIP of files I'd like tested. module2.zip

thisiskeithb commented 2 years ago

Closing since there’s now a PR: https://github.com/MarlinFirmware/Marlin/pull/24455

Bob-the-Kuhn commented 2 years ago

PR has been merged. Please try the latest bugfix-2.1.x.

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.

MarlinFirmware / Marlin