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
VTK doesn't build (freetype headers problem) #594
Comments
@remram44 @doutriaux1 Can you try this branch in devel: vtk_use_cdat_packages |
Please merge if works. |
@aashish24 I never had the issue, so let's wait to hear from @remram44 first ok? |
Sure.. but you may want to make sure that it does not break build on your system. Checking on that would be helpful as well. |
I'm building right now |
Thanks. Make sure that run you ctest afterwards just to make sure it didn't break your code.
|
If you can, please run CTEST before and after to see if there's any difference. From: Aashish Chaudhary <notifications@github.commailto:notifications@github.com> Thanks. Make sure that run you ctest afterwards just to make sure it didn't break your code. cd build_dir — |
that is correct. You want to run ctest before as well to be precise. But I assumed that you already have done it and for most of us 8 tests are failing.
|
@remram44 Any update? |
Unfortunately I won't be able to build on OS X until tonight, however running CTest on Debian showed a lot more failing tests than expected. |
This is not good! Can you send us the CTEST "Failed" list? From: Remi Rampin <notifications@github.commailto:notifications@github.com> Unfortunately I won't be able to build on OS X until tonight, however running CTest on Debian showed a lot more failing testshttps://gist.github.com/remram44/6c8302cdaf2d3350e47d than expected. — |
Built on OS X, results (29% tests passed, 200 tests failed out of 282). |
OS X 10.8 or 10.9? From: Remi Rampin <notifications@github.commailto:notifications@github.com> Built on OS X, resultshttps://gist.github.com/remram44/81357eabd3da05089df0 (29% tests passed, 200 tests failed out of 282). — |
This is 10.9. |
Looks like initial attribute issue: /Users/remirampin/.uvcdat/initial.attributes |
Yes, we need to get the new initial.attributes file out so others can avoid this problem. From: Aashish Chaudhary <notifications@github.commailto:notifications@github.com> Looks like initial attribute issue: /Users/remirampin/.uvcdat/initial.attributes — |
@aashish24 I have a branch that fixes the "no initial.attributes" bug. |
@aashish24 and another one with a touched up initial.atrributes 614 and 611 are the branches. @williams13 you sent me some templates, etc.. yesterday, is that all you need for diagnostics? Or do you have more in the work? |
For diagnostics, this is all I have for now. Either user UVWG (lower case uvwg), UVWG_1of3, UVWG_2or3, and UVWG_3of3. Or some other descriptive name. I am okay with it as long as I can call it. From: Charles <notifications@github.commailto:notifications@github.com> @aashish24https://github.com/aashish24 and another one with a touched up initial.atrributes 614 and 611 are the branches. @williams13https://github.com/williams13 you sent me some templates, etc.. yesterday, is that all you need for diagnostics? Or do you have more in the work? — |
@remram44 Can you try this:
then build and test as ususal. |
Results on Debian: https://gist.github.com/remram44/84045ef50a2af1adf7c1 (76% tests passed, 68 tests failed out of 286) |
You need to do the following: cd HOME mv .uvcdat .uvcdat.old source /Users/williams13/work/uvcdate/build_uvcdat/install/bin/setup_runtime.sh (point to wherever you built UV-CDAT) uvcdat This will generate a new .uvcdat directory with the appropriate initial.attribute file cd /Users/williams13/work/uvcdate/build_uvcdat/ (go to the directory where you built UV-CDAT) ctest –j4 Let us know what you get. Results on Debian: https://gist.github.com/remram44/84045ef50a2af1adf7c1 (76% tests passed, 68 tests failed out of 286) — |
Alright, new logs (same results, obviously): Results on Debian: https://gist.github.com/remram44/ea8f68e9bb81a01f5466 (76% tests passed, 68 tests failed out of 286) |
I cannot speak to Debian. Didn't know we had someone building on this flavor of Linux. But the Mac 10.9 should have the results below. @aashish and @doutriaux how can we get Remi on the same page as the rest of us. WE have got to get his changes into master! All the 282/286 Test #20: testUnstructGrid ................................... Passed 5.55 sec 98% tests passed, 6 tests failed out of 286 Total Test time (real) = 605.19 sec The following tests FAILED: From: Remi Rampin <notifications@github.commailto:notifications@github.com> Alright, new logs (same results, obviously): Results on Debian: https://gist.github.com/remram44/ea8f68e9bb81a01f5466 (76% tests passed, 68 tests failed out of 286) — |
If you look closely at LastTest.log, you'll see things like that: Are the thresholds too strict? |
@williams13 Remi's changes are in. This is my change (well my fix to the bug). @remram44 the threshold is bit strict but on Mac we should have gotten close to what we all get. Can you send me the diff.png images in your build directory (just few of them?) @Looks like the build issue is fixed but we need to make sure that it didn't cause these new test failures. @doutriaux1 can you try this branch? |
The threshold is definitely not too strict because adding the logo to the pictures did not break ANY test last time we merged it back accident, that was actually scary! C. From: Remi Rampin <notifications@github.commailto:notifications@github.com> If you look closely at LastTest.loghttps://gist.githubusercontent.com/remram44/3af7f200ff3af849c77d/raw/osx10.9-LastTest.log, you'll see things like that: image fails comparison threshold 0.0736404483779>0.05 Are the thresholds too strict? — |
Remi, In your build directory under testing/vcs there should be the pictures produced by failed test and also the same file with a _diff.png that shows (poor man implementation) the diff with original. Please take a look to see if they are very different or not. Send us some example of ones that you think are identical and where failure is due to bad threshold. Thanks, C. On 09/10/2014 01:10 PM, Remi Rampin wrote: If you look closely at LastTest.loghttps://gist.githubusercontent.com/remram44/3af7f200ff3af849c77d/raw/osx10.9-LastTest.log, you'll see things like that: image fails comparison threshold 0.0736404483779>0.05 Are the thresholds too strict? — |
Seems like it. Can you turn off the anti-aliasing on your system driver? Just for testing |
But @aashish24 we do turn off antialiasing (per your instructions a while back). Do we not want to set
|
@remram44 I don't think it's the issue, we are turning off antialiasing. |
@remram44 @aashish24 ok it is VTK related... I just built this branch on my ubunutu and now I get the same failed tests as @remram44 ! Aashish how can these flags affect the png output so much. Not merging until we figured out what's going on. @aashish24 did you run ctest on your computer when you pushed these changes? |
same on mac, tests fail. @aashish24 did it work for you? |
It did work for me on Ubuntu but lets not merge unless we figure it out at this point. One thing would be is that now we are using freetype from cdat. I am wonderin gif that's causing the images to be sligthtly diffirent. Can you post the diff image? Thanks for testing it. |
@aashish24 @remram44 already posted the diff image |
That cannot be diff image if the diffirence is that small.? |
the second one. |
Ahh... got it -;) yes it is the text rendering. Can you show me where the text rendering code is? |
if vcs2vtk.py around line 698.
I'm using TextActor. Maybe we should be using textActor2D? |
1 similar comment
if vcs2vtk.py around line 698.
I'm using TextActor. Maybe we should be using textActor2D? |
Let me have a look |
@doutriaux1 what font is used by the vcs? vcs.elements["font"][vcs.elements["fontNumber"][tt.font]] Where do I find that information? |
Packages/vcs/Lib/init.py
|
|
Looks like discussion on the original issue continues on #664? I don't know. |
OS X 10.9 build stuck on this error, building VTK:
Full log
The text was updated successfully, but these errors were encountered: