Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Typo in the docstring for echelon_coordinate_vector in FreeModules Documentation #13780

Closed
KPanComputes opened this issue Nov 30, 2012 · 6 comments

Comments

@KPanComputes
Copy link

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."

Component: documentation

Keywords: FreeModule, echelon_coordinate_vector

Author: Kannappan Sampath

Reviewer: Julian Rueth

Merged: sage-5.8.beta2

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

@KPanComputes
Copy link
Author

Attachment: 13780_echelon_coordinate_vector.patch.gz

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

@saraedum
Copy link
Member

comment:2

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.

@saraedum
Copy link
Member

Reviewer: Julian Rueth

@saraedum
Copy link
Member

Author: Kannappan Sampath

@KPanComputes
Copy link
Author

comment:3

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... :)

@jdemeyer
Copy link

Merged: sage-5.8.beta2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants