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

[BUG] Compile Error Z_PROBE_OFFSET_RANGE #22662

Closed
sarvenn opened this issue Aug 29, 2021 · 7 comments
Closed

[BUG] Compile Error Z_PROBE_OFFSET_RANGE #22662

sarvenn opened this issue Aug 29, 2021 · 7 comments

Comments

@sarvenn
Copy link

sarvenn commented Aug 29, 2021

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

Yes, and the problem still exists.

Bug Description

27th Aug bugfix 2 compile error.
I added the terminal error messeges below. It seems to be due to source code files.
compile error.txt

Bug Timeline

new

Expected behavior

Successful compile

Actual behavior

Unsuccesful compile

Steps to Reproduce

Using both Autobuild Marlin and Platformio build task have the same error results

Version of Marlin Firmware

bugfix2 27th Aug

Printer model

Ender 3 pro

Electronics

BTT SKR Mini E3 V2 & TFT35 E3 V3

Add-ons

No response

Bed Leveling

MBL Manual Bed Leveling

Your Slicer

Prusa Slicer

Host Software

No response

Additional information & file uploads

No response

@ellensp
Copy link
Contributor

ellensp commented Aug 29, 2021

You need matching Configuration.h files eg it must have this line
https://github.com/MarlinFirmware/Marlin/blob/bugfix-2.0.x/Marlin/Configuration.h#L1248

@sarvenn
Copy link
Author

sarvenn commented Aug 29, 2021

I'm not even using a Z probe. Here are my config files.
config files.zip

@ellensp
Copy link
Contributor

ellensp commented Aug 29, 2021

you have
//#define Z_PROBE_OFFSET_RANGE_MIN -20
//#define Z_PROBE_OFFSET_RANGE_MAX 20
You have #define MESH_BED_LEVELING which uses these defines
Enabled these defines.

@sarvenn
Copy link
Author

sarvenn commented Aug 29, 2021

I also tried for STM32F103RE_btt. Result is the same with STM32F103RC_btt

Problem refers to menu_bed_leveling.cpp file.

@sarvenn
Copy link
Author

sarvenn commented Aug 29, 2021

I think I found the reason of the problem. By default these line were inactive.
I enabled these commands and compiled again. It is successful now.
I think this should be enabled by default.
Actually I use the ender 3 pro skr mini e3 v2 config files and do some minor changes. manual MBL was enabled by default in that config. So Z probe offset range should be enabled by default as well. May be this can be added to sanity check as well. If any of the mesh bed leveling is enabled then z probe offset range also should be enabled. I don't have knowledge about coding. I'm sure you will find the best solution to overcome this error.

// For M851 give a range for adjusting the Z probe offset
#define Z_PROBE_OFFSET_RANGE_MIN -20
#define Z_PROBE_OFFSET_RANGE_MAX 20

@ellensp
Copy link
Contributor

ellensp commented Aug 29, 2021

PR recreated. closing.

@ellensp ellensp closed this as completed Aug 29, 2021
@github-actions
Copy link

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 Oct 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants