sagemath / sage

Main repository of SageMath
https://www.sagemath.org
Other
1.43k stars 479 forks source link

sage-coverage screws up with lambda functions as default arguments #4005

Closed 89c6e537-b2e3-45e6-882d-d4957b74ffe5 closed 16 years ago

89c6e537-b2e3-45e6-882d-d4957b74ffe5 commented 16 years ago

Component: doctest coverage

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

89c6e537-b2e3-45e6-882d-d4957b74ffe5 commented 16 years ago

Apply to scripts repo.

85eec1a4-3d04-4b4d-b711-d4db03337c41 commented 16 years ago
comment:1

Attachment: trac_4005_sage-coverage.patch.gz

This patch is motivated by

    def min_spanning_tree(self, weight_function=lambda e: 1,
                          algorithm='Kruskal',
                          starting_vertex=None ):

from sage/graphs/graph.py.

Positive review.

Cheers,

Michael

85eec1a4-3d04-4b4d-b711-d4db03337c41 commented 16 years ago
comment:2

rlm points out that this is not perfect, but it fixes this one specific issue.

Cheers,

Michael

85eec1a4-3d04-4b4d-b711-d4db03337c41 commented 16 years ago
comment:3

Merged in Sage 3.1.2.alpha3