Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moving a couple processing tools to MTH5 #290

Merged
merged 12 commits into from
Mar 21, 2025
Merged

Moving a couple processing tools to MTH5 #290

merged 12 commits into from
Mar 21, 2025

Conversation

kujaku11
Copy link
Owner

@kujaku11 kujaku11 commented Mar 17, 2025

When processing data from an MTH5 there are a few key objects that have been developed in aurora, then ported to mtpy-v2. However, these should not be exclusive to those two packages, they could be used by all users agnostic to processing codes. Therefore, moving RunSummary and KernelDataset objects to mth5 is the best solution. This will close Issue simpeg/aurora#376.

  • RunSummary is an object that collects run information for the stations to be processed from MTH5s

  • KernelDataset is an object that organizes the run information into processing runs for processing from MTH5s.

  • Create processing module

  • Move RunSummary and all global variables to mth5.processing

  • Move KernelDataset and all global variables to mth5.processing

  • Add tests

  • Update tests in aurora

  • Update tests in 'mtpy-v2

  • Add documentation

@kujaku11 kujaku11 added the enhancement New feature or request label Mar 17, 2025
@kujaku11 kujaku11 requested a review from kkappler March 17, 2025 17:31
@kujaku11 kujaku11 self-assigned this Mar 17, 2025
@kujaku11 kujaku11 marked this pull request as ready for review March 21, 2025 23:06
@kujaku11 kujaku11 merged commit 8b77f9a into patches Mar 21, 2025
8 checks passed
@kujaku11 kujaku11 deleted the processing branch March 21, 2025 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants