Changes in newFPGA interface

lsst-ts · Jun 11, 2024 · b1f4386 · b1f4386
1 parent 93eb41b
commit b1f4386
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 11 deletions.
diff --git a/src/vmscli.cpp b/src/vmscli.cpp
@@ -47,7 +47,7 @@ class VMScli : public SimpleFPGACliApp {
     int record(command_vec cmds);
 
 protected:
-    virtual SimpleFPGA* newFPGA(const char* dir) override;
+    virtual SimpleFPGA* newFPGA(const char* dir, bool& fpga_singleton) override;
 
 private:
     std::string _subsystem = "";
@@ -145,10 +145,11 @@ int VMScli::record(command_vec cmds) {
     return 0;
 }
 
-SimpleFPGA* VMScli::newFPGA(const char* dir) {
+SimpleFPGA* VMScli::newFPGA(const char* dir, bool& fpga_singleton) {
     _settings = SettingReader::instance().loadVMSApplicationSettings(_subsystem);
     SPDLOG_INFO("Creating FPGA");
     FPGA::instance().populate(&_settings);
+    fpga_singleton = true;
 
     return &FPGA::instance();
 }

diff --git a/utils/vmsrawdump.cpp b/utils/vmsrawdump.cpp
@@ -97,15 +97,6 @@ using namespace std::chrono;
 void dump_file(const char *filename) {
     std::ifstream file;
 
-    for (int i = 1; i <= 8; i++) {
-        std::cout << "Acc" << +i;
-        if (i < 8) {
-            std::cout << ",";
-        }
-    }
-
-    std::cout << std::endl;
-
     try {
         // auto file = std::ifstream(argv[i], std::ifstream::in |
         // std::fstream::binary);
@@ -129,6 +120,11 @@ void dump_file(const char *filename) {
         uint8_t sensors = subsystem[len - 1];
         subsystem[len - 1] = 0;
 
+        if (!(sensors == 3 || sensors == 6)) {
+            std::cerr << "Expect to see 3 or 6 sensors in " << filename << ", saw " << +sensors << "."
+                      << std::endl;
+        }
+
         std::cout << subsystem << " time (UTC)";
 
         int i;