Opened 8 years ago

Closed 8 years ago

#16568 closed task (fixed)

remove desolve_system_strings()

Reported by: Ralf Stephan Owned by:
Priority: minor Milestone: sage-6.4
Component: calculus Keywords: deprecation, removal, beginner
Cc: Merged in:
Authors: Sergey Bykov Reviewers: Karl-Dieter Crisman
Report Upstream: N/A Work issues:
Branch: 7ef2572 (Commits, GitHub, GitLab) Commit: 7ef2572d4c80968c6b44cc926ff4e48491fd35c1
Dependencies: Stopgaps:

Status badges

Description

kcrisman in comment 54 of #8734:

... see #8132 where it was first said to be obsolete, and it hasn't been in the global namespace since before 2010. That is pretty much equivalent to a deprecation to me. However, we should keep any non-overlapping examples ...

Change History (14)

comment:1 Changed 8 years ago by Karl-Dieter Crisman

Keywords: beginner added
Priority: majorminor

comment:2 Changed 8 years ago by For batch modifications

Milestone: sage-6.3sage-6.4

comment:3 Changed 8 years ago by Sergey Bykov

Branch: u/captaintrunky/remove_desolve_system_strings__

comment:4 Changed 8 years ago by Sergey Bykov

Authors: Sergey Bykov
Commit: b9e84768a23321550136de609e8d88cb06aca79d
Status: newneeds_review

New commits:

b9e8476Obsolete function desolve_system_string is removed

comment:5 Changed 8 years ago by Karl-Dieter Crisman

Hi! Thanks for joining Sage development. Just curious - do you think that any of the examples in this function are worth rescuing for doctesting purposes (I mean if they were changed to the standard desolve syntax)? It's always worth checking for this.

comment:6 in reply to:  5 Changed 8 years ago by Sergey Bykov

Replying to kcrisman:

Hi! Thanks for joining Sage development. Just curious - do you think that any of the examples in this function are worth rescuing for doctesting purposes (I mean if they were changed to the standard desolve syntax)? It's always worth checking for this.

Hi, thanks. I'm going to agree with you on this. It is not so time consuming to modify and add these little tests, while an additional feedback is often useful.

comment:7 Changed 8 years ago by git

Commit: b9e84768a23321550136de609e8d88cb06aca79d210b69f474129b1c7b3b29c1547f046da0aa96ba

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

210b69fAdditional test for 'desolve()' method have been recovered from obsolete 'desolve_system_string()' routine

comment:8 Changed 8 years ago by Karl-Dieter Crisman

Status: needs_reviewneeds_work

Nice! But you need to do a bit more indentation for the last block, and should probably do :trac:`16568` for the Trac reference.

comment:9 Changed 8 years ago by git

Commit: 210b69f474129b1c7b3b29c1547f046da0aa96ba32cbad4d5f08dccc45f1bc0c3cfdbdf455cb5973

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

32cbad4Formatting fixes

comment:10 Changed 8 years ago by Sergey Bykov

Status: needs_workneeds_review

comment:11 Changed 8 years ago by Karl-Dieter Crisman

Dedent

Now type show(P1), show(P2) to view these plots.

so it is at the same indentation level as other text for the documentation to look right. But otherwise this is great.

comment:12 Changed 8 years ago by git

Commit: 32cbad4d5f08dccc45f1bc0c3cfdbdf455cb59737ef2572d4c80968c6b44cc926ff4e48491fd35c1

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

7ef2572Formatting fixes

comment:13 Changed 8 years ago by Karl-Dieter Crisman

Reviewers: Karl-Dieter Crisman
Status: needs_reviewpositive_review

comment:14 Changed 8 years ago by Volker Braun

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