Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Stack Overflow in BoundingBox #5217

Closed
magistere opened this issue Dec 22, 2013 · 6 comments
Closed

Stack Overflow in BoundingBox #5217

magistere opened this issue Dec 22, 2013 · 6 comments

Comments

@magistere
Copy link
Contributor

There are problems with Winston plotting on recent Julia build:

julia> using Winston

julia> plot(sin(1:10))
error during Tk callback: 
ERROR: stack overflow
 in BoundingBox at graphics.jl:86 (repeats 79996 times)
Evaluation succeeded, but an error occurred while showing value of type FramedPlot:
ERROR: stack overflow
 in BoundingBox at graphics.jl:86 (repeats 79996 times)

julia>

I assume it is related with similar recent issue #5201
@jiahao
Copy link
Member

jiahao commented Dec 22, 2013

Have you tried updating within the last 24-48 hours? There were a number of these stack overflows whose root cause was recently fixed (e.g. JuliaInterop/ZMQ.jl#39).

@timholy
Copy link
Member

timholy commented Dec 22, 2013

I noticed this too, fixed.

@magistere
Copy link
Contributor Author

Tim, I still have error after your fix:

julia> plot(sin(1:10))
error during Tk callback: 
ERROR: no method convert(Type{Vec2}, Points)
 in setindex! at array.jl:413
 in _make_ticks at /home/magistere/.julia/Winston/src/Winston.jl:861
 in make at /home/magistere/.julia/Winston/src/Winston.jl:890
 in boundingbox at /home/magistere/.julia/Winston/src/Winston.jl:2486
 in exterior at /home/magistere/.julia/Winston/src/Winston.jl:1228
 in interior at /home/magistere/.julia/Winston/src/Winston.jl:1711
 in compose at /home/magistere/.julia/Winston/src/Winston.jl:1775
 in page_compose at /home/magistere/.julia/Winston/src/Winston.jl:1793
 in page_compose at /home/magistere/.julia/Winston/src/Winston.jl:1781
 in anonymous at /home/magistere/.julia/Winston/src/tk.jl:42
 in draw at /home/magistere/.julia/Tk/src/tkwidget.jl:251
 in configure at /home/magistere/.julia/Tk/src/tkwidget.jl:246
 in init_canvas at /home/magistere/.julia/Tk/src/tkwidget.jl:406
 in anonymous at /home/magistere/.julia/Tk/src/tkwidget.jl:385
 in jl_tcl_callback at /home/magistere/.julia/Tk/src/tkwidget.jl:145
 in tcl_doevent at /home/magistere/.julia/Tk/src/tkwidget.jl:20
 in wait_initialized at /home/magistere/.julia/Tk/src/tkwidget.jl:434
 in getgc at /home/magistere/.julia/Tk/src/tkwidget.jl:443
 in anonymous at /home/magistere/.julia/Winston/src/tk.jl:39
 in draw at /home/magistere/.julia/Tk/src/tkwidget.jl:251
 in display at /home/magistere/.julia/Winston/src/tk.jl:44
 in tk at /home/magistere/.julia/Winston/src/tk.jl:33
 in display at /home/magistere/.julia/Winston/src/Winston.jl:2620
 in display at multimedia.jl:151
FramedPlot(...)

@magistere
Copy link
Contributor Author

@jiahao I've used the most recent Julia version when I've sent this issue. But stack overflow still exists for me.

@timholy
Copy link
Member

timholy commented Dec 22, 2013

Totally-separate error. You need to update to master on Winston:

JuliaGraphics/Winston.jl@23bb603

@magistere
Copy link
Contributor Author

Tim, thank you for fixing both bugs. Plotting works now.

gitfoxi pushed a commit to gitfoxi/julia that referenced this issue Dec 23, 2013
Merge remote-tracking branch 'upstream/master'
e25f4ad 
* upstream/master: (89 commits)
  fix JuliaLang#5225
  update pcre
  fix off-by-1 in isqrt. closes JuliaLang#4884
  Add more keywords to ctags regex, plus README
  annotate the types of arguments for derived trigonometric & hyperbolic functions
  fix doc for && and || and update helpdb
  only show ccall literal address warning in imaging mode. closes JuliaLang#5215
  minor update of hypot to ensure consistency of output types
  Fix JuliaLang#5217
  silence compiler warning
  hopefully more robust way of getting github URL (don't assume module name is Pkg name)
  add text/html writemime for MethodList and Method (fix JuliaLang#4952)
  update NEWS
  doc: `import M: single,name` syntax, close JuliaLang#5214
  clean up native finalizers code
  specialized abs2 for bool
  remove use of callback API in REPL
  Some error message cleanup to fix segfault when transposing sparse vector with illegal values.
  test/git*.jl: don't use `echo` to read-and-write from processes.
  test/git*.jl: don't use `echo` to read-and-write from processes.
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jiahao @magistere @timholy