Ensure proper colorer range for depth range

.github/workflows/test.yml

@@ -84,6 +84,7 @@ jobs:
         working-directory: ${{github.workspace}}/tests
         run: |
              if [ "$RUNNER_OS" == "Linux" ]; then
+               sudo apt-get install imagemagick
                ./bandage_command_line_tests.sh ${{github.workspace}}/build/BandageNG
              elif [ "$RUNNER_OS" == "macOS" ]; then
                brew install imagemagick

graph/nodecolorer.cpp

@@ -75,15 +75,25 @@ QColor DepthNodeColorer::get(const GraphicsItemNode *node) {
     double depth = deBruijnNode->getDepth();
 
     double lowValue = g_settings->lowDepthValue, highValue = g_settings->highDepthValue;
+
     if (g_settings->autoDepthValue) {
-        lowValue = m_graph->m_firstQuartileDepth;
-        highValue = m_graph->m_thirdQuartileDepth;
+        if (m_scope.graphScope() == DEPTH_RANGE) {
+            lowValue = m_scope.minDepth();
+            highValue = m_scope.maxDepth();
+        } else {
+            lowValue = m_graph->m_firstQuartileDepth;
+            highValue = m_graph->m_thirdQuartileDepth;
+        }
     }
 
     float fraction = (depth - lowValue) / (highValue - lowValue);
     return tinycolormap::GetColor(fraction, colorMap(g_settings->colorMap)).ConvertToQColor();
 }
 
+void DepthNodeColorer::saveScopeReference(graph::Scope& scope) {
+    m_scope = scope;
+}
+
 QColor UniformNodeColorer::get(const GraphicsItemNode *node) {
     const DeBruijnNode *deBruijnNode = node->m_deBruijnNode;
 

graph/nodecolorer.h

@@ -17,6 +17,8 @@
 
 #pragma once
 
+#include "graphscope.h"
+
 #include <QColor>
 #include <QSharedPointer>
 
@@ -51,6 +53,7 @@ class INodeColorer {
     static std::unique_ptr<INodeColorer> create(NodeColorScheme scheme);
 
     [[nodiscard]] NodeColorScheme scheme() const { return m_scheme; }
+
 protected:
     NodeColorScheme m_scheme;
     QSharedPointer<AssemblyGraph> &m_graph;

graph/nodecolorers.h

@@ -27,10 +27,18 @@
 
 class DepthNodeColorer : public INodeColorer {
 public:
-    using INodeColorer::INodeColorer;
-
+    explicit DepthNodeColorer(NodeColorScheme scheme)
+        : INodeColorer(scheme),
+          m_scope(graph::Scope::wholeGraph())
+    {
+    }
     QColor get(const GraphicsItemNode *node) override;
     [[nodiscard]] const char* name() const override { return "Color by depth"; };
+
+    void saveScopeReference(graph::Scope& scope);
+
+private:
+    graph::Scope m_scope;
 };
 
 class UniformNodeColorer : public INodeColorer {

ui/mainwindow.cpp

@@ -88,7 +88,8 @@ MainWindow::MainWindow(QString fileToLoadOnStartup, bool drawGraphAfterLoad) :
     QMainWindow(nullptr),
     ui(new Ui::MainWindow), m_imageFilter("PNG (*.png)"),
     m_fileToLoadOnStartup(fileToLoadOnStartup), m_drawGraphAfterLoad(drawGraphAfterLoad),
-    m_uiState(NO_GRAPH_LOADED), m_blastSearchDialog(nullptr), m_alreadyShown(false)
+    m_uiState(NO_GRAPH_LOADED), m_blastSearchDialog(nullptr), m_alreadyShown(false),
+    m_scope(graph::Scope::wholeGraph())
 {
     ui->setupUi(this);
 
@@ -1004,7 +1005,7 @@ void MainWindow::drawGraph() {
     QString errorMessage;
     g_settings->doubleMode = ui->doubleNodesRadioButton->isChecked();
 
-    auto scope =
+    m_scope =
             graph::scope(g_settings->graphScope,
                          ui->startingNodesLineEdit->text(),
                          ui->minDepthSpinBox->value(), ui->maxDepthSpinBox->value(),
@@ -1015,7 +1016,7 @@ void MainWindow::drawGraph() {
                          ui->nodeDistanceSpinBox->value());
 
     auto startingNodes = graph::getStartingNodes(&errorTitle, &errorMessage,
-                                                 *g_assemblyGraph, scope);
+                                                 *g_assemblyGraph, m_scope);
 
     if (!errorMessage.isEmpty()) {
         QMessageBox::information(this, errorTitle, errorMessage);
@@ -1024,7 +1025,7 @@ void MainWindow::drawGraph() {
 
     resetScene();
     g_assemblyGraph->resetNodes();
-    g_assemblyGraph->markNodesToDraw(scope, startingNodes);
+    g_assemblyGraph->markNodesToDraw(m_scope, startingNodes);
     layoutGraph();
 }
 
@@ -1381,6 +1382,9 @@ void MainWindow::switchColourScheme(int idx) {
         if (!g_assemblyGraph->m_csvHeaders.empty())
             colorer->setColumnIdx(0);
         ui->tagsComboBox->setVisible(true);
+    } else if (scheme == DEPTH_COLOUR) {
+        dynamic_cast<DepthNodeColorer*>(&*g_settings->nodeColorer)->saveScopeReference(m_scope);
+        ui->tagsComboBox->setVisible(false);
     } else {
         ui->tagsComboBox->setVisible(false);
     }

ui/mainwindow.h

@@ -71,6 +71,7 @@ class MainWindow : public QMainWindow
     bool m_drawGraphAfterLoad;
     UiState m_uiState;
     GraphSearchDialog * m_blastSearchDialog;
+    graph::Scope m_scope;
 
     bool m_alreadyShown;