Opened 12 months ago

Closed 12 months ago

Last modified 12 months ago

#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:

Status badges

Description

tarball - see checksums.ini

Change History (18)

comment:1 Changed 12 months ago by dimpase

  • Summary changed from conver cunningham_tables to a new style package to convert cunningham_tables to a new style package

comment:2 Changed 12 months ago by git

  • Commit changed from 19095288fdd60da4a075ac9c0f9447303e93880d to 5066177ea5f0119ab8363d07ac925b6133c5d105

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

5066177convert cunningham_tables to a new style package

comment:3 Changed 12 months ago by git

  • Commit changed from 5066177ea5f0119ab8363d07ac925b6133c5d105 to ede9b61fd7c151babd1853d4995e7395db41f5f2

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

ede9b61convert cunningham_tables to a new style package

comment:4 Changed 12 months ago by mkoeppe

Is main.gz really supposed to be on the branch?

comment:5 Changed 12 months ago by git

  • Commit changed from ede9b61fd7c151babd1853d4995e7395db41f5f2 to 1c9e941ed315cafa15eb3913b577c91aaba4ee51

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

1c9e941convert cunningham_tables to a new style package

comment:6 Changed 12 months ago by dimpase

  • Status changed from new to needs_review

comment:7 Changed 12 months ago by git

  • Commit changed from 1c9e941ed315cafa15eb3913b577c91aaba4ee51 to dcae6e74e1998d3e663e2483ea5bd335ee01eb7d

Branch pushed to git repo; I updated commit sha1. New commits:

dcae6e7correct doctest tags

comment:8 Changed 12 months ago by dimpase

doctests now are back, and work, too.

comment:9 Changed 12 months ago by mkoeppe

  • Reviewers set to Matthias Koeppe
  • Status changed from needs_review to positive_review

comment:10 Changed 12 months ago by slelievre

  • Cc jdemeyer added

comment:11 follow-up: Changed 12 months ago by 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.

comment:12 Changed 12 months ago by slelievre

Also, a download url at archive.org scares me a little as brittle.

comment:13 Changed 12 months ago by slelievre

Software Heritage to the rescue?

comment:15 in reply to: ↑ 11 ; follow-up: Changed 12 months ago by dimpase

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 12 months ago by slelievre

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 12 months ago by vbraun

  • Branch changed from u/dimpase/packages/cutables to dcae6e74e1998d3e663e2483ea5bd335ee01eb7d
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:18 Changed 12 months ago by dimpase

  • 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.

Last edited 12 months ago by dimpase (previous) (diff)
Note: See TracTickets for help on using tickets.