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

Implement Xiangqi game #802

Closed
huynq55 opened this issue Mar 13, 2022 · 2 comments
Closed

Implement Xiangqi game #802

huynq55 opened this issue Mar 13, 2022 · 2 comments
Labels
contribution welcome It's a nice feature! But we do not have the time to do it ourselves. Contribution welcomed!

Comments

@huynq55
Copy link

huynq55 commented Mar 13, 2022

Hello, I want to implement the Xiangqi game. I'm referring the Chess game implementation as an example for Xiangqi. For chess, there is chess folder which has files like chess_board, chess-common. I want to create the same xiangqi folder with same files (my programming skill is not so good, so i'm just mimicking Chess). Will the files be recognized by the compiler when I run build_and_run_tests.sh? How can I make sure that my implementation will be merged into open_spiel? Thanks!

@lanctot
Copy link
Collaborator

lanctot commented Mar 14, 2022

Hi @huynq55,

Great!

The best place to start is the "Adding a game" section of the developer guide: https://github.com/deepmind/open_spiel/blob/master/docs/developer_guide.md

The files will be found if they are added to the appropriate CMakeLists.txt files. I recommend copying what is done in chess there as well.

Once you are happy with the implementation and all the tests pass, please submit a pull request. Then we will review it and probably ask you to change a few things and then once the fixes are all done we will import it and it gets merged after that.

@lanctot lanctot added the contribution welcome It's a nice feature! But we do not have the time to do it ourselves. Contribution welcomed! label Mar 25, 2022
@lanctot lanctot closed this as completed Nov 30, 2022
@lanctot lanctot reopened this Nov 30, 2022
@lanctot
Copy link
Collaborator

lanctot commented Nov 30, 2022

I've added this game to Call for New Games thread, so closing this one for now until anybody volunteers to work on it. Thanks for the suggestion!

@lanctot lanctot closed this as completed Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution welcome It's a nice feature! But we do not have the time to do it ourselves. Contribution welcomed!
Projects
None yet
Development

No branches or pull requests

2 participants