-
Notifications
You must be signed in to change notification settings - Fork 23
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
fix build issues and migrate to ue4.23 #5
base: master
Are you sure you want to change the base?
Conversation
I cannot get this to work. It crashes when I hit play. In your code I noticed that at some point you have one ERHIFeatureLevel: flag set to SM5 and one to ES3_1. I set both to ES3_1 . I get the following issue: I only got some Oculus errors in the log, but don't think that's related (I didn't install SteamVR on this machine):
|
Yes, I noticed that inconsistency with SM5 and ES3_1. I created another branch at https://github.com/NuLL3rr0r/ImsvGraphVis/commits/migration and tried to do it version by version. Unfortunately, I am not that much familiar with shaders. The code does not crash on 4.20 but is not working as expected either. So, I gave up on it. |
Hi, @NuLL3rr0r, I have also tried with Vulkan (on a Win10 platform) and I get the same error. But I think the problem may be due to the shaders themselves, as in the Vulkan documentation it is specified that:
In addition, in the UE4 docs, it is specified that there is a cross-compiler for transforming HLSL into GLSL: My suspicion is that this does not work correctly for Vulkan, as the error is on line 56 in the SplineComputeShaderCommon.usf file, which is: It does not like the : symbol before the register keyword. This is standard HLSL syntax, and UE4 seems to be unable to resolve it for VULKAN. I would try with pure OpenGL if you are using Linux. When I try to launch with DirectX on Windows I do get further, but there is still a crash later on (so the shader compiled successfully). It occurs when I hit the Play button:
The code that fails in that file is:
When debugging I can see that it fails when processing a Layout with Name: And two resources:
It fails with index i = 0, so the resource was UBMT_TEXTURE. In addition I can get the nodes of the graph to display - it only crashes if I enable the edges (so I think that finding a solution for this crash would make the whole project work again). For example, commenting out the contents of: So far, I was not able to fix this last crash and make the edge rendering work. I suspect that it should be possible to disable FMobileReflectionCaptureShaderParameters or bypass it, but I haven't found out how. |
@avrdan thank you so much for the progress and detailed information. If I recall correctly, I already did try to run it on pure OpenGL on Linux. Unfortunately, Epic did deprecated OpenGL in favor of Vulkan on Linux. Basically, we Linux users are stuck to Vulkan and it is not possible to run the engine on OpenGL in newer versions of the engine anymore. |
First of all, thank you so much for such a great project.
Second, I tried to port ImsvGraphVis over to the Unreal Engine 4.23 and figured out all the build issues due to UE4 API changes since 4.17 in a couple of hours. So, I decided to create this pull request.
Although, there is a runtime crash that I was not able to figure out since I am not much familiar with HLSL and shaders in general. When I run the project, I get the following error:
I am running Unreal Engine under Gentoo/Vulkan with NVIDIA drivers on 1070M and run it with -vulkan flag.
By the way, here is the full log if it helps: