Details

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

      Description

      Markdown: https://bitbucket.org/mdavis4290/molly-flavonoid-rnaseq/src/main/72_F3H_PollenTube/DESeq_analysis/DESeq-Clusters.Rmd

      • Add labels
      • Square figure
      • Make all axis' the same length
      • Circle clusters
      • Improve markdown with in-line R coding

      Note: some of these requests were changed due to not visually working.

        Attachments

          Issue Links

            Activity

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

            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.

            Show
            ann.loraine Ann Loraine added a comment - - edited 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.
            Hide
            ann.loraine Ann Loraine added a comment -

            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] thinks more or less is needed.

            Show
            ann.loraine Ann Loraine added a comment - 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] thinks more or less is needed.
            Hide
            ann.loraine Ann Loraine added a comment -

            Moving back to "To-Do" for Molly to review the preceding comments by me, following the most recent comment made by [~molly].

            I am happy with it being closed, or worked-on a bit more, as she sees fit.

            Show
            ann.loraine Ann Loraine added a comment - Moving back to "To-Do" for Molly to review the preceding comments by me, following the most recent comment made by [~molly] . I am happy with it being closed, or worked-on a bit more, as she sees fit.
            Hide
            Mdavis4290 Molly Davis added a comment -

            Branch: https://bitbucket.org/mdavis4290/molly3-flavonoid-rnaseq/branch/IGBF-3369
            Pull Request: https://bitbucket.org/hotpollen/flavonoid-rnaseq/pull-requests/19

            Notes: Dr. Loraine has already reviewed the cosmetic changes to the PCA plots and this pull request are just the cosmetic code changes to the plots. The commit should only include the new Markdown and output file of the new figures. I will make more changes to the writing later to then also include the knitted PDF. Thanks!

            Show
            Mdavis4290 Molly Davis added a comment - Branch : https://bitbucket.org/mdavis4290/molly3-flavonoid-rnaseq/branch/IGBF-3369 Pull Request : https://bitbucket.org/hotpollen/flavonoid-rnaseq/pull-requests/19 Notes : Dr. Loraine has already reviewed the cosmetic changes to the PCA plots and this pull request are just the cosmetic code changes to the plots. The commit should only include the new Markdown and output file of the new figures. I will make more changes to the writing later to then also include the knitted PDF. Thanks!
            Hide
            ann.loraine Ann Loraine added a comment -

            PR is merged. Moving to Done.

            Show
            ann.loraine Ann Loraine added a comment - PR is merged. Moving to Done.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: