<p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +2</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/17583">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/17583/2/src/osmo_gsm_tester/srs_enb.py">File src/osmo_gsm_tester/srs_enb.py:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/17583/2/src/osmo_gsm_tester/srs_enb.py@253">Patch Set #2, Line 253:</a> <code style="font-family:monospace,monospace">    def num_cells(self):</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Avoid adding getter for no good reason if nobody's going to use them. Let's try keeping the APIs at the minimum necessary so that other object implementation don't need to provide them too later.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/17583/2/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl">File src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/17583/2/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl@89">Patch Set #2, Line 89:</a> <code style="font-family:monospace,monospace">% if enb.get('num_cells') == '2':</code></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">We could change all this into a for loop at some point. Fyi counter of the loop is available as ${loop.index}. Eg:<br>%for trx in osmo_bts_octphy.trx_list:<br> trx ${loop.index}<br>%endfor</pre></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/17583">change 17583</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-gsm-tester/+/17583"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-gsm-tester </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I6d1658bf820163aee03b64b0f4ac68046c243f37 </div>
<div style="display:none"> Gerrit-Change-Number: 17583 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: srs_andre <andre@softwareradiosystems.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 24 Mar 2020 14:09:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>