Skip to content

Conversation

@ronaldex
Copy link

i'm not using preheating of the tools, but want to use prime-lines. Currently the macro depends on the tools being preheated.

The change accepts the params already used in PRINT_START to be passed to make the PRIME_LINES macro so it can be used without preheating.

In PRINT_START (and related macros) it can be now called with the rawparams: PRIME_LINES {rawparams}

@TheSin-
Copy link
Collaborator

TheSin- commented Sep 29, 2025

This will break for those passing rawparams from print start cars can’t be changed like this, use logic from default print start and assume the same params are used.

@ronaldex
Copy link
Author

I don’t think I completely understand you. By calling it with {rawparams}, I meant that I followed the standard PRINT_START parameters as described in the StealthChanger documentation:

https://github.com/DraftShift/StealthChanger/wiki/Slicers

Of course, anyone can define or pass their own parameters to PRINT_START.

description: Prime tools before printing.
  Tx_TEMP= Temperature per tool to prime
  TOOL= Initial tool number, optional.

As far as I can tell, the script is currently copied (not symlinked) during initial installation and isn’t called by default, since it’s part of the extra-macros.

@TheSin-
Copy link
Collaborator

TheSin- commented Sep 29, 2025

Correct keep the Param names the same

@ronaldex
Copy link
Author

If you are referring to this from a backwards compatibility perspective, the macro invocation has already changed regardless, as the tools to be primed must now be explicitly specified.

As a fallback, I could consider defaulting to the preheated tools when no temperature values are provided for the individual tools?

If you mean that I should follow the parameters from the PRINT_START documentation, that is what I have done.

@TheSin-
Copy link
Collaborator

TheSin- commented Sep 29, 2025

I’ll just fix it when o have time I’ll keep this open to sort it

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.

3 participants