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

Sikkmod: SSAO with Crytek-method gives weird artifacting #37

Open
Eonfge opened this issue Jul 29, 2024 · 8 comments
Open

Sikkmod: SSAO with Crytek-method gives weird artifacting #37

Eonfge opened this issue Jul 29, 2024 · 8 comments
Labels
old bug out of scope The bug already existed in the original Doom3 mod, out of scope for me, but contributions welcome

Comments

@Eonfge
Copy link

Eonfge commented Jul 29, 2024

ssao-crytek

game/caverns1.map at the starting location

Details

Discovered while looking into #36

I also found that a few other SSAO methods are really dark, which might be in the spirit of Doom 3, but it might also point towards some technical problems.

Version

1.5.4 RC2, which you can find here: flathub/org.dhewm3.Dhewm3#26

@Eonfge
Copy link
Author

Eonfge commented Jul 29, 2024

Also minor complaint. notarget 1 doesn't seem to do shit against those spiders. Is that scripted?

@DanielGibson
Copy link
Member

DanielGibson commented Jul 29, 2024

I found that some sikkmod effects don't look right when gamma in shaders is enabled.
No idea if that would help here..
I want to try to fix the gamma in shaders issues eventually, if possible, but not before 1.5.4

Also minor complaint. notarget 1 doesn't seem to do shit against those spiders. Is that scripted?

Don't know either.

Also note that I have no intentions of fixing Sikkmod (or any other mods here), unless the bugs were introduced by the dhewm3 port (i.e. don't happen when using the original mod with Vanilla Doom3) or cause crashes or similar.
It's just too many mods for this to be feasible :-/

However, I'll keep such bug reports open in case someone else wants to work on them.

@Eonfge
Copy link
Author

Eonfge commented Jul 29, 2024

Also note that I have no intentions of fixing Sikkmod (or any other mods here), unless the bugs were introduced by the dhewm3 port (i.e. don't happen when using the original mod with Vanilla Doom3) or cause crashes or similar.

That's totally reasonable. You're already doing the community a great service while you maintain Dhewm 3, and it doesn't have to be all on you. Perhaps you should create a label and flag issues with "Out of current scope" or"'assistance appreciated" so you easily communicate which issues you want to address and which ones not.

In this case, I would flag this issue and leave it as it.

@DanielGibson
Copy link
Member

Well, we could try to determine if this issue also happens with orig Doom3/Sikkmod

@Eonfge
Copy link
Author

Eonfge commented Jul 29, 2024

Well... that gives a different bug:

ssao-crytek-proton

The checkerboxed pattern that is on the entire screen is caused by the same SSAO

@DanielGibson
Copy link
Member

Define "that" ;)
Is this original Sikkmod, or gamma in shaders disabled, or what?

Btw, I created the old bug out of scope label

@Eonfge
Copy link
Author

Eonfge commented Jul 29, 2024

That: SSAO + Crytek method

I tried this with Vanilla Doom 3 using Proton 9 from Valve. I don't have access to a Windows XP computer so I can't tell you how period correct this rendering is, but I think that Sikkmod did some OpenGL trickery that no longer flies.

@DanielGibson
Copy link
Member

It can also heavily depend on the driver.
For example, the soft particle code from TDM I ported to dhewm3 only seemed to work properly with nvidia's driver, until I fixed the shader.
No idea if that was different when it was written or no one ever noticed the missing particles (they weren't rendered at all with other drives I tested, incl. some Mesa drivers and AMDs Windows driver)..

Anyway, I think we can probably apply the new label here;)

@DanielGibson DanielGibson added the old bug out of scope The bug already existed in the original Doom3 mod, out of scope for me, but contributions welcome label Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
old bug out of scope The bug already existed in the original Doom3 mod, out of scope for me, but contributions welcome
Development

No branches or pull requests

2 participants