-
Notifications
You must be signed in to change notification settings - Fork 359
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
Commit the files that should have gone into #8612 #8613
Conversation
(Fix the variable transparency case that was broken by the #8255 commit.)
The CI jobs tell me that |
I failed to commit also the modified tests. You should now see the opposite situation. |
But what do you get when running the tests? I get
|
|
Did you update for this PR? I latter added the modified scripts. Note that now to have variable transparency we must now use the option -Q+t. i.e., |
Here is what I have in my log file:
|
Hmm, my runs were done with a release build. Could it be the reason? |
I've updated to the latest version.
I also tried the release build. The image is still different from the baseline image, so it fails. |
Can't make sense out of this. I am using |
The "BACKGROUND" word is likely added by macOS Preview. I don't see it in the Adobe PDF reader. |
Yes, likely but the SC I posted was taken in my computer as seen from Ghostscript. |
Any reason to not to merge this PR? |
Summary of changed imagesThis is an auto-generated report of images that have changed on the DVC remote
Image diff(s)Report last updated at commit d0c540b |
I've updated the baseline image for |
I can’t see a difference between the two. And note that only the second row, first col subplot has changed. |
When I do a We need to find a solution/replacement to this DVC thing. I simply can't get it to work. |
What's the dvc version you're using? |
C:\progs_cygw\GMTdev\gmt5\master>dvc --version But note that all 3 OSs fail this CI test. |
(Fix the variable transparency case that was broken by the #8612 commit.)