#29251 closed defect (duplicate)

plot3d method of Graphics objects broken with threejs

Reported by: egourgoulhon Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: graphics Keywords: graphics, plot3d, threejs
Cc: paulmasson Merged in:
Authors: Reviewers: Paul Masson, Eric Gourgoulhon
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #29227 Stopgaps:

Status badges

Description

As reported in https://ask.sagemath.org/question/50051/, 3d objects constructed by the method plot3d() of class Graphics do not display properly with threejs. A minimal example is

sage: g = plot(sin(x), (x, 0, 2*pi))
sage: g.plot3d()
/home/eric/sage/9.1.develop/local/lib/python3.7/site-packages/sage/repl/rich_output/display_manager.py:592: 
RichReprWarning: Exception in _rich_repr_ while displaying object: 
'Line' object has no attribute 'all'
  RichReprWarning,

Note that

sage: show(g.plot3d(), viewer='jmol')
Launched jmol viewer for Graphics3d Object

works well. Besides, if one change the value of the argument z from 0 (the default) to 2:

sage: g.plot3d(z=2)
Launched html viewer for Graphics3d Object

one does not get any error message but a wrong threejs display: the curve is located at z=-2 instead of z=2.

Change History (5)

comment:1 Changed 17 months ago by egourgoulhon

  • Cc paulmasson added

comment:2 Changed 17 months ago by paulmasson

  • Dependencies set to #29227

Will be fixed by #29227

comment:3 Changed 17 months ago by paulmasson

  • Milestone changed from sage-9.1 to sage-duplicate/invalid/wontfix
  • Reviewers set to Paul Masson
  • Status changed from new to needs_review

comment:4 Changed 17 months ago by egourgoulhon

  • Reviewers changed from Paul Masson to Paul Masson, Eric Gourgoulhon
  • Status changed from needs_review to positive_review

I confirm that the issue reported here is fixed by #29227.

comment:5 Changed 17 months ago by chapoton

  • Resolution set to duplicate
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.