AlexsLemonade / refinebio-examples

Example workflows for refine.bio data
https://www.refine.bio
Other
11 stars 5 forks source link

ORA RNA-seq: Part 1 - The Set Up #394

Closed cansavvy closed 3 years ago

cansavvy commented 3 years ago

Analysis Purpose

Add an "RNA-seq" version of ORA pathway analysis. This PR adds the "Set Up" part 2 will add the actual ORA and plot making.

344

Pull Request Stage

This is a Refined PR - needs review of big concepts and outline. Draft Review occurred here: #381 not much changed.

To help reviewers save time, I'm still using slightly different tags to alert you to the few places that have changed in this module so far.

After your initial review, I will make a commit that deletes all these tags. (Unless you don't want the tags in the first place, then I can just delete them).

**IDENTICAL TO MICROARRAY EXAMPLE** means that until a different tag shows up, all the info there is 99% the same as the microarray example. Any differences I put in parenthesis like **IDENTICAL TO MICROARRAY EXAMPLE** (except microarray. --> rnaseq)

**REVIEW** doesn't mean it's 100% polished, but it does mean that it differs from the microarray example.

Strategy

Main changes from the Microarray are:

Most everything else is pretty identical to microarray ORA.

Concerns/Questions for reviewers:

I think wording and descriptions of the WGCNA analysis are the main focus here. Do they make sense but aren't going off the trail too much?

Analysis Pull Request Check List (roughly in order):

Content checks

Formatting Checks

Add datasets to S3

Docker/Snakemake rendering components

cansavvy commented 3 years ago

Thanks for this review. I went ahead and ported over the relevant wording changes to microarray. There was really just one. But I also went ahead and addressed #393

Anything else left to address? @jashapiro