Skip to content

Display engine plugins in gcode - #2364

Merged
HellAholic merged 3 commits into
mainfrom
CURA-11212_add_plugins_to_header
Aug 24, 2026
Merged

Display engine plugins in gcode#2364
HellAholic merged 3 commits into
mainfrom
CURA-11212_add_plugins_to_header

Conversation

@wawanbreton

Copy link
Copy Markdown
Contributor

Add the list of active engine plugins in the generated gcode. Syntax is based on the post-processing plugins: https://github.com/Ultimaker/Cura/blob/05e2c8937da42b48f8502f3ac69bcf8ed44b7307/plugins/PostProcessingPlugin/PostProcessingPlugin.py#L94

Result is something like:

;ENGINE PLUGINS
;  [valve (GCODE_PATHS_MODIFY)]

CURA-11212

@github-actions

github-actions Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Test Results

31 tests   31 ✅  5s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 996da95.

♻️ This comment has been updated with latest results.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NPM package is failing, I believe we should add #ifdef ENABLE_PLUGINS blocks

Comment thread src/gcode_export/gcodeExport.cpp
Comment thread src/gcode_export/gcodeExport.cpp
CURA-11212

Co-authored-by: Casper Lamboo <c.lamboo@ultimaker.com>

@rburema Remco Burema (rburema) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've got some remarks. I'd be willing to accept as-is if you can explain though :-)

Comment thread include/plugins/slotproxy.h
Comment thread src/gcode_export/gcodeExport.cpp
@HellAholic
HellAholic merged commit e731ce0 into main Aug 24, 2026
35 checks passed
@HellAholic
HellAholic deleted the CURA-11212_add_plugins_to_header branch August 24, 2026 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants