Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Phantom go #867

Merged
merged 35 commits into from
Aug 8, 2022
Merged

Phantom go #867

merged 35 commits into from
Aug 8, 2022

Conversation

Syor
Copy link
Contributor

@Syor Syor commented Jun 24, 2022

No description provided.

Syor and others added 30 commits June 24, 2022 14:33
…ith stones visible in players observation

Added a visual test for this method
Reverted changes in inicialization of new state in mentioned method, because methods from PhantomGoState are needed
… played by opponent for observation, resulting in terminal state
… reliably generates new resampled states

Added a test to test this method on huge number of states with random histories
… of the infoState

Added a visual test for Cloning
…nfostate, now only resamples a metaposition, infostate is in progress
Added a method to check metaposition equality
refactored metapositon resampling test
Syor added 2 commits June 24, 2022 15:01
Refactored the way an observationString is created
Three variables added to better describe the state
Generated new playthrough to match the Observer class
Removed unimportant commented code
@lanctot
Copy link
Collaborator

lanctot commented Jun 24, 2022

Thanks! There's a conflict in pyspiel_test.py, can you resolve it?

@lanctot lanctot mentioned this pull request Jun 27, 2022
Copy link
Collaborator

@lanctot lanctot left a comment

Choose a reason for hiding this comment

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

Can you add the game and its description to the games page? (docs/games.md)

@lanctot
Copy link
Collaborator

lanctot commented Jul 2, 2022

(You can ignore the failing wheels tests, they will be fixed once #873 is merged on Monday.. but you will have to rerun ./install.sh locally)



std::vector<VirtualPoint> HandicapStones(int num_handicap) {
if (num_handicap < 2 || num_handicap > 9) return {};
Copy link
Member

Choose a reason for hiding this comment

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

Would be better to raise an error in most of these cases.

open_spiel/games/phantom_go.cc Outdated Show resolved Hide resolved
open_spiel/games/phantom_go.cc Outdated Show resolved Hide resolved
open_spiel/games/phantom_go.cc Outdated Show resolved Hide resolved
@Syor
Copy link
Contributor Author

Syor commented Jul 21, 2022

@lanctot > Can you add the game and its description to the games page? (docs/games.md)

Sorry for the delay. I am not sure what is the proper way to edit it, is there some editor?

@lanctot
Copy link
Collaborator

lanctot commented Jul 22, 2022

@lanctot > Can you add the game and its description to the games page? (docs/games.md)

Sorry for the delay. I am not sure what is the proper way to edit it, is there some editor?

It is plain text file. But don't worry about it, I can do it on my end.

@lanctot
Copy link
Collaborator

lanctot commented Jul 22, 2022

Hi @Syor , the test is failing because it appears there is some Linux-specific code (specifically, the header <asm-generic/ioctls.h>. Can you modify the test so that it can run on MacOS?

@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 Aug 4, 2022
@OpenSpiel OpenSpiel merged commit 0d32821 into google-deepmind:master Aug 8, 2022
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.

None yet

4 participants