Third section of methylation #955
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cgreene
requested changes
Jul 19, 2019
There was a problem hiding this comment.
A bit of feedback. Can you take a look through and merge what you agree with and see if there are additional changes that you want to make? There needs to be a little bit more detail on the other VAE paper too, and ideally some synthesis of the results. If that paragraph gets too long, it may be good to split to VAE / non-VAE. In that case, it may not make sense to move the sentence that I noted.
content/04.study.md
Outdated
| @@ -47,6 +47,26 @@ Deep learning applied to gene expression data is still in its infancy, but the f | |||
| Many previously untestable hypotheses can now be interrogated as deep learning enables analysis of increasing amounts of data generated by new technologies. | |||
| For example, the effects of cellular heterogeneity on basic biology and disease etiology can now be explored by single-cell RNA-seq and high-throughput fluorescence-based imaging, techniques we discuss below that will benefit immensely from deep learning approaches. | |||
|
|
|||
| ### DNA Methylation (to remove after merge) | |||
There was a problem hiding this comment.
Pretty sure this will cause merge conflicts, so noting here that we should merge this last.
Updates based on pull-request peer-review. Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com>
cgreene
reviewed
Jul 22, 2019
Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com>
cgreene
requested changes
Jul 22, 2019
Additional clarity Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com>
Updates based on peer-review Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com>
cgreene
approved these changes
Jul 22, 2019
cgreene
requested changes
Jul 22, 2019
resolve citation error Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com>
|
All checks passed. |
cgreene
approved these changes
Jul 23, 2019
dhimmel
pushed a commit
that referenced
this pull request
Jul 23, 2019
This build is based on f000c4c. This commit was created by the following Travis CI build and job: https://travis-ci.org/greenelab/deep-review/builds/562448060 https://travis-ci.org/greenelab/deep-review/jobs/562448061 [ci skip] The full commit message that triggered this build is copied below: Third section of methylation (#955) * Update 04.study.md * Apply suggestions from code review Updates based on pull-request peer-review. Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com> * Update 04.study.md * Apply suggestions from code review Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com> * Update content/04.study.md Additional clarity Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com> * Apply suggestions from code review Updates based on peer-review Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com> * Update content/04.study.md resolve citation error Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com>
dhimmel
pushed a commit
that referenced
this pull request
Jul 23, 2019
This build is based on f000c4c. This commit was created by the following Travis CI build and job: https://travis-ci.org/greenelab/deep-review/builds/562448060 https://travis-ci.org/greenelab/deep-review/jobs/562448061 [ci skip] The full commit message that triggered this build is copied below: Third section of methylation (#955) * Update 04.study.md * Apply suggestions from code review Updates based on pull-request peer-review. Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com> * Update 04.study.md * Apply suggestions from code review Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com> * Update content/04.study.md Additional clarity Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com> * Apply suggestions from code review Updates based on peer-review Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com> * Update content/04.study.md resolve citation error Co-Authored-By: Casey Greene <cgreene@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is the 3 of 3 for sections of the methylation section addition to the Deep Review.