-
Notifications
You must be signed in to change notification settings - Fork 250
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
ERROR: LoadError: UndefVarError: pangolayout not defined #1206
Comments
Seems like this is a known issue in Compose.jl relating to scopes: The workaround proposed in the first issue still works, i.e. add a pangolayout definition to pango.jl (https://github.com/d9w/Compose.jl/commit/c2acf841a9faddd8785697c915f106a8b52ebd7d for an example) |
i'm working on fixing libpango in Compose. sorry it's taking so long. |
@bjarthur, I had another look at the error after I received your message. Before the error, there is a warning that looks relevant to
I have included the full output below.
|
+1 Ubuntu 16.04, Julia 1.0.1, using Juno. I'm stacking layers of |
it's related to text. i'd bet if you turn off all labels, keys, and titles the error would go away. i'm still working on pango support. quite difficult actually with the new BinaryBuilder system. might still be awhile. |
Great thanks. I actually did the same as @d9w and just added a single line to pango.jl in Compose.jl, and that seems to fix the issue: EDIT: this workaround is not a long term fix rather just temporarily avoids the issue and produces poorly colored plots. |
Hi @montyvesselinov , I have mostly been using Gadfly but not contributed back to the repo much yet. The workaround above is probably not something we want to cement into the repo, however, I agree this issue is bit of a show stopper for Julia 0.7 upgrades. I'd defer to @bjarthur or @d9w on whether to include d9w/Compose.jl@c2acf84 / dehann/Compose.jl@7793350 as a temporary fix. As example with the workaround, my figures are looking something like this (black background not intended): I appreciate though that the binary building issue bjarther offered to work on is difficult to deal with due to upstream changes. One alternative might be that |
@dehann Thank you very much for your note. I highly appreciate your efforts. I am a big fan gadfly. The reason I am pushing for this is to be able to release our codes that depend on gadfly / compose. They work for me because I use the fixes above I cannot releases because the tests fail at the moment. |
it's going to be awhile before the pango issue is resolved. i'd be fine merging this workaround into master for the time being. |
do you guys need GiovineItalia/Compose.jl#326 too? |
I think so. Thank you very much!
On Tue, Nov 6, 2018 at 05:09 Ben Arthur ***@***.***> wrote:
do you guys need GiovineItalia/Compose.jl#326
<GiovineItalia/Compose.jl#326> too?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1206 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFXxeL7i56twM181_amsI8cRLfL8mhXQks5usXwEgaJpZM4XUXQw>
.
--
-monty
|
Both GiovineItalia/Compose.jl#325 and GiovineItalia/Compose.jl#326 appear to get Gadfly working for me. |
I understand 326 as making Thanks again for working on the long term fix! |
re. pango support in Gadfly... what features of pango do you guys need that is not recreated in Compose/src/fontfallback.jl? in other words, if you just |
@bjarthur, I just |
I've encountered an error, "pangolayout not defined".
I have the latest Cairo, Compose, and Gadfly.
Does anyone know of a workaround?
The text was updated successfully, but these errors were encountered: