Support braces in variable names for writeReport

pik-piam / magclass

R package | Data Class and Tools for Handling Spatial-Temporal Data

GNU Lesser General Public License v3.0

4 stars 24 forks source link

Closed mikapfl closed 2 years ago

mikapfl commented 2 years ago

Hi,

This adds support for variable names containing braces in every part of the variable name to writeReport.

Also adds a battery of tests for weird variable names.

Please review this thoroughly, it is my first "proper" pull request and my first time writing R code for production.

Cheers,

Mika

mikapfl commented 2 years ago

New changes:

fixed a bug in the for loop around unitsplit which ended up converting dataframes to character instead of greping through the values in the dataframes.
converted unitsplit back to a fully vectorized function for speed