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

Caddy (vehicle) plays incorrect door/entry animation #1318

Closed
tut001 opened this issue Mar 25, 2020 · 15 comments · Fixed by #1600
Closed

Caddy (vehicle) plays incorrect door/entry animation #1318

tut001 opened this issue Mar 25, 2020 · 15 comments · Fixed by #1600
Labels
bug Something isn't working
Milestone

Comments

@tut001
Copy link
Member

tut001 commented Mar 25, 2020

Describe the bug
When entering Golf Caddy ID 457, it plays a door opening animation both when entering from passenger and driver side.

To reproduce

  1. Spawn ID 457
  2. Enter from either passenger or driver seat
  3. Notice you're opening an invisible door

Expected behaviour
The player should enter the vehicle by directly stepping inside.

Screenshots
N/A

Version
Client: Multi Theft Auto v1.5.7-release-20460

Additional context
Qaisjp suspects the vs2019 issue may be related
Strix later found the answer, #1318 (comment)

@tut001 tut001 added the bug Something isn't working label Mar 25, 2020
@zo3n
Copy link

zo3n commented Mar 25, 2020

Tractor as well
by the way noticed something about this invisible door:

you know when you normally enter a vehicle, and door remains open, so when you let go of W, the player closes the door automatically by also doing animation of doing it?
if this happens with invisible door of tractor, ped will attempt to close the door always, whenever letting go of W key, but it'll never close because it doesn't exist

@qaisjp qaisjp added this to To do in vs2019 via automation Mar 25, 2020
@zo3n
Copy link

zo3n commented Mar 25, 2020

Just to show it off better, here's a video demonstration:
https://streamable.com/u7p9m

@qaisjp
Copy link
Contributor

qaisjp commented Mar 25, 2020

Thank you so much for the video! Please can you install https://nightly.mtasa.com/mtasa-1.5.7-rc-20417-20200301.exe and check if that still happens?

@zo3n
Copy link

zo3n commented Mar 25, 2020

That fixed it. Both Caddy and Tractor act normal now.

@qaisjp
Copy link
Contributor

qaisjp commented Mar 25, 2020

Thanks for confirming. Okay, definitely a VS 2019 problem. Release the kraken!

@qaisjp qaisjp added this to the 1.6 milestone Mar 25, 2020
@zo3n
Copy link

zo3n commented Apr 5, 2020

Uh, problem reappeared in version 20488

@Dutchman101
Copy link
Member

Dutchman101 commented Apr 5, 2020

Uh, problem reappeared in version 20488

No.. it just hasn't been fixed yet, the build you tested with (which you said "fixed" it) was simply an old build from before upgrading to VS2019.

This is probably undefined behavior somewhere that gets exposed with VS2019 only, and it needs to be located in order to get fixed.

@ccw808
Copy link
Member

ccw808 commented Jun 24, 2020

Does the bug occur with a custom (debug or release) build?

@ghost
Copy link

ghost commented Jun 24, 2020

Does the bug occur with a custom (debug or release) build?

I can reproduce it on both, and sometimes this happens: https://cdn.discordapp.com/attachments/366384007535001612/725386799610069083/2020-06-24_21-23-39.mp4
In the video, we can see how the vehicle door closing animation gets played in a loop. It happens randomly.

@Dutchman101
Copy link
Member

Dutchman101 commented Jun 29, 2020

Tractor as well

Many more vehicles are affected.. for example, kart, mower, vortex, bandito, baggage.. the list goes on beyond that. It seems like all vehicles without door are affected, those with "jump (or step) right in" animation.

@ccw808 if there's any GTA hook optimization in netcode like last time with a similar hard-to-locate VS2019 bug, this information can help narrow it down?

@ccw808
Copy link
Member

ccw808 commented Jun 29, 2020

there's any GTA hook optimization in netcode like last time

I think the problem is in the OS code

@ghost ghost pinned this issue Jul 5, 2020
@StrixG
Copy link
Contributor

StrixG commented Aug 6, 2020

This issue is caused by c2dc463 and is not related to VS2019

@ghost
Copy link

ghost commented Aug 13, 2020

This is strange. I can still reproduce this bug.

@Vadya963
Copy link

there is no error, the client: 20621 server: 20624

@ghost
Copy link

ghost commented Aug 13, 2020

there is no error, the client: 20621 server: 20624

yes

This is strange. I can still reproduce this bug.

I was using the old server build. Full rebuild fixed the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants