Skip to content

Commit

Permalink
Trac #12673: Deprecate sage.media.wav, since it is totally undocteste…
Browse files Browse the repository at this point in the history
…d, and evidently full of confusing bugs

As I think http://480.sagenb.org/home/pub/137/ amply illustrates, the
mere existence of the untested and very buggy module sage.media.wav
causes more confusion than it is worth for now.  It would be best to
remove it from Sage, and only add something back as a result of a new
(probably student) project to do things right.

URL: https://trac.sagemath.org/12673
Reported by: was
Ticket author(s): Matthias Koeppe
Reviewer(s): Travis Scrimshaw
  • Loading branch information
Release Manager committed Aug 29, 2021
2 parents 1baf0e7 + 2bf2bd0 commit c326905
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/sage/all.py
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,7 @@

from sage.games.all import *

from sage.media.all import *
lazy_import('sage.media.wav', 'Wave', as_='wave', deprecation=12673)

from sage.logic.all import *

Expand Down
2 changes: 2 additions & 0 deletions src/sage/media/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
from sage.misc.superseded import deprecation
deprecation(12673, "the package sage.media is deprecated")
9 changes: 9 additions & 0 deletions src/sage/media/wav.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,15 @@
- Bobby Moretti and Gonzalo Tornaria (2007-07-01): First version
- William Stein (2007-07-03): add more
- Bobby Moretti (2007-07-03): add doctests
This module (and all of ``sage.media``) is deprecated.
EXAMPLES::
sage: import sage.media
doctest:warning...
DeprecationWarning: the package sage.media is deprecated
See http://trac.sagemath.org/12673 for details.
"""

import math
Expand Down

0 comments on commit c326905

Please sign in to comment.