Skip to content

Fix CORDEX ALADIN53 tas coordinates#3110

Open
bouweandela wants to merge 4 commits into
mainfrom
fix-aladin53-tas
Open

Fix CORDEX ALADIN53 tas coordinates#3110
bouweandela wants to merge 4 commits into
mainfrom
fix-aladin53-tas

Conversation

@bouweandela

@bouweandela bouweandela commented Jun 15, 2026

Copy link
Copy Markdown
Member

Description

  • Correct the coordinate system for ALADIN53 tas.
  • Promote projection_x_coordinate and projection_y_coordinate from iris.coords.AuxCoord to iris.coords.DimCoords if they were demoted during load because they are not monotonic.

Closes #3105

Image with the current main branch:
image

Image with this branch:
image

Recipe to generate the above images:

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 air temperature (tas).
  authors:
    - loosveldt-tomas_saskia
    - andela_bouwe
  maintainer:
    - loosveldt-tomas_saskia

preprocessors:
  tas_pp:
    custom_order: true
    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

diagnostics:
  bias:
    variables:
      tas:
        project: CORDEX
        mip: day
        domain: EUR-11
        exp: historical
        timerange: 2003/2005
        preprocessor: tas_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:
            show_stats: false

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 Jun 15, 2026
@codecov

codecov Bot commented Jun 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.18%. Comparing base (b03c23d) to head (7373fbd).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3110   +/-   ##
=======================================
  Coverage   96.18%   96.18%           
=======================================
  Files         271      271           
  Lines       15913    15926   +13     
=======================================
+ Hits        15306    15319   +13     
  Misses        607      607           

☔ View full report in Codecov by Harness.
📢 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 marked this pull request as ready for review June 15, 2026 14:42
@bouweandela bouweandela requested review from Copilot and ghossh and removed request for Copilot June 15, 2026 14:42
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

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Dataset problem: wrong values for false_northing and false_easting in CORDEX ALADIN53 tas day grid mapping in EUR-11 domain

1 participant