Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(4041)

Issue 339960043: Iterate through multiconfigurations of propogation delay (FS)

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 years, 4 months ago by tonythomas
Modified:
6 years, 4 months ago
Reviewers:
david.tomic11
CC:
dkostic, kirill.sc, seregibenjamin, tonythomas, cynthiamao122_gmail.com, liangzijie1437_gmail.com
Visibility:
Public.

Description

Iterate through multiconfigurations of propogation delay (FS) BUG=REP-

Patch Set 1 #

Patch Set 2 : Repeat and plot for mulitple propogation deltas #

Patch Set 3 : Merge branch 'master' into multiconf-delta #

Patch Set 4 : Add distribution to jitter experiments, fix spellbee #

Patch Set 5 : Fix spell bee in flags.multiconf_delta #

Unified diffs Side-by-side diffs Delta from patch set Stats (+93 lines, -16 lines) Patch
M experiments/setups/configuration.json View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M launch_lx3.py View 1 2 3 4 12 chunks +92 lines, -15 lines 0 comments Download

Messages

Total messages: 7
tonythomas
Co-authored with Ben, David.
6 years, 4 months ago (2017-12-05 16:28:46 UTC) #1
tonythomas
this would allow us to explain why we chose propogation delta constant.
6 years, 4 months ago (2017-12-12 14:45:28 UTC) #2
david.tomic11
+1 LGTM
6 years, 4 months ago (2017-12-12 14:48:38 UTC) #3
tonythomas
Ready for review. Untested though.
6 years, 4 months ago (2017-12-12 14:57:16 UTC) #4
tonythomas
tested.
6 years, 4 months ago (2017-12-12 15:01:36 UTC) #5
tonythomas
Merge it when ready: https://gits-15.sys.kth.se/REP/lx3/pull/36/
6 years, 4 months ago (2017-12-12 15:02:01 UTC) #6
david.tomic11
6 years, 4 months ago (2017-12-12 15:02:21 UTC) #7
On 2017/12/12 15:01:36, tonythomas wrote:
> tested.

merging now.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b