-
Notifications
You must be signed in to change notification settings - Fork 19
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
xwindow driver conventions #8
Comments
I'd be interested to know your thoughts on this Daniel? |
my personal goal is not 100% compatibility with PGPLOT if giza is used as a standalone library. Where we should have 100% compatibility with PGPLOT is when giza is called through the PGPLOT API. So we could fix this by ensuring the behaviour when called via giza-pgplot matches that of the original. For example, this is already done with colour indices. Giza has different default line colours to PGPLOT, but when called through the PGPLOT API will use line colours matching the original PGPLOT |
OK, that implies we should swap the default black/white on XW
… On 28 Nov 2018, at 11:27 am, Daniel Price ***@***.***> wrote:
my personal goal is not 100% compatibility with PGPLOT if giza is used as a standalone library. Where we should have 100% compatibility with PGPLOT is when giza is called through the PGPLOT API. So we could fix this by ensuring the behaviour when called via giza-pgplot matches that of the original.
For example, this is already done with colour indices. Giza has different default line colours to PGPLOT, but when called through the PGPLOT API will use line colours matching the original PGPLOT
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#8 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AOnymuvgRxr-DSmDKrBOUKce3FbJEdBtks5uzdhngaJpZM4XyZph>.
|
The /xwin driver in the pgplot mode has col 1 = white and col 0 = black.
This is the reverse of the behaviour of real pgplot (draws on white on black).
I don't have a huge problem with this, but if the goal is compatibility it should be changed.
Karl
The text was updated successfully, but these errors were encountered: