Updated Hdf5Recorder.cpp and Hdf5Recorder.h

Author: Zaina Shaikh

Simulator/Recorders/Hdf5Recorder.cpp

@@ -189,13 +189,21 @@ void Hdf5Recorder::compileHistories()
                } else if (variableInfo.hdf5Datatype_ == PredType::NATIVE_INT) {
                   vector<int> dataBuffer(variableInfo.variableLocation_.getNumElements());
                   for (size_t i = 0; i < variableInfo.variableLocation_.getNumElements(); ++i) {
-                     if constexpr (std::is_same_v<
+                     // For type int, a distinction needs to be made between vertexType and regular integers
+                     // Since vertexType is an enum class, it needs to first be converted to an integer before storing
+                     
+                     // 'decltype' determines the type at compile time 
+                     // 'decay_t' removes any references/const from the type
+                     // This simplifies comparisons through 'is_same_v' to ensure type matches vertexType
+                     if (std::is_same_v<
                                       vertexType,
                                       std::decay_t<decltype(get<vertexType>(
                                          variableInfo.variableLocation_.getElement(i)))>>) {
+                        // If type matches vertexType, convert to int before storing
                         dataBuffer[i] = static_cast<int>(
                            get<vertexType>(variableInfo.variableLocation_.getElement(i)));
                      } else {
+                        // Otherwise, store as a regular integer  
                         dataBuffer[i] = get<int>(variableInfo.variableLocation_.getElement(i));
                      }
                   }

Simulator/Recorders/Hdf5Recorder.h

@@ -138,11 +138,19 @@ class Hdf5Recorder : public Recorder {
             } else if (hdf5Datatype_ == PredType::NATIVE_INT) {
                vector<int> dataBuffer(variableLocation_.getNumElements());
                for (int i = 0; i < variableLocation_.getNumElements(); ++i) {
-                  if constexpr (std::is_same_v<vertexType, std::decay_t<decltype(get<vertexType>(
+                  // For type int, a distinction needs to be made between vertexType and regular integers
+                  // Since vertexType is an enum class, it needs to first be converted to an integer before storing
+                     
+                  // 'decltype' determines the type at compile time 
+                  // 'decay_t' removes any references/const from the type
+                  // This simplifies comparisons through 'is_same_v' to ensure type matches vertexType
+                  if (std::is_same_v<vertexType, std::decay_t<decltype(get<vertexType>(
                                                               variableLocation_.getElement(i)))>>) {
+                     // If type matches vertexType, convert to int before storing
                      dataBuffer[i]
                         = static_cast<int>(get<vertexType>(variableLocation_.getElement(i)));
                   } else {
+                     // Otherwise, store as a regular integer 
                      dataBuffer[i] = get<int>(variableLocation_.getElement(i));
                   }
                }