Blender Git Commits

Feature Collada: Add support for Continuous Acceleration (Fcurve handles)

This allows us to create smoother FCurves during importing Collada
Animation curves. Possibly this should become an option ionstead of
a fixed import feature.

Refactor Collada: use BCKeyPoint to construct the exported FCurves

Before the handles where taken from the FCurves. But this caused
odd behavior when exporting Object hierarhcies with parent_inverse
matrices. Then the handles where calculated wrong. Now there wil be no
handles for those cases. (we might still consider to optionally
generate handles here (to be decided)

Refactor Collada: Moved check for animated curve one level up

This avoid to call the curve exporter for every curve,
regardless i it is actually exported or not.

Refactopr Collada: Changed BCValueMap to contain Values+curve handles

This makes it easiuer to later construct the export curves.
This may later be removed again in favor of storing the data
within the FCurves themself. (I am still somewhat undecided here)

More work on opti edge split

Refactor Collada: Changed BCAnimationCurve methods to const methods

Refactor Collada: Added new Class BCKeyPoint

Refactor Collada: Removed unused parameter from method

BCAnimationCurve::add_value(const float val, const int frame_index)

Fix Collada: Keep existing FCurves available in Sampler

The FCurves are needed when we export Keyframes. In that case
we want to preserve the tangent data. Hence we want to keep
the original FCurves available while exporting.

Collada refactor: Added meaningful Constructor for BCAnimationSampler

The processing was unnecessarily split into creating the Sampler
and initialising it later. This can safely be done in one go
within a Sampler constructor

Also removed the not needed reference to the Sampler inside
the AnimationExporter instance. The Sampler is now a simple
local variable in AnimationExporter::exportAnimations()

Cleanup Collada: Removed unneeded Constructor

Refactor Collada: Renamed Enumerations to avoid confusion

The Curve animation type enumeration looked almost identical
to the enumeration that describes if the exporter shall
export sample frames or keyframes.

Details

old names:
BC_ANIMATION_TYPE_SAMPLE,
BC_ANIMATION_TYPE_KEYS

replaced by the more descriptive names:
BC_ANIMATION_EXPORT_SAMPLES,
BC_ANIMATION_EXPORT_KEYS

Refactor Collada: moved Object sampling code into its own private method

Refactor: Moved unrelated class methods to collada_utils

Refactor: Renamed classes, updated comments for better reading

Cleanup collada: improved class description comment

Refactor Collada: Moved static class functions to collada_utils

Reason: The functions had nothing to do with the class

Cleanup Collada: moved typedefs to more intuitive locations

cleanup: indentation and class method declarations

Refactor Collada: Major rewrite of the AnimationExporter

* Mostly reorganized the classes and made them more readable.
* The animation sampler now can be used for any animation curve.
Before the sampler only looked at curves which are supported by
Standard Collada 1.4. However the Collada exporter currently
ignores all animation curves which are not covered by the 1.4.1
Collada Standards. There is still some room for improvements
here (work in progres)
* Removed some unnecessary dependencies and cleaned up many
functions which have become unnecessary and no longer used.
* moved functions between classes to better match the class
responsibilities.
* took care that the functions are kept small and understandable
* removed intermediate structures which are no longer needed
* cleanly separated the Animation sample part from the
Collada part so that they can be treated independently from each
other

Known issues:

* Some exports do currently not work reliably, among those
are the camera animations, material animations and light animations
those animations will be added back next (work in progres)

* Exporting animation curves with keyframes (and tangents)
sometimes results in odd curves (when parent inverse matrix is involved)
This needs to be checked in more depth.

* Export of "all animations in scene" is disabled because the
Collada Importer can not handle this reliably at the
moment (work in progres).