Page MenuHome

BVH importer improvements
ClosedPublic

Authored by Sybren A. Stüvel (sybren) on Nov 7 2015, 2:52 PM.

Details

Summary

Errors are now reported to the user via Operator.report(), instead of
either being printed to the terminal or raised as an exception.

Updating the scene is now performed in separate functions, to make it
easier to control execution flow in the case of broken BVH files.

Diff Detail

Repository
rBA Blender Add-ons
Branch
arcpatch-D1609

Event Timeline

Sybren A. Stüvel (sybren) retitled this revision from to BVH importer improvements.
Sybren A. Stüvel (sybren) updated this object.

Generally seems fine, but would rather report be an optional argument. Theres some convention of having load(context, filepath) for importers already, and all other args as optional.

Sybren A. Stüvel (sybren) edited edge metadata.

Made 'report' arg optional, defaulting to 'print'

Made filepath arg non-optional, and following args keyword-only

This revision is now accepted and ready to land.Nov 7 2015, 3:32 PM
This revision was automatically updated to reflect the committed changes.