bigtreetech / BIGTREETECH-TouchScreenFirmware

support TFT35 V1.0/V1.1/V1.2/V2.0/V3.0, TFT28, TFT24 V1.1, TFT43, TFT50, TFT70
GNU General Public License v3.0
1.32k stars 1.65k forks source link

TFT35 freezes before Purging #2895

Closed MrKuskov closed 8 months ago

MrKuskov commented 9 months ago

Description

Biqu B1 Printer

Steps to reproduce

I am sending a file for printing flowrate_0_PLA_7m36s.zip all G-code commands pass normally until the M117 Purging command line 291 in the file. Then the screen freezes. If you send it to print from Marlin mode, then the file is printed. But it is not possible to go back to "Touch mode". The mode does not switch when holding the knob. The same happens when printing any files.

Expected behavior

Actual behavior

Hardware Variant

GD_TFT35_V3.0_B1_V27.x ### TFT Firmware Version & Main Board Firmware details

The TFT firmware version from February 20th. From February 17, it worked fine. Marlin 2.1.2.2

Additional Information

digant73 commented 9 months ago

If you are on the latest TFT firmware, do you have the new setting command_checksum enabled on TFT's config.ini file? eventually try to disable it (although it should not be related to the issue) and let me know the result. Thanks

When you start a print from a mode (e.g Marlin mode with serial_always_on enabled on config.ini) you cannot move to the other mode (e.g. Touch mode) and viceversa. You have to wait the print is completed or aborted.

EDIT: I tested your gcode file on two printers without any issue. Before M117 Purging gcode the nozzle temp needs to reach its target. After it was reached, the print started and terminated without any issue

MrKuskov commented 9 months ago

If you are on the latest TFT firmware, do you have the new setting command_checksum enabled on TFT's config.ini file? eventually try to disable it (although it should not be related to the issue) and let me know the result. Thanks

Yes. The function is available, I tried to turn it on and off in the screen settings. my screen settings via the link

EDIT: I tested your gcode file on two printers without any issue. Before M117 Purging gcode the nozzle temp needs to reach its target. After it was reached, the print started and terminated without any issue

Yes, in Marlin mode, I also type without problems. Printing freezes after line G1 X6.1 Y20 Z0.3 F5000.0; Move to start position line 290 in the code. When the hotend starts moving to the specified line, the screen stops responding. The "M117 Purging" line is no longer being executed. The command is executed without problems via Pronterface.

MrKuskov commented 9 months ago

I found out that printing is interrupted in "Touch mode" only if the card is installed in the USB or SD of the screen. Printing is normal when installed in the SD board.

digant73 commented 9 months ago

@MrKuskov try with another SD card, if not already done. I have no issue at all on my printers (5 prints without any problem)

MrKuskov commented 9 months ago

I flashed the version from January 23rd. https://github.com/MrKuskov/BIGTREETECH-TouchScreenFirmware/blob/2ae78153bf685e901e504a04a88d9b6021450057/Copy%20to%20SD%20Card%20root%20directory%20to%20update/BIGTREE_GD_TFT35_V3.0_B1.27.x.bin Everything works correctly in it.

digant73 commented 9 months ago

@MrKuskov please could you verify if the attached fw is properly working? You need to add the new command_checksum setting in config.ini file

BIGTREE_GD_TFT35_V3.0_B1.27.x.zip

MrKuskov commented 9 months ago

No, it doesn't work. The screen froze when the hotend moved to the starting point, but the hotend executed three more commands: completed two cleaning lines and moved to the center of the table. enabling or disabling the checksum command does not affect the operation. The screen freezes anyway

digant73 commented 9 months ago

@MrKuskov ok thanks, I will continue to investigate. Could you please attach your config.ini file? so I will try to use it

MrKuskov commented 9 months ago

I have made a fork from your repository and am making changes to it for both the marlin firmware and the screen firmware https://github.com/MrKuskov/BIGTREETECH-TouchScreenFirmware/blob/master/Copy%20to%20SD%20Card%20root%20directory%20to%20update/config.ini

digant73 commented 9 months ago

I have made a fork from your repository and am making changes to it for both the marlin firmware and the screen firmware https://github.com/MrKuskov/BIGTREETECH-TouchScreenFirmware/blob/master/Copy%20to%20SD%20Card%20root%20directory%20to%20update/config.ini

ok. I will use your config.ini file to try to replicate the issue

digant73 commented 9 months ago

@MrKuskov Even with your config.ini file I have no issue. the only thing I saw is that command M280 at line 282 was skipped by Marlin (I received the Unknow Command message) due it is not configured/supported on my printer.

MrKuskov commented 9 months ago

I don't think it's about config.ini, I use default config.ini and the screen freezes on the firmware from February 20th. Now I'm back on the January 17th firmware and everything is working correctly. When flashing between versions, I use only a bin file with the same config.ini. The only thing is that when I upgrade to the old version, I get one error about the missing cheksum. Maybe the screen theme is to blame for the error? I'm using The Round Miracle Theme

digant73 commented 9 months ago

@MrKuskov thanks for the clarifications. Every details is very important to replicate a bug and so to find and fix the issue. Yes the last version (you have the issue with) added only the new setting command_checksum. Is the working fw from January 17 or February 17? I will try to use also Round Miracle Theme (I'm currently on Unified Menu Material theme). Also are you using a language pack (e.g. loaded language_de.ini) or you are on hardcoded english language?

MrKuskov commented 9 months ago

Sorry I made a mistake. The firmware version is from January 23rd. This one. I use the ru language pack. But I tried switching to the en version. The screen freezes the same way.

digant73 commented 9 months ago

ok, understood. If possible please update to latest version (merged today Feb-23). If you still have the issue, try to disable, one by one, LED Event feature and Comment parsing feature and let me know if without one of this feature the issue is no more present. thanks

MrKuskov commented 9 months ago

@digant73 Yes!!! The Event Led function is to blame for the screen freezes. When it is turned off, everything works correctly. If the function is enabled during printing, the screen freezes immediately.

digant73 commented 9 months ago

ok thanks. Do you have a LED strip configured in your printer or a simple LED on the hotend?

MrKuskov commented 9 months ago

Multi-color RGB lights from Biqu B1

digant73 commented 9 months ago

could you let me know which Marlin settings reported at the beginning of config.ini file (on section Supported Marlin Firmware Versions) are enabled and disabled on your Marlin?

MrKuskov commented 9 months ago

I have enabled all the settings specified at the beginning of the config.ini file in my Marlin Configuration.h Configuration_adv.h except for Z_STEPPER_AUTO_ALIGN. I have only one axis so far.

MrKuskov commented 9 months ago

If the function Event LED is enabled during printing, the screen freezes immediately.

But if I go to the LED menu, I can turn on any backlight colors and the screen works correctly.

digant73 commented 9 months ago

Ok. I will check on LED Event code

digant73 commented 9 months ago

please try if the issue is still present with the attached fw (enable again LED Event feature). Try also with PID process.

BIGTREE_GD_TFT35_V3.0_B1.27.x.zip

MrKuskov commented 9 months ago

@digant73 everything is working correctly. very good. thanks

digant73 commented 9 months ago

ok. but remember that I commented out just a piece of code (the color on the knob is not set at the end of the process) to have a confirmation of the issue. I will provide a PR with the fix. Leave this bug report open, it will be automatically closed once the PR is merged. Thanks for support.

EDIT: Please if possible let me know if the attached fw is ok (that should fix the issue).

BIGTREE_GD_TFT35_V3.0_B1.27.x.zip

MrKuskov commented 8 months ago

EDIT: Please if possible let me know if the attached fw is ok (that should fix the issue).

Everything works.

digant73 commented 8 months ago

thanks. The fix was also merged in the official btt fw just today.

github-actions[bot] commented 5 months ago

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.