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.

Sign up for GitHub

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

Jump to bottom

Update docstring for KLIFS RMSD values #122

Merged
merged 2 commits into from
Dec 8, 2021
Merged

Update docstring for KLIFS RMSD values #122

merged 2 commits into from
Dec 8, 2021

Conversation

dominiquesydow
Copy link
Contributor

@dominiquesydow dominiquesydow commented Nov 17, 2021 •
edited
Loading

Description

Correct docstring for "structure.rmsd1" and "structure.rmsd2" in StructuresProvider class.

Todos

  • Update docstring
    • RMSD1: RMSD between the structure and the reference template for the conserved pocket residues
    • RMSD2: RMSD between the structure and the reference template for the full pocket

Questions

None

Status

  • Ready to go

@dominiquesydow dominiquesydow self-assigned this Nov 17, 2021
@dominiquesydow dominiquesydow added the module-databases-klifs Concerns opencadd.databases.klifs module label Nov 17, 2021
@codecov-commenter
Copy link

codecov-commenter commented Nov 17, 2021 •
edited
Loading

Codecov Report

Merging #122 (f71f812) into master (7f23496) will not change coverage.
The diff coverage is n/a.

@dominiquesydow dominiquesydow merged commit d46ca3e into master Dec 8, 2021
@dominiquesydow dominiquesydow deleted the fix-klifs-rmsd-docs branch December 8, 2021 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@dominiquesydow dominiquesydow

Labels
module-databases-klifs Concerns opencadd.databases.klifs module
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dominiquesydow @codecov-commenter