Opened 8 years ago
Closed 8 years ago
#13780 closed defect (fixed)
Typo in the docstring for echelon_coordinate_vector in FreeModules Documentation
Reported by: | knsam | Owned by: | mvngu |
---|---|---|---|
Priority: | trivial | Milestone: | sage-5.8 |
Component: | documentation | Keywords: | FreeModule, echelon_coordinate_vector |
Cc: | Merged in: | sage-5.8.beta2 | |
Authors: | Kannappan Sampath | Reviewers: | Julian Rueth |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
There is a minor typo in the docstring for the function: echelon_coordinate_vector
in the FreeModules? reported in the sage-support mailing list https://groups.google.com/d/topic/sage-support/qhzchfrq-5M/discussion by Philipp Kerling.
The description of the function "echelon_coordinate_vector" of sage.modules.free_module.FreeModule_submodule_with_basis_pid says: "Write v in terms of the user basis for self." while it should correctly read: "Write v in terms of the echelonized basis for self."
Attachments (1)
Change History (6)
Changed 8 years ago by
comment:1 Changed 8 years ago by
- Status changed from new to needs_review
comment:2 follow-up: ↓ 3 Changed 8 years ago by
- Reviewers set to Julian Rueth
The patchbot complained about this patch which does not make much sense. I told it to rerun tests. Let's see what happens this times.
comment:3 in reply to: ↑ 2 Changed 8 years ago by
Replying to saraedum:
The patchbot complained about this patch which does not make much sense. I told it to rerun tests. Let's see what happens this times.
Thanks for looking into this! The patchbot seems to be happy now, inferring from its green light... :)
comment:4 Changed 8 years ago by
- Status changed from needs_review to positive_review
comment:5 Changed 8 years ago by
- Merged in set to sage-5.8.beta2
- Resolution set to fixed
- Status changed from positive_review to closed
Trivial Documentation Fix. Easy to review, hence. :)