Animation loop doesn't work with imported 3D scene

[guilhermefelipecgs]

Godot version:
9423f23

OS/device including version:
Linux 4.17.8

Issue description:
"Animation Looping" doesn't work when you have an "imported" animation player like this:

Steps to reproduce:
Import an 3D animation from Blender using better collada plugin;
Enable Loop Animation;
Press F5;

Minimal reproduction project:
Loop Test.zip

[karroffel]

If you use BetterCollada you can name your animation TheAnimation-loop and it will be imported with loop enabled.

The problem is that imported animations are read-only. You can change them in the editor, but they are only changed in memory and not on disk, so as soon as you start running the game, a new process is spawned which reads data from disk.

This should be better documented and maybe a "copy-on-write" could be made for resources?

[reduz]

If you ask them to save to disk, new tracks added and loop info will be kept on reimport. Maybe it should be the default behavior.

…

On Tue, Jul 31, 2018, 06:08 Thomas Herzog ***@***.***> wrote: If you use BetterCollada you can name your animation TheAnimation-loop and it will be imported with loop enabled. The problem is that imported animations are read-only. You can change them in the editor, but they are only changed in memory and not on disk, so as soon as you start running the game, a new process is spawned which reads data from disk. This should be better documented and maybe a "copy-on-write" could be made for resources? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#20467 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AF-Z24ZfUJwE2Y_9X9qwHJ7lCkGU_FSsks5uMB6jgaJpZM4Vg7vS> .

[guilhermefelipecgs]

If you use BetterCollada you can name your animation TheAnimation-loop and it will be imported with loop enabled.

Didn't work for me, I open an issue with a demo godotengine/collada-exporter#82.

[myuniquename]

I have same issue with imported with glb animation file. I don't think it's it's collada specific issue.

The workaround is:

• create new animation player in the scene
• copy-paste animation from imported animation player to the new one
• in the animation tree use newly created animation player (not the one imported from glb)

[Sl3dge78]

After a bit of digging around I found out that there already exists a nice warning that pops up if the flag "track_set_imported" is set on the animation track. The thing is that this flag only gets set when choosing "Files" as Storage option in the importer.
Which I think is the opposite of what we want because changes get populated properly on anim files stored outside of the original one.
Working on a fix for this...