Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] The M115 command does not work #17922

Closed
radek8 opened this issue May 8, 2020 · 11 comments
Closed

[BUG] The M115 command does not work #17922

radek8 opened this issue May 8, 2020 · 11 comments

Comments

@radek8
Copy link
Contributor

radek8 commented May 8, 2020

The M115 command does not work in the current version of Marlin bugfix 2.0

if allowed
#define M115_GEOMETRY_REPORT

related to the repair #16741

@thisiskeithb
Copy link
Member

Please follow the Issue Template when filing a bug report so we don't have to hunt for information.

### Description

<!-- Description of the bug or requested feature -->

### Steps to Reproduce

<!-- If this is a Bug Report, please describe the steps needed to reproduce the issue -->

1. [First Step]
2. [Second Step]
3. [and so on...]

**Expected behavior:** [What you expect to happen]

**Actual behavior:** [What actually happens]

#### Additional Information

* Include a ZIP file containing your `Configuration.h` and `Configuration_adv.h` files.
* Provide pictures or links to videos that clearly demonstrate the issue.
* See [How Can I Contribute](#how-can-i-contribute) for additional guidelines.

@radek8
Copy link
Contributor Author

radek8 commented May 9, 2020

A device that communicates via the serial port and calls the M115 command, for example the TFT display, does not work when the function is enabled
#define M115_GEOMETRY_REPORT

Expected behavior:

M115 dis

Actual behavior:

M115 eror

@sjasonsmith
Copy link
Contributor

Post your configs.

@thisiskeithb
Copy link
Member

Please follow the Issue Template.

@radek8
Copy link
Contributor Author

radek8 commented May 9, 2020

Post your configs.

no need to send configuration
by commenting #define M115_GEOMETRY_REPORT
output M115 is OK
@guruathwal should check if the settings work as intended.

@sjasonsmith
Copy link
Contributor

Every issue should have configs posted. Not posting configs requires someone to waste their time creating their own configs which may or may not exhibit the same issue.

If you won’t spend the time to post your configs, why should someone else spend their time trying to help?

@radek8
Copy link
Contributor Author

radek8 commented May 9, 2020

Configuration

config.zip

@sjasonsmith
Copy link
Contributor

OK, I've reproduced it. That is some pretty hilarious output. Far more entertaining than I expected.

@sjasonsmith
Copy link
Contributor

There was an infinite loop in some recently added print macros. I've posted a PR to fix it.
@radek8 if you need to use this feature you can look at the PR linked above and easily implement the change yourself.

@sjasonsmith
Copy link
Contributor

@radek8 the fix for this has been merged into bugfix-2.0.x. Please try it out and close the issue once you verify it is fixed.

@github-actions
Copy link

github-actions bot commented Jul 9, 2020

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.

@github-actions github-actions bot locked and limited conversation to collaborators Jul 9, 2020
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

4 participants