Explain what path the DLSS files should be in.#136
Conversation
|
Making PRs for this is just you trying to be relevant and "make a contribution" |
I'm afraid auto-download isn't allowed under NVIDIA's DLSS EULA, which prohibits automated redistribution or programmatic fetching of the runtime libraries. That said, I do think we can make a big difference with a simple logging improvement, by explicitly printing the expected absolute file path in the error message. For example, telling users exactly where to place the DLL file. This is especially helpful because the actual runtime directory isn't always the obvious mods folder. With modern Minecraft launchers, the working directory can vary significantly. Giving users the exact absolute path removes all the guesswork and saves them from digging through launcher logs just to figure out where "the right spot" even is. |
Radiance is throwing an error when the DLSS libraries are missing.
I noticed the error is very vague. Mutiple people got this error after installing on Modrinth, even after putting the dll files in what seems like the right spot.
I don't think missing DLSS should cause an error that crashes the game though, at worst it should disable DLSS functionality. Ideally I would also replace radianceDir.resolve with a new function like resolveAndCheck(radianceDir, "nvngx_dlssd.dll") which then throws an error or just writes to the console which exact location every missing file is so the user can directly check that location. You can also make a function resolveAndCheck("nvngx_dlssd.dll") that doesn't need the path as param.
radianceDir should not be a static Path, it should be a property. I see that it's only referenced outside the class at 3 other places (Options.java and ModuleEntry.java). These can be provided the path via dependency injection or just from the RadianceClient instance.
It would also be pretty easy to auto download the dll files if that's allowed, or you can just add a link to where to download the dll files that gets logged in console or shown in a pop-up.
Let me know if you'd want me to implement anything.
Here's the stack trace for anyone facing this problem: