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

[BUG] MKS TFT_LVGL_UI does not support relative extrusion #27318

Closed
1 task done
andrademf opened this issue Jul 28, 2024 · 3 comments · May be fixed by #22595
Closed
1 task done

[BUG] MKS TFT_LVGL_UI does not support relative extrusion #27318

andrademf opened this issue Jul 28, 2024 · 3 comments · May be fixed by #22595

Comments

@andrademf
Copy link

andrademf commented Jul 28, 2024

Did you test the latest bugfix-2.1.x code?

Yes, and the problem still exists.

Bug Description

TFT_LVGL_UI MKS implementation has its own filament runout logic, and using that combined with relative extrusion generates issues when resuming the printing.

My main guess for the root cause of the issue is the usage of G90/G91 to lift the Z axis when going to the park position, causing the running gcode to lose its references for extrusion when resuming.

image
(OrcaSlicer option to used relative extrusion)

Bug Timeline

No response

Expected behavior

Printing interruption might not change current movement configuration

Actual behavior

The extruder runs backwards when resume the printing, in some cases even ejecting the filament from the extruder

Steps to Reproduce

  1. Setup a printer with TFT_LVGL_UI
  2. Use relative extrusion on the slicer
  3. Start a print
  4. Trigger the filament runout sensor
  5. Restablish the filament sensor
  6. Press "Resume"

Version of Marlin Firmware

Bugfix-2.1.x

Printer model

Two Trees SP5

Electronics

MKS Robin Nano V3.1

LCD/Controller

MKS TFT 3.5

Other add-ons

MKS Wifi

Bed Leveling

None

Your Slicer

Other (explain below)

Host Software

SD Card (headless)

Don't forget to include

  • A ZIP file containing your Configuration.h and Configuration_adv.h.

Additional information & file uploads

Configuration.zip

gcodes.zip

Suggestion

Instead of using relative movement to lift the nozzle before moving, the current Z position, which is available right above, can be used to calculate the lifted height. In this way, it is not requried to use G90/G91, which causes the side effect of also resetting the relative extrusion defined at the beggining of the gcode.

image

@thisiskeithb
Copy link
Member

MKS' LVGL UI needs a lot of work in all kinds of areas, but we have an open PR that will address filament runout:

@andrademf
Copy link
Author

Thanks @thisiskeithb!

Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Sep 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants