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

Add graal to CI #220

Closed
wants to merge 1 commit into from
Closed

Add graal to CI #220

wants to merge 1 commit into from

Conversation

masklinn
Copy link
Contributor

@masklinn masklinn commented Sep 29, 2024

For reference, as of opening this PR, the run tests step takes:

Notes:

  • graal-24 seems to run a lot faster than graal-23 did, I remember much worse from the previous attempt, even if it's still a good 4x behind cpython it's competitive with pypy
  • although the overall CI run is way behind (by a minute) because graal has much higher "set up python" and "install test dependencies" overhead, the former is almost certainly because doesn't find graal in cache (why tho?), the latter I'm less sure both have to build a pyyaml wheel from src dist but graal triggers a bunch of downloads & tries to install libyaml-dev so there might be missing bits for graal in the CI setup

Caveats:

  • cpython also runs re2 tests, though that ought be basically nil compared to running the "pure python" test suite twice

@masklinn
Copy link
Contributor Author

masklinn commented Oct 8, 2024

Handled as part of #217

@masklinn masklinn closed this Oct 8, 2024
@masklinn masklinn deleted the enable-graal branch October 8, 2024 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant