Skip to content

Conversation

@laspsandoval
Copy link
Contributor

@laspsandoval laspsandoval commented Jan 9, 2026

Change Summary

Overview

Complete change in cdf structure and updates to variable attributes based on Andriy's feedback. Also note that I intend to calculate each instrument's timestamp (Issue: #2571 ).

Updated Files: Major changes

  • create_xarray.py
    • Basically a complete rework of this file. A lot has changed with the database. CoDICE data structure is now more clearly defined. And after speaking with Andriy I realized the need for an epoch for each instrument.
  • imap_ialirt_l1_variable_attrs.yaml
    • A lot of feedback from Andriy for this.

Updated Files: Minor changes

  • imap_processing/ialirt/utils/constants.py
    • Organization of dimensions, energy bins, and data types.
  • constants.py
    • Added constants based on CoDICE Algorithm Document page 64-65. These are not stored in DynamoDB (website documentation is still needed here), but they will be used in the cdf.
  • imap_ialirt_global_cdf_attrs.yaml
    • Global description as recommended
  • parse_mag.py
    • Find centerpoint of sc timestamp (instead of first timestamp)
  • process_codice.py
    • Find centerpoint of sc timestamp (instead of first timestamp)
  • process_hit.py
    • Find centerpoint of sc timestamp (instead of first timestamp)
  • process_swapi.py
    • Find centerpoint of sc timestamp (instead of first timestamp)
  • process_swe.py
    • Find centerpoint of sc timestamp (instead of first timestamp)

Testing

test_create_xarray.py

@laspsandoval laspsandoval self-assigned this Jan 9, 2026
@laspsandoval laspsandoval marked this pull request as draft January 9, 2026 15:51
@laspsandoval laspsandoval marked this pull request as ready for review January 9, 2026 20:22
@laspsandoval laspsandoval requested a review from greglucas January 9, 2026 20:22
@laspsandoval laspsandoval marked this pull request as draft January 9, 2026 21:31
@laspsandoval laspsandoval marked this pull request as ready for review January 10, 2026 00:32
@laspsandoval laspsandoval marked this pull request as draft January 10, 2026 00:46
@laspsandoval laspsandoval marked this pull request as ready for review January 14, 2026 19:52
@laspsandoval
Copy link
Contributor Author

@greglucas I think this is ready for your review when you have the chance. Really appreciate it.

@laspsandoval
Copy link
Contributor Author

pre-commit.ci autofix

Copy link
Collaborator

@greglucas greglucas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding that helper function for the header!

@laspsandoval laspsandoval merged commit 37b0292 into IMAP-Science-Operations-Center:dev Jan 22, 2026
14 checks passed
@laspsandoval laspsandoval deleted the andriy_cdf_response branch January 22, 2026 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants