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.

Sign up for GitHub

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

Jump to bottom

Features #369

Open
wants to merge 58 commits into
base: patches
Choose a base branch
from
Open

Features #369

wants to merge 58 commits into from
+322 −888

Conversation

kkappler
Copy link
Collaborator

@kkappler kkappler commented Jan 4, 2025

This mostly is associated with refactoring and added functionality in mt_metadata and mth5. This will change syntax in several of aurora's methods. These are being tracked in issue #364

@kkappler
Test new pathing for Band object in mt_metadata
f914a30 
- this requires reinstalling mt_metadata over mtpy
- I'm not sure whether or not to comment out the aurora install ..
... this may get quite complicated.  This push is a test to see how complicated
@kkappler
Reduce number of instances for test and update mt_metadata branch
cb86a49
@kkappler
try suppress mtpy-v2 reinstall
eed21ba
@kkappler
update import get_fft_harmonics to new path in mt_metadata
cb1225e
@kkappler
replace some calls to decimation attrs with access to time_series_dec…
df77152 
…imation in FCDecimation
@kkappler
change var name and add dtype (local tests pass)
5fa3fe7
@kkappler
add some dtyping and doc
4931648
@kkappler
add doc/notes
12d4cf4
@kkappler
add TODO
f0abee1
@kkappler
add typehint, tests pass locally
f7cc243
@kkappler
updates for partial compatability with mt_metadata issue 238 (local t…
65732fa 
…ests pass)
@kkappler
access recoloring via stft
ab90d8b
@kkappler
add a variation on tests
264dfc6
@kkappler
comment out notebooks for clearer error messages
c21793d
@kkappler
towards mt_metadata 235
503758b
@kkappler
Update per changes in mt_metadata
6e69bca 
- rename extra_pre_fft_detrend_type to per_window_detrend_type
- add explicit assignment of per_window_detrend_type="linear" in config creator
@kkappler
try another way to make tests pass
6a27f92
@kkappler
add test (towards issue #363)
c0a885d
@kkappler
echo some env info
41a414b
@kkappler
fix typo
5a630e2
@kkappler
update window imports
5b44182
@kkappler
towards mth5 spectrograms (partial factoring)
031202c
@kkappler
add TODO and clarify argument
14c6507
@kkappler
add some TODOs
f290c61
@kkappler
factor add_spectrogram_to_mth5 from add_fcs_to_mth5
5dab8b9
@kkappler
try another way to make tests pass
e0601d7
@kkappler
remane xrts to xrds
91fbc7b
@kkappler
modify tests to preinstall branches of mth5, mt_metadata
96b3844
@kkappler
cleanup yaml
33e88c4
@kkappler
uncommnet ipynb tests
914998a
kkappler and others added 28 commits January 4, 2025 12:13
@kkappler
Merge pull request #367 from simpeg/try_fix_tests
80ab668 
Try fix tests
@kkappler
Merge pull request #366 from simpeg/fix_mth5_issue_271
8484fb1 
MTH5 spectrograms prep (2 of several)
@kkappler
Merge pull request #365 from simpeg/fix_mt_metadata_issue_241
5fc4a0f 
MTH5 spectrograms prep (1 of several)
@kkappler
update doc and TODOs
a2a10ec
@kkappler
remove unused sample_rate validation
831a58f
@kkappler
replace decimation.window with decimation.stft.window
8f9172f
@kkappler
apply updates from mt_metadata to access stft attrs
e0e8e54
@kkappler
update syntax to new mt_metadata decimation
2f4a10d 
- also a bit of doc and dtyping
@kkappler
update ipynb for mt_metadata decimation
c229015
@kkappler
update ipynb for mt_metadata decimation
55557b5
@kkappler
update syntax to new mt_metadata decimation
3849b89
remove spectrogram (will replace with mth5 Spectrogram)
ae76455 
- minor changes to doc
replace aurora's extract_band method with mth5 method
aef6c2b
replace apply_prewhitening with mth5 method
3ec3ed5
replace aurora's apply_recoloring method with mth5's version
384a08d
suppress ipynb from tests
5cd5a84
re-add tests
57e7037
replace �_add_spectrorgrams_to_mth5 with mth5 method
e7aaf4f
replace �read_back_fcs with mth5 method
614c889
suppress ipynb tests
286a01e
replace duplicate method with mt_metadata
962906e
@kkappler
add doc
13fc2b8 
- these docstrings were in pipelines/fourier_coefficients, which is about to be deleted
@kkappler
replace imports from pipelines/fourier_coefficients with mth5
966b911 
- if tests pass we should be able to delete pipelines/fourier_coefficients
@kkappler
remove fourier_coefficients.py
3d53ddc 
- these tools are now in mth5
@kkappler
reenable ipynb tests
8eb8a24
@kkappler
update imports in ipynb
eaea779
@kkappler
replace run_ts_to_stft_scipy with mth5 method
9a53fee
@kkappler
add some typehints
07f55a2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kkappler