Msg Difference Per Period #73

DanielMwendwa · 2020-03-02T13:12:36Z

…periods (the time period for firebase is a constant number)

…rameter

… message difference between two firebase time periods'

mno_analysis_tools/compute_msg_difference_btwn_periods.py

Co-Authored-By: Isaack Mwenda <[email protected]>

…target_message_direction} message direction' Co-Authored-By: Isaack Mwenda <[email protected]>

Co-Authored-By: Isaack Mwenda <[email protected]>

… output file

…nce_per_two_firebase_period_output_file_path

…od with is constant

… the console

' script to 'compute_msg_difference_btwn_two_firebase_time_periods.py'

This script has been renamed

…s/RapidProTools into msg_diff_per_period

IsaackMwenda

Hi Daniel lets test this over a subset test dataset before merging

mno_analysis_tools/compute_msg_difference_btwn_two_firebase_time_periods.py

…case & update time period to firebase time period Co-Authored-By: Isaack Mwenda <[email protected]>

…_period Co-Authored-By: Isaack Mwenda <[email protected]>

Co-Authored-By: Isaack Mwenda <[email protected]>

DanielMwendwa added 3 commits March 2, 2020 16:11

added script to compute message difference between two firebase time …

e4e35bc

…periods (the time period for firebase is a constant number)

copied and unmodified code from computed msg per period PR

0951c8b

remove writing to msg per period file and CLI file name arg

97341ee

DanielMwendwa force-pushed the msg_diff_per_period branch from 8f31a9d to 97341ee Compare March 3, 2020 20:11

DanielMwendwa added 5 commits March 3, 2020 23:13

updated 'log.set_project_name' & ArgumentParser description

f435c70

added comment on computing message difference code block

cd62415

removed forward slash included unknowingly

deec56f

renamed filename variable to write computed output

7e2b2f2

updated python argparse argument to optional and specified default pa…

a379abf

…rameter

DanielMwendwa requested a review from IsaackMwenda March 4, 2020 14:35

updated argparse help msg for target operator to 'Operator to compute…

8b0b60c

… message difference between two firebase time periods'

IsaackMwenda requested changes Mar 5, 2020

View reviewed changes

DanielMwendwa and others added 16 commits March 5, 2020 14:58

Update log info from 'operator {target_operator}' -> '{target_operator}'

e0228a1

Co-Authored-By: Isaack Mwenda <[email protected]>

Update log info: 'message direction {target_message_direction}' to '{…

2db36c7

…target_message_direction} message direction' Co-Authored-By: Isaack Mwenda <[email protected]>

Updated comment from computing msg btwn 2 datetime to 2 firebase time

467189a

Co-Authored-By: Isaack Mwenda <[email protected]>

updated name of the variable containing the data we're writing to the…

265ee92

… output file

updated default timeframe to 10 seconds

75800a1

updated log info from writing to two files to one file

06531f1

renamed argparse output file output path argument to messages_differe…

bc99116

…nce_per_two_firebase_period_output_file_path

updated comment on computing number of mesages to state firebase peri…

3b3ea1e

…od with is constant

increased line width

ea9013a

removed unnecessary before a word

abf187f

format printing variable to console

ef49ee6

remove action argument from parser

8fefb1d

format printing variable to console

59b0ac0

updated log info to be consistent with the variables to be printed on…

f22dca4

… the console

remove information from log info specified ealier

fd92488

renamed 'compute_msg_difference_btwn_periods.py

8fc654c

' script to 'compute_msg_difference_btwn_two_firebase_time_periods.py'

DanielMwendwa requested a review from IsaackMwenda March 5, 2020 14:33

Delete compute_msg_difference_btwn_periods.py

5ee9242

This script has been renamed

DanielMwendwa added 2 commits March 5, 2020 17:56

Merge branch 'master' into msg_diff_per_period

5982e44

Merge branch 'msg_diff_per_period' of https://github.com/AfricasVoice…

422e3b5

…s/RapidProTools into msg_diff_per_period

IsaackMwenda approved these changes Mar 6, 2020

View reviewed changes

DanielMwendwa and others added 7 commits March 6, 2020 12:30

Update help message by removing white space, changing 'json' to upper…

3de7992

…case & update time period to firebase time period Co-Authored-By: Isaack Mwenda <[email protected]>

rename variable messages_per_period -> messages_per_two_firebase_time…

ddd2f8d

…_period Co-Authored-By: Isaack Mwenda <[email protected]>

updated variable containing data to write to output file

41fe232

Co-Authored-By: Isaack Mwenda <[email protected]>

Merge branch 'master' into msg_diff_per_period

604f50d

updated output file pathname variable

7e68cd7

updated variable reference after being updated earlier

f1eaf1c

match variable name to the comment

eed729b

DanielMwendwa merged commit 70bda1f into master Mar 6, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Msg Difference Per Period #73

Msg Difference Per Period #73

DanielMwendwa commented Mar 2, 2020

IsaackMwenda left a comment

Msg Difference Per Period #73

Msg Difference Per Period #73

Conversation

DanielMwendwa commented Mar 2, 2020

IsaackMwenda left a comment

Choose a reason for hiding this comment