tomvandeneede / p2pp

Palette2/3 Post Processing tool for Prusa Slicer
143 stars 40 forks source link

Erroneous variable layer height at 0.xx5mm #159

Open bartrv opened 1 year ago

bartrv commented 1 year ago

Using Prusa Slicer 2.5.0: When using a layer height of 0.075mm I get the std. variable layer height warning even when Printer Settings "Enable variable layer height feature" is unchecked, and [Print Settings](Support Material) synchronize with object layers is checked, AND 'Generate support material" is turned off . Changing to 0.070mm or 0.050mm removes the warning but .070 causes layer consistency/step issues and .050 takes to long to print. Looking at the code... mcf.py line 309, 310 and the try block at line 350 -> when using 0.070 the "try" yields {0,0,...}, when using 0.075 it yields {0,7,0,7,...}

I propose changing: lv, flh, olh and the associated calculations from 100 to 1000, and +0.001 to +0.0001 eg... lv = INT(lv +0.001)100) - flv --> lv = INT(lv +0.0001)1000) - flv If these values are used elsewhere, perhaps only modifying the add and multiplier locally in the variable_layer_height check is sufficient. Since the MK4 has 0.9 steppers, finer layer height increments may start to be a consideration.

tomvandeneede commented 1 year ago

Will have a look, can you send me the unprocessed code file for this print for testing reference

On 11 Apr 2023, at 19:28, bartrv @.***> wrote:

Using Prusa Slicer 2.5.0: When using a layer height of 0.075mm I get the std. variable layer height warning even when Printer Settings <x-msg://2/General> "Enable variable layer height feature" is unchecked, and [Print Settings](Support Material) synchronize with object layers is checked, AND 'Generate support material" is turned off . Changing to 0.070mm or 0.050mm removes the warning but .070 causes layer consistency/step issues and .050 takes to long to print. Looking at the code... mcf.py line 309, 310 and the try block at line 350 -> when using 0.070 the "try" yields {0,0,...}, when using 0.075 it yields {0,7,0,7,...}

I propose changing: lv, flh, olh and the associated calculations from 100 to 1000, and +0.001 to +0.0001 eg... lv = INT(lv +0.001)100) - flv --> lv = INT(lv +0.0001)1000) - flv If these values are used elsewhere, perhaps only modifying the add and multiplier locally in the variable_layer_height check is sufficient. Since the MK4 has 0.9 steppers, finer layer height increments may start to be a consideration.

— Reply to this email directly, view it on GitHub https://github.com/tomvandeneede/p2pp/issues/159, or unsubscribe https://github.com/notifications/unsubscribe-auth/AK6CHJLCRCTRLUTRBB25YHTXAWIDFANCNFSM6AAAAAAW2SJB3A. You are receiving this because you are subscribed to this thread.

bartrv commented 1 year ago

070 and 075 unprocessed gcode and mcfx attached p2pp_2023.01Test_F1F2_025x075_Unp.zip 075_Unprocessed.zip p2pp_2023.01Test_F1F2_025x070_unp.zip 070_Unprocessed.zip

tomvandeneede commented 1 year ago

Will be addressed in 9.0.2