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 #276

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

Features #276

wants to merge 91 commits into from

Conversation

kujaku11
Copy link
Owner

@kujaku11 kujaku11 commented Jan 3, 2025 •
edited by kkappler
Loading

Adding ability to include FC features.

Nice to haves include:

  • An accessor for runts to yield a spectrogram

kujaku11 and others added 30 commits December 13, 2024 13:00
@kujaku11
Create features.py
faa3321
@kujaku11
starting features
738be02
@kkappler
add some placeholders for functions to write
d92a222
add covariance
3a1a4f2
@kkappler
bring aurora spectrogram to mth5
3d79dc3
@kkappler
Move fc_tools from util to time_series/spectre
2806314 
- update test imports
@kkappler
move test_fc_tools to tests/timeseries/spectre
d5179c3
@kkappler
fix import paths
52a9932
@kkappler
merge fc_tools from features branch
6008fa2
@kkappler
Merge branch 'fix_issue_269' into kk/features
2d4db64
@kkappler
remove features-specific placeholders
7f1a740
@kkappler
rename fc_tools to multiple_station
dd28188
@kkappler
doc / housekeeping
20ccfcf
@kkappler
rename file
0bf5d02
@kkappler
doc / housekeeping
2225bcf
@kkappler
Merge branch 'fix_issue_269' into kk/features
9c867f8
@kkappler
add feature stuffs back after fix_issue_269 merge
a276ec4
@kkappler
Move test from aurora over to mth5
ebcbc5c
@kkappler
Merge branch 'fix_issue_269' into kk/features
b0f0bae
@kkappler
fix import
a3b62b9
@kkappler
Merge branch 'fix_issue_269' into kk/features
3f9d2b8
@kkappler
import Spectrogram to __init__
769b6ea
@kkappler
Merge branch 'fix_issue_269' into kk/features
0dbc60d
@kkappler
add extract band method, needed for spectrogram
2b00f68
@kkappler
Merge branch 'fix_issue_269' into kk/features
b03c8aa
@kkappler
minor housekeeping changes
ca4cfc3
@kkappler
Merge branch 'patches' into fix_issue_271
5d2d19b
@kkappler
Merge branch 'patches' into fix_issue_269
2c131cc
@kkappler
add modules for stft
37a3217
@kkappler
add TODO
8f2addf
kkappler and others added 30 commits January 10, 2025 18:42
@kkappler
add separate method for general 2d and covariance xarrays
8e2c85c 
- update tests to pass
@kkappler
add cross_powers method from cursor
f24d08d
@kkappler
clean up spectrogram.cross_powers
5920804
@kkappler
add commneted lines
89e597e
@kkappler
update xarray helpers
4c6778d
@kkappler
update crosspower to only operate on valid bands
35399be 
- add some tests
@kkappler
add TODO note to doc
deef033
@kkappler
remove dependance on aurora
3b71ba6
@kkappler
Merge pull request #277 from kujaku11/cursor_001
dfe98a0 
Cursor 001
@kkappler
gitignore pycharm folders
9c9987f
@kkappler
change imports to work from higher level
f99adac 
- also add some AI generated typehints
@kkappler
invert order of class definition for typehinting
97e1278
@kkappler
minor updates to typehints
2b56a33
make rowvar argument settable in covariance_xr
ae46ebe
@kkappler
update default bias in covariance
1c97f7c
@kujaku11
adjusting file structure figure
9bdf711
@kujaku11
Update features.py
a64f709
@kujaku11
Update features.py
929d36c
@kujaku11
adding in how features are accessed and organized
9283c87
@kujaku11
adding fc and ts features, got to decimation level
2b5931d
Synthetic Data Updates
fdc9abd 
- generally working towards the ability to reuse the tools in
  make_mth5_from_ascii to make more general mth5.
- the way things were set up were completely dependent on using data
  from the legacy EMTF ascii files
- I have tried to make a SyntheticRun object be able to optionally store a
  dataframe in place, and only load the legacy file if there is not a
  dataframe present
- also improve docstrings and typehinting
move one-off upsampleing into legacy emtf ascii class
3e46a59 
- some tidying and doc
@kujaku11
updated logic for setting save path
2ba6552
@kujaku11
Merge branch 'features' of https://github.com/kujaku11/mth5 into feat…
5450735 
…ures
@kujaku11
updating how features are implemented
a16727b
@kujaku11
fixing client tests
650f1f4
Minor update to making synthetioc data
e098bec 
- factor method to get target folder out of create_mth5_synthetic_file
- remove unused import
@kkappler
Add features integrated test
1c1a387 
- generate some cross powers
- store them as features
- close the h5
- open the h5
- read the features
- assert equal to the originally computed values
@kkappler
add some doc and notes
b6419d2
@kkappler
add multiple cross power channels to test
7c8fbf2
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.
3 participants
@kujaku11 @codecov-commenter @kkappler