#29834 closed defect (fixed)
convert cunningham_tables to a new style package
Reported by: | dimpase | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-9.2 |
Component: | packages: optional | Keywords: | |
Cc: | mkoeppe, jdemeyer | Merged in: | |
Authors: | Dima Pasechnik | Reviewers: | Matthias Koeppe |
Report Upstream: | N/A | Work issues: | |
Branch: | dcae6e7 (Commits, GitHub, GitLab) | Commit: | |
Dependencies: | Stopgaps: |
Description
tarball - see checksums.ini
Change History (18)
comment:1 Changed 2 years ago by
- Summary changed from conver cunningham_tables to a new style package to convert cunningham_tables to a new style package
comment:2 Changed 2 years ago by
- Commit changed from 19095288fdd60da4a075ac9c0f9447303e93880d to 5066177ea5f0119ab8363d07ac925b6133c5d105
comment:3 Changed 2 years ago by
- Commit changed from 5066177ea5f0119ab8363d07ac925b6133c5d105 to ede9b61fd7c151babd1853d4995e7395db41f5f2
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
ede9b61 | convert cunningham_tables to a new style package
|
comment:4 Changed 2 years ago by
Is main.gz really supposed to be on the branch?
comment:5 Changed 2 years ago by
- Commit changed from ede9b61fd7c151babd1853d4995e7395db41f5f2 to 1c9e941ed315cafa15eb3913b577c91aaba4ee51
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
1c9e941 | convert cunningham_tables to a new style package
|
comment:6 Changed 2 years ago by
- Status changed from new to needs_review
comment:7 Changed 2 years ago by
- Commit changed from 1c9e941ed315cafa15eb3913b577c91aaba4ee51 to dcae6e74e1998d3e663e2483ea5bd335ee01eb7d
Branch pushed to git repo; I updated commit sha1. New commits:
dcae6e7 | correct doctest tags
|
comment:8 Changed 2 years ago by
doctests now are back, and work, too.
comment:9 Changed 2 years ago by
- Reviewers set to Matthias Koeppe
- Status changed from needs_review to positive_review
comment:10 Changed 2 years ago by
- Cc jdemeyer added
comment:11 follow-up: ↓ 15 Changed 2 years ago by
The file build/pkgs/cunningham_tables/read_cunningham_prime_factors.py
starts:
+# python2/Sage script used to create the package +# (needs porting to python3 Sage)
Was that done? If so, consider removing the comment -- in a follow-up ticket if it's too late here.
comment:12 Changed 2 years ago by
Also, a download url at archive.org scares me a little as brittle.
comment:13 Changed 2 years ago by
Software Heritage to the rescue?
comment:14 Changed 2 years ago by
comment:15 in reply to: ↑ 11 ; follow-up: ↓ 16 Changed 2 years ago by
Replying to slelievre:
The file
build/pkgs/cunningham_tables/read_cunningham_prime_factors.py
starts:+# python2/Sage script used to create the package +# (needs porting to python3 Sage)Was that done? If so, consider removing the comment -- in a follow-up ticket if it's too late here.
no, I added this comment -- this script is broken atm, in the sense it doesn't work with Python3-based Sage.
comment:16 in reply to: ↑ 15 Changed 2 years ago by
Replying to dimpase:
Replying to slelievre:
+# (needs porting to python3 Sage)Was that done? If so, consider removing the comment -- in a follow-up ticket if it's too late here.
no, I added this comment -- this script is broken atm, in the sense it doesn't work with Python3-based Sage.
Is there a follow-up ticket for that? Sage 9.2 will be Py3-only.
comment:17 Changed 2 years ago by
- Branch changed from u/dimpase/packages/cutables to dcae6e74e1998d3e663e2483ea5bd335ee01eb7d
- Resolution set to fixed
- Status changed from positive_review to closed
comment:18 Changed 2 years ago by
- Commit dcae6e74e1998d3e663e2483ea5bd335ee01eb7d deleted
the script in question is only needed to create the package source; as long as we are happy with what we have, no need to run it, so it can be left as it is for the time being.
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
convert cunningham_tables to a new style package