mriscoc / Ender3V2S1

This is optimized firmware for Ender3 V2/S1 3D printers.
Other
2.51k stars 345 forks source link

G92 doesn't behave preserve physical limits and physical mesh offsets #1312

Open lagnat opened 1 month ago

lagnat commented 1 month ago

Did you test with a precompiled firmware?

Yes, and the problem still exists.

Bug Description

Bug Timeline

No response

Expected behavior

No response

Actual behavior

No response

Steps to Reproduce

These are my observations (steps to reproduce)

Version of Professional Firmware

FIRMWARE_NAME:Marlin 2.1.3 MRiscoC Ender3V2-422-BLTUBL-T13-MPC, based on bugfix-2.1.x (Jan 27 2024 14:31:38) SOURCE_CODE_URL:github.com/mriscoc/Ender3V2S1 PROTOCOL_VERSION:1.0 MACHINE_TYPE:Ender3V2-422-BLTUBL-T13-MPC EXTRUDER_COUNT:1 UUID:cede2a2f-41a2-4748-9b12-c55c62f367ff

Printer model

3V2

Electronics

4.2.2

LCD/Controller

No response

Other add-ons

No response

Bed Leveling

UBL Bilinear mesh

Your Slicer

None

Host Software

OctoPrint

Additional information & file uploads

No response

mriscoc commented 1 month ago

This behavior can be expected when NO_WORKSPACE_OFFSETS is enabled as it is by default in this firmware. This firmware implements variable physical limits as a better approach to keep leveling accuracy. https://github.com/mriscoc/Ender3V2S1/wiki/How-to-set-the-home-offsets

Physical limits can be set via menu or using custom G-codes
https://github.com/mriscoc/Ender3V2S1/wiki/Custom-C-gcodes

lagnat commented 1 month ago

My feeling is that the mesh should respect G92 even when home offsets are disabled. I'd even go so far as to say that you should disable G92 if your intention is to simplify coordinate calculations however it is my hope that you accept this as a bug and consider determining the level of effort to fix it.

Thanks!

Edit: You said workspace offsets but I misunderstood that as home offsets.