Skip to content

Ignore None dataset timestamps in Scene bounds - #3427

Draft
sapunyangkut wants to merge 1 commit into
pytroll:mainfrom
sapunyangkut:codex/issue-2883-resample-none-sensors
Draft

Ignore None dataset timestamps in Scene bounds#3427
sapunyangkut wants to merge 1 commit into
pytroll:mainfrom
sapunyangkut:codex/issue-2883-resample-none-sensors

Conversation

@sapunyangkut

Copy link
Copy Markdown

Summary

  • ignore None dataset timestamps before calculating Scene.start_time and Scene.end_time
  • fall back to reader times when every dataset timestamp is missing
  • cover mixed valid/missing timestamps and all-missing fallback behavior

Closes #2883.

Validation

  • focused regression cases: 3 passed on the exact upstream base and patch
  • complete scene initialization module: 21 passed on the exact upstream base and patch
  • adjacent configuration checks: 2 passed on the exact upstream base and patch
  • git diff --check, reverse patch validation, and a bounded manual security review passed
  • the current publication environment could not recollect pytest because pathlib_abc is not installed; no dependencies were installed for this submission

Checklist

AI assistance disclosure

This change was prepared and tested with OpenAI Codex. It is intentionally opened as a Draft for maintainer review. No automated issue/PR comments or Ready transition will be made.

Exclude None-valued dataset timestamps before computing Scene start and end bounds so valid dataset or reader times remain usable.

AI-assisted by OpenAI Codex; submitted as a Draft for maintainer review.
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.

Scene.start_time computation fails for datasets/composites with start_time=None (and same for end_time)

1 participant