<div dir="ltr">I am not understanding why you want to use threshold values to calculate new sensor values. We can certainly add this feature but it can not be based on name rather we have to think differently to get. It would really help to decide on design once we know it's actual usage. I can't think of any situation here.<div><br></div><div>Regards</div><div>-Vijay</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Dec 29, 2020 at 6:49 PM Harvey Wu (吳崇維) <<a href="mailto:Harvey.Wu@quantatw.com">Harvey.Wu@quantatw.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> On Wed, Dec 30, 2020 at 10:19 AM Harvey Wu (吳崇維)<br>
> <<a href="mailto:Harvey.Wu@quantatw.com" target="_blank">Harvey.Wu@quantatw.com</a>> wrote:<br>
> ><br>
> > Hi All,<br>
> ><br>
> ><br>
> ><br>
> > The current phosphor-virtual-sensor DbusParam value is from<br>
> DBus “Value” property which interface is<br>
> “xyz.openbmc_project.Sensor.Value”.<br>
> ><br>
> > Because I want to get the property “CriticalHigh” value on DBus, could I<br>
> add a new feature “Interface” property in json file? The property can be<br>
> “CriticalHigh” , “CriticalLow” , “WarningHigh” or “WarningLow”.<br>
> <br>
> <br>
> According to<br>
> <a href="https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub" rel="noreferrer" target="_blank">https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub</a>.<br>
> com%2Fopenbmc%2Fphosphor-virtual-sensor%2Fblob%2Fmaster%2Fvirtual_<br>
> sensor_config.json%23L8-L14&data=04%7C01%7CHarvey.Wu%40quantat<br>
> <a href="http://w.com" rel="noreferrer" target="_blank">w.com</a>%7C1479ebd5aedd4b8a7c3308d8ac6a4393%7C179b032707fc4973ac73<br>
> 8de7313561b2%7C1%7C0%7C637448919693850853%7CUnknown%7CTWFpb<br>
> GZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6<br>
> Mn0%3D%7C1000&sdata=0KyrnRkZKr9M59DEg%2BnyUruiege%2B3EDXb<br>
> mxzR%2B6xeng%3D&reserved=0,<br>
> the threshold properties are already supported.<br>
<br>
The "Threshold" in json seems only can set the sensor critical or warning value, but I want to use the critical or warning value in "Expression".<br>
After check the newest commit, this function seems doesn't implement in current source code.<br>
<br>
> <br>
> --<br>
> BRs,<br>
> Lei YU<br>
<br>
<br>
Harvey Wu<br>
</blockquote></div>