Custom models using obj's not working

[dragonwhisper92]

Current files https://github.com/Dragonwhisper92/FF7

custom models wont load

 

all i see in game is a purple/black texture (or the missing texture texture) and text that reads ff7:ff7:buster_sword#inventory

 

please help me fix this

[Draco18s]

This event:

https://github.com/Dragonwhisper92/FF7/blob/master/main/java/com/finalfantasy7mod/proxy/ClientProxy.java#L44.

Runs after this event:

https://github.com/Dragonwhisper92/FF7/blob/master/main/java/com/finalfantasy7mod/proxy/ClientProxy.java#L21

 

Problematic Code problem numbers 2, 7

Code Style issue number 1

 

Edited December 31, 2017 by Draco18s

[dragonwhisper92]

10 minutes ago, Draco18s said:

This event:

https://github.com/Dragonwhisper92/FF7/blob/master/main/java/com/finalfantasy7mod/proxy/ClientProxy.java#L44.

Runs after this event:

https://github.com/Dragonwhisper92/FF7/blob/master/main/java/com/finalfantasy7mod/proxy/ClientProxy.java#L21

This i can fix and will do soon

 

11 minutes ago, Draco18s said:

Problematic Code problem numbers 2, 7

If you could help me fix this that would be great

 

11 minutes ago, Draco18s said:

Code Style issue number 1

Yes i know about this but prefer doing it this way

[dragonwhisper92]

9 hours ago, diesieben07 said:

I also prefer calling my car "tree". Very helpful in communicating with others!

well first off thanks for the helpful comment secondly is it such a big problem, like i bet there hundreds of other code styles we should all follow, I've done it this way as i find it easier, if that's an issue feel free to change it for me, if not please stop commenting

[dragonwhisper92]

So i now have the model render but the textures wont any help???

https://github.com/Dragonwhisper92/FF7

Edited December 31, 2017 by dragonwhisper92

[dragonwhisper92]

15 hours ago, Draco18s said:

Problematic Code problem numbers 2

also i thought i was doing this correct? where am i doing it wrong?

[Draco18s]

I can't tell. You have your code redirecting things all over the place and while it looks fine, I can't spot the issue.

Use the debugger.

 

One of the things that might make things easier is to not let Item classes handle their own models. There is nothing that requires access to protected values.

 

For example, if I look at my code...and ignore lines 181 to 196 (those lines handle custom state mappers and custom mesh definitions, which DO require access to protected values) you'll see that I can register all the models I need without doing anything fancy. I do use a wrapper object for the array (due to having migrated the code from 1.10 where the ModelRegistryEvent didn't exist and I was registering items and models at the same time).

[dragonwhisper92]

7 hours ago, Draco18s said:

I can't tell. You have your code redirecting things all over the place and while it looks fine, I can't spot the issue.

Use the debugger.

 

One of the things that might make things easier is to not let Item classes handle their own models. There is nothing that requires access to protected values.

 

For example, if I look at my code...and ignore lines 181 to 196 (those lines handle custom state mappers and custom mesh definitions, which DO require access to protected values) you'll see that I can register all the models I need without doing anything fancy. I do use a wrapper object for the array (due to having migrated the code from 1.10 where the ModelRegistryEvent didn't exist and I was registering items and models at the same time).

Finally a helpful comment thanks so much, ill defiantly look into these and see what I can come up with