Opened 7 years ago

Closed 7 years ago

#20216 closed enhancement (fixed)

Deprecate PARI nth_prime, prime_list, primes_up_to_n

Reported by: Jeroen Demeyer Owned by:
Priority: major Milestone: sage-7.1
Component: interfaces Keywords:
Cc: Luca De Feo Merged in:
Authors: Jeroen Demeyer Reviewers: Luca De Feo
Report Upstream: N/A Work issues:
Branch: e4858c6 (Commits, GitHub, GitLab) Commit: e4858c69f9f5e5e07a8577f96d844265dddd9c52
Dependencies: #20210, #20205, #20213 Stopgaps:

Status badges

Description (last modified by Jeroen Demeyer)

Use the standard PARI/GP names prime() and primes() instead.

Change History (15)

comment:1 Changed 7 years ago by Jeroen Demeyer

Dependencies: #20213#20210, #20205, #20213

comment:2 Changed 7 years ago by Jeroen Demeyer

Description: modified (diff)
Summary: Deprecate PARI nth_primeDeprecate PARI nth_prime and prime_list

comment:3 Changed 7 years ago by Jeroen Demeyer

Description: modified (diff)

comment:4 Changed 7 years ago by Jeroen Demeyer

Description: modified (diff)

comment:5 Changed 7 years ago by Jeroen Demeyer

Description: modified (diff)

comment:6 Changed 7 years ago by Jeroen Demeyer

Summary: Deprecate PARI nth_prime and prime_listDeprecate PARI nth_prime, prime_list, primes_up_to_n

comment:7 Changed 7 years ago by Jeroen Demeyer

Branch: u/jdemeyer/deprecate_pari_nth_prime

comment:8 Changed 7 years ago by Jeroen Demeyer

Commit: d5c934c5164255e352cee9ddbc780d0759b6e185
Status: newneeds_review

New commits:

03458eaUpgrade cysignals package
dce67fcMove memory functions to cysignals
4bb8337Rename sage_malloc -> sig_malloc and friends
5ab73c1Get rid of factorint_withproof_sage in PARI interface
c0ed97aStop using deprecated PARI factoring features
edc5ce2Merge branch 't/20205/get_rid_of_factorint_withproof_sage_in_pari_interface' into HEAD
5fb408dReplace pari_catch_sig_on by sig_on
d5c934cDeprecate PARI nth_prime and prime_list

comment:9 Changed 7 years ago by Luca De Feo

Shouldn't the doc

Return the vector of the first `n` primes, the primes in the
interval `[n, end]` or the list of primes up to `end`.

read

Return the pari vector of the first `n` primes, the primes in the
interval `[n, end]` or the pari vector of primes up to `end`.

instead?

comment:10 Changed 7 years ago by Luca De Feo

Branch: u/jdemeyer/deprecate_pari_nth_primeu/defeo/deprecate_pari_nth_prime

comment:11 Changed 7 years ago by Luca De Feo

Commit: d5c934c5164255e352cee9ddbc780d0759b6e1857355781bf73e5798224ddfce2283e7e4dae3a436

I made the rewording. This ticket is good to go for me.


New commits:

7355781Reworded docstring

comment:12 Changed 7 years ago by git

Commit: 7355781bf73e5798224ddfce2283e7e4dae3a436e4858c69f9f5e5e07a8577f96d844265dddd9c52

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

e4858c6Reworded docstring

comment:13 Changed 7 years ago by Jeroen Demeyer

Status: needs_reviewneeds_work

comment:14 Changed 7 years ago by Luca De Feo

Reviewers: Luca De Feo
Status: needs_workpositive_review

Oups, sorry for the indentation nonsense.

comment:15 Changed 7 years ago by Volker Braun

Branch: u/defeo/deprecate_pari_nth_primee4858c69f9f5e5e07a8577f96d844265dddd9c52
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.