Skip to content

Comments

refactor(algorithms): add type hints to cfr.py#1412

Merged
lanctot merged 1 commit intogoogle-deepmind:masterfrom
SH20RAJ:refactor/cfr-types
Jan 13, 2026
Merged

refactor(algorithms): add type hints to cfr.py#1412
lanctot merged 1 commit intogoogle-deepmind:masterfrom
SH20RAJ:refactor/cfr-types

Conversation

@SH20RAJ
Copy link
Contributor

@SH20RAJ SH20RAJ commented Dec 18, 2025

refactor(algorithms): add type hints to cfr.py

Added type hints to the CFRSolver, _CFRSolverBase classes and helper functions.
Cleaned up imports and improved docstrings.
Verified by running cfr_test.py.

@lanctot
Copy link
Collaborator

lanctot commented Dec 18, 2025

Thanks!

@SH20RAJ
Copy link
Contributor Author

SH20RAJ commented Dec 18, 2025

Will you merge it....

Thanks!

@lanctot
Copy link
Collaborator

lanctot commented Dec 18, 2025

Will you merge it....

Almost surely, but it has to past tests and be reviewed first. Please also note that we're backlogged (several PRs currently in flight before this one and we're all busy), so there may be a delay. But we tend to merge most things unless there's a serious issue or the quality is too low.

@lanctot lanctot added imported This PR has been imported and awaiting internal review. Please avoid any more local changes, thanks! merged internally The code is now submitted to our internal repo and will be merged in the next github sync. labels Jan 13, 2026
@lanctot lanctot merged commit 3d2e560 into google-deepmind:master Jan 13, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

imported This PR has been imported and awaiting internal review. Please avoid any more local changes, thanks! merged internally The code is now submitted to our internal repo and will be merged in the next github sync.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants