Skip to content

Range picking crashing #4355

Closed
Closed

Description

@lpatiny

If we load this spectrum and we do not phase correct but just do range detection it will crash

13c.fid.dx.zip

Activity

  1. targos commented on Sep 1, 2026

    @targos
    Member
    Image

    The combinations quickly explode.
    Luc is wondering why this part of the code is so complex.

  2. lpatiny commented on Sep 1, 2026

    @lpatiny
    MemberAuthor

    I would just take the possible candidates to match the first peak and the possible candidates to match the second peak and then evaluate all the possibilities based on the J between those 2 peaks and try to match the rest "precisely". For 13C the 1Hz seems reasonable.

  3. jobo322 commented on Sep 3, 2026

    @jobo322
    Member

    This function uses brutal force combination, and there is several noise taken as peaks, I am looking a better implementation.

  4. jobo322 commented on Sep 3, 2026

    @jobo322
    Member

    You can see the peaks around the solvent signal

    img_ecaadd530f66

  5. jobo322 commented on Sep 16, 2026

    @jobo322
    Member

    The new version of nmr-processing was updated in:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions