docs: add XML documentation to PepperDash.Core project

Co-authored-by: andrew-welker <1765622+andrew-welker@users.noreply.github.com>

src/PepperDash.Core/Comm/CommunicationGather.cs

@@ -88,6 +88,9 @@ namespace PepperDash.Core
 		/// Disconnects this gather from the Port's TextReceived event. This will not fire LineReceived
 		/// after the this call.
 		/// </summary>
+  /// <summary>
+  /// Stop method
+  /// </summary>
 		public void Stop()
 		{
 			Port.TextReceived -= Port_TextReceived;

src/PepperDash.Core/Comm/CommunicationStreamDebugging.cs

@@ -25,6 +25,9 @@ namespace PepperDash.Core
         /// <summary>
         /// The current debug setting
         /// </summary>
+        /// <summary>
+        /// Gets or sets the DebugSetting
+        /// </summary>
         public eStreamDebuggingSetting DebugSetting { get; private set; }
 
         private uint _DebugTimeoutInMs;
@@ -44,6 +47,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Indicates that receive stream debugging is enabled
         /// </summary>
+        /// <summary>
+        /// Gets or sets the RxStreamDebuggingIsEnabled
+        /// </summary>
         public bool RxStreamDebuggingIsEnabled{ get; private set; }
 
         /// <summary>
@@ -65,6 +71,9 @@ namespace PepperDash.Core
         /// Sets the debugging setting and if not setting to off, assumes the default of 30 mintues
         /// </summary>
         /// <param name="setting"></param>
+        /// <summary>
+        /// SetDebuggingWithDefaultTimeout method
+        /// </summary>
         public void SetDebuggingWithDefaultTimeout(eStreamDebuggingSetting setting)
         {
             if (setting == eStreamDebuggingSetting.Off)
@@ -81,6 +90,9 @@ namespace PepperDash.Core
         /// </summary>
         /// <param name="setting"></param>
         /// <param name="minutes"></param>
+        /// <summary>
+        /// SetDebuggingWithSpecificTimeout method
+        /// </summary>
         public void SetDebuggingWithSpecificTimeout(eStreamDebuggingSetting setting, uint minutes)
         {
             if (setting == eStreamDebuggingSetting.Off)
@@ -135,6 +147,9 @@ namespace PepperDash.Core
     /// The available settings for stream debugging
     /// </summary>
     [Flags]
+    /// <summary>
+    /// Enumeration of eStreamDebuggingSetting values
+    /// </summary>
     public enum eStreamDebuggingSetting
     {
         /// <summary>

src/PepperDash.Core/Comm/ControlPropertiesConfig.cs

@@ -8,6 +8,9 @@ namespace PepperDash.Core
     /// <summary>
     /// Config properties that indicate how to communicate with a device for control
     /// </summary>
+    /// <summary>
+    /// Represents a ControlPropertiesConfig
+    /// </summary>
     public class ControlPropertiesConfig
     {
         /// <summary>

src/PepperDash.Core/Comm/EventArgs.cs

@@ -32,6 +32,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the Client
+  /// </summary>
 		public ISocketStatus Client { get; private set; }
 
         /// <summary>
@@ -62,6 +65,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+        /// <summary>
+        /// Gets or sets the State
+        /// </summary>
         public ServerState State { get; private set; }
 
         /// <summary>
@@ -156,6 +162,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Text
+        /// </summary>
         public string Text { get; private set; }
 
         /// <summary>

src/PepperDash.Core/Comm/GenericSecureTcpIpClient.cs

@@ -81,6 +81,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Port on server
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Port
+        /// </summary>
         public int Port { get; set; }
 
         /// <summary>
@@ -151,6 +154,9 @@ namespace PepperDash.Core
         /// <summary>
         /// bool to track if auto reconnect should be set on the socket
         /// </summary>
+        /// <summary>
+        /// Gets or sets the AutoReconnect
+        /// </summary>
         public bool AutoReconnect { get; set; }
 
         /// <summary>
@@ -190,6 +196,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Bool to show whether the server requires a preshared key. This is used in the DynamicTCPServer class
         /// </summary>
+        /// <summary>
+        /// Gets or sets the SharedKeyRequired
+        /// </summary>
         public bool SharedKeyRequired { get; set; }
 
         /// <summary>
@@ -209,6 +218,9 @@ namespace PepperDash.Core
         /// <summary>
         /// SharedKey is sent for varification to the server. Shared key can be any text (255 char limit in SIMPL+ Module), but must match the Shared Key on the Server module
         /// </summary>
+        /// <summary>
+        /// Gets or sets the SharedKey
+        /// </summary>
         public string SharedKey { get; set; }
 
         /// <summary>
@@ -224,6 +236,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Bool showing if socket is ready for communication after shared key exchange
         /// </summary>
+        /// <summary>
+        /// Gets or sets the IsReadyForCommunication
+        /// </summary>
         public bool IsReadyForCommunication { get; set; }
 
         /// <summary>
@@ -344,6 +359,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Just to help S+ set the key
         /// </summary>
+        /// <summary>
+        /// Initialize method
+        /// </summary>
         public void Initialize(string key)
         {
             Key = key;
@@ -421,6 +439,9 @@ namespace PepperDash.Core
         /// Deactivate the client
         /// </summary>
         /// <returns></returns>
+        /// <summary>
+        /// Deactivate method
+        /// </summary>
         public override bool Deactivate()
         {
             if (_client != null)
@@ -434,6 +455,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Connect Method. Will return if already connected. Will write errors if missing address, port, or unique key/name.
         /// </summary>
+        /// <summary>
+        /// Connect method
+        /// </summary>
         public void Connect()
         {
             ConnectionCount++;
@@ -565,6 +589,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+        /// <summary>
+        /// Disconnect method
+        /// </summary>
         public void Disconnect()
         {
             this.LogVerbose("Disconnect Called");
@@ -588,6 +615,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Does the actual disconnect business
         /// </summary>
+        /// <summary>
+        /// DisconnectClient method
+        /// </summary>
         public void DisconnectClient()
         {
             if (_client == null) return;
@@ -848,6 +878,9 @@ namespace PepperDash.Core
         /// <summary>
         /// General send method
         /// </summary>
+        /// <summary>
+        /// SendText method
+        /// </summary>
         public void SendText(string text)
         {
             if (!string.IsNullOrEmpty(text))
@@ -877,6 +910,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+        /// <summary>
+        /// SendBytes method
+        /// </summary>
         public void SendBytes(byte[] bytes)
         {
             if (bytes.Length > 0)

src/PepperDash.Core/Comm/GenericSecureTcpIpClient_ForServer.cs

@@ -82,6 +82,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Port on server
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Port
+        /// </summary>
         public int Port { get; set; }
 
         /// <summary>
@@ -115,6 +118,9 @@ namespace PepperDash.Core
         /// <summary>
         /// SharedKey is sent for varification to the server. Shared key can be any text (255 char limit in SIMPL+ Module), but must match the Shared Key on the Server module
         /// </summary>
+        /// <summary>
+        /// Gets or sets the SharedKey
+        /// </summary>
         public string SharedKey { get; set; }
 
         /// <summary>
@@ -125,6 +131,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Defaults to 2000
         /// </summary>
+        /// <summary>
+        /// Gets or sets the BufferSize
+        /// </summary>
         public int BufferSize { get; set; }
 
         /// <summary>
@@ -338,6 +347,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Just to help S+ set the key
         /// </summary>
+        /// <summary>
+        /// Initialize method
+        /// </summary>
         public void Initialize(string key)
         {
             Key = key;
@@ -397,6 +409,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Connect Method. Will return if already connected. Will write errors if missing address, port, or unique key/name.
         /// </summary>
+        /// <summary>
+        /// Connect method
+        /// </summary>
         public void Connect()
         {
             ConnectionCount++;
@@ -528,6 +543,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+        /// <summary>
+        /// Disconnect method
+        /// </summary>
         public void Disconnect()
         {
             this.LogVerbose("Disconnect Called");
@@ -806,6 +824,9 @@ namespace PepperDash.Core
         /// <summary>
         /// General send method
         /// </summary>
+        /// <summary>
+        /// SendText method
+        /// </summary>
         public void SendText(string text)
         {
             if (!string.IsNullOrEmpty(text))
@@ -835,6 +856,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+        /// <summary>
+        /// SendBytes method
+        /// </summary>
         public void SendBytes(byte[] bytes)
         {
             if (bytes.Length > 0)

src/PepperDash.Core/Comm/GenericSecureTcpIpServer.cs

@@ -60,6 +60,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+        /// <summary>
+        /// Delegate for ServerHasChokedCallbackDelegate
+        /// </summary>
         public delegate void ServerHasChokedCallbackDelegate();
 
         #endregion
@@ -106,6 +109,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 3 by default
         /// </summary>
+        /// <summary>
+        /// Gets or sets the MonitorClientMaxFailureCount
+        /// </summary>
         public int MonitorClientMaxFailureCount { get; set; }
 
         /// <summary>
@@ -192,6 +198,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Port Server should listen on
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Port
+        /// </summary>
         public int Port { get; set; }
 
         /// <summary>
@@ -226,6 +235,9 @@ namespace PepperDash.Core
         /// SharedKey is sent for varification to the server. Shared key can be any text (255 char limit in SIMPL+ Module), but must match the Shared Key on the Server module. 
         /// If SharedKey changes while server is listening or clients are connected, disconnect and stop listening will be called
         /// </summary>
+        /// <summary>
+        /// Gets or sets the SharedKey
+        /// </summary>
         public string SharedKey { get; set; }
 
         /// <summary>
@@ -250,6 +262,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Milliseconds before server expects another heartbeat. Set by property HeartbeatRequiredIntervalInSeconds which is driven from S+
         /// </summary>
+        /// <summary>
+        /// Gets or sets the HeartbeatRequiredIntervalMs
+        /// </summary>
         public int HeartbeatRequiredIntervalMs { get; set; }
 
         /// <summary>
@@ -260,6 +275,9 @@ namespace PepperDash.Core
         /// <summary>
         /// String to Match for heartbeat. If null or empty any string will reset heartbeat timer
         /// </summary>
+        /// <summary>
+        /// Gets or sets the HeartbeatStringToMatch
+        /// </summary>
         public string HeartbeatStringToMatch { get; set; }
 
         //private timers for Heartbeats per client
@@ -278,6 +296,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Defaults to 2000
         /// </summary>
+        /// <summary>
+        /// Gets or sets the BufferSize
+        /// </summary>
         public int BufferSize { get; set; }
 
         /// <summary>
@@ -341,6 +362,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Disconnects all clients and stops the server
         /// </summary>
+        /// <summary>
+        /// KillServer method
+        /// </summary>
         public void KillServer()
         {
             ServerStopped = true;
@@ -356,6 +380,9 @@ namespace PepperDash.Core
         /// Initialize Key for device using client name from SIMPL+. Called on Listen from SIMPL+
         /// </summary>
         /// <param name="key"></param>
+        /// <summary>
+        /// Initialize method
+        /// </summary>
         public void Initialize(string key)
         {
             Key = key;
@@ -397,6 +424,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Start listening on the specified port
         /// </summary>
+        /// <summary>
+        /// Listen method
+        /// </summary>
         public void Listen()
         {
             ServerCCSection.Enter();
@@ -455,6 +485,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Stop Listeneing
         /// </summary>
+        /// <summary>
+        /// StopListening method
+        /// </summary>
         public void StopListening()
         {
 			try
@@ -478,6 +511,9 @@ namespace PepperDash.Core
         /// Disconnects Client
         /// </summary>
         /// <param name="client"></param>
+        /// <summary>
+        /// DisconnectClient method
+        /// </summary>
         public void DisconnectClient(uint client)
         {
             try
@@ -493,6 +529,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Disconnect All Clients
         /// </summary>
+        /// <summary>
+        /// DisconnectAllClientsForShutdown method
+        /// </summary>
         public void DisconnectAllClientsForShutdown()
         {
             Debug.Console(1, this, Debug.ErrorLogLevel.Notice, "Disconnecting All Clients");
@@ -533,6 +572,9 @@ namespace PepperDash.Core
         /// Broadcast text from server to all connected clients
         /// </summary>
         /// <param name="text"></param>
+        /// <summary>
+        /// BroadcastText method
+        /// </summary>
         public void BroadcastText(string text)
         {
             CCriticalSection CCBroadcast = new CCriticalSection();
@@ -566,6 +608,9 @@ namespace PepperDash.Core
         /// </summary>
         /// <param name="text"></param>
         /// <param name="clientIndex"></param>
+        /// <summary>
+        /// SendTextToClient method
+        /// </summary>
         public void SendTextToClient(string text, uint clientIndex)
         {
             try
@@ -634,6 +679,9 @@ namespace PepperDash.Core
         /// </summary>
         /// <param name="clientIndex"></param>
         /// <returns></returns>
+        /// <summary>
+        /// GetClientIPAddress method
+        /// </summary>
         public string GetClientIPAddress(uint clientIndex)
         {
             Debug.Console(1, this, Debug.ErrorLogLevel.Notice, "GetClientIPAddress Index: {0}", clientIndex);

src/PepperDash.Core/Comm/GenericSshClient.cs

@@ -44,6 +44,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Address of server
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Hostname
+        /// </summary>
         public string Hostname { get; set; }
 
         /// <summary>
@@ -54,11 +57,17 @@ namespace PepperDash.Core
         /// <summary>
         /// Username for server
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Username
+        /// </summary>
         public string Username { get; set; }
 
         /// <summary>
         /// And... Password for server.  That was worth documenting!
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Password
+        /// </summary>
         public string Password { get; set; }
 
         /// <summary>
@@ -126,6 +135,9 @@ namespace PepperDash.Core
         /// Millisecond value, determines the timeout period in between reconnect attempts.
         /// Set to 5000 by default
         /// </summary>
+        /// <summary>
+        /// Gets or sets the AutoReconnectIntervalMs
+        /// </summary>
         public int AutoReconnectIntervalMs { get; set; }
 
         SshClient Client;
@@ -200,6 +212,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Connect to the server, using the provided properties.
         /// </summary>
+        /// <summary>
+        /// Connect method
+        /// </summary>
         public void Connect()
         {
             // Don't go unless everything is here
@@ -327,6 +342,9 @@ namespace PepperDash.Core
 		/// <summary>
 		/// Disconnect the clients and put away it's resources.
 		/// </summary>
+  /// <summary>
+  /// Disconnect method
+  /// </summary>
 		public void Disconnect()
 		{
 			ConnectEnabled = false;
@@ -476,6 +494,9 @@ namespace PepperDash.Core
 		/// Sends text to the server
 		/// </summary>
 		/// <param name="text"></param>
+  /// <summary>
+  /// SendText method
+  /// </summary>
 		public void SendText(string text)
 		{
 		    try
@@ -513,6 +534,9 @@ namespace PepperDash.Core
         /// Sends Bytes to the server
         /// </summary>
         /// <param name="bytes"></param>
+  /// <summary>
+  /// SendBytes method
+  /// </summary>
 		public void SendBytes(byte[] bytes)
 		{
             try
@@ -551,6 +575,9 @@ namespace PepperDash.Core
 /// <summary>
 /// Fired when connection changes
 /// </summary>
+/// <summary>
+/// Represents a SshConnectionChangeEventArgs
+/// </summary>
 public class SshConnectionChangeEventArgs : EventArgs
 	{
         /// <summary>
@@ -561,16 +588,25 @@ public class SshConnectionChangeEventArgs : EventArgs
         /// <summary>
         /// Connection Status represented as a ushort
         /// </summary>
+  /// <summary>
+  /// Gets or sets the UIsConnected
+  /// </summary>
 		public ushort UIsConnected { get { return (ushort)(Client.IsConnected ? 1 : 0); } }
 
         /// <summary>
         /// The client
         /// </summary>
+  /// <summary>
+  /// Gets or sets the Client
+  /// </summary>
 		public GenericSshClient Client { get; private set; }
 
         /// <summary>
         /// Socket Status as represented by
         /// </summary>
+  /// <summary>
+  /// Gets or sets the Status
+  /// </summary>
 		public ushort Status { get { return Client.UStatus; } }
 
         /// <summary>

src/PepperDash.Core/Comm/GenericTcpIpClient.cs

@@ -61,6 +61,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Port on server
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Port
+        /// </summary>
         public int Port { get; set; }
 
         /// <summary>
@@ -138,6 +141,9 @@ namespace PepperDash.Core
 		/// <summary>
 		/// bool to track if auto reconnect should be set on the socket
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the AutoReconnect
+  /// </summary>
 		public bool AutoReconnect { get; set; }
 
         /// <summary>
@@ -234,6 +240,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Just to help S+ set the key
         /// </summary>
+        /// <summary>
+        /// Initialize method
+        /// </summary>
         public void Initialize(string key)
         {
             Key = key;
@@ -255,6 +264,9 @@ namespace PepperDash.Core
         /// 
         /// </summary>
         /// <returns></returns>
+  /// <summary>
+  /// Deactivate method
+  /// </summary>
 		public override bool Deactivate()
 		{
             RetryTimer.Stop();
@@ -270,6 +282,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Attempts to connect to the server
         /// </summary>
+  /// <summary>
+  /// Connect method
+  /// </summary>
 		public void Connect()
 		{
             if (string.IsNullOrEmpty(Hostname))
@@ -337,6 +352,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Attempts to disconnect the client
         /// </summary>
+  /// <summary>
+  /// Disconnect method
+  /// </summary>
 		public void Disconnect()
 		{
             try
@@ -357,6 +375,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Does the actual disconnect business
         /// </summary>
+        /// <summary>
+        /// DisconnectClient method
+        /// </summary>
         public void DisconnectClient()
         {
             if (_client != null)
@@ -449,6 +470,9 @@ namespace PepperDash.Core
 		/// <summary>
 		/// General send method
 		/// </summary>
+  /// <summary>
+  /// SendText method
+  /// </summary>
 		public void SendText(string text)
 		{
 			var bytes = Encoding.GetEncoding(28591).GetBytes(text);
@@ -462,6 +486,9 @@ namespace PepperDash.Core
 		/// <summary>
 		/// This is useful from console and...?
 		/// </summary>
+  /// <summary>
+  /// SendEscapedText method
+  /// </summary>
 		public void SendEscapedText(string text)
 		{
 			var unescapedText = Regex.Replace(text, @"\\x([0-9a-fA-F][0-9a-fA-F])", s =>
@@ -476,6 +503,9 @@ namespace PepperDash.Core
         /// Sends Bytes to the server
         /// </summary>
         /// <param name="bytes"></param>
+  /// <summary>
+  /// SendBytes method
+  /// </summary>
 		public void SendBytes(byte[] bytes)
 		{
             if (StreamDebugging.TxStreamDebuggingIsEnabled)
@@ -511,6 +541,9 @@ namespace PepperDash.Core
     /// <summary>
     /// Configuration properties for TCP/SSH Connections
     /// </summary>
+ /// <summary>
+ /// Represents a TcpSshPropertiesConfig
+ /// </summary>
 	public class TcpSshPropertiesConfig
 	{
         /// <summary>
@@ -532,6 +565,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Passord credential
         /// </summary>
+  /// <summary>
+  /// Gets or sets the Password
+  /// </summary>
 		public string Password { get; set; }
 
 		/// <summary>
@@ -542,11 +578,17 @@ namespace PepperDash.Core
 		/// <summary>
 		/// Defaults to true
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the AutoReconnect
+  /// </summary>
 		public bool AutoReconnect { get; set; }
 
 		/// <summary>
 		/// Defaults to 5000ms
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the AutoReconnectIntervalMs
+  /// </summary>
 		public int AutoReconnectIntervalMs { get; set; }
 
         /// <summary>

src/PepperDash.Core/Comm/GenericTcpIpClient_ForServer.cs

@@ -71,6 +71,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Port on server
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Port
+        /// </summary>
         public int Port { get; set; }
 
         /// <summary>
@@ -104,6 +107,9 @@ namespace PepperDash.Core
         /// <summary>
         /// SharedKey is sent for varification to the server. Shared key can be any text (255 char limit in SIMPL+ Module), but must match the Shared Key on the Server module
         /// </summary>
+        /// <summary>
+        /// Gets or sets the SharedKey
+        /// </summary>
         public string SharedKey { get; set; }
 
         /// <summary>
@@ -114,6 +120,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Defaults to 2000
         /// </summary>
+        /// <summary>
+        /// Gets or sets the BufferSize
+        /// </summary>
         public int BufferSize { get; set; }
 
         /// <summary>
@@ -291,6 +300,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Just to help S+ set the key
         /// </summary>
+        /// <summary>
+        /// Initialize method
+        /// </summary>
         public void Initialize(string key)
         {
             Key = key;
@@ -313,6 +325,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Connect Method. Will return if already connected. Will write errors if missing address, port, or unique key/name.
         /// </summary>
+        /// <summary>
+        /// Connect method
+        /// </summary>
         public void Connect()
         {
             ConnectionCount++;
@@ -444,6 +459,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+        /// <summary>
+        /// Disconnect method
+        /// </summary>
         public void Disconnect()
         {
             this.LogVerbose("Disconnect Called");
@@ -671,6 +689,9 @@ namespace PepperDash.Core
         /// <summary>
         /// General send method
         /// </summary>
+        /// <summary>
+        /// SendText method
+        /// </summary>
         public void SendText(string text)
         {
             if (!string.IsNullOrEmpty(text))
@@ -700,6 +721,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+        /// <summary>
+        /// SendBytes method
+        /// </summary>
         public void SendBytes(byte[] bytes)
         {
             if (bytes.Length > 0)

src/PepperDash.Core/Comm/GenericTcpIpServer.cs

@@ -54,6 +54,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+        /// <summary>
+        /// Delegate for ServerHasChokedCallbackDelegate
+        /// </summary>
         public delegate void ServerHasChokedCallbackDelegate();
 
         #endregion
@@ -84,6 +87,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 3 by default
         /// </summary>
+        /// <summary>
+        /// Gets or sets the MonitorClientMaxFailureCount
+        /// </summary>
         public int MonitorClientMaxFailureCount { get; set; }
 
         /// <summary>
@@ -173,6 +179,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Port Server should listen on
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Port
+        /// </summary>
         public int Port { get; set; }
 
         /// <summary>
@@ -207,6 +216,9 @@ namespace PepperDash.Core
         /// SharedKey is sent for varification to the server. Shared key can be any text (255 char limit in SIMPL+ Module), but must match the Shared Key on the Server module. 
         /// If SharedKey changes while server is listening or clients are connected, disconnect and stop listening will be called
         /// </summary>
+        /// <summary>
+        /// Gets or sets the SharedKey
+        /// </summary>
         public string SharedKey { get; set; }
 
         /// <summary>
@@ -231,6 +243,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Milliseconds before server expects another heartbeat. Set by property HeartbeatRequiredIntervalInSeconds which is driven from S+
         /// </summary>
+        /// <summary>
+        /// Gets or sets the HeartbeatRequiredIntervalMs
+        /// </summary>
         public int HeartbeatRequiredIntervalMs { get; set; }
 
         /// <summary>
@@ -241,6 +256,9 @@ namespace PepperDash.Core
         /// <summary>
         /// String to Match for heartbeat. If null or empty any string will reset heartbeat timer
         /// </summary>
+        /// <summary>
+        /// Gets or sets the HeartbeatStringToMatch
+        /// </summary>
         public string HeartbeatStringToMatch { get; set; }
 
         //private timers for Heartbeats per client
@@ -259,6 +277,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Defaults to 2000
         /// </summary>
+        /// <summary>
+        /// Gets or sets the BufferSize
+        /// </summary>
         public int BufferSize { get; set; }
 
         /// <summary>
@@ -322,6 +343,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Disconnects all clients and stops the server
         /// </summary>
+        /// <summary>
+        /// KillServer method
+        /// </summary>
         public void KillServer()
         {
             ServerStopped = true;
@@ -337,6 +361,9 @@ namespace PepperDash.Core
         /// Initialize Key for device using client name from SIMPL+. Called on Listen from SIMPL+
         /// </summary>
         /// <param name="key"></param>
+        /// <summary>
+        /// Initialize method
+        /// </summary>
         public void Initialize(string key)
         {
             Key = key;
@@ -377,6 +404,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Start listening on the specified port
         /// </summary>
+        /// <summary>
+        /// Listen method
+        /// </summary>
         public void Listen()
         {
             ServerCCSection.Enter();
@@ -434,6 +464,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Stop Listening
         /// </summary>
+        /// <summary>
+        /// StopListening method
+        /// </summary>
         public void StopListening()
         {
             try
@@ -457,6 +490,9 @@ namespace PepperDash.Core
         /// Disconnects Client
         /// </summary>
         /// <param name="client"></param>
+        /// <summary>
+        /// DisconnectClient method
+        /// </summary>
         public void DisconnectClient(uint client)
         {
             try
@@ -472,6 +508,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Disconnect All Clients
         /// </summary>
+        /// <summary>
+        /// DisconnectAllClientsForShutdown method
+        /// </summary>
         public void DisconnectAllClientsForShutdown()
         {
             Debug.Console(1, this, Debug.ErrorLogLevel.Notice, "Disconnecting All Clients");
@@ -512,6 +551,9 @@ namespace PepperDash.Core
         /// Broadcast text from server to all connected clients
         /// </summary>
         /// <param name="text"></param>
+        /// <summary>
+        /// BroadcastText method
+        /// </summary>
         public void BroadcastText(string text)
         {
             CCriticalSection CCBroadcast = new CCriticalSection();
@@ -545,6 +587,9 @@ namespace PepperDash.Core
         /// </summary>
         /// <param name="text"></param>
         /// <param name="clientIndex"></param>
+        /// <summary>
+        /// SendTextToClient method
+        /// </summary>
         public void SendTextToClient(string text, uint clientIndex)
         {
             try
@@ -613,6 +658,9 @@ namespace PepperDash.Core
         /// </summary>
         /// <param name="clientIndex"></param>
         /// <returns>IP address of the client</returns>
+        /// <summary>
+        /// GetClientIPAddress method
+        /// </summary>
         public string GetClientIPAddress(uint clientIndex)
         {
             Debug.Console(1, this, Debug.ErrorLogLevel.Notice, "GetClientIPAddress Index: {0}", clientIndex);

src/PepperDash.Core/Comm/GenericUdpServer.cs

@@ -150,6 +150,9 @@ namespace PepperDash.Core
         /// <param name="key"></param>
         /// <param name="address"></param>
         /// <param name="port"></param>
+        /// <summary>
+        /// Initialize method
+        /// </summary>
         public void Initialize(string key, string address, ushort port)
         {
             Key = key;
@@ -187,6 +190,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Enables the UDP Server
         /// </summary>
+        /// <summary>
+        /// Connect method
+        /// </summary>
         public void Connect()
         {
             if (Server == null)
@@ -224,6 +230,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Disabled the UDP Server
         /// </summary>
+        /// <summary>
+        /// Disconnect method
+        /// </summary>
         public void Disconnect()
         {
             if(Server != null)
@@ -292,6 +301,9 @@ namespace PepperDash.Core
         /// General send method
         /// </summary>
         /// <param name="text"></param>
+        /// <summary>
+        /// SendText method
+        /// </summary>
         public void SendText(string text)
         {
             var bytes = Encoding.GetEncoding(28591).GetBytes(text);
@@ -309,6 +321,9 @@ namespace PepperDash.Core
         /// 
         /// </summary>
         /// <param name="bytes"></param>
+        /// <summary>
+        /// SendBytes method
+        /// </summary>
         public void SendBytes(byte[] bytes)
         {
             if (StreamDebugging.TxStreamDebuggingIsEnabled)
@@ -323,6 +338,9 @@ namespace PepperDash.Core
     /// <summary>
     /// 
     /// </summary>
+ /// <summary>
+ /// Represents a GenericUdpReceiveTextExtraArgs
+ /// </summary>
 	public class GenericUdpReceiveTextExtraArgs : EventArgs
 	{
         /// <summary>

src/PepperDash.Core/Comm/TcpClientConfigObject.cs

@@ -5,6 +5,9 @@ namespace PepperDash.Core
     /// <summary>
     /// Client config object for TCP client with server that inherits from TcpSshPropertiesConfig and adds properties for shared key and heartbeat
     /// </summary>
+    /// <summary>
+    /// Represents a TcpClientConfigObject
+    /// </summary>
     public class TcpClientConfigObject
     {
         /// <summary>

src/PepperDash.Core/CommunicationExtras.cs

@@ -41,6 +41,9 @@ namespace PepperDash.Core
 	/// <summary>
 	/// Represents a device that uses basic connection
 	/// </summary>
+    /// <summary>
+    /// Defines the contract for IBasicCommunication
+    /// </summary>
     public interface IBasicCommunication : ICommunicationReceiver
 	{
         /// <summary>
@@ -150,6 +153,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the Bytes
+  /// </summary>
 		public byte[] Bytes { get; private set; }
 
         /// <summary>
@@ -228,6 +234,9 @@ namespace PepperDash.Core
         /// </summary>
         /// <param name="text"></param>
         /// <returns></returns>
+  /// <summary>
+  /// GetEscapedText method
+  /// </summary>
 		public static string GetEscapedText(string text)
 		{
 			var bytes = Encoding.GetEncoding(28591).GetBytes(text);
@@ -239,6 +248,9 @@ namespace PepperDash.Core
         /// </summary>
         /// <param name="text"></param>
         /// <returns></returns>
+        /// <summary>
+        /// GetDebugText method
+        /// </summary>
         public static string GetDebugText(string text)
         {
             return Regex.Replace(text, @"[^\u0020-\u007E]", a => GetEscapedText(a.Value));

src/PepperDash.Core/Config/PortalConfigReader.cs

@@ -67,6 +67,9 @@ namespace PepperDash.Core.Config
 		/// </summary>
 		/// <param name="doubleConfig"></param>
 		/// <returns></returns>
+  /// <summary>
+  /// MergeConfigs method
+  /// </summary>
 		public static JObject MergeConfigs(JObject doubleConfig)
 		{
 			var system = JObject.FromObject(doubleConfig["system"]);

src/PepperDash.Core/Conversion/Convert.cs

@@ -6,13 +6,22 @@ using Crestron.SimplSharp;
 
 namespace PepperDash.Core
 {
+    /// <summary>
+    /// Represents a EncodingHelper
+    /// </summary>
     public class EncodingHelper
     {
+        /// <summary>
+        /// ConvertUtf8ToAscii method
+        /// </summary>
         public static string ConvertUtf8ToAscii(string utf8String)
         {
             return Encoding.ASCII.GetString(Encoding.UTF8.GetBytes(utf8String), 0, utf8String.Length);
         }
 
+        /// <summary>
+        /// ConvertUtf8ToUtf16 method
+        /// </summary>
         public static string ConvertUtf8ToUtf16(string utf8String)
         {
             return Encoding.Unicode.GetString(Encoding.UTF8.GetBytes(utf8String), 0, utf8String.Length);

src/PepperDash.Core/Device.cs

@@ -8,6 +8,9 @@ namespace PepperDash.Core
 	/// <summary>
 	/// The core event and status-bearing class that most if not all device and connectors can derive from.
 	/// </summary>
+ /// <summary>
+ /// Represents a Device
+ /// </summary>
 	public class Device : IKeyName
 	{
 
@@ -18,6 +21,9 @@ namespace PepperDash.Core
 		/// <summary>
 		/// Name of the devie
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the Name
+  /// </summary>
 		public string Name { get; protected set; }
 		/// <summary>
 		/// 
@@ -86,6 +92,9 @@ namespace PepperDash.Core
 		/// Adds a post activation action
 		/// </summary>
 		/// <param name="act"></param>
+  /// <summary>
+  /// AddPostActivationAction method
+  /// </summary>
 		public void AddPostActivationAction(Action act)
 		{
 			if (_PostActivationActions == null)
@@ -96,6 +105,9 @@ namespace PepperDash.Core
 		/// <summary>
 		/// Executes the preactivation actions
 		/// </summary>
+  /// <summary>
+  /// PreActivate method
+  /// </summary>
 		public void PreActivate()
 		{
 			if (_PreActivationActions != null)
@@ -117,6 +129,9 @@ namespace PepperDash.Core
 		/// all post-activation at end. Classes needing additional logic to 
 		/// run should override CustomActivate()
 		/// </summary>
+  /// <summary>
+  /// Activate method
+  /// </summary>
 		public bool Activate()
 		{
 			//if (_PreActivationActions != null)
@@ -130,6 +145,9 @@ namespace PepperDash.Core
 		/// <summary>
 		/// Executes the postactivation actions
 		/// </summary>
+  /// <summary>
+  /// PostActivate method
+  /// </summary>
 		public void PostActivate()
 		{
 			if (_PostActivationActions != null)
@@ -152,6 +170,9 @@ namespace PepperDash.Core
 		/// do not need to call base.CustomActivate()
 		/// </summary>
 		/// <returns>true if device activated successfully.</returns>
+  /// <summary>
+  /// CustomActivate method
+  /// </summary>
 		public virtual bool CustomActivate() { return true; }
 
 		/// <summary>
@@ -184,6 +205,9 @@ namespace PepperDash.Core
         /// <remarks>The returned string is formatted as "{Key} - {Name}". If the <c>Name</c> property is
         /// null or empty,  "---" is used in place of the name.</remarks>
         /// <returns>A string that represents the object, containing the key and name in the format "{Key} - {Name}".</returns>
+  /// <summary>
+  /// ToString method
+  /// </summary>
 		public override string ToString()
 		{
 			return string.Format("{0} - {1}", Key, string.IsNullOrEmpty(Name) ? "---" : Name);

src/PepperDash.Core/EthernetHelper.cs

@@ -7,6 +7,9 @@ namespace PepperDash.Core
     /// <summary>
     /// Class to help with accessing values from the CrestronEthernetHelper class 
     /// </summary>
+ /// <summary>
+ /// Represents a EthernetHelper
+ /// </summary>
 	public class EthernetHelper
 	{
 		/// <summary>
@@ -27,6 +30,9 @@ namespace PepperDash.Core
 		/// <summary>
 		/// 
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the PortNumber
+  /// </summary>
 		public int PortNumber { get; private set; }
 
 		private EthernetHelper(int portNumber)

src/PepperDash.Core/EventArgs.cs

@@ -19,16 +19,25 @@ namespace PepperDash.Core
 		/// <summary>
 		/// Boolean ushort value property
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the IntValue
+  /// </summary>
 		public ushort IntValue { get { return (ushort)(State ? 1 : 0); } }
 		
 		/// <summary>
 		/// Boolean change event args type
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the Type
+  /// </summary>
 		public ushort Type { get; set; }
 		
 		/// <summary>
 		/// Boolean change event args index
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the Index
+  /// </summary>
 		public ushort Index { get; set; }
 		
 		/// <summary>
@@ -67,6 +76,9 @@ namespace PepperDash.Core
 	/// <summary>
 	/// Ushort change event args
 	/// </summary>
+ /// <summary>
+ /// Represents a UshrtChangeEventArgs
+ /// </summary>
 	public class UshrtChangeEventArgs : EventArgs
 	{
 		/// <summary>
@@ -77,11 +89,17 @@ namespace PepperDash.Core
 		/// <summary>
 		/// Ushort change event args type
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the Type
+  /// </summary>
 		public ushort Type { get; set; }
 		
 		/// <summary>
 		/// Ushort change event args index
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the Index
+  /// </summary>
 		public ushort Index { get; set; }
 		
 		/// <summary>
@@ -120,6 +138,9 @@ namespace PepperDash.Core
 	/// <summary>
 	/// String change event args
 	/// </summary>
+ /// <summary>
+ /// Represents a StringChangeEventArgs
+ /// </summary>
 	public class StringChangeEventArgs : EventArgs
 	{
 		/// <summary>
@@ -130,11 +151,17 @@ namespace PepperDash.Core
 		/// <summary>
 		/// String change event args type
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the Type
+  /// </summary>
 		public ushort Type { get; set; }
 
 		/// <summary>
 		/// string change event args index
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the Index
+  /// </summary>
 		public ushort Index { get; set; }
 
 		/// <summary>

src/PepperDash.Core/JsonStandardObjects/EventArgs and Constants.cs

@@ -35,11 +35,17 @@ namespace PepperDash.Core.JsonStandardObjects
 		/// <summary>
 		/// Device change event args type
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the Type
+  /// </summary>
 		public ushort Type { get; set; }
 
 		/// <summary>
 		/// Device change event args index
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the Index
+  /// </summary>
 		public ushort Index { get; set; }
 
 		/// <summary>

src/PepperDash.Core/JsonStandardObjects/JsonToSimplDevice.cs

@@ -58,6 +58,9 @@ namespace PepperDash.Core.JsonStandardObjects
 		/// </summary>
 		/// <param name="uniqueID"></param>
 		/// <param name="deviceKey"></param>
+  /// <summary>
+  /// Initialize method
+  /// </summary>
 		public void Initialize(string uniqueID, string deviceKey)
 		{
 			// S+ set EvaluateFb low

src/PepperDash.Core/JsonStandardObjects/JsonToSimplDeviceConfig.cs

@@ -50,11 +50,17 @@ namespace PepperDash.Core.JsonStandardObjects
 	/// <summary>
 	/// Device communication parameter class
 	/// </summary>
+ /// <summary>
+ /// Represents a ComParamsConfig
+ /// </summary>
 	public class ComParamsConfig
 	{
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the baudRate
+  /// </summary>
 		public int baudRate { get; set; }
         /// <summary>
         /// 
@@ -89,10 +95,16 @@ namespace PepperDash.Core.JsonStandardObjects
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the simplBaudRate
+  /// </summary>
 		public ushort simplBaudRate { get { return Convert.ToUInt16(baudRate); } }
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the simplDataBits
+  /// </summary>
 		public ushort simplDataBits { get { return Convert.ToUInt16(dataBits); } }
         /// <summary>
         /// 
@@ -146,10 +158,16 @@ namespace PepperDash.Core.JsonStandardObjects
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the simplPort
+  /// </summary>
 		public ushort simplPort { get { return Convert.ToUInt16(port); } }
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the simplAutoReconnect
+  /// </summary>
 		public ushort simplAutoReconnect { get { return (ushort)(autoReconnect ? 1 : 0); } }
         /// <summary>
         /// 
@@ -195,6 +213,9 @@ namespace PepperDash.Core.JsonStandardObjects
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the simplControlPortNumber
+  /// </summary>
 		public ushort simplControlPortNumber { get { return Convert.ToUInt16(controlPortNumber); } }
 
 		/// <summary>
@@ -210,6 +231,9 @@ namespace PepperDash.Core.JsonStandardObjects
 	/// <summary>
 	/// Device properties class
 	/// </summary>
+ /// <summary>
+ /// Represents a PropertiesConfig
+ /// </summary>
 	public class PropertiesConfig
 	{
         /// <summary>
@@ -229,10 +253,16 @@ namespace PepperDash.Core.JsonStandardObjects
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the simplDeviceId
+  /// </summary>
 		public ushort simplDeviceId { get { return Convert.ToUInt16(deviceId); } }
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the simplEnabled
+  /// </summary>
 		public ushort simplEnabled { get { return (ushort)(enabled ? 1 : 0); } }
 
 		/// <summary>

src/PepperDash.Core/JsonToSimpl/Constants.cs

@@ -91,6 +91,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the ValueType
+  /// </summary>
 		public SPlusType ValueType { get; private set; }
         /// <summary>
         /// 
@@ -125,6 +128,9 @@ namespace PepperDash.Core.JsonToSimpl
 	/// <summary>
 	/// S+ types enum
 	/// </summary>
+ /// <summary>
+ /// Enumeration of SPlusType values
+ /// </summary>
 	public enum SPlusType
 	{
         /// <summary>

src/PepperDash.Core/JsonToSimpl/Global.cs

@@ -23,6 +23,9 @@ namespace PepperDash.Core.JsonToSimpl
 		/// </summary>
 		/// <param name="master">New master to add</param>
         /// 
+  /// <summary>
+  /// AddMaster method
+  /// </summary>
 		public static void AddMaster(JsonToSimplMaster master)
 		{
 			if (master == null)
@@ -52,6 +55,9 @@ namespace PepperDash.Core.JsonToSimpl
 		/// <summary>
 		/// Gets a master by its key.  Case-insensitive
 		/// </summary>
+  /// <summary>
+  /// GetMasterByFile method
+  /// </summary>
 		public static JsonToSimplMaster GetMasterByFile(string file)
 		{
 			return Masters.FirstOrDefault(m => m.UniqueID.Equals(file, StringComparison.OrdinalIgnoreCase));

src/PepperDash.Core/JsonToSimpl/JsonToSimplArrayLookupChild.cs

@@ -8,6 +8,9 @@ namespace PepperDash.Core.JsonToSimpl
     /// <summary>
     /// Used to interact with an array of values with the S+ modules
     /// </summary>
+ /// <summary>
+ /// Represents a JsonToSimplArrayLookupChild
+ /// </summary>
 	public class JsonToSimplArrayLookupChild : JsonToSimplChildObjectBase
 	{
         /// <summary>
@@ -79,6 +82,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// <summary>
         /// Process all values
         /// </summary>
+  /// <summary>
+  /// ProcessAll method
+  /// </summary>
 		public override void ProcessAll()
 		{
 			if (FindInArray())

src/PepperDash.Core/JsonToSimpl/JsonToSimplChildObjectBase.cs

@@ -31,11 +31,17 @@ namespace PepperDash.Core.JsonToSimpl
 		/// <summary>
 		/// Use a callback to reduce task switch/threading
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the SetAllPathsDelegate
+  /// </summary>
 		public SPlusValuesDelegate SetAllPathsDelegate { get; set; }
 
         /// <summary>
         /// Unique identifier for instance
         /// </summary>
+  /// <summary>
+  /// Gets or sets the Key
+  /// </summary>
 		public string Key { get; protected set; }
 
 		/// <summary>
@@ -52,6 +58,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// <summary>
         /// Indicates if the instance is linked to an object
         /// </summary>
+  /// <summary>
+  /// Gets or sets the LinkedToObject
+  /// </summary>
 		public bool LinkedToObject { get; protected set; }
 
         /// <summary>
@@ -96,6 +105,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// Sets the path prefix for the object
         /// </summary>
         /// <param name="pathPrefix"></param>
+  /// <summary>
+  /// SetPathPrefix method
+  /// </summary>
 		public void SetPathPrefix(string pathPrefix)
 		{
 			PathPrefix = pathPrefix;
@@ -113,6 +125,9 @@ namespace PepperDash.Core.JsonToSimpl
 		/// <summary>
 		/// Set the JPath for a ushort out index.
 		/// </summary>
+  /// <summary>
+  /// SetUshortPath method
+  /// </summary>
 		public void SetUshortPath(ushort index, string path)
 		{
 			Debug.Console(1, "JSON Child[{0}] SetUshortPath {1}={2}", Key, index, path);
@@ -123,6 +138,9 @@ namespace PepperDash.Core.JsonToSimpl
 		/// <summary>
 		/// Set the JPath for a string output index. 
 		/// </summary>
+  /// <summary>
+  /// SetStringPath method
+  /// </summary>
 		public void SetStringPath(ushort index, string path)
 		{
 			Debug.Console(1, "JSON Child[{0}] SetStringPath {1}={2}", Key, index, path);
@@ -134,6 +152,9 @@ namespace PepperDash.Core.JsonToSimpl
 		/// Evalutates all outputs with defined paths. called by S+ when paths are ready to process
 		/// and by Master when file is read.
 		/// </summary>
+  /// <summary>
+  /// ProcessAll method
+  /// </summary>
 		public virtual void ProcessAll()
 		{
 			if (!LinkedToObject)
@@ -277,6 +298,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// </summary>
         /// <param name="key"></param>
         /// <param name="theValue"></param>
+  /// <summary>
+  /// USetBoolValue method
+  /// </summary>
 		public void USetBoolValue(ushort key, ushort theValue)
 		{
 			SetBoolValue(key, theValue == 1);
@@ -287,6 +311,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// </summary>
         /// <param name="key"></param>
         /// <param name="theValue"></param>
+  /// <summary>
+  /// SetBoolValue method
+  /// </summary>
 		public void SetBoolValue(ushort key, bool theValue)
 		{
 			if (BoolPaths.ContainsKey(key))
@@ -298,6 +325,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// </summary>
         /// <param name="key"></param>
         /// <param name="theValue"></param>
+  /// <summary>
+  /// SetUShortValue method
+  /// </summary>
 		public void SetUShortValue(ushort key, ushort theValue)
 		{
 			if (UshortPaths.ContainsKey(key))
@@ -309,6 +339,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// </summary>
         /// <param name="key"></param>
         /// <param name="theValue"></param>
+  /// <summary>
+  /// SetStringValue method
+  /// </summary>
 		public void SetStringValue(ushort key, string theValue)
 		{
 			if (StringPaths.ContainsKey(key))
@@ -320,6 +353,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// </summary>
         /// <param name="keyPath"></param>
         /// <param name="valueToSave"></param>
+  /// <summary>
+  /// SetValueOnMaster method
+  /// </summary>
 		public void SetValueOnMaster(string keyPath, JValue valueToSave)
 		{
 			var path = GetFullPath(keyPath);

src/PepperDash.Core/JsonToSimpl/JsonToSimplFileMaster.cs

@@ -22,11 +22,17 @@ namespace PepperDash.Core.JsonToSimpl
         /// <summary>
         /// Filepath to the actual file that will be read (Portal or local)
         /// </summary>
+        /// <summary>
+        /// Gets or sets the ActualFilePath
+        /// </summary>
         public string ActualFilePath { get; private set; }
 
         /// <summary>
         /// 
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Filename
+        /// </summary>
         public string Filename { get; private set; }
         /// <summary>
         /// 
@@ -194,6 +200,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// Sets the debug level
         /// </summary>
         /// <param name="level"></param>
+        /// <summary>
+        /// setDebugLevel method
+        /// </summary>
         public void setDebugLevel(uint level)
         {
             Debug.SetDebugLevel(level);

src/PepperDash.Core/JsonToSimpl/JsonToSimplFixedPathObject.cs

@@ -5,6 +5,9 @@ namespace PepperDash.Core.JsonToSimpl
     /// <summary>
     /// 
     /// </summary>
+ /// <summary>
+ /// Represents a JsonToSimplFixedPathObject
+ /// </summary>
 	public class JsonToSimplFixedPathObject : JsonToSimplChildObjectBase
 	{
         /// <summary>

src/PepperDash.Core/JsonToSimpl/JsonToSimplGenericMaster.cs

@@ -8,6 +8,9 @@ namespace PepperDash.Core.JsonToSimpl
     /// <summary>
     /// Generic Master
     /// </summary>
+ /// <summary>
+ /// Represents a JsonToSimplGenericMaster
+ /// </summary>
 	public class JsonToSimplGenericMaster : JsonToSimplMaster
     {
 		/*****************************************************************************************/
@@ -23,6 +26,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// <summary>
         /// Callback action for saving
         /// </summary>
+  /// <summary>
+  /// Gets or sets the SaveCallback
+  /// </summary>
 		public Action<string> SaveCallback { get; set; }
 
 		/*****************************************************************************************/
@@ -60,6 +66,9 @@ namespace PepperDash.Core.JsonToSimpl
 		/// Loads JSON into JsonObject, but does not trigger evaluation by children
 		/// </summary>
 		/// <param name="json"></param>
+  /// <summary>
+  /// SetJsonWithoutEvaluating method
+  /// </summary>
 		public void SetJsonWithoutEvaluating(string json)
 		{
 			try
@@ -75,6 +84,9 @@ namespace PepperDash.Core.JsonToSimpl
 		/// <summary>
 		/// 
 		/// </summary>
+  /// <summary>
+  /// Save method
+  /// </summary>
 		public override void Save()
 		{
 			// this code is duplicated in the other masters!!!!!!!!!!!!!

src/PepperDash.Core/JsonToSimpl/JsonToSimplMaster.cs

@@ -41,6 +41,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// <summary>
         /// A unique ID
         /// </summary>
+  /// <summary>
+  /// Gets or sets the UniqueID
+  /// </summary>
 		public string UniqueID { get; protected set; }
 
 		/// <summary>
@@ -57,6 +60,9 @@ namespace PepperDash.Core.JsonToSimpl
 		/// This will be prepended to all paths to allow path swapping or for more organized
 		/// sub-paths
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the PathPrefix
+  /// </summary>
 		public string PathPrefix { get; set; }
 
 		/// <summary>
@@ -86,6 +92,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the JsonObject
+  /// </summary>
 		public JObject JsonObject { get; protected set; }
 
 		/*****************************************************************************************/
@@ -120,6 +129,9 @@ namespace PepperDash.Core.JsonToSimpl
 		/// Adds a child "module" to this master
 		/// </summary>
 		/// <param name="child"></param>
+  /// <summary>
+  /// AddChild method
+  /// </summary>
 		public void AddChild(JsonToSimplChildObjectBase child)
 		{
 			if (!Children.Contains(child))
@@ -131,6 +143,9 @@ namespace PepperDash.Core.JsonToSimpl
 		/// <summary>
 		/// Called from the child to add changed or new values for saving
 		/// </summary>
+  /// <summary>
+  /// AddUnsavedValue method
+  /// </summary>
 		public void AddUnsavedValue(string path, JValue value)
 		{
 			if (UnsavedValues.ContainsKey(path))
@@ -179,6 +194,9 @@ namespace PepperDash.Core.JsonToSimpl
             /// </summary>
             /// <param name="json"></param>
             /// <returns></returns>
+   /// <summary>
+   /// ParseArray method
+   /// </summary>
 			public static JArray ParseArray(string json)
 			{
 				#if NET6_0

src/PepperDash.Core/JsonToSimpl/JsonToSimplPortalFileMaster.cs

@@ -21,6 +21,9 @@ namespace PepperDash.Core.JsonToSimpl
         /// <summary>
         /// File path of the actual file being read (Portal or local)
         /// </summary>
+        /// <summary>
+        /// Gets or sets the ActualFilePath
+        /// </summary>
         public string ActualFilePath { get; private set; }
 
 		/*****************************************************************************************/
@@ -128,6 +131,9 @@ namespace PepperDash.Core.JsonToSimpl
 		/// 
 		/// </summary>
 		/// <param name="level"></param>
+  /// <summary>
+  /// setDebugLevel method
+  /// </summary>
 		public void setDebugLevel(uint level)
 		{
 			Debug.SetDebugLevel(level);

src/PepperDash.Core/Logging/CrestronEnricher.cs

@@ -9,6 +9,9 @@ using System.Threading.Tasks;
 
 namespace PepperDash.Core.Logging
 {
+    /// <summary>
+    /// Represents a CrestronEnricher
+    /// </summary>
     public class CrestronEnricher : ILogEventEnricher
     {
         static readonly string _appName;
@@ -27,6 +30,9 @@ namespace PepperDash.Core.Logging
         }
             
 
+        /// <summary>
+        /// Enrich method
+        /// </summary>
         public void Enrich(LogEvent logEvent, ILogEventPropertyFactory propertyFactory)
         {
             var property = propertyFactory.CreateProperty("App", _appName);

src/PepperDash.Core/Logging/Debug.cs

@@ -80,11 +80,17 @@ namespace PepperDash.Core
         /// <summary>
         /// Debug level to set for a given program.
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Level
+        /// </summary>
         public static int Level { get; private set; }
 
         /// <summary>
         /// When this is true, the configuration file will NOT be loaded until triggered by either a console command or a signal
         /// </summary>
+        /// <summary>
+        /// Gets or sets the DoNotLoadConfigOnNextBoot
+        /// </summary>
         public static bool DoNotLoadConfigOnNextBoot { get; private set; }
 
         private static DebugContextCollection _contexts;
@@ -96,6 +102,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Version for the currently loaded PepperDashCore dll
         /// </summary>
+        /// <summary>
+        /// Gets or sets the PepperDashCoreVersion
+        /// </summary>
         public static string PepperDashCoreVersion { get; private set; } 
 
         private static CTimer _saveTimer;
@@ -233,6 +242,9 @@ namespace PepperDash.Core
             };
         }
 
+        /// <summary>
+        /// UpdateLoggerConfiguration method
+        /// </summary>
         public static void UpdateLoggerConfiguration(LoggerConfiguration config)
         {
             _loggerConfiguration = config;
@@ -240,6 +252,9 @@ namespace PepperDash.Core
             _logger = config.CreateLogger();
         }
 
+        /// <summary>
+        /// ResetLoggerConfiguration method
+        /// </summary>
         public static void ResetLoggerConfiguration()
         {
             _loggerConfiguration = _defaultLoggerConfiguration;
@@ -319,6 +334,9 @@ namespace PepperDash.Core
         /// Callback for console command
         /// </summary>
         /// <param name="levelString"></param>
+        /// <summary>
+        /// SetDebugFromConsole method
+        /// </summary>
         public static void SetDebugFromConsole(string levelString)
         {
             try
@@ -371,6 +389,9 @@ namespace PepperDash.Core
         /// Sets the debug level
         /// </summary>
         /// <param name="level"> Valid values 0-5</param>
+        /// <summary>
+        /// SetDebugLevel method
+        /// </summary>
         public static void SetDebugLevel(uint level)
         {
             if(!_logLevels.TryGetValue(level, out var logLevel))
@@ -385,6 +406,9 @@ namespace PepperDash.Core
             SetDebugLevel(logLevel);
         }
 
+        /// <summary>
+        /// SetDebugLevel method
+        /// </summary>
         public static void SetDebugLevel(LogEventLevel level)
         {
             _consoleLoggingLevelSwitch.MinimumLevel = level;
@@ -402,6 +426,9 @@ namespace PepperDash.Core
                 CrestronConsole.PrintLine($"Error saving console debug level setting: {err}");
         }
 
+        /// <summary>
+        /// SetWebSocketMinimumDebugLevel method
+        /// </summary>
         public static void SetWebSocketMinimumDebugLevel(LogEventLevel level)
         {
             _websocketLoggingLevelSwitch.MinimumLevel = level;            
@@ -414,6 +441,9 @@ namespace PepperDash.Core
             LogMessage(LogEventLevel.Information, "Websocket debug level set to {0}", _websocketLoggingLevelSwitch.MinimumLevel);
         }
 
+        /// <summary>
+        /// SetErrorLogMinimumDebugLevel method
+        /// </summary>
         public static void SetErrorLogMinimumDebugLevel(LogEventLevel level)
         {
             _errorLogLevelSwitch.MinimumLevel = level;
@@ -426,6 +456,9 @@ namespace PepperDash.Core
             LogMessage(LogEventLevel.Information, "Error log debug level set to {0}", _websocketLoggingLevelSwitch.MinimumLevel);
         }
 
+        /// <summary>
+        /// SetFileMinimumDebugLevel method
+        /// </summary>
         public static void SetFileMinimumDebugLevel(LogEventLevel level)
         {
             _errorLogLevelSwitch.MinimumLevel = level;
@@ -442,6 +475,9 @@ namespace PepperDash.Core
         /// Callback for console command
         /// </summary>
         /// <param name="stateString"></param>
+        /// <summary>
+        /// SetDoNotLoadOnNextBootFromConsole method
+        /// </summary>
         public static void SetDoNotLoadOnNextBootFromConsole(string stateString)
         {
             try
@@ -464,6 +500,9 @@ namespace PepperDash.Core
         /// Callback for console command
         /// </summary>
         /// <param name="items"></param>
+  /// <summary>
+  /// SetDebugFilterFromConsole method
+  /// </summary>
 		public static void SetDebugFilterFromConsole(string items)
 		{
 			var str = items.Trim();
@@ -559,6 +598,9 @@ namespace PepperDash.Core
         /// </summary>
         /// <param name="deviceKey"></param>
         /// <returns></returns>
+        /// <summary>
+        /// GetDeviceDebugSettingsForKey method
+        /// </summary>
         public static object GetDeviceDebugSettingsForKey(string deviceKey)
         {
             return _contexts.GetDebugSettingsForKey(deviceKey);
@@ -581,6 +623,9 @@ namespace PepperDash.Core
         /// <summary>
         /// 
         /// </summary>
+        /// <summary>
+        /// ShowDebugLog method
+        /// </summary>
         public static void ShowDebugLog(string s)
         {
             var loglist = CrestronLogger.PrintTheLog(s.ToLower() == "all");
@@ -595,6 +640,9 @@ namespace PepperDash.Core
         /// <param name="message">Message template</param>
         /// <param name="device">Optional IKeyed device. If provided, the Key of the device will be added to the log message</param>
         /// <param name="args">Args to put into message template</param>
+        /// <summary>
+        /// LogMessage method
+        /// </summary>
         public static void LogMessage(Exception ex, string message, IKeyed device = null, params object[] args)
         {
             using (LogContext.PushProperty("Key", device?.Key))
@@ -623,16 +671,25 @@ namespace PepperDash.Core
            _logger.Write(level, message, args);
         }
 
+        /// <summary>
+        /// LogMessage method
+        /// </summary>
         public static void LogMessage(LogEventLevel level, Exception ex, string message, params object[] args)
         {
             _logger.Write(level, ex, message, args);
         }
 
+        /// <summary>
+        /// LogMessage method
+        /// </summary>
         public static void LogMessage(LogEventLevel level, IKeyed keyed, string message, params object[] args)
         {
             LogMessage(level, message, keyed, args);
         }
 
+        /// <summary>
+        /// LogMessage method
+        /// </summary>
         public static void LogMessage(LogEventLevel level, Exception ex, IKeyed device, string message, params object[] args)
         {
             using (LogContext.PushProperty("Key", device?.Key))
@@ -642,6 +699,9 @@ namespace PepperDash.Core
         }
 
         #region Explicit methods for logging levels
+        /// <summary>
+        /// LogVerbose method
+        /// </summary>
         public static void LogVerbose(IKeyed keyed, string message, params object[] args)
         {
             using(LogContext.PushProperty("Key", keyed?.Key))
@@ -650,6 +710,9 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// LogVerbose method
+        /// </summary>
         public static void LogVerbose(Exception ex, IKeyed keyed, string message, params object[] args)
         {
             using(LogContext.PushProperty("Key", keyed?.Key))
@@ -658,16 +721,25 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// LogVerbose method
+        /// </summary>
         public static void LogVerbose(string message, params object[] args)
         {
             _logger.Write(LogEventLevel.Verbose, message, args);
         }
 
+        /// <summary>
+        /// LogVerbose method
+        /// </summary>
         public static void LogVerbose(Exception ex, string message, params object[] args)
         {
             _logger.Write(LogEventLevel.Verbose, ex, null, message, args);
         }
 
+        /// <summary>
+        /// LogDebug method
+        /// </summary>
         public static void LogDebug(IKeyed keyed, string message, params object[] args)
         {
             using (LogContext.PushProperty("Key", keyed?.Key))
@@ -676,6 +748,9 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// LogDebug method
+        /// </summary>
         public static void LogDebug(Exception ex, IKeyed keyed, string message, params object[] args)
         {
             using (LogContext.PushProperty("Key", keyed?.Key))
@@ -684,16 +759,25 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// LogDebug method
+        /// </summary>
         public static void LogDebug(string message, params object[] args)
         {
             _logger.Write(LogEventLevel.Debug, message, args);
         }
 
+        /// <summary>
+        /// LogDebug method
+        /// </summary>
         public static void LogDebug(Exception ex, string message, params object[] args)
         {
             _logger.Write(LogEventLevel.Debug, ex, null, message, args);
         }
 
+        /// <summary>
+        /// LogInformation method
+        /// </summary>
         public static void LogInformation(IKeyed keyed, string message, params object[] args)
         {
             using (LogContext.PushProperty("Key", keyed?.Key))
@@ -702,6 +786,9 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// LogInformation method
+        /// </summary>
         public static void LogInformation(Exception ex, IKeyed keyed, string message, params object[] args)
         {
             using (LogContext.PushProperty("Key", keyed?.Key))
@@ -710,16 +797,25 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// LogInformation method
+        /// </summary>
         public static void LogInformation(string message, params object[] args)
         {
             _logger.Write(LogEventLevel.Information, message, args);
         }
 
+        /// <summary>
+        /// LogInformation method
+        /// </summary>
         public static void LogInformation(Exception ex, string message, params object[] args)
         {
             _logger.Write(LogEventLevel.Information, ex, null, message, args);
         }
 
+        /// <summary>
+        /// LogWarning method
+        /// </summary>
         public static void LogWarning(IKeyed keyed, string message, params object[] args)
         {
             using (LogContext.PushProperty("Key", keyed?.Key))
@@ -728,6 +824,9 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// LogWarning method
+        /// </summary>
         public static void LogWarning(Exception ex, IKeyed keyed, string message, params object[] args)
         {
             using (LogContext.PushProperty("Key", keyed?.Key))
@@ -736,16 +835,25 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// LogWarning method
+        /// </summary>
         public static void LogWarning(string message, params object[] args)
         {
             _logger.Write(LogEventLevel.Warning, message, args);
         }
 
+        /// <summary>
+        /// LogWarning method
+        /// </summary>
         public static void LogWarning(Exception ex, string message, params object[] args)
         {
             _logger.Write(LogEventLevel.Warning, ex, null, message, args);
         }
 
+        /// <summary>
+        /// LogError method
+        /// </summary>
         public static void LogError(IKeyed keyed, string message, params object[] args)
         {
             using (LogContext.PushProperty("Key", keyed?.Key))
@@ -754,6 +862,9 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// LogError method
+        /// </summary>
         public static void LogError(Exception ex, IKeyed keyed, string message, params object[] args)
         {
             using (LogContext.PushProperty("Key", keyed?.Key))
@@ -762,16 +873,25 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// LogError method
+        /// </summary>
         public static void LogError(string message, params object[] args)
         {
             _logger.Write(LogEventLevel.Error, message, args);
         }
 
+        /// <summary>
+        /// LogError method
+        /// </summary>
         public static void LogError(Exception ex, string message, params object[] args)
         {
             _logger.Write(LogEventLevel.Error, ex, null, message, args);
         }
 
+        /// <summary>
+        /// LogFatal method
+        /// </summary>
         public static void LogFatal(IKeyed keyed, string message, params object[] args)
         {
             using (LogContext.PushProperty("Key", keyed?.Key))
@@ -780,6 +900,9 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// LogFatal method
+        /// </summary>
         public static void LogFatal(Exception ex, IKeyed keyed, string message, params object[] args)
         {
             using (LogContext.PushProperty("Key", keyed?.Key))
@@ -788,11 +911,17 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// LogFatal method
+        /// </summary>
         public static void LogFatal(string message, params object[] args)
         {
             _logger.Write(LogEventLevel.Fatal, message, args);
         }
 
+        /// <summary>
+        /// LogFatal method
+        /// </summary>
         public static void LogFatal(Exception ex, string message, params object[] args)
         {
             _logger.Write(LogEventLevel.Fatal, ex, null, message, args);
@@ -869,6 +998,9 @@ namespace PepperDash.Core
         /// Logs to Console when at-level, and all messages to error log
         /// </summary>
         [Obsolete("Use LogMessage methods, Will be removed in 2.2.0 and later versions")]
+        /// <summary>
+        /// Console method
+        /// </summary>
         public static void Console(uint level, ErrorLogLevel errorLogLevel,
             string format, params object[] items)
         {
@@ -881,6 +1013,9 @@ namespace PepperDash.Core
         /// it will only be written to the log.
         /// </summary>
         [Obsolete("Use LogMessage methods, Will be removed in 2.2.0 and later versions")]
+        /// <summary>
+        /// ConsoleWithLog method
+        /// </summary>
         public static void ConsoleWithLog(uint level, string format, params object[] items)
         {
             LogMessage(level, format, items);
@@ -1005,6 +1140,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Error level to for message to be logged at
         /// </summary>
+        /// <summary>
+        /// Enumeration of ErrorLogLevel values
+        /// </summary>
         public enum ErrorLogLevel
         {
             /// <summary>

src/PepperDash.Core/Logging/DebugConsoleSink.cs

@@ -11,10 +11,16 @@ using System.Text;
 
 namespace PepperDash.Core
 {
+    /// <summary>
+    /// Represents a DebugConsoleSink
+    /// </summary>
     public class DebugConsoleSink : ILogEventSink
     {
         private readonly ITextFormatter _textFormatter;
 
+        /// <summary>
+        /// Emit method
+        /// </summary>
         public void Emit(LogEvent logEvent)
         {
             if (!Debug.IsRunningOnAppliance) return;            
@@ -44,6 +50,9 @@ namespace PepperDash.Core
 
     public static class DebugConsoleSinkExtensions
     {
+        /// <summary>
+        /// DebugConsoleSink method
+        /// </summary>
         public static LoggerConfiguration DebugConsoleSink(
                              this LoggerSinkConfiguration loggerConfiguration,
                                               ITextFormatter formatProvider = null)

src/PepperDash.Core/Logging/DebugContext.cs

@@ -38,6 +38,9 @@ namespace PepperDash.Core
         /// </summary>
         /// <param name="key"></param>
         /// <returns></returns>
+        /// <summary>
+        /// GetDebugContext method
+        /// </summary>
         public static DebugContext GetDebugContext(string key)
         {
             var context = Contexts.FirstOrDefault(c => c.Key.Equals(key, StringComparison.OrdinalIgnoreCase));
@@ -92,6 +95,9 @@ namespace PepperDash.Core
         /// Callback for console command
         /// </summary>
         /// <param name="levelString"></param>
+        /// <summary>
+        /// SetDebugFromConsole method
+        /// </summary>
         public void SetDebugFromConsole(string levelString)
         {
             try
@@ -114,6 +120,9 @@ namespace PepperDash.Core
         /// Sets the debug level
         /// </summary>
         /// <param name="level"> Valid values 0 (no debug), 1 (critical), 2 (all messages)</param>
+        /// <summary>
+        /// SetDebugLevel method
+        /// </summary>
         public void SetDebugLevel(int level)
         {
             if (level <= 2)
@@ -143,6 +152,9 @@ namespace PepperDash.Core
         /// <summary>
         /// Appends a device Key to the beginning of a message
         /// </summary>
+        /// <summary>
+        /// Console method
+        /// </summary>
         public void Console(uint level, IKeyed dev, string format, params object[] items)
         {
             if (SaveData.Level >= level)
@@ -191,6 +203,9 @@ namespace PepperDash.Core
         /// </summary>
         /// <param name="errorLogLevel"></param>
         /// <param name="str"></param>
+        /// <summary>
+        /// LogError method
+        /// </summary>
         public void LogError(Debug.ErrorLogLevel errorLogLevel, string str)
         {
             string msg = string.Format("App {0}:{1}", InitialParametersClass.ApplicationNumber, str);

src/PepperDash.Core/Logging/DebugCrestronLoggerSink.cs

@@ -5,8 +5,14 @@ using Serilog.Events;
 
 namespace PepperDash.Core.Logging
 {
+    /// <summary>
+    /// Represents a DebugCrestronLoggerSink
+    /// </summary>
     public class DebugCrestronLoggerSink : ILogEventSink
     {
+        /// <summary>
+        /// Emit method
+        /// </summary>
         public void Emit(LogEvent logEvent)
         {
             if (!Debug.IsRunningOnAppliance) return;

src/PepperDash.Core/Logging/DebugErrorLogSink.cs

@@ -11,6 +11,9 @@ using System.Threading.Tasks;
 
 namespace PepperDash.Core.Logging
 {
+    /// <summary>
+    /// Represents a DebugErrorLogSink
+    /// </summary>
     public class DebugErrorLogSink : ILogEventSink
     {
         private ITextFormatter _formatter;
@@ -24,6 +27,9 @@ namespace PepperDash.Core.Logging
             {LogEventLevel.Error, (msg) => ErrorLog.Error(msg) },
             {LogEventLevel.Fatal, (msg) => ErrorLog.Error(msg) }
         };
+        /// <summary>
+        /// Emit method
+        /// </summary>
         public void Emit(LogEvent logEvent)
         {
             string message;

src/PepperDash.Core/Logging/DebugExtensions.cs

@@ -6,66 +6,105 @@ namespace PepperDash.Core.Logging
 {
     public static class DebugExtensions
     {
+        /// <summary>
+        /// LogException method
+        /// </summary>
         public static void LogException(this IKeyed device, Exception ex, string message, params object[] args)
         {
             Log.LogMessage(ex, message, device, args);
         }
 
+        /// <summary>
+        /// LogVerbose method
+        /// </summary>
         public static void LogVerbose(this IKeyed device, Exception ex, string message, params object[] args)
         {
             Log.LogMessage(LogEventLevel.Verbose, ex, message, device, args);
         }
 
+        /// <summary>
+        /// LogVerbose method
+        /// </summary>
         public static void LogVerbose(this IKeyed device, string message, params object[] args)
         {
             Log.LogMessage(LogEventLevel.Verbose, device, message, args);
         }
 
+        /// <summary>
+        /// LogDebug method
+        /// </summary>
         public static void LogDebug(this IKeyed device, Exception ex, string message, params object[] args)
         {
             Log.LogMessage(LogEventLevel.Debug, ex, message, device, args);
         }
 
+        /// <summary>
+        /// LogDebug method
+        /// </summary>
         public static void LogDebug(this IKeyed device, string message, params object[] args)
         {
             Log.LogMessage(LogEventLevel.Debug, device, message, args);
         }
 
+        /// <summary>
+        /// LogInformation method
+        /// </summary>
         public static void LogInformation(this IKeyed device, Exception ex, string message, params object[] args)
         {
             Log.LogMessage(LogEventLevel.Information, ex, message, device, args);
         }
 
+        /// <summary>
+        /// LogInformation method
+        /// </summary>
         public static void LogInformation(this IKeyed device, string message, params object[] args)
         {
             Log.LogMessage(LogEventLevel.Information, device, message, args);
         }
 
+        /// <summary>
+        /// LogWarning method
+        /// </summary>
         public static void LogWarning(this IKeyed device, Exception ex, string message, params object[] args)
         {
             Log.LogMessage(LogEventLevel.Warning, ex, message, device, args);
         }
 
+        /// <summary>
+        /// LogWarning method
+        /// </summary>
         public static void LogWarning(this IKeyed device, string message, params object[] args)
         {
             Log.LogMessage(LogEventLevel.Warning, device, message, args);
         }
 
+        /// <summary>
+        /// LogError method
+        /// </summary>
         public static void LogError(this IKeyed device, Exception ex, string message, params object[] args)
         {
             Log.LogMessage(LogEventLevel.Error, ex, message, device, args);
         }
 
+        /// <summary>
+        /// LogError method
+        /// </summary>
         public static void LogError(this IKeyed device, string message, params object[] args)
         {
             Log.LogMessage(LogEventLevel.Error, device, message, args);
         }
 
+        /// <summary>
+        /// LogFatal method
+        /// </summary>
         public static void LogFatal(this IKeyed device, Exception ex, string message, params object[] args)
         {
             Log.LogMessage(LogEventLevel.Fatal, ex, message, device, args);
         }
 
+        /// <summary>
+        /// LogFatal method
+        /// </summary>
         public static void LogFatal(this IKeyed device, string message, params object[] args)
         {
             Log.LogMessage(LogEventLevel.Fatal, device, message, args);

src/PepperDash.Core/Logging/DebugMemory.cs

@@ -7,6 +7,9 @@ namespace PepperDash.Core.Logging
     /// <summary>
     /// Class to persist current Debug settings across program restarts
     /// </summary>
+ /// <summary>
+ /// Represents a DebugContextCollection
+ /// </summary>
 	public class DebugContextCollection
 	{
         /// <summary>
@@ -39,6 +42,9 @@ namespace PepperDash.Core.Logging
 		/// </summary>
 		/// <param name="contextKey"></param>
 		/// <param name="level"></param>
+  /// <summary>
+  /// SetLevel method
+  /// </summary>
 		public void SetLevel(string contextKey, int level)
 		{
 			if (level < 0 || level > 2)
@@ -51,6 +57,9 @@ namespace PepperDash.Core.Logging
 		/// </summary>
 		/// <param name="contextKey"></param>
 		/// <returns></returns>
+  /// <summary>
+  /// GetOrCreateItem method
+  /// </summary>
 		public DebugContextItem GetOrCreateItem(string contextKey)
 		{
 			if (!_items.ContainsKey(contextKey))
@@ -65,6 +74,9 @@ namespace PepperDash.Core.Logging
         /// <param name="deviceKey"></param>
         /// <param name="settings"></param>
         /// <returns></returns>
+        /// <summary>
+        /// SetDebugSettingsForKey method
+        /// </summary>
         public void SetDebugSettingsForKey(string deviceKey, object settings)
         {
             try
@@ -89,6 +101,9 @@ namespace PepperDash.Core.Logging
         /// </summary>
         /// <param name="deviceKey"></param>
         /// <returns></returns>
+        /// <summary>
+        /// GetDebugSettingsForKey method
+        /// </summary>
         public object GetDebugSettingsForKey(string deviceKey)
         {
             return DeviceDebugSettings[deviceKey];

src/PepperDash.Core/Logging/DebugWebsocketSink.cs

@@ -21,6 +21,9 @@ using Serilog.Formatting.Json;
 
 namespace PepperDash.Core
 {
+    /// <summary>
+    /// Represents a DebugWebsocketSink
+    /// </summary>
     public class DebugWebsocketSink : ILogEventSink
     {
         private HttpServer _httpsServer;
@@ -47,6 +50,9 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// Gets or sets the IsRunning
+        /// </summary>
         public bool IsRunning { get => _httpsServer?.IsListening ?? false; }
         
 
@@ -105,6 +111,9 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// Emit method
+        /// </summary>
         public void Emit(LogEvent logEvent)
         {
             if (_httpsServer == null || !_httpsServer.IsListening) return;
@@ -116,6 +125,9 @@ namespace PepperDash.Core
 
         }
 
+        /// <summary>
+        /// StartServerAndSetPort method
+        /// </summary>
         public void StartServerAndSetPort(int port)
         {
             Debug.Console(0, "Starting Websocket Server on port: {0}", port);
@@ -193,6 +205,9 @@ namespace PepperDash.Core
             }
         }
 
+        /// <summary>
+        /// StopServer method
+        /// </summary>
         public void StopServer()
         {
             Debug.Console(0, "Stopping Websocket Server");
@@ -204,6 +219,9 @@ namespace PepperDash.Core
 
     public static class DebugWebsocketSinkExtensions
     {
+        /// <summary>
+        /// DebugWebsocketSink method
+        /// </summary>
         public static LoggerConfiguration DebugWebsocketSink(
                              this LoggerSinkConfiguration loggerConfiguration,
                                               ITextFormatter formatProvider = null)
@@ -212,6 +230,9 @@ namespace PepperDash.Core
         }
     }
 
+    /// <summary>
+    /// Represents a DebugClient
+    /// </summary>
     public class DebugClient : WebSocketBehavior
     {
         private DateTime _connectionTime;

src/PepperDash.Core/PasswordManagement/PasswordClient.cs

@@ -5,6 +5,9 @@ namespace PepperDash.Core.PasswordManagement
     /// <summary>
     /// A class to allow user interaction with the PasswordManager
     /// </summary>
+ /// <summary>
+ /// Represents a PasswordClient
+ /// </summary>
 	public class PasswordClient
 	{
 		/// <summary>
@@ -59,6 +62,9 @@ namespace PepperDash.Core.PasswordManagement
 		/// Retrieve password by index
 		/// </summary>
 		/// <param name="key"></param>
+  /// <summary>
+  /// GetPasswordByIndex method
+  /// </summary>
 		public void GetPasswordByIndex(ushort key)
 		{
 			OnUshrtChange((ushort)PasswordManager.Passwords.Count, 0, PasswordManagementConstants.PasswordManagerCountChange);
@@ -81,6 +87,9 @@ namespace PepperDash.Core.PasswordManagement
 		/// Password validation method
 		/// </summary>
 		/// <param name="password"></param>
+  /// <summary>
+  /// ValidatePassword method
+  /// </summary>
 		public void ValidatePassword(string password)
 		{
 			if (string.IsNullOrEmpty(password))
@@ -99,6 +108,9 @@ namespace PepperDash.Core.PasswordManagement
 		/// password against the selected password when the length of the 2 are equal
 		/// </summary>
 		/// <param name="data"></param>
+  /// <summary>
+  /// BuildPassword method
+  /// </summary>
 		public void BuildPassword(string data)
 		{
 			PasswordToValidate = String.Concat(PasswordToValidate, data);
@@ -111,6 +123,9 @@ namespace PepperDash.Core.PasswordManagement
 		/// <summary>
 		/// Clears the user entered password and resets the LEDs
 		/// </summary>
+  /// <summary>
+  /// ClearPassword method
+  /// </summary>
 		public void ClearPassword()
 		{
 			PasswordToValidate = "";

src/PepperDash.Core/PasswordManagement/PasswordManager.cs

@@ -7,6 +7,9 @@ namespace PepperDash.Core.PasswordManagement
     /// <summary>
     /// Allows passwords to be stored and managed
     /// </summary>
+ /// <summary>
+ /// Represents a PasswordManager
+ /// </summary>
 	public class PasswordManager
 	{
 		/// <summary>
@@ -71,6 +74,9 @@ namespace PepperDash.Core.PasswordManagement
 		/// </summary>
 		/// <param name="key"></param>
 		/// <param name="password"></param>
+  /// <summary>
+  /// UpdatePassword method
+  /// </summary>
 		public void UpdatePassword(ushort key, string password)
 		{
 			// validate the parameters
@@ -152,6 +158,9 @@ namespace PepperDash.Core.PasswordManagement
 		/// Method to change the default timer value, (default 5000ms/5s)
 		/// </summary>
 		/// <param name="time"></param>
+  /// <summary>
+  /// PasswordTimerMs method
+  /// </summary>
 		public void PasswordTimerMs(ushort time)
 		{
 			PasswordTimerElapsedMs = Convert.ToInt64(time);

src/PepperDash.Core/SystemInfo/EventArgs and Constants.cs

@@ -71,6 +71,9 @@ namespace PepperDash.Core.SystemInfo
 	/// <summary>
 	/// Processor Change Event Args Class
 	/// </summary>
+ /// <summary>
+ /// Represents a ProcessorChangeEventArgs
+ /// </summary>
 	public class ProcessorChangeEventArgs : EventArgs
 	{
         /// <summary>
@@ -117,6 +120,9 @@ namespace PepperDash.Core.SystemInfo
 	/// <summary>
 	/// Ethernet Change Event Args Class
 	/// </summary>
+ /// <summary>
+ /// Represents a EthernetChangeEventArgs
+ /// </summary>
 	public class EthernetChangeEventArgs : EventArgs
 	{
         /// <summary>
@@ -168,6 +174,9 @@ namespace PepperDash.Core.SystemInfo
 	/// <summary>
 	/// Control Subnet Chage Event Args Class
 	/// </summary>
+ /// <summary>
+ /// Represents a ControlSubnetChangeEventArgs
+ /// </summary>
 	public class ControlSubnetChangeEventArgs : EventArgs
 	{
         /// <summary>
@@ -214,6 +223,9 @@ namespace PepperDash.Core.SystemInfo
 	/// <summary>
 	/// Program Change Event Args Class
 	/// </summary>
+ /// <summary>
+ /// Represents a ProgramChangeEventArgs
+ /// </summary>
 	public class ProgramChangeEventArgs : EventArgs
 	{
         /// <summary>

src/PepperDash.Core/SystemInfo/SystemInfoToSimpl.cs

@@ -103,6 +103,9 @@ namespace PepperDash.Core.SystemInfo
 		/// <summary>
 		/// Gets the current ethernet info
 		/// </summary>
+  /// <summary>
+  /// GetEthernetInfo method
+  /// </summary>
 		public void GetEthernetInfo()
 		{
 			OnBoolChange(true, 0, SystemInfoConstants.BusyBoolChange);
@@ -164,6 +167,9 @@ namespace PepperDash.Core.SystemInfo
 		/// <summary>
 		/// Gets the current control subnet info
 		/// </summary>
+  /// <summary>
+  /// GetControlSubnetInfo method
+  /// </summary>
 		public void GetControlSubnetInfo()
 		{
 			OnBoolChange(true, 0, SystemInfoConstants.BusyBoolChange);
@@ -206,6 +212,9 @@ namespace PepperDash.Core.SystemInfo
 		/// Gets the program info by index
 		/// </summary>
 		/// <param name="index"></param>
+  /// <summary>
+  /// GetProgramInfoByIndex method
+  /// </summary>
 		public void GetProgramInfoByIndex(ushort index)
 		{
 			if (index < 1 || index > 10)
@@ -266,6 +275,9 @@ namespace PepperDash.Core.SystemInfo
 		/// <summary>
 		/// Gets the processor uptime and passes it to S+
 		/// </summary>
+  /// <summary>
+  /// RefreshProcessorUptime method
+  /// </summary>
 		public void RefreshProcessorUptime()
 		{
 			try
@@ -287,6 +299,9 @@ namespace PepperDash.Core.SystemInfo
 		/// Gets the program uptime, by index, and passes it to S+
 		/// </summary>
 		/// <param name="index"></param>
+  /// <summary>
+  /// RefreshProgramUptimeByIndex method
+  /// </summary>
 		public void RefreshProgramUptimeByIndex(int index)
 		{
 			try
@@ -308,6 +323,9 @@ namespace PepperDash.Core.SystemInfo
 		/// Sends command to console, passes response back using string change event
 		/// </summary>
 		/// <param name="cmd"></param>
+  /// <summary>
+  /// SendConsoleCommand method
+  /// </summary>
 		public void SendConsoleCommand(string cmd)
 		{
 			if (string.IsNullOrEmpty(cmd))

src/PepperDash.Core/Web/BouncyCertificate.cs

@@ -35,6 +35,9 @@ namespace PepperDash.Core
             return issuerCertificate;
         }
 
+        /// <summary>
+        /// IssueCertificate method
+        /// </summary>
         public X509Certificate2 IssueCertificate(string subjectName, X509Certificate2 issuerCertificate, string[] subjectAlternativeNames, KeyPurposeID[] usages)
         {
             // It's self-signed, so these are the same.
@@ -56,6 +59,9 @@ namespace PepperDash.Core
             return ConvertCertificate(certificate, subjectKeyPair, random);
         }
 
+        /// <summary>
+        /// CreateCertificateAuthorityCertificate method
+        /// </summary>
         public X509Certificate2 CreateCertificateAuthorityCertificate(string subjectName, string[] subjectAlternativeNames, KeyPurposeID[] usages)
         {
             // It's self-signed, so these are the same.
@@ -78,6 +84,9 @@ namespace PepperDash.Core
             return ConvertCertificate(certificate, subjectKeyPair, random);
         }
 
+        /// <summary>
+        /// CreateSelfSignedCertificate method
+        /// </summary>
         public X509Certificate2 CreateSelfSignedCertificate(string subjectName, string[] subjectAlternativeNames, KeyPurposeID[] usages)
         {
             // It's self-signed, so these are the same.
@@ -305,6 +314,9 @@ namespace PepperDash.Core
             return convertedCertificate;
         }
 
+        /// <summary>
+        /// WriteCertificate method
+        /// </summary>
         public void WriteCertificate(X509Certificate2 certificate, string outputDirectory, string certName)
         {
             // This password is the one attached to the PFX file. Use 'null' for no password.
@@ -332,6 +344,9 @@ namespace PepperDash.Core
                 }
             }
         }
+        /// <summary>
+        /// AddCertToStore method
+        /// </summary>
         public bool AddCertToStore(X509Certificate2 cert, System.Security.Cryptography.X509Certificates.StoreName st, System.Security.Cryptography.X509Certificates.StoreLocation sl)
         {
             bool bRet = false;

src/PepperDash.Core/Web/RequestHandlers/DefaultRequestHandler.cs

@@ -5,6 +5,9 @@ namespace PepperDash.Core.Web.RequestHandlers
 	/// <summary>
 	/// Web API default request handler
 	/// </summary>
+ /// <summary>
+ /// Represents a DefaultRequestHandler
+ /// </summary>
 	public class DefaultRequestHandler : WebApiBaseRequestHandler
 	{
 		/// <summary>

src/PepperDash.Core/Web/RequestHandlers/WebApiBaseRequestAsyncHandler.cs

@@ -142,6 +142,9 @@ namespace PepperDash.Core.Web.RequestHandlers
         /// Process request
         /// </summary>
         /// <param name="context"></param>
+        /// <summary>
+        /// ProcessRequest method
+        /// </summary>
         public void ProcessRequest(HttpCwsContext context)
         {
             if (!_handlers.TryGetValue(context.Request.HttpMethod, out Func<HttpCwsContext, Task> handler))

src/PepperDash.Core/Web/RequestHandlers/WebApiBaseRequestHandler.cs

@@ -144,6 +144,9 @@ namespace PepperDash.Core.Web.RequestHandlers
 		/// Process request
 		/// </summary>
 		/// <param name="context"></param>
+  /// <summary>
+  /// ProcessRequest method
+  /// </summary>
 		public void ProcessRequest(HttpCwsContext context)
 		{
 			Action<HttpCwsContext> handler;

src/PepperDash.Core/Web/WebApiServer.cs

@@ -28,21 +28,33 @@ namespace PepperDash.Core.Web
 		/// <summary>
 		/// Web API server key
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the Key
+  /// </summary>
 		public string Key { get; private set; }
 
 		/// <summary>
 		/// Web API server name
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the Name
+  /// </summary>
 		public string Name { get; private set; }
 
 		/// <summary>
 		/// CWS base path, will default to "/api" if not set via initialize method
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the BasePath
+  /// </summary>
 		public string BasePath { get; private set; }
 
 		/// <summary>
 		/// Indicates CWS is registered with base path
 		/// </summary>
+  /// <summary>
+  /// Gets or sets the IsRegistered
+  /// </summary>
 		public bool IsRegistered { get; private set; }
 
 		/// <summary>
@@ -140,6 +152,9 @@ namespace PepperDash.Core.Web
 		/// <summary>
 		/// Initializes CWS class
 		/// </summary>
+  /// <summary>
+  /// Initialize method
+  /// </summary>
 		public void Initialize(string key, string basePath)
 		{
 			Key = key;
@@ -165,6 +180,9 @@ namespace PepperDash.Core.Web
 		/// Removes a route from CWS
 		/// </summary>
 		/// <param name="route"></param>
+  /// <summary>
+  /// RemoveRoute method
+  /// </summary>
 		public void RemoveRoute(HttpCwsRoute route)
 		{
 			if (route == null)
@@ -179,6 +197,9 @@ namespace PepperDash.Core.Web
 		/// <summary>
 		/// Returns a list of the current routes
 		/// </summary>
+  /// <summary>
+  /// GetRouteCollection method
+  /// </summary>
 		public HttpCwsRouteCollection GetRouteCollection()
 		{
 			return _server.Routes;
@@ -225,6 +246,9 @@ namespace PepperDash.Core.Web
 		/// <summary>
 		/// Stop CWS instance
 		/// </summary>
+  /// <summary>
+  /// Stop method
+  /// </summary>
 		public void Stop()
 		{
 			try

src/PepperDash.Core/WebApi/Presets/Preset.cs

@@ -5,6 +5,9 @@ namespace PepperDash.Core.WebApi.Presets
     /// <summary>
     /// Represents a preset
     /// </summary>
+ /// <summary>
+ /// Represents a Preset
+ /// </summary>
 	public class Preset
 	{
         /// <summary>
@@ -15,26 +18,41 @@ namespace PepperDash.Core.WebApi.Presets
         /// <summary>
         /// User ID
         /// </summary>
+  /// <summary>
+  /// Gets or sets the UserId
+  /// </summary>
 		public int UserId { get; set; }
 
         /// <summary>
         /// Room Type ID
         /// </summary>
+  /// <summary>
+  /// Gets or sets the RoomTypeId
+  /// </summary>
 		public int RoomTypeId { get; set; }
 
         /// <summary>
         /// Preset Name
         /// </summary>
+  /// <summary>
+  /// Gets or sets the PresetName
+  /// </summary>
 		public string PresetName { get; set; }
 
         /// <summary>
         /// Preset Number
         /// </summary>
+  /// <summary>
+  /// Gets or sets the PresetNumber
+  /// </summary>
 		public int PresetNumber { get; set; }
 
         /// <summary>
         /// Preset Data
         /// </summary>
+  /// <summary>
+  /// Gets or sets the Data
+  /// </summary>
 		public string Data { get; set; }
 
         /// <summary>
@@ -51,6 +69,9 @@ namespace PepperDash.Core.WebApi.Presets
 	/// <summary>
 	/// 
 	/// </summary>
+ /// <summary>
+ /// Represents a PresetReceivedEventArgs
+ /// </summary>
 	public class PresetReceivedEventArgs : EventArgs
 	{
         /// <summary>
@@ -61,11 +82,17 @@ namespace PepperDash.Core.WebApi.Presets
         /// <summary>
         /// S+ helper
         /// </summary>
+        /// <summary>
+        /// Gets or sets the ULookupSuccess
+        /// </summary>
         public ushort ULookupSuccess { get { return (ushort)(LookupSuccess ? 1 : 0); } }
 
         /// <summary>
         /// The preset
         /// </summary>
+        /// <summary>
+        /// Gets or sets the Preset
+        /// </summary>
         public Preset Preset { get; private set; }
 
 		/// <summary>

src/PepperDash.Core/WebApi/Presets/User.cs

@@ -19,16 +19,25 @@ namespace PepperDash.Core.WebApi.Presets
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the ExternalId
+  /// </summary>
 		public string ExternalId { get; set; }
 
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the FirstName
+  /// </summary>
 		public string FirstName { get; set; }
 
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the LastName
+  /// </summary>
 		public string LastName { get; set; }
 	}
 
@@ -46,11 +55,17 @@ namespace PepperDash.Core.WebApi.Presets
         /// <summary>
         /// For stupid S+
         /// </summary>
+        /// <summary>
+        /// Gets or sets the ULookupSuccess
+        /// </summary>
         public ushort ULookupSuccess { get { return (ushort)(LookupSuccess ? 1 : 0); } }
 
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the User
+  /// </summary>
 		public User User { get; private set; }
 
 		/// <summary>
@@ -73,6 +88,9 @@ namespace PepperDash.Core.WebApi.Presets
 	/// <summary>
 	/// 
 	/// </summary>
+ /// <summary>
+ /// Represents a UserAndRoomMessage
+ /// </summary>
 	public class UserAndRoomMessage
 	{
         /// <summary>
@@ -83,11 +101,17 @@ namespace PepperDash.Core.WebApi.Presets
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the RoomTypeId
+  /// </summary>
 		public int RoomTypeId { get; set; }
 
         /// <summary>
         /// 
         /// </summary>
+  /// <summary>
+  /// Gets or sets the PresetNumber
+  /// </summary>
 		public int PresetNumber { get; set; }
 	}
 }

src/PepperDash.Core/WebApi/Presets/WebApiPasscodeClient.cs

@@ -28,6 +28,9 @@ namespace PepperDash.Core.WebApi.Presets
         /// <summary>
         /// Unique identifier for this instance
         /// </summary>
+  /// <summary>
+  /// Gets or sets the Key
+  /// </summary>
 		public string Key { get; private set; }
 
 		//string JsonMasterKey;
@@ -77,6 +80,9 @@ namespace PepperDash.Core.WebApi.Presets
         /// Gets the user for a passcode
         /// </summary>
         /// <param name="passcode"></param>
+  /// <summary>
+  /// GetUserForPasscode method
+  /// </summary>
 		public void GetUserForPasscode(string passcode)
 		{
             // Bullshit duplicate code here... These two cases should be the same 
@@ -115,6 +121,9 @@ namespace PepperDash.Core.WebApi.Presets
 		/// </summary>
 		/// <param name="roomTypeId"></param>
 		/// <param name="presetNumber"></param>
+  /// <summary>
+  /// GetPresetForThisUser method
+  /// </summary>
 		public void GetPresetForThisUser(int roomTypeId, int presetNumber)
 		{
 			if (CurrentUser == null)
@@ -212,6 +221,9 @@ namespace PepperDash.Core.WebApi.Presets
 		/// </summary>
 		/// <param name="roomTypeId"></param>
 		/// <param name="presetNumber"></param>
+  /// <summary>
+  /// SavePresetForThisUser method
+  /// </summary>
 		public void SavePresetForThisUser(int roomTypeId, int presetNumber)
 		{
 			if (CurrentPreset == null)

src/PepperDash.Core/XSigUtility/Tokens/XSigAnalogToken.cs

@@ -59,6 +59,9 @@ namespace PepperDash.Core.Intersystem.Tokens
         /// </summary>
         /// <param name="offset"></param>
         /// <returns></returns>
+        /// <summary>
+        /// GetTokenWithOffset method
+        /// </summary>
         public override XSigToken GetTokenWithOffset(int offset)
         {
             if (offset == 0) return this;
@@ -69,6 +72,9 @@ namespace PepperDash.Core.Intersystem.Tokens
         /// 
         /// </summary>
         /// <returns></returns>
+        /// <summary>
+        /// ToString method
+        /// </summary>
         public override string ToString()
         {
             return Index + " = 0x" + Value.ToString("X4");
@@ -80,6 +86,9 @@ namespace PepperDash.Core.Intersystem.Tokens
         /// <param name="format"></param>
         /// <param name="formatProvider"></param>
         /// <returns></returns>
+        /// <summary>
+        /// ToString method
+        /// </summary>
         public string ToString(string format, IFormatProvider formatProvider)
         {
             return Value.ToString(format, formatProvider);

src/PepperDash.Core/XSigUtility/Tokens/XSigDigitalToken.cs

@@ -57,6 +57,9 @@ namespace PepperDash.Core.Intersystem.Tokens
         /// </summary>
         /// <param name="offset"></param>
         /// <returns></returns>
+        /// <summary>
+        /// GetTokenWithOffset method
+        /// </summary>
         public override XSigToken GetTokenWithOffset(int offset)
         {
             if (offset == 0) return this;
@@ -67,6 +70,9 @@ namespace PepperDash.Core.Intersystem.Tokens
         /// 
         /// </summary>
         /// <returns></returns>
+        /// <summary>
+        /// ToString method
+        /// </summary>
         public override string ToString()
         {
             return Index + " = " + (Value ? "High" : "Low");
@@ -77,6 +83,9 @@ namespace PepperDash.Core.Intersystem.Tokens
         /// </summary>
         /// <param name="formatProvider"></param>
         /// <returns></returns>
+        /// <summary>
+        /// ToString method
+        /// </summary>
         public string ToString(IFormatProvider formatProvider)
         {
             return Value.ToString(formatProvider);

src/PepperDash.Core/XSigUtility/Tokens/XSigSerialToken.cs

@@ -63,6 +63,9 @@ namespace PepperDash.Core.Intersystem.Tokens
         /// </summary>
         /// <param name="offset"></param>
         /// <returns></returns>
+        /// <summary>
+        /// GetTokenWithOffset method
+        /// </summary>
         public override XSigToken GetTokenWithOffset(int offset)
         {
             if (offset == 0) return this;
@@ -73,6 +76,9 @@ namespace PepperDash.Core.Intersystem.Tokens
         /// 
         /// </summary>
         /// <returns></returns>
+        /// <summary>
+        /// ToString method
+        /// </summary>
         public override string ToString()
         {
             return Index + " = \"" + Value + "\"";

src/PepperDash.Core/XSigUtility/XSigHelpers.cs

@@ -52,6 +52,9 @@ namespace PepperDash.Core.Intersystem
         /// </summary>
         /// <param name="xSigSerialization">XSig state resolver.</param>
         /// <returns>Bytes in XSig format for each token within the state representation.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(IXSigSerialization xSigSerialization)
         {
             return GetBytes(xSigSerialization, 0);
@@ -63,6 +66,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="xSigSerialization">XSig state resolver.</param>
         /// <param name="offset">Offset to which the data will be aligned.</param>
         /// <returns>Bytes in XSig format for each token within the state representation.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(IXSigSerialization xSigSerialization, int offset)
         {
             var tokens = xSigSerialization.Serialize();
@@ -82,6 +88,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="index">1-based digital index</param>
         /// <param name="value">Digital data to be encoded</param>
         /// <returns>Bytes in XSig format for digtial information.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(int index, bool value)
         {
             return GetBytes(index, 0, value);
@@ -94,6 +103,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="offset">Index offset.</param>
         /// <param name="value">Digital data to be encoded</param>
         /// <returns>Bytes in XSig format for digtial information.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(int index, int offset, bool value)
         {
             return new XSigDigitalToken(index + offset, value).GetBytes();
@@ -105,6 +117,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="startIndex">Starting index of the sequence.</param>
         /// <param name="values">Digital signal value array.</param>
         /// <returns>Byte sequence in XSig format for digital signal information.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(int startIndex, bool[] values)
         {
             return GetBytes(startIndex, 0, values);
@@ -117,6 +132,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="offset">Index offset.</param>
         /// <param name="values">Digital signal value array.</param>
         /// <returns>Byte sequence in XSig format for digital signal information.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(int startIndex, int offset, bool[] values)
         {
             // Digital XSig data is 2 bytes per value
@@ -134,6 +152,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="index">1-based analog index</param>
         /// <param name="value">Analog data to be encoded</param>
         /// <returns>Bytes in XSig format for analog signal information.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(int index, ushort value)
         {
             return GetBytes(index, 0, value);
@@ -146,6 +167,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="offset">Index offset.</param>
         /// <param name="value">Analog data to be encoded</param>
         /// <returns>Bytes in XSig format for analog signal information.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(int index, int offset, ushort value)
         {
             return new XSigAnalogToken(index + offset, value).GetBytes();
@@ -157,6 +181,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="startIndex">Starting index of the sequence.</param>
         /// <param name="values">Analog signal value array.</param>
         /// <returns>Byte sequence in XSig format for analog signal information.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(int startIndex, ushort[] values)
         {
             return GetBytes(startIndex, 0, values);
@@ -169,6 +196,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="offset">Index offset.</param>
         /// <param name="values">Analog signal value array.</param>
         /// <returns>Byte sequence in XSig format for analog signal information.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(int startIndex, int offset, ushort[] values)
         {
             // Analog XSig data is 4 bytes per value
@@ -186,6 +216,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="index">1-based serial index</param>
         /// <param name="value">Serial data to be encoded</param>
         /// <returns>Bytes in XSig format for serial signal information.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(int index, string value)
         {
             return GetBytes(index, 0, value);
@@ -198,6 +231,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="offset">Index offset.</param>
         /// <param name="value">Serial data to be encoded</param>
         /// <returns>Bytes in XSig format for serial signal information.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(int index, int offset, string value)
         {
             return new XSigSerialToken(index + offset, value).GetBytes();
@@ -209,6 +245,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="startIndex">Starting index of the sequence.</param>
         /// <param name="values">Serial signal value array.</param>
         /// <returns>Byte sequence in XSig format for serial signal information.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(int startIndex, string[] values)
         {
             return GetBytes(startIndex, 0, values);
@@ -221,6 +260,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="offset">Index offset.</param>
         /// <param name="values">Serial signal value array.</param>
         /// <returns>Byte sequence in XSig format for serial signal information.</returns>
+        /// <summary>
+        /// GetBytes method
+        /// </summary>
         public static byte[] GetBytes(int startIndex, int offset, string[] values)
         {
             // Serial XSig data is not fixed-length like the other formats

src/PepperDash.Core/XSigUtility/XSigTokenStreamReader.cs

@@ -48,6 +48,9 @@ namespace PepperDash.Core.Intersystem
         /// <param name="stream">Input stream</param>
         /// <param name="value">Result</param>
         /// <returns>True if successful, otherwise false.</returns>
+        /// <summary>
+        /// TryReadUInt16BE method
+        /// </summary>
         public static bool TryReadUInt16BE(Stream stream, out ushort value)
         {
             value = 0;
@@ -65,6 +68,9 @@ namespace PepperDash.Core.Intersystem
         /// </summary>
         /// <returns>XSigToken</returns>
         /// <exception cref="ArgumentOutOfRangeException">Offset is less than 0.</exception>
+        /// <summary>
+        /// ReadXSigToken method
+        /// </summary>
         public XSigToken ReadXSigToken()
         {
             ushort prefix;
@@ -114,6 +120,9 @@ namespace PepperDash.Core.Intersystem
         /// Reads all available XSig tokens from the stream.
         /// </summary>
         /// <returns>XSigToken collection.</returns>
+        /// <summary>
+        /// ReadAllXSigTokens method
+        /// </summary>
         public IEnumerable<XSigToken> ReadAllXSigTokens()
         {
             var tokens = new List<XSigToken>();

src/PepperDash.Core/XSigUtility/XSigTokenStreamWriter.cs

@@ -47,6 +47,9 @@ namespace PepperDash.Core.Intersystem
         /// Write XSig data gathered from an IXSigStateResolver to the stream.
         /// </summary>
         /// <param name="xSigSerialization">IXSigStateResolver object.</param>
+        /// <summary>
+        /// WriteXSigData method
+        /// </summary>
         public void WriteXSigData(IXSigSerialization xSigSerialization)
         {
             WriteXSigData(xSigSerialization, 0);
@@ -57,6 +60,9 @@ namespace PepperDash.Core.Intersystem
         /// </summary>
         /// <param name="xSigSerialization">IXSigStateResolver object.</param>
         /// <param name="offset">Index offset for each XSigToken.</param>
+        /// <summary>
+        /// WriteXSigData method
+        /// </summary>
         public void WriteXSigData(IXSigSerialization xSigSerialization, int offset)
         {
             if (xSigSerialization == null)
@@ -70,6 +76,9 @@ namespace PepperDash.Core.Intersystem
         /// Write XSigToken to the stream.
         /// </summary>
         /// <param name="token">XSigToken object.</param>
+        /// <summary>
+        /// WriteXSigData method
+        /// </summary>
         public void WriteXSigData(XSigToken token)
         {
             WriteXSigData(token, 0);
@@ -80,6 +89,9 @@ namespace PepperDash.Core.Intersystem
         /// </summary>
         /// <param name="token">XSigToken object.</param>
         /// <param name="offset">Index offset for each XSigToken.</param>
+        /// <summary>
+        /// WriteXSigData method
+        /// </summary>
         public void WriteXSigData(XSigToken token, int offset)
         {
             WriteXSigData(new[] { token }, offset);
@@ -108,6 +120,9 @@ namespace PepperDash.Core.Intersystem
         /// </summary>
         /// <param name="tokens">XSigToken objects.</param>
         /// <param name="offset">Index offset for each XSigToken.</param>
+        /// <summary>
+        /// WriteXSigData method
+        /// </summary>
         public void WriteXSigData(IEnumerable<XSigToken> tokens, int offset)
         {
             if (offset < 0)
@@ -127,6 +142,9 @@ namespace PepperDash.Core.Intersystem
         /// <summary>
         /// Disposes of the internal stream if specified to not leave open.
         /// </summary>
+        /// <summary>
+        /// Dispose method
+        /// </summary>
         public void Dispose()
         {
             if (!_leaveOpen)

src/PepperDash.Essentials.Core/Factory/DeviceFactory.cs

@@ -12,12 +12,29 @@ using System.Linq;
 
 namespace PepperDash.Essentials.Core
 {
+    /// <summary>
+    /// Wrapper class for device factory information
+    /// </summary>
     public class DeviceFactoryWrapper
     {
+        /// <summary>
+        /// Gets or sets the device type
+        /// </summary>
         public Type Type { get; set; }
+        
+        /// <summary>
+        /// Gets or sets the description of the device factory
+        /// </summary>
         public string Description { get; set; }
+        
+        /// <summary>
+        /// Gets or sets the factory method for creating devices
+        /// </summary>
         public Func<DeviceConfig, IKeyed> FactoryMethod { get; set; }
 
+        /// <summary>
+        /// Initializes a new instance of the DeviceFactoryWrapper class
+        /// </summary>
         public DeviceFactoryWrapper()
         {
             Type = null;
@@ -25,8 +42,14 @@ namespace PepperDash.Essentials.Core
         }
     }
 
+    /// <summary>
+    /// Factory class for loading and managing device types
+    /// </summary>
     public class DeviceFactory
     {
+        /// <summary>
+        /// Initializes a new instance of the DeviceFactory class and loads all device type factories
+        /// </summary>
         public DeviceFactory()
         {
             var assy = Assembly.GetExecutingAssembly();