considered num_udp_interafces for jungfrau and quad structure for eiger

include/aare/RawMasterFile.hpp

@@ -82,8 +82,10 @@ class RawMasterFile {
     size_t m_pixels_x{};
     size_t m_bitdepth{};
     size_t m_num_udp_interfaces_per_module = 1;
+    uint8_t m_quad = 0;
 
     xy m_geometry{};
+    xy m_udp_interfaces_per_module{1, 1};
 
     size_t m_max_frames_per_file{};
     // uint32_t m_adc_mask{}; // TODO! implement reading
@@ -101,7 +103,6 @@ class RawMasterFile {
     std::optional<size_t> m_digital_samples;
     std::optional<size_t> m_transceiver_samples;
     std::optional<size_t> m_number_of_rows;
-    std::optional<uint8_t> m_quad;
 
     std::optional<ROI> m_roi;
 
@@ -120,18 +121,18 @@ class RawMasterFile {
     size_t max_frames_per_file() const;
     size_t bitdepth() const;
     size_t frame_padding() const;
-    size_t num_udp_interfaces_per_module() const;
+    xy udp_interfaces_per_module() const;
     const FrameDiscardPolicy &frame_discard_policy() const;
 
     size_t total_frames_expected() const;
     xy geometry() const;
     size_t n_modules() const;
+    uint8_t quad() const;
 
     std::optional<size_t> analog_samples() const;
     std::optional<size_t> digital_samples() const;
     std::optional<size_t> transceiver_samples() const;
     std::optional<size_t> number_of_rows() const;
-    std::optional<uint8_t> quad() const;
 
     std::optional<ROI> roi() const;
 

include/aare/defs.hpp

@@ -3,16 +3,15 @@
 #include "aare/Dtype.hpp"
 
 #include <array>
-#include <stdexcept>
 #include <cassert>
 #include <cstdint>
 #include <cstring>
+#include <stdexcept>
 #include <string>
 #include <string_view>
 #include <variant>
 #include <vector>
 
-
 /**
  * @brief LOCATION macro to get the current location in the code
  */
@@ -20,28 +19,24 @@
     std::string(__FILE__) + std::string(":") + std::to_string(__LINE__) +      \
         ":" + std::string(__func__) + ":"
 
-
-
 #ifdef AARE_CUSTOM_ASSERT
 #define AARE_ASSERT(expr)                                                      \
-    if (expr)\
+    if (expr) {                                                                \
-        {}\
+    } else                                                                     \
-    else\
         aare::assert_failed(LOCATION + " Assertion failed: " + #expr + "\n");
 #else
 #define AARE_ASSERT(cond)                                                      \
-    do { (void)sizeof(cond); } while(0)
+    do {                                                                       \
+        (void)sizeof(cond);                                                    \
+    } while (0)
 #endif
 
-
 namespace aare {
 
 inline constexpr size_t bits_per_byte = 8;
 
 void assert_failed(const std::string &msg);
 
-
-
 class DynamicCluster {
   public:
     int cluster_sizeX;
@@ -179,9 +174,9 @@ template <typename T> struct t_xy {
 };
 using xy = t_xy<uint32_t>;
 
-
 /**
- * @brief Class to hold the geometry of a module. Where pixel 0 is located and the size of the module
+ * @brief Class to hold the geometry of a module. Where pixel 0 is located and
+ * the size of the module
  */
 struct ModuleGeometry {
     int origin_x{};
@@ -193,8 +188,8 @@ struct ModuleGeometry{
 };
 
 /**
- * @brief Class to hold the geometry of a detector. Number of modules, their size and where pixel 0 
+ * @brief Class to hold the geometry of a detector. Number of modules, their
- * for each module is located
+ * size and where pixel 0 for each module is located
  */
 struct DetectorGeometry {
     int modules_x{};
@@ -221,7 +216,6 @@ struct ROI{
     }
 };
 
-
 using dynamic_shape = std::vector<ssize_t>;
 
 // TODO! Can we uniform enums between the libraries?
@@ -243,7 +237,8 @@ enum class DetectorType {
     Gotthard2,
     Xilinx_ChipTestBoard,
 
-    //Additional detectors used for defining processing. Variants of the standard ones.
+    // Additional detectors used for defining processing. Variants of the
+    // standard ones.
     Moench03 = 100,
     Moench03_old,
     Unknown

src/RawFile.cpp

@@ -149,15 +149,23 @@ RawMasterFile RawFile::master() const { return m_master; }
 void RawFile::find_geometry() {
 
     for (size_t col = 0; col < m_master.geometry().col;
-         col += m_master.num_udp_interfaces_per_module())
+         col += m_master.udp_interfaces_per_module().col)
-        for (size_t row = 0; row < m_master.geometry().row; ++row) {
+        for (size_t row = 0; row < m_master.geometry().row;
-            for (size_t port = 0;
+             row += m_master.udp_interfaces_per_module().row) {
-                 port < m_master.num_udp_interfaces_per_module(); ++port) {
+            for (size_t port_row = 0;
+                 port_row < m_master.udp_interfaces_per_module().row;
+                 ++port_row)
+                for (size_t port_col = 0;
+                     port_col < m_master.udp_interfaces_per_module().col;
+                     ++port_col) {
                     ModuleGeometry g;
-                g.origin_x = (col + port) * m_master.pixels_x();
+                    g.origin_x = (col + port_col) * m_master.pixels_x();
-                g.origin_y = row * m_master.pixels_y();
+                    g.origin_y = (row + port_row) *
-                g.row_index = row;
+                                 m_master.pixels_y(); // TODO: quad doesnt seem
-                g.col_index = col + port;
+                                                      // to have an effect
+                    g.row_index = m_master.quad() ? (row + port_row + 1) % 2
+                                                  : (row + port_row);
+                    g.col_index = col + port_col;
                     g.width = m_master.pixels_x();
                     g.height = m_master.pixels_y();
                     m_geometry.module_pixel_0.push_back(g);

src/RawFile.test.cpp

@@ -200,9 +200,27 @@ TEST_CASE("check find_geometry", "[.integration][.files][.rawfile]") {
                                  ModuleGeometry{0, 0, 256, 512, 0, 0},
                                  ModuleGeometry{512, 0, 256, 512, 0, 1},
                                  ModuleGeometry{0, 256, 256, 512, 1, 0},
-                                 ModuleGeometry{512, 256, 256, 512, 1, 1}}}});
+                                 ModuleGeometry{512, 256, 256, 512, 1, 1}}}},
+
+        TestParameters{
+            "raw/jungfrau_2modules_2interfaces/run_master_0.json", 4,
+            DetectorGeometry{1, 4, 1024, 1024, 0, 0,
+                             std::vector<ModuleGeometry>{
+                                 ModuleGeometry{0, 0, 256, 1024, 0, 0},
+                                 ModuleGeometry{0, 256, 256, 1024, 1, 0},
+                                 ModuleGeometry{0, 512, 256, 1024, 2, 0},
+                                 ModuleGeometry{0, 768, 256, 1024, 3, 0}}}},
+        TestParameters{
+            "raw/W13_230320/"
+            "W13_vthreshscan_m21C_300V_800eV_vrpre3400_master_0.json",
+            2,
+            DetectorGeometry{1, 2, 512, 512, 0, 0,
+                             std::vector<ModuleGeometry>{
+                                 ModuleGeometry{0, 0, 256, 512, 0, 0},
+                                 ModuleGeometry{0, 256, 256, 512, 1, 0}}}});
 
     auto fpath = test_data_path() / test_parameters.master_filename;
+
     REQUIRE(std::filesystem::exists(fpath));
 
     RawFileTestWrapper f(fpath, "r");
@@ -242,7 +260,7 @@ TEST_CASE("check find_geometry", "[.integration][.files][.rawfile]") {
 TEST_CASE("Open multi module file with ROI",
           "[.integration][.files][.rawfile]") {
 
-    auto fpath = test_data_path() / "SingleChipROI/Data_master_0.json";
+    auto fpath = test_data_path() / "raw/SingleChipROI/Data_master_0.json";
     REQUIRE(std::filesystem::exists(fpath));
 
     RawFile f(fpath, "r");

src/RawMasterFile.cpp

@@ -139,11 +139,11 @@ size_t RawMasterFile::n_modules() const {
     return m_geometry.row * m_geometry.col;
 }
 
-size_t RawMasterFile::num_udp_interfaces_per_module() const {
+xy RawMasterFile::udp_interfaces_per_module() const {
-    return m_num_udp_interfaces_per_module;
+    return m_udp_interfaces_per_module;
 }
 
-std::optional<uint8_t> RawMasterFile::quad() const { return m_quad; }
+uint8_t RawMasterFile::quad() const { return m_quad; }
 
 // optional values, these may or may not be present in the master file
 // and are therefore modeled as std::optional
@@ -267,10 +267,13 @@ void RawMasterFile::parse_json(const std::filesystem::path &fpath) {
         // not a scan
     }
     try {
-        m_num_udp_interfaces_per_module = j.at("Number of UDP Interfaces");
+        m_udp_interfaces_per_module = {j.at("Number of UDP Interfaces"), 1};
     } catch (const json::out_of_range &e) {
-        if (m_type == DetectorType::Eiger)
+        if (m_type == DetectorType::Eiger && m_quad == 1)
-            m_num_udp_interfaces_per_module = 2;
+            m_udp_interfaces_per_module = {2, 1};
+        else if (m_type == DetectorType::Eiger) {
+            m_udp_interfaces_per_module = {1, 2};
+        }
     }
 
     try {
@@ -347,9 +350,6 @@ void RawMasterFile::parse_raw(const std::filesystem::path &fpath) {
                 if (m_type == DetectorType::Moench) {
                     m_type = DetectorType::Moench03_old;
                 }
-                if (m_type == DetectorType::Eiger) {
-                    m_num_udp_interfaces_per_module = 2;
-                }
             } else if (key == "Timing Mode") {
                 m_timing_mode = StringTo<TimingMode>(value);
             } else if (key == "Image Size") {
@@ -407,10 +407,18 @@ void RawMasterFile::parse_raw(const std::filesystem::path &fpath) {
                     static_cast<uint32_t>(std::stoi(value.substr(1, pos))),
                     static_cast<uint32_t>(std::stoi(value.substr(pos + 1)))};
             } else if (key == "Number of UDP Interfaces") {
-                m_num_udp_interfaces_per_module = std::stoi(value);
+                m_udp_interfaces_per_module = {
+                    static_cast<uint32_t>(std::stoi(value)), 1};
             }
         }
     }
+
+    if (m_type == DetectorType::Eiger && m_quad == 1) {
+        m_udp_interfaces_per_module = {2, 1};
+    } else if (m_type == DetectorType::Eiger) {
+        m_udp_interfaces_per_module = {1, 2};
+    }
+
     if (m_pixels_x == 400 && m_pixels_y == 400) {
         m_type = DetectorType::Moench03_old;
     }

src/RawMasterFile.test.cpp

@@ -153,7 +153,7 @@ TEST_CASE("Parse a master file in old .raw format",
     REQUIRE(std::filesystem::exists(fpath));
     RawMasterFile f(fpath);
 
-    CHECK(f.num_udp_interfaces_per_module() == 1);
+    CHECK(f.udp_interfaces_per_module() == xy{1, 1});
     CHECK(f.n_modules() == 2);
     CHECK(f.geometry().row == 2);
     CHECK(f.geometry().col == 1);