<p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25087">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-bsc/+/25087/2/doc/manuals/chapters/bts.adoc">File doc/manuals/chapters/bts.adoc:</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-bsc/+/25087/2/doc/manuals/chapters/bts.adoc@482">Patch Set #2, Line 482:</a> <code style="font-family:monospace,monospace">`access-control-class-ramping-chan-load` VTY command.</code></p><ul><li>"will actually be checked always" sounds a bit strange to me, how about "will always be checked"</li><li>"the" missing before "start up procedure", before `access-control-class-ramping-step-interval` and before `access-control-class-ramping-chan-load`</li><li>This sentence is very long and therefore hard to read. How about:</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">... VTY command. It will either keep, ...</p><p style="white-space: pre-wrap; word-wrap: break-word;">instead of</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> ... VTY command, and will either keep, ...</pre></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25087/2/doc/manuals/chapters/bts.adoc@526">Patch Set #2, Line 526:</a> <code style="font-family:monospace,monospace"><6> The time until the subset size is further increased or decreased based on current channel load check (in seconds)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">In the other lines, the exact config value is mentioned instead of a generic description.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So here I would put the 30s, e.g.</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  <6> 30 seconds until the subset size is further increased or decreased based on the current channel load check</pre></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25087">change 25087</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/+/25087"/><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: I8701386b35ccdebfc843fd00535091648e52f739 </div>
<div style="display:none"> Gerrit-Change-Number: 25087 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 28 Jul 2021 16:40:47 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>