Merge pull request #113 from open-ephys-plugins/issue-109

Added check for when hub IDs are not known

Author: bparks13

Source/OnixSource.cpp

@@ -401,6 +401,16 @@ bool OnixSource::initializeDevices(device_map_t deviceTable, bool updateStreamIn
 				return false;
 			}
 		}
+		else
+		{
+			Onix1::showWarningMessageBoxAsync(
+				"Unkown Hub ID",
+				"Discovered hub ID " + std::to_string(hubId) + " (" + onix_hub_str(hubId) + ") on " + OnixDevice::getPortName(hubIndex) + " which does not match any currently implemented hubs."
+			);
+			devicesFound = false;
+			sources.clear();
+			return false;
+		}
 	}
 
 	// NB: Search for passthrough devices, and initialize any headstages found in passthrough mode
@@ -416,7 +426,7 @@ bool OnixSource::initializeDevices(device_map_t deviceTable, bool updateStreamIn
 
 			auto EEPROM = std::make_unique<HeadStageEEPROM>(index, context);
 			uint32_t hsid = EEPROM->GetHeadStageID();
-			LOGD("Detected headstage ", hsid);
+			LOGD("Detected headstage ", onix_hub_str(hsid));
 
 			if (hsid == ONIX_HUB_HSNP2E)
 			{
@@ -484,6 +494,16 @@ bool OnixSource::initializeDevices(device_map_t deviceTable, bool updateStreamIn
 
 				hubNames.insert({ OnixDevice::getOffset(polledBno->getDeviceIdx()), NEUROPIXELSV1E_HEADSTAGE_NAME });
 			}
+			else
+			{
+				Onix1::showWarningMessageBoxAsync(
+					"Unkown Hub ID",
+					"Discovered hub ID " + std::to_string(hsid) + " (" + onix_hub_str(hsid) + ") on " + OnixDevice::getPortName(OnixDevice::getHubIndexFromPassthroughIndex(index)) + " which does not match any currently implemented hubs."
+				);
+				devicesFound = false;
+				sources.clear();
+				return false;
+			}
 		}
 	}