ersilia-os / eos8fth

GNU General Public License v3.0
0 stars 0 forks source link

BioModel Discussion Issue for eos8fth #8

Open Zainab-ik opened 2 months ago

Zainab-ik commented 2 months ago

Summary !!!

GemmaTuron commented 2 months ago

Yes, we could consider that a similar approach as ZairaChem, good observation

Zainab-ik commented 2 months ago
Zainab-ik commented 2 months ago

@GemmaTuron The model is ready for review.

GemmaTuron commented 2 months ago

Hi @Zainab-ik

I do not understand why some target organisms are added as Taxon and others as Property. If you want to understand more about the output you can run the model from the web implementation even. I do not understand this question, can you clarify? What is not clear about the classifiers 24 different classifiers were trained for REDIAL, i'm confused on how that'd fit into the metadata. can we go with just classification models as the metadata. The publication is a Nature paper, so even if it is not in Pubmed, it is the right link to cite

In the methds they say: ll models were optimized and selected based on the validation F1 score. so there must be some crossvalidation happening

Zainab-ik commented 2 months ago
GemmaTuron commented 2 months ago

Hi @Zainab-ik

What is the In Vitro field? what does it refer to? And the Active in Assay and Inactive in Assay? I had not seen them before

Zainab-ik commented 2 months ago

Hi @Zainab-ik

What is the In Vitro field? what does it refer to? And the Active in Assay and Inactive in Assay? I had not seen them before

The invitro is for data generation and thus removed. The output of the model is either active in an assay or inactive in an assay. All redundant field removed.

GemmaTuron commented 2 months ago

Ok, wouldn't all model outputs be "active in an assay" or "inactive in an assay" how is this different from "Active" and "Inactive"? Maybe it is redundant?

Zainab-ik commented 2 months ago

Ok, wouldn't all model outputs be "active in an assay" or "inactive in an assay" how is this different from "Active" and "Inactive"? Maybe it is redundant?

It's actually the same ontology term that it is annotated with, and the output was based on Redial-2020 prediction. I don't think it's redundant. It fits better as an output for this model rather than a property, whereas, it's a property that's general to all model. When a model predicts a compound, it's either active for a particular bioactivity or not. And for this model, It's an assay activity.

GemmaTuron commented 2 months ago

Hi Zainab,

I am not convinced of the difference, as this model provides a bunch of bioactivity predictions actually, I do not understand what you are referring to as "assay" Let's mark this to discuss in our next meeting

Zainab-ik commented 2 months ago

Interpretation: The model returns the probability of 1 (active) in each assay. Good drugs are active in CPE, 3CL and are inactive in cytotox, hCYTOX and ACE2 and/or are active in at least one of the following: AlphaLISA, CoV-PPE, MERS-PPE, while inactive in the counter screen, respectively: TruHit, CoV-PPE_cs, MERS-PPE_cs.

I considered this interpretation in the repository when annotating.

GemmaTuron commented 1 month ago

Hi @Zainab-ik

We did not discuss that, but I think Active Inactive is enough, we do not need to add the Active in Assay, as it is redundant