Skip to content

Conversation

dulte
Copy link
Collaborator

@dulte dulte commented Sep 9, 2025

Change Summary

Makes o3 resample as daily max instead of mean. Done through an option in the config

Checklist

  • Start with a draft-PR
  • The PR title is a good summary of the changes
  • PR is set to AeroTools and a tentative milestone
  • Documentation reflects the changes where applicable
  • Tests for the changes exist where applicable
  • Tests pass locally
  • Tests pass on CI
  • At least 1 reviewer is selected
  • Make PR ready to review

@dulte dulte requested a review from charlienegri September 9, 2025 09:08
@dulte dulte self-assigned this Sep 9, 2025
@dulte dulte added the CAMS2_83 Issues related to the CAMS2_83 contract label Sep 9, 2025
@dulte dulte added this to the m2025-10 milestone Sep 9, 2025
Copy link

codecov bot commented Sep 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.96%. Comparing base (ae39f18) to head (badf371).

Additional details and impacted files
@@            Coverage Diff            @@
##           main-dev    #1728   +/-   ##
=========================================
  Coverage     77.96%   77.96%           
=========================================
  Files           167      167           
  Lines         22150    22150           
=========================================
  Hits          17270    17270           
  Misses         4880     4880           
Flag Coverage Δ
unittests 77.96% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@dulte dulte marked this pull request as draft September 9, 2025 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CAMS2_83 Issues related to the CAMS2_83 contract

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant