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

Replaced HardcoreDarkness with DynamicDarkness #817

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Nekulian
Copy link

[Dynamic Darkness] 0.4.x suffers from performance issues when used in conjunction with [OptiFine]
[Dynamic Darkness] 0.3.1 in my experience has no issues.
[Dynamic Darkness] 0.2.1 also doesn't have any issues from my experience.
Other modpacks seems to use 0.2.1 in particular so I've decided to go for this one, based on the assumption that 0.3.1 might have an issue that I'm not personally aware of

If any issues are noticed, then I implore anyone reading this comment to report back.

[Dynamic Darkness] 0.4.x suffers from performance issues when used in conjunction with [OptiFine]

[Dynamic Darkness 0.3.1] in my experience has no issues
[Dynamic Darkness 0.2.1] also doesnt have any issues in my experience.
Other modpacks seems to use this version in particular so I've decided to go for this one.

If any issues are noticed, then I implore anyone reading this comment to report back.
@Nekulian
Copy link
Author

Nekulian commented Dec 18, 2021

Personally recommend that you use [Pixel Reality - Lite] in conjunction with this mod if you want more shadow contrast
it seems to work as intended with 0.3.1 (and probably 0.2.1 though that is something I've yet to test)

Here's a link to the lightmap
https://www.curseforge.com/minecraft/texture-packs/pixel-reality-luminance/files/2404003

@CalaMariGold
Copy link
Contributor

I'd like to note that Dynamic Darkness recently updated, which fixed performance issues when used in conjunction with OptiFine.

I've used it in my own modpack in place of Hardcore Darkness for a bit now and it's worked just fine.

@ambrosia13
Copy link
Contributor

Looks good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants