-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
MultipleSeqAlignment position mapping #4249
base: master
Are you sure you want to change the base?
Conversation
…q` when it returns a `str`
…apping between alignment indexes and original sequence positions
…fferent seq) position
The automated checks are failing due to your Python style, please see https://github.com/biopython/biopython/blob/master/CONTRIBUTING.rst - at least try running the tool black on your Python code. |
@@ -904,13 +1093,13 @@ def substitutions(self): | |||
|
|||
Any weights associated with the sequences are taken into account when | |||
calculating the substitution matrix. For example, given the following | |||
multiple sequence alignment:: | |||
multiple sequence alignment: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please revert this change, the double colon is RST markup indicating the following indented code is a literal block. Same for the change below.
I wonder if it would be best to more closely match the new alignment object's mapping via |
I hereby agree to dual licence this and any previous contributions under both
the Biopython License Agreement AND the BSD 3-Clause License.
I have read the
CONTRIBUTING.rst
file, have runpre-commit
locally, and understand that continuous integration checks will be used to
confirm the Biopython unit tests and style checks pass with these changes.
I have added my name to the alphabetical contributors listings in the files
NEWS.rst
andCONTRIB.rst
as part of this pull request, am listedalready, or do not wish to be listed. (This acknowledgement is optional.)
Closes #4248