3MFConsortium / spec_core

3MF's Core specification
BSD 2-Clause "Simplified" License
55 stars 16 forks source link

Specify requirement on ZIP64 and streaming extension #44

Closed bubnikv closed 3 years ago

bubnikv commented 3 years ago

ZIP64 and streaming extensions are required to consume, ZIP64 should only be produced if necessary.

hejllukas commented 3 years ago

I attach some examples for ZIP64 and streaming extension. Example 1. - No-ZIP64_No-Streaming.3mf.zip - ZIP without ZIP64 and also without streaming extension. Example 2. - No-ZIP64_Streaming.3mf.zip - ZIP without ZIP64 but with streaming extension. Example 3. - No-ZIP64_Streaming_Prealocated.3mf.zip - ZIP without ZIP64 but with streaming extension. Also, it contains preallocated a phony custom block (only for the file 3D/3dmodel.model) with an ID 0x9999 in the Extra field of the Local file header for possible enabling ZIP64 extensions while the file is compressed. Example 4. - ZIP64_No-Streaming.3mf.zip - ZIP with ZIP64 extension but without streaming extension. ZIP64 extension is forced for all files that even don't need ZIP64 extension, Example 5. - ZIP64_Streaming.3mf.zip - ZIP with ZIP64 extension and also with streaming extension. Both these extensions are used simultaneously only for the file 3D/3dmodel.model.

jordig100 commented 3 years ago

The files contain a relationship to a missing thumbnail, so the file is incorrect.

Could you please update the files just adding the thumbnail? Thanks

hejllukas commented 3 years ago

Sorry, my bad, I didn't notice it. Thanks for reporting it. I updated the previous post with re-uploaded ZIPs files with the thumbnail.