Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-3455

Visualize log fold-changes and p values to compare edgeR and DESeq2 results

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      Compare control-vs-treatment DESeq2 and edgeR differential expression results by visualizing log fold-changes and p-values between methods.

        Attachments

          Issue Links

            Activity

            ann.loraine Ann Loraine created issue -
            ann.loraine Ann Loraine made changes -
            Field Original Value New Value
            Epic Link IGBF-3446 [ 22548 ]
            ann.loraine Ann Loraine made changes -
            Link This issue is blocked by IGBF-3448 [ IGBF-3448 ]
            ann.loraine Ann Loraine made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            Hide
            ann.loraine Ann Loraine added a comment -
            Show
            ann.loraine Ann Loraine added a comment - How to deploy a shiny app: https://datasciencegenie.com/how-to-embed-a-shiny-app-on-website/
            ann.loraine Ann Loraine made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            ann.loraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ]
            Hide
            ann.loraine Ann Loraine added a comment -

            For review, see:

            • CompareDEGenesByMethod.Rmd - can build it
            • CompareDEGenesByMethod.pdf - makes sense, typos aren't too bad (meaning: it's understandable)
            • results/CvT-compare-SL5.txt - check number of rows, NA values make sense, SL5 has extra column indicating SL4 gene name, if available
            • results/CvT-compare-SL4.txt - check number of rows, NA values make sense
            Show
            ann.loraine Ann Loraine added a comment - For review, see: CompareDEGenesByMethod.Rmd - can build it CompareDEGenesByMethod.pdf - makes sense, typos aren't too bad (meaning: it's understandable) results/CvT-compare-SL5.txt - check number of rows, NA values make sense, SL5 has extra column indicating SL4 gene name, if available results/CvT-compare-SL4.txt - check number of rows, NA values make sense
            ann.loraine Ann Loraine made changes -
            Sprint Fall 2 2023 Sep 17 [ 178 ] Fall 2 2023 Sep 17, Fall 3 2023 Oct 2 [ 178, 179 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            Mdavis4290 Molly Davis made changes -
            Assignee Molly Davis [ molly ]
            ann.loraine Ann Loraine made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            ann.loraine Ann Loraine made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            ann.loraine Ann Loraine made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            ann.loraine Ann Loraine made changes -
            Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
            ann.loraine Ann Loraine made changes -
            Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
            Mdavis4290 Molly Davis made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            Hide
            Mdavis4290 Molly Davis added a comment - - edited

            Testing:

            • Build: Ran the mark down through each chunk and then also knitted and had no issues/errors.
            • Comprehension: Markdown makes sense and explains perfectly what the goal and results are.
            • SL5 Output: there is an issue with the description column for the output file. It seems it might just be a placement issue within the data frame. SL4 column seems to be holding the description and missing just the gene names for SL4.
            • SL4 Output: there is an issue with the description column for the output file. It seems it might just be a placement issue within the data frame. Description isn't present in the output file at all.

            Note: When I check bitbucket the output files on there do not have the description issue so I am going to retry updating my fork and rerunning. After comparing the output files it seems mine is showing a basemean column and the one on bitbucket is not which is why the description columns are not in the right place

            Show
            Mdavis4290 Molly Davis added a comment - - edited Testing : Build: Ran the mark down through each chunk and then also knitted and had no issues/errors. Comprehension: Markdown makes sense and explains perfectly what the goal and results are. SL5 Output: there is an issue with the description column for the output file. It seems it might just be a placement issue within the data frame. SL4 column seems to be holding the description and missing just the gene names for SL4. SL4 Output: there is an issue with the description column for the output file. It seems it might just be a placement issue within the data frame. Description isn't present in the output file at all. Note: When I check bitbucket the output files on there do not have the description issue so I am going to retry updating my fork and rerunning. After comparing the output files it seems mine is showing a basemean column and the one on bitbucket is not which is why the description columns are not in the right place
            Hide
            Mdavis4290 Molly Davis added a comment -

            I went ahead and cloned the main repo and did run into the same issue. Might need to try on another computer to see if this is an output issue only for me or for everyone.

            Moving back to to do!

            Show
            Mdavis4290 Molly Davis added a comment - I went ahead and cloned the main repo and did run into the same issue. Might need to try on another computer to see if this is an output issue only for me or for everyone. Moving back to to do!
            Mdavis4290 Molly Davis made changes -
            Assignee Molly Davis [ molly ]
            Mdavis4290 Molly Davis made changes -
            Status Post-merge Testing In Progress [ 10003 ] To-Do [ 10305 ]
            Mdavis4290 Molly Davis made changes -
            Assignee Ann Loraine [ aloraine ]
            Hide
            ann.loraine Ann Loraine added a comment -

            Thanks Molly! I will fix as noted in previous comment.

            Show
            ann.loraine Ann Loraine added a comment - Thanks Molly! I will fix as noted in previous comment.
            ann.loraine Ann Loraine made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            The SL4 data file appears to be fine. I checked it with:

            > fn="results/CvT-compare-SL4.txt"
            > d = read.csv(fn,header = T,quote = "",sep="\t")
            > names(d)
             [1] "gene"          "group1"        "group2"        "padj.edgeR"    "pvalue.edgeR" 
             [6] "logFC.edgeR"   "padj.DESeq2"   "pvalue.DESeq2" "logFC.DESeq2"  "description"  
            

            I checked the SL5 output file with:

            > fn="results/CvT-compare-SL5.txt"
            > d = read.csv(fn,header = T,quote = "",sep="\t")
            > names(d)
             [1] "gene"          "group1"        "group2"        "padj.edgeR"    "pvalue.edgeR" 
             [6] "logFC.edgeR"   "padj.DESeq2"   "pvalue.DESeq2" "logFC.DESeq2"  "description"  
            [11] "SL4"
            

            Example data rows for the SL5 data:

            > d$description[1:4]
            [1] "Complex 1 LYR protein (AHRD V3.3 *** A0A2U1NHM6_ARTAN) ITAG4.0:Solyc02g085937.1"
            [2] "Complex 1 LYR protein (AHRD V3.3 *** A0A2U1NHM6_ARTAN) ITAG4.0:Solyc02g085937.1"
            [3] "LeHSP110/ClpB heat shock protein ITAG4.0:Solyc02g088610.4"                      
            [4] NA                                                                               
            > d$SL4[1:4]
            [1] "Solyc02g085937.1" "Solyc02g085937.1" "Solyc02g088610.4" NA       
            

            The files look right to me. Moving this forward to "Needs Testing" for [~molly] to take another look.

            Show
            ann.loraine Ann Loraine added a comment - - edited The SL4 data file appears to be fine. I checked it with: > fn= "results/CvT-compare-SL4.txt" > d = read.csv(fn,header = T,quote = "",sep=" \t") > names(d) [1] "gene" "group1" "group2" "padj.edgeR" "pvalue.edgeR" [6] "logFC.edgeR" "padj.DESeq2" "pvalue.DESeq2" "logFC.DESeq2" "description" I checked the SL5 output file with: > fn= "results/CvT-compare-SL5.txt" > d = read.csv(fn,header = T,quote = "",sep=" \t") > names(d) [1] "gene" "group1" "group2" "padj.edgeR" "pvalue.edgeR" [6] "logFC.edgeR" "padj.DESeq2" "pvalue.DESeq2" "logFC.DESeq2" "description" [11] "SL4" Example data rows for the SL5 data: > d$description[1:4] [1] "Complex 1 LYR protein (AHRD V3.3 *** A0A2U1NHM6_ARTAN) ITAG4.0:Solyc02g085937.1" [2] "Complex 1 LYR protein (AHRD V3.3 *** A0A2U1NHM6_ARTAN) ITAG4.0:Solyc02g085937.1" [3] "LeHSP110/ClpB heat shock protein ITAG4.0:Solyc02g088610.4" [4] NA > d$SL4[1:4] [1] "Solyc02g085937.1" "Solyc02g085937.1" "Solyc02g088610.4" NA The files look right to me. Moving this forward to "Needs Testing" for [~molly] to take another look.
            ann.loraine Ann Loraine made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            ann.loraine Ann Loraine made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            ann.loraine Ann Loraine made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            ann.loraine Ann Loraine made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            ann.loraine Ann Loraine made changes -
            Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
            ann.loraine Ann Loraine made changes -
            Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
            ann.loraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ] Molly Davis [ molly ]
            Hide
            Mdavis4290 Molly Davis added a comment -

            Everything looks good now! Moving to done!

            Show
            Mdavis4290 Molly Davis added a comment - Everything looks good now! Moving to done!
            Mdavis4290 Molly Davis made changes -
            Assignee Molly Davis [ molly ]
            Mdavis4290 Molly Davis made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            Mdavis4290 Molly Davis made changes -
            Resolution Done [ 10000 ]
            Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
            Mdavis4290 Molly Davis made changes -
            Assignee Ann Loraine [ aloraine ]

              People

              • Assignee:
                ann.loraine Ann Loraine
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: