fix: set default coords to -1

Author: mvishiu11

Framework/include/QualityControl/TrendingTaskConfig.h

@@ -65,7 +65,7 @@ struct TrendingTaskConfig : PostProcessingConfig {
   // legend configuration
   struct LegendConfig {
     int nColumns{ 1 };
-    float x1{ 0.30f }, y1{ 0.20f }, x2{ 0.55f }, y2{ 0.35f }; // NDC coords
+    float x1{ -1.f }, y1{ -1.f }, x2{ -1.f }, y2{ -1.f }; // NDC coords
   };
 
   // this corresponds to one canvas which can include multiple graphs

Framework/src/TrendingTask.cxx

@@ -347,10 +347,10 @@ TCanvas* TrendingTask::drawPlot(const TrendingTaskConfig::Plot& plotConfig)
 
   // Legend
   TLegend* legend = nullptr;
-  if (plotConfig.legend.x1 && plotConfig.legend.y1 && plotConfig.legend.x2 && plotConfig.legend.y2) {
+  if (plotConfig.legend.x1 >= 0 && plotConfig.legend.y1 >= 0 && plotConfig.legend.x2 >= 0 && plotConfig.legend.y2 >= 0) {
     legend = new TLegend(plotConfig.legend.x1, plotConfig.legend.y1,
                          plotConfig.legend.x2, plotConfig.legend.y2,
-                         /*header=*/nullptr, /*option=*/"NDC");
+                         nullptr, "NDC");
     if (plotConfig.legend.nColumns > 0) {
       legend->SetNColumns(plotConfig.legend.nColumns);
     }

Framework/src/TrendingTaskConfig.cxx

@@ -74,10 +74,10 @@ TrendingTaskConfig::TrendingTaskConfig(std::string id, const boost::property_tre
 
     LegendConfig leg;
     leg.nColumns = plotConfig.get<int>("legend.nColumns", 1);
-    leg.x1 = plotConfig.get<float>("legend.x1", 0.30f);
-    leg.y1 = plotConfig.get<float>("legend.y1", 0.20f);
-    leg.x2 = plotConfig.get<float>("legend.x2", 0.55f);
-    leg.y2 = plotConfig.get<float>("legend.y2", 0.35f);
+    leg.x1 = plotConfig.get<float>("legend.x1", -1.f);
+    leg.y1 = plotConfig.get<float>("legend.y1", -1.f);
+    leg.x2 = plotConfig.get<float>("legend.x2", -1.f);
+    leg.y2 = plotConfig.get<float>("legend.y2", -1.f);
 
     plots.push_back({ plotConfig.get<std::string>("name"),
                       plotConfig.get<std::string>("title", ""),