Skip to content

Add fixes for CORDEX Aladin models#3059

Draft
bouweandela wants to merge 3 commits into
mainfrom
aladin
Draft

Add fixes for CORDEX Aladin models#3059
bouweandela wants to merge 3 commits into
mainfrom
aladin

Conversation

@bouweandela
Copy link
Copy Markdown
Member

@bouweandela bouweandela commented Apr 30, 2026

Description

Closes #2840

Link to documentation:


Before you get started

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.


To help with the number pull requests:

@bouweandela bouweandela added the fix for dataset Related to dataset-specific fix files label Apr 30, 2026
@bouweandela
Copy link
Copy Markdown
Member Author

bouweandela commented Apr 30, 2026

Recipe for testing:

Details
documentation:
  title: |
    Example recipe that loads CORDEX-CMIP5 datasets.
  description: |
    This is an example recipe that loads CORDEX-CMIP5 datasets.
    It computes the climatology for surface temperature (ts).
  authors:
    - loosveldt-tomas_saskia
    - andela_bouwe
  maintainer:
    - loosveldt-tomas_saskia

preprocessors:
  ts_pp:
    custom_order: true
    monthly_statistics:
      operator: mean
    climate_statistics:
      period: full
      operator: mean
    mask_landsea:
      mask_out: sea

datasets:
  - dataset: ALADIN53
    driver: CNRM-CERFACS-CNRM-CM5
    ensemble: r1i1p1
    institute: CNRM
    rcm_version: v1
    supplementary_variables:
    - mip: fx
      short_name: sftlf
  - dataset: ALADIN63
    driver: CNRM-CERFACS-CNRM-CM5
    ensemble: r1i1p1
    institute: CNRM
    rcm_version: v2
    supplementary_variables:
      - short_name: sftlf
        mip: fx

diagnostics:
  bias:
    variables:
      ts:
        short_name: ts
        project: CORDEX
        mip: day
        domain: EUR-11
        exp: historical
        timerange: 2003/2005
        preprocessor: ts_pp
    scripts:
      plot:
        plot_folder: '{plot_dir}'
        plot_filename: '{plot_type}_{real_name}_{alias}_{mip}'
        facet_used_for_labels: 'alias'
        script: monitor/multi_datasets.py
        plots:
          map:
            common_cbar: true
            fontsize: 10
            show_stats: false
            plot_kwargs_bias:
              levels: [-10.0, -7.5, -5.0, -2.5, 0.0, 2.5, 5.0, 7.5, 10.0]
              x_pos_stats_avg: 0.5
              x_pos_stats_bias: 0.5

@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

❌ Patch coverage is 32.60870% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.98%. Comparing base (6d4aa72) to head (2664c3f).

Files with missing lines Patch % Lines
...or/_fixes/cordex/cnrm_cerfacs_cnrm_cm5/aladin53.py 35.48% 20 Missing ⚠️
...or/_fixes/cordex/cnrm_cerfacs_cnrm_cm5/aladin63.py 26.66% 11 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3059      +/-   ##
==========================================
- Coverage   96.16%   95.98%   -0.19%     
==========================================
  Files         270      271       +1     
  Lines       15827    15873      +46     
==========================================
+ Hits        15220    15235      +15     
- Misses        607      638      +31     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bouweandela bouweandela reopened this May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix for dataset Related to dataset-specific fix files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dataset problem: Fixing the grids in ALADIN53 and ALADIN63 datasets

1 participant