Change all occurrences of "method" to "algorithm" in coding/code_bounds.py

[rbeezer]

This is a follow-on to #6094. More places where the keyword argument "method=" should be changed to "algorithm=".

CC: @wdjoyner

Component: coding theory

Author: Rob Beezer

Reviewer: Minh Van Nguyen

Issue created by migration from https://trac.sagemath.org/ticket/7971

[rbeezer]

comment:1

This patch will build, and the tests in sage/coding all pass. But I don't have Guava installed, which is needed for most of the affected doctests.

David - maybe you can run the optional tests as part of a review?

[rbeezer]

Author: Rob Beezer

[rbeezer]

Attachment: trac_7971_method_algorithm.patch.gz

[sagetrac-mvngu]

comment:3

Passes all doctests on Sage 4.3.1.rc0 with or without the patch. To run the optional doctests that require Guava: After applying the patch, install the optional Guava package by installing the package gap_packages-4.4.10_6.spkg. Running doctest on "sage/coding/code_bounds.py" with the options

-t -long -optional

results in all doctests passed. Positive review.

[sagetrac-mvngu]

Reviewer: Minh Van Nguyen

[rbeezer]

comment:4

This needs work (deprecation warnings) and so should just be handled as part of #6094.

I've marked this as "needs work" but should be marked some form of invalid.

[johanrosenkilde]

comment:5

The currently newest uploaded patch for Trac #6094 -- needing a positive review -- handles all these cases, so this Trac should just be closed.