Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0006864opensim[REGION] OpenSim Corepublic2013-11-23 23:552013-12-18 18:32
Assigned To 
Statuspatch includedResolutionopen 
PlatformOSOS Version
Product Versionmaster (dev code) 
Target VersionFixed in Version 
Summary0006864: Avatar Physics not shared between viewers
DescriptionTo match the current viewer spec it would be nice if Avatar Physics switch would be enabled. My understanding from Cinder Biscuits of Firestorm is it is a switch that needs to be enabled in the region software code. The effect is viewerside but without passing the information via the region it is not viewable by others, only the person who is running the avi physics. This is NOT part of Bullet or ODE Physics or the physics engine.
Steps To ReproduceDownload any free Avi Physics and wear them in world in any compatible viewer (which I believe is all of them now). You will see the effect locally, but nobody else will.
TagsNo tags attached.
Git Revision or version number
Run Mode Grid (Multiple Regions per Sim)
Physics EngineODE
Script Engine
Environment.NET / Windows64
Mono VersionNone
ViewerFirestorm, Kokua, Sing.
Attached Filesdiff file icon avatar_physics_params.diff [^] (3,785 bytes) 2013-12-05 18:26 [Show Content]

- Relationships

-  Notes
justincc (administrator)
2013-12-05 16:07

I'm not aware of any such switch. Can you describe it in more detail?
wbalazic (reporter)
2013-12-05 16:34

This is the one I was telling you about with Cinder at the community conference. I'm going to send this to Cinder and Tonya so they can provide details on this for you.
cinderblocks (reporter)
2013-12-05 18:31

Uploaded an ugly patch, I know. Quick and dirty, but here's a patch for all the extra avatar_lad params that avatar physics requires.

Avatar physics doesn't really use a switch per se, it adds another optional wearable type WT_PHYSICS which the avatar can wear that contains these parameters. They are adjusted the same way any other shape or clothing type is. I have no idea how this translate to simulator code, but I'm pretty sure it just requires another wearable type with the parameters I added (poorly and with naughty tab indentation) in the patch.

Hope this helps, let me know if you need further explanation on the viewer side of things.
justincc (administrator)
2013-12-18 17:44

Thanks cinderblocks. I applied the patch to the branch justincc-master in git. This is a temporary branch for master minus the in-process merge which is currently the cause of various breaking problems in master.

However, I failed to realise that in master, AvatarAppearance actually already contains parameters for the breast physics from the merge. However, some of the parameter names are different. This won't affect the viewer, but if you get a chance could you compare the master and justincc-master versions of AvatarAppearance and tell me which param names are more accurate (since some of them differ). Thanks.
wbalazic (reporter)
2013-12-18 18:32

Aren't there more parameters than just breast physics in this? The viewer avi physics section shows other areas that are configurable besides breasts, or does just that one cover all the areas?

- Issue History
Date Modified Username Field Change
2013-11-23 23:55 wbalazic New Issue
2013-12-05 16:07 justincc Note Added: 0024818
2013-12-05 16:34 wbalazic Note Added: 0024822
2013-12-05 18:26 cinderblocks File Added: avatar_physics_params.diff
2013-12-05 18:31 cinderblocks Note Added: 0024835
2013-12-18 17:44 justincc Note Added: 0024889
2013-12-18 17:44 justincc Status new => patch included
2013-12-18 18:32 wbalazic Note Added: 0024890

Copyright © 2000 - 2012 MantisBT Group
Powered by Mantis Bugtracker