Some models refuse to parse

[spassvogel]

A number of models will not parse properly. When centroids of geometry are being calculated there is a lookup on vertices using face, but in some cases this is a null reference, throwing an error.

Stack trace: THREE.Vector3.add (three.js:1531) THREE.Geometry.computeCentroids (three.js:8530) Mesh.parse (ColladaLoader.js:2402) Geometry.parse (ColladaLoader.js:2314) parseLib (ColladaLoader.js:247) parse (ColladaLoader.js:137) request.onreadystatechange (ColladaLoader.js:84)

How to reproduce: Uncomment out on of the models specified in model loader js files (eg PersModelLoader) that have PARSE ERROR!! at the end. Load the client and observe error being thrown.

[spassvogel]

List of models and any parsing / texture problems that occur

Rome

These models work. They use texture: textures/skins/structural/rome_struct_2.png

• [x] rome_barracks.dae
• [x] rome_blacksmith_struct_b.dae
• [x] rome_corral.dae
• [x] rome_dock.dae
• [x] rome_civic_house.dae
• [x] rome_fortress.dae
• [x] romefc.dae (farm)_
• [x] roman_house_a.dae
• [x] roman_house_b.dae
• [x] roman_house_c.dae
• [x] rome_market_struct1.dae
• [x] rome_market_struct2.dae
• [x] rome_scout_tower.dae
• [x] rome_storehouse.dae
• [x] rome_wall_gate.dae
• [x] rome_wall_long.dae
• [x] rome_wall_medium.dae
• [x] rome_wall_short.dae
• [x] rome_wall_tower.dae

These models use a custom texture.

• [ ] rome_temple_big.dae (uses textures/skins/structural/rome_struct.png)
• [ ] rome_civic_center.dae (I can't find the texture belonging to this :/ REMOVE material Map__379-Material in dae file)
• [ ] rome_temple_2.dae (uses textures/skins/structural/rome_struct_2.png BUT also another material`for the floor tiles that I cant find. REMOVE Map__1-Material in dae file)
• [ ] rome_civic_house.dae (I can't find the texture belonging to this :/ REMOVE Map__1-Material)

Hellenes

• [ ] hele_corral (missing roof)

[spassvogel]

List of models and any parsing / texture problems that occur

• Mauryans*

These models work. They use texture: art/textures/skins/structural/maur_struct.png

• [x] maur_wall_tower_struct.dae

These models have some problems

• [ ] maur_market.dae (refuses to load, vertex errors)
• [ ] maur_market_struct.dae (parts missing)
• [ ] maur_barracks.dae (TEXTURE MISSING)
• [ ] maur_storehouse.dae (This model uses art/textures/skins/structural/maur_struct_b.png)
• [ ] maur_farmstead.dae (This model uses art/textures/skins/structural/maur_struct_b.png)
• [ ] maur_fortress.dae (roof missing)
• [ ] maur_house_b.dae (parts missing)
• [ ] maur_wall_gate_palisade.dae (parts missing)
• [ ] maur_big_stupa_struct.dae (parts missing)
• [ ] maur_corral.dae (This model uses art/textures/skins/structural/maur_struct_b.png)
• [ ] maur_farmstead_prop_b.dae (TEXTURE MISSING)
• [ ] maur_big_stupa_gates.dae (TEXTURE MISSING, maybe this has to be combined with big_stupa_struct.dae)
• [ ] maur_house_d.dae parts missing)

(.. at this point I gave up. these are the other models, unchecked)

• [ ] maur_wall_gate_struct.dae
• [ ] maur_house_c_struct_b.dae
• [ ] maur_barracks_a_struct.dae
• [ ] maur_wall_long_palisade.dae
• [ ] maur_civic_centre.dae
• [ ] maur_stables_water.dae
• [ ] maur_house_a.dae
• [ ] maur_house_c.dae
• [ ] maur_docks_struct_b.dae
• [ ] maur_cc_struct.dae
• [ ] maur_scout_tower.dae
• [ ] maur_temple.dae
• [ ] maur_market_stalls_cloth.dae
• [ ] maur_misc_01.dae
• [ ] maur_farmstead_prop_a.dae
• [ ] maur_temple_struct.dae
• [ ] maur_docks_struct

[spassvogel]

Carthagenians

• [ ] kart_tower (the model is missing from the game?)

[spassvogel]

Possible solution: http://stackoverflow.com/questions/8678205/collada-to-json

1. Load models into Blender
2. Export to JSON
3. Load JSON models instead of using ColladaLoader
4. Profit

[mvoda]

Could work, but having some models in JSON rather than Collada would cause problems with our tablet thingy which only supports Collada, Wavefront or STL.

[flatmap13]

I think we have enough models, we could just leave out the ones that cause problems....?

[mvoda]

Yes, that's what I'm also thinking. :+1:

[spassvogel]

Well, I mean it could work for the beta, and I guess even the final prototype. It still sucks though because choosing anything other than Iber or Pers means you're gimping yourself because you can build less buildings ....

[flatmap13]

The users can build any type of building right? The user is not restricted to some race? Or am I missing something?

[spassvogel]

I thought that you had to choose an initial race (/empire /build style whatev) and you could gradually unlock the other ones? Like a rewarding mechanism?

[flatmap13]

We already restrict the types of structures the player can build with the resource mechanics right? And you "unlock" them by generating enough resources to build them. I don't really see why we also should restrict the player per race as well.

[spassvogel]

Well yeah, but the players are supposed to be doing this for at least 10 weeks... I think it's good to have some more progression mechanics in there. Like, having built 10 houses of race X enables you to make a house of race Y and so on.

[mvoda]

Neah, this seems too forced to me. I think allowing everybody to build everything from the start (as long as they have enough resources) works better.

[spassvogel]

OKay but then we're dropping the part where you have to choose your race at the beginning.

[flatmap13]

Yeah, I see no problem with that. Saves us work as well.

[spassvogel]

Fixed