Expose correlation package functionality #66

Activity

1. MuhammadAbeerAkmal commented on Aug 31, 2026

   

   MuhammadAbeerAkmal

   on Aug 31, 2026

   Contributor

   More actions

   @vcnainala I would like to work on this. I have looked at nmr-correlation's actual API: buildCorrelationData(spectra, options) and the existing nmr-cli codebase already has everything needed to fetch/parse spectra from a URL (prase-spectra.ts), so this should mostly be wiring that into a new command, following the same pattern as peaks-to-nmrium.

   One thing I need from you: nmr-correlation doesn't ship any built-in default tolerance values, the tolerance option is just an empty object if not provided. What should the default H/C tolerances be for this command? Happy to open a PR once I have that.
2. vcnainala commented on Aug 31, 2026

   

   vcnainala

   on Aug 31, 2026

   MemberAuthor

   More actions

   @MuhammadAbeerAkmal thank you for your interest. we can have the following default tolerance values

   const options = {
     tolerance: {
       C: 0.25,
       H: 0.02,
     },
     mf: 'C11H14N2O', // molecular formula
   };
   
3. hamed-musallam

   linked a pull request that will close this issuefeat(nmr-cli): add nmr-correlation as a new command #140on Sep 10, 2026