laminlabs / lamin-usecases

Use cases.
https://docs.lamin.ai
Apache License 2.0
5 stars 0 forks source link

✨ Add perturbation use-case #142

Closed Zethson closed 17 hours ago

Zethson commented 1 month ago
github-actions[bot] commented 4 days ago

🚀 Deployed on https://66f2c423c38feb18dc1d77b4--lamin-usecases-ddcd.netlify.app

sunnyosun commented 1 day ago

Could you make this a single cell and hide output? I think the download bar gives a weird whitespace:

Screenshot 2024-09-23 at 17 39 39
sunnyosun commented 1 day ago

Could we have all features lower case, here "DepMap_ID":

Screenshot 2024-09-23 at 17 41 06
sunnyosun commented 1 day ago

Could you put "CRISPRi" instead of "CRISPR"? (in contrast to "CRISPRa" is overexpression)?

Could you call the 2nd type "compound" instead of "drug"? So it's consistent with the registry name

Screenshot 2024-09-23 at 17 41 57
sunnyosun commented 1 day ago

This here is still protein, could you change it to gene please?

Screenshot 2024-09-23 at 17 45 09
sunnyosun commented 1 day ago

I'd suggest:

wl.GeneticTreatment(
    system="CRISPRi",
    name="sgGPX4-1",
)

You can then directly use GeneticTreatment to validate perturbation column:

Screenshot 2024-09-23 at 17 46 50
sunnyosun commented 1 day ago

This is quite long as you are repeating the process 4 times, could we simplify with a loop?

Screenshot 2024-09-23 at 17 50 58
sunnyosun commented 1 day ago

Is it possible to map out the compound names manually? Relying on search here feels quite hacky:

Screenshot 2024-09-23 at 17 59 27
Zethson commented 21 hours ago

Is it possible to map out the compound names manually? Relying on search here feels quite hacky:

Screenshot 2024-09-23 at 17 59 27

Required renaming a compound which got created using a synonym in the data because else .get(name="bla") would not find it. search() results in less code but .get() is less hacky.