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

Messages printed twice in terminal #916

Closed
hroncok opened this issue Mar 21, 2018 · 7 comments
Closed

Messages printed twice in terminal #916

hroncok opened this issue Mar 21, 2018 · 7 comments
Labels
2.x Non-breaking features and fixes suitable for inclusion in future Printrun 2.x minor releases Fix commited Not Regression

Comments

@hroncok
Copy link
Collaborator

hroncok commented Mar 21, 2018

On 610f9ff

Right panel:

Connecting...
start
Printer is now online.
echo:Marlin 1.1.0.11
echo: Last Updated: 2016-04-27 12:00 | Author: (Aleph Objects Inc., LulzBot Mini)
...

Terminal:

Connecting...
Connecting...
Printer is now online.
Printer is now online.
@hroncok hroncok added the 2.x Non-breaking features and fixes suitable for inclusion in future Printrun 2.x minor releases label Mar 21, 2018
@hroncok
Copy link
Collaborator Author

hroncok commented Mar 21, 2018

Or load a file, if you don't have a printer around:

Loading file: /envy/dj/rostock_max/razor-holder.gcode
Loading file: /envy/dj/rostock_max/razor-holder.gcode
Loaded /envy/dj/rostock_max/razor-holder.gcode, 29565 lines
Loaded /envy/dj/rostock_max/razor-holder.gcode, 29565 lines
3041.45mm of filament used in this print
3041.45mm of filament used in this print
The print goes:
The print goes:
- from -31.91 mm to 31.91 mm in X and is 63.83 mm wide
- from -31.91 mm to 31.91 mm in X and is 63.83 mm wide
- from -15.41 mm to 11.25 mm in Y and is 26.66 mm deep
- from -15.41 mm to 11.25 mm in Y and is 26.66 mm deep
- from 0.00 mm to 18.05 mm in Z and is 18.05 mm high
- from 0.00 mm to 18.05 mm in Z and is 18.05 mm high
Estimated duration: 120 layers, 0:52:29
Estimated duration: 120 layers, 0:52:29

@rockstorm101
Copy link
Collaborator

This is not a regression, it happened to me also with version 1.6.0.

@kliment
Copy link
Owner

kliment commented Mar 22, 2018

This should now be fixed in both 2.x and master - we were adding an extra console handler even though one had already been added, and not resetting the handlers when adding it.

@rockstorm101
Copy link
Collaborator

I can confirm this is now solved, I see no more duplicated messages after 38e6c34.

@kliment
Copy link
Owner

kliment commented Mar 22, 2018

@hroncok can you test this works correctly for you now and close if so?

@hroncok
Copy link
Collaborator Author

hroncok commented Mar 22, 2018 via email

@hroncok
Copy link
Collaborator Author

hroncok commented Mar 22, 2018

Confirmed.

@hroncok hroncok closed this as completed Mar 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.x Non-breaking features and fixes suitable for inclusion in future Printrun 2.x minor releases Fix commited Not Regression
Projects
None yet
Development

No branches or pull requests

3 participants