-
Notifications
You must be signed in to change notification settings - Fork 32
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.
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
WARNINGS and Tk error when using view() with latest Julia v0.4.0 (OSX 10.9.5) #57
Comments
The repeated warnings should have been fixed by JuliaLang/julia#9291, if you have a sufficiently-modern 0.4. But I need to fix these deprecation warnings within ImageView, because they also cause a major performance hit. I've not been looking at big images recently, so I haven't had much incentive. BTW, you might also be interested in https://github.com/timholy/TestImages.jl. If you'd like to generalize it to allow reading directly into OpenCV.jl's format, that would be fine with me. The main tricky thing would be to avoid the penalty for loading both Images and OpenCV. Winston contains an example of how to configure an environment variable to determine which graphical backend (Tk or Gtk) gets loaded. We could also check |
Oh wait, you said your julia is completely up to date. But, let me guess: because of Cxx you are using LLVM-svn (or LLVM-3.5), and your backtraces are broken. (What does |
I think my Julia 0.4-dev build from this morning should hopefully be modern enough :) backtrace (very very very long process. . .) says JULIA test/backtrace
* backtrace
SUCCESS Full trace is here. Any suggestions? |
That's for your whole build (you must have had something "stale" so it rebuilt julia for you). Here's a more useful test:
Notice that I got useful file names and line numbers for many of these. But I basically have a PR almost cued up that will fix this; just waiting on JuliaLang/julia#9278 (comment). Or I'll do it myself, or you can submit a |
OK, I see that the trace is for the whole build (and that it rebuilt it again) but I don´t get your point about something being "stale". . . I simply took the the fresh master from github, and after updating the folder (including deps/llvm folders) with Here is my output for the test you suggested: julia> @noinline function myfunction()
backtrace()
end
myfunction (generic function with 1 method)
julia> bt = myfunction()
1-element Array{Ptr{Void},1}:
Ptr{Void} @0x000000010d140f6e
julia> for b in bt
@show Profile.lookup(b)
end
Profile.lookup(b) = Base.Profile.LineInfo("rec_backtrace","/Users/maximilianosuster/julia-v0.4.0/src/task.c",623,true,4514385774) |
Not sure I get your point fully about the planned fixes.
The biggest trouble for me right now seems to be the |
Easiest if I just show you. See #58. It will break ImageView on 0.3 currently because no one added |
See this PR in Compat for how I fixed iceil, iround, and ifloor: JuliaLang/Compat.jl#20 |
Thanks for sharing the examples. Got it, seems straightforward. It seems that Compat.jl #25 asks to keep |
Sure, go ahead and make the addition to Compat. |
You also should tag a new version of Compat. Let me know if you have questions about how to do that (it's no different than tagging one of your own packages, once your PR gets accepted). |
Thanks. I think I will manage since I have tagged my own package. |
Looks like I forgot to hit "comment" on this, so it's out of order (it's about your backtraces): All I meant was that most of that output you put in the gist was from building julia, and not the result from running the test. Only the last three lines were from running the test. But that test definitely shows there's something broken about your backtraces. (It's not your fault; see JuliaLang/julia#7910.) That's why you're seeing zillions of warnings, and I'm not. |
Got it. Thanks a lot for the explanation. I will be back today with the PR submission for Compat.jl. Its basically done but I want to make sure that it works for all 5 methods of sizehint!. |
After pulling julia-v0.4.0 master, rebuilding julia, doing Pkg.update(), I encountered the following with ImageView (OSX 10.9.5):
The WARNING goes on forever and the only way to stop it in the REPL is to use ctrl C.
The first time it actually reported a Tk error, but I did not see it the second time I tested.
The text was updated successfully, but these errors were encountered: