Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • SUNSET SUNSET
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 43
    • Issues 43
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 12
    • Merge requests 12
  • 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
  • SUNSETSUNSET
  • Merge requests
  • !204

Add script to compute Anomalies in full cross-validation

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged vagudets requested to merge dev_add-cross-anomalies into master Apr 16, 2025
  • Overview 8
  • Commits 24
  • Pipelines 10
  • Changes 30

Hi @nperez, here is the MR to add Crossval_anomalies.R to master.

Before merging, I have two things I wanted to ask:

  1. I compared the results of this multiApply version to the results of the original version. Everything is the same except for the forecast limits, which were a little different. I realized it's because in the original version, they are computed after rounding the thresholds to two decimal points (https://earth.bsc.es/gitlab/es/sunset/-/blob/dev_cross/modules/Crossval/Crossval_anomalies.R#L150) while the limits for hindcast and obs are calculated before rounding. In this new version, all the limits are calculated before rounding. Is that okay?
  2. I wanted to add Crossval_metrics.R as well, but I have noticed that for the RPS and RPSS it may not work as expected when more than one set of percentiles is requested, because the loop over categories rewrites the results as $rps/$rpss each time. So I think it may be better to wait until we have discussed how we want to approach this case, and fixed this. Let me know what you think.

P.S. I think the pipeline is failing because esarchive is not mounted on the gitlab runners yet, I have run it locally and the tests pass.

Victòria

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: dev_add-cross-anomalies