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 added this to the January 2026 milestone Jan 9, 2026
@laspsandoval laspsandoval changed the title I-ALiRT - Andriy cdf response DRAFT I-ALiRT - Andriy cdf response Jan 9, 2026
@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.

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