test: remove offset for character controller

[aixaCode]

What does this PR change?

In the previous version, the character controller's pivot point was located in center, which required us to adjust position synchronization accordingly. This setup led to issues with other clients, especially in scenarios where characters standing on the ground should have a y-coordinate of 0. To address this, we've adjusted the pivot point to sit at the ground level. This eliminates the need to account for an offset and ensuring consistency across different clients decentraland/sdk#903

To implement this adjustment, several changes were necessary:

• Modified the character controller and realigned all colliders accordingly.
• An offset was introduced to the camera-following script to maintain its previous behaviour
• The ground-checking logic was streamlined to remove the offset.
• Removed offset from kernel when synchronizing the position.

This modification impacts scenes that rely on distance calculation based on player position, as the pivot point was moved.

Release plan:

• Conduct thorough initial testing to assess the extent of affected scenes and the severity of any issues.
• Based on our findings, we will decide on our communication strategy. We plan to share the updated branch with creators, allowing them to evaluate the impact on their scenes. We will provide some time for them to prepare for the change.
• If all is successful, we will roll out the update on a pre-announced date

This approach will allow us to smoothly introduce this important change, minimizing disruption and ensuring a seamless transition for all users.

How to test the changes?

1. Launch the explorer
2. Check the character movement is working as before
3. Inspect scenes, especially with trigger areas and validate they work properly.

Our Code Review Standards

https://github.com/decentraland/unity-renderer/blob/master/docs/code-review-standards.md

Copilot summary

copilot:summary

[github-actions]

After the CI passes:

Web

This branch can be previewed at:

• https://play.decentraland.org/?explorer-branch=test/removing-character-controller-offset
• https://play.decentraland.zone/?explorer-branch=test/removing-character-controller-offset

Desktop:

If you have the launcher installed (download launcher) you can press open on the following link:

• Open custom protocol
• Open custom protocol with Developer Mode

SDK 6/7:

• Open SDK7 scenes playground
• Open SDK7 scenes in preview
• Open SDK6 scenes in preview

More

• Code coverage report: https://renderer-artifacts.decentraland.org/branch-coverage/test/removing-character-controller-offset
• Benchmark report: https://renderer-artifacts.decentraland.org/branch-benchmark/test/removing-character-controller-offset/index.html

[Ludmilafantaniella]

❌ The fix fails in areas with automatic triggers, such as in the case of the DCL scene (-107,-94) or True X Energy (100,77)
❌ realm=MVMF21Meta.dcl.eth&position=-63%2C91

Screen.Recording.2024-02-23.at.11.48.26.mov

[aixaCode]

❌ Failed scenes:

• -107,-95

🟢 Passed scenes:

• https://sdk-test-scenes.decentraland.zone/

[Kinerius]

Nice!

[Ludmilafantaniella]

✅ Verified! No major blockers were found. character movement is working as before

[aixaCode]

We are going to just go with changing it for SDK7 in that PR #6123