Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add support for GraalPy#434

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
maurycy wants to merge2 commits intopython:main
base:main
Choose a base branch
Loading
frommaurycy:graalpy
Draft

Conversation

@maurycy
Copy link
Contributor

@maurycymaurycy commentedOct 26, 2025
edited
Loading

It's an alternative Python implementation, as perpython/pythondotorg#2797 and supported byhttps://github.com/actions/setup-python

@maurycymaurycy changed the titleAdd support forgraalpy-24.0Add support for GraalPyOct 26, 2025
@maurycymaurycy marked this pull request as draftOctober 26, 2025 21:48
@maurycy
Copy link
ContributorAuthor

@timfel What do you think?

It mostly worked:

5 benchmarks failed:- dask (Benchmark died)- gc_collect (Benchmark died)- networkx (Benchmark died)- networkx_connected_components (Benchmark died)- networkx_k_core (Benchmark died)

It wasextremely slow:

I cancelled it after an hour.

Is it premature? Does it require much more powerful runner?

@timfel
Copy link

@maurycy we run the pyperformance benchmarks internally, but they require a powerful runner and lots more warmup than PyPy or CPython per benchmark. One hour is not nearly enough, we give it 18 cores and 64G of RAM and it runs in about 2.5 hours

@timfel
Copy link

As for the benchmarks, the networkx benchmarks should work, that's a bug on our side. But dask and gc_collect as are won't, the GC benchmarks in general just don't make much sense on GraalPy, since we have a completely different GC, so neither the benchmark nor any assertions make sense for us. The dask one doesn't work because we don't support thedis module, we have no current plans for this. So those would have to be excluded for GraalPy

@hugovk
Copy link
Member

This also increases the CI runs from around13 minutes to over2.5 hours.

@maurycy
Copy link
ContributorAuthor

maurycy commentedOct 27, 2025
edited
Loading

@maurycy we run the pyperformance benchmarks internally, but they require a powerful runner and lots more warmup than PyPy or CPython per benchmark. One hour is not nearly enough, we give it 18 cores and 64G of RAM and it runs in about 2.5 hours

That was just tests, though:

Other tests finished in ~10 minutes

FYI: The whole benchmark runs in ~1h on cpython on i9-12900K, 128G DDR4

@timfel
Copy link

@maurycy if it's anything like in our internal setup, running the pip subprocesses to install dependencies is easily the worst part of the runtime. Thesecreating venv for benchmark steps are easily 20x slower than on CPython, and when 2s turn into 40s times ~90 benchmarks this fills the time quickly.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@maurycy@timfel@hugovk

[8]ページ先頭

©2009-2026 Movatter.jp