<div dir="ltr"><div>Hi Jonathan,</div><div><br></div><div>Thanks a lot for the quick review and suggestions to improve the code readability.</div><div><br></div><div>We incorporated all the changes but didn't see any improvement in the results.</div><div><br></div><div>However, we finally noticed that it was the packet size used in our simulations that was affecting the throughput. It was earlier set to 1000 bytes, and after making it 1500 bytes (including headers) we note that the throughput is not affected.</div><div><br></div><div>We have uploaded the new graphs on the same wiki link:</div><div><a href="https://github.com/Daipu/COBALT/wiki/Light-Traffic" target="_blank">https://github.com/Daipu/COBALT/wiki/Light-Traffic</a></div><div><br></div><div>We're not sure why packet size is affecting the throughput so largely. Is it the expected behavior?</div><div><br></div><div>Thanks and Regards</div><div>Jendaipou Palmei</div><div>Shefali Gupta<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 27, 2018 at 8:07 PM Jonathan Morton <<a href="mailto:chromatix99@gmail.com">chromatix99@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> On 27 Nov, 2018, at 4:10 pm, Jendaipou Palmei <<a href="mailto:jendaipoupalmei@gmail.com" target="_blank">jendaipoupalmei@gmail.com</a>> wrote:<br>
> <br>
> We have made the changes in the code as suggested.<br>
> <br>
> Here is the diff after making the changes:<br>
> <br>
> <a href="https://github.com/Daipu/COBALT/commit/033db330287e2072bad94ac441f8aed774678a7d" rel="noreferrer" target="_blank">https://github.com/Daipu/COBALT/commit/033db330287e2072bad94ac441f8aed774678a7d</a><br>
> <br>
> Additionally, we have also updated the values of 'Pincrement' and 'Pdecrement' (parameters of BLUE) in ns-3 to match the ones used in the Cake implementation in Linux.<br>
> <br>
> Does the diff look correct?<br>
<br>
It does look like an improvement.  How's the behaviour?<br>
<br>
NB: the decimal values used in the constants are not quite precise equivalents of the Linux values, since you have dropped some of the trailing digits.  You could use (1./256) and (1./4096) to both make them precise and more human-readable.<br>
<br>
 - Jonathan Morton<br>
<br>
</blockquote></div>