Skip to content
Snippets Groups Projects
Commit 17bdb8d4 authored by Simon Heybrock's avatar Simon Heybrock
Browse files

Re #18234. Small cleanup in MantidMatrixModel.

parent a354fcb4
No related merge requests found
...@@ -283,15 +283,10 @@ bool MantidMatrixModel::checkMonitorCache(int row) const { ...@@ -283,15 +283,10 @@ bool MantidMatrixModel::checkMonitorCache(int row) const {
if (m_monCache.contains(row)) { if (m_monCache.contains(row)) {
isMon = m_monCache.value(row); isMon = m_monCache.value(row);
} else { } else {
try { const auto &specInfo = m_workspace->spectrumInfo();
size_t wsIndex = static_cast<size_t>(row); size_t wsIndex = static_cast<size_t>(row);
auto det = m_workspace->getDetector(wsIndex); isMon = specInfo.hasDetectors(wsIndex) && specInfo.isMonitor(wsIndex);
isMon = det->isMonitor(); m_monCache.insert(row, isMasked);
m_monCache.insert(row, isMon);
} catch (std::exception &) {
m_monCache.insert(row, false);
isMon = false;
}
} }
return isMon; return isMon;
} else { } else {
...@@ -311,15 +306,10 @@ bool MantidMatrixModel::checkMaskedCache(int row) const { ...@@ -311,15 +306,10 @@ bool MantidMatrixModel::checkMaskedCache(int row) const {
if (m_maskCache.contains(row)) { if (m_maskCache.contains(row)) {
isMasked = m_maskCache.value(row); isMasked = m_maskCache.value(row);
} else { } else {
const auto &spectrumInfo = m_workspace->spectrumInfo(); const auto &specInfo = m_workspace->spectrumInfo();
size_t wsIndex = static_cast<size_t>(row); size_t wsIndex = static_cast<size_t>(row);
if (spectrumInfo.hasDetectors(wsIndex)) { isMasked = specInfo.hasDetectors(wsIndex) && specInfo.isMasked(wsIndex);
isMasked = spectrumInfo.isMasked(wsIndex); m_maskCache.insert(row, isMasked);
m_maskCache.insert(row, isMasked);
} else {
m_maskCache.insert(row, false);
isMasked = false;
}
} }
return isMasked; return isMasked;
} else { } else {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment