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

Implement FR - Issue#302 #305

Closed
wants to merge 7 commits into from
Closed

Implement FR - Issue#302 #305

wants to merge 7 commits into from

Conversation

Thinkersbluff
Copy link

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

Start the Bed Mesh Validation Pattern with one of the Full Circle pads instead of the front left corner
Increase the amount primed from 2mm to 15mm before drawing the Bed Mesh Validation pattern
@Thinkersbluff Thinkersbluff changed the title Implement fr issue#302 Implement FR - Issue#302 Jan 30, 2022
@Sebazzz
Copy link
Collaborator

Sebazzz commented Jan 30, 2022

Same remark as in #304, use bed size divided by half. 🙃

@Thinkersbluff
Copy link
Author

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?

@Thinkersbluff
Copy link
Author

Thinkersbluff commented Jan 30, 2022

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.
@Sebazzz
Copy link
Collaborator

Sebazzz commented Feb 3, 2022

Is this PR still relevant now #304 got merged?

@Thinkersbluff
Copy link
Author

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. 🤪

@Sebazzz
Copy link
Collaborator

Sebazzz commented Feb 3, 2022

Apologies, I only focused on the duplicate BED_SIZE in the gcode that causes a conflict.

@Thinkersbluff
Copy link
Author

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?

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants