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

OpenGL context version #45

Open
paraboul opened this issue Jan 10, 2017 · 4 comments
Open

OpenGL context version #45

paraboul opened this issue Jan 10, 2017 · 4 comments
Assignees

Comments

@paraboul
Copy link
Member

SDL doesn't seem to chose the best version available.

We need to figure out what's the lowest and highest version we want (only core profile?).
Does skia accepts core profile only?

@paraboul paraboul self-assigned this Jan 10, 2017
@verpeteren verpeteren self-assigned this Jan 10, 2017
@verpeteren
Copy link
Contributor

#46

@verpeteren verpeteren removed their assignment Jan 10, 2017
@efyx efyx mentioned this issue Jan 24, 2017
@cookiengineer
Copy link
Contributor

cookiengineer commented Apr 13, 2017

What's the minimum OpenGL version that skia needs to function properly?

Today I tried out nidium on an old Thinkpad (core2duo with i915 intel driver), but apparently the OpenGL 1.5 is too low for it to create a proper context.

Is there a way to use/compile skia with Vulkan support? As Intel backported most of their OpenGL stuff already to Vulkan, I guess that would help to support some legacy environments. But I have no clue about whether that gets too complex to be worth the effort for skia.

ninja edit:// Just reading through https://skia.org/user/special/vulkan

@paraboul
Copy link
Member Author

paraboul commented Apr 13, 2017

@cookiengineer OpenGL 2.1 is the minimum.

Indeed Skia now supports Vulkain (IIRC it's now used on Android).
However Nidium itself has a lot a OpenGL code and is also using it for WebGL (though I guess the ANGLE transpiler we're using could take care of this).

Unless someone is able to investigate this, we've more important task to finish before considering this.

@cookiengineer
Copy link
Contributor

Jup, I understand completely and it would be probably a huge overhead to add a shader compiler like ANGLE for it that could translate between vulkan, opengl, webgl and directx.

I kind of wanted to add this just so that others can find the issue and the minimum OpenGL version if they happen to have a legacy machine :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants