Skip to content

SceneModel changes#84

Closed
yuberee wants to merge 6 commits intoFacepunch:masterfrom
yuberee:SceneModel-Changes
Closed

SceneModel changes#84
yuberee wants to merge 6 commits intoFacepunch:masterfrom
yuberee:SceneModel-Changes

Conversation

@yuberee
Copy link
Copy Markdown
Contributor

@yuberee yuberee commented Dec 2, 2025

Some minor grievances I've been running into for SceneModel

  • Make ModelChanged public to subscribe to
  • Make SceneModel.GetBoneVelocity() and SceneModel.HasBoneOverrides public
  • Add SkinnedModelRenderer.GetBoneVelocity()
  • Document methods, fix some oversights

Comment thread engine/Sandbox.Engine/Scene/Components/Render/ModelRenderer.cs Outdated
@yuberee yuberee force-pushed the SceneModel-Changes branch from 161fdfa to 0877c3b Compare December 3, 2025 12:53
@garrynewman garrynewman added the triaged triaged pull-requests are replicated on the internal sbox repo label Dec 5, 2025
@sboxbot sboxbot added the accepted this pull request was accepted, hurrah! label Dec 7, 2025
@sboxbot
Copy link
Copy Markdown
Contributor

sboxbot commented Dec 7, 2025

This PR has been merged upstream.

@sboxbot sboxbot closed this Dec 7, 2025
@yuberee yuberee deleted the SceneModel-Changes branch January 18, 2026 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted this pull request was accepted, hurrah! triaged triaged pull-requests are replicated on the internal sbox repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants