Skip to content
This repository was archived by the owner on Sep 7, 2022. It is now read-only.

Conversation

@zhuxingwei
Copy link
Contributor

@zhuxingwei zhuxingwei commented Apr 6, 2022

In this PR we are trying to support OpenGL backend on Mac/iOS and external texture on it, mainly including the following changes:
(1) fix the C# side codes so that the UIWidgetsPanel can now handle both Metal and OpenGLCore/OpenGLES backends on Mac/iOS respectively via different C++ bindings.

(2) refine the UIWidgets engine codes on the C++ side to fix issues caused by OpenGL context switching: Specifically, since we have to deal with both the openGL context of UIWidgets engine itself and that of Unity engine, we need a more careful approach to guarantee that the two kinds of openGL context doesn't mess around with each other. This situation doesn't not happen for flutter.

(3) refine the UIWidgetsPanel as well as SurfaceManager codes on C++ side to support both Metal and OpenGLCore/OpenGLES backends

(4) add the support of external native texture for OpenGLCore/OpenGLES backends on Mac/iOS respectively

(5) update the build code a bit so that it now support building iOS plugin with Mac 12 SDK

In this PR, we also add support for linear colorspace support on all platforms.

@zhuxingwei zhuxingwei changed the title 【WIP】Zxw/3d mac Zxw/3d mac Apr 14, 2022

static bool UIWidgetsDisabled;

void DisableUIWidgets() {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found that this log is wrong.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

which log ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Debug.Log("Please change graphic api for UIWidgets.\n" +
                      "Metal for iOS and MacOS.\n" +
                      "Direct3D11 for Windows\n" +
                      "Vulkan for Android\n");

(not this pr's change)

@zhuxingwei zhuxingwei merged commit 085c978 into zxw/3d May 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants