fix T53839: better default for curve resolution in abc import
AbandonedPublic
Actions

Authored by Philipp Oeser (lichtwerk) on Jan 30 2018, 5:17 PM.

Details

Reviewers

Summary

rB5bdbc88ab81891eff0 introduced a user property for curve resolution.
When this is not set (e.g. from other software) imported curves defaulted to a resolution of 1.
This is now changed to a resolution of 12 (matching blenders default for new curves)

Diff Detail

Repository: rB Blender

Event Timeline

Philipp Oeser (lichtwerk) created this revision.Jan 30 2018, 5:17 PM

Philipp Oeser (lichtwerk) mentioned this in T53839: .abc Loads On Ubuntu Fails To Load On Windows 7.Jan 30 2018, 5:20 PM

Also see e12c31100. Guessing for default values is a tricky thing, especially since Alembic doesn't seem to have a standardised property for the curve resolution (I'm gladly proven wrong on this one). Having resolu=12 is fine for a simple case like T53839, but brings down Blender to a crawl when importing animated hair from another package.

Hi @Sybren A. Stüvel (sybren) , seeing the wider scope (case of hair), I guess current behaviour (default resolution of 1) seems indeed the wiser choice... user can always up the resolution by hand...
Will abandon this then and also archive T53839...

Revision Contents
Changeset List

Path

Size

source/

blender/

alembic/

intern/

abc_curves.cc

2 lines

Diff 9905

View Options

source/blender/alembic/intern/abc_curves.cc

fix T53839: better default for curve resolution in abc importAbandonedPublicActions