Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • s2dv s2dv
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 16
    • Issues 16
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 2
    • Merge requests 2
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Terraform modules
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Earth SciencesEarth Sciences
  • s2dvs2dv
  • Merge requests
  • !136

Cross-validation in .get_probs, RPS and RPSS

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Carlos Delgado Torres requested to merge develop-crossRPS into master Feb 13, 2023
  • Overview 6
  • Commits 5
  • Pipelines 2
  • Changes 6

Hi @aho and @erifarov,

We have included the possibility of computing the quantiles between the probabilistic categories in cross-validation mode (i.e., leaving out the data of the time step for which the terciles are being calculated). To do so, we have modified .get_probs (where the quantiles are calculated), RPS and RPSS.

@jcos, we have slightly modified some lines in the part of the code that uses weights. We have checked that we obtain the same results as the previous version. Anyway, could you please have a look at it and let us know if anything does not make sense to you? Thanks!!

@nmilders, you can start using these functions in cross-validation mode (setting cross.val = TRUE). Could you please test it with the evaluation tool and report any problems that you may find?

Besides, ROCSS should be modified to also have the possibility of computing the probabilities in cross-validation mode. There is a merge request to include this function. @aho, what do you think is better? Can we include ROCSS in the master branch and then create another merge request to include the cross-validation into this function?

Please let us know if you want to discuss anything.

Best regards,
@nperez and Carlos

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: develop-crossRPS