updated the patch. PFA<div><br></div><div>Do you think its necessary to update the file in trac ?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Dec 20, 2012 at 8:50 AM, Mohammed Rashad <span dir="ltr"><<a href="mailto:mohammedrashadkm@gmail.com" target="_blank">mohammedrashadkm@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Anna,<div><br></div><div>Adding  a check for ultimatelistctrl widget and if not available can be done. But remember using the old implementation will crash wxGUI.</div>

<div><br></div><div>How about If no ultimatelistctrl is not available we can add a copy of it our own with a warning</div>
<div><br></div><div>Btw, I dont have a source build of wxpython. This comes with Ubuntu 12.04 LTS</div><div><br></div><div><br></div></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">On Thu, Dec 20, 2012 at 4:56 AM, Anna Kratochvílová <span dir="ltr"><<a href="mailto:kratochanna@gmail.com" target="_blank">kratochanna@gmail.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On Wed, Dec 19, 2012 at 4:39 PM, Mohammed Rashad<br>
<<a href="mailto:mohammedrashadkm@gmail.com" target="_blank">mohammedrashadkm@gmail.com</a>> wrote:<br>
> Hi All,<br>
><br>
> gui/wxpython/modules/colorrules.py, providing inteface for setting<br>
> colortable for raster and vector.<br>
><br>
> As I reported on trac[1] several weeks ago there is a bug in the code which<br>
> happens if no. color rules for raster map is big (eg: 32763 This for<br>
> nc_spm_08/landsat 4,3,2 color composite) . This will result in crashing the<br>
> whole wxGUI!<br>
><br>
> The problem is for each rule colorrules.py create a checkbox,textCtrl and<br>
> ColorSelect.<br>
><br>
> Creating these widgets will hangup the UI.<br>
><br>
> A solution is to use a list control agw provides one such thing called<br>
> ultimatelistCtrl.<br>
><br>
> I had updated the colorrules.py and made a patch for it. Anyone Please<br>
> review and update the file<br>
<br>
</div>Hi Rashad,<br>
<br>
I can't even test your patch at the moment because UltimateListCtrl is<br>
not supported in wxPython 2.8.10.1 (which is installed on my<br>
computer). It seems that it is available from 2.8.11, however in the<br>
grass requirements, there is wxPython >= 2.8.1.1. I agree that this<br>
one is pretty old but still we must decide if it is worth changing the<br>
requirements. I think that the version I have is still used a lot so<br>
we should support it. You are using this widget to show the color I<br>
suppose. I am not aware of any other good solution. Maybe you can a<br>
condition on wxPython version and if it is high enough use this<br>
widget, otherwise use the current implementation?<br>
<br>
Anna<br>
<div><br>
><br>
> PFA patch<br>
><br>
> [0] <a href="http://trac.osgeo.org/grass/ticket/1816" target="_blank">http://trac.osgeo.org/grass/ticket/1816</a><br>
><br>
><br>
> --<br>
> Regards,<br>
>    Rashad<br>
><br>
><br>
</div>> _______________________________________________<br>
> grass-dev mailing list<br>
> <a href="mailto:grass-dev@lists.osgeo.org" target="_blank">grass-dev@lists.osgeo.org</a><br>
> <a href="http://lists.osgeo.org/mailman/listinfo/grass-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/grass-dev</a><br>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div><font face="arial, helvetica, sans-serif">Regards,<br>   Rashad</font></div><br>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div><font face="arial, helvetica, sans-serif">Regards,<br>   Rashad</font></div><br>
</div>