myst6re / makoureactor

Final Fantasy VII field archive editor
GNU General Public License v3.0
55 stars 11 forks source link

Walkmesh and Background not visible [develop branch] #165

Closed dangarfield closed 1 year ago

dangarfield commented 1 year ago

After a while away, I'm getting back into FF7 and I've seen some recent commits, but cannot view the walkmesh or backgrounds in the WalkmeshWidget. Great job as always.

Mac: Ventura 13.0.1, Intel Core i9, Radeon Pro Integrated Makou Reactor: Unstable Build based on 1beed4e0fc17e2562e2e2b349f97d39fe59bdf51 (Downloaded today - 12 Dec)

It's also worth mentioning that I can't see any field models. Neither in this latest release or 1.8.5 (macosx).

Application Logs:

default 10:36:29.854743+0000    Makou Reactor   No persisted cache on this platform.
default 10:36:29.855473+0000    Makou Reactor   Failed to copy the SysCfgDict MG key with error: 0
default 10:36:29.903829+0000    Makou Reactor   Current system appearance, (HLTB: 1), (SLS: 0)
default 10:36:29.903969+0000    Makou Reactor   Current system appearance, (HLTB: 1), (SLS: 0)
default 10:36:29.905098+0000    Makou Reactor   Post-registration system appearance: (HLTB: 1)
default 10:36:29.926738+0000    Makou Reactor   Received configuration update from daemon (initial)
default 10:36:29.946515+0000    Makou Reactor   CHECKIN: pid=97369
default 10:36:29.955502+0000    Makou Reactor   CHECKEDIN: pid=97369 asn=0x0-0x108af89f foreground=1
default 10:36:29.958590+0000    Makou Reactor   FRONTLOGGING: version 1
default 10:36:29.958744+0000    Makou Reactor   Registered, pid=97369 ASN=0x0,0x108af89f
default 10:36:29.963671+0000    Makou Reactor   BringForward: pid=97369 asn=0x0-0x108af89f bringForward=1 foreground=1 uiElement=0 launchedByLS=1 modifiersCount=1 allDisabled=0
default 10:36:29.964054+0000    Makou Reactor   BringFrontModifier: pid=97369 asn=0x0-0x108af89f Modifier 0 hideAfter=0 hideOthers=0 dontMakeFrontmost=0 mouseDown=0/0 seed=0/0
default 10:36:29.964211+0000    Makou Reactor   BringForward: pid=97369 asn=0x0-0x108af89f
default 10:36:29.964256+0000    Makou Reactor   SetFrontProcess: asn=0x0-0x108af89f options=0
default 10:36:30.251182+0000    Makou Reactor   order window: 8330 op: 0 relative: 0 related: 0
default 10:36:30.404206+0000    Makou Reactor   order window front conditionally: 8331 related: 0
default 10:36:30.406568+0000    Makou Reactor   Registering for test daemon availability notify post.
default 10:36:30.406712+0000    Makou Reactor   notify_get_state check indicated test daemon not ready.
default 10:36:30.406843+0000    Makou Reactor   notify_get_state check indicated test daemon not ready.
default 10:36:30.406965+0000    Makou Reactor   notify_get_state check indicated test daemon not ready.
default 10:36:30.453321+0000    Makou Reactor   Received DFR status change 0. status = 0x1
default 10:36:30.454102+0000    Makou Reactor   Received DFR status change 1. status = 0x1
default 10:36:30.457878+0000    Makou Reactor   SignalReady: pid=97369 asn=0x0-0x108af89f
default 10:36:30.458374+0000    Makou Reactor   SIGNAL: pid=97369 asn=0x0x-0x108af89f
default 10:36:30.579164+0000    Makou Reactor   Initializing connection
default 10:36:30.579221+0000    Makou Reactor   Removing all cached process handles
default 10:36:30.579292+0000    Makou Reactor   Sending handshake request attempt #1 to server
default 10:36:30.579318+0000    Makou Reactor   Creating connection to com.apple.runningboard
default 10:36:30.580313+0000    Makou Reactor   Handshake succeeded
default 10:36:30.580358+0000    Makou Reactor   Identity resolved as app<application.org.myst6re.makoureactor.134349048.134349180(504)>
default 10:36:30.587327+0000    Makou Reactor   Reacting to DFR status change 0. status = 0x1
default 10:36:30.591950+0000    Makou Reactor   Shortcuts Contextual Actions enabled
default 10:36:30.592758+0000    Makou Reactor   Reacting to DFR status change 1. status = 0x1
default 10:36:30.597038+0000    Makou Reactor   SetFrontProcess: asn=0x0-0x108af89f options=1
default 10:36:36.190782+0000    Makou Reactor   order window: 833e op: 0 relative: 0 related: 0
default 10:36:36.254601+0000    Makou Reactor   order window: 833e op: 1 relative: 833c related: 0
default 10:36:36.254868+0000    Makou Reactor   +[CATransaction synchronize] called within transaction
default 10:36:36.257067+0000    Makou Reactor   order window front conditionally: 833c related: 0
default 10:36:36.257353+0000    Makou Reactor   order window: 833e op: 1 relative: 833c related: 0
default 10:36:36.427704+0000    Makou Reactor   +[CATransaction synchronize] called within transaction
default 10:36:38.975218+0000    Makou Reactor   order window: 833e op: 0 relative: 0 related: 0
default 10:36:38.978071+0000    Makou Reactor   order window: 833e op: 0 relative: 0 related: 0
default 10:36:38.978669+0000    Makou Reactor   order window: 833c op: 0 relative: 0 related: 0
default 10:36:38.997384+0000    Makou Reactor   order window: 8331 op: 1 relative: 833c related: 0
default 10:36:38.997471+0000    Makou Reactor   order window: 833c op: 0 relative: 0 related: 0
default 10:36:38.997978+0000    Makou Reactor   order window: 833c op: 0 relative: 0 related: 0
default 10:36:38.998647+0000    Makou Reactor   order window: 833c op: 0 relative: 0 related: 0
default 10:36:39.001879+0000    Makou Reactor   order window: 833f op: 0 relative: 0 related: 0
default 10:36:39.003844+0000    Makou Reactor   order window: 833c op: 0 relative: 0 related: 0
default 10:36:39.004631+0000    Makou Reactor   order window: 833e op: 0 relative: 0 related: 0
default 10:36:39.034116+0000    Makou Reactor   order window front conditionally: 8331 related: 0
default 10:36:39.040776+0000    Makou Reactor   order window: 8344 op: 1 relative: 8331 related: 0
default 10:36:39.304451+0000    Makou Reactor   order window: 8344 op: 1 relative: 8331 related: 0
default 10:36:39.429657+0000    Makou Reactor   order window front conditionally: 8331 related: 0
default 10:36:39.694217+0000    Makou Reactor   order window: 8344 op: 0 relative: 0 related: 0
default 10:36:39.694781+0000    Makou Reactor   order window: 8344 op: 0 relative: 0 related: 0
default 10:36:40.276700+0000    Makou Reactor   LSExceptions shared instance invalidated for timeout.
default 10:37:44.273802+0000    Makou Reactor   QOpenGLShader::compile(Vertex): ERROR: 0:1: '' :  version '130' is not supported
default 10:37:44.273842+0000    Makou Reactor   *** Problematic Vertex shader source code ***
#version 130
#define lowp
#define mediump
#define highp
#line 1

in vec4 a_position;
in vec4 a_color;
in vec2 a_texcoord;

out vec4 v_color;
out vec2 v_texcoord;

uniform mat4 modelMatrix;
uniform mat4 projectionMatrix;
uniform mat4 viewMatrix;
uniform float pointSize;

void main()
{
    mat4 mat = projectionMatrix * viewMatrix * modelMatrix;

    gl_Position = mat * a_position;
    gl_PointSize = pointSize;
    v_color = a_color;
    v_texcoord = a_texcoord;
}

***
default 10:37:44.273868+0000    Makou Reactor   Cannot compile main.vert "ERROR: 0:1: '' :  version '130' is not supported\n"
default 10:37:44.273886+0000    Makou Reactor   Warning during main.vert compilation "ERROR: 0:1: '' :  version '130' is not supported\n"
default 10:37:44.274003+0000    Makou Reactor   QOpenGLShader::compile(Fragment): ERROR: 0:1: '' :  version '130' is not supported
default 10:37:44.274035+0000    Makou Reactor   *** Problematic Fragment shader source code ***
#version 130
#ifdef GL_KHR_blend_equation_advanced
#extension GL_ARB_fragment_coord_conventions : enable
#extension GL_KHR_blend_equation_advanced : enable
#endif
#define lowp
#define mediump
#define highp
#line 1

in vec4 v_color;
in vec2 v_texcoord;

out vec4 o_color;

uniform sampler2D tex;

void main()
{
    vec4 _color = v_color;

    // Integrate texture color with the current vertex color
    if (v_texcoord.x > 0.0 || v_texcoord.y > 0.0)
    {
        vec4 _texColor = texture2D(tex, v_texcoord.xy);

        if (_texColor.a == 0.0) discard;

        _color *= _texColor;
    }

    o_color = _color;
}

***
default 10:37:44.274055+0000    Makou Reactor   Cannot compile main.frag "ERROR: 0:1: '' :  version '130' is not supported\n"
default 10:37:44.274074+0000    Makou Reactor   Warning during main.frag compilation "ERROR: 0:1: '' :  version '130' is not supported\n"
default 10:37:44.274111+0000    Makou Reactor   QOpenGLShader::link: ERROR: One or more attached shaders not successfully compiled
default 10:37:44.274132+0000    Makou Reactor   Cannot link the program "ERROR: One or more attached shaders not successfully compiled\n"
default 10:37:44.274148+0000    Makou Reactor   QOpenGLShader::link: ERROR: One or more attached shaders not successfully compiled
default 10:37:44.274163+0000    Makou Reactor   Cannot bind the program
default 10:37:44.274179+0000    Makou Reactor   QOpenGLShaderProgram::uniformLocation(tex): shader program is not linked
default 10:37:44.281755+0000    Makou Reactor   order window: 834b op: 1 relative: 0 related: 0
myst6re commented 1 year ago

Unfortunately I know this issue, Mac OS does not support this shader. Use the Window or Linux version for now

I have two possible solutions in mind for a future fix:

CC @julianxhokaxhiu

dangarfield commented 1 year ago

I just read that too. That was news to me, I didn't realise about the history of Apple and OpenGL.

Great job as always. Much appreciated. I'll leave it up to you as to whether you leave this open or close this issue. No urgency from my perspective.

myst6re commented 1 year ago

I keep the issue, thanks for opening one :)

julianxhokaxhiu commented 1 year ago

I have this still in my TODO, thanks for the heads up. I'll check what progress has been made on QT6 now and how easy would it be to replace the GL layer with a more cross-platform rendering one. The last time I checked the QT6 framework was not providing a low-level library a-la bgfx for eg. But if there's one LMK and I'll be happy to port the code!

myst6re commented 1 year ago

It might be intersting, but in Qt 6.4, QQuickWidget can now render QML scene with Metal

julianxhokaxhiu commented 1 year ago

Thanks, I'll check it in the next days and I'll let you know if I succeeded in getting something up and running.

Can you please assign this issue to me so I do not forget? Ty!

myst6re commented 1 year ago

Thank you!

myst6re commented 1 year ago

Fixed by @julianxhokaxhiu