[SofaCore] Restore xml and python different naming conventions

[alxbilger]

Based on the discussion started in #2631 and finished during a dev meeting, this PR restores the old behavior:

• Two different conventions based on whether the scene is in XML or Python
• XML suffixes start at 1

Nevertheless, the class that provides the names is still common.

[ci-depends-on https://github.com/sofa-framework/SofaPython3/pull/241]

---

By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).

---

Reviewers will merge this pull-request only if

• it builds with SUCCESS for all platforms on the CI.
• it does not generate new warnings.
• it does not generate new unit test failures.
• it does not generate new scene test failures.
• it does not break API compatibility.
• it is more than 1 week old (or has fast-merge label).

[sofabot]

[ci-depends-on] detected during build #1.

To unlock the merge button, you must

• Merge or close Restore xml and python different naming conventions SofaPython3#241
  For this build, SofaPython3/ExternalProjectConfig.cmake.in will be edited by builders with
  GIT_REPOSITORY https://github.com/sofa-framework/SofaPython3
  GIT_TAG 2ca795da0c1b85e4b8ebf4a402ba1a57ae4f2317

[alxbilger]

[ci-build][with-all-tests]

[sofabot]

[ci-depends-on] detected during build #2.

To unlock the merge button, you must

• Merge or close Restore xml and python different naming conventions SofaPython3#241
  For this build, SofaPython3/ExternalProjectConfig.cmake.in will be edited by builders with
  GIT_REPOSITORY https://github.com/sofa-framework/SofaPython3
  GIT_TAG 2ca795da0c1b85e4b8ebf4a402ba1a57ae4f2317

[EulalieCoevoet]

Hi,

Me again... Sorry I didn't notice but the old behavior has not been really restored. Before, the default name was FreeMotionAnimationLoop, for instance, and now it is freeMotionAnimationLoop which still breaks my (our) scenes.

[damienmarchal]

To me this change is a perfect use case for the Lifcycle feature management introduced in:
sofa-framework/SofaPython3#184

With such it would be trivial when loading a scene to write something like that:

from Sofa.Lifecycle import __feature__

def myOldNamingScene(root):
    """A scene or prefab that use the old naming scheme"""
     pass

def myNewNamingScene(root):
    """A scene or prefab that use the new naming scheme"""
    pass

def createScene(root):
     with __feature__("uniform-naming-scheme",True):
           myNewNamingScene(root)
           
     myOldNamingScene(root)                     

[alxbilger]

Before, the default name was FreeMotionAnimationLoop, for instance, and now it is freeMotionAnimationLoop which still breaks my (our) scenes.

Sorry, I thought it was the opposite due to https://peps.python.org/pep-0008/#function-and-variable-names. So, let's merge #2801