Skip to content
This repository has been archived by the owner on Aug 27, 2023. It is now read-only.

No code M117 #305

Open
thomaskilian opened this issue Oct 1, 2018 · 9 comments
Open

No code M117 #305

thomaskilian opened this issue Oct 1, 2018 · 9 comments

Comments

@thomaskilian
Copy link

Is there a reason why M117 is not implemented? Cura does put them in the generated code. So why not just accept this M-code and skip the rest of the line? Shouldn't be too difficult, right? I'd do that, but I'd like to avoid diving deep into gcode_process.c and find out how it actually works (if possible). Any hint on how to implement that quickly?

@Wurstnase
Copy link
Collaborator

Wurstnase commented Oct 2, 2018

Because there is no complete support for displays in Teacup. Afaik M117 will show a message on a display. So there is acutally no reason to implement something which will do nothing.

Edit: Probably your start gcode or end gcode has some lines with M117 which could be easily removed.

@thomaskilian
Copy link
Author

thomaskilian commented Oct 2, 2018

I use, probably like anyone else, Octoprint to feed my 3D printer. Once an unknown opcode (like M117) is sent, Teacup echos the remainder of the current line one char after the other. That causes Octoprint to just hang and stop further printing. So rather than that I'd wish Teacup could just reply with some well-formed warning or to skip the opcode.

P.S. No, it's not start/end. Cura is putting that in when using a post-processing macro.

@Wurstnase
Copy link
Collaborator

Wurstnase commented Oct 2, 2018 via email

@thomaskilian
Copy link
Author

Looking into Marlin I just see "Error: \n". I'll check how Octoprint reacts on that and whether it probably accepts something like a warning. So you can save that effort here.

I'd be happy to just know how to skip until \n in the gcode_process.c.

@thomaskilian
Copy link
Author

I'm a bit confused now. The malicious M117 is preceded by a M107 where Teacup responds with "E: Bad M-code" and Octoprint happily continuing. It's probably that 107 has no extra parameters. So, why not add a skip to newline after that?

@phord
Copy link
Collaborator

phord commented Oct 3, 2018

I don't think I've seen this before. Also, teacup does process the entire g-code before determining if it's valid. So I'm confused. Maybe some parameter field isn't recognized or the message includes something that triggers us to bail. Can you provide the exact command that causes this failure?

@thomaskilian
Copy link
Author

thomaskilian commented Oct 3, 2018

Sure: M117 Printing... ch@L 1
It causes Teacup to echo chars enclosed in question marks. Can't post the result since I disassembled the Arduino at the moment.

@thomaskilian
Copy link
Author

Fixed my board. Here's the echo from Teacup upon the above command: ?R??I??I??C??H??@??L?ok

@thomaskilian
Copy link
Author

Uh! I think it was just because the code was compiled with DEBUG. That makes it echo unknown chars :-/

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants