-
Notifications
You must be signed in to change notification settings - Fork 80
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
Implement FR - Issue#302 #305
Implement FR - Issue#302 #305
Conversation
Start the Bed Mesh Validation Pattern with one of the Full Circle pads instead of the front left corner
Implement [FR]#301
Increase the amount primed from 2mm to 15mm before drawing the Bed Mesh Validation pattern
Same remark as in #304, use bed size divided by half. 🙃 |
Here, I did not know how to submit two changes that are totally independent of each other, modify the same line of code, but may not both be approved. If the actual change is ok to commit as is, we could update the #301 implementation to start from that new baseline, to achieve the same result. Would that be ok? |
As I review the changes, though, I don't see the edit to line 80 that I made in the request I thought I submitted. 😳 Still learning how this GitHub works and my O'Reilly guidebook is way out of date... I have updated the file in the pull request branch, to fix that. |
Assumes #301 is approved first. Otherwise, need to back-out that change before committing this one.
Is this PR still relevant now #304 got merged? |
Yes please. This one asks for a longer prime before the pattern starts. I think it actually proposes to revert a change you made earlier, so you may have a reason to challenge this one. I have depth perception issues up close to the nozzle and am not quick enough to snag that drool before the printer drags it into the pattern, with a priming of only 2mm…. If I forget or fail anyway at 15mm, it gives me a bigger blob to grab for, as it makes the tour of the bed. 🤪 |
Apologies, I only focused on the duplicate BED_SIZE in the gcode that causes a conflict. |
Have I done the PR incorrectly? You did ask me to change this PR to use BED_SIZE / 2, but I did that before the other PR was merged into extui. I don’t see a conflict now, but if GitHub is seeing a conflict, what is it that Ihaveto change, to resolve that? |
Description
Increase the amount primed from 2mm to 15mm before drawing the Bed Mesh Validation pattern.
Add a prompt for the user to remind them to get their tweezers ready, to remove the drool and the primed amount, before the extruder begins drawing the pattern.
Implement this change as follows:
Modify line 72 of MeshValidationHandler.cpp to replace P2 with P15
Modify line 80 of MeshValidationHandler.cpp to add "Prepare tweezers" to the "Starting..." message.<!--
Requirements
No special requirements.
Benefits
See Issue #302.
Configurations
Use the CF6.1-Final configs.
Related Issues
#302