search

Sage-Bionetworks / NF_LandscapePaper_2019

This repository hosts all the code used to generate analyses and figures for the landscape paper
3 stars 1 forks source link

Update multiplier table to add transplantation type #64

Closed sgosline closed 4 years ago

sgosline commented 4 years ago

Currently I cannot filter xenograft data from multiplier table. Can you add this back in?

allaway commented 4 years ago

Should I just remove all xenograft data from the multiplier analysis? This way we won't inadvertently include it. What do you think?

sgosline commented 4 years ago

Sure - maybe exclude the cell line too?

allaway commented 4 years ago

Sound good!

On Wed, Nov 20, 2019 at 1:29 PM Sara JC Gosline notifications@github.com wrote:

Sure - maybe exclude the cell line too?

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/Sage-Bionetworks/NF_LandscapePaper_2019/issues/64?email_source=notifications&email_token=AE3WNSGN7U4S3RFZWCL33E3QUWT23A5CNFSM4JPKIU3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEVFD2A#issuecomment-556421608, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE3WNSHIRJ3QKZ7EG5JFREDQUWT23ANCNFSM4JPKIU3A .

jaybee84 commented 4 years ago

Are we excluding all cell lines from our analyses? Currently I have some included in one of the RFs.

On Nov 20, 2019, at 1:29 PM, Robert Allaway notifications@github.com wrote:

Sound good!

On Wed, Nov 20, 2019 at 1:29 PM Sara JC Gosline notifications@github.com wrote:

Sure - maybe exclude the cell line too?

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/Sage-Bionetworks/NF_LandscapePaper_2019/issues/64?email_source=notifications&email_token=AE3WNSGN7U4S3RFZWCL33E3QUWT23A5CNFSM4JPKIU3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEVFD2A#issuecomment-556421608, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE3WNSHIRJ3QKZ7EG5JFREDQUWT23ANCNFSM4JPKIU3A .

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Sage-Bionetworks/NF_LandscapePaper_2019/issues/64?email_source=notifications&email_token=AEIWMUKCQAMQBYV7TA2IYULQUWT4XA5CNFSM4JPKIU3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEVFH6A#issuecomment-556422136, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEIWMUPNYC2XP6XYHDIAMTLQUWT4XANCNFSM4JPKIU3A.

Jineta Banerjee, PhD Research Scientist Sage Bionetworks https://sagebionetworks.org/ Email: jineta.banerjee@sagebionetworks.org Phone: (206) 928 8264

allaway commented 4 years ago

Actually, I already got rid of the cell line in a previous revision. :)

On Wed, Nov 20, 2019 at 1:29 PM Robert Allaway robert.j.allaway@gmail.com wrote:

Sound good!

On Wed, Nov 20, 2019 at 1:29 PM Sara JC Gosline notifications@github.com wrote:

Sure - maybe exclude the cell line too?

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/Sage-Bionetworks/NF_LandscapePaper_2019/issues/64?email_source=notifications&email_token=AE3WNSGN7U4S3RFZWCL33E3QUWT23A5CNFSM4JPKIU3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEVFD2A#issuecomment-556421608, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE3WNSHIRJ3QKZ7EG5JFREDQUWT23ANCNFSM4JPKIU3A .

allaway commented 4 years ago

Yes, primary nerve sheath tumors only @jaybee84 :)

Removed the PDXs in the latest rebuild of this table: https://www.synapse.org/#!Synapse:syn21046991/tables/

Uploading the new results now.