HIT L1A - Add attributes to science datasets

[vmartinez-cu]

This PR adds L1A science data attributes to the yaml file and updates L1A processing to assign attributes to science datasets

Closes #1056
Closes #1193

Updated Files

• imap_processing/cdf/config/imap_hit_l1a_variable_attrs.yaml
  • added science dataset attributes
• imap_processing/hit/l0/decom_hit.py
  • removed comment
• imap_processing/hit/l1a/hit_l1a.py
  • Updated code that adds attributes to science dataset
• imap_processing/hit/l2/hit_l2.py
  • Minor update to code that adds dimension labels to datasets to match approach in L1A

Testing

Updated tests where needed. Mostly related to validating coordinates that now have dimension labels

[Copilot]

Pull Request Overview

This PR adds science data attributes to L1A datasets and updates tests and data processing to support the new attribute scheme. Key changes include:

• Adding new dimension label attributes (e.g., zenith_label, azimuth_label) in L1A and L2 dataset processing.
• Updating tests to verify the presence of new coordinate labels and converting coordinate collections to sets.
• Minor comment updates in decom_hit.py.

Reviewed Changes

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

Show a summary per file

File	Description
imap_processing/tests/hit/test_hit_l2.py	Added new coordinate label assertions for scientific data attributes.
imap_processing/tests/hit/test_hit_l1b.py	Converted coordinate validations from list to set and added new label keys.
imap_processing/hit/l2/hit_l2.py	Updated coordinate assignment to directly add label arrays for dimensions.
imap_processing/hit/l1a/hit_l1a.py	Revised dataset iteration to use a dictionary for logical sources and updated coordinate attribute assignment.
imap_processing/hit/l0/decom_hit.py	Updated comment to remove obsolete reference to the YAML file.

Comments suppressed due to low confidence (2)

imap_processing/hit/l1a/hit_l1a.py:292

• The removal of coordinate assignment for each data variable (previously done with ds[field].assign_coords(dims)) may affect downstream processing that expects these mappings. Please confirm that this change is intentional and that the new approach properly handles coordinate assignments.

        for field in ds.data_vars.keys():

imap_processing/tests/hit/test_hit_l1b.py:388

• [nitpick] Changing the coordinate comparison from a list to a set assumes that the order of coordinates is irrelevant. Please verify that this change is aligned with the expected behavior of the dataset structure.

    assert valid_coords == set(l1b_standard_rates_dataset.coords), (

[sdhoyt]

Looks good!