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

More PRESS support and signal processing #2

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

darrencl
Copy link
Contributor

@darrencl darrencl commented Mar 11, 2020

Press support:

  • Extract fids, simple averaging, and spectrum to support PRESS

Signal processing:

  • HSVD water suppression to work with AxisArray and added test
  • Auto phase correction
  • Ernst estimation of zero and first order phase (inspired by suspect.py, see here)
  • Baseline correction based on ALS (Asymmetric Least Square)
  • Frequency alignment

Minor: add downsample keyword arg to spectrum function

Closes #3

* Extract fids, simple averaging, and spectrum to support
  PRESS
* HSVD water suppression to work with AxisArray and added
  test
* Minor: add downsample keyword arg to spectrum function
@darrencl darrencl requested a review from c42f March 11, 2020 00:58
@darrencl darrencl force-pushed the more-press-processing branch from 60dfee6 to 8fba1f5 Compare March 11, 2020 01:44
@darrencl darrencl changed the title More 1-D processing and improve HSVD w-sup More PRESS support and improve HSVD w-sup Mar 11, 2020
@darrencl darrencl changed the title More PRESS support and improve HSVD w-sup More PRESS support and signal processing Mar 12, 2020
@darrencl
Copy link
Contributor Author

@c42f I am also adding the phase correction, which is inspired by suspect.py

@darrencl darrencl force-pushed the more-press-processing branch from 3fa57f6 to 62129d3 Compare March 13, 2020 06:36
@darrencl darrencl force-pushed the more-press-processing branch 3 times, most recently from 33e793a to f0f49bf Compare March 16, 2020 03:14
@darrencl
Copy link
Contributor Author

Hey @c42f , I am pretty much done with this PR, could you take a quick look to check if what I added makes sense?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Frequency domain processing: Phase correction
1 participant