omni360 / kuda

Automatically exported from code.google.com/p/kuda
GNU General Public License v2.0
0 stars 0 forks source link

particleSystem.showBoxes function missing from Kuda 1.3.2 #27

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

particleSystem.showBoxes();
particleSystem.hideBoxes();

missing from Kuda 1.3.2 build.  I tried using an alternative:

hemi.curve.showBoxes(particleSystem.boxes);
hemi.curve.hideBoxes(particleSystem.boxes);

but this doesn' work the same way... at least in the case that I tested see 
attached screenshot.  The bases show up in a different location from the 
particle system.

Original issue reported on code.google.com by raj...@gmail.com on 18 May 2011 at 5:03

Attachments:

GoogleCodeExporter commented 9 years ago
Ok, I wrote a unit test that shows the problem. 

http://labs.rajdye.com/superb/kuda/unit_tests/unit4.html

So I think the solution would be to restore the showBoxes() and hideBoxes() 
functions and implement them in the way that they were for Kuda 1.3.2

Original comment by raj...@gmail.com on 18 May 2011 at 5:17

GoogleCodeExporter commented 9 years ago
Added an optional second parameter to hemi.curve.showBoxes() to pass the parent 
transform of the particle system (if it's not the default client root 
transform). Updated unit test 4 for this change.

Also changed the constructor for hemi.curve.ParticleSystem so that the optional 
parent transform is now a part of the configuration object rather than a 
separate parameter. See hemi.curve.createSystem() for documentation on the 
config parameters. Updated unit tests and samples for this change.

I'm intentionally leaving showBoxes as a utility function of hemi.curve, since 
it's more of a debug utility than explicit functionality of a particle system.

Original comment by erik.kit...@gmail.com on 19 May 2011 at 4:53

GoogleCodeExporter commented 9 years ago
Thanks I se now that it works. Thanks for updating the unit tests.

Original comment by raj...@gmail.com on 20 May 2011 at 1:09