Skip to content

Add pyresample, trollimage, trollsift and pykdtree to show_versions defaults - #3435

Closed
khlaifiabilel wants to merge 1 commit into
pytroll:mainfrom
khlaifiabilel:add-pytroll-deps-show-versions
Closed

Add pyresample, trollimage, trollsift and pykdtree to show_versions defaults#3435
khlaifiabilel wants to merge 1 commit into
pytroll:mainfrom
khlaifiabilel:add-pytroll-deps-show-versions

Conversation

@khlaifiabilel

@khlaifiabilel khlaifiabilel commented Jul 31, 2026

Copy link
Copy Markdown

Adds the four Pytroll packages requested in #3116 to the default package list of satpy.utils.show_versions, so check_satpy() output includes them.

Happy to also add pycoast, pydecorate, and geotiepoints here if desired - they were floated in the issue but not confirmed.

Copilot AI review requested due to automatic review settings July 31, 2026 20:21

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates Satpy’s environment diagnostics so satpy.utils.show_versions() (and therefore check_satpy()) reports additional common Pytroll dependencies by default, aligning the output with expectations from issue #3116.

Changes:

  • Add pyresample, trollimage, trollsift, and pykdtree to the default show_versions package list.
  • Add a new contributor entry to AUTHORS.md.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
satpy/utils.py Extends the default dependency list printed by show_versions/check_satpy to include additional Pytroll packages.
AUTHORS.md Adds the contributor requested by the PR checklist.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@djhoese

djhoese commented Jul 31, 2026

Copy link
Copy Markdown
Member

Duplicate of #3426

@djhoese djhoese marked this as a duplicate of #3426 Jul 31, 2026
@djhoese djhoese closed this Jul 31, 2026
@khlaifiabilel

Copy link
Copy Markdown
Author

Apologies for the noise - I missed that #3426 already covered this. Thanks for the quick triage!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add additional dependencies to show_versions and check_satpy

3 participants