<p style="white-space: pre-wrap; word-wrap: break-word;">the related ticket was a sysmocom internal one, so let me copy the remark here:</p><p style="white-space: pre-wrap; word-wrap: break-word;">"<br>we have plenty of other timers already configurable in the osmocom universe,<br>and we never suffix them with the unit. Rather, we document the unit in the<br>help so that if you press tab, you will get some information as to what the unit<br>is.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't argue the existing behavior is better than your implemented one, but for<br>the sake of consistency I would think it makes sense to align with the existing<br>behavior. See for example the various tXXXX that are configurable in BSC an MSC.<br>"</p><p style="white-space: pre-wrap; word-wrap: break-word;">I even tried to just copy the T... implementation from osmo-bsc, but it doesn't match. These timers here have names, not numbers. The T timers sometimes have really convoluted units that the user needs to familiarize with anyway, while all of these SCCP timers are a plain duration in "real time". A less important aspect: they are specified in ITU-T, not 3GPP.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I see the consistency POV, and actually also considered making everything one unit. That would be seconds then. After contemplating back and forth I ended up with this implementation instead.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If all are seconds, users then need to multiply/divide by 60 in the head for most timers, which are in the range of minutes.</p><p style="white-space: pre-wrap; word-wrap: break-word;">For the few shorter timers of about 10 seconds, I lose the ability to set sub-second timers. Not sure if it's relevant in practice, but for a value of a few seconds I thought it better to also allow higher resolution. But I definitely don't want to supply a timer of 11 minutes in milliseconds, UI wise. A timer as low as 20 minutes would already surpass a million.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If we defined longer-specified timers as in-seconds, and shorter ones as in-milliseconds. If you really want that I can do it, but I don't like the inconsistency/non-readability between the individual sccp timers then.</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  sccp-timer ias 300<br>  sccp-timer iar 660<br>  sccp-timer rel 10000</pre><p style="white-space: pre-wrap; word-wrap: break-word;">That really reads like "wtf".</p><p style="white-space: pre-wrap; word-wrap: break-word;">For me the best fixed unit choice then would be seconds for all timers, and losing the sub-second value range.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I agree that consistency is good, yet it doesn't really match / help here IMHO.</p><p style="white-space: pre-wrap; word-wrap: break-word;">How about optional units? All default to seconds, but for convenience a 'm' or 'ms' can be added to spare the factor of 60 / to allow sub-second values?</p><p><a href="https://gerrit.osmocom.org/11119">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11119">change 11119</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/11119"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-sccp </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I28a7362aa838e648ecc9b26ee53dbcade81a9d65 </div>
<div style="display:none"> Gerrit-Change-Number: 11119 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 27 Sep 2018 12:35:03 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>