-
Notifications
You must be signed in to change notification settings - Fork 122
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
Vignettes #979
Comments
ggplot2 comparison #980 |
Thanks @nickbearman and @aduhour Yes, PRs welcome! What is important is that My idea was to structure the vignettes in such a way that it is useful for all types of users (beginners/experiences, theory/application driven). My proposal is something like this: tmap_basics:
tmap_examples:
tmap_theory:
tmap_advanced
tmap_versus:
Let me know if you agree or have suggestions to this structure and the topics Two final remarks:
Let me know if you agree on the list of topics. When we've settled on this, we can coordinate/distribute the work load to write them. Thx |
Thanks @mtennekes. I will have a read of your comment and reply in due course. I am away for the rest of December, so it will probably be in early Jan. One other comment is do you have thoughts / a structure / an outline for the various bits of documentation that exist for tmap. So this would include:
I was trying to explain how this was organized to @aduhour and realized I didn't understand it myself....! Thanks! |
😅 Totally understandable. Actually, the vignettes will not be part of the CRAN package, but only deployed online at https://r-tmap.github.io/tmap/ The help files are just very brief/to-the-point descriptions of all functions and arguments: most is already there, but if you notice that something is incomplete and important for the CRAN submission, please let us know. Good question about which version the vignettes are based on. Currently always the GH version (master branch), but the question is what to do when v4 is on CRAN. (@olivroy ?) |
Good question about which version the vignettes are based on. Currently always the GH version (master branch), but the question is what to do when v4 is on CRAN. (@olivroy ?) We can reenable site dev mode once tmap v4 hits CRAN. https://r-tmap.github.io/ would show CRAN vignettes while |
Hello! I can make a PR about https://github.com/r-tmap/tmap/blob/master/vignettes/01_basics_vv.Rmd, so you can consider my contribution so that I can make useful suggestions in the future. Thanks! |
Thanks a lot @aduhour, much appreciated! |
I added a PR about visual variables vignette. I hope it will be helpful. #992 |
Thx @aduhour I've shortened it a bit; e.g. removed the visual constant from the table, because that are for default. In essence every visual variable can be either a value or data-variable. (So I used 'visual variable' a bit broader now). |
FYI, new: https://r-tmap.github.io/tmap/articles/03_basics_facets.html What is the easiest way to cross-link between vignettes and between vignette and roxygen2 docs (@olivroy) ? |
It is easy to cross link from vignette to docs. Functions are automatically hyperlinked. We'd need to pepper-spray links to vignettes in docs. For example, in |
Yes, we need some pepper-spray (love this analogy)! Indeed via @Seealso, but also urls in each details paragraph. Before we do this, we need to make use of a standard text for the 'details' section in the tm layer functions ( |
We can do something like this #' @seealso `r .link_vig("04_basics_legends")` with this in R/docs.R .link_vig = function(title) {
paste0("Learn more in the [", title, " vignette]", "(https://r-tmap.github.io/tmap/articles/", title, ")")
} |
New vignettes drafted: versus mapsf, basic components and advanced positioning. @olivroy This |
It is simply to surround with backticks :) https://pkgdown.r-lib.org/articles/linking.html |
Hi, Thanks so much for updating the vignettes, it really helps a lot! thanks again! |
Thx for the reminder @gmgui ! Now I think all vignettes I wanted to write have been written. There may be some (or many) typos, and some minor layout thinks, but it should be sufficient for now. Let me know if something is still missing or can be improved. |
Hi,
We (@aduhour & I) are looking at the Vignettes - https://github.com/r-tmap/tmap/blob/master/vignettes/01_basics_vv.Rmd
We were planning to look through them and update them where possible.
For 1. visual variables (01_basics_vv.Rmd), it looks similar to https://r-tmap.github.io/tmap_vv.html.
Should @aduhour copy that into 01_basics_vv.Rmd?
I have looked at 03_versus_ggplot2.html and will post a PR shortly.
If there is a better way of doing this, or something else we should do, please let us know.
Thanks!
Working at FOSS4G #978
The text was updated successfully, but these errors were encountered: