only export keyframes for node animations

[paroj]

This function processess the objects animation frame by frame, insted of using its keyframes

Im not fully happy with this. While I understand the need of it, there are serious drawbacks.

• what are the FPS of the animation? 24 as the blender default? This might or might not do what the user expects.
• by getting rid of the keyframes, there are no keyframes to easily modify the animation in code
• unrolling the animation in .skeleton is less evil as that is a binary format not edited by humans. .scenes on the other hand seem to be designed to be human readable. But with unrolling animations you end with something like this:
  

I think in this case, I would just for once advocate for a user-switch (default-off):

• only write node keyframes

(we can also merge this as is and add that later)

Originally posted by @paroj in #86 (comment)

[oldmanauz]

The code for this feature exists, it's commented out though.
Was it disabled for a reason?
Do you want it re-enabled?

[paroj]

Was it disabled for a reason?

yes, see: #86 (comment)

[sercero]

@oldmanauz I was the one who implemented this feature.

I was unable to come up with a solution on how to export the keyframes, it didn't seem as simple as exporting every frame.

[oldmanauz]

@sercero I see your logic. Is it better to have a well documented part-implementation that might suit some than potentially never having a solution?

[paroj]

I would prefer having that part implementation

[sercero]

@oldmanauz, @paroj: of course I have no problem with more options, mostly my concern is that the option to export every keyframe is still present.

Also, I acknowledge that there are cases when exporting only the keyframes is enough, so it is useful of course to have that option as well.

Thanks!