Viewers icon indicating copy to clipboard operation
Viewers copied to clipboard

[Bug] Instances of one series are splitted

Open narbek-abd opened this issue 1 year ago • 3 comments

Describe the Bug

028bd0c0-4e389405-e7476fe9-1c31fff6-1306d902.zip Here is anonymized study.

image They are in the same series, but splitted

Steps to Reproduce

tried open on last version and on 3.7

The current behavior

splitted

The expected behavior

Expected instances to be in the same series so i could scroll in single viewer

OS

all

Node version

all

Browser

all

narbek-abd avatar Oct 11 '24 08:10 narbek-abd

Screencast from 11.10.2024 15:11:57.webm Removing MG from isSingleImageModality function resolved the issue. But i guess it will broke somewhere else? pls help

narbek-abd avatar Oct 11 '24 10:10 narbek-abd

Here is complete code. Worked for all different kind of mammo studies. But i'm not sure

function getDisplaySetsFromSeries(instances) {
  // If the series has no instances, stop here
  if (!instances || !instances.length) {
    throw new Error('No instances were provided');
  }
  const hasMultiSourceImageSequenceInstance = instances.some(
    i => (i.SourceImageSequence || []).length > 1
  );
...
}

const isSingleImageModality = (modality, hasMultiSourceImageSequenceInstance) => {
  return (
    modality === 'CR' ||
    (modality === 'MG' && !hasMultiSourceImageSequenceInstance) ||
    modality === 'DX'
  );
};

narbek-abd avatar Oct 15 '24 08:10 narbek-abd

Had the same issue but this seems to be the reason.

arturojain avatar Oct 15 '24 15:10 arturojain