Does anyone here work on Superslicer?
-
Just FYI for anyone interested, the Prusa Team seems to have taken over the development of this feature. They have already changed a couple things:
The string to identify the thumbnail seems to have been changed to
thumbnail_OQI
Also, there will now be an option in settings to select thumbnail format: PNG, JPG, and QOI. It won't be automatically selected for RRF (unless we intervene/request that change).
So, basically, it looks like this will make it into stock PrusaSlicer, but be aware of the possible changes.
https://github.com/prusa3d/PrusaSlicer/commit/87cff55856ae14096cd57f8e2542f1f1caef7167
-
Coming soon to a PanelDue near you...
-
Very cool! Then the investment in a 7-inch Due panel paid off after all...
-
@dc42 quick check in to align:
- about image size and resolution: what did you settle with now?
- whats the expected header comment you want to parse? I think PrusaSlicer devs used a different format than our original proposal with @bot and my Cura plugin?
; thumbnail_QOI begin 32x32 1234
vs.; QOI thumbnail begin 32x32 1234
-
-
@dc42 https://github.com/Duet3D/DuetSoftwareFramework/blob/75a1a4083062c6fbe0a5801685ab3371ede0b8b8/src/DuetControlServer/Files/ImageProcessing/PrusaSlicerImageParser.cs#L74 and https://github.com/Duet3D/RepRapFirmware/blob/5adb96401f7d8f6e79658ff8211a18785a73cb36/src/Storage/FileInfoParser.cpp#L977 tell two different stories for 3.4-rc1...
-
@resam thanks, we'll change DSF to use the new string too.
-
@resam is the Cura plugin for QOI thumbnail images available yet? If it is then I will mention it in 3.4.0rc1 release announcement.
-
@dc42 could you test this generated file - to make sure it works, before I submit the plugin to Ultimaker for review: debug.gcode
I still don't know which image resolutions you want, so right now it contains:
- 64x64
- 128x128
- 160x160
- 256x256
Ultimaker Marketplace Review usually takes a few days, but you can alway install the plugin in manually with these instructions https://github.com/Kriechi/Cura-DuetRRFPlugin/#manual-installation
You can add that to the release announcement now. -
@resam there's a problem with it, the start comment should be
; thumbnail_QOI begin
instead of
; thumbnail_OQI begin
and the end comment should read
; thumbnail end
instead of
; thumbnail_OQI end
Then it should work as expected.
-
@chrishamm lol - yeah, good idea of mine to ask for a sanity check Thanks for catching that typo!
I'll ask again about resolutions: what do you want, or what can you/DCW/RRF/DSF handle? I read that PanelDue can do a max. of 160x160.
-
@resam said in Does anyone here work on Superslicer?:
yeah, good idea of mine to ask for a sanity check Thanks for catching that typo!
Actually that typo was made by the devs of PrusaSlicer; but they later corrected it.
I don't have a latest build of PrusasSlicer, so I don't know whether they use thumbnail_end or thumbnail_QOI end.
-
@dc42 indeed! https://github.com/prusa3d/PrusaSlicer/commit/086662b6a1c7d7a14e35a90e6d46445f6871ac24 fixes it - so I probably copy-pasted it.
The end tag is the same tag as the begin, so
thumbail_QOI
. See https://github.com/prusa3d/PrusaSlicer/commit/87cff55856ae14096cd57f8e2542f1f1caef7167#diff-e09699df9f3fa4c292ba50b9f88dffe95b2d2d5c0a58d9d81441f9da5031c97bR49 -
@resam thanks for that. We'll need to change RRF and probably DSF then.
-
@dc42 ok - my Cura plugin behaves the same way as PrusaSlicer.
-
@dc42 please also update the code comment at https://github.com/Duet3D/RepRapFirmware/blob/09bf6d01b6028c7d7d162f0e2b60ca8a07849901/src/Storage/FileInfoParser.cpp#L959-L961
and am I reading RRF constant
MaxThumbnails = 3
correctly, that the fourth thumbnail will not be parsed by RRF? My Cura plugin currently outputs 4 sizes... so again my question about resolutions? -
@resam That limitation applies to RRF but not DSF. For the DWC filelist the ideal resolution is 48x48px and there is no real upper limit for previews. For PanelDue the maximum thumbnail height appears to be 6 x rowTextHeight. That means we have effective height limits of 126px for small displays and 192px for larger displays.
-
For PanelDue the recommended image size is 160x160 pixels. That size works on all three screen sizes.
-
@dc42 said in Does anyone here work on Superslicer?:
ecause QOI for
I work with ideamaker and have the possibility to add content of data into gcode file. Thumbnail 640x640 or any other value.
Would be nice if it would be used on your side.
thx
Deiter -
@icy_snake for PanelDue the optimum size is 160x160. DWC can use a wider variety of sizes, however RRF can only use the first 3 thumbnail images in the file.