rushabhshah10 / opennero

Automatically exported from code.google.com/p/opennero
Other
0 stars 0 forks source link

Review rtNEAT code changes on /branches/merge #14

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Purpose of code changes on this branch:

This code has three sets of changes:

 - merge from the creativeit branch (in particular for object manipulation)
 - rtNEAT changes from the rtNEAT package comparison work
 - collision detection and collision resolution code

When reviewing my code changes, please focus on:

 - rtNEAT fix from the rtNEAT package comparison (Revision #412)
 - test that the new rtNEAT code compiles and runs without errors and performs no worse on the new NERO environment

After the review, I'll merge this branch into:
/trunk

Original issue reported on code.google.com by ikarpov on 13 Jul 2010 at 3:36

GoogleCodeExporter commented 9 years ago
Several header files have "^@"'s in them that make them uncompilable in Linux. 

Original comment by AdamDz...@gmail.com on 13 Jul 2010 at 6:45

GoogleCodeExporter commented 9 years ago
thanks, AdamDziuk, I'll fix in a moment.

Original comment by ikarpov on 13 Jul 2010 at 7:03

GoogleCodeExporter commented 9 years ago
Actually just checked out the branch and it compiles fine on xbox. Which lines 
are giving you trouble?

Original comment by ikarpov on 13 Jul 2010 at 7:14

GoogleCodeExporter commented 9 years ago
Looks like two compile issues that only turn up when compiling in non-Release 
mode. Should be fixed in Revision 421.

Original comment by ikarpov on 13 Jul 2010 at 7:57

GoogleCodeExporter commented 9 years ago
Works better than before on flag running tasks. Testing Explore/Exploit next.

Original comment by AdamDz...@gmail.com on 13 Jul 2010 at 9:01

GoogleCodeExporter commented 9 years ago
Explore Exploit still doesn't work correctly, will demonstrate during meeting, 
but it doesn't work worse. rtneat changes look like they're good to merge in.

Original comment by AdamDz...@gmail.com on 13 Jul 2010 at 9:08