Skip to content
GitLab

CRPS() and CRPSS(): Add parameter na.rm

Open vagudets requested to merge dev-crpss into master

Hi @cdelgado and @ptrascas,

I found this branch dev-crpss with the following changes:

  1. Adding the 'na.rm' parameter to CRPS and CRPSS, by @cdelgado
  2. Fixing a bug in the computation of the mean of the observations in .CRPS() and .CRPSS(), by @ptrascas

I have reviewed the changes, added unit tests, made some adjustments and fixed the pipeline. If you agree, I can merge these changes to the master branch so that they will be included in the next release (which should be soon).

Let me know if you have any issues with this. Also in the future please remember to open a merge request for the changes! 👼🏽

Thanks,

Victòria