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: Optifine CIT Cache #973

Closed
wants to merge 6 commits into from

Conversation

CalMWolfs
Copy link
Collaborator

@CalMWolfs CalMWolfs commented Feb 2, 2024

Written by @nea89o

Gain 20-50% extra performance while using a skyblock cit pack
Will not work in dev environment!
/shoptifinecache shows stats about the cache over the last 100 ticks

Test in hub 1 looking forward while using furfsky
No cit pack
no cit

With cit pack
cit

With cit pack and option on
with option

Conclusion is that the setting works well but cit packs are still horrible for performance

@hannibal002 hannibal002 added this to the Version 0.23 milestone Feb 2, 2024
@CalMWolfs
Copy link
Collaborator Author

Closed by the request of @nea89o

@CalMWolfs CalMWolfs closed this Feb 4, 2024
@qtlunya
Copy link
Contributor

qtlunya commented Feb 4, 2024

Fair enough, but it's unfortunate that it's in a cheat mod, so we can't exactly go around recommending it to people.

@CalMWolfs
Copy link
Collaborator Author

Ill port it with another performance improvement patch i am working on into its own standalone mod

I feel that this is the best solution and sorry for any problems I caused through this pull request that was never my intention.

@jani270
Copy link
Collaborator

jani270 commented Feb 4, 2024

Fair enough, but it's unfortunate that it's in a cheat mod, so we can't exactly go around recommending it to people.

Ill port it with another performance improvement patch i am working on into its own standalone mod

More standalone mods is always a good idea.

@CalMWolfs CalMWolfs deleted the optifine-cit-thing branch February 10, 2024 02:24
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.

5 participants