Page MenuHome

Fix T55542: BAT doesn't pack smoke cache
AbandonedPublic

Authored by Philipp Oeser (lichtwerk) on Jul 4 2018, 1:26 PM.

Details

Diff Detail

Event Timeline

Looks nice, although it's missing a unit test for the tracer and a unit test for the packer.

blender_asset_tracer/trace/modifier_walkers.py
212

Since the only difference between the calls is the extension, it's probably better to just have a little dict lookup, like:

extensions = {
  cdefs.PTCACHE_FILE_PTCACHE: cdefs.PTCACHE_EXT,
  cdefs.PTCACHE_FILE_OPENVDB: cdefs.PTCACHE_EXT_VDB
}

add unittests
use dict lookup for cache extension

Philipp Oeser (lichtwerk) marked an inline comment as done.Jul 5 2018, 1:38 PM
Philipp Oeser (lichtwerk) added inline comments.
tests/test_tracer.py
184

noticed path can be longer then 64 [if no name for the cache is provided, since hex blows it up]

Sybren A. Stüvel (sybren) accepted this revision.EditedJul 5 2018, 4:02 PM

lgtm, but maybe reduce the number of frames to 10.

This revision is now accepted and ready to land.Jul 5 2018, 4:02 PM

commited with less (10) cache frames in tests
rBAST4e1c2d1bbb72da1fc48134fb3bc46af75aec34fb