Blizzard / Blizzard/s2client-proto
DebugGameState gas does nothing
- 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