Opened 11 years ago
Closed 4 years ago
#5768 closed defect (invalid)
improve coverage of generators.pyx
Reported by: | was | Owned by: | mabshoff |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | doctest coverage | Keywords: | |
Cc: | mvngu | Merged in: | |
Authors: | Reviewers: | Jeroen Demeyer | |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
Change History (9)
comment:1 Changed 11 years ago by
comment:2 Changed 10 years ago by
- Cc mvngu added
- Report Upstream set to N/A
Current status of that function:
cpdef get_from_index(self, i): """ EXAMPLES: sage: from sage.structure.generators import Generators_list sage: gens = Generators_list(ZZ, [5,9], Rings) sage: gens.get_from_index(0) 5 """
Though still
SCORE /Users/.../sage-4.4.2/devel/sage/sage/structure/generators.pyx: 11% (5 of 45) Missing documentation: * __init__(self, Generators gens): * next(self): * __iter__(self): * __init__(self, obj, index_set, category): * get_from_index(self, i): * __contains__(self, x): * __call__(self, i): * __getitem__(self, i): * __iter__(self): * __len__(self): * index_set(self): * category(self): * obj(self): * count(self): * list(self): * _repr_(self): * count(self): * __cmp__(self, other_unty): * _repr_(self): * __reduce__(self): * make_finite_gens(obj, n, index_set, category): * __reduce__(self): * __cmp__(left, _right): * _repr_(self): * __init__(self, obj, category): * count(self): * __getitem__(self, i): * __init__(self, obj, category = None): * get_from_index(self, i): * __contains__(self, x): * __iter__(self): * index_set(self): * count(self): * list(self): * __init__(self, G, index_set, category, computing_function): * get_from_index(self, i): * count(self): * list(self): * _repr_(self): Missing doctests: * __init__(self, obj, n, index_set, category): Possibly wrong (function name doesn't occur in doctests): * make_list_gens(*args):
So there is yet work to be done.
comment:3 Changed 7 years ago by
Quick note, it seems this file is currently in 5.7.beta3
referenced only in structure/category_object.pyx
to handle the to-be-deprecated^{(tm)} ParentWithGens
. So this file might be obsolete...
comment:4 Changed 7 years ago by
- Milestone changed from sage-5.11 to sage-5.12
comment:5 Changed 6 years ago by
- Milestone changed from sage-6.1 to sage-6.2
comment:6 Changed 6 years ago by
- Milestone changed from sage-6.2 to sage-6.3
comment:7 Changed 6 years ago by
- Milestone changed from sage-6.3 to sage-6.4
comment:8 Changed 4 years ago by
- Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
- Status changed from new to needs_review
This module will be completely removed in #21382.
comment:9 Changed 4 years ago by
- Resolution set to invalid
- Reviewers set to Jeroen Demeyer
- Status changed from needs_review to closed
Note: See
TracTickets for help on using
tickets.
This file is just scary. The second function I tried to document is:
I couldn't find any object in all of Sage with a
_gen_
method -- I see now way this could ever work. Hence I can't see any wayGenerators_naturals
would ever work. This whole file just scares me. I don't get it at all.I give up.
This file was evidently 100% part of this changeset:
I rue the day I ever allowed a single non-doctested function into Sage.