<p style="white-space: pre-wrap; word-wrap: break-word;">Very good work. This worked out of the box for me in our setup. There are a few minor suggestions from my side. Please have a look.</p><p>Patch set 3:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20521">View Change</a></p><p>4 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/+/20521/3/src/osmo_gsm_tester/obj/rfemu_gnuradio_zmq.py">File src/osmo_gsm_tester/obj/rfemu_gnuradio_zmq.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/+/20521/3/src/osmo_gsm_tester/obj/rfemu_gnuradio_zmq.py@192">Patch Set #3, Line 192:</a> <code style="font-family:monospace,monospace">                self.broker.cmd_set_relative_gain_on_local_port(cell['zmq_enb_peer_port'], (max_att_db - db)/max_att_db)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">i'll provide an update for this calculation</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20521/3/src/osmo_gsm_tester/obj/rfemu_gnuradio_zmq.py@196">Patch Set #3, Line 196:</a> <code style="font-family:monospace,monospace">        return 12 # maximum cell_gain value in srs. Is this correct value?</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">also here</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20521/3/sysmocom/scenarios/mod-enb-cells-2ca.conf">File sysmocom/scenarios/mod-enb-cells-2ca.conf:</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/+/20521/3/sysmocom/scenarios/mod-enb-cells-2ca.conf@12">Patch Set #3, Line 12:</a> <code style="font-family:monospace,monospace">     dl_earfcn: 2850</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">this causes conflicts with my recent update, but we need to add a new scenario anyway</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20521/3/utils/bin/osmo-gsm-tester_zmq_broker.py">File utils/bin/osmo-gsm-tester_zmq_broker.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/+/20521/3/utils/bin/osmo-gsm-tester_zmq_broker.py@19">Patch Set #3, Line 19:</a> <code style="font-family:monospace,monospace">class GrBroker(gr.top_block):</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">this script works well, however, I am a bit worried about the current file location though. I expect to have at least two or three broker scripts in the future, if no more. Would it be possible to move it to the scenario folder so customization are possible?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20521">change 20521</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/+/20521"/><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: I6282cda400558dcb356276786d91e6388524c5b1 </div>
<div style="display:none"> Gerrit-Change-Number: 20521 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: srs_andre <andre@softwareradiosystems.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 14 Oct 2020 13:52:58 +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>