Sveino / Inst4CIM-KG

Instance of CIM Knowledge Graph
Apache License 2.0
5 stars 1 forks source link

cim:ActivePowerPerFrequency.unit is wrong #76

Open VladimirAlexiev opened 2 months ago

VladimirAlexiev commented 2 months ago

See https://github.com/qudt/qudt-public-repo/issues/970 cim:ActivePowerPerFrequency is used with only one prop: cim:ExternalNetworkInjection.governorSCD "Power Frequency Bias. This is the change in power injection divided by the change in frequency and negated. A positive value of the power frequency bias provides additional power injection upon a drop in frequency."

So I've corrected the unit as follows:

delete {cim:ActivePowerPerFrequency.unit cims:isFixed "WPers"}
insert {cim:ActivePowerPerFrequency.unit cims:isFixed "WPerHz"}
where  {cim:ActivePowerPerFrequency.unit cims:isFixed "WPers"};
Sveino commented 2 months ago

If we make this change we would also need to update the EAP model. I know that this was discussed in TF13. I found the following issues:

However, I see for GeneratingUnit we are using PerCent:

https://github.com/Sveino/Inst4CIM-KG/blob/8a1846e8f766dd89a2ff5142d597344326b343ac/source/CGMES/v3.0/RDFS2020/61970-600-2_Equipment-AP-Voc-RDFS2020_v3-0-0.rdf#L4181-L4187

I assume that this is an error and we should update it as PerCent.

VladimirAlexiev commented 1 month ago
griddigit-ci commented 3 weeks ago

We will need CIM issue here I guess