Patches #359
Merged
Patches #359
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- process_cas04_single_station was not processing, just making. - renamed it to make_cas04_single_station.ipynb. - removed it form index.rst as it is not a very interesting tutorial - it should be moved to MTH5 from being part of the docs. - process_cas04_multiple_station.ipynb should be the goto example as it was in the joss paper
- towards #358 (it is fixed but needs a test) - adding some tools in test_processing (_make_processing_config_and_kernel_dataset) to use for test in test_fourier_coefficients
- make_processing_config_and_kernel_dataset() - this fucntion has application in multiple tests - so move it to an importable location
- Now checking that the FCs are detected by TFK after build
- confirm that if the window type in the processing config does not match the window type used to create FCs, then FCs would be ignored for processing.
- use mtpy instead
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
So far:
Minor changes for earthscope 2024 workshop