Legacy unit prefixes for bit rate

[akosladanyi]

https://github.com/jdegre/5GC_APIs/blob/e14e017101b2d25f6e1bdb7af851d709c89eadbe/TS29571_CommonData.yaml#L2470-L2473

The units for bit rate seem to use a legacy notation: bps, Kbps, Mbps, Gbps, Tbps [1].

These should be bps, Kibps, Mibps, Gibps, Tibps (multiplier = 1024) or bps, kbps, Mbps, Gbps, Tbps (multiplier = 1000).

[1] https://en.wikipedia.org/wiki/Binary_prefix

[jdegre]

Thanks for the feedback @akosladanyi

The intention is to have these units as "multiplier = 1000", so the only one that seems to be wrong is "Kbps" that should've been "kbps".

Now, the problem is how to fix this in a backwards-compatible way. I guess the simplest way would be to add a clarification in the description indicating that "Kbps" is used by mistake to actually mean the same thing as "kbps".

Simply adding "kbps" as an allowed unit in the regex pattern would not work, I'm afraid, since a sender using this "new" unit format will not interoperate with an older receiver which only understands "Kbps".

I'll make sure the issue is discussed at next 3GPP meeting, and we'll see how to address this problem. Thanks for reporting it!