Documentation: derive_vars_dt()

[Fanny-Gautier]

Please select a category the issue is focused on?

Function Documentation

Let us know where something needs a refresh or put your idea here!

Could the error/warning message or the function's documentation be updated for the permitted values of date_imputation? Or further checks applied for the argument date_imputation?

When not "properly" used (e.g. "15" instead of "mid") the warning message is not very clear and the results are scary. Currently "15" seems permitted.

It's not visible when there is no partial date to impute, but as soon as there is data (which wasn't the case with previous data transfers) to impute, there's this weird message that the user needs to pay attention to and understand.

[bms63]

Probably should improve that error message and not have mutate() in it! @pharmaverse/admiral

I think we could apply a check on date_imputation for sure.

[bundfussr]

Yes, we should check the validity of date_imputation. Most likely the invalid date_imputation value causes the warnings.