Export get current animation name

[iommu]

Describe your changes

Exported the internal AnimationManager::GetAnimationName as interactor::getAnimationName so that libf3d can access animation name data.

Issue ticket number and link if any

#2699

Checklist for finalizing the PR

• I have performed a self-review of my code
• I have added tests for new features and bugfixes *(see notes)
• I have added documentation for new features
• If it is a modifying the libf3d API, I have updated bindings *(see notes)
• If it is a modifying the .github/workflows/versions.json, I have updated docker_timestamp

Notes:

• Java bindings do not exist for any interactor code ans so have not been added for this change
• Python tests do not exist for any interactor animation code (But I would be happy to contribute that if that would be accepted)

Continuous integration

Please write a comment to run CI, eg: \ci fast.
See here for more info.

[github-actions]

You are modifying libf3d public API! ⚠️Please update bindings accordingly⚠️!
You can find them in their respective directories: c, python, java, webassembly.

[iommu]

Question regarding the docs : the checklist specifically mentions the QUICKSTART docs. So I'm assuming that, as this feature doesn't really fit being exposed as a CLI option, this doesn't need any changes to that?

[mwestphal]

A suggested API change

[mwestphal]

Hey @iommu

Need any help moving forward ?

[iommu]

Hi, sorry yes I was a bit unsure what you meant by your last comment of
getAnimationName vs availableAnimationNames is that a possible future change or is that something that you would like done for this merge request?

[mwestphal]

Hi, sorry yes I was a bit unsure what you meant by your last comment of getAnimationName vs availableAnimationNames is that a possible future change or is that something that you would like done for this merge request?

Lets try to do it in this PR :)

[mwestphal]

Question regarding the docs : the checklist specifically mentions the QUICKSTART docs. So I'm assuming that, as this feature doesn't really fit being exposed as a CLI option, this doesn't need any changes to that?

yes, unrelated to QUICKSTART indeed.

[mwestphal]

api change

[mwestphal]

a question

[mwestphal]

\ci fast

[mwestphal]

almost there, ready for full CI imo :)

[iommu]

\ci full

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.64%. Comparing base (6838c62) to head (ffc8ab5).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2702   +/-   ##
=======================================
  Coverage   96.64%   96.64%           
=======================================
  Files         142      142           
  Lines       12918    12933   +15     
=======================================
+ Hits        12484    12499   +15     
  Misses        434      434           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[mwestphal]

Thanks for the contribution @iommu !