[1.11.2] Multiple Questions

[JTK222]

Heyho folks,

even though I don't like to ask and prever to figure stuff out my self, I think this might be best for my mod and it's users

 

My first question is:

How can I load Models for Blocks and Items only if required. The following thread inspired me to do this: http://www.minecraftforge.net/forum/topic/55133-how-many-models-is-too-many/#comment-262773

I tried the stuff mentioned in there, but it sadly caused only crashes. And in addition I think this can be done a bunch easier, all I want is to load a model according to it's blockstate json as it's required.

I would appreciate if anybody could help me with this, as my Mod increases the RAM usage already by 1GB and that is of course to much D:

 

My second Question:

How could I port my current Storage Blocks to Capabilities, I got it working without the problem, I never understood really how it was working (made it 1 (or even more) year ago).

Now as I've understood at least NBT and Capabilities I want also try to move the storage to capabilities, but this time I would like to understand it earlier and not a few months later. 

I would be happy about maybe a link to a tutorial with a good Documentation or something similar where it's explained.

Even though my biggest trouble with it was always the the GUI and synchronization stuff.

Here is my current code if anybody is interested: 

https://github.com/DarkRoleplay/Dark-Roleplay-Medieval/blob/master/src/main/java/net/drpmedieval/common/gui/container/ContainerCrate.java
https://github.com/DarkRoleplay/Dark-Roleplay-Medieval/blob/master/src/main/java/net/drpmedieval/common/blocks/storage/Crate.java
https://github.com/DarkRoleplay/Dark-Roleplay-Medieval/blob/master/src/main/java/net/drpmedieval/common/blocks/tileentitys/TileEntityCrate.java

I would really appreciate any kind of help and thanks in advance.

[Jay Avery]

1 hour ago, JTK222 said:

My first question is:

How can I load Models for Blocks and Items only if required. The following thread inspired me to do this: http://www.minecraftforge.net/forum/topic/55133-how-many-models-is-too-many/#comment-262773

I tried the stuff mentioned in there, but it sadly caused only crashes. And in addition I think this can be done a bunch easier, all I want is to load a model according to it's blockstate json as it's required.

I would appreciate if anybody could help me with this, as my Mod increases the RAM usage already by 1GB and that is of course to much D:

Post the code you tried and the crash(es) that resulted.

 

It can't really be done in a much easier way. The standard vanilla system bakes all models at start-up, so if you want to do something other than that, you need to alter the model loading and baking steps yourself. But that said, it really isn't as complicated as it might seem. For a simple system (i.e. one that would be simple enough to render using ordinary JSON blockstates), you can do it all within one object which implements ICustomModelLoader, IModel, and IBakedModel:

• Register your object with ModelLoaderRegistry.registerLoader.
• accepts should check for your block (probably by checking that the ResourceLocation path contains your block's registry name).
• loadModel should load all the models you will need from ModelLoaderRegistry.getModel (or one of the variant methods like getModelOrLogError) by passing the ResourceLocations of the json files, and store those in fields to use later. Then it can return itself.
• bake should do nothing except store the parameters as fields to use later, and return itself.
• getQuads should do the actual baking as needed. From the IBlockState parameter, decide which model(s) are needed for the block. Use your previously-stored IModel fields, call bake (using your previously-stored parameters) and then getQuads and return the result.
  • You should also have a cache to store models which have already been created. In simple cases it can just be a Map<IBlockState, List<BakedQuad>>. The first step in your getQuads should be to check whether the state has a cached result (and return it if so), and the last step should be to add the result to the cache if it's not already present.

For simple cases, a lot of this can be done automatically and the same for every case (rather than needing to write it individually for every block). I have an abstract class which I use for all the stuff that's the same every time here. Then I only have to write loadModel and getQuads individually, which is not really any more complicated than writing a big messy blockstates file.

[jeffryfisher]

Can users fake it by adding  virtual memory? Then unused models would be paged out.

[JTK222]

@Jay Avery Thank you! Your explanation helped me a lot ^^ And I got it working (somehow)

But one question is there a way to get the Model Path that would be used by default for a blockstate?

I am asking because I have to many different blocks and making the loadModel and getQuads methods would take to long.

In case no such method exists I will write something for that by myself.

[Jay Avery]

What do you mean by the default model path for a blockstate? For a normally-rendered block you'd have to write a blockstates file telling it which model(s) to render depending on the state - in this case you're just doing the same thing in Java instead of json. Or did I misunderstand your question?

[JTK222]

I would like to read the path from the json, as making it in java would cause to much work. I have currently 50 Blocks that require a delayed baking process.

And changing all that to java would take a bunch of time D:

[Jay Avery]

Oh I see! I'd try looking at how the normal model loading system reads blockstates files, and see if you can use or copy some of the code from there.

[JTK222]

Ok found a solution, only one problem left D:

 

I cant figure out in which format the Keys for the variants in ModelBlockDefinition are.
The variable is private and the only getter method for this returns it's values. Does anybody maybe know the format?
Or tell me how to figure it out by myself? (I guess that would be possible with reflections even though I don't have much experience with that)

[Leviathan143]

15 hours ago, JTK222 said:

Ok found a solution, only one problem left D:

 

I cant figure out in which format the Keys for the variants in ModelBlockDefinition are.
The variable is private and the only getter method for this returns it's values. Does anybody maybe know the format?
Or tell me how to figure it out by myself? (I guess that would be possible with reflections even though I don't have much experience with that)

The keys look to be the variant names. To clarify, they are the strings you put before a variant to specify the conditions that should cause that variant to be used. Examples of variant names include "facing=up", "snowy=true", "facing=top, part=top", etc.

 

I figured this out by creating exit breakpoints on both of the constructors of ModelBlockDefinition. Then I launched in debug mode; when execution suspended due to one of the breakpoints, I hovered over mapVariants in the source and looked at the string representation of it in the lower popup. I'm using Eclipse, but IDEA should have something similar.

 

Edited May 11, 2017 by Leviathan143

[JTK222]

@Leviathan143 Thank you! But I figured it already out yesterday night using reflexions.

 

@Jay Avery Ok, so it can find the models and everything there are no more errors. But the problem is that the Block is now invisible D:
I've been trying now the whole day to find the cause for this but everything seems right. Here is the source code: https://github.com/DarkRoleplay/Dark-Roleplay-Medieval/blob/1.11/src/main/java/net/dark_roleplay/medieval/client/model_baking/DelayedBakedModel.java

[Jay Avery]

Are the appropriate methods being called (e.g. accepts, loadModel and bake during startup, getQuads when the block is placed or the state is changed)? If the methods are being called correctly, step through them (or use print statements) to see whether the flow is what you expect it to be (e.g. is a condition returning false when you expect it to be true, is something null at the wrong time). I can see you've already got some print and log statements in the code, could you show a log from starting up the game and placing one of your blocks?

[JTK222]

Latest.log: https://pastebin.com/C3NFivLm
fml-client-latest.log: https://pastebin.com/TJ07CDet

I have removed a few of the debug messages becase there were over thousand lines for my block D:
 

As I said everything should work. The Models get loaded they just don't render. All I could imagine is to test if the Baked Models do contain any quads D:

[Jay Avery]

7 minutes ago, JTK222 said:

Latest.log: https://pastebin.com/C3NFivLm
fml-client-latest.log: https://pastebin.com/TJ07CDet

I have removed a few of the debug messages becase there were over thousand lines for my block D:
 

As I said everything should work. The Models get loaded they just don't render. All I could imagine is to test if the Baked Models do contain any quads D:

That's what I'd try next! See what result contains just before it's returned from getQuads. It may be empty for some reason, and then you could backtrack and find out why it's not being filled.

[JTK222]

Ok... it seems that the list never gets filled. There are also no quads during baking. I have no clue why that's not working.

I think I have to figure out how that stuff with breakpoints works to find a solution

[Jay Avery]

As I suspected! I didn't use your blockstates file technique, so my bet is that it's somewhere in that process that models aren't being found or loaded correctly.

 

I just had another poke around. In the vanilla blockstates process, BlockStateLoader.load is called with a specific Gson as the second parameter, not a generic new parameterless constructor one. The method ModelBlockDefinition.parseFromReader calls it using its own Gson which seems to be set up to read blockstates, so I'd give that a shot! (Disclaimer, I have no direct experience of reading json resources, but this is what I would be doing if I wanted to work it out :P).

 

Edit: But yes, becoming comfortable with using breakpoints and the debugger is always a good idea!

Edited May 11, 2017 by Jay Avery

[JTK222]

Well. I kind of fixed it.... all I had to do is to pass null as the side on getBakedQuads methods.

There are still a few bugs, but as I've been sitting on this for more than 5 days now I will make a break on that.