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

Add first draft of OpenCADD-KLIFS paper #112

Merged
merged 21 commits into from
Nov 8, 2021
Merged

Conversation

dominiquesydow
Copy link
Contributor

@dominiquesydow dominiquesydow commented Oct 27, 2021

Description

Draft for OpenCADD-KLIFS paper (to be submitted to JOSS).

Todos

Notable points that this PR has either accomplished or will accomplish.

  • Read https://joss.readthedocs.io/en/latest/submitting.html
  • Add Summary: "A summary describing the high-level functionality and purpose of the software for a diverse, non-specialist audience."
  • Add Statement of Need: "A Statement of Need section that clearly illustrates the research purpose of the software."
    • Useful to facilitate KLIFS-based kinase research
    • No other KLIFS-dedicated Python packages available to our knowledge, however dynamically generated Python API via bravado
  • Add OpenCADD-KLIFS TOC figure
  • Check paper rendering here: https://whedon.theoj.org/
  • Update OpenCADD docs

Questions

  • Submitting submodules to JOSS possible? Has been done in the past - also read here.
  • Name OpenCADD-KLIFS ok?
  • @jaimergp are you good with being a co-author? He is :)

Status

  • Ready to go

@dominiquesydow dominiquesydow self-assigned this Oct 27, 2021
@codecov-commenter
Copy link

codecov-commenter commented Oct 27, 2021

Codecov Report

Merging #112 (205821e) into master (ce20b50) will decrease coverage by 0.00%.
The diff coverage is n/a.

Copy link
Member

@AndreaVolkamer AndreaVolkamer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dominiquesydow really great mini-paper!!!
Small comments in line.

papers/opencadd-klifs/paper.md Show resolved Hide resolved
papers/opencadd-klifs/paper.md Outdated Show resolved Hide resolved
papers/opencadd-klifs/paper.md Show resolved Hide resolved
papers/opencadd-klifs/paper.md Outdated Show resolved Hide resolved
papers/opencadd-klifs/paper.md Outdated Show resolved Hide resolved
papers/opencadd-klifs/paper.md Outdated Show resolved Hide resolved
papers/opencadd-klifs/paper.md Outdated Show resolved Hide resolved
papers/opencadd-klifs/paper.md Outdated Show resolved Hide resolved
papers/opencadd-klifs/paper.md Show resolved Hide resolved
papers/opencadd-klifs/paper.md Outdated Show resolved Hide resolved
@dominiquesydow dominiquesydow added the module-databases-klifs Concerns opencadd.databases.klifs module label Nov 1, 2021
@jaimergp
Copy link
Contributor

jaimergp commented Nov 1, 2021

Added a couple of tiny comments! It reads super well and goes straight to the point. Like the code behind! Awesome!

@dominiquesydow dominiquesydow merged commit 8a6d164 into master Nov 8, 2021
@dominiquesydow dominiquesydow deleted the opencadd-klifs-paper branch November 8, 2021 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module-databases-klifs Concerns opencadd.databases.klifs module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants