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

Extrusion Rate Smoothing: Round speed values to prevent needless & trivial speed fluctuations #5911

Conversation

igiannakas
Copy link
Contributor

@igiannakas igiannakas commented Jun 28, 2024

Description

Currently extrusion rate smoothing outputs a float to the set speed command. However this results on occasion in trivial speed fluctuations midway through continuous speed segments like seen below. This may or may not have an effect on print output; however it should be addressed to eliminate redundant gcode commands.

image

By rounding the speed to whole numbers this issue is eliminated, while also allowing speed adjustments to continue to be emitted where the speed delta is more than 1mm/sec

image

@igiannakas igiannakas marked this pull request as ready for review June 28, 2024 13:54
@igiannakas igiannakas changed the title Extrusion Rate Smoothing: Round speed values to prevent needless speed fluctuations Extrusion Rate Smoothing: Round speed values to prevent needless & trivial speed fluctuations Jun 28, 2024
…ed-values-to-prevent-unecessary-minor-speed-fluctuations
Copy link
Owner

@SoftFever SoftFever left a comment

Choose a reason for hiding this comment

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

Nice, looks good

@SoftFever SoftFever merged commit b478104 into SoftFever:main Jun 29, 2024
12 checks passed
@igiannakas igiannakas deleted the Extrusion-Rate-Smoothing-Round-speed-values-to-prevent-unecessary-minor-speed-fluctuations branch June 29, 2024 14:17
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.

2 participants