splewis / csgo-practice-mode

CS:GO Sourcemod plugin for private team/individual practice servers
GNU General Public License v3.0
455 stars 70 forks source link

Spawn points are incorrect #47

Closed gg closed 7 years ago

gg commented 7 years ago

Expected behavior

All 5v5 competitive spawn points, and only 5v5 competitive spawn points should be considered when using the .spawn, .bestspawn, and .worstspawn commands.

Actual behavior

Steps to reproduce

Examples:

de_cbble CT

15 expected de_cbble competitive CT 5v5 spawn points (using map_showspawnpoints) cbble competitive CT spawns using map_showspawnpoints

Only 9 de_cbble CT spawn points are considered (4 of which are Wingman mode spawn points):

de_train CT

de_train T

12 expected spawn points, but only 7 spawn points are available via the plugin.

splewis commented 7 years ago

The addition of the wingman spawns is a bug (hopefully there's a property on them I can find to differentiate them), but the first image you show is misleading.

The spots on the left (on the A site itself) aren't included in the practicemode command since they're lower priority (based on their m_iPriority value) spawns that shouldn't be given in 5v5's. There is a bug (which happens on all servers, as far as I know) where the pistol rounds (or maybe just the 1st pistol) causes players to get any of the spawns - including those A site ones - but I don't think it's correct to include those in practicemode still.

I don't think your train example is correct. I've never spawned that close to Z in a 5v5 (excluding the mentioned pistol bug, as I mentioned).

edit: checked the sdk, there is an "enabled" field on info_player_counterterrorist at least, so presumably that's what wingman is setting on the new spawns.

gg commented 7 years ago

Ah, I didn't know about the first round spawn bug. Thanks for educating me. I agree that it's not correct to include these additional spawns in practicemode.

edit: checked the sdk, there is an "enabled" field on info_player_counterterrorist at least, so presumably that's what wingman is setting on the new spawns.

That's good to hear.

Thanks for creating this plugin -- my friends and I really appreciate it!

splewis commented 7 years ago

Confirmed that https://github.com/splewis/csgo-practice-mode/commit/b8071d16704fcff9f171b3ca1c79c03d69103bee fixed this.