-
Notifications
You must be signed in to change notification settings - Fork 0
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
Cran submission 0.7.6 #33
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general, looks very good to me! I requested some optional changes according to my preferences, however, this could also be submitted as is.
NEWS.md
Outdated
@@ -1,8 +1,10 @@ | |||
# eatTools 0.7.6 | |||
|
|||
* add argument checks using the checkmate package | |||
* bugfix in `halveString()` for splitting at the last instance of a regular expression | |||
* new function `rbind_fill_vector()` allows `rbind()` for non-dimensional vectors of unequal length, filling empty entries with NA |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NIT: vectors are unidimensional and not non-dimensional? (r didnt i get what you meant?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I changed it to "unidimensional"
R/makeTria.r
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does this funtion do? According to initial comment and man-page a matrix should be input, but instead a data.frame is taken and returned. Further, I expected according to description that it converts full covariance matrices to triangles but this is just re-sort of columns according to the amount of missings in these columns. This should be described in man-page and further checks should be added that input corresponds to what is needed (dfr and cols with unequal amounts of NAs).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added some checks in the function code and a details section in the description. The function should actually only be called internally and is only exported to the namespace so that eatAnalysis
and eatModel
packages can access it.
Some final adjustments so that the package can be submitted to CRAN.