updated measurement node to match selected units from user and convert it properly

rotatingMachine.html

@@ -24,8 +24,7 @@
           warmup: { value: 0 },
           shutdown: { value: 0 },
           cooldown: { value: 0 },
-          machineCurve: { value: {}},
+          machineCurve : { value: {}},
-          flowNumber: { value: 1, required: true },
 
           //define asset properties
           uuid: { value: "" },
@@ -128,10 +127,6 @@
     <label for="node-input-cooldown"><i class="fa fa-clock-o"></i> Cooldown Time</label>
     <input type="number" id="node-input-cooldown" style="width:60%;" />
   </div>
-  <div class="form-row">
-    <label for="node-input-flowNumber"><i class="fa fa-clock-o"></i> Flow Number</label>
-    <input type="number" id="node-input-flowNumber" style="width:60%;" />
-  </div>
 
   <!-- Asset fields injected here -->
   <div id="asset-fields-placeholder"></div>

src/nodeClass.js

@@ -63,8 +63,7 @@ class nodeClass {
       },
       functionality: {
         positionVsParent: uiConfig.positionVsParent
-      },
+      }
-      flowNumber: uiConfig.flowNumber
     };
 
     // Utility for formatting outputs
@@ -115,8 +114,8 @@ class nodeClass {
     try {
           const mode = m.currentMode;
           const state = m.state.getCurrentState();
-          const flow = Math.round(m.measurements.type("flow").variant("predicted").position('downstream').getCurrentValue());
+          const flow = Math.round(m.measurements.type("flow").variant("predicted").position('downstream').getCurrentValue('m3/h'));
-          const power = Math.round(m.measurements.type("power").variant("predicted").position('upstream').getCurrentValue());
+          const power = Math.round(m.measurements.type("power").variant("predicted").position('atequipment').getCurrentValue('kW'));
           let symbolState;
           switch(state){
             case "off":

src/specificClass.js

@@ -48,7 +48,17 @@ class Machine {
     this.errorMetrics = new nrmse(errorMetricsConfig, this.logger);
 
     // Initialize measurements
-    this.measurements = new MeasurementContainer();
+    this.measurements = new MeasurementContainer({
+      autoConvert: true,
+      windowSize: 50,
+      defaultUnits: {
+        pressure: 'mbar',
+        flow: this.config.general.unit,
+        power: 'kW',
+        temperature: 'C'
+      }
+    });
+
     this.interpolation = new interpolation();
 
     this.flowDrift = null;
@@ -68,94 +78,97 @@ class Machine {
       this.updatePosition();
     });
 
-    // used for holding the source and sink unit operations or other object with setInfluent / getEffluent method for e.g. recirculation.
+    //When state changes look if we need to do other updates
-    this.upstreamSource = null;
+    this.state.emitter.on("stateChange", (newState) => {
-    this.downstreamSink = null;
+      this.logger.debug(`State change detected: ${newState}`);
+      this._updateState();
+    });
 
     this.child = {}; // object to hold child information so we know on what to subscribe
     this.childRegistrationUtils = new childRegistrationUtils(this); // Child registration utility
 
   }
 
+  _updateState(){
+    const isOperational = this._isOperationalState();
+    if(!isOperational){
+      //overrule the last prediction this should be 0 now
+      this.measurements.type("flow").variant("predicted").position("downstream").value(0);
+    }
+  }
+
   /*-------------------               Register child events               -------------------*/
   registerChild(child, softwareType) {
-    if(!child) {
+    this.logger.debug('Setting up child event for softwaretype ' + softwareType);
-      this.logger.error(`Invalid ${softwareType} child provided.`);
-      return;
-    }
 
-    switch (softwareType) {
+    if(softwareType === "measurement"){
-      case "measurement":
+          const position = child.config.functionality.positionVsParent;
-        this.logger.debug(`Registering measurement child...`);
+          const distance = child.config.functionality.distanceVsParent || 0;
-        this._connectMeasurement(child);
+          const measurementType = child.config.asset.type;
-        break;
+          const key = `${measurementType}_${position}`;
-      case "reactor":
+          //rebuild to measurementype.variant no position and then switch based on values not strings or names.
-        this.logger.debug(`Registering reactor child...`);
+          const eventName = `${measurementType}.measured.${position}`;
-        this._connectReactor(child);
-        break;
-      default:
-        this.logger.error(`Unrecognized softwareType: ${softwareType}`);
-    }
-  }
 
-  _connectMeasurement(measurementChild) {
+          this.logger.debug(`Setting up listener for ${eventName} from child ${child.config.general.name}`);
-    const position = measurementChild.config.functionality.positionVsParent;
+          // Register event listener for measurement updates
-    const distance = measurementChild.config.functionality.distanceVsParent || 0;
+          child.measurements.emitter.on(eventName, (eventData) => {
-    const measurementType = measurementChild.config.asset.type;
+          this.logger.debug(`🔄 ${position} ${measurementType} from ${eventData.childName}: ${eventData.value} ${eventData.unit}`);
-    //rebuild to measurementype.variant no position and then switch based on values not strings or names.
-    const eventName = `${measurementType}.measured.${position}`;
-
-    this.logger.debug(`Setting up listener for ${eventName} from child ${measurementChild.config.general.name}`);
-    // Register event listener for measurement updates
-    measurementChild.measurements.emitter.on(eventName, (eventData) => {
-      this.logger.debug(`🔄 ${position} ${measurementType} from ${eventData.childName}: ${eventData.value} ${eventData.unit}`);
-
-      // Store directly in parent's measurement container
-      this.measurements
-        .type(measurementType)
-        .variant("measured")
-        .position(position)
-        .value(eventData.value, eventData.timestamp, eventData.unit);      
       
-      // Call the appropriate handler
+
-      switch (measurementType) {
+          console.log(` Emitting... ${eventName} with data:`);
-        case 'pressure':
+          // Store directly in parent's measurement container
-          this.updateMeasuredPressure(eventData.value, position, eventData);
+          this.measurements
-          break;
+            .type(measurementType)
+            .variant("measured")
+            .position(position)
+            .value(eventData.value, eventData.timestamp, eventData.unit);
           
-        case 'flow':
+          // Call the appropriate handler
-          this.updateMeasuredFlow(eventData.value, position, eventData);
+          this._callMeasurementHandler(measurementType, eventData.value, position, eventData);
-          break;
+      });
-          
+    }
-        default:
-          this.logger.warn(`No handler for measurement type: ${measurementType}`);
-          // Generic handler - just update position
-          this.updatePosition();
-      }
-    });
   }
 
-  _connectReactor(reactorChild) {
+// Centralized handler dispatcher
-    this.downstreamSink = reactorChild; // downstream from the pumps perpective
+_callMeasurementHandler(measurementType, value, position, context) {
+  switch (measurementType) {
+    case 'pressure':
+      this.updateMeasuredPressure(value, position, context);
+      break;
+      
+    case 'flow':
+      this.updateMeasuredFlow(value, position, context);
+      break;
+      
+    case 'temperature':
+      this.updateMeasuredTemperature(value, position, context);
+      break;
+      
+    default:
+      this.logger.warn(`No handler for measurement type: ${measurementType}`);
+      // Generic handler - just update position
+      this.updatePosition();
+      break;
   }
+}
 
-  //---------------- END child stuff -------------//
+//---------------- END child stuff -------------//
 
-  // Method to assess drift using errorMetrics
+    // Method to assess drift using errorMetrics
-  assessDrift(measurement, processMin, processMax) {
+    assessDrift(measurement, processMin, processMax) {
-    this.logger.debug(`Assessing drift for measurement: ${measurement} processMin: ${processMin} processMax: ${processMax}`);
+      this.logger.debug(`Assessing drift for measurement: ${measurement} processMin: ${processMin} processMax: ${processMax}`);
-    const predictedMeasurement = this.measurements.type(measurement).variant("predicted").position("downstream").getAllValues().values;
+      const predictedMeasurement = this.measurements.type(measurement).variant("predicted").position("downstream").getAllValues().values;
-    const measuredMeasurement = this.measurements.type(measurement).variant("measured").position("downstream").getAllValues().values;
+      const measuredMeasurement = this.measurements.type(measurement).variant("measured").position("downstream").getAllValues().values;
 
-    if (!predictedMeasurement || !measuredMeasurement) return null;
+      if (!predictedMeasurement || !measuredMeasurement) return null;
-    
+      
-    return this.errorMetrics.assessDrift(
+      return this.errorMetrics.assessDrift(
-      predictedMeasurement,
+        predictedMeasurement,
-      measuredMeasurement,
+        measuredMeasurement,
-      processMin,
+        processMin,
-      processMax
+        processMax
-    );
+      );
-  }
+    }
 
   reverseCurve(curve) {
     const reversedCurve = {};
@@ -500,36 +513,26 @@ class Machine {
 
   // NEW: Flow handler
   updateMeasuredFlow(value, position, context = {}) {
-
     if (!this._isOperationalState()) {
       this.logger.warn(`Machine not operational, skipping flow update from ${context.childName || 'unknown'}`);
       return;
     }
 
     this.logger.debug(`Flow update: ${value} at ${position} from ${context.childName || 'child'}`);
-
-    if (this.upstreamSource && this.downstreamSink) {
-      this.updateSourceSink();
-    }
     
     // Store in parent's measurement container
     this.measurements.type("flow").variant("measured").position(position).value(value, context.timestamp, context.unit);
     
     // Update predicted flow if you have prediction capability
     if (this.predictFlow) {
-      this.measurements.type("flow").variant("predicted").position("atEquipment").value(this.predictFlow.outputY || 0);
+      this.measurements.type("flow").variant("predicted").position("downstream").value(this.predictFlow.outputY || 0);
     }
   }
 
-  updateSourceSink() {
-    // Handles flow according to the configured "flow number"
-    this.logger.debug(`Updating source-sink pair: ${this.upstreamSource.config.functionality.softwareType} - ${this.downstreamSink.config.functionality.softwareType}`);
-    this.downstreamSink.setInfluent = this.upstreamSource.getEffluent[this.config.flowNumber];
-  }
-
   // Helper method for operational state check
   _isOperationalState() {
     const state = this.state.getCurrentState();
+    this.logger.debug(`Checking operational state ${this.state.getCurrentState()} ? ${["operational", "accelerating", "decelerating"].includes(state)}`);
     return ["operational", "accelerating", "decelerating"].includes(state);
   }
 
@@ -553,6 +556,9 @@ class Machine {
       this.calcDistanceBEP(efficiency,cog,minEfficiency);
 
     }
+
+    
+
   }
 
   calcDistanceFromPeak(currentEfficiency,peakEfficiency){
@@ -583,7 +589,6 @@ class Machine {
     };
   }
 
-
    // Calculate the center of gravity for current pressure
   calcCog() {