kaosat-dev / Blenvy

Bevy Code & Blender addon for a simple workflow to add & edit Bevy components in Blender
Other
617 stars 57 forks source link

gltf_auto_export v0.13, does not export animated objects correctly #147

Closed kaosat-dev closed 9 months ago

kaosat-dev commented 9 months ago

@janhohenheim this is for the issue that you found with the export of the animated objects Btw, I am experiencing some very weird things: image

janhohenheim commented 9 months ago

@kaosat-dev can confirm. I don't know why, but the fox is only exported when you right click the "Fox" collection and "select Objects" image which results in this gltf test.zip which can be opened in https://gltf-viewer.donmccurdy.com/ successfully.

Using only the "root", I also get an empty gltf.

kaosat-dev commented 9 months ago

@janhohenheim Thanks for the double check! I am not 100% sure it is related, I need to double check the Blender 4.0 release notes. It might have worked until now because of the hack-ish way I was doing things before. (not the first time that a hack works better than clean code in Blender ...)

So this issue is driving me nuts: the best I managed so far is this: image

Sorry for the inconvenience, once again !

janhohenheim commented 9 months ago

Well, I just started a first person project, so until I need moving NPCs I'm fine, hehe

kaosat-dev commented 9 months ago

@janhohenheim haha, all is good then ;) Joke's on me, I forgot to enable exporting animation in my automated tests :D I will double check things, but I think it should be solved, PR very soon

kaosat-dev commented 9 months ago

@janhohenheim the fix & related tests are up ..however, there is something weird going on:

Screenshot from 2024-02-28 23-40-52 Screenshot from 2024-02-28 23-51-45

Tempted to still merge this PR (as it seems to work in all cases I could muster except for Foxtrot's version of the fox), but if you have some time & any ideas on what is going on, would love to get your feedback

janhohenheim commented 9 months ago

Wow that looks... interesting. Poor fox! Yeah, if it works with everything else it should be good enough for a merge. I'll just reimport the fox. Is it maybe a scaling issue? Have you tried "Object mode -> CTRL A -> apply all transforms" beforehand?

kaosat-dev commented 9 months ago

Poor fox indeed:D Also , bingo , scaling issue: it seems both our versions have scaling problems , applying transforms does solve some of it but not quite as you can see below :) I would still consider it a "not exporter" issue ? (yes, for once I am sorta lazy :D ) Screen Capture_select-area_20240229101031

janhohenheim commented 9 months ago

Haha, great! Yeah, not an exporter issue :)