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

Feature request : more descriptive gcode comments #2865

Open
zeograd opened this Issue May 30, 2015 · 2 comments

Comments

Projects
None yet
2 participants
@zeograd

zeograd commented May 30, 2015

Hi,
in order to better integrate postprocessing scripts (like one I'm currently working on), I'd like to see more information leaked via the verbose gcode mode.

As of now, all perimeters are attached the comment ' ; perimeter' without details whether they are the most external shell (or not) nor whether they are shells for the outside of the shape or shells for the inside (for the holes in the model).

As I'd like to distinguish those different perimeter types, I've added the following trivial patch for figuring out which perimeter are external and not, and I assume that the first external perimeter from a layer is the outer one, non external perimeters up to the next external one are extra outer perimeters. But I think it would be more robust to generate gcode comments telling explicitely in which situation we are.

https://gist.github.com/zeograd/fd4fd0d4e67ecb1a463f

I propose to have those 4 comments in gcode:
; perimeter external outer
; perimeter outer
; perimeter external inner
; perimeter inner

(or any equivalent rewording).

I can try to work on a patch for doing so if pointed at the right direction to distinguish outer from inner shells.
Thanks in advance.

@lordofhyphens

This comment has been minimized.

Show comment
Hide comment
@lordofhyphens

lordofhyphens Jul 27, 2016

Member

I don't know about differentiating that far down, but I believe that it would be very helpful to at least call out the printing moves that are already covered/tweakable in settings for speed, etc.

So that would be:

  • external perimeters
  • small perimeters
  • other perimeters
  • infill
  • solid infill
  • top solid infill
  • support
    • skirt
    • brim
  • support interface
  • gap fill
  • bridges
  • travel
  • Move to layer

Most of these are already present.

Member

lordofhyphens commented Jul 27, 2016

I don't know about differentiating that far down, but I believe that it would be very helpful to at least call out the printing moves that are already covered/tweakable in settings for speed, etc.

So that would be:

  • external perimeters
  • small perimeters
  • other perimeters
  • infill
  • solid infill
  • top solid infill
  • support
    • skirt
    • brim
  • support interface
  • gap fill
  • bridges
  • travel
  • Move to layer

Most of these are already present.

@lordofhyphens lordofhyphens added this to the 1.3.1 milestone Jul 27, 2016

@lordofhyphens lordofhyphens self-assigned this Jul 27, 2016

@lordofhyphens

This comment has been minimized.

Show comment
Hide comment
@lordofhyphens

lordofhyphens Jul 27, 2016

Member

Related: #3132

Member

lordofhyphens commented Jul 27, 2016

Related: #3132

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment