region2/mu-comparison warn when combined symbol length is too great for basename()

ncss-tech / soilReports

An R package that assists with the setup and operation of a collection of soil data summary, comparison, and evaluation reports. These reports are primarily used by USDA-NRCS soil scientists in both initial and update mapping.

15 stars 5 forks source link

region2/mu-comparison warn when combined symbol length is too great for basename() #93

Closed dylanbeaudette closed 3 years ago

dylanbeaudette commented 6 years ago

A simple check on max symbol length should be used to trigger a suggestion on shorter symbol names. Currently, very long symbol names will cause basename() to throw an error and stop the report.

A toggle in config.R for disabling QC output would be useful for those cases where long symbol names are required.

brownag commented 3 years ago

This has been resolved with a warning that will inform the user of the options to customize the default output file names in config.R, reverting to "safe" names in the meantime. Another case of this is when there are many symbols, rather than symbols with long names. This issue will be closed by #107.