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:

Status badges

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)

13780_echelon_coordinate_vector.patch (641 bytes) - added by knsam 8 years ago.
Trivial Documentation Fix. Easy to review, hence. :)

Download all attachments as: .zip

Change History (6)

Changed 8 years ago by knsam

Trivial Documentation Fix. Easy to review, hence. :)

comment:1 Changed 8 years ago by knsam

  • Status changed from new to needs_review

comment:2 follow-up: Changed 8 years ago by saraedum

  • Authors set to Kannappan Sampath
  • 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 knsam

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 saraedum

  • Status changed from needs_review to positive_review

comment:5 Changed 8 years ago by jdemeyer

  • Merged in set to sage-5.8.beta2
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.