-
Notifications
You must be signed in to change notification settings - Fork 66
VCS won't plot when every value is exactly zero #648
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
Comments
looking at it now. |
@aashish24 please review and let @potter2 @williams13 @painter1 know when in master, they need this for the ACME png testing code. |
in branch: issue_648_constant_value |
Okay.. ....sure. |
Sorry.. failing some tests: The following tests FAILED: 94: vtkPNGReader (0x37bbe90): Unable to open file /home/chaudhary/tools/uvcdat/src.git/testing/vcs/test_vcs_basic_scatter_zero.png |
At that point we diverged too much from original pictures. I cannot/don’t want to regenerate all pix for every branch, please merge all my branches in, they pass tests on my systems. ALL of them. Once they are all in, I will merge master into my logo branch and regenerate all the pix ok? Thanks, C. From: Aashish Chaudhary <notifications@github.commailto:notifications@github.com> Sorry.. failing some tests: The following tests FAILED: 94: vtkPNGReader (0x37bbe90): Unable to open file /home/chaudhary/tools/uvcdat/src.git/testing/vcs/test_vcs_basic_scatter_zero.png — |
Does this also fix the buttons to blend the colors? Gerald (Jerry) Potter From: Charles <notifications@github.commailto:notifications@github.com> At that point we diverged too much from original pictures. I cannot/don’t want to regenerate all pix for every branch, please merge all my branches in, they pass tests on my systems. ALL of them. Once they are all in, I will merge master into my logo branch and regenerate all the pix ok? Thanks, C. From: Aashish Chaudhary <notifications@github.commailto:notifications@github.commailto:notifications@github.com> Sorry.. failing some tests: The following tests FAILED: 94: vtkPNGReader (0x37bbe90): Unable to open file /home/chaudhary/tools/uvcdat/src.git/testing/vcs/test_vcs_basic_scatter_zero.png — — |
nope... |
@doutriaux1 this tests is failing by quite a bit of large number: image fails comparison threshold 87.5885568018>0.09 It cannot be small diffirence. |
which test? |
4: image fails comparison threshold 87.5885568018>0.09 |
that might be that the source image is not in the repo actually. Can you double check please. I've only seen this PNG error when source is missing. |
yes that is exactly it the files are on my system but not in the repo.... Pushing them back into the branch right now. |
Sure.. I just sent you an email. I will help here..will report back soon. |
ok anyway file is in now. Sorry about will setup the "staging" branch now. |
This was fixed in an earlier version. If I have one time point with every value exactly zero, nothing happen when I try to plot. This doesn't happen very often but when comparing a forecast with analysis, the first time-step is exactly zero. If I shift the first time to the second time step the plot looks fine. DV3D handle the zero value by just plotting zeros for the entire map.
The text was updated successfully, but these errors were encountered: