Blizzard / Blizzard/s2client-proto

DebugGameState gas does nothing

Open
#196 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
4k
Forks
440
PR merge metrics
No merged PRs in 30d

Description

Hello, after looking what `DebugGameState` can do noticed that `gas = 8` does nothing 🤷‍♂️ also there is no such call in [reference](https://github.com/Blizzard/s2client-api/blob/614acc00abb5355e4c94a1b0279b46e9d845b7ce/src/sc2api/sc2_client.cc#L1045-L1087) cpp implementations. Wondering if this is something well known or not and may be it should be removed from proto at least to not confuse anyone

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing the DebugGameState definition in the protocol files and compare it with the referenced src/sc2api/sc2_client.cc implementation. Confirm whether gas is unsupported or unused, then determine whether the proto field should be removed; done means the protocol no longer exposes a misleading setting or the behavior is documented.

Written by the indexing model from the issue text.

Assessment

Domain
api
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.