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

M127 public #251

Merged
merged 18 commits into from
Jul 29, 2024
Merged

M127 public #251

merged 18 commits into from
Jul 29, 2024

Conversation

HinTak
Copy link
Collaborator

@HinTak HinTak commented Jun 28, 2024

The m127 part is quite mundane - GrVkExtensionFlags were gone (withdrawn from public headers).

I have include the doc fix (#249) for apple shader compilation errors. Also while working on that, I found that it is good that old python code using m87-like API continues to work mostly, it is a bit confusing if one is looking at current m12x c code for reference (like trying to rewrite the sdl2 c example in python) , but having to go backward. So it would be nice to have newly written python code mimicking current c code API wise.

To that end, I found that while between m87 and m116, SkSurfaceProps(inittype) was removed, a simpler constructor SkSurfaceProps() was added to replace it - when you just want "any" and don't care - and we simply commented out the removed one only. There are a few things I'd like to add, so that the python version of the sdl example mimic the c sdl example better. etc. So a few more things to add to this. This should build fine.

Make skia.GrDirectContexts (namespace) an alias to skia.GrDirectContext
Make GrBackendRenderTargets (namespace) an alias to GrBackendRenderTarget

Allow direct call to GrBackendRenderTargets::MakeGL
Adding direct call to SkSurfaces::WrapBackendRenderTarget

Make Surface.Raster the same as Surface.MakeRaster
@HinTak
Copy link
Collaborator Author

HinTak commented Jul 19, 2024

@kyamagu just FYI, the latest bunch of additions to the m127 pull are misc things I found while porting the official SDL2 skia example in google's repo (before they decided it is not particularly special and removed it) in c to python, and the additions basically allow newly written python code to look more like current upstream m1xx skia c API, instead of being stuck in looking like m87. (And the m87-like ones still works - and am just adding redirections, or direct calls to new, where old api was emulated with new).

@HinTak
Copy link
Collaborator Author

HinTak commented Jul 19, 2024

@kyamagu it looks like CI is failing because it can't get to the github centos host. I an assuming it is either a transient problem (e.g. Microsoft Windows is broken around the world at the moment) or a permanent one I.e. we need to switch to a newer Linux host. Will re-run the CI again later, and look at this again if it doesn't work after re-try. Don't worry.

@HinTak
Copy link
Collaborator Author

HinTak commented Jul 19, 2024

Centos 7 was EOL on 1st July 2024. Part of manylinux we used. Updating and hope it should be okay now.

…yment target matches.

We need and build for 10.13 in Skia. The default is 10.9, too low for us.
@HinTak HinTak mentioned this pull request Jul 27, 2024
@HinTak HinTak merged commit 6de2dce into kyamagu:main Jul 29, 2024
13 checks passed
@HinTak HinTak deleted the m127-public branch July 29, 2024 03:17
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.

1 participant