convert mandelbrot/julia interact to jupyter notebook

[fchapoton]

as another step to python3

Depends on #25373

CC: @bbarros50 @bhutz @sagetrac-atowsley @kcrisman @embray @jdemeyer @jm58660

Component: notebook

Author: Frédéric Chapoton, Jeroen Demeyer

Branch/Commit: 4de168d

Reviewer: Frédéric Chapoton, Jeroen Demeyer

Issue created by migration from https://trac.sagemath.org/ticket/24994

[fchapoton]

comment:1

work in progress, not yet working as expected..

---

New commits:

a9e184d

first tentative of conversion of mandelbrot interact to jupyter interact

[fchapoton]

Commit: a9e184d

[fchapoton]

Branch: public/24994

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

52ce4a5

second pass at interactive mandelbrot/julia in ipython notebook

[sagetrac-git]

Changed commit from a9e184d to 52ce4a5

[sagetrac-git]

Changed commit from 52ce4a5 to c65a01c

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

c65a01c

fixing colors and other details

[sagetrac-git]

Changed commit from c65a01c to 7cbf77b

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

7cbf77b

trying to fix doctests..

[fchapoton]

comment:5

ok, working now. Except 3 failing doctests that I just fail to write using ellipsis !

[fchapoton]

Author: Frédéric Chapoton

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

4467110

pep8 and pyflakes cleanup

[sagetrac-git]

Changed commit from 7cbf77b to 4467110

[sagetrac-git]

Changed commit from 4467110 to 588ac04

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

588ac04

fixing doctests (ugly)

[jdemeyer]

comment:9

This is bad:

M = mandelbrot_interact
...adjust various attributes of M...
return M

The problem is that you are changing and returning a unique global object. Instead of having

@interact.widget
def mandelbrot_interact(...

it would be better to just define mandelbrot_interact as an ordinary function and call interact.widget() inside the mandelbrot_plot function. That way, you end up with non-unique widgets.

Can you explain the <CSI-2K> appearing in the doctests? I feel uncomfortable accepting that without knowing where it comes from.

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

8bdbb0d

made the suggested changes

[sagetrac-git]

Changed commit from 588ac04 to 8bdbb0d

[fchapoton]

comment:11

Better like that ?

[fchapoton]

comment:12

ping ?

[fchapoton]

comment:13

ping ?

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

ad085dc	upgrade to ipywidgets 7.2.0 and widgetsnbextension 3.20
4b925be	first tentative of conversion of mandelbrot interact to jupyter interact

[sagetrac-git]

Changed commit from 8bdbb0d to 4b925be

[fchapoton]

comment:31

Is this still "needing work", Jeroen ?

This is the place where docbuild currently fails on the python3 patchbot. See

https://patchbot.sagemath.org/log/0/Ubuntu/18.04/x86_64/4.15.0-20-generic/petitbonum/2018-05-15%2012:37:45?plugin=docbuild

[fchapoton]

Changed dependencies from #25074 to none

[fchapoton]

comment:33

This is working fine, the only issue is comment:29

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

8143016

first tentative of conversion of mandelbrot interact to jupyter interact

[sagetrac-git]

Changed commit from 4b925be to 8143016

[jdemeyer]

comment:35

OK, I'll work on this today.

[jdemeyer]

Dependencies: #25373

[sagetrac-git]

Changed commit from 8143016 to e4523c4

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

2e75d4c	Add IPython support for Cython functions
f25a0e8	first tentative of conversion of mandelbrot interact to jupyter interact
e4523c4	Improve Mandelbrot/Julia interact

[sagetrac-git]

Changed commit from e4523c4 to 56b571f

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

56b571f

Improve Mandelbrot/Julia interact

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

4de168d

Improve Mandelbrot/Julia interact

[sagetrac-git]

Changed commit from 56b571f to 4de168d

[fchapoton]

Reviewer: Frédéric Chapoton, Jeroen Demeyer

[fchapoton]

comment:41

I think I can reasonably set this to positive now.

[vbraun]

Changed branch from public/24994 to 4de168d