-
Notifications
You must be signed in to change notification settings - Fork 7
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
Blender 2.8 Mac OS X shader compilation errors #7
Comments
Thanks for the report, @skarnproject! I will be updating these scripts right after the majority of the UI code is written. A note: early on when I started dev on the addon_common (a long time ago now), I decided to wait on updating the shaders to see how quickly 2.80 was adopted, how much support it has on OSX, what OpenGL API was going to be available, etc. OSX OpenGL has always been a thorn in my side when it comes to writing shader code in Blender. I thought that I might have to support 2.79 along with 2.80. However, I'm leaning more toward dropping 2.79 support and focus on supporting supporting 2.80, which would simplify things a lot. Checking in with @patmo141 to see if he would use CC in Blender 2.79, or if he is planning to update his tools to 2.80. |
Thank you for the reply. Supporting both versions is a nightmare given how the API is different. I'd personally go for 2.8 only. By the way, I have updated the shaders myself to give a test to your UI framework in 2.80, as I am going to use it in my addon for viewport widgets. There is an issue with FSM erroring out that main_main was not registered. It happens when I am running a provided test operator. |
If you would, do a Pull Request. Would be interested in your change. As for the FSM issue, how is your project set up? Do you have a repo that I can look at? |
I need it tested before doing a PR since I don't know opengl, I just did it according to documentation. But since I could never run it, I did not test it. Speaking of the repo, I don't have it publicly available yet. I just added addon_common folder to my project, that's it (the operator got registered automatically). Then I went into Blender and ran the test operator. I am thinking it could maybe be related to automatic registration of operators? The system that Animation Nodes addon is also using for automatically registering bpy classes, you can see it there. |
ok. Are you using |
Yes, I am using CookieCutter. I just ran an operator provided in the test.py file of this repo. From what i can see, I can't spot any difference between the usage of CookieCutter in ExtraCut and Retopoflow compared to this. The error is:
|
which branch are you using? also, I haven't updated a better template to start from is to look at |
I am using the B280 branch. I see now that test.py contains some old style rendering code. I will try to setup a sample operator using CookieCutter using the Retopoflow one as an example, if that is the working one. I will see if I can update the rendering code to OpenGL core profile then. |
I will be continuing use of CC in 2.79 for a while, however I can freeze the dependency at current revision so don't stress about pushing on to 2.8. I will be developing new tools in 2.8 so my vote is not to let 2.79 compatibility slow down 2.8 progress |
@skarnproject were you able to set up a sample CookieCutter operator? I haven't taken the plunge yet but soon I will. |
@patmo141 I actually took a break from doing my Blender projects due to some reasons. Now I am getting back to this and will try asap. Is the Mac OS X compatibility issue supposed to be resolved now? |
would you like to create a github repo and work together on creating a few simple cookie cutter examples? |
Sure, if you want to collaborate on that, please add me to the repo. I will be working on that stuff actively after January 1 st. |
The lib is using OpenGL shaders version #120 while Blender 2.8 got moved to core profile. Is there any chance rendering can be updated to the core profile as well? Mac OS X has no support for compatibility profile.
The text was updated successfully, but these errors were encountered: