Bug 5030 - IQM version 2 support
Status: RESOLVED FIXED
Alias: None
Product: ioquake3
Classification: Unclassified
Component: Misc
Version: GIT MASTER
Hardware: All All
: P3 normal
Assignee: Zachary J. Slater
QA Contact: ioquake3 bugzilla mailing list
URL:
Depends on:
Blocks:
 
Reported: 2011-06-08 13:29 EDT by Zack Middleton
Modified: 2011-06-11 16:20:38 EDT
2 users (show)

See Also:


Attachments
IQM version 2 support, drops version 1 support. Adds a warning for unsupported versions. (4.37 KB, patch)
2011-06-08 13:29 EDT, Zack Middleton
Update iqm.h for IQM v2, fixed a IQM warning (1.17 KB, patch)
2011-06-11 13:49 EDT, Zack Middleton

Description Zack Middleton 2011-06-08 13:29:10 EDT
Created attachment 2761 [details]
IQM version 2 support, drops version 1 support. Adds a warning for unsupported versions.

IQM format was updated on May 31 2011 to version 2. See http://lee.fov120.com/iqm/ The change is joints and poses now save quaternion w component.

The attached patch does not continue to support version 1, does anyone think ioq3 should?
Comment 1 Monk 2011-06-08 18:21:07 EDT
No, ioq3 should not support the older, broke version.  This is so new for ioq3 at least that there's no large library of "old" content that won't work anymore.  I mean, looking at this:

The IQM development kit can be downloaded here (iqm-2011-06-05.tar.gz, 2324KB). It includes:
- format descriptions for IQM and IQE 
- IQM compiler for MD5, SMD, and IQE models 
- Blender exporter (versions 2.49, 2.56, and 2.57) for both IQM and IQE formats 
- sample code/demos for loading and rendering IQM models 

If the exporters and the compiler are all updated to version 2, then again, I think it's wise to go with the latest bugfixed version.  The toolchain's updated so I think there's no disadvantage to ioq3.

BTW, thanks for updating the patch to the latest!
Comment 2 Thilo Schulz 2011-06-11 11:17:32 EDT
thanks alot! applied to r2014
Comment 3 Zack Middleton 2011-06-11 13:49:25 EDT
Created attachment 2774 [details]
Update iqm.h for IQM v2, fixed a IQM warning

Sorry, I missed including the iqm.h changes in the patch. Also fixed a warning I introduced in another IQM patch.
Comment 4 Zack Middleton 2011-06-11 13:53:12 EDT
Changed status to reopened.
Comment 5 Thilo Schulz 2011-06-11 16:20:38 EDT
well, cid