Skip to content

fix: unset animationAssetIds to allow player-set animations#170

Open
intheviewof wants to merge 1 commit into
Windows81:mainfrom
intheviewof:main
Open

fix: unset animationAssetIds to allow player-set animations#170
intheviewof wants to merge 1 commit into
Windows81:mainfrom
intheviewof:main

Conversation

@intheviewof

@intheviewof intheviewof commented May 22, 2026

Copy link
Copy Markdown

quick hack to allow player specified r15 animations (walk, run, etc) by unsetting hardcoded asset ids. this allow users to edit their GameConfig.toml and specify the IDs of animations they wish to use (i.e: Stylish Run)
tested & working on 2018 and 2021. i only tested this very lightly but this single change shouldn't have any negative effects

not my discovery, but i can't remember who found it first

tested with the following accessory data:

retrieve_avatar_call_mode = "lua"
retrieve_avatar = '''
function(user_id_num, user_code)
    local default_avatar = {
        type = 'R15',
        items = {},
        scales = {
            height = 1,
            width = 1,
            head = 1,
            depth = 1,
            proportion = 0,
            body_type = 0,
        },
        colors = {
            head = BrickColor.Random().Number,
            left_arm = BrickColor.Random().Number,
            left_leg = BrickColor.Random().Number,
            right_arm = BrickColor.Random().Number,
            right_leg = BrickColor.Random().Number,
            torso = BrickColor.Random().Number,
        },
    }

    if user_code == 'Yuri' then
        default_avatar.items = {
			125157548394108,
            7581780596,
			4603805914,
			20573078,
			12413229126,
			619512153,
        }
        default_avatar.colors = {
            head = 148, 
            left_arm = 148,
            left_leg = 148,
            right_arm = 148,
            right_leg = 148,
            torso = 148,
        }
        default_avatar.scales = {
            height = 1,
            width = 1,
            head = 1,
            depth = 1,
            proportion = 0,
            body_type = 0, 
        }

    elseif user_code == 'Player2' then
        default_avatar.items = {
			12596168028,
            8262048015,
            7259172148,
            6471948627,
            6380112434,
            6238565077,
            6067201766,
            6067201270,
            4660621665,
            4466228735,
            6202647707,
            7317773,
        }
        default_avatar.colors = {
            head = 125, 
            left_arm = 125,
            left_leg = 125,
            right_arm = 125,
            right_leg = 125,
            torso = 125,
        }
        default_avatar.scales = {
            height = 1,
            width = 1,
            head = 1,
            depth = 1,
            proportion = 0,
            body_type = 0, 
        }
		
		elseif user_code == 'Yuri_alt' then
        default_avatar.items = {
			125157548394108,
            7581780596,
			4603805914,
			20573078,
			12413229126,
			619512153,
        }
        default_avatar.colors = {
            head = 125, 
            left_arm = 125,
            left_leg = 125,
            right_arm = 125,
            right_leg = 125,
            torso = 125,
        }
        default_avatar.scales = {
            height = 1,
            width = 1,
            head = 1,
            depth = 1,
            proportion = 0,
            body_type = 0, 
        }


    end
    return default_avatar
end
'''

Summary by CodeRabbit

  • Bug Fixes
    • Character appearance endpoint now returns empty animation asset identifiers instead of previously hardcoded animation mappings.

Review Change Stack

this appears to allow users to specify custom animations in their avatar in GameConfig.toml
@coderabbitai

coderabbitai Bot commented May 22, 2026

Copy link
Copy Markdown
Contributor

Walkthrough

The v463 character appearance endpoint now returns an empty animationAssetIds object instead of hardcoded animation ID mappings. All other response fields remain unchanged.

Changes

Character appearance endpoint

Layer / File(s) Summary
v463 avatar animation assets response
Source/web_server/endpoints/avatar.py
animationAssetIds simplified from populated animation mapping to empty object in avatar JSON response.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Title accurately reflects the main change: unsetting animationAssetIds to enable custom animations, matching the code modification and PR objectives.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Windows81 Windows81 changed the title unset animationAssetIds to allow player-set animations fix: unset animationAssetIds to allow player-set animations May 22, 2026
@Windows81 Windows81 changed the title fix: unset animationAssetIds to allow player-set animations fix: unset animationAssetIds to allow player-set animations May 22, 2026
@frostyyyy1

Copy link
Copy Markdown
Collaborator

@intheviewof Stylish Run also does not work for me, even if remove those animation ids.

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.

2 participants