Skip to content

Remove unnessary dependencies and pin iris < 3.16 - #3216

Merged
bouweandela merged 4 commits into
mainfrom
update-deps
Sep 4, 2026
Merged

Remove unnessary dependencies and pin iris < 3.16#3216
bouweandela merged 4 commits into
mainfrom
update-deps

Conversation

@bouweandela

@bouweandela bouweandela commented Sep 4, 2026

Copy link
Copy Markdown
Member

Description

Reduce the size of the conda environments.

Size on current main:

2.1G	.pixi/envs/default/
822M	.pixi/envs/esmvalcore/

Size with this pull request:

1.8G    .pixi/envs/default/
519M	.pixi/envs/esmvalcore/
  • Move esmvaltool-sample-data to the test dependencies as it is not required for normal runs.
  • Pin iris to < 3.16 because the improved string handling in iris 3.16 appears to be incompatible with ncdata Incompatible with iris 3.16 SciTools/ncdata#255
  • Remove the intake-esm dependency because ESMValCore does not currently support intake-esm, this is still work in progress in Intake-ESM Integration based on #1218 #2690
  • Use the prov-core conda-forge package, which has fewer dependencies than the prov conda-forge package
  • Update to the most recent build of esgf-pyclient, which no longer pulls in myproxyclient

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

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.48%. Comparing base (539b0fe) to head (9829c71).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3216   +/-   ##
=======================================
  Coverage   96.47%   96.48%           
=======================================
  Files         282      282           
  Lines       16407    16407           
=======================================
+ Hits        15829    15830    +1     
+ Misses        578      577    -1     

☔ 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 changed the title Remove unnessary dependencies and pin iris Remove unnessary dependencies and pin iris < 3.16 Sep 4, 2026
@bouweandela bouweandela added the installation Installation problem label Sep 4, 2026
@bouweandela
bouweandela marked this pull request as ready for review September 4, 2026 10:02

@schlunma schlunma left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Sad that we need to pin Iris 😢

@bouweandela
bouweandela merged commit 7e2f106 into main Sep 4, 2026
6 checks passed
@bouweandela
bouweandela deleted the update-deps branch September 4, 2026 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

installation Installation problem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants