Skip to content

Added the prefixes to the roles#55

Open
timur-berenstein wants to merge 5 commits into
clp-research:mainfrom
timur-berenstein:clemgame_roles_prefixes
Open

Added the prefixes to the roles#55
timur-berenstein wants to merge 5 commits into
clp-research:mainfrom
timur-berenstein:clemgame_roles_prefixes

Conversation

@timur-berenstein

Copy link
Copy Markdown
Contributor

Added the prefixes to the roles. For example: codenames -> "roles": [{"name": "Cluegiver", "prefix": ["CLUE:", "TARGETS:"]}, {"name": "Guesser", "prefix": "GUESS:"}]
Exception: textmapworld_graphreasoning
Reasoning: to complicated prefix e.g.: {"action":"GO: east","graph":{"nodes":["Sunroom","Kitchen"],"edges":{"north":[("Sunroom","Kitchen")],"south":[("Kitchen","Sunroom")],"east":[],"west":[]}}}

…{"name": "Cluegiver", "prefix": ["CLUE:", "TARGETS:"]}, {"name": "Guesser", "prefix": "GUESS:"}]

Exception: textmapworld_graphreasoning
Reasoning: to complicated prefix e.g.: {"action":"GO: east","graph":{"nodes":["Sunroom","Kitchen"],"edges":{"north":[("Sunroom","Kitchen")],"south":[("Kitchen","Sunroom")],"east":[],"west":[]}}}
@phisad

phisad commented Aug 13, 2025

Copy link
Copy Markdown
Collaborator

Hi @zarathuuustra ,

the "roles" should use the same structure for all clemgame.json in the repo (all games!)!

That is,

  • all player should become dicts
  • and all prefixes should become lists (since you found that some roles have multiple prefixes)

In addition,

  • the player name should be identical to the player class name e.g. Word Describer instead of Describer

phisad and others added 3 commits August 13, 2025 09:38
…ucture with players as dicts and prefixes as lists.
…emgame_roles_prefixes

# Conflicts:
#	wordle/clemgame.json

@phisad phisad left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Looks better now. Two more things:

  • privateshared Questioner is not playable. This should be removed.
  • wordle with clue doesnt have the critic player. This should be removed.

…ordle with clue (regarding the critic player) have been fixed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants