UCSC-MedBook / patient-care

Clincian facing portal showing pathways, signatures, and genes of interest
2 stars 1 forks source link

Create job to download filtered data using Yulia's scripts #29

Closed mokolodi1 closed 8 years ago

mokolodi1 commented 8 years ago

See "Outlier analysis" email from July 13th.

e-t-k commented 8 years ago

branch: f-exprAndVarFilters

Requires the following repos:

Tasks [not necessarily exhaustive haha]

Phase 1 : creating the sample group filter tool

Phase 2 : applying it to outlier analysis

e-t-k commented 8 years ago

Currently: TESTING



code review

Next step, once its made live:

updating sample groups / data sets on prod to match these changes

Right now, which data sets are 'raw' and unfiltered ? which had the filters 'baked in' before uploading to medbook? I don't know

mokolodi1 commented 8 years ago

What do you mean by baked in before uploading to MedBook?

Teo

On Mon, Aug 1, 2016 at 1:06 PM, e-t-k notifications@github.com wrote:

Currently: TESTING

Next step, once its made live:

updating sample groups / data sets on prod to match these changes

Right now, which data sets are 'raw' and unfiltered ? which had the filters 'baked in' before uploading to medbook? I don't know

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/UCSC-MedBook/patient-care/issues/29#issuecomment-236691552, or mute the thread https://github.com/notifications/unsubscribe-auth/ABRIv0BXoYo9iymh_gWjTom3edtQpsuCks5qblGsgaJpZM4JOFhG .

e-t-k commented 8 years ago

What do you mean by baked in

My recollection is that for the current "CKCC reference cohort" data set in the prod db, I was told that these filters had been applied manually before the samples were uploaded to medbook. (the sample group for the filtering would have been the entire data set.) Don't know if any other data sets are known to be like this. Do i recall correctly? since this data set was imported before I got here...

Re-applying filters to a sample group where the filter is already "baked in" - expression level filter would have no effect (since all relevant genes already removed) but variance filter would remove another 20% of genes.

mokolodi1 commented 8 years ago

Ah, I see.

The CKCC Reference Cohort is the only one.

On Mon, Aug 1, 2016 at 1:32 PM, e-t-k notifications@github.com wrote:

What do you mean by baked in

My recollection is that for the current "CKCC reference cohort" data set in the prod db, I was told that these filters had been applied manually before the samples were uploaded to medbook. (the sample group for the filtering would have been the entire data set.) Don't know if any other data sets are known to be like this. Do i recall correctly? since this data set was imported before I got here...

Re-applying filters to a sample group where the filter is already "baked in" - expression level filter would have no effect (since all relevant genes already removed) but variance filter would remove another 20% of genes.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/UCSC-MedBook/patient-care/issues/29#issuecomment-236698585, or mute the thread https://github.com/notifications/unsubscribe-auth/ABRIv2r4ZW0xIxW1aHm2N8WkdhrxpxfZks5qblfrgaJpZM4JOFhG .

e-t-k commented 8 years ago

Issues found in testing (dev box):

Issues found in code review with Teo

Issues found in testing(staging) ... Jobs fail with, roughly, " undefined is not a function at server.js at 105". Suggests that JobClasses is not correctly being populated with my new JobClass. docker problem? looking into this... Update: docker-compose up wasn't building the new code. (This turns out to be the documented behavior when there's an existing image.). fixed.

e-t-k commented 8 years ago

-- When I'm running a job to generate filters, Teo, looking at the same sample group, can't see that that job is running, and the button to generate filters is available to him.

Fix : let the sample group subscribe to all jobs of that type associated with that sample group, regardless of owner.

e-t-k commented 8 years ago

This is live, & sample groups / data sets etc have been updated to new cohort .