gpl: add HPWL vs Iteration chart in debug mode

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>

Author: maliberty

src/gpl/src/graphics.cpp

@@ -55,7 +55,16 @@ Graphics::Graphics(utl::Logger* logger,
       logger_(logger),
       mode_(Nesterov)
 {
-  gui::Gui::get()->registerRenderer(this);
+  gui::Gui* gui = gui::Gui::get();
+  gui->registerRenderer(this);
+
+  // Setup the chart
+  chart_ = gui->addChart("GPL");
+  chart_->setAxisLabel("Iteration", odb::horizontal);
+  chart_->setAxisLabel("HPWL (μm)", odb::vertical);
+  chart_->setAxisFormat("%d", odb::horizontal);
+  chart_->setAxisFormat("%.2e", odb::vertical);
+
   initHeatmap();
   if (inst) {
     for (size_t idx = 0; idx < nbc_->getGCells().size(); ++idx) {
@@ -226,7 +235,7 @@ void Graphics::drawSingleGCell(const GCell* gCell, gui::Painter& painter)
   }
 
   // Highlight selection (highest priority)
-  if (gCell == nbc_->getGCellByIndex(selected_)) {
+  if (selected_ != kInvalidIndex && gCell == nbc_->getGCellByIndex(selected_)) {
     color = gui::Painter::kYellow;
     color.a = 180;
   }
@@ -293,7 +302,7 @@ void Graphics::drawNesterov(gui::Painter& painter)
   }
 
   // Draw lines to neighbors
-  if (nbc_->getGCellByIndex(selected_)) {
+  if (selected_ != kInvalidIndex && nbc_->getGCellByIndex(selected_)) {
     painter.setPen(gui::Painter::kYellow, true);
     for (GPin* pin : nbc_->getGCellByIndex(selected_)->gPins()) {
       GNet* net = pin->gNet();
@@ -385,6 +394,12 @@ void Graphics::reportSelected()
   }
 }
 
+void Graphics::addIter(const int iter)
+{
+  odb::dbBlock* block = pbc_->db()->getChip()->getBlock();
+  chart_->addPoint(iter, block->dbuToMicrons(nbc_->getHpwl()));
+}
+
 void Graphics::cellPlot(bool pause)
 {
   gui::Gui::get()->redraw();

src/gpl/src/graphics.h

@@ -54,6 +54,9 @@ class Graphics : public gui::Renderer, public gui::HeatMapDataSource
   // Draw the graphics; optionally pausing afterwards
   void cellPlot(bool pause = false);
 
+  // Update the chart for the current iter
+  void addIter(int iter);
+
   // Draw the MBFF mapping
   void mbffMapping(const LineSegs& segs);
   void mbffFlopClusters(const std::vector<odb::dbInst*>& ffs);
@@ -130,6 +133,7 @@ class Graphics : public gui::Renderer, public gui::HeatMapDataSource
   LineSegs mbff_edges_;
   std::vector<odb::dbInst*> mbff_cluster_;
   Mode mode_;
+  gui::Chart* chart_{nullptr};
 
   void initHeatmap();
   void drawNesterov(gui::Painter& painter);

src/gpl/src/nesterovPlace.cpp

@@ -423,6 +423,7 @@ int NesterovPlace::doNesterovPlace(int start_iter)
 
     const int debug_start_iter = npVars_.debug_start_iter;
     if (graphics_ && (debug_start_iter == 0 || iter + 1 >= debug_start_iter)) {
+      graphics_->addIter(iter);
       bool update
           = (iter == 0 || (iter + 1) % npVars_.debug_update_iterations == 0);
       if (update) {

src/gui/src/chartsWidget.cpp

@@ -33,9 +33,9 @@ class GuiChart : public Chart
 
   void setAxisLabel(const std::string& label,
                     odb::Orientation2D orientation) override;
-  void setAxisFormat(const std::string& label,
+  void setAxisFormat(const std::string& format,
                      odb::Orientation2D orientation) override;
-  void addPoint(const double x, const double y) override;
+  void addPoint(double x, double y) override;
 
  private:
   QChart* chart_;