Deprecate pds3.py

[jnspitale]

Fixes #184, #185, and #186

Summary by CodeRabbit

• Refactor

  • Updated label parsing across Cassini and Galileo instrument modules to use a new parsing approach.

• Bug Fixes

  • Fixed an iteration bug in Cassini UVIS that could skip processing of windowed data.

• Chores

  • Added test framework import to support unit testing.

[coderabbitai]

Walkthrough

Replaces legacy PDS3 label handling with pdsparser.Pds3Label(...).as_dict() in Cassini (UVIS, VIMS) and Galileo (SSI) hosts; removes a pds3 public import; fixes an iteration bug in uvis.get_qube; and adds a unittest import to tests/hosts/hst/__init__.py.

Changes

Cohort / File(s)	Summary
Cassini UVIS oops/hosts/cassini/uvis.py	Replaced pds3.fast_dict(filespec) with pdsparser.Pds3Label(filespec, method='fast').as_dict(); removed from oops.hosts import pds3 import; fixed loop for w in range(len(line0)) (previously for w in len(line0)).
Cassini VIMS oops/hosts/cassini/vims.py	Replaced multiple pds3.fast_dict(...) usages (constants and from_file) with pdsparser.Pds3Label(..., method='fast').as_dict() to obtain label dicts.
Galileo SSI oops/hosts/galileo/ssi/__init__.py	Replaced pds3.get_label(filespec) with pdsparser.Pds3Label(filespec).as_dict() and removed dependence on pds3 import for label retrieval.
Tests (HST) tests/hosts/hst/__init__.py	Added import unittest at module top.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related issues

• Use pdsparser.PdsLabel.from_file() instead of pds3.get_label() in host modules #186: Both replace legacy pds3 label-reading calls with pdsparser.Pds3Label(...).as_dict(), aligning label-parsing changes across host modules.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Deprecate pds3.py' is clear and directly related to the main change: replacing pds3 module usage with pdsparser across multiple files.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into main
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[markshowalter]

One optional comment is that, in general, you should probably use method='fast' by default, and only abandon it if you discover empirically that it doesn't always work for some data set. But that's up to Joe.