cheminfo / nmrium

React component to display and process nuclear magnetic resonance (NMR) spectra.
https://docs.nmrium.org
MIT License
55 stars 25 forks source link

unhandled error in xyz auto signals picking #2483

Closed targos closed 1 year ago

targos commented 1 year ago
observedFrequencies is mandatory

{
  source: 'webpack://_N_E/node_modules/nmrium/lib/component/reducer/Reducer.js',
  line: 328,
  column: 56,
  name: 'draft'
}
{
  source: 'webpack://_N_E/node_modules/nmrium/lib/component/reducer/actions/ZonesActions.js',
  line: 47,
  column: 22,
  name: 'detectZones'
}
{
  source: 'webpack://_N_E/node_modules/nmrium/lib/data/data2d/Spectrum2D/zones/detectZones.js',
  line: 4,
  column: 18,
  name: 'getDetectionZones'
}
{
  source: 'webpack://_N_E/node_modules/nmrium/lib/data/data2d/Spectrum2D/zones/getDetectionZones.js',
  line: 23,
  column: 0,
  name: 'dataMatrix'
}
{
  source: 'webpack://_N_E/node_modules/nmrium/lib/data/data2d/Spectrum2D/zones/getDetectionZones.js',
  line: 31,
  column: 35,
  name: 'data'
}
{
  source: 'webpack://_N_E/node_modules/nmr-processing/lib-esm/xyz/xyzAutoZonesPicking.js',
  line: 4,
  column: 20,
  name: 'xyzAutoSignalsPicking'
}
{
  source: 'webpack://_N_E/node_modules/nmr-processing/lib-esm/xyz/xyzAutoSignalsPicking.js',
  line: 25,
  column: 0,
  name: null
}
hamed-musallam commented 1 year ago

i tried all the possible scenarios but I still can not reproduce this bug, maybe the imported file is corrupted

hamed-musallam commented 1 year ago

maybe it is related to the predicted 2D spectra, I'm checking

hamed-musallam commented 1 year ago

fixed by https://github.com/cheminfo/nmrium/pull/2487