Skip to content

[BUG] Z_SAFE_HOMING_X_POINT no longer allowed outside of print bed. #16943

@marciot

Description

@marciot

Currently if the Z_SAFE_HOMING_X_POINT or Z_SAFE_HOMING_Y_POINT is outside of the print bed, as it is on LulzBot TAZ printers, the assertion will fail. This regression was introduced by commit 0fd0554. I do not think it makes sense to have Z_SAFE_HOMING be restricted to points within the bed. In the case of the TAZ, it needs to reach a button besides the print bed, and in the case of the LulzBot Bio, it is used to position the needle outside of the bed so that it can the axis deep enough to trigger a Z_MIN without crushing the needle.

Instead of:

#if ENABLED(Z_SAFE_HOMING)
  static_assert(WITHIN(Z_SAFE_HOMING_X_POINT, X_MIN_BED, X_MAX_BED), "Z_SAFE_HOMING_X_POINT can't be reached by the nozzle.");
  static_assert(WITHIN(Z_SAFE_HOMING_Y_POINT, Y_MIN_BED, Y_MAX_BED), "Z_SAFE_HOMING_Y_POINT can't be reached by the nozzle.");
#endif

I propose:

#if ENABLED(Z_SAFE_HOMING)
  static_assert(WITHIN(Z_SAFE_HOMING_X_POINT,X_MIN_POS, X_MAX_POS), "Z_SAFE_HOMING_X_POINT can't be reached by the nozzle.");
  static_assert(WITHIN(Z_SAFE_HOMING_Y_POINT,Y_MIN_POS, Y_MAX_POS), "Z_SAFE_HOMING_Y_POINT can't be reached by the nozzle.");
#endif

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions