Skip to content

COMP: Install xarray in Python CI and report missing-dependency skips - #7004

Open
jdotc1 wants to merge 2 commits into
InsightSoftwareConsortium:mainfrom
jdotc1:fix/xarray-python-ci-20261024
Open

jdotc1 wants to merge 2 commits into
InsightSoftwareConsortium:mainfrom
jdotc1:fix/xarray-python-ci-20261024

Conversation

@jdotc1

@jdotc1 jdotc1 commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

Install xarray in macOS and Windows Python CI and mark PythonXarrayTest as skipped when xarray is absent. Addresses the CI checks in #6950; 4D wrapping remains a separate follow-up.

Validation
  • Parsed all three Python pipeline YAML files and checked xarray installation commands.
  • Checked the CTest skip registration and whitespace.
  • Full native CTest was unavailable locally (no CMake/Pixi build); pipeline validation is pending.
AI assistance

Automated coding assistance prepared this CI configuration change and ran static checks. CI results and contributor review are required before marking ready.

@github-actions github-actions Bot added type:Compiler Compiler support or related warnings type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct labels Oct 24, 2026
@jdotc1
jdotc1 marked this pull request as ready for review October 25, 2026 00:09
@greptile-apps

greptile-apps Bot commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] The PR appears safe to merge.

Summary

Adds xarray to the macOS and Windows Python CI dependency installs.

  • macOS and Windows Python CI install xarray and report missing-package skips.

Reviews (1) · Last reviewed commit: "STYLE: Format xarray test skip property" · Reviewed by Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:Compiler Compiler support or related warnings type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant