Skip to content

Fix PixelVectors cache#1212

Merged
campagnola merged 1 commit intopyqtgraph:developfrom
erikmansson:fix-pv-cache
May 29, 2020
Merged

Fix PixelVectors cache#1212
campagnola merged 1 commit intopyqtgraph:developfrom
erikmansson:fix-pv-cache

Conversation

@erikmansson
Copy link
Copy Markdown
Contributor

Suggested fix for issue #1189 where I stumbled upon an issue with the PixelVectors cache. See the issue for more details.

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented May 28, 2020

Hmm, all linux CI runs are failing, but not just on this PR, but on another one as well... wonder if there is some upstream issue.

@2xB
Copy link
Copy Markdown
Contributor

2xB commented May 29, 2020

The linux CI tests fail in pyvirtualdisplay, and that was recently updated, i.e. yesterday and the day before yesterday: https://pypi.org/project/PyVirtualDisplay/#history

It causes an issue in pytest-xvfb, already reported here:
The-Compiler/pytest-xvfb#22

We could either ignore it for now, awaiting the update in one of the above modules, or pin pyvirtualdisplay to version 0.2.5.

More info:

End of traceback:

  File "/usr/share/miniconda/envs/test-environment-3.6/lib/python3.6/site-packages/pyvirtualdisplay/abstractdisplay.py", line 313, in stop
    raise XStartError("stop() is called before start().")
pyvirtualdisplay.abstractdisplay.XStartError: stop() is called before start().

End of internal pytest error stack:

INTERNALERROR>   File "/usr/share/miniconda/envs/test-environment-3.7/lib/python3.7/site-packages/pytest_xvfb.py", line 46, in start
INTERNALERROR>     self._virtual_display.cmd.extend(self.args)
INTERNALERROR> AttributeError: 'Display' object has no attribute 'cmd'

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented May 29, 2020

@2xB figured it had to do w/ the virtual display, but hadn't had a chance to do anything more than glance at it... I'll read a bit more when I get the kids to bed and try and figure out a course of action....on one hand we should ignore, on the other hand, I hate red from the CI.

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented May 29, 2020

Closing/Reopening the PR to trigger a new CI run.

@j9ac9k j9ac9k closed this May 29, 2020
@j9ac9k j9ac9k reopened this May 29, 2020
@campagnola campagnola merged commit 2d59ce8 into pyqtgraph:develop May 29, 2020
@campagnola
Copy link
Copy Markdown
Member

Looks great to me. Thanks @erikmansson !

@erikmansson erikmansson deleted the fix-pv-cache branch May 30, 2020 14:18
zhujun98 added a commit to zhujun98/EXtra-foam that referenced this pull request Jun 4, 2020
zhujun98 added a commit to zhujun98/EXtra-foam that referenced this pull request Jun 15, 2020
zhujun98 added a commit to zhujun98/EXtra-foam that referenced this pull request Jun 17, 2020
zhujun98 added a commit to zhujun98/EXtra-foam that referenced this pull request Jun 17, 2020
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.

4 participants