open_spiel
open_spiel copied to clipboard
Added Python efr algorithm implementation
A Python implementation of the EFR (https://arxiv.org/abs/2102.06973) algorithm with the deviation types defined in the proposing paper. The implementation was developed as part of my undergraduate dissertation and I thought it would be beneficial if the implementation could be made available as part of OpenSpiel.
Uses a shortcut for external only deviations as found in https://github.com/dmorrill10/hr_edl_experiments (original experimental code)
Any guidance or feedback would be appreciated as this is my first contribution to an open source project.
Todo:
- [x] Look into issues with behavioural deviations
- [x] Look into possible incorrect definition with CSPS deviations
- [x] Add docstrings for all functions
- [x] Remove discounting code
- [x] Refactor code
- [x] Ensure adherence to Google Python style
- [x] Add updates to OpenSpiel docs
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.
Thanks @Jamesflynn1 !
Please let us know when it is ready for a review.
Hi @Jamesflynn1, if any of your recent commits have addressed the comments, can you reply to them and/or mark them as resolved?
Hi @Jamesflynn1, if any of your recent commits have addressed the comments, can you reply to them and/or mark them as resolved?
Hi @lanctot, apologies I haven't replied sooner, I have been very busy with university coursework and graduate applications.
I ran pylint with the Google style guide and I was still some of style issues (spaces after comments and blank lines), I'll take another look now and mark off the solved issues.
Hi @lanctot,
I have tried to refractor the files using the linked pylintrc file and I'm not getting all the commenting suggestions that has been pointed out. I've fixed (hopefully) all instances of the issues you have pointed out. However, I think it's possible that I have a different pylintrc file (I get no linting issues when I pylint the file) so to catch all issues would it be possible to compare files?
Any assistance would be appreciated!
Thanks, James
No problem, I can try to fix these on our side. Apologies for the delays.
Hi @Jamesflynn1 , sorry for the extreme lateness. Did you ever verify the convergence matches what was expected from the papers? I'd like to import this, but it would be good to know that it's correct.