Improve dagster logging when there are missing upstream dependencies

[tmplummer]

Improved dagster logging when there are missing upstream dependencies:

1. Removed the dead branch (if not dependency_inputs: in run_job) — confirmed unreachable since ProcessingInputCollection has no bool/len.
2. SPICE error now names the missing kernel types: Missing SPICE files ({spice_types}) between {start} and {end}.
3. Generic science-file error now includes the time range: No science files were discovered between {start} and {end}....

Closes: #1511

[Copilot]

Pull request overview

This PR improves observability in Dagster when a job is skipped due to missing upstream dependencies, aligning with issue #1511’s goal of making “what was missing” clearer in logs and skip reasons.

Changes:

• Removes an unused dependency-missing branch in run_job and relies on MissingDependenciesError for skip flow.
• Improves SPICE missing-dependency errors by including the configured SPICE kernel types in the message.
• Improves the generic “no science files” error by including the target time window.

---

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