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

Fixed issue preventing headgear and top bone posing for some races #1440

Merged
merged 2 commits into from
Feb 3, 2025

Conversation

ergoxiv
Copy link
Collaborator

@ergoxiv ergoxiv commented Feb 2, 2025

Changes:

  • Renamed NopHookViewModel to NopHook.
  • Added an unhandled exception event to NOP hooks to improve their stability.
  • Added three new hooks (kineDriverPosition, kineDriverRotation, kineDriverScale) to fix Helmet Bone #1422.
  • Changed pose mode toggle freeze order to freeze physics after positions. This will make Viera ear length increases when freezing physics #1319 less frequent of an issue.
  • Minor cleanup and additional comments to address service.

@StoiaCode StoiaCode linked an issue Feb 2, 2025 that may be closed by this pull request
public static IntPtr Territory { get; private set; }
public static IntPtr GPose { get; private set; }
public static IntPtr TimeAsm { get; private set; }
public static IntPtr Framework { get; set; }
public static IntPtr PlayerTargetSystem { get; set; }
public static IntPtr AnimationSpeedPatch { get; set; }
public static IntPtr KineDriverPosition { get; private set; }
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you think its worthwhile to comment on what exactly the KineDriver things are for, even if we only have a brief description? As I understood right now they are only responsible for one thing and chances are it'll only get harder to understand what exactly they are for in the future.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, good point. I ammended the latest commit to include a comment on the kinematic driver's use case.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

God damn it, Visual Studio nuked the includes. I just noticed 😆

@ergoxiv ergoxiv force-pushed the hotfix/visor-posing-fix branch from a427629 to f07b075 Compare February 2, 2025 20:29
@ergoxiv ergoxiv force-pushed the hotfix/visor-posing-fix branch from f07b075 to fc5a677 Compare February 2, 2025 20:33
@StoiaCode StoiaCode merged commit 3baa980 into imchillin:testing Feb 3, 2025
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.

Helmet Bone
2 participants