-
Notifications
You must be signed in to change notification settings - Fork 58
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
CommunityShaders.json is generated blank with PrivateProfilereDirector < 0.6.2 #624
Comments
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Documentating my analysis here as I get distracted too often. I inserted some debug logs to find out where the issue originates. Within state.cpp method Save progresses fine until the following line, stopping there: If I dig deeper into Upscaling::SaveSettings method (Upscaling.cpp) I can see that the flow proceeds until: The class iniSettingCollection does not have a WriteSetting method but inherits from SettingCollectionList from CommonLibSSE NR. As this is a foundational library I do not think there is much use digging into this class. May have to try some debug output of settings / wait or try-catch. |
Thanks. You should consider perma linking to the code in the repo so we can easily cross reference. Otherwise I gotta dig into whatever branch it's on to find the code. |
This comment was marked as outdated.
This comment was marked as outdated.
Just learned something about permalinks :-) Introducing a 5 second wait before doing any save did not help |
I just had a fluke. |
I saw that PrivateProfileRedirector was updated on 18th June: This update led to a lot of changes to INI processing features. Most were reimplemented to conform known specifications and assumtions based on the mod scope (Bethesda games and their mods), but the new implementation may differ from the old versions behavior. The new behavior is, hopefully, more correct. I tested with CommunityShaders_AIO-2024-10-11T23-30Z and PrivateProfileRedirector version 0.5.3 and 0.6.2. So simple fix, users reporting the issue should be asked to update PrivateProfileRedirector. |
Probably should detect privateprofileredirector and either require an update or just not try to save settings. |
I believe we can use Commonlib's LookupLoadedModByName to detect the plugin, but I am not sure how to check for the version |
No that won't do it. LookupLoadedModByName is just for plugins. PPD is a dll so we'd have to detect it onload like we detect other dlls. We'll probably want to do a version check or something and spit out messages, etc. Might be a good first issue for someone or I might get to it. |
Thank you for the hint. Feel free to ingore my attempts and implement this if you like. |
No worries. You contributed enough finding out the cause. |
Linked issue: 536
Issue identified in CS version: 2024-09-17 VR Screen Space Reflections - RC and all following builds up to 2024-10-10
Skyrim version used: SkyrimVR
Reporting users: jepster
Reproducable: Yes, every start of the game
Initial analysis:
Workaround: The workaround for this is to obtain a valid CS.json and place it in overwrite or a mod \SKSE\Plugins. With the next game start the shadercache is created correctly and everything is fine.
The text was updated successfully, but these errors were encountered: