Resolve "(LearnerND) add iso-surface plot feature"
Closes #112 (closed)
Merge request reports
Activity
@Jorn there are no commits in this MR.
Now there is :)
Quick demo:
import adaptive adaptive.notebook_extension() def f(xyz): x, y, z = xyz return x**4 + y**4 + z**4 - (x**2+y**2+z**2)**2 learner = adaptive.LearnerND(f, bounds=[(-1,1), (-1,1), (-1,1)]) runner = adaptive.Runner(learner, goal=lambda l:l.npoints>1000) runner.live_info()
and when it's finished:
from ipywidgets import interact_manual interact_manual(learner.plot_isosurface, level=(-5.99,-.01,0.01), hull_opacity=(0,1,0.2))
or
learner.plot_isosurface(level=-0.5)
Edited by Jorn Hoofwijkadded 8 commits
-
46f2d8d4...724f9890 - 7 commits from branch
master
- 4761fd7e - add iso_surface_plot to adaptive learnerND
-
46f2d8d4...724f9890 - 7 commits from branch
- Resolved by Jorn Hoofwijk
- Resolved by Bas Nijholt
added 2 commits
added 2 commits
mentioned in merge request !127 (merged)
- Resolved by Jorn Hoofwijk
added 4 commits
-
3e660922...3a69cd99 - 2 commits from branch
master
- f07282c4 - add iso_surface_plot to adaptive learnerND
- f518d872 - improve 'notebook_extension'
-
3e660922...3a69cd99 - 2 commits from branch
added 1 commit
- 3f611327 - set lighting such that the plots look even better
added 2 commits
This plotly MR may be relevant: https://github.com/plotly/plotly.js/pull/2752
(but it's work in progress on their side)
Edited by Anton AkhmerovIt would be great if we could add some example to the documentation.
However, I don't think that the plotly plots will correctly show up ATM.
Related: https://github.com/spatialaudio/nbsphinx/issues/128
added 24 commits
-
260df457...8cf6ead1 - 20 commits from branch
master
- d22cd4a0 - add iso_surface_plot to adaptive learnerND
- b2008fa8 - improve 'notebook_extension'
- 7c453813 - add plotly to environment.yml
- 1810e2bf - also add an isoline feature
Toggle commit list-
260df457...8cf6ead1 - 20 commits from branch
added 2 commits
added 4 commits
-
82605cfe...e6d7fc09 - 3 commits from branch
master
- 6c22ce7e - add 'plot_isosurface' and 'plot_isoline' to adaptive learnerND
-
82605cfe...e6d7fc09 - 3 commits from branch
added 1 commit
- 2fdfaa58 - add 'plot_isosurface' and 'plot_isoline' to adaptive learnerND
added 5 commits
-
190db37a...1119d950 - 3 commits from branch
master
- a54799eb - add 'plot_isosurface' and 'plot_isoline' to adaptive learnerND
- e13e1d52 - merge 'get_isosurface' and 'get_isoline'
-
190db37a...1119d950 - 3 commits from branch
added 5 commits
- f758d4b0 - documentation improvements
- d1e4e270 - improve 'notebook_extension' and add 'plotly' as a dependency
- 8eaecaec - add 'LearnerND.plot_3D' and add an example to the docs
- a6c3f3a6 - add 'plot_isosurface' and 'plot_isoline' to adaptive learnerND
- bcc4be84 - merge 'get_isosurface' and 'get_isoline'
Toggle commit listadded 4 commits
-
bcc4be84...ad7a3d23 - 3 commits from branch
master
- 0547a7d9 - add 'plot_isosurface' and 'plot_isoline' to adaptive learnerND
-
bcc4be84...ad7a3d23 - 3 commits from branch
mentioned in issue #117
added 7 commits
-
0547a7d9...6cdd24e9 - 6 commits from branch
master
- 72449d98 - add 'plot_isosurface' and 'plot_isoline' to adaptive learnerND
-
0547a7d9...6cdd24e9 - 6 commits from branch
added 1 commit
- 143b29fd - add 'plot_isosurface' and 'plot_isoline' to adaptive learnerND
added 1 commit
- 5708c4ca - add 'plot_isosurface' and 'plot_isoline' to adaptive learnerND
@jbweston and @anton-akhmerov I rebased this, made some simplifications, and improved the docstrings.
Merge if you are satisfied.
Nice work @Jorn!
- Resolved by Joseph Weston
- Resolved by Joseph Weston