search

MD-Anderson-Bioinformatics / NGCHM-R

An R library for creating Next-Generation Clustered Heat Maps (NG-CHM)
http://bioinformatics.mdanderson.org/main/NG-CHM-V2:Overview
7 stars 3 forks source link

Load NGCHMSupportFiles when package loaded #50

Closed marohrdanz closed 3 months ago

marohrdanz commented 3 months ago

When the package is loaded, if environment variables SHAIDYMAPGEN and NGCHMWIDGETPATH are not set, check if NGCHMSupportFiles is available and load it. If not, install NGCHMSupportFiles and load it. Give the user feedback about all this.

I'm pretty sure CRAN is ok with this proposed behavior. I'm basing this on the sentence "Downloads of additional software or data as part of package installation or startup should only use secure download mechanisms (e.g., ‘https’). For downloads of more than a few MB, ensure that a sufficiently large timeout is set." from https://cran.r-project.org/web/packages/policies.html. However, do a little more investigation to ensure this procedure aligns with CRAN policies.

marohrdanz commented 3 months ago

After further investigation, it seems CRAN does NOT allow loading or installing other packages. When running R CMD CHECK --as-cran, NOTEs are generated if commands like library() or require() are in the code.