Skip invalid Landsat QA asset

[Renato L. de F. Cunha (renatolfc)]

The Landsat downloader currently fetches the generic qa asset even though no workflow consumes it. For the scene reported in #116, Planetary Computer serves that asset as HTML while advertising it as a TIFF, so stacking fails before the usable qa_pixel mask is reached.

This PR removes only the unused qa key and keeps qa_pixel and qa_radsat. A focused regression checks that contract.

Closes #116.

[Copilot]

Pull request overview

This PR updates the Planetary Computer Landsat collection configuration to avoid downloading the generic qa asset, which can be served as invalid (HTML) content despite being advertised as a TIFF, causing downstream stacking failures (Issue #116).

Changes:

• Remove the unused Landsat qa asset key from LandsatCollection.asset_keys.
• Add a regression test to ensure qa is excluded and qa_pixel remains included.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/vibe_lib/vibe_lib/planetary_computer.py	Removes qa from Landsat asset download list to avoid invalid QA assets causing failures.
src/vibe_lib/tests/test_planetary_computer.py	Adds a regression test asserting qa exclusion (and qa_pixel inclusion).

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

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