PixarAnimationStudios / OpenUSD

Universal Scene Description
http://www.openusd.org
Other
5.99k stars 1.18k forks source link

1GB inputs to usdzip generate 2GB usdz file #686

Closed jcowles closed 5 years ago

jcowles commented 5 years ago

Description of Issue

Subject says it all.

Steps to Reproduce

  1. Localize files into a directory
  2. cd into the directory
  3. run "usdzip Volund.usdz ."

First problem, usdzip attempts to zip the temp usdz file: usdzip Volund.usdz . Skipping empty file '.\Volund.vhdmao'.

When complete, the generated usdz file is 2GB, but the source files are 1GB total. Let me know if you want my actual source files.

System Information (OS, Hardware)

Windows 10

Package Versions

USD 18.11

Build Flags

--openimageio --alembic

drwave commented 5 years ago

Sounds like an awesomely pathological alignment bug...

Sent from my iPad

On Nov 3, 2018, at 7:13 PM, Jeremy Cowles notifications@github.com wrote:

Description of Issue

Subject says it all.

Steps to Reproduce

Localize files into a directory cd into the directory run "usdzip Volund.usdz ." First problem, usdzip attempts to zip the temp usdz file: usdzip Volund.usdz . Skipping empty file '.\Volund.vhdmao'.

When complete, the generated usdz file is 2GB, but the source files are 1GB total. Let me know if you want my actual source files.

System Information (OS, Hardware)

Windows 10

Package Versions

USD 18.11

Build Flags

--openimageio --alembic

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

jcowles commented 5 years ago

Here's the full source: https://drive.google.com/file/d/1LKpTCtulCfXXolYO4tNVQ8gPUTW3S955/view?usp=sharing

spiffmon commented 5 years ago

This was due to duplicate archiving, which is fixed in our dev tree, and I believe was the result of commit 9bfcf499715c7b08fabff99959e38e83d7ab20d1 .

Jeremy I'd like to note that your give usdzip command-line won't produce a legit usdz archive... when not using the "--asset" or "--arkitAsset" flags, you must provide the root/default usd file as the first file following the usdz file. I successfully made an archive like so:

usdzip Volund.usdz VolundModel.usda *

We should definitely document this better.

--spiff --spiff

On Sat, Nov 3, 2018 at 7:31 PM Jeremy Cowles notifications@github.com wrote:

Here's the full source:

https://drive.google.com/file/d/1LKpTCtulCfXXolYO4tNVQ8gPUTW3S955/view?usp=sharing

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/PixarAnimationStudios/USD/issues/686#issuecomment-435636951, or mute the thread https://github.com/notifications/unsubscribe-auth/AF7qaNxQV4AMTN79YA4tO2D3kg1GO5uUks5urlFrgaJpZM4YNGUm .

jtran56 commented 5 years ago

Filed as internal issue #USD-4897.

c64kernal commented 5 years ago

Hey @jcowles , we're going to close this out based on @spiffmon comment. Let us know if you still run into it in dev. -George & Spiff

jcowles commented 5 years ago

Thanks guys!