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

Read total length from .end_of_bowden_to_sensor when loading to nozzle #91

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Imnotkelby
Copy link

Had an issue ( #82 ) where ERCF would fault when loading filament from reverse bowden to the toolhead sensor, but shortly after would say "Filament loaded successfully". Thought this might have been a timing issue initially, but I was able to add the end_of_bowden_to_sensor variable to the load length and this stopped the fault from happening.

@EtteGit
Copy link
Owner

EtteGit commented Apr 7, 2022

I don't mind updating this code line, but I don't think that is an issue per se.

The move prior to this (i.e. the move of the ercf and extruder in sync, for a length of 'printer["gcode_macro _ERCF_VAR"].end_of_bowden_to_sensor|float - 7', is done from the "end of bowden" position. Hence, in principle, you end up at 7mm from the toolhead sensor. The next move, i.e. the ERCF_HOME_EXTRUDER, needs an upper range. Using 30.0 gives already 23.0 mm of "room", which is already higher than the possible "slack" from the encoder resolution and global loading move uncertainty.

That being said, I don't mind upping this 30.0 mm upper value, but I don't think the 'printer["gcode_macro _ERCF_VAR"].end_of_bowden_to_sensor|float - 7' is better suited.

I propose a value of 50.0 mm, would that work in your specific case?

@Imnotkelby
Copy link
Author

Thank you for the response and the explanation of how this feature works. I spent a few hours trying to understand it, then did some more testing over the last few days. It seems like my calibrated distance from the ERCF to the extruder is off. Got one cart to work fine, but having troubles with another.

For my situation, I was able to make it work with 40mm.

@EtteGit
Copy link
Owner

EtteGit commented Jun 17, 2022

Hi. I'm catching up on github stuff. Do you still see this issue or can I close this?

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