open_spiel icon indicating copy to clipboard operation
open_spiel copied to clipboard

Added Python efr algorithm implementation

Open Jamesflynn1 opened this issue 1 year ago • 6 comments

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

Jamesflynn1 avatar Jul 08 '23 20:07 Jamesflynn1

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.

google-cla[bot] avatar Jul 08 '23 20:07 google-cla[bot]

Thanks @Jamesflynn1 !

Please let us know when it is ready for a review.

lanctot avatar Jul 22 '23 10:07 lanctot

Hi @Jamesflynn1, if any of your recent commits have addressed the comments, can you reply to them and/or mark them as resolved?

lanctot avatar Oct 31 '23 15:10 lanctot

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.

Jamesflynn1 avatar Nov 13 '23 22:11 Jamesflynn1

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

Jamesflynn1 avatar Jan 14 '24 20:01 Jamesflynn1

No problem, I can try to fix these on our side. Apologies for the delays.

lanctot avatar Jan 28 '24 11:01 lanctot

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.

lanctot avatar Apr 16 '24 09:04 lanctot