-
Notifications
You must be signed in to change notification settings - Fork 29
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
Updated help file for BayesianSetupGenerateParallel.R #255
Updated help file for BayesianSetupGenerateParallel.R #255
Conversation
|
#' By default a copy of your workspace is exported. However, depending on your workspace this can be very inefficient. | ||
#' | ||
#' Alternatively you can specify the environments and packages in the likelihood function (e.g. BayesianTools::VSEM() instead of VSEM()). | ||
#' @param parallel should a parallel R cluster be used? If set to T, machine will |
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.
With the line breaks, I can't see which words have changed in the diff because all seems to be changed - could you change back to how it was before?
#' By default a copy of your workspace is exported. However, depending on your workspace this can be very inefficient. | ||
#' | ||
#' Alternatively you can specify the environments and packages in the likelihood function (e.g. BayesianTools::VSEM() instead of VSEM()). | ||
#' @param parallel should a parallel R cluster be used? If set to T, machine will |
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.
Why "machine"?
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.
Probably you mean the operating system
Hi @TahminaMojumder - can you look at this PR - also note that the checks don't run, do they run for you on your machine? I suspect that this is because a line in the code was erased. |
Hi Tahmina, In this branch, the checks produce an error, the slash n is apparently not supported by Roxygen, needs to be corrected |
Hi @TahminaMojumder, a) I think you accidentally changed a few other files as well, see file changes of the last commit - I would revert this, just because of tidiness b) For the list, don't use itemise, use standard markdown commands, this is better to read / maintain. See https://mpn.metworx.com/packages/roxygen2/7.1.1/articles/rd-formatting.html#lists-2 |
|
|
There were some files that did not have changes but was still committed. Maybe, when I made changes from the windows it was creating white spaces and considering the white spaces as a change. |
No description provided.