Merge pull request #204 from astrorama/fix/output_number_of_sources

fixed regression in number of sources detected output

Author: ayllon

SEImplementation/SEImplementation/Output/TableOutput.h

@@ -41,13 +41,14 @@ class TableOutput : public Output {
       Euclid::Table::Table table {m_rows};
       m_table_handler(table);
     }
-    auto written = m_rows.size();
+    m_total_rows_written += m_rows.size();
     m_rows.clear();
-    return written;
+    return m_total_rows_written;
   }
 
   TableOutput(SourceToRowConverter source_to_row, TableHandler table_handler, size_t flush_size)
-          : m_source_to_row(source_to_row), m_table_handler(table_handler), m_flush_size(flush_size) {
+          : m_source_to_row(source_to_row), m_table_handler(table_handler),
+            m_flush_size(flush_size), m_total_rows_written(0) {
   }
 
   void outputSource(const SourceInterface& source) override {
@@ -62,6 +63,7 @@ class TableOutput : public Output {
   TableHandler m_table_handler;
   std::vector<Euclid::Table::Row> m_rows {};
   size_t m_flush_size;
+  size_t m_total_rows_written;
 };
 
 } /* namespace SourceXtractor */

SEImplementation/src/lib/Configuration/OutputConfig.cpp

@@ -78,10 +78,9 @@ void OutputConfig::initialize(const UserValues& args) {
 
   auto& format = args.at(OUTPUT_FILE_FORMAT).as<std::string>();
   m_format = format_map.at(format);
-  m_flush_size = args.at(OUTPUT_FLUSH_SIZE).as<int>();
-  if (m_flush_size < 0) {
-    m_flush_size = 0;
-  }
+
+  int flush_size = args.at(OUTPUT_FLUSH_SIZE).as<int>();
+  m_flush_size = (flush_size >= 0) ? flush_size : 0;
 }
 
 std::string OutputConfig::getOutputFile() {