secondlife / secondlife/viewer

Re-enable animation constraints on animation verifier

Open
#3,631 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug stale team:viewer
Dominant language
C++
Stars
299
Forks
146
Avg merge
1d 9h
Merged PRs (30d)
88

Description

Animation constraints were disabled about 5 or so years ago because of a viewer crash bug regarding chain length / constraint targets. This bug has since been fixed as far as I can tell (At least, my repro animation doesn't crash the Linden viewer or mainstream TPVs).

Given the fact that invalid chain lengths / constraint targets are now ignored by the viewer, there isn't really a need to verify the chain length anymore.

It would be really nice if we could have the ability to add constraints in animations again!

This is basically just IK for bones, think feet automatically aligning to the ground's curvature, or shape independant animations that can put the hands on the head, without it looking like the hands are floating above or inside the head.

(This feature ALREADY EXISTS in the viewer. In fact, default animations use this mechanism. It was disabled on the server side animation verifier a few years back due to a bug that I found that has since been fixed.)

https://secondlife.canny.io/admin/board/feature-requests/p/re-enable-animation-constraints-on-animation-verifier

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No source file or test is named. Start by locating the server-side animation verifier and reviewing the linked Canny feature request alongside its constraint validation logic. Done means valid animation constraints are accepted again while invalid chain lengths or targets remain safely ignored without viewer crashes.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
game-dev
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.