-
Notifications
You must be signed in to change notification settings - Fork 25
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
feat: two dimensions phase correction filter #2930
Conversation
Deploying with Cloudflare Pages
|
032788e
to
60986c7
Compare
60986c7
to
36ed9f2
Compare
chore: update nmr-processing to version 11.11.0 chore: fix prettier
36ed9f2
to
cc391ac
Compare
eee38af
to
54f26ee
Compare
54f26ee
to
bb06d51
Compare
I play with it seems to me quite intuitive. 2D phasing is never an easy task. I could be merge now and we can add issues after. I would prefer a toggle button rather that a drop down menu to select Vertical / Horizontal This could be a group button rather than a toggle button so that user sees he can switch. |
There is an issue because we need to apply on the 2D the phase correction at least before switching between axes. We could try to apply the 2D phase correction on mouse up (once we finished phasing PH0 and PH1). Same is true when we edit the value directly. |
…ropdown menu to switch between directions
9d638be
to
1a21386
Compare
When we change the pH values we apply them to the 2D. We debounce the applying on the 2D for 250 ms. |
b121d6f
to
a890c91
Compare
Debouncing is a good idea but if redrawing takes a long time it could still be problematic. We will see in the future if it is okay or if we should only recalculate when we release the pressed button on the button PH0 or PH1. |
No description provided.