-
-
Notifications
You must be signed in to change notification settings - Fork 224
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
Error on Sprite Calculation When Moving the Character [Canary, Tibia 11 spr/dat] #1022
Comments
i only test 8.6 and 13.10 g_game.enableFeature(GameEnhancedAnimations)
g_game.enableFeature(GameIdleAnimations) ups , me bad testg_game.getLocalPlayer():setOutfit({type=63}) main repo spr 10.98 (glitching + error terminal)✅ GameEnhancedAnimations
main repo spr 8.6 (work)❌ GameEnhancedAnimations main repo assets 13.40 (work) |
Alright, so what's the exact configuration that glitches? I don't even know what's tibia 11 spr, it looks like regular 1098? |
@lucasromagnoli please provide exact looktype that fails with images from object builder, so I can inspect your options. |
@kokekanon, @nekiro thanks for helping. What I noticed is that the issue occurs with outfits that have the idle and walking groups. If we look closely, when the character is stationary, it uses the first animation from the walking sequence. However, for outfits that are only animations, the calculation works correctly. Outfit 128 - BugOutfit 243 - Animation working |
In my personal case I did a clean installation: exe.: compiled by myself(solution release GL) and also github downloader (cmake) = https://github.com/mehah/otclient/actions/runs/12637388851 test with tfs 1.4.2 10.98 in my case there is no crash, but glitching . |
Priority
High
Area
What happened?
Problem:
When the character moves, the client fails to correctly calculate the sprite and triggers the following assertion error:
Assertion failed: index < m_spritesIndex.size()
This happens specifically at this line of code.
Environment:
Operating System: Windows
Canary (latest commit)
Tibia 11 (spr/dat files)
Notes:
This issue started occurring after merging PR #1019.
Reverting the changes from the PR fixes the problem.
What OS are you seeing the problem on?
Windows
Code of Conduct
The text was updated successfully, but these errors were encountered: