Lighting is wrong and missing volumetric effects #226
Labels
No labels
accessibility
autorelease: pending
autorelease: tagged
bug
dependencies
documentation
duplicate
enhancement
github_actions
good first issue
help wanted
invalid
question
testing wanted
wontfix
No milestone
No assignees
1 participant
Due date
No due date set.
Dependencies
No dependencies set
Reference
MagicBeanDev/Hookshot#226
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
GPU : rtx 4090
CPU : Intel Core i7-13700KF
RAM: 32 GB
First off ,thank you for your efforts. It appears that the lighting is wrong/ overblown. Also the volumetic fog/mist is entirely missing. I have attached 2 screenshots of how it looks vs how it's supposed to look.
recompilation:
Switch 2:
Same symptoms here on NVIDIA / Linux: one flat, overblown light in Kokiri Forest, no soft falloff and no cast shadows. Light flashing and VRAM stalls are gone since today's lighting/VRAM commits, but this part remains, so it doesn't look vendor- or OS-specific.
Setup
mainat2a9fe7f(Linux support) with SDL3 instead of SDL2. Renderer code is unchanged frommain.e0f3ccfa, 5.5 min session in Kokiri Forest, 55 fps average, graphics memory peak 7.7 GBPossibly related, from the log (a lead, not a confirmed cause)
In one burst at ~184–185 s, compute programs were dispatched with a storage buffer the replay could not bind (
vk: storage buffer NOT bound (reads zeros)):755a777ada52677d78ccf14779ec3efb7650519f88e59b19102f5b0abb0f1e63a54f19e927d8faeff0fed2bbccae7bdc,cb2e06ddfb9d7d27,5b9ba94d1c822e80,499dbfd0721d2858,3e6fe279463b020eNotes:
vk_draw.cpp), so all 40 were used up in that burst. I can't tell whether these programs keep running unbound afterwards or whether it was only a scene change.a54f19e927d8faef's "size" is0x3f800000(float 1.0), so that c3+0x60 slot looks like a constant read as an address/size pair rather than a buffer descriptor.texture slot ... is not registered: sampled as the dummy(slots 0x46040, 0x2e000, 0xde040, 0xc6000), and nodoes not lowerprograms.Since volumetric fog and light gathering are usually compute passes, these unbound compute buffers seemed worth flagging next to the missing fog/shadows.
I'm happy to run with
RX_VK_DUMP_SSBO/RX_VK_PEEK_PROG=755a777a(or anything else useful) and post the output.Thanks. Lighting that is too bright (bloom, overexposure) or too dark (shadows, faces, NPCs) is tracked in #36. https://github.com/NotRexed/Hookshot/releases/tag/foliage-linux-updater brings EvilOOT's lighting work (Link, signs and ferns are lit again, and the game's own fog), so please try it; what remains is followed in #36. Closing as a duplicate. Mabec-games' finding (storage buffers not bound in 755a777ada52677d and four other programs) is linked from #36.