Skip to content

Conversation

@ctessum
Copy link
Contributor

@ctessum ctessum commented Mar 20, 2015

No description provided.

@kortschak
Copy link
Member

kortschak commented Mar 20, 2015 via email

@dgryski
Copy link

dgryski commented Mar 20, 2015

@kortschak bug introduced in #26

@kortschak
Copy link
Member

kortschak commented Mar 20, 2015 via email

@ctessum
Copy link
Contributor Author

ctessum commented Mar 20, 2015

I'm not sure how to add a test that doesn't require visual inspection for this.

@kortschak
Copy link
Member

kortschak commented Mar 20, 2015 via email

@kortschak
Copy link
Member

Closing because we have a competing proposal with a test in #182.

@kortschak kortschak closed this Mar 26, 2015
sbinet added a commit to sbinet-gonum/plot that referenced this pull request Mar 27, 2015
Address gonum#179 by translating the image to (0,-h) so (0,0) is at the
bottom-left corner.

Add a test decorated with (!windows && x11) tags (the travis-ci nodes
may or may not have x11 installed and configured.)

Add a dummy canvas_nox11.go so 'go test ./...' also works on windows.
sbinet added a commit to sbinet-gonum/plot that referenced this pull request Mar 29, 2015
- fixes gonum#179 for X11 too by translating the image to (0,-h) so (0,0)
  is at the bottom-left corner.
- 'go test ./...' will skip the X11 test if $DISPLAY is not defined.
- setup xvfb in travis-ci to test X11
sbinet added a commit to sbinet-gonum/plot that referenced this pull request Jun 14, 2019
In 4f1973d, we added bootstrap code
to fix gonum#179 and start xvfb for the benefit of the vg/vgx11 backend.

vg/vgx11 has since been removed from gonum/plot.
So remove the xvfb startup as well.
sbinet added a commit that referenced this pull request Jun 15, 2019
In 4f1973d, we added bootstrap code
to fix #179 and start xvfb for the benefit of the vg/vgx11 backend.

vg/vgx11 has since been removed from gonum/plot.
So remove the xvfb startup as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants