Lazy build the layered strategies, so it is not initialized before it's actually needed. Some models will never use their LayeredCache, so there's no use to initializing it before it's needed. #890
Labels
performance
Better or worse performance
Comments
1 task
SigmundGranaas
added a commit
that referenced
this issue
Apr 14, 2024
## Baked model selection rewrite Closes #870 Closes #865 **Goal**: Rebuild the model selection process to be more flexible and easier to maintain ## Changes * Removes the old monolith class that handled everything before * Removed itemstack cache keys * Removed the old * Removed dependency on the deprecated Fabric Model selection api * Adds a new 1-1 binding between items and the Unbaked model class * Implemented using Fabric's new ModelLoadingPlugin * Adds a system of decorator strategies to be able to quickly set up and build a BakedModel selection process * Added the following selectors: AsyncStrategy, SingleCachedStrategy, LayeredCachedStrategy, DefaultedBakedModelStrategy, ModelBakerStrategy ## Compatibility * Adds compat for Modernfix * Adds compat for Emissive Textures plugin ## Further work - [ ] #890
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This might make the model system more performant but it probably won't make a huge difference.
The text was updated successfully, but these errors were encountered: