Skip to content

Comments

Remove Julia, Rust, and Go API tests from default optional dependencies#1437

Closed
lanctot wants to merge 3 commits intomasterfrom
lanctot-patch-80
Closed

Remove Julia, Rust, and Go API tests from default optional dependencies#1437
lanctot wants to merge 3 commits intomasterfrom
lanctot-patch-80

Conversation

@lanctot
Copy link
Collaborator

@lanctot lanctot commented Jan 12, 2026 •
edited
Loading

  • Julia API test now failing, so remove building other language APIs as default optional dependencies (also remove Rust and Go as default optional dependency as they've been unmaintained for a long time)

This was referenced Jan 12, 2026
Closed
Closed
@lanctot lanctot changed the title Fix wheel tests - update requirements.txt to add upper-bound to scipy version Fix GitHub Actions CI wheel tests Jan 12, 2026
@alexunderch
Copy link
Contributor

alexunderch commented Jan 12, 2026

@lanctot I think it's time to take out intel wheels on master as well

@lanctot
Copy link
Collaborator Author

lanctot commented Jan 12, 2026

@lanctot I think it's time to take out intel wheels on master as well

Yes I made that part of #1424

@lanctot lanctot changed the title Fix GitHub Actions CI wheel tests Remove Julia, Rust, and Go API tests from default optional dependencies Jan 12, 2026
@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 Jan 12, 2026
@lanctot
Copy link
Collaborator Author

lanctot commented Jan 12, 2026

Applied in 49d9e2f

@lanctot lanctot closed this Jan 12, 2026
gmweinberg pushed a commit to gmweinberg/open_spiel that referenced this pull request Jan 12, 2026
@lanctot
Merge pull request google-deepmind#1437 from google-deepmind:lanctot-…
49d9e2f 
…patch-80

PiperOrigin-RevId: 855295543
Change-Id: I207f062d61e3f5f5adc85fc2bf37e9685f25902c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lanctot @alexunderch