Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CMCL-0000: FreeLookOnSphericalSurface sample is improved #935

Merged
merged 6 commits into from Feb 28, 2024

Conversation

glabute
Copy link
Collaborator

@glabute glabute commented Feb 20, 2024

Purpose of this PR

FreeLookOnSphericalSurface sample is improved, adding a moving surface and second camera.
This is part of the ongoing improvement of the SimplePlayerController ecosystem.

Testing status

  • Added an automated test
  • Passed all automated tests
  • Manually tested

Documentation status

  • Updated CHANGELOG
  • Updated README (if applicable)
  • Commented all public classes, properties, and methods
  • Updated user documentation

Technical risk

Only the samples have changed.

@codecov-commenter
Copy link

codecov-commenter commented Feb 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2239d9a) 26.67% compared to head (ee65a1f) 26.67%.
Report is 2 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #935   +/-   ##
=======================================
  Coverage   26.67%   26.67%           
=======================================
  Files         246      246           
  Lines       27573    27573           
=======================================
  Hits         7356     7356           
  Misses      20217    20217           

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

Copy link
Collaborator

@AntoineCharton AntoineCharton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice! Just wondering if the sphere camera should have it's own sample? Is it necessary to animate the cube I haven't seen anything interacting with the animation?

@glabute
Copy link
Collaborator Author

glabute commented Feb 23, 2024

Try making the player walk on the cube

@glabute glabute merged commit 1d804c5 into main Feb 28, 2024
9 checks passed
@glabute glabute deleted the dev/improve-FreeLookOnSpehericalSurface-sample branch February 28, 2024 17:21
glabute added a commit that referenced this pull request Mar 11, 2024
* Add missing length check to 3rdPersonFollow

* CMCL-1551: add missing empty check when overriding a blend stack (#930)

* add missing empty check

* Update CHANGELOG.md

* Better filtering for vcam Navel Gazing warning (#927)

* Revert "Update CinemachineThirdPersonFollow.cs"

This reverts commit c15efbe.

* Update CMWaveform.compute

* Update CMWaveform.compute

* Update README.md

* another tweak - binding modes this time

* Revert "another tweak - binding modes this time"

This reverts commit 91a0b6c.

* Fix typo in Editor min req version (#936)

* CMCL-0000: FreeLookOnSphericalSurface sample is improved (#935)

* FreeLookOnSphericalSurface sample is improved, adding a moving surface and second camera.

* forgot to commit

* tewaking blend hints

* binding mode tweaks

* Cleanup, tweaking

* cosmetic

* bugfix in AddCustomBlendable

* deprecate RangeSliderAttribute (#944)

* Add some missing null checks in shot editor to handle missing scripts

* CMCL-0000: add isDelayed to some fields (#939)

* add isDelayed to some fields

* Fix outdated tooltip, add "Custom" label to SensorSize presets dropdown

* Update LensSettings.cs

* statedrivencamera bugfix for minactivationtime, add CancelWait (#938)

* CMCL-1565: Deoccluder improvements, and add CinemachineShotQualityEvaluator (#899)

* Add CinemachineDecollider

* Added Cinemachine Terran Decollider

* Update RuntimeUtility.cs

* Fix OrbitalFollow inappropriate vertical damping

* Update CinemachineDecollider.cs

* small fixes

* Add CinemachineShotQualityEvaluator

* Add deoccluder Resolve towards follow target option

* Terrain Decollider improvements

* Deoccluder evaluates shot quality only when evaluation is enabled

* Add doc and editor for shot quality evaluator

* Remove TerrainDecollider

* Add terrain functionality to Decollider

* Remove decollider (to another PR)

* Update CinemachineShotQualityEvaluator.md

* Update TableOfContents.md

* Update CHANGELOG.md

* Update CHANGELOG.md

* Remove ResolveTowardsFollowTarget

* Update CHANGELOG.md

* Update CinemachineOrbitalFollow.cs

* Update CinemachineOrbitalFollow.cs

* Update CinemachineDeoccluder.cs

* Add a comment

* CMCL-0000: Fix OrbitalFollow inappropriate vertical damping (#942)

* Fix OrbitalFollow inappropriate vertical damping

* small fixes

* Revert "small fixes"

This reverts commit 7d32f6a.

* new strategy

* Setting isDelayed was causing some exceptions on domain reload

* Add another missing isDelayed

* Axis recentering happens independently on axes (#937)

---------

Co-authored-by: Gregory Labute <gregoryl@unity3d.com>
Co-authored-by: Sébastien Duverne <55094336+sebastienduverne@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants