-
-
Notifications
You must be signed in to change notification settings - Fork 561
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
IntegerListsLex speedup #13840
Comments
comment:2
Fixed the doctest in |
This comment has been minimized.
This comment has been minimized.
Dependencies: #14045 |
Reviewer: Mike Hansen |
comment:5
Looks good to me. |
comment:6
How was this patch file created? It is missing a "User" field, you should create a |
Attachment: trac_13840-integer_list_speedup-ts.patch.gz Fixed doctest |
comment:7
Fixed. |
Merged: sage-5.7.beta3 |
This patch speeds up
IntegerListsLex
by removing calls to sage'sinfinity
and instead usesfloat('+inf')
andfloat('-inf')
.Before the patch:
After:
Depends on #14045
CC: @sagetrac-sage-combinat @nthiery
Component: combinatorics
Keywords: optimization, integerlistlex
Author: Travis Scrimshaw
Reviewer: Mike Hansen
Merged: sage-5.7.beta3
Issue created by migration from https://trac.sagemath.org/ticket/13840
The text was updated successfully, but these errors were encountered: