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

Add intent profiles for Method High Speed #20129

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

alanbjorklund
Copy link
Contributor

These intent modes were tuned by Eric MacNeil. The jerk & accel values will be converted into a accel_config by the Makerbot writer plugin, and the Method firmware will interpret these values for each path.

The self support speed settings were unchanged from the previous defaults, but the model speed was increased for ABSR, PLA & TPLA.

PP-544

New intent modes include:

ABSR High Speed (MX, MXL)
ABSR High Speed Solid (MX, MXL)
PLA High Speed (M, MX, MXL)
PLA High Speed Solid (M, MX, MXL)
TPLA High Speed (M, MX, MXL)
TPLA High Speed Solid (M, MX, MXL)

These changes were tested extensively in the New York printer lab, as well as by Alan in New Jersey. The most commonly observed failure mode was self support under-extrusion. This failure mode was eliminated by reverting the self support settings to the previous defaults.

These intent modes were tuned by Eric MacNeil.  The jerk &
accel values will be converted into a accel_config by the
Makerbot writer plugin, and the Method firmware will interpret
these values for each path.

The self support speed settings were unchanged from the previous
defaults, but the model speed was increased for ABSR, PLA & TPLA.

PP-544
Copy link
Contributor

github-actions bot commented Jan 14, 2025

Test Results

23 466 tests  +23 466   23 464 ✅ +23 464   45s ⏱️ +45s
     1 suites +     1        2 💤 +     2 
     1 files   +     1        0 ❌ ±     0 

Results for commit 8fca5d9. ± Comparison against base commit 8a4e7f1.

♻️ This comment has been updated with latest results.

Copy link
Member

@rburema rburema left a comment

Choose a reason for hiding this comment

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

Looks good to me, as long as you're sure that the alterations made to the ..._base_... file really are OK for the entire Method-series (also when not using the high-speed profiles).

Another(?) really small nitpick is that in the interface, some profiles now shows up as 'Fast - Highspeedsolid' (funnily enough it would be totally valid if written in Dutch, since we have compound words like that). I'm OK with keeping it like this at least for now, but I thought you might like to know.

@alanbjorklund
Copy link
Contributor Author

alanbjorklund commented Jan 22, 2025

@rburema The settings in the base profile are the current defaults assigned via the firmware. The current defaults get overridden only when the high speed / high speed solid intents are selected. The lack of a space in the names "highspeed" & "highspeedsolid" is something we noticed, but there was no obvious solution so we didn't consider this a blocker for release.

@rburema
Copy link
Member

rburema commented Jan 22, 2025

@alanbjorklund Thanks, that makes sense. I agree, not a blocker.

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