
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN">

<html><body>

<p>Le 2014-05-20 23:02, Burlen Loring a écrit :</p>

<blockquote type="cite" style="padding-left:5px; border-left:#1010ff 2px solid; margin-left:5px"><!-- html ignored --><!-- head ignored --><!-- meta ignored -->Hi All,<br /><br /> I really love the massif-visualizer, it's been tremendously helpful digging into memory consumption issues in a complex application that has to run for long periods of time!!<br /><br /> I found that the legend could occlude a fairly large portion of the chart making it hard to see and interact with the chart. Since the project is so helpful to me I wanted to contribute. I've added ability to control the placement of the legend and change it's font size. For instance it could be placed beside the chart now. I also added a "save as" menu item so that the charts can be saved to an image for sharing with collaborators. <br /><br /> here are images produced using the new "save as" feature showing the original issue and new placement control, and a screen shot of the settings dialog.<br /><a href="http://www.hpcvis.com/vis/images/massif-vis/original-legend-floating-12pt.png">original</a>, <a href="http://www.hpcvis.com/vis/images/massif-vis/with-legend-east-top-10pt.png">legend-east-top-10pt</a>, <a href="http://www.hpcvis.com/vis/images/massif-vis/settings.png">settings</a><br /><br /> I'm sending git style patches with this mail, which I hope could be reviewed and applied assuming they are up to snuf. I hope that these are welcome. If there's a better way to submit changes for review, please point me to it.<br /><br /> Thanks<br /> Burlen<br /><!-- html ignored --><br />

<pre>_______________________________________________

Massif-visualizer mailing list

<a href="mailto:Massif-visualizer@kde.org">Massif-visualizer@kde.org</a>

<a href="https://mail.kde.org/mailman/listinfo/massif-visualizer">https://mail.kde.org/mailman/listinfo/massif-visualizer</a>

</pre>

</blockquote>

<p>Hi,<br /> <br /> First, thanks for contributing to massif-visualizer.<br /> <br /> I'm not the one to say wether your patches could be accepted or not, but I find the idea of moving the legend pleasant. I haven't tested them yet, but I already saw some little details that should be addressed, such as missing i18n() calls.<br /> <br /> Btw, the place to submit patches is: git.reviewboard.kde.org. It's easier to review them there.<br /> <br /> Let's wait for Milian's answer. ;)<br /> <br /> Thanks,<br /> Arnold</p>

<div> </div>

</body></html>