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

Sovol SV06 pre-set shaper frequencies #1098

Merged

Conversation

marbocub
Copy link
Contributor

@marbocub marbocub commented Sep 3, 2024

Requirements

Description

The #1088 enabled input shaper. But shaper frequencies in the config are still the Marlin's default of 40.0Hz. These are not SV06 hardware specific frequencies.

The vibration frequency profile of X-axis measured by an accelerometer on the head is:
image

The vibration frequency profile of Y-axis measured by an accelerometer on the bed is:
image

The results indicate X resonant is around 55-57Hz and Y resonant is around 43-45Hz.

The results came from a stock hardware directly on a wood desk. Of course, someone may get different results from different hardware settings. However, in the Internet, it sometimes found similar results with aboves, e.g. https://insertnewline.com/blog/SV06-klipper-input-shaping/.

I think the current setting 40Hz is far from the real X axis resonant 55Hz. This draft pre-sets shaper frequencies for X and Y axes of SV06 based on above results.

note:
Most config examples have the shaper frequencies of 40.0Hz. I found only the following two configs have pre-set frequencies.

  • Sunlu/T3 (X:30.0, Y:40.0)
  • Two Trees/SP-5_V3 (X:55, Y:55)

I'm not sure whether it's good idea to pre-set. But I would be happy if the frequencies was pre-set when I install a new version of Marlin for the first time. Therefore I created this draft.

Benefits

Related Issues

@thinkyhead thinkyhead marked this pull request as ready for review September 5, 2024 19:10
@thinkyhead thinkyhead merged commit b805767 into MarlinFirmware:import-2.1.x Sep 5, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants