Skip to content
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

Removed pyglet.window.get_platform() #35

Closed
wants to merge 2 commits into from

Conversation

arihantparsoya
Copy link
Member

Fixes #10 .

Pyglets `platform` API has been depriciated hence it should not be used.

`config` (defined in line 46) is created using `screen`.
@abhikpal
Copy link
Member

Hmm. interesting. This seems to break on my machine (Debian 9 running on a ThinkPad)... The window opens but fails to display anything. I do see some grey boxes occasionally, though. Here's what I get:
pr35

@arihantparsoya
Copy link
Member Author

arihantparsoya commented Mar 1, 2018

Can you show your config variable when it is working and not working?

@arihantparsoya
Copy link
Member Author

@abhikpal , I have pushed a new commit in this PR. Can you check if this new configuration works on your computer?

based on the comments by @kevinstadler I haved added double_buffer=1, sample_buffers=1.

See: p5py#10 (comment)
@arihantparsoya
Copy link
Member Author

Ping @abhikpal .

@abhikpal abhikpal added this to the Pre-VisPy release milestone Jun 5, 2018
@abhikpal abhikpal self-assigned this Jun 5, 2018
@abhikpal abhikpal modified the milestones: Pre-VisPy release, Port backend to VisPy Jun 12, 2018
@abhikpal abhikpal closed this Jul 18, 2018
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.

2 participants