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

Observation model #22

Merged
merged 21 commits into from
Sep 2, 2020
Merged

Observation model #22

merged 21 commits into from
Sep 2, 2020

Conversation

t-kimber
Copy link
Contributor

@t-kimber t-kimber commented Aug 17, 2020

Description

This PR deals with the observation model implemented in kinoml (theory and implementation)

Todos

  • check observation model in the api for all 4 measurement types (and measurement units)
    • pIC50
    • pKi
    • pKd
    • %displ.
  • adapt kinoml workflow (with theory and image)

Question

  • Where should the explanation (the theory behind the observation model) live?

Status

  • Ready to go

@t-kimber
Copy link
Contributor Author

t-kimber commented Aug 19, 2020

TO BE CHECKED!

In:

@t-kimber t-kimber self-assigned this Aug 19, 2020
@t-kimber t-kimber mentioned this pull request Aug 19, 2020
# FIXME: this might be upside down -- check!
import torch

return 100 * (1 - 1 / (1 + inhibitor_conc / torch.exp(dG_over_KT)))
return 100 * (1 - 1 / (1 +1 / torch.exp(dG_over_KT)))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do we need to hardcode this? Don't we have the argument in the function? Also, why are we changing units now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I need to speak with John for this observation model. I thought that this measurement type was in the clear, but I realized at the end of last week that something is fishy and I need to loop in John asap.

kinoml/core/measurements.py Show resolved Hide resolved
kinoml/core/measurements.py Show resolved Hide resolved
@jaimergp
Copy link
Collaborator

jaimergp commented Sep 1, 2020

RuntimeError: Found dtype Double but expected Float

This should be fixed by last commits.

@jaimergp jaimergp mentioned this pull request Sep 1, 2020
5 tasks
@jaimergp jaimergp mentioned this pull request Sep 2, 2020
16 tasks
@jaimergp jaimergp merged commit 7133238 into api Sep 2, 2020
@jaimergp jaimergp deleted the observation-model branch September 2, 2020 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants