-
Notifications
You must be signed in to change notification settings - Fork 68
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
autot tests seem to fail from master #1930
Comments
interestingly removing my pre-exisiting uvcmetrics_test_data directory and re-installing it leads:
I think the md5 checking fails |
No, I don't. While images seem to be correct, everything seems to be shifted a little. Maybe it is some sort of round up difference. What kind of mac is that? What os? |
I am rebuilding everything fresh on my laptop from master. I will let you know the results. |
It is interesting that only the tall version of the tests fail not the long ones. |
I just run the tests from master and a clean build on my laptop. This is what fails: |
@chaosphere2112 I see that you observe the same failures I think that
Might not work I'm going to add the x.setbgoutputdimension command as well just to be sure @chaosphere2112 I remember you saying something about geometry and bg mode |
yes I think that that is it I see the log on cdash:
image do not have the same size. |
I do get a gazillion of these now though:
|
@doutriaux1 Currently in VTK any renderers associated with a render window that is in background mode when it is resized will not resize. The render window itself will, and dumping it into a PNG should still output at the correct dimensions. So, probably not the same issue. |
In any case the picture generated by my mac are not the same size as the baselines from @danlipsa so I'm looking into this right now. |
If size is the issue, just use a smaller size in the tests. I assume my laptop has higher resolution than yours, that is why it works on mine. I had to resize the pictures when going from the desktop to the laptop as well. I think this is related to |
@danlipsa but we need to make sure they consistently get out at the same size accross device, will push a fix in a minute |
@danlipsa any idea why I get this on my mac from today's master:
I'm taking care of the diags ones
The text was updated successfully, but these errors were encountered: