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] SanityCheck reports error when GRID_MAX_POINTS_X is set to 2 #26190

Closed
1 task done
JudithRichardson opened this issue Aug 20, 2023 · 3 comments
Closed
1 task done

Comments

@JudithRichardson
Copy link

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

Yes, and the problem still exists.

Bug Description

I am using a Lulzot Taz6. I am trying to update to the latest Marlin version (bugfix-2.1.x) from an older version (2.1.1). The Taz uses the nozzle as the probe, and needs to do bed levelling by probing on 4 washers in the corners of the bed. I do not have a BLTouch on this machine.
I need to use AUTO_BED_LEVELING_LINEAR to probe at these four specific positions. I have set the PROBING_MARGIN_* variables to specify these positions relative to the X/Y_MIN/MAX_POS and X/Y_BED_SIZE values.
In the older version setting #define GRID_MAX_POINTS_X 2 compiles without complaint, and the code runs fine on the Taz, probing at the intended 4 points.
In the bugfix-2.1.x SanityCheck.h has
#error "GRID_MAX_POINTS_[XY] must be between 3 and 255."
which kills the build

Bug Timeline

This seems to have changed some time between 2.1.1 and 2.1.2

Expected behavior

I expected it to continue to compile with GRID_MAX_POINTS_X 2

Actual behavior

Fails to compile

Steps to Reproduce

#define AUTO_BED_LEVELING_LINEAR
#define GRID_MAX_POINTS_X 2
try to compile in VSCode

Version of Marlin Firmware

bugfix_2.1.x

Printer model

Lulzbot Taz 6

Electronics

stock electronics

Add-ons

No response

Bed Leveling

ABL Linear grid

Your Slicer

Prusa Slicer

Host Software

Repetier Host

Don't forget to include

  • A ZIP file containing your Configuration.h and Configuration_adv.h.

Additional information & file uploads

I haven't included the full configuration files since you can reproduce this by just changing 2 lines in the distribution files

@ellensp
Copy link
Contributor

ellensp commented Aug 20, 2023

was added in #25831

@thisiskeithb
Copy link
Member

Fixed in #26199

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

No branches or pull requests

3 participants