<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not really sure we really gain something with this patch, specially since ms_power is probably also stored the same way and you are then leaving each one with different metrics.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">In my opinion we should not store weird wire encoded formats, requiring code duplication of numeric conversion calculations all over the place. The conversions should be at those places that define that they require a weird format, as it is now: the RSL encoding needs divided-by-two, so apply it there. The meas_vis is (unfortunately) defined as div-two, so apply it there, and so on. But internally we now just store the plain dB value and calculate with it without duplicating the wire decoding step everywhere.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Vadim tells me that ms_power is some magic weird value and I'm not going to take that one on now, because I am working on BS Power.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24421">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24421">change 24421</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-bsc/+/24421"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I229db1d6bcf532af95aff56b2ad18b5ed9d81616 </div>
<div style="display:none"> Gerrit-Change-Number: 24421 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 27 May 2021 14:40:33 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>