incompatabilities between 'intro numpy' and 'image processing' about scientific-python-lectures HOT 8 CLOSED

On Fri, Aug 26, 2011 at 03:04:33AM -0700, esc wrote:

it seems like some of the commits for the 'intro numpy' break the 'image processing'.

We discussed this a bit with @pv, and I believe that he is working on
fixing that. After this is fixed, we can cherry pick from emmanuelle's
branch.

from scientific-python-lectures.

Two alternatives:

• make the new plot directive to work like the old one: http://github.com/pv/scipy-lecture-notes/tree/newplot
• revert to the old one: http://github.com/pv/scipy-lecture-notes/tree/oldplot

from scientific-python-lectures.

• pv [email protected] [110826]:

  Two alternatives:

  • make the new plot directive to work like the old one: http://github.com/pv/scipy-lecture-notes/tree/newplot
  • revert to the old one: http://github.com/pv/scipy-lecture-notes/tree/oldplot

I think the issue was also the plot settings in conf.py (5a387d9).
@emmanuelle what was the exact problem?

V-

from scientific-python-lectures.

Oh, sorry, i just now saw that you have removed that exact code in both options. Next step is probably to test this with the image processing tutorial.

from scientific-python-lectures.

Hi,

i made a branch pu:

This contains the oldplot branch. We (Gael + I) decided to use the old version of the plot as an interim solution. In the long run we would however like to get away from the plot extension and instead devise a scheme for people to inlclude graphics directly. This would save us future problems like this one. Sphinx directives such as literalinclude may help here.

The pu branch also contains a cherry pick of d299916 for the image processing tutorial. Also the image processing tutorial graphics now have the right size again.
@emmanuelle if you have a minute please just have a quick look and let me know if i can merge this to master.

Also note that on my machine [TM] this currently only builds with a HG version of Sphinx.... yeah....

from scientific-python-lectures.

Here is the diff for pu: master...pu

from scientific-python-lectures.

I think that this is fixed and that we can close the issue. @esc can you confirm, please?

from scientific-python-lectures.

+1

See also: 2349533, 0262450, 2349533 and c29001d

from scientific-python-lectures.