TASEmulators / TASEmulators/BizHawk
Basic Bot should validate initial state
Nobody has claimed this yet.
- Dominant language
- C#
- Stars
- 2.8k
- Forks
- 468
- PR merge metrics
- No merged PRs in 30d
Description
Summary
While looking at #4560 I encountered issues with Basic Bot failing to work due to improper setup. I think it should be able to detect these problems and tell the user:
- The selected state/branch slot doesn't exist
- Recording mode is turned off
In either of these situations, it will appear to just plain not work. If there is no savestate in the selected slot, there will be a HUD message about no state but this does not appear until after pausing or stopping the bot.
Host env.
BizHawk 2.11 dev
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the Basic Bot entry point that initializes the selected state or branch slot and recording mode. Check how missing savestates currently produce the HUD message, then verify both invalid setup conditions are reported before pausing or stopping the bot. Done means the user is told when the slot is missing or recording mode is off.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100