[Werewolf V0.2] - Game Logic #426
Conversation
SohierDane
changed the title
| wolves = [p for p in self.state.alive_players() if p.role.team == Team.WEREWOLVES] | ||
| villagers = [p for p in self.state.alive_players() if p.role.team == Team.VILLAGERS] | ||
|
|
||
| if not wolves: |
There was a problem hiding this comment.
@hannw It looks like we're missing a state for the case where a single player causes the game to end early, such as by timing out or delivering invalid responses. It looks like penalizing just the player causing the error wouldn't be too tricky to add here. Would it be a reasonable lift to integrate with the other error handling?
| perceived_threat_level: Optional[PerceivedThreatLevel] = Field( | ||
| default=PerceivedThreatLevel.SAFE, access=DataAccessLevel.PERSONAL | ||
| ) | ||
| action: Optional[BaseAction] = Field(default=None, access=DataAccessLevel.PERSONAL) |
There was a problem hiding this comment.
@hannw Looks like this line is the cause of a Pydantic deprecation alert:
PydanticDeprecatedSince20: Using extra keyword arguments on Fieldis deprecated and will be removed. Usejson_schema_extra instead. (Extra keys: 'access'). Deprecated in Pydantic V2.0 to be removed in V3.0.
Gemini suggests instead:
action: Optional[BaseAction] = Field(default=None, json_schema_extra={"access": DataAccessLevel.PERSONAL})
There was a problem hiding this comment.
Also affects lines 37 and 40.
No description provided.