<div dir="ltr">Hi Sriram,<div><br></div><div>Those changes sound great, yes I would definitely love to see the patches (ideally split by separate feature if that is easy).</div><div><br></div><div> - Daniel</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Apr 30, 2013 at 8:23 AM, Murali, Sriram <span dir="ltr"><<a href="mailto:sriram.murali@intel.com" target="_blank">sriram.murali@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">






<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class="MsoNormal">Hi Daniel,<u></u><u></u></p>
<p class="MsoNormal">I made some changes to the LNT perf reporting tool to make it more user friendly by adding some features:<u></u><u></u></p>
<p><u></u><span>1.<span style="font:7.0pt "Times New Roman"">     
</span></span><u></u>Make the sidebar and the navigation bar stationary, so that it is easy to navigate the site<u></u><u></u></p>
<p><u></u><span>2.<span style="font:7.0pt "Times New Roman"">     
</span></span><u></u>Have the pop-down menu for the items in the navigation bar, activate upon hovering the mouse, rather than clicking the item<u></u><u></u></p>
<p><u></u><span>3.<span style="font:7.0pt "Times New Roman"">     
</span></span><u></u>Add a nav-link in the sidebar for the graph button, so that it is easy to navigate to the bottom of the page in order to graph<u></u><u></u></p>
<p><u></u><span>4.<span style="font:7.0pt "Times New Roman"">     
</span></span><u></u>Add drop down boxes for Runs and Compare to (in the sidebar) that pops-down upon hover, rather than links to 4 neighboring runs.<u></u><u></u></p>
<p><u></u><span>5.<span style="font:7.0pt "Times New Roman"">     
</span></span><u></u>Make the execution_time  and compilation_time results table as sortable and a checkbox to select all the items in the table.
<u></u><u></u></p>
<p style="margin-left:1.0in">
<u></u><span>a.<span style="font:7.0pt "Times New Roman"">     
</span></span><u></u>Clicking the workload url in the table will open a new window instead of a tab, for simpler navigation.<u></u><u></u></p>
<p style="margin-left:1.0in">
<u></u><span>b.<span style="font:7.0pt "Times New Roman"">     
</span></span><u></u>Next Steps: Add a pop-up to display the graph upon hovering over the workload url, instead of opening a new window.<u></u><u></u></p>
<p style="margin-left:1.0in">
<u></u><span>c.<span style="font:7.0pt "Times New Roman"">     
</span></span><u></u>Next Steps: Split the results table into multiple sub-tables depending upon the suite (MultiSource/Applications, …SingleSource/UnitTest) and make use of select-all checkbox and graph all the workloads in the corresponding suite.<u></u><u></u></p>

<p style="margin-left:1.0in">
<u></u><span>d.<span style="font:7.0pt "Times New Roman"">     
</span></span><u></u>Next Steps: Add a Geomean button to plot the geomean of all the workloads instead of separate plots for each workload.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">If you feel that some of these improvements are valuable, then I can create patches and upload them in Phabricator for review.<u></u><u></u></p>
<p class="MsoNormal">Thanks!<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">With regards,<u></u><u></u></p>
<p class="MsoNormal">Sriram<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">--<u></u><u></u></p>
<p class="MsoNormal">Sriram Murali<u></u><u></u></p>
<p class="MsoNormal">SSG/DPD/ECDL/DMP<u></u><u></u></p>
<p class="MsoNormal"><a href="tel:%2B1%20%28519%29%20772%20%E2%80%93%202579" value="+15197722579" target="_blank">+1 (519) 772 – 2579</a><u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>

</blockquote></div><br></div>