seandavi / MicroBioMap

https://seandavi.github.io/MicroBioMap/
Other
3 stars 3 forks source link

Clean up non-idiomatic code in vignette #14

Open seandavi opened 10 months ago

seandavi commented 10 months ago

from @antagomir

While I went through the vignette, I noticed some minor points that you may find useful for your next version. Use as you wish:

  1. colData(cpd)$country (and other similar cases) can be also written more shortly just like: cpd$country
  2. ggplot(as.data.frame(colData(cpd)))... -> the as.data.frame() was necessary until very recently, but not needed any more since the issue was solved ; the latest ggplot2 version allows this:
    ggplot(colData(cpd))... 

    We are adding a reference to this data resource in the OMA Gitbook (beta version, under active development)