Conversation
Co-Authored-By: Neha Talluri <78840540+ntalluri@users.noreply.github.com> Co-Authored-By: Oliver Faulkner Anderson <112665860+oliverfanderson@users.noreply.github.com> Co-Authored-By: Altaf Barelvi <altafayyubibarelvi@gmail.com>
|
A question I’d appreciate feedback on: Currently, we generate separate source, target, and prize files for each pathway, but we combine all pathways into each thresholded interactome. Should we also create a combined list of sources, targets, and prizes? Should we also combine the gold standard as well? Or would it be better to keep separate interactomes for each individual pathway (keep it the way it is)? |
|
We should have separate gold standards. |
|
When this is reviewed (or before) we should do tests to see how connected the networks are after thresholding, adding back the pathway data, and removing proteins that don't have uniprot ids. |
|
Also there is a chance we can use more panther pathways, we should look to see what else we can use from pathway commons. |
|
@oliverfanderson @ctrlaltaf For the gold standard nodes (and potentially the edges), should we exclude source, target, and prize nodes when defining it? Currently, it looks like we’re including these nodes in the gold standard for each pathway. These nodes overlap with the gold standard, but that overlap should happen naturally, not by construction/being predefined. I’m concerned this could inflate our precision and recall metrics, because of a form of data leakage. |
Plan to keep all of them in the gold standard. But update the evaluation code to deal with the sources/targets/prizes being in the gold standard and shown as a different baseline where those are all set as frequency 1.0. |
|
Should we also consider how sparse an interactome becomes after applying a threshold to the STRING interactome? When we filter by size, we implicitly accounting for the decrease in graph density as well. Would it make more sense to treat size and density as separate variables when evaluating performance? However, does testing for density even matter in this context; are there any interactomes that aren’t already highly connected? I’m thinking we should first threshold the interactomes, then select only those that are highly connected (e.g., density ≥ 0.85). From that subset, we could choose a few to represent different size scales. |
|
I will be updating how we create interactomes for the Panther pathways dataset. Current: New: For example, in the STRING interaction networks, when using only physical interactions and experimental edge scores, we could aim to keep 25% of all edges.
Now we will be construct new interactomes by removing X% of edges and then adding all edges from all chosen PANTHER pathways. We will only keep downsampled interacomes that satisfy specified properties for a given set of sources and targets. Proposed brute-force method for Panther pathways interactomes:
Randomly remove X edges from the full STRING interactome
Verify that the new network maintains the following properties:
If the properties above are not satisfied, repeat the process with a different random sample. |
|
For this dataset, we are planning on using it for all of the evaluations. I was deciding if we need to use all of the pathways, and I don't think we need to. I decided on a couple that we can use: Balanced Skewed Tiny When making the interactomes, I want to add all of these pathways on the thresholded interactomes and uphold the properties above. I need to double check if I used any of these will break the rules for pilot data/runs; but since we are making a new dataset that wasn't used for my thesis, I think we will be okay. |
|
Made minor changes to fix the interactome fetching - these shouldn't cause any conflicts, nor were the changes I wanted to make as mentioned in Slack. [If they do, feel free to force push.] |
I'm also going to change the way I feed in pathways into the Snakefile since it's weird locally.
There was a problem hiding this comment.
This won't be the final list. We still need to check for pathways that have at least one source and at least one target. That will help us finalize this list.
| "Notch signaling pathway", | ||
| "JAK/STAT signaling pathway", | ||
| "Interleukin signaling pathway", | ||
| // TODO: These pathways do not have necessary data to work |
There was a problem hiding this comment.
I think these do, I think the file you sent me had a messed up folder structure because of the "/" so my script didn't find information on it. I have to go in a get these stats manually.
We also refactor the cache system (again!) to reduce code indirection, and make the entire repository a python package for referencing other modules throughout the codebase.
Co-Authored-By: Neha Talluri 78840540+ntalluri@users.noreply.github.com
Co-Authored-By: Oliver Faulkner Anderson 112665860+oliverfanderson@users.noreply.github.com
Co-Authored-By: Altaf Barelvi altafayyubibarelvi@gmail.com