Comments (8)
Thanks @TomKellyGenetics.
I have added examples for the three functions you mentioned and added extra info and the screenshot to the launch_shiny_app help file. I've also added a presize-package helpfile and the app screenshot to the readme (as per your suggestions).
A quick question regarding the examples... you say "usage for parameters without defaults explained in examples"... as the examples are on the help pages and the help pages already explain the parameters, isn't this a bit over the top? Examples in most helpfiles are very minimal...
I'm actually quite surprised that the helpfiles aren't there... I can see them when I install through remotes...
In general, one of our github actions actually builds all our package documentation for us (including the pkgdown site) each time something is merged in to the main branch, so everything should be up to date.... although this might not be the case for 0.2.2 on my fork...
Regarding your vignette suggestion, it would indeed be nice, but to do it well would require a little bit of time I think. Something to think about for the (near) future... is that OK?
from presize.
Installing remotes::install_github("CTU-Bern/presize")
installs presize version 0.2.1 and remotes::install_github("aghaynes/presize")
installs versions 0.2.2. In both cases the documentation is now available in R. 👍
Is the pgkdown
and devtools::document()
workflow triggered when merging to CTU-Bern/presize
? I see the examples are updated on your branch with comments in the code. Presuming this is merged into the package, this is acceptable.
I'd prefer more explanation of the examples such as how these apply to real-world data or study design. A vignette would be the best place for long-form documentation like this. I understand it would take more time. As examples are also explained well in the README and manuscript, this is not a strict requirement for publication.
from presize.
It is odd that for you, installing via remotes::install_github("CTU-Bern/presize")
installs v0.2.1... I get 0.2.2, as it should be...
> remotes::install_github("ctu-bern/presize")
Downloading GitHub repo ctu-bern/presize@HEAD
[truncated for brevity...]
** testing if installed package keeps a record of temporary installation path
* DONE (presize)
> packageVersion("presize")
[1] ‘0.2.2’
Yes, pkgdown
and devtools::document
are both triggered automatically on a merge (technically a push, but that's the same thing in a PR). It's handled by this action.
Re the vignette, can we consider the issue resolved if we agree to include a vignette in the next version (0.2.3 presumably, assuming CRAN accept the current version as is) of the package?
One question regarding html vignettes (as I notice that you have a few on at least one of your packages), if I may... in another package, we tried to include a html vignette and got a message back from CRAN that you have to indicate external dependencies (such as pandoc; they even set the package as to-be-archived, so we had to reformat everything for PDF vignettes). I've looked at other packages with html vignettes and could never determine where that is done... googling has also never helped me. Can you shed some light on that for me?
from presize.
You're correct, installing from GItHub now gives the correct version:
> packageVersion("presize")
[1] '0.2.2'
A possible explanation for this is that I cloned the repository and run devtools
locally to build the documentation. I may have been using an outdated version last week if the package was updated in the meantime.
I'm okay for version 0.2.2 to be published without a vignette, although I would encourage it for the next release. For HTML vignettes you have 2 options.
-
Provide vignettes in Rmarkdown. They will be automatically built as HTML on CRAN the header of the Rmarkdown file will tell CRAN which Vignette builder to use. See this example. You can check vignette building with devtools.
-
You can provide pre-generated static HTML vignettes with some modifications these are CRAN compatible.
The current docs are acceptable to me, although I understand some changes may be needed to pass CRAN checks.
from presize.
Great, thanks for the info @TomKellyGenetics :)
from presize.
@TomKellyGenetics, I guess that this issue can be closed?
from presize.
If you have more questions about the vignettes, please let me know! Otherwise the issues raised here have been addressed. I've installed the updated package from GitHub and verified that the documentation including examples is available in the R command line. 👍
from presize.
Thanks @TomKellyGenetics! :)
from presize.
Related Issues (20)
- Integrate `prec_cronb` into shiny app HOT 1
- Error in prec_meandiff()? HOT 2
- pre_riskratio lwr limit setting HOT 1
- check that vectors can be passed to all functions HOT 1
- JOSS review HOT 12
- checks HOT 1
- Improve community guidelines HOT 4
- Improve manuscript for JOSS Review HOT 2
- Reporting error: sensitivity and specificity in shiny app HOT 3
- Zenodo
- Link to JOSS review
- vignette
- prediction model methods
- mean or mu? HOT 4
- condition > 1 warning if passing multiple `conf.width`s to `prec_prop`
- Equivalent of power for presize functions HOT 1
- Adding template text for sample size calculation reporting
- clarify ICC description regarding interactions
- `prec_alpha` method for Cronbach's alpha
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from presize.