forked from johnomics/basicr
-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pull request to correct a mistake in the intro to R on variables. #3
Open
CristianRiccio
wants to merge
45
commits into
bioinformatics-core-shared-training:master
Choose a base branch
from
cambiotraining:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Pull request to correct a mistake in the intro to R on variables. #3
CristianRiccio
wants to merge
45
commits into
bioinformatics-core-shared-training:master
from
cambiotraining:master
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
example of adding proportions to count data
remove some arguments previously used for slide layout
Use script to create patients data
R notebook presentations
Hi Christian, So sorry but I'm away at the moment and am not able to look into it. Was it just the problem with the variable names? |
Hi Mark, |
Add explanation on how genes were selected to create the heatmap (and pick new set of genes).
… of colours and plotting characters is used rather than points
…th red normal curve
Remove the variables used to create the data frame. Hopefully encourage people to subset using the actual columns from the data frame
- do a match by selecting the symbol column by name rather than index - replace t variable name - remove sample names from heatmap
In the cor function, instead of using the abbreviation use="c", it is clearer to write use="complete.obs".
clarity of use parameter in cor function
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.
Hi Mark,
I am not very comfortable yet with Github but I am trying to ask you to correct a mistake on the naming of variables in the introduction to R. Variables can contain '.' or '_'.