matplotlib-users

[Matplotlib-users] mpl 0.99 and py2exe

[Werner F. B. <wer...@fr...>]

I have run into a bit of problem using 0.99 and py2exe.

I am getting errors "TypeError: unsupported operand type(s) for +=: 
'NoneType' and 'str'" in mlab.pyo after py2exe'd my application.

This is caused by this type of code, as one normally uses the optimize 
option with py2exe which means that the docs are stripped from the .pyo 
files.
"psd.__doc__ = psd.__doc__ % kwdocd"

I changed it to:
if psd.__doc__ is not None:
    psd.__doc__ = psd.__doc__ % kwdocd
else:
    psd.__doc__ = ""

Above is a bit of a hack, if someone can suggest how to correct this in 
a way which would get accepted as a patch I would search all modules 
correct them.

Werner

Re: [Matplotlib-users] mpl 0.99 and py2exe

[Werner F. B. <wer...@fr...>]

Werner F. Bruhin wrote:
> I have run into a bit of problem using 0.99 and py2exe.
>
> I am getting errors "TypeError: unsupported operand type(s) for +=: 
> 'NoneType' and 'str'" in mlab.pyo after py2exe'd my application.
>
> This is caused by this type of code, as one normally uses the optimize 
> option with py2exe which means that the docs are stripped from the .pyo 
> files.
> "psd.__doc__ = psd.__doc__ % kwdocd"
>
> I changed it to:
> if psd.__doc__ is not None:
>     psd.__doc__ = psd.__doc__ % kwdocd
> else:
>     psd.__doc__ = ""
>
> Above is a bit of a hack, if someone can suggest how to correct this in 
> a way which would get accepted as a patch I would search all modules 
> correct them.
>   
Maybe a nicer solution would be:
if __debug__:
    psd.__doc__ = psd.__doc__ % kwdocd

Also that would mean that above is not run when running Python as 
"python -O script.py or python -OO script.py where the first solution 
means that with -O the it would still be executed.

Werner

Re: [Matplotlib-users] mpl 0.99 and py2exe

[Jouni K. S. <jk...@ik...>]

"Werner F. Bruhin" <wer...@fr...> writes:

>> I am getting errors "TypeError: unsupported operand type(s) for +=: 
>> 'NoneType' and 'str'" in mlab.pyo after py2exe'd my application.

I think this has been fixed on the trunk for good, by changing all
docstring modifications to use decorators (defined in docstring.py) that
check for nonexistent docstrings. The changes are perhaps too big to
apply on the 0.99 branch.

>> I changed it to:
>> if psd.__doc__ is not None:
>>     psd.__doc__ = psd.__doc__ % kwdocd
>> else:
>>     psd.__doc__ = ""
>>
>> Above is a bit of a hack, if someone can suggest how to correct this in 
>> a way which would get accepted as a patch I would search all modules 
>> correct them.

I think either this or something like

    psd.__doc__ = psd.__doc__ and (psd.__doc__ % kwdocd)

should be acceptable as a bugfix.

-- 
Jouni K. Seppänen
http://www.iki.fi/jks

Re: [Matplotlib-users] mpl 0.99 and py2exe

[Werner F. B. <wer...@fr...>]

Jouni K. Seppänen wrote:
> "Werner F. Bruhin" <wer...@fr...> writes:
>
>   
>>> I think this has been fixed on the trunk for good, by changing all
>>> docstring modifications to use decorators (defined in docstring.py) that
>>> check for nonexistent docstrings. The changes are perhaps too big to
>>> apply on the 0.99 branch.
>>>   
>>>       
>> As it is fixed in trunk is there a need/much use for a patch to 0.99?
>> I.e. how far of is the next release which includes what ever is in
>> trunk?
>>     
>
> I think John Hunter said somewhere that he plans to release 1.0 in a few
> months, but software release schedules are notoriously difficult to
> estimate, and with volunteer-driven open-source software even more so. I
> think the answer depends on how many people use py2exe, about which I
> have absolutely no idea.
>   
I don't think it is worse a patch as the work around is very easy.  I
documented what is needed for a py2exe setup.py with mpl 0.99 on the
py2exe site at the end of the following page
http://www.py2exe.org/index.cgi/MatPlotLib .

While doing this I noted one more problem if one uses backend_wx, the
wxPython version check will always fail if py2exe'd.  I added "if not
hasattr(sys, 'frozen'):" on line 113 and indented lines 114 to 131.
Hopefully someone could make this change for the next version of mpl.

Werner