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

Revise edgeR differential expression code to output results tables

    Details

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

      Description

      The current version does not write out DE gene results. Also, the name of the file is not specific enough.

      For this task:

      • Change name of FindDifferentiallyExpressedGenes.Rmd (and its knitted .PDF) to: FindControlVsStressDEGenes-edgeR.Rmd
      • Write out a consolidated .txt file reporting differential expression results

        Attachments

          Issue Links

            Activity

            Hide
            ann.loraine Ann Loraine added a comment - - edited

            I pushed my changes to the main branch in the team repository. Sorry - this means you have to rebase your branch onto main.

            To test:

            • Proof-read knitted .PDF and point out text that does not make sense or formatting that looks weird or bad
            • Read .Rmd file and check for possible bugs or unnecessary statements and code
            • Check for literal values that should instead be reported via in-line R code
            • Attempt to knit the .Rmd file and make note of any errors
            • Open the new output file in RStudio and visually inspect it - look for numbers that seem nonsensical
            Show
            ann.loraine Ann Loraine added a comment - - edited I pushed my changes to the main branch in the team repository. Sorry - this means you have to rebase your branch onto main. To test: Proof-read knitted .PDF and point out text that does not make sense or formatting that looks weird or bad Read .Rmd file and check for possible bugs or unnecessary statements and code Check for literal values that should instead be reported via in-line R code Attempt to knit the .Rmd file and make note of any errors Open the new output file in RStudio and visually inspect it - look for numbers that seem nonsensical
            Hide
            Mdavis4290 Molly Davis added a comment - - edited

            Testing:

            • Proof-read knitted .PDF: Looks professional and easy to comprehend. The new section for writing results also looks good
            • Read .Rmd file: Everything ran on my machine flawlessly and no issues. Everything matches up well with the knitted pdf file as well.
            • Check in-line R code: I don't see any bugs for the in-line coding.
            • Attempt to knit .Rmd file: I had no issues knitting the .Rmd file myself on my machine.
            • Open the new output file: The CvT.txt file did save to the results folder and does open as an excel file but could look a little cleaner based off of my machine. For example, I see no column names so you don't know what the numbers are from.
            Show
            Mdavis4290 Molly Davis added a comment - - edited Testing : Proof-read knitted .PDF : Looks professional and easy to comprehend. The new section for writing results also looks good Read .Rmd file : Everything ran on my machine flawlessly and no issues. Everything matches up well with the knitted pdf file as well. Check in-line R code : I don't see any bugs for the in-line coding. Attempt to knit .Rmd file : I had no issues knitting the .Rmd file myself on my machine. Open the new output file : The CvT.txt file did save to the results folder and does open as an excel file but could look a little cleaner based off of my machine. For example, I see no column names so you don't know what the numbers are from.
            Hide
            ann.loraine Ann Loraine added a comment -

            Thank you [~molly] for noticing the error in the results file.
            I have re-pushed a new version.

            Note that I also changed the name to include "edgeR" in the file name.
            The results file is now named "CvT-edgeR.txt".

            Show
            ann.loraine Ann Loraine added a comment - Thank you [~molly] for noticing the error in the results file. I have re-pushed a new version. Note that I also changed the name to include "edgeR" in the file name. The results file is now named "CvT-edgeR.txt".
            Hide
            Mdavis4290 Molly Davis added a comment -

            After reviewing all of the commits made to the main repo and viewing the output file "CvT-edgeR.txt", it seems everything is in order now and looks great! Moving to done!

            Show
            Mdavis4290 Molly Davis added a comment - After reviewing all of the commits made to the main repo and viewing the output file "CvT-edgeR.txt", it seems everything is in order now and looks great! Moving to done!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: