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.3k stars 1.65k forks source link

[BUG] (Filament Sensor don't work in touch mode, if it is connected to the base board) #2004

Closed Ari-SSO closed 3 years ago

Ari-SSO commented 3 years ago

Description

The runout filament sensor do not function correctly in touch mode, my environment:

The problems appears only when i use the tocuh mode in the display (The mode that normaly I use).

Steps to reproduce

  1. Configure filament sensor in marlin firmware
  2. run M119 to verify thar the sensor detect correctly the ,presence or absence, of filament.
  3. Set the TFT to touch mode
  4. Set, in TFT, the runout filament sensor ON
  5. Print something with and without filament in the the printer (Always display a message reports that there is not filamente)
  6. Repeat 4 and 5, but with OFF and SMART option.

Expected behavior If there is filamente then print normaly, else stop printing, (This if you set the filamente sensor on, if off contiune printing without filament)

In touch mode, depending on runout filament sensor setting in TFT: Filament sensor ON: With or without filament, it always says that filament is missing and cannot continue printing. Filament Sensor OFF: It never detects whether or not it has filament (I guess this is correct). Filament Sensor SMART: It starts printing and after a few seconds, it says that there is no filament. When pressing "Resume" it continues but says again that there is no filament (This regardless of whether or not there is a filament). And the same thing always repeats.

Hardware Variant

Firmware TFT35 v3.0: BIGTREE_TFT35_V3.0.27.x.bin

TFT Firmware Version & Main Board Firmware details

BTT TFT35 v3.0 BTT SKR 1.3 Filamente sensor connected to SKR 1.3 board, PIN 1.29 Firmware TFT35 v3.0: BIGTREE_TFT35_V3.0.27.x.bin Firmware SKR 1.3: Marlin Bugfix (Version 2.0.9, according to the configuration.h file) BIGTREETECH-TouchScreenFirmware-Configurations.zip Marlin.zip

Additional Information

kisslorand commented 3 years ago

Put M75 in the start of your gCode file. If I had a cent for each time I wrote this sentence... Wouldn't it be easier to search around and check it it has been discussed or not? The TFT has no way to know if filament sensor connected to the motherboard is triggered or not unless someone (in our case Marlin on the motherboard) tells him so. Marlin is not transmitting filament sensor info unless there's a print ongoing. So Marlin doesn't know that the TFT is actually printing, not just sending gCodes, unless someone (in our case the TFT) tells him so. M75 tells Marlin that there's a print ongoing and Marling knowing this will send info about the filament sensor connected to it.

I hope it helps...

guruathwal commented 3 years ago

The filament sensor option in the TFT is only for connecting the sensor to the TFT itself. In your case, the sensor is connected to the mainboard so the filament sensor settings on the TFT are useless. you will have to control it through marlin mode or through gcodes.

oldman4U commented 3 years ago

We should add a note to the filament sensor part of config.ini that the Settings here have only to be activated in case the sensor is connected to the TFT and that you have to add M75 to the start gCode in case it is connected to the mainboard. Who wants to add this?

Ari-SSO commented 3 years ago

if you add M75 at start code, may be also add M77 at end code, for consistency. If you are OK, then i will add it in the config.ini...

oldman4U commented 3 years ago

Please do so. But please add a note and do not change the gCode itself.

Ari-SSO commented 3 years ago

In which branch i must add this? in develop branch?

Thanks!

oldman4U commented 3 years ago

You just make a PR. BTT merges it correctly.

Ari-SSO commented 3 years ago

it is done, i hope it is ok.

Thank to all of you!

bigtreetech commented 3 years ago

fixed in #2017, so close here

oldman4U commented 3 years ago

Thank you. I will change the order of the text and the text itself a bit and hope this is ok for you.

Happy printing

oldman

Ari-SSO commented 3 years ago

Yes, please!... I Know that my english is not the best of the world.

Thanks!

oldman4U commented 3 years ago

No problem at all. It was more the order of the information and to make it shorter. Check my PR.

Happy printing

github-actions[bot] commented 6 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.