Does anyone here work on Superslicer?

[resam]

@bot @dc42 on the Cura side, I am in full control of the plugin and therefore the full slicer, so we have all the freedom and flexibility, but I would like us to try hard and keep feature parity with @bot 's work in PrusaSlicer (and others).

We could make a "thumbnail negotiation" between RRF and the slicer. The slicer could query any information from the printer and generate+assemble the perfect thumbnails (is PanelDue connected? which screen size is available? only DWC in PNG needed?) -- as compared to a static config in the Slicer that is unaware of what the printer is capable of.

[giostark]

OT/ team at work , such a nice reading /OT

[bot]

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:

@dc42

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

9 bubnikv committed to prusa3d/PrusaSlicer

WIP: Implemented support for QOI G-code thumbnail format as requested
by the RepRapFirmware team due to their low RAM budget for decompression.
Bundled the QOI image compression library.

[dc42]

Coming soon to a PanelDue near you...

[medicusdkfz]

@dc42

Very cool! Then the investment in a 7-inch Due panel paid off after all...

[resam]

@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]

@resam I think PrusaSlicer uses

thumbnail_QOI begin

so we'll look for that.

[resam]

@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...

[dc42]

@resam thanks, we'll change DSF to use the new string too.

[dc42]

@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.

[resam]

@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.

[chrishamm]

@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.

[resam]

@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.

[dc42]

@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.

[resam]

@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

0 bubnikv committed to prusa3d/PrusaSlicer

Fixed typo in thumbnail_QOI tag.

9 bubnikv committed to prusa3d/PrusaSlicer

WIP: Implemented support for QOI G-code thumbnail format as requested
by the RepRapFirmware team due to their low RAM budget for decompression.
Bundled the QOI image compression library.

[dc42]

@resam thanks for that. We'll need to change RRF and probably DSF then.

[resam]

@dc42 ok - my Cura plugin behaves the same way as PrusaSlicer.

[resam]

@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?

[chrishamm]

@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.

[dc42]

For PanelDue the recommended image size is 160x160 pixels. That size works on all three screen sizes.