State of cleaning up Linear Codes

Issue #121 new
David Lucas repo owner created an issue

A list of methods in linear_code.py that should be changed is available in https://bitbucket.org/lucasdavid/sage_coding_project/wiki/Exhaustive%20list%20of%20problems%20with%20methods%20in%20linear_code.py

As some presents naming issues, documentation issues, forced use of an optional package and/or weird code, it's definitely a part of the clean-up session.

Comments (4)

  1. David Lucas reporter
    • Removed useless commented code, rewrote __eq__ and dual_code. Changes available in trac #19492.

    Some timings are given below: Dual_codes.png eq_comparison.png

    Edit -- Of course, one should read "eq" instead of "dual" in the second plot collection's legend...

  2. David Lucas reporter

    dual_code and __eq__ cleaned-up in trac #19492 positive reviewed. Some commented lines removed in trac #19492 positive reviewed. I know have a generic implementation of __ne__ in trac #19743 closed.

    Wiki page updated accordingly.

  3. Log in to comment