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

Change python_Serpent implementation #6

Open
wants to merge 5 commits into
base: python3
Choose a base branch
from

Conversation

koujiandou-mhfg
Copy link

python_Serpent uses pyserpent.py found on http://www.cl.cam.ac.uk/~fms27/serpent/. It is human readable but extremely slow.

I replaced pyserpent.py with serpent.py. Though the writer of this code says "These implementations are converted directly from the C code and are thus very ugly Python, but the code works although it’s slow," it is 250x faster than pyserpent.py I expect.

I made it more readable on 2086c20, it still runs 200x faster than pyserpent.py.

In the same way as #5, new version of serpent.py runs on Python 2.7 and Python 3.5.

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