srsran / srsRAN_4G

Open source SDR 4G software suite from Software Radio Systems (SRS) https://docs.srsran.com/projects/4g
https://www.srsran.com
GNU Affero General Public License v3.0
3.44k stars 1.13k forks source link

srsenb stdout without space #645

Closed avalori1 closed 2 years ago

avalori1 commented 3 years ago

Issue Description

The output for srsenb in some cases is missing a space between adiacent metrics. This makes parsing automatically tedious (and possibly even ambiguous). For example, splitting a line may lead to ['46', '15', '0', '8.0', '392', '1', '0', '0%', '12.4', '36.010.0', '4.3k', '1', '0', '0%', '0.0'] where 36.010.0 may be interpreted as one of the two options:

  1. 36.01 and 0.0
  2. 36.0 and 10.0

Setup Details

I am running the below, but seems that is happening with all the versions... Built in Release mode using commit 45486b6e2 on branch master.

Expected Behavior

Having a space in between the metric whatever are the numerical values

Actual Behaviour

As described above, in some cases the space is missing

Steps to reproduce the problem

Just run srsenb extensively, with different UE and/or conditions.

Additional Information

Additionally, to even further simplify the parsing, would be good to have unique column names (e.g. brateU and brateD for the two), rather than (or in addition to) having the DL and UL descriptions above, which are easy to read by eye, but less when parsing the text file automatically.

andrepuschmann commented 3 years ago

Thanks for the reporting. We are aware of the issue and look into it.

faluco commented 3 years ago

This has been fixed and will be available for the next release.

Also included for next release is a JSON report file that will be more suitable for parsing.

andrepuschmann commented 2 years ago

All this is now available in the official release.