luc-github / ESP3D-WEBUI

A Web UI for ESP8266 or ESP32 based boards connected to 3D printers / CNC
GNU General Public License v3.0
756 stars 305 forks source link

[BUG] Serial communication issue #353

Closed dryas closed 1 year ago

dryas commented 1 year ago

Describe the bug Whenever I try to get feedback via a GCODE it doesn't seem to work like it should be. On the machine info page for example as far as I know some info is collected via GCODE. When I open it, I only get a small "ERROR" message. When I try to run M114 I simply get an "ok" in the terminal. The position of XYZ are not updated automatically and as the response of M114 is only "ok" I also can't get it to work with automatic polling. On the other side M105 seems to work. M503 is also only returning an "ok". It seems like there is some issue in the serial communication (probably on line breaks?) or the parsing engine. Additional: M115 returns only one line, M119 also only an "ok" and no endstop info (see screenshot).

To Reproduce Steps to reproduce the behavior: Run M114 in the terminal, run M503 in the terminal. Response is only "ok", but no more data.

Expected behavior M114 returns the position of the extruder M503 returns the settings of the printer I can use M114 to update the XYZ positons on the jog panel.

Screenshots image

ESP3D Firmware: (please complete the following information)

Target Firmware: (please complete the following information)

Board used (please complete the following information):

Browser used (please complete the following information):

Additional context Few questions on that:

github-actions[bot] commented 1 year ago

Thank your for submiting, please be sure you followed template or your issue may be dismissed.

luc-github commented 1 year ago

did you enabled the polling commands M114;M105 in settings ?

dryas commented 1 year ago

In the Marlin or in the ESP3D settings? They are activated in Marlin. I haven't seen them in the ESP3D config. Let me have a look.

luc-github commented 1 year ago

in webui interface tab

dryas commented 1 year ago

Like this? Yes, they are added as "polling" commands. Temperature works, but not position (M114). image

luc-github commented 1 year ago

you can add like this or to existing one just separate by ;

luc-github commented 1 year ago

another solution is to enable auto report in Marlin so no need to do polling

dryas commented 1 year ago

Ah, good to know. Then I will probably combinate them later on. However, M114 does not update the positions on the Jog control and for example when I open the Machine tab I only get an error (but I'm not sure if this is already implemented in to the WebUI3.0).

Regards the auto report: Will it work with SD card uploads? I use the MKS protocol and I've read somewhere that this could be result in complications and interferences when commands get mixed in the communication.

image
luc-github commented 1 year ago

Yes it is implemented but may be not enabled in Marlin What is the result of M503 in webui terminal ?

luc-github commented 1 year ago

on MKS protocol yes it may be safer to not use autoreport , sorry I missed you used it

luc-github commented 1 year ago

if you use MKS protocol - I am guessing M503 / M114 will report nothing https://github.com/luc-github/ESP3D/discussions/620 Unless they fix the issue in Marlin

luc-github commented 1 year ago

@dryas issue be closed ?

dryas commented 1 year ago

@luc-github Yeah, I've investigated a little bit more and it seems like MKS has encapsulated the communication in their logic somehow. I've moved back to the traditional serial protocol without the possibilities to upload.

Thanks for all your hard work. You could be proud of what you have created!

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