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
Speed up graph generation using Cython #9143
Comments
comment:1
Every time I ask about "yield" statements in Cython, I always get the same response: very close, not there yet. It will be very easy for me to fix this, once they are implemented. Until then, it's probably ten times the work... |
comment:2
Ah, okay, thanks for the update. I'm CCing robertwb so that he sees (yet again) another vote for yield statements in Cython. I'm sure they'll come in time. |
comment:3
Well, we're very close, but not there yet ;). Realistically, I'm 90% sure they'll be implemented by the time the summer is out, now that I don't have to spend every waking minute on my thesis. |
comment:4
Replying to @robertwb:
+1! I was also going to mention in response to
that I believe in one-person-one-vote, and I must disclose that I have already voted for this :) |
comment:5
This is now part of #9559. |
comment:10
As mentioned in comment:5, the problem has been fixed, so this ticket can be closed as a duplicate of #9559.
|
Reviewer: Dave Morris |
It's amazing how slow our graph generator is:
Compare this to nauty's C implementation (with approximately the same algorithm)
Notice that the vast majority of the time is spent in some python calls, which presumably could be much, much faster if we instead used the underlying C structure via Cython.
CC: @nathanncohen @rlmill @robertwb
Component: graph theory
Reviewer: Dave Morris
Issue created by migration from https://trac.sagemath.org/ticket/9143
The text was updated successfully, but these errors were encountered: