Hello vvvelichkov,
I'd like you to do a code review. Please visit
https://gerrit.osmocom.org/c/gr-gsm/+/24082
to review the following change.
Change subject: Python 3 compatibility changes in fn_time.py
......................................................................
Python 3 compatibility changes in fn_time.py
- Use range instead of xrange as it has been removed
- Use integer division operator (//)
- Import uniform and fn_time_delta_cpp only when started as a script
- Remove the unused imports
Change-Id: I268e0aec8fb8e6d490bfb8b0e9e6d169a9f6c352
---
M python/misc_utils/fn_time.py
1 file changed, 4 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/gr-gsm refs/changes/82/24082/1
diff --git a/python/misc_utils/fn_time.py b/python/misc_utils/fn_time.py
index 17c6295..93342bd 100644
--- a/python/misc_utils/fn_time.py
+++ b/python/misc_utils/fn_time.py
@@ -20,9 +20,6 @@
# Boston, MA 02110-1301, USA.
#
#
-from math import floor, ceil
-from random import uniform
-from grgsm import fn_time_delta_cpp
__hyper_frame = 26*51*2048
__symb_rate = 13.0e6/48.0
@@ -72,10 +69,13 @@
if __name__ == "__main__":
+ from random import uniform
+ from grgsm import fn_time_delta_cpp
+
fn1 = 10000
ts_ref = 4
time1 = 10.5
- for fn2 in xrange(__hyper_frame/2+fn1-10,__hyper_frame/2*10+fn1+100,10):
+ for fn2 in range(__hyper_frame//2+fn1-10,__hyper_frame//2*10+fn1+100,10):
ts_x = int(uniform(0,8))
time2 = time1 + (fn2-fn1)*__frame_period + (ts_x-ts_ref)*__ts_period
error = uniform(-6200,6200)
--
To view, visit https://gerrit.osmocom.org/c/gr-gsm/+/24082
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: gr-gsm
Gerrit-Branch: master
Gerrit-Change-Id: I268e0aec8fb8e6d490bfb8b0e9e6d169a9f6c352
Gerrit-Change-Number: 24082
Gerrit-PatchSet: 1
Gerrit-Owner: ptrkrysik <ptrkry...@gmail.com>
Gerrit-Reviewer: vvvelichkov <vvvelich...@gmail.com>
Gerrit-MessageType: newchange
