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

[Feature]: New Shader Support (Engine) #1406

Merged
merged 23 commits into from
Nov 1, 2023

Conversation

yohjimane
Copy link
Contributor

@yohjimane yohjimane commented Jul 24, 2023

  • Support Ascii's screen space shaders
  • Support Kennedith/KennShade's enhanced shaders
  • Support Beef's NVG shaders
  • Support Anomaly gasmask shaders

@github-actions github-actions bot added AI Artificial Intelligence Renderer labels Jul 24, 2023
@yohjimane yohjimane requested a review from Xottab-DUTY July 26, 2023 01:01
@yohjimane yohjimane force-pushed the feature/shader-support-engine branch 3 times, most recently from f7097fc to df15d2e Compare July 29, 2023 15:52
@yohjimane yohjimane force-pushed the feature/shader-support-engine branch 11 times, most recently from 63c6100 to f4e6fbc Compare August 9, 2023 15:56
@Xottab-DUTY Xottab-DUTY added Enhancement Player Experience Call of Pripyat This can affect Call of Pripyat compatibility, or related to this game. labels Aug 27, 2023
@yohjimane yohjimane force-pushed the feature/shader-support-engine branch from f4e6fbc to 6c3a3b4 Compare August 28, 2023 13:51
@yohjimane
Copy link
Contributor Author

Due to size, this branch will no longer receive any more updates. Any additional updates for this functionality (e.g. UI for new options, script functions ported to engine, etc) will be in separate PRs.

As of now, the support for using new shaders is in place, so I consider this feature complete.

@Neloreck
Copy link
Contributor

@Xottab-DUTY
Any blockers / help with testing needed?

@Xottab-DUTY
Copy link
Member

Xottab-DUTY commented Sep 21, 2023

@Xottab-DUTY Any blockers / help with testing needed?

Any reviews are appreciated.

But we currently have FPS regression after multithreaded renderer changes, and little graphical bugs when multithreading is enabled. I want to make sure that I won't add any additional work to @vTurbine (which I asked to fix these problems), so I'm waiting until at least FPS regression is fixed before merging this PR.

@Xottab-DUTY Xottab-DUTY force-pushed the feature/shader-support-engine branch from 039211d to bff6f77 Compare November 1, 2023 20:25
vis cleanup is done in CDetailManager::UpdateVisibleM. For reference –
d6e2d6a.
@Xottab-DUTY Xottab-DUTY merged commit 0b5417f into OpenXRay:dev Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AI Artificial Intelligence Call of Pripyat This can affect Call of Pripyat compatibility, or related to this game. Enhancement Environment (Weather) Player Experience Renderer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support new shaders from Ascii1457, Kennshade, TheRealBeef, Anomaly Team
3 participants