-
Notifications
You must be signed in to change notification settings - Fork 8
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
timeseries/spectre (Fix issue 269) #270
Conversation
Bump version 0.4.9 --> 0.4.10
- update test imports
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## patches #270 +/- ##
===========================================
- Coverage 62.54% 62.47% -0.07%
===========================================
Files 164 167 +3
Lines 17523 17606 +83
===========================================
+ Hits 10959 11000 +41
- Misses 6564 6606 +42
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
- this depends on mt_metadata issue 239 and the tests.yml were updated accordingly
- still needs more tests - updated task list in aurora issue 364
Continue 271 after changes in mt_metadata
update fourier coefficient method for checking fc parameters
Fix issue 271
mth5/utils/
during initial development totimeseries/spectre
.fc_tools.py
tomultiple_station.py
Spectrogam
class from aurora into mth5 (this is a fairly general class that can be used to wrangle FCs, and should not be aurora-specific)@kujaku11 the PR to bring this into
features
is an FYI to clarify these changes from what is already in patches. When you approve please consider merging patches into main as well to allow aurora to import these methods from mth5 moving forward.