Skip to content
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

Walkmesh and Background not visible [develop branch] #165

Closed
dangarfield opened this issue Dec 12, 2022 · 8 comments
Closed

Walkmesh and Background not visible [develop branch] #165

dangarfield opened this issue Dec 12, 2022 · 8 comments
Assignees

Comments

@dangarfield
Copy link
Contributor

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 1beed4e (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
Copy link
Owner

myst6re commented Dec 12, 2022

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:

  • Move to Core Profile for mac OSX (I already tried that, but there is something Mac does not like about glDrawElements)
  • Move to Qt Quick which support Metal renderer natively

CC @julianxhokaxhiu

@dangarfield
Copy link
Contributor Author

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
Copy link
Owner

myst6re commented Dec 12, 2022

I keep the issue, thanks for opening one :)

@julianxhokaxhiu
Copy link
Contributor

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
Copy link
Owner

myst6re commented Dec 12, 2022

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

@julianxhokaxhiu
Copy link
Contributor

julianxhokaxhiu commented Dec 12, 2022

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
Copy link
Owner

myst6re commented Dec 12, 2022

Thank you!

@myst6re
Copy link
Owner

myst6re commented Feb 21, 2023

Fixed by @julianxhokaxhiu

@myst6re myst6re closed this as completed Feb 21, 2023
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

No branches or pull requests

3 participants