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

Edit PCA plot markdown and commit knitted file

    Details

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

      Description

      Dr. Loraine comments:

      • Figure out a nicer way to present the result from sanity checking. Currently, the message letting the reader know that the test passed does not fit on the page, making it harder to follow what is going on. See: p. 3, Muday-DESeq-PCA-Plots.pdf
      • First two words of the section of text appearing after the plot are confusing. It says: "The following plot.." suggesting the plot will be after this text. But the plot is actually appearing before the text, not after it. However, there is only one plot, so it's obvious what the text is talking about.
      • To me, this statement describing part "A" sounds nonsensical: "We also see that over a longer duration of time that there is even more significant variance between the samples." I see filled shapes spread about evenly across the two "slash"-shaped clusters. There is no such trend. Also, I don't think there is something such as "significant variance" in the statistical sense. Maybe you mean to say: "a lot" ?
      • Regarding "B": I agree with the statement made about temperature. I recommend wording it a little differently to refer back to the image itself a bit more. How about: "PC1 cleanly separates points into two visually obvious groups, one containing only earlier-stage sample points and the second containing only-later-stage points."
      • Regarding "C": I agree, there's no trend here. There are two, maybe three clusters, plus a singleton. Not obvious trends are there with respect to any of the variables examined. The current statement: "C) The OE3 genotype was an analysis of just OE3 samples and had no clear indication of clustering or separation of points except for maybe a little at 75 minutes." is correct. But I think you should change the word "just" to "only," or, better yet, say "This plot and the code used to produce it included OE3 genotype samples only." Also, we can easily check this is true by looking at the included code.
      • Regarding "D": There's an obvious visual separation between the treatment (heat stress) samples and the control (no-heat stress sample) in the PC2 dimension. There is also a clear grouping based on early-versus-later sample points in the PC1 dimension. Editorial comment: feel "D" is the most exciting image.
      • This statement again needs to be connected to the plot itself: "The ARE genotype has the largest variation out of all PCA plots and was analyzed with just ARE samples." Why do you say this about variance? How can the reader make the same conclusion about variance for themselves. What do they need to look at in the image? Explain this better and talk about what it means in the Discussion. Recall that the true "result" here is your observation about what the plot looks like.

      Comments on the Discussion:

      • First sentence is just filler. Remove it.
      • Second sentence has a lot of filler words. Reword it to remove unnecessary words. Change: "The goal was to answer these questions for the experiment to be successful:" to "Question (or questions) this Markdown aims to answer:"
      • There are a lot of things an author can do with a Discussion, but the ultimate goal should be the same in each case: Explain the Results.
      • There are a lot of choices about how to organize a Discussion section. Here are some:
      • Re-list the questions from the Introduction.
      • Edit by including one or two paragraphs following each question stating the answer to the question, and evidence. Interleave answers so that each re-stated question is followed by its answer and evidence for the answer.
      • Then list the questions, one-by-one, and say what the answer is, using the Results.
      • If the question is: "Was the experiment successful" and you have arrived at an answer to this question, you have to explain it more.
        I feel it is too vague, anyway.

      Suggestion: Read the comments and then spend an afternoon or a couple of hours to make some improvements. If you like it better, submit a new PR. If you don't, we can close this.

      As far as I am concerned, I feel I can look at the plots and come to my own conclusions about them. I don't need a Markdown to explain it to me, because I can read code and understand it, generally. Someone else might not, however. I would recommend investing at least another half-day in making improvements, but not much more, unless Molly Davis thinks more or less is needed.

        Attachments

          Issue Links

            Activity

            Mdavis4290 Molly Davis created issue -
            Mdavis4290 Molly Davis made changes -
            Field Original Value New Value
            Epic Link IGBF-3277 [ 22158 ]
            Mdavis4290 Molly Davis made changes -
            Link This issue relates to IGBF-3369 [ IGBF-3369 ]
            Mdavis4290 Molly Davis made changes -
            Description *Dr. Loraine comments*:

            Figure out a nicer way to present the result from sanity checking. Currently, the message letting the reader know that the test passed does not fit on the page, making it harder to follow what is going on. See: p. 3, Muday-DESeq-PCA-Plots.pdf
            First two words of the section of text appearing after the plot are confusing. It says: "The following plot.." suggesting the plot will be after this text. But the plot is actually appearing before the text, not after it. However, there is only one plot, so it's obvious what the text is talking about.

            To me, this statement describing part "A" sounds nonsensical: "We also see that over a longer duration of time that there is even more significant variance between the samples." I see filled shapes spread about evenly across the two "slash"-shaped clusters. There is no such trend. Also, I don't think there is something such as "significant variance" in the statistical sense. Maybe you mean to say: "a lot" ?
            Regarding "B": I agree with the statement made about temperature. I recommend wording it a little differently to refer back to the image itself a bit more. How about: "PC1 cleanly separates points into two visually obvious groups, one containing only earlier-stage sample points and the second containing only-later-stage points."
            Regarding "C": I agree, there's no trend here. There are two, maybe three clusters, plus a singleton. Not obvious trends are there with respect to any of the variables examined. The current statement: "C) The OE3 genotype was an analysis of just OE3 samples and had no clear indication of clustering or separation of points except for maybe a little at 75 minutes." is correct. But I think you should change the word "just" to "only," or, better yet, say "This plot and the code used to produce it included OE3 genotype samples only." Also, we can easily check this is true by looking at the included code.
            Regarding "D": There's an obvious visual separation between the treatment (heat stress) samples and the control (no-heat stress sample) in the PC2 dimension. There is also a clear grouping based on early-versus-later sample points in the PC1 dimension. Editorial comment: feel "D" is the most exciting image.
            This statement again needs to be connected to the plot itself: "The ARE genotype has the largest variation out of all PCA plots and was analyzed with just ARE samples." Why do you say this about variance? How can the reader make the same conclusion about variance for themselves. What do they need to look at in the image? Explain this better and talk about what it means in the Discussion. Recall that the true "result" here is your observation about what the plot looks like.

            Comments on the Discussion:
            First sentence is just filler. Remove it.
            Second sentence has a lot of filler words. Reword it to remove unnecessary words. Change: "The goal was to answer these questions for the experiment to be successful:" to "Question (or questions) this Markdown aims to answer:"
            More thoughts:
            There are a lot of things an author can do with a Discussion, but the ultimate goal should be the same in each case: Explain the Results.
            There are a lot of choices about how to organize a Discussion section. Here are some:
            Listify:
            Re-list the questions from the Introduction.
            Edit by including one or two paragraphs following each question stating the answer to the question, and evidence. Interleave answers so that each re-stated question is followed by its answer and evidence for the answer.
            Then list the questions, one-by-one, and say what the answer is, using the Results.

            If the question is: "Was the experiment successful" and you have arrived at an answer to this question, you have to explain it more.
            I feel it is too vague, anyway.

            Suggestion: Read the comments and then spend an afternoon or a couple of hours to make some improvements. If you like it better, submit a new PR. If you don't, we can close this.
            As far as I am concerned, I feel I can look at the plots and come to my own conclusions about them. I don't need a Markdown to explain it to me, because I can read code and understand it, generally. Someone else might not, however. I would recommend investing at least another half-day in making improvements, but not much more, unless Molly Davis thinks more or less is needed.
            *Dr. Loraine comments*:

            * Figure out a nicer way to present the result from sanity checking. Currently, the message letting the reader know that the test passed does not fit on the page, making it harder to follow what is going on. See: p. 3, Muday-DESeq-PCA-Plots.pdf
            * First two words of the section of text appearing after the plot are confusing. It says: "The following plot.." suggesting the plot will be after this text. But the plot is actually appearing before the text, not after it. However, there is only one plot, so it's obvious what the text is talking about.

            * To me, this statement describing part "A" sounds nonsensical: "We also see that over a longer duration of time that there is even more significant variance between the samples." I see filled shapes spread about evenly across the two "slash"-shaped clusters. There is no such trend. Also, I don't think there is something such as "significant variance" in the statistical sense. Maybe you mean to say: "a lot" ?
            * Regarding "B": I agree with the statement made about temperature. I recommend wording it a little differently to refer back to the image itself a bit more. How about: "PC1 cleanly separates points into two visually obvious groups, one containing only earlier-stage sample points and the second containing only-later-stage points."
            * Regarding "C": I agree, there's no trend here. There are two, maybe three clusters, plus a singleton. Not obvious trends are there with respect to any of the variables examined. The current statement: "C) The OE3 genotype was an analysis of just OE3 samples and had no clear indication of clustering or separation of points except for maybe a little at 75 minutes." is correct. But I think you should change the word "just" to "only," or, better yet, say "This plot and the code used to produce it included OE3 genotype samples only." Also, we can easily check this is true by looking at the included code.
            * Regarding "D": There's an obvious visual separation between the treatment (heat stress) samples and the control (no-heat stress sample) in the PC2 dimension. There is also a clear grouping based on early-versus-later sample points in the PC1 dimension. Editorial comment: feel "D" is the most exciting image.
            * This statement again needs to be connected to the plot itself: "The ARE genotype has the largest variation out of all PCA plots and was analyzed with just ARE samples." Why do you say this about variance? How can the reader make the same conclusion about variance for themselves. What do they need to look at in the image? Explain this better and talk about what it means in the Discussion. Recall that the true "result" here is your observation about what the plot looks like.

            Comments on the Discussion:
            First sentence is just filler. Remove it.
            Second sentence has a lot of filler words. Reword it to remove unnecessary words. Change: "The goal was to answer these questions for the experiment to be successful:" to "Question (or questions) this Markdown aims to answer:"
            More thoughts:
            There are a lot of things an author can do with a Discussion, but the ultimate goal should be the same in each case: Explain the Results.
            There are a lot of choices about how to organize a Discussion section. Here are some:
            Listify:
            Re-list the questions from the Introduction.
            Edit by including one or two paragraphs following each question stating the answer to the question, and evidence. Interleave answers so that each re-stated question is followed by its answer and evidence for the answer.
            Then list the questions, one-by-one, and say what the answer is, using the Results.

            If the question is: "Was the experiment successful" and you have arrived at an answer to this question, you have to explain it more.
            I feel it is too vague, anyway.

            Suggestion: Read the comments and then spend an afternoon or a couple of hours to make some improvements. If you like it better, submit a new PR. If you don't, we can close this.
            As far as I am concerned, I feel I can look at the plots and come to my own conclusions about them. I don't need a Markdown to explain it to me, because I can read code and understand it, generally. Someone else might not, however. I would recommend investing at least another half-day in making improvements, but not much more, unless Molly Davis thinks more or less is needed.
            Mdavis4290 Molly Davis made changes -
            Description *Dr. Loraine comments*:

            * Figure out a nicer way to present the result from sanity checking. Currently, the message letting the reader know that the test passed does not fit on the page, making it harder to follow what is going on. See: p. 3, Muday-DESeq-PCA-Plots.pdf
            * First two words of the section of text appearing after the plot are confusing. It says: "The following plot.." suggesting the plot will be after this text. But the plot is actually appearing before the text, not after it. However, there is only one plot, so it's obvious what the text is talking about.

            * To me, this statement describing part "A" sounds nonsensical: "We also see that over a longer duration of time that there is even more significant variance between the samples." I see filled shapes spread about evenly across the two "slash"-shaped clusters. There is no such trend. Also, I don't think there is something such as "significant variance" in the statistical sense. Maybe you mean to say: "a lot" ?
            * Regarding "B": I agree with the statement made about temperature. I recommend wording it a little differently to refer back to the image itself a bit more. How about: "PC1 cleanly separates points into two visually obvious groups, one containing only earlier-stage sample points and the second containing only-later-stage points."
            * Regarding "C": I agree, there's no trend here. There are two, maybe three clusters, plus a singleton. Not obvious trends are there with respect to any of the variables examined. The current statement: "C) The OE3 genotype was an analysis of just OE3 samples and had no clear indication of clustering or separation of points except for maybe a little at 75 minutes." is correct. But I think you should change the word "just" to "only," or, better yet, say "This plot and the code used to produce it included OE3 genotype samples only." Also, we can easily check this is true by looking at the included code.
            * Regarding "D": There's an obvious visual separation between the treatment (heat stress) samples and the control (no-heat stress sample) in the PC2 dimension. There is also a clear grouping based on early-versus-later sample points in the PC1 dimension. Editorial comment: feel "D" is the most exciting image.
            * This statement again needs to be connected to the plot itself: "The ARE genotype has the largest variation out of all PCA plots and was analyzed with just ARE samples." Why do you say this about variance? How can the reader make the same conclusion about variance for themselves. What do they need to look at in the image? Explain this better and talk about what it means in the Discussion. Recall that the true "result" here is your observation about what the plot looks like.

            Comments on the Discussion:
            First sentence is just filler. Remove it.
            Second sentence has a lot of filler words. Reword it to remove unnecessary words. Change: "The goal was to answer these questions for the experiment to be successful:" to "Question (or questions) this Markdown aims to answer:"
            More thoughts:
            There are a lot of things an author can do with a Discussion, but the ultimate goal should be the same in each case: Explain the Results.
            There are a lot of choices about how to organize a Discussion section. Here are some:
            Listify:
            Re-list the questions from the Introduction.
            Edit by including one or two paragraphs following each question stating the answer to the question, and evidence. Interleave answers so that each re-stated question is followed by its answer and evidence for the answer.
            Then list the questions, one-by-one, and say what the answer is, using the Results.

            If the question is: "Was the experiment successful" and you have arrived at an answer to this question, you have to explain it more.
            I feel it is too vague, anyway.

            Suggestion: Read the comments and then spend an afternoon or a couple of hours to make some improvements. If you like it better, submit a new PR. If you don't, we can close this.
            As far as I am concerned, I feel I can look at the plots and come to my own conclusions about them. I don't need a Markdown to explain it to me, because I can read code and understand it, generally. Someone else might not, however. I would recommend investing at least another half-day in making improvements, but not much more, unless Molly Davis thinks more or less is needed.
            *Dr. Loraine comments*:

            * Figure out a nicer way to present the result from sanity checking. Currently, the message letting the reader know that the test passed does not fit on the page, making it harder to follow what is going on. See: p. 3, Muday-DESeq-PCA-Plots.pdf
            * First two words of the section of text appearing after the plot are confusing. It says: "The following plot.." suggesting the plot will be after this text. But the plot is actually appearing before the text, not after it. However, there is only one plot, so it's obvious what the text is talking about.

            * To me, this statement describing part "A" sounds nonsensical: "We also see that over a longer duration of time that there is even more significant variance between the samples." I see filled shapes spread about evenly across the two "slash"-shaped clusters. There is no such trend. Also, I don't think there is something such as "significant variance" in the statistical sense. Maybe you mean to say: "a lot" ?
            * Regarding "B": I agree with the statement made about temperature. I recommend wording it a little differently to refer back to the image itself a bit more. How about: "PC1 cleanly separates points into two visually obvious groups, one containing only earlier-stage sample points and the second containing only-later-stage points."
            * Regarding "C": I agree, there's no trend here. There are two, maybe three clusters, plus a singleton. Not obvious trends are there with respect to any of the variables examined. The current statement: "C) The OE3 genotype was an analysis of just OE3 samples and had no clear indication of clustering or separation of points except for maybe a little at 75 minutes." is correct. But I think you should change the word "just" to "only," or, better yet, say "This plot and the code used to produce it included OE3 genotype samples only." Also, we can easily check this is true by looking at the included code.
            * Regarding "D": There's an obvious visual separation between the treatment (heat stress) samples and the control (no-heat stress sample) in the PC2 dimension. There is also a clear grouping based on early-versus-later sample points in the PC1 dimension. Editorial comment: feel "D" is the most exciting image.
            * This statement again needs to be connected to the plot itself: "The ARE genotype has the largest variation out of all PCA plots and was analyzed with just ARE samples." Why do you say this about variance? How can the reader make the same conclusion about variance for themselves. What do they need to look at in the image? Explain this better and talk about what it means in the Discussion. Recall that the true "result" here is your observation about what the plot looks like.

            *Comments on the Discussion*:
            * First sentence is just filler. Remove it.
            * Second sentence has a lot of filler words. Reword it to remove unnecessary words. Change: "The goal was to answer these questions for the experiment to be successful:" to "Question (or questions) this Markdown aims to answer:"
            * There are a lot of things an author can do with a Discussion, but the ultimate goal should be the same in each case: Explain the Results.
            * There are a lot of choices about how to organize a Discussion section. Here are some:
            * Re-list the questions from the Introduction.
            * Edit by including one or two paragraphs following each question stating the answer to the question, and evidence. Interleave answers so that each re-stated question is followed by its answer and evidence for the answer.
            * Then list the questions, one-by-one, and say what the answer is, using the Results.
            * If the question is: "Was the experiment successful" and you have arrived at an answer to this question, you have to explain it more.
            I feel it is too vague, anyway.

            *Suggestion*: Read the comments and then spend an afternoon or a couple of hours to make some improvements. If you like it better, submit a new PR. If you don't, we can close this.

            As far as I am concerned, I feel I can look at the plots and come to my own conclusions about them. I don't need a Markdown to explain it to me, because I can read code and understand it, generally. Someone else might not, however. I would recommend investing at least another half-day in making improvements, but not much more, unless Molly Davis thinks more or less is needed.
            Mdavis4290 Molly Davis made changes -
            Summary Edit PCA plot markdown and create knitted file Edit PCA plot markdown and commit knitted file
            Mdavis4290 Molly Davis made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            Mdavis4290 Molly Davis made changes -
            Status In Progress [ 3 ] To-Do [ 10305 ]
            Mdavis4290 Molly Davis made changes -
            Sprint Summer 5 2023 July 10 [ 174 ] Summer 6 2023 July 24 [ 175 ]
            Mdavis4290 Molly Davis made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            Mdavis4290 Molly Davis made changes -
            Story Points 1 2
            Mdavis4290 Molly Davis made changes -
            Status In Progress [ 3 ] To-Do [ 10305 ]
            Mdavis4290 Molly Davis made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            ann.loraine Ann Loraine made changes -
            Sprint Summer 6 2023 July 24 [ 175 ] Summer 6 2023 July 24, Summer 7 2023 Aug 7 [ 175, 176 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            Mdavis4290 Molly Davis made changes -
            Assignee Molly Davis [ molly ]
            Mdavis4290 Molly Davis made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            Mdavis4290 Molly Davis made changes -
            Assignee Robert Reid [ robertreid ]
            robofjoy Robert Reid made changes -
            Assignee Robert Reid [ robertreid ] Molly Davis [ molly ]
            Mdavis4290 Molly Davis made changes -
            Assignee Molly Davis [ molly ] Robert Reid [ robertreid ]
            robofjoy Robert Reid made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            robofjoy Robert Reid made changes -
            Assignee Robert Reid [ robertreid ] Molly Davis [ molly ]
            Mdavis4290 Molly Davis made changes -
            Status First Level Review in Progress [ 10301 ] To-Do [ 10305 ]
            Mdavis4290 Molly Davis made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            Mdavis4290 Molly Davis made changes -
            Assignee Molly Davis [ molly ]
            Mdavis4290 Molly Davis made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            Mdavis4290 Molly Davis made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            Mdavis4290 Molly Davis made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            Mdavis4290 Molly Davis made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            Mdavis4290 Molly Davis made changes -
            Assignee Ann Loraine [ aloraine ]
            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 ]
            ann.loraine Ann Loraine made changes -
            Sprint Summer 6 2023 July 24, Summer 7 2023 Aug 7 [ 175, 176 ] Summer 6 2023 July 24, Summer 7 2023 Aug 7, Summer 8 2023 Aug 21 [ 175, 176, 177 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            ann.loraine Ann Loraine made changes -
            Comment [ I am trying the approach shown in Section 3.3 of [edgeRUsersGuide.pdf|https://bioconductor.org/packages/release/bioc/vignettes/edgeR/inst/doc/edgeRUsersGuide.pdf]

            I am going to work on it for a few more hours and then call it quits for the day.

            (¡Mi clase de español empieza a las siete!)

            I think we ought to try this same linear modeling approach using the two libraries, using code that produces equivalent outputs, and then compare results. Others would probably want to read about it. ]
            ann.loraine Ann Loraine made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            ann.loraine Ann Loraine made changes -
            Status Post-merge Testing In Progress [ 10003 ] To-Do [ 10305 ]
            ann.loraine Ann Loraine made changes -
            Assignee Molly Davis [ molly ]
            ann.loraine Ann Loraine made changes -
            Sprint Summer 6 2023 July 24, Summer 7 2023 Aug 7, Summer 8 2023 Aug 21 [ 175, 176, 177 ] Testing 2 : 4 Nov - 15 Nov, Summer 6 2023 July 24, Summer 7 2023 Aug 7, Summer 8 2023 Aug 21 [ 82, 175, 176, 177 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            Mdavis4290 Molly Davis made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            Mdavis4290 Molly Davis made changes -
            Assignee Molly Davis [ molly ]
            Mdavis4290 Molly Davis made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            Mdavis4290 Molly Davis made changes -
            Assignee Ann Loraine [ aloraine ]
            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 -
            Assignee Ann Loraine [ aloraine ] Molly Davis [ molly ]
            Mdavis4290 Molly Davis made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            Mdavis4290 Molly Davis made changes -
            Assignee Molly Davis [ molly ] Ann Loraine [ aloraine ]
            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 -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            ann.loraine Ann Loraine made changes -
            Resolution Done [ 10000 ]
            Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
            ann.loraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ] Molly Davis [ molly ]

              People

              • Assignee:
                Mdavis4290 Molly Davis
                Reporter:
                Mdavis4290 Molly Davis
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: