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

Roundstart takes about 20 seconds between the moment you were spawned in and the moment the game actually starts #18637

Closed
goonstation-issuebot opened this issue Apr 26, 2024 · 9 comments
Labels
C-Bug A bug that impacts usage of a feature E-Bug Hall of Fame Bugs which are very funny and memorable, and should be preserved for future generations. E-FUCK [Dev Only] FUCK E-Verified This bug has been verified to occur P-Major A bug that significantly impacts the usage of a feature.

Comments

@goonstation-issuebot
Copy link

Labels

[BUG][MAJOR]

Description

Roundstart takes about 20 seconds between the moment you were spawned in and the moment the game actually starts

Steps to reproduce

1- Roundstart
2- Spawn in
3- Wait 20 seconds before the game is actually launched

Expected Behavior

The usual 2-3 second roundstat

Additional Information & Screenshots

It wasn't happening before, but since a day or two, it's almost constantly 15+ seconds of wait.

Reported by: Chatauscours
Client version: 515.1635
On server: Goonstation 3 Roleplay: Morty
Active test merges: #18435, #18561, and #18275
Round log date: 2024-04-26-11-31
Reported on: 2024-04-26 11:38:01
Map: DONUT2

@keywordlabeler keywordlabeler bot added C-Bug A bug that impacts usage of a feature P-Major A bug that significantly impacts the usage of a feature. labels Apr 26, 2024
@Chatauscours
Copy link
Contributor

Adding verified tag, since Iit seems it happens to everyone, see
https://discord.com/channels/182249960895545344/890223118482800790/1233151068301430803

@Chatauscours Chatauscours added the E-Verified This bug has been verified to occur label Apr 26, 2024
@Chatauscours
Copy link
Contributor

buuug

@TobleroneSwordfish
Copy link
Contributor

Currently being worked on, still no idea what's causing the delay.

@Chatauscours
Copy link
Contributor

Chatauscours commented Apr 27, 2024

I don't know if it's of any help, but I've noticed over the rounds that X is CE / HOP / RD / MD gets displayed very early on, as soon as you spawn in, however X is Captain gets displayed only after the delay. @TobleroneSwordfish

@pgmzeta
Copy link
Contributor

pgmzeta commented Apr 27, 2024

Currently being worked on, still no idea what's causing the delay.

gut check here but i removed a SPAWN(0) that would be called during job distribution in #18482 - worth adding back to test?

@TobleroneSwordfish
Copy link
Contributor

gut check here but i removed a SPAWN(0) that would be called during job distribution in #18482 - worth adding back to test?

!!!

@TobleroneSwordfish
Copy link
Contributor

Okay yep, it was almost certainly #18482 - the SPAWN was around a choose_name call which blocks the call stack until the user chooses their name. So the entire roundstart process was being blocked until the clown decided which version of "Honk McHonkingson" they wanted to call themselves today.
Easy to miss in that big block of mostly uselessly SPAWNed code!

@TobleroneSwordfish
Copy link
Contributor

I'll leave this issue open until we can verify it's no longer occurring on live.

@TobleroneSwordfish TobleroneSwordfish added the E-Bug Hall of Fame Bugs which are very funny and memorable, and should be preserved for future generations. label Apr 27, 2024
@pgmzeta
Copy link
Contributor

pgmzeta commented Apr 27, 2024

😔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Bug A bug that impacts usage of a feature E-Bug Hall of Fame Bugs which are very funny and memorable, and should be preserved for future generations. E-FUCK [Dev Only] FUCK E-Verified This bug has been verified to occur P-Major A bug that significantly impacts the usage of a feature.
Projects
None yet
Development

No branches or pull requests

4 participants