MarlinFirmware / Marlin

Marlin is an optimized firmware for RepRap 3D printers based on the Arduino platform. Many commercial 3D printers come with Marlin installed. Check with your vendor if you need source code for your specific machine.
https://marlinfw.org
GNU General Public License v3.0
16.22k stars 19.22k forks source link

Extruder heating message only shows 'S' on Full Graphics LCD status line #5944

Closed WheresWaldo closed 7 years ago

WheresWaldo commented 7 years ago

I am trying to use RCBugFix up to commit eeb4902 (May 2nd) and appear to have an issue that I am not sure is configuration or something else.

Bed heating shows the proper message of 'Bed Heating...' while the bed is coming up to temperature. As soon as it switches to hotend heating (there is only one extruder) the LCD status message changes to just the letter 'S' and not the 'E0 Heating...' that I expected. I have tried to go through Configuration.h with a fine tooth comb and cannot find any part of it that would change the way the message should read. All other functions of the printer appear to be working well.

The slicer is S3D and the start up gcode is always the same (except for temps obviously) and it looks like this:


M82
M106 S0
M140 S90
M190 S90
M104 S235 T0
M109 S235 T0
M82
G1 Z15
G21
G28
G29
G90
M117 Printing ...
ghost commented 7 years ago

Can you see if you have the same issue I had here: #5876?

WheresWaldo commented 7 years ago

You did a lot of testing let me try some of the same and let you know. This might be a duplicate issue. I'll get back to you in about an hour (my family is still sleeping).

benlye commented 7 years ago

Also looks like the same issue (but different message) as #5894. I get 'LLED.' when heating the extruder.

WheresWaldo commented 7 years ago

I haven't had a chance to test yet, but I just got a report that said someone else using the exact firmware and configuration had "h 1m 9089s" on the status line when heating switched to extruder (T0).

ghost commented 7 years ago

I did a partial reversion to get the message back until this is resolved.

In marlin_main.cpp, I commented out this line:

//if (code_value_temp_abs() > thermalManager.degHotend(target_extruder)) status_printf(0, "E%i %s", target_extruder + 1, PSTR(MSG_HEATING));

and inserted this line:

if (code_value_temp_abs() > thermalManager.degHotend(target_extruder)) LCD_MESSAGEPGM(MSG_HEATING);

Also commented out this line:

//if (code_value_temp_abs() > thermalManager.degHotend(target_extruder)) status_printf(0, "E%i %s", target_extruder + 1, PSTR(MSG_HEATING));

and inserted this line:

if (code_value_temp_abs() > thermalManager.degHotend(target_extruder)) LCD_MESSAGEPGM(MSG_HEATING);

This is temporary, but it does put the correct message of Heating... back on the status line.

thinkyhead commented 7 years ago

5948 should fix this.

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