Skip to content
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

Open
nickbearman opened this issue Dec 7, 2024 · 7 comments
Open

Vignettes #979

nickbearman opened this issue Dec 7, 2024 · 7 comments

Comments

@nickbearman
Copy link
Contributor

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

@nickbearman
Copy link
Contributor Author

ggplot2 comparison #980

@mtennekes
Copy link
Member

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:

  • visual variables
  • scales
  • facets
  • legends
  • modes (plot and view)
  • basemaps
  • layout
  • exporting maps

tmap_examples:

  • choropleth
  • bubble map
  • raster map (input from @gilbertocamara)
  • bivarite choropleth
  • gridmap
  • topographic map

tmap_theory:

  • grammar of graphics
  • map projections

tmap_advanced

  • options
  • mode extensions
  • layer extensions

tmap_versus:

  • ggplot2
  • mapview
  • leaflet

Let me know if you agree or have suggestions to this structure and the topics

Two final remarks:

  • @Nowosad and I will update/rewrite the book https://github.com/r-tmap/tmap-book (contributions welcome). So in other words, the vignettes don't need to be detailed. Furthermore, it would be preferable if we can use tmap's own datasets.
  • For the v4 release we don't need all the vignettes, and also not in too much detail.

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

@nickbearman
Copy link
Contributor Author

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!
Nick.

@mtennekes
Copy link
Member

I was trying to explain how this was organized to @aduhour and realized I didn't understand it myself....!

😅 Totally understandable.

Actually, the vignettes will not be part of the CRAN package, but only deployed online at https://r-tmap.github.io/tmap/
Two reasons: overlap (as you've pointed out) and the other is the CRAN package size restriction (image rich vignettes are large).

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 ?)

@olivroy
Copy link
Contributor

olivroy commented Dec 9, 2024

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
https://r-tmap.github.io/dev would show the development version vignettes.

@aduhour
Copy link

aduhour commented Dec 11, 2024

Hello!
Thank you for your replies. I am interested, as a user of tmap, in to cotribute in what I can.
I could read today about what is needed and the structure suggestions.

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!
Andrés

@mtennekes
Copy link
Member

Thanks a lot @aduhour, much appreciated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants