Updated bed mesh configuration for Creality Ender 3 S1 and S1 Pro

[Auxx]

Existing mesh_min values start probing too far away from the edge.

Additionally mesh_max Y value of 197 is out of bed movement range (197 + bltouch.y_offset = 237.5) and causes Y stepper to overhit, which can potentially damage the printer in the long run.

Signed-off-by: Aleksandr Ivanov aux@hexmode.org

[github-actions]

Thank you for your contribution to Klipper. Unfortunately, a reviewer has not assigned themselves to this GitHub Pull Request. All Pull Requests are reviewed before merging, and a reviewer will need to volunteer. Further information is available at: https://www.klipper3d.org/CONTRIBUTING.html

There are some steps that you can take now:

1. Perform a self-review of your Pull Request by following the steps at: https://www.klipper3d.org/CONTRIBUTING.html#what-to-expect-in-a-review
   If you have completed a self-review, be sure to state the results of that self-review explicitly in the Pull Request comments. A reviewer is more likely to participate if the bulk of a review has already been completed.
2. Consider opening a topic on the Klipper Discourse server to discuss this work. The Discourse server is a good place to discuss development ideas and to engage users interested in testing. Reviewers are more likely to prioritize Pull Requests with an active community of users.
3. Consider helping out reviewers by reviewing other Klipper Pull Requests. Taking the time to perform a careful and detailed review of others work is appreciated. Regular contributors are more likely to prioritize the contributions of other regular contributors.

Unfortunately, if a reviewer does not assign themselves to this GitHub Pull Request then it will be automatically closed. If this happens, then it is a good idea to move further discussion to the Klipper Discourse server. Reviewers can reach out on that forum to let you know if they are interested and when they are available.

Best regards,
~ Your friendly GitIssueBot

PS: I'm just an automated script, not a human being.

[Auxx]

I reviewed this change according to the steps in the CONTRIBUTING document and everything looks good to me

[KevinOConnor]

Thanks. @Fisheiyy - do you have any comments on this change?

Separately, commits need to have a signed-off-by line for me to commit them - see: https://www.klipper3d.org/CONTRIBUTING.html

-Kevin

[Fisheiyy]

Thanks. @Fisheiyy - do you have any comments on this change?

Separately, commits need to have a signed-off-by line for me to commit them - see: https://www.klipper3d.org/CONTRIBUTING.html

-Kevin

Looks fine to me

[github-actions]

Unfortunately a reviewer has not assigned themselves to this GitHub Pull Request and it is therefore being closed. It is a good idea to move further discussion to the Klipper Discourse server. Reviewers can reach out on that forum to let you know if they are interested and when they are available.

Best regards,
~ Your friendly GitIssueBot

PS: I'm just an automated script, not a human being.

[Sineos]

@Auxx - any update?

[Auxx]

@Sineos not sure I can provide any updates. I'm not a Klipper developer, not part of the team, I don't have rights to do anything with PR. I can't assign it to myself, I can't approve it, I can't merge it. So it's up to Klipper devs to merge it.

[Bobo1239]

@Auxx You need to add your Signed-off-by: line to the actual commit and not just the pull request description. You can do so using git commit --amend --signoff. Afterwards you can force-push to the branch using git push --force.

[Sineos]

Thanks @Bobo1239
@Auxx see @KevinOConnor's comment.

[github-actions]

Unfortunately a reviewer has not assigned themselves to this GitHub Pull Request and it is therefore being closed. It is a good idea to move further discussion to the Klipper Discourse server. Reviewers can reach out on that forum to let you know if they are interested and when they are available.

Best regards,
~ Your friendly GitIssueBot

PS: I'm just an automated script, not a human being.

[Sineos]

@KevinOConnor

• Sign off is there (at least in the PR text)
• Reviewed and approved by @Fisheiyy

As a minor change, seems good to go.

[KevinOConnor]

Thanks. I merged this PR. Sorry that I didn't see the updated signed-off-by line earlier.

-Kevin