-
Notifications
You must be signed in to change notification settings - Fork 90
Description
I would like to propose a couple of minor additions for electrical metering purposes...
-
Add a class brick:Electrical_Energy_Sensor, as a subclass of brick:Energy_Sensor.
-
Add "peak" to the list of literals in brick:AggregationShape (i.e. allow "peak" to be the object of a triple with predicate brick:aggregationFunction).
-
Rename brick:PowerComplexityShape to be brick:ElectricalComplexityShape, and rename brick:PowerFlowShape to be brick:ElectricalFlowShape.
Rationale for these changes...
-
An Electrical_Energy_Sensor is a mathematical integration of Electrical_Power_Sensor, and is a type that is more specific than brick:Energy_Sensor. It's a type that we have required in our project work, and which we have used internally in our own patches to Brick.
-
Peak aggregation was a project requirement we encountered, and it seems like a common need others will have. This allows a modeller to set an aggregation over any metering Point, rather than being limited to brick:Peak_* classes.
-
This is just a minor detail about clarity. These shapes are not really specific to 'power' as distinct from energy, but concern electricity in general (whether energy or power). I know renaming classes is discouraged, but these shapes are 'internal' in the sense that they're used by automated validation tools rather than being referenced in models, so I think this refactoring should be safe.
Please let me know what you think!