Skip to content

Conversation

@katjaweigel
Copy link
Contributor

@katjaweigel katjaweigel commented Nov 18, 2025

Description

In the documentation the preprocessor name "rolling_window_statistics" is missing in the example. Therefore copying the example in a recipe leads to an error.

Closes #2891

Link to documentation: https://esmvaltool--2890.org.readthedocs.build/projects/ESMValCore/en/2890/recipe/preprocessor.html#rolling-window-statistics


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:

@codecov
Copy link

codecov bot commented Nov 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.63%. Comparing base (f17b767) to head (c533761).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2890   +/-   ##
=======================================
  Coverage   95.63%   95.63%           
=======================================
  Files         263      263           
  Lines       15492    15492           
=======================================
  Hits        14816    14816           
  Misses        676      676           

☔ 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.

@katjaweigel katjaweigel marked this pull request as ready for review November 18, 2025 15:28
@katjaweigel katjaweigel moved this from Todo to In Progress in ESMValTool Workshop 2025 Nov 18, 2025
@katjaweigel katjaweigel added the documentation Improvements or additions to documentation label Nov 18, 2025
@katjaweigel katjaweigel self-assigned this Nov 18, 2025
@katjaweigel katjaweigel moved this from In Progress to Ready for Review in ESMValTool Workshop 2025 Nov 18, 2025
@bouweandela bouweandela moved this from Ready for Review to In Review in ESMValTool Workshop 2025 Nov 19, 2025
Co-authored-by: Bouwe Andela <b.andela@esciencecenter.nl>
@bouweandela bouweandela merged commit a661eb2 into main Nov 19, 2025
4 checks passed
@bouweandela bouweandela deleted the fix_small_doc_issue_preproc branch November 19, 2025 11:50
@github-project-automation github-project-automation bot moved this from In Review to Done in ESMValTool Workshop 2025 Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Documentation: Missing preprocessor name in example for Rolling window statistics.

3 participants