Skip to content

Support SciPy sparse matrices for alpharank code, implement Stochasically Stable Distributions#1404

Draft
l-ouis wants to merge 1 commit into
google-deepmind:masterfrom
l-ouis:master
Draft

Support SciPy sparse matrices for alpharank code, implement Stochasically Stable Distributions#1404
l-ouis wants to merge 1 commit into
google-deepmind:masterfrom
l-ouis:master

Conversation

@l-ouis

@l-ouis l-ouis commented Dec 5, 2025

Copy link
Copy Markdown

Evolutionary dynamic Markov chains are sparse matrices, so using SciPy sparse matrices and operations can save memory.

Also implements SSD: https://arxiv.org/pdf/1207.1424, which is an exact algorithm that computes the stochastically stable distribution of a Markov chain (and can be used similarly as alpharank)

  • move Kosaraju.py into ssd.py and fix formatting
  • support sparse matrices in the SSD reduce step

@Eric-Ewing

@google-cla

google-cla Bot commented Dec 5, 2025

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@lanctot

lanctot commented Dec 6, 2025

Copy link
Copy Markdown
Collaborator

Thanks @l-ouis ! Can you sign the CLA?

Comment thread open_spiel/python/algorithms/psro_v2/psro_v2.py

@lanctot lanctot 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.

Can you fix the indentation to be consistent? (two-space indent)

E.g kosaraju.py appears to use 4-space indent but there might be others.|

Edit: original_ssd.py as well. Or maybe you don't intend to keep that one in the PR? (i.e. is it the original implementation that you're fitting into OpenSpiel, for instance)

@lanctot lanctot added the waiting Waiting to hear back from contributor (tests failed or thread reply / code update required) label Mar 9, 2026
@l-ouis l-ouis force-pushed the master branch 2 times, most recently from adcf5f8 to 961944b Compare April 20, 2026 18:12
@l-ouis

l-ouis commented Apr 20, 2026

Copy link
Copy Markdown
Author

Fixed indentation / moved kosaraju logic into ssd.py. Also implemented more correct sparsity logic to the ssd portion.

@lanctot lanctot removed waiting Waiting to hear back from contributor (tests failed or thread reply / code update required) labels Apr 20, 2026
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.

3 participants