MERGE PR 281

sm64pc#281

Author: quentin452

Makefile

@@ -39,6 +39,8 @@ TARGET_BITS ?= 0
 
 # Disable texture fixes by default (helps with them purists)
 TEXTURE_FIX ?= 0
+# Disable high poly models by default (all hail the purists)
+HD_MODELS ?= 0
 # Enable extended options menu by default
 EXT_OPTIONS_MENU ?= 1
 # Disable text-based save-files by default
@@ -601,6 +603,12 @@ ifeq ($(TEXTURE_FIX),1)
   CFLAGS += -DTEXTURE_FIX
 endif
 
+# Check high poly models options
+ifeq ($(HD_MODELS),1)
+  CC_CHECK += -DHD_MODELS
+  CFLAGS += -DHD_MODELS
+endif
+
 # Check for extended options menu option
 ifeq ($(EXT_OPTIONS_MENU),1)
   CC_CHECK += -DEXT_OPTIONS_MENU

actors/bowser/hd_model.inc.c

@@ -51,7 +51,11 @@ UNUSED static const u64 binid_10 = 10;
 #include "warp_collision/collision.inc.c"
 UNUSED static const u64 binid_11 = 11;
 
+#ifdef HD_MODELS
+#include "mario_cap/hd_model.inc.c"
+#else
 #include "mario_cap/model.inc.c"
+#endif
 UNUSED static const u64 binid_12 = 12;
 
 #include "power_meter/model.inc.c"

actors/common1.c

@@ -6,7 +6,11 @@
 #include "make_const_nonconst.h"
 
 // Note: This bin does not use bin IDs, unlike the other segmented bins.
+#ifdef HD_MODELS
+#include "mario/hd_model.inc.c"
+#else
 #include "mario/model.inc.c"
+#endif
 
 #include "bubble/model.inc.c"
 

actors/group0.c

@@ -14,7 +14,11 @@ UNUSED static const u64 binid_1 = 1;
 #include "yellow_sphere/model.inc.c"
 UNUSED static const u64 binid_2 = 2;
 
+#ifdef HD_MODELS
+#include "bowser/hd_model.inc.c"
+#else
 #include "bowser/model.inc.c"
+#endif
 #include "bowser/anims/data.inc.c"
 #include "bowser/anims/table.inc.c"
 #include "bowser/flames_pos.inc.c"

actors/group12.c

@@ -1,18 +1,15 @@
 """
 This module attempts to parse the ``model.inc.c`` files and extract the
-3D models within as standard Wavefront OBJ files. 
-
+3D models (including normals and texture coordinates) within as standard 
+Wavefront OBJ files. 
 Example:
     Specify the path to the ``.inc.c`` file and a directory where to save 
     the extracted ``.obj`` files.
-
         $ python c2obj.py ./actors/mario/model.inc.c ./actors/mario/obj/
-
 This is a work in progress and it currently has some serious limitations:
-    * It only extracts geometry information, so no textures or any other info
     * It makes assumptions about the layout of the code in the C source
+    * It assumes that the encoded texture coordinates are for a 32x32 map
     * It hasn't been properly tested.
-
 """
 
 def parse(filename, output_directory):
@@ -23,7 +20,7 @@ def parse(filename, output_directory):
             mkdir(output_directory)
         except OSError:
             print(f'Could not use output directory {output_directory}.')
-        
+
     vtx_def = 'static const Vtx '
     vtx_data = {}
     reading_vtx = False
@@ -55,14 +52,14 @@ def parse(filename, output_directory):
 
             if line.startswith(gfx_def):
                 from datetime import datetime
-                
+
                 current_gfx_name = line.split(' ')[2][:-2]
                 current_gfx_file = open(path.join(output_directory, current_gfx_name + '.obj'), 'w')
-                current_gfx_file.write("# Armando Arredondo's SM64 Wavefront OBJ Geometry Converter\n")
-                current_gfx_file.write('# File Created: {}\n\n'.format(datetime.now()))
+                current_gfx_file.write(f"# Armando Arredondo's SM64 Wavefront OBJ Geometry Converter\n")
+                current_gfx_file.write(f'# File Created: {datetime.now()}\n\n')
                 reading_gfx = True
                 continue
-            
+
             if line == end_of_block:
                 if reading_vtx:
                     vtx_data[current_vtx_name] = current_vtx_data
@@ -74,23 +71,23 @@ def parse(filename, output_directory):
                     current_gfx_vertices = 0
                     reading_gfx = False
                     gfx_count += 1
-                
+
                 continue
-            
+
             if reading_vtx:
                 line = line.replace('{', '[').replace('}', ']')
                 tri = eval(line[:-1])[0]
                 current_vtx_data.append(tri)
                 continue
-            
+
             if reading_gfx:
                 if line.startswith(insert_vert_call):
                     args = line[len(insert_vert_call):].split(',')
                     current_vtx_name = args[0]
-                    
+
                     if current_gfx_vertices > 0:
                         current_gfx_file.write(f'# {current_gfx_faces} faces\n\n')
-                    
+
                     current_gfx_faces = 0
                     current_vtx_vertices = len(vtx_data[current_vtx_name])
                     current_gfx_vertices += current_vtx_vertices
@@ -107,31 +104,53 @@ def parse(filename, output_directory):
                         current_gfx_file.write('vn {:.3f} {:.3f} {:.3f}\n'.format(*n))
                     current_gfx_file.write(f'# {current_vtx_vertices} vertex normals\n\n')
 
+                    texture_info = any(_has_texture_info(tri) for tri in current_vtx_data)
+                    if texture_info:
+                        for tri in current_vtx_data:
+                            u = _decode_texture(tri[2][0])
+                            v = 1 - _decode_texture(tri[2][1])
+                            current_gfx_file.write('vt {:.3f} {:.3f}\n'.format(u, v))
+                        current_gfx_file.write(f'# {current_vtx_vertices} texture coords\n\n')
+
                     current_gfx_file.write(f'g {current_vtx_name}\n\n')
-                
+
                 elif line.startswith(insert_2tri_call):
                     args = line[len(insert_2tri_call):].split(',')
                     correction = current_gfx_vertices - current_vtx_vertices + 1
                     indexes = [eval(args[i]) + correction for i in [0, 1, 2, 4, 5, 6]]
-                    current_gfx_file.write('f {0}//{0} {1}//{1} {2}//{2}\n'.format(*indexes[:3]))
-                    current_gfx_file.write('f {0}//{0} {1}//{1} {2}//{2}\n'.format(*indexes[3:]))
+                    if texture_info:
+                        current_gfx_file.write('f {0}/{0}/{0} {1}/{1}/{1} {2}/{2}/{2}\n'.format(*indexes[:3]))
+                        current_gfx_file.write('f {0}/{0}/{0} {1}/{1}/{1} {2}/{2}/{2}\n'.format(*indexes[3:]))
+                    else:
+                        current_gfx_file.write('f {0}//{0} {1}//{1} {2}//{2}\n'.format(*indexes[:3]))
+                        current_gfx_file.write('f {0}//{0} {1}//{1} {2}//{2}\n'.format(*indexes[3:]))
                     current_gfx_faces += 2
 
                 elif line.startswith(insert_1tri_call):
                     args = line[len(insert_1tri_call):].split(',')
                     correction = current_gfx_vertices - current_vtx_vertices + 1
                     indexes = [eval(args[i]) + correction for i in [0, 1, 2]]
-                    current_gfx_file.write('f {0}//{0} {1}//{1} {2}//{2}\n'.format(*indexes))
+                    if texture_info:
+                        current_gfx_file.write('f {0}/{0}/{0} {1}/{1}/{1} {2}/{2}/{2}\n'.format(*indexes))
+                    else:
+                        current_gfx_file.write('f {0}//{0} {1}//{1} {2}//{2}\n'.format(*indexes))
                     current_gfx_faces += 1
 
                 continue
-    
+
     print(f'{gfx_count} models extracted.')
 
+def _has_texture_info(tri):
+    (u, v) = tri[2]
+    return int(u) != 0 or int(v) != 0
+
 def _decode_normal(x):
     y = x if x <= 127 else x - 255
     return y / 127
 
+def _decode_texture(x):
+    return x / (2**10)
+
 if __name__ == "__main__":
     import argparse
     parser = argparse.ArgumentParser()