Skip to content
This repository has been archived by the owner on Mar 9, 2021. It is now read-only.

Weird way while going up the stairs in CSs #2500

Open
3 tasks done
Gradius2 opened this issue Jan 27, 2021 · 5 comments
Open
3 tasks done

Weird way while going up the stairs in CSs #2500

Gradius2 opened this issue Jan 27, 2021 · 5 comments

Comments

@Gradius2
Copy link

Gradius2 commented Jan 27, 2021

  • I have searched existing issues (https://github.com/topaz-next/topaz/issues) to see if the issue has already been opened
  • I have checked the commit log to see if the issue has been resolved since my server was last updated
  • This issue occurs on release branch (if other branch, please specify)

Additional Information (Steps to reproduce/Expected behavior) :

I was pretty sure I wasn't seeing things, so I decided to check up some official CS on youtube to compare/confirm those claims, result I was correct.

On any CS involving stairs, the chars beheaves very weird (or wrong) way.

Official CS:
https://youtu.be/-XfXj4UQcAk?t=269

Look @ Aldo going up the stairs (topaz server).
https://user-images.githubusercontent.com/11011463/105931318-b914b600-6029-11eb-9be9-d409b3ffcf23.mp4

@kaincenteno
Copy link
Contributor

Weird i thought CS don't rely on anything from the server since they are client side >.<

I guess this would be related to navmeshes then?

@zach2good
Copy link
Contributor

Interesting find!

My guess would be that this is a side effect of the speed and speedsub for Aldo's SQL entry being set to 40/40, instead of the modern default of 50/50. Then the client takes that information and determines "If you're going at this speed up the stairs, you are faster than your default speed, so play run animation". That would be my guess. You could change those values and run the CS and see if it fixes it 🤷‍♂️

  `speed` tinyint(3) unsigned NOT NULL DEFAULT '40',
  `speedsub` tinyint(3) unsigned NOT NULL DEFAULT '40',

INSERT INTO `npc_list` VALUES (17809493,'Aldo','Aldo',0,0.000,0.000,0.000,0,40,40,0,0,0,6,27,0x00004E0000000000000000000000000000000000,32,NULL,1);

@Gradius2
Copy link
Author

Isn't only with 'aldo' is any NPC in CSes. Hmm, perhaps is really the !speed cmd, I'll check it again with other CSes.

@TeoTwawki
Copy link
Contributor

Interesting find!

My guess would be that this is a side effect of the speed and speedsub for Aldo's SQL entry being set to 40/40, instead of the modern default of 50/50. Then the client takes that information and determines "If you're going at this speed up the stairs, you are faster than your default speed, so play run animation". That would be my guess. You could change those values and run the CS and see if it fixes it man_shrugging

  `speed` tinyint(3) unsigned NOT NULL DEFAULT '40',
  `speedsub` tinyint(3) unsigned NOT NULL DEFAULT '40',

INSERT INTO `npc_list` VALUES (17809493,'Aldo','Aldo',0,0.000,0.000,0.000,0,40,40,0,0,0,6,27,0x00004E0000000000000000000000000000000000,32,NULL,1);

I would wager it is lower than 40 actually. Objects aren't universally set to a single speed liek the copy pasta db rows tend to be. I can find multiple examples of retail npcs moving at 12, 25, 40, 50... Speedsub tends to be 40 or 50 regardless of their regular speed, but speed is all over the place - especially in cut scene events! Zeid in Qu Bia Arena (archlich fight) is another prominent example of stutter-stair steps. The way forward is still the same: we need to verify they are using retail accurate speed/speedsub and see if that solves it.

@ShiyoKozuki
Copy link

Happens to players and every single NPC.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants