Skip to content

M127 public#251

Merged
HinTak merged 18 commits into
skia-python:mainfrom
HinTak:m127-public
Jul 29, 2024
Merged

M127 public#251
HinTak merged 18 commits into
skia-python:mainfrom
HinTak:m127-public

Conversation

@HinTak
Copy link
Copy Markdown
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.

HinTak added 6 commits July 19, 2024 15:33
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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 skia-python:main Jul 29, 2024
@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