<p>fixeria has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmocom-bb/+/16091">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">trx_toolkit/data_msg.py: fix: NOPE.ind also contains C/I filed<br><br>Change-Id: I281fb7387a83fec7e097ebf8650c95713d3f70e9<br>---<br>M src/target/trx_toolkit/data_msg.py<br>1 file changed, 6 insertions(+), 9 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/91/16091/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/target/trx_toolkit/data_msg.py b/src/target/trx_toolkit/data_msg.py</span><br><span>index ec59b85..c96dddb 100644</span><br><span>--- a/src/target/trx_toolkit/data_msg.py</span><br><span>+++ b/src/target/trx_toolkit/data_msg.py</span><br><span>@@ -630,7 +630,8 @@</span><br><span>                 if self.toa256 < self.TOA256_MIN or self.toa256 > self.TOA256_MAX:</span><br><span>                     raise ValueError("ToA256 %d is out of range" % self.toa256)</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-               if self.ver >= 0x01:</span><br><span style="color: hsl(120, 100%, 40%);">+               # Version specific parameters (omited for NOPE.ind)</span><br><span style="color: hsl(120, 100%, 40%);">+           if self.ver >= 0x01 and not self.nope_ind:</span><br><span>                        if type(self.mod_type) is not Modulation:</span><br><span>                            raise ValueError("Unknown Rx modulation type")</span><br><span> </span><br><span>@@ -650,6 +651,8 @@</span><br><span>                   if self.tsc not in self.TSC_RANGE:</span><br><span>                           raise ValueError("TSC %d is out of range" % self.tsc)</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+           # Version specific parameters (also present in NOPE.ind)</span><br><span style="color: hsl(120, 100%, 40%);">+              if self.ver >= 0x01:</span><br><span>                      if self.ci is None:</span><br><span>                          raise ValueError("C/I is not set")</span><br><span> </span><br><span>@@ -779,10 +782,7 @@</span><br><span>                      buf.append(mts)</span><br><span> </span><br><span>                  # C/I: Carrier-to-Interference ratio (in centiBels)</span><br><span style="color: hsl(0, 100%, 40%);">-                     if not self.nope_ind:</span><br><span style="color: hsl(0, 100%, 40%);">-                           buf += struct.pack(">h", self.ci)</span><br><span style="color: hsl(0, 100%, 40%);">-                  else:</span><br><span style="color: hsl(0, 100%, 40%);">-                           buf += bytearray(2)</span><br><span style="color: hsl(120, 100%, 40%);">+                   buf += struct.pack(">h", self.ci)</span><br><span> </span><br><span>           return buf</span><br><span> </span><br><span>@@ -799,10 +799,7 @@</span><br><span>                        self.parse_mts(hdr[8])</span><br><span> </span><br><span>                   # C/I: Carrier-to-Interference ratio (in centiBels)</span><br><span style="color: hsl(0, 100%, 40%);">-                     if not self.nope_ind:</span><br><span style="color: hsl(0, 100%, 40%);">-                           self.ci = struct.unpack(">h", hdr[9:11])[0]</span><br><span style="color: hsl(0, 100%, 40%);">-                        else:</span><br><span style="color: hsl(0, 100%, 40%);">-                           self.ci = None</span><br><span style="color: hsl(120, 100%, 40%);">+                        self.ci = struct.unpack(">h", hdr[9:11])[0]</span><br><span> </span><br><span>         # Generates message specific burst</span><br><span>   def gen_burst(self):</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmocom-bb/+/16091">change 16091</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/osmocom-bb/+/16091"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmocom-bb </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I281fb7387a83fec7e097ebf8650c95713d3f70e9 </div>
<div style="display:none"> Gerrit-Change-Number: 16091 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>