#20565: Fix LinearCode.wtdist_gap method
-------------------------------------+-------------------------------------
Reporter: arpitdm | Owner: arpitdm
Type: defect | Status: needs_work
Priority: minor | Milestone: sage-7.3
Component: coding theory | Resolution:
Keywords: | Merged in:
Authors: Arpit Merchant | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/arpitdm/fix_wtdist_gap_method | 111426739cd1bb4eb395117ff5000d9aafdc23ac
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by dlucas):
> One more thing. We need to have the original wtdist_gap documentation
also. The doctest there fails with the following message
Usually, I just delete the old documentation and leave only the code of
the old method.
See for instance what I did with `ReedSolomonCode` in
`code_constructions.py`.
My argument is the following: as it's an old feature we keep only for
retrocompatibility purposes, I don't see the point of keeping its
documentation. Indeed, users which have old code using the deprecated
feature already know how the old method works and are using it correctly,
so they don't need the documentation. On any other case, people are not
supposed to use the old feature, and maintaining proper documentation for
an old feature might confuse them and lead them to use it - while we don't
want them to do that!
> When I run the wtdist_gap method, it gives a bunch of other deprecation
warnings. Are these fine or do I need to change anything here?
That's fine, don't worry!
David
--
Ticket URL: <http://trac.sagemath.org/ticket/20565#comment:16>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
