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

pa calib: print flow value and acceleration #7178

Merged
merged 3 commits into from
Oct 24, 2024

Conversation

buzzhuzz
Copy link
Contributor

Print flow value and acceleration for PA pattern calibration. This should help keep track during adaptive PA calibration.

Description

Modify Pattern PA calibration to print current Flow Rate and Acceleration values which should help following Adaptive PA calibration guidelines.

Those values printed below PA values:

  • last line is Acceleration
  • second to last line is Flow Rate

Screenshots/Recordings/Graphs

Screenshot from 2024-10-21 17-09-01

Tests

Manual tests with several speed ratios: checked printed flow rate matches one reported by gcode preview.

Print flow value and acceleration for PA pattern calibration.
This should help keep track during adaptive PA calibration.
@igiannakas
Copy link
Contributor

NICE! thank you for this, that's fantastic!

@kylek29
Copy link

kylek29 commented Oct 22, 2024

I do like this, especially for those of us with Bambu printers because they don't give us any of this information.

@SoftFever
Copy link
Owner

Hi @buzzhuzz
Nice one, this is very handy 👍
I have two pieces of feedback:

  1. Could we round the flow rate value to one decimal place?
  2. With the number rounded as suggested in 1, could we make the box thinner?
Screenshot 2024-10-24 at 12 10 52 AM

Rework pa pattern legent generation to correctly estimate
width of the legend section.

Current flow value now has variable length which is a longest of
PA value and the acceleration for a given test.

Few examples:
1. PA value are 4 characters:        0.04
   Acceleratioion 3 chars:           400
   Flow value will be 4 chars long:  7.98, or 11.3

2. PA:      0.018
   Accel:   1000
   Flow:    12.35, or 6.345

3. PA:      0.04
   Accel:   15000
   Flow:    34.34, or 4.567

Rework number-to-string conversion flow to correctly round values at
given precision.
@buzzhuzz
Copy link
Contributor Author

@SoftFever, thanks for your feedback.

I have found a bug in previous implementation which causes legend section to be too-wide in some cases.
Additionally, flow value precision now trimmed to not exceed length of PA value and acceleration strings.

For example:
Screenshot from 2024-10-24 11-58-14

Screenshot from 2024-10-24 11-59-02
Screenshot from 2024-10-24 11-59-55

@SoftFever
Copy link
Owner

@SoftFever, thanks for your feedback.

I have found a bug in previous implementation which causes legend section to be too-wide in some cases. Additionally, flow value precision now trimmed to not exceed length of PA value and acceleration strings.

Perfect!
Thanks :)

@SoftFever SoftFever merged commit 983f1a8 into SoftFever:main Oct 24, 2024
15 checks passed
@buzzhuzz buzzhuzz deleted the dbuzz/pa-pattern-print-flow-accel branch October 24, 2024 21:37
@malventano
Copy link

I do like this, especially for those of us with Bambu printers because they don't give us any of this information.

Note for BBL printer users: I've been doing lots of Adaptive PA testing on an X1C, and it appears BBL is doing something in FW which interferes with some of the intention of what Adaptive PA attempts to accomplish. Measurements and calibration at a fixed speed track as you would expect, but this is static. If I change layer height (to change flow), but also change speed (to return flow back to its original value), I get a very different PA result. Since the current Adaptive PA does not take speed as an input, any prints at speeds other than where the Adaptive PA cal runs were performed will be printed with 'inaccurate' PA, since the BBL FW is adding in its own compensation above and beyond the requested PA value.

This was off by enough for me to go back to 'normal' PA on my X1C as it was getting better results for prints of differing speeds.

@seedeeze510
Copy link

I'm new to 3D printing, and I'm just trying to learn as much as I can. I have a filament where I set the Max Volumetric Speed to 8 mm³/s and the Pattern shows 7.92, which I thought is close enough. But when I set the Max Volumetric Speed to 35 mm³/s, the Pattern shows 16.2. So I'm curious, where does this PA calibration get the flow rate value?

@igiannakas
Copy link
Contributor

I'm new to 3D printing, and I'm just trying to learn as much as I can. I have a filament where I set the Max Volumetric Speed to 8 mm³/s and the Pattern shows 7.92, which I thought is close enough. But when I set the Max Volumetric Speed to 35 mm³/s, the Pattern shows 16.2. So I'm curious, where does this PA calibration get the flow rate value?

Speed of print and layer height. Increasing either should give you a higher flow. Also make sure your machine has enough acceleration to actually reach those speeds within the test distance - from start of the line till the corner.

this is a useful calculator for acceleration: https://blog.prusa3d.com/calculator_3416/

@seedeeze510
Copy link

I'm new to 3D printing, and I'm just trying to learn as much as I can. I have a filament where I set the Max Volumetric Speed to 8 mm³/s and the Pattern shows 7.92, which I thought is close enough. But when I set the Max Volumetric Speed to 35 mm³/s, the Pattern shows 16.2. So I'm curious, where does this PA calibration get the flow rate value?

Speed of print and layer height. Increasing either should give you a higher flow. Also make sure your machine has enough acceleration to actually reach those speeds within the test distance - from start of the line till the corner.

this is a useful calculator for acceleration: https://blog.prusa3d.com/calculator_3416/

Thank you for the clear answer and also for the calculator link. Cheers!

@rolox7
Copy link

rolox7 commented Nov 22, 2024

Big noob here, where is this thing?

Thanks!!

@seedeeze510
Copy link

It's in Prepare > Calibration (the one on top, not the one inline with Prepare) > Pressure advance
image

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.

7 participants