code rev#1
Open
smwindecker wants to merge 2 commits intoPJordano-Lab:masterfrom
Open
Conversation
… at line 14 (formula() wasn't accepting spaces)
This file contains hidden or 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
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 Jorge et al,
As part of a BES hackathon to review data and code provided with published papers I downloaded this repo and ran the code to review whether I could reproduce the analyses. I wanted to reach out because I was almost entirely successful (huge congrats on that!) but there were two issues that I thought you may be interested in knowing/addressing, which I've addressed in this very small pull request.
The only code I couldn't run was in
Code.Rmdat line 2046 using thevif_func(). It errored at line 14form_in <- formula(paste(val, '~', form))because the column names had spaces which formula() doesn't like. I added one line to remove the spaces from the column names and it works fine.The only other change I needed to make to run the code was related to file paths. In your
Code.Rmdfile they were hardcoded as follows:seeds_trees <- read.csv("~/Documents/GitHub/MS_JuniperusProgeny/Data/seeds_trees.csv", sep=";")The PR fixes these paths so they are generically usable for anyone opening the code in an rstudio project (ie
seeds_trees <- read.csv("../Data/seeds_trees.csv", sep=";"). Alternatively, you could use the here package or provide the user an option at the top of the script to point to the directory location.Hope these small changes are of interest, I've submitted them in good faith simply because I'd gone to the effort of rerunning the code anyway. Of course, feel free to ignore!