Skip to content

Improvements to arenaresults3 renderer#330

Merged
jonbinney merged 2 commits intomainfrom
jdb/arenaresults3-improvements
Jan 29, 2026
Merged

Improvements to arenaresults3 renderer#330
jonbinney merged 2 commits intomainfrom
jdb/arenaresults3-improvements

Conversation

@jonbinney
Copy link
Owner

@jonbinney jonbinney commented Jan 29, 2026

arenaresults3 is intended to be the "super verbose but also very explicit" renderer. I've made two changes to it in this PR:

  • for each player, summarize their results both as P1 and separately as P2
  • show total results for each of those

Using this test command:

python deep_quoridor/src/play.py \
	-N 3 -W 0 -mx 40 \
	-t 10 \
	-i 34 \
	-r computationtimes arenaresults3 eloresults progressbar \
	--num-workers=0 \
	-p \
	simple:nick=s_numba,use_rust=false,max_depth=4,heuristic=1,discount_factor=1\
	simple:nick=s_rust,use_rust=true,max_depth=4,heuristic=1,discount_factor=1\
	greedy:p_random=0.5

The results now look like:

+----------------------------+------+--------+------+
| Matchup                    | Wins | Losses | Ties |
+----------------------------+------+--------+------+
| P1 greedy (p_random=0.5)   |      |        |      |
|   vs s_numba               | 0%   | 100%   | 0%   |
|   vs s_rust                | 0%   | 100%   | 0%   |
|   total                    | 0%   | 100%   | 0%   |
|                            |      |        |      |
| P2 greedy (p_random=0.5)   |      |        |      |
|   vs s_numba               | 60%  | 40%    | 0%   |
|   vs s_rust                | 60%  | 40%    | 0%   |
|   total                    | 60%  | 40%    | 0%   |
|                            |      |        |      |
| P1 s_numba                 |      |        |      |
|   vs greedy (p_random=0.5) | 40%  | 60%    | 0%   |
|   vs s_rust                | 0%   | 100%   | 0%   |
|   total                    | 20%  | 80%    | 0%   |
|                            |      |        |      |
| P2 s_numba                 |      |        |      |
|   vs greedy (p_random=0.5) | 100% | 0%     | 0%   |
|   vs s_rust                | 100% | 0%     | 0%   |
|   total                    | 100% | 0%     | 0%   |
|                            |      |        |      |
| P1 s_rust                  |      |        |      |
|   vs greedy (p_random=0.5) | 40%  | 60%    | 0%   |
|   vs s_numba               | 0%   | 100%   | 0%   |
|   total                    | 20%  | 80%    | 0%   |
|                            |      |        |      |
| P2 s_rust                  |      |        |      |
|   vs greedy (p_random=0.5) | 100% | 0%     | 0%   |
|   vs s_numba               | 100% | 0%     | 0%   |
|   total                    | 100% | 0%     | 0%   |
|                            |      |        |      |
+----------------------------+------+--------+------+

Copy link
Collaborator

@alejandromarcu alejandromarcu left a comment

Choose a reason for hiding this comment

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

looks great, very easy to read the table!

@jonbinney jonbinney merged commit bed0e3d into main Jan 29, 2026
1 check passed
@jonbinney jonbinney deleted the jdb/arenaresults3-improvements branch February 3, 2026 14:51
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