> On Jan. 31, 2011, 5:48 p.m., Merov Linden wrote: > > It seems to me that the new code can completely replace the old one, i.e. > > the search based on the map is not adding anything different or is it?
There are several reasons that replacing the code may provide a different result. The list of avatars returned by the function is based on NearMeRange which could be a greater value than RenderFarClip. In that case, replacing the code would only return the list of avatars in RenderFarClip range and would ignore a greater NearMeRange. In the case of a person owning an island, they may have NearMeList set to 1200 so they could see the presence of any avatar in their region at any altitude and yet have RenderFarClip set to 128. I realize that at this point its not really "NearMe". Also I've noticed in the sandboxes that quite often a person shows up in the region avatar list before showing up in the viewer object list and in my personal implementation of a radar floater I mark the avatar names that are not visible. In the released versions of the Nearby list, the Zoom-in option is not available for avatars that are not visible so designers must have considered this situation. I guess my other reason would be changing it in llworld wouldn't be the right place to have the function getAvatars if it ignored map data and only used character data. Maybe creating a new getAvatars in LLCharacters would be the right way to do it then. > On Jan. 31, 2011, 5:48 p.m., Merov Linden wrote: > > indra/newview/llworld.cpp, lines 1475-1477 > > <http://codereview.secondlife.com/r/132/diff/1/?file=751#file751line1475> > > > > I fail to see how the "1000m" condition is tested in this code. Same > > for the "only do this when we are retrieving uuid's". It can only provide the function if it has been storing uuid's (avatar_ids not null) or it would provide duplicates. The reason its for over 1000m is the viewer object list position is ignored if it had already added the uuid from the region data. I.e, the region avatar position is incorrect if the user is over 1024m and will not be added to the list as its not in NearMeRange. - Twisted ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/132/#review297 ----------------------------------------------------------- On Jan. 31, 2011, 3:34 p.m., Twisted Laws wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/132/ > ----------------------------------------------------------- > > (Updated Jan. 31, 2011, 3:34 p.m.) > > > Review request for Viewer. > > > Summary > ------- > > This modifies the getAvatars function in llworld to also include any avatars > that are found within the range from the LLCharactes list as well (the list > of avatars that is in the viewer object list). This should make sure that > anyone that you visually see within range shows up in the list. Note that > changing it in this function also affects > LLFloaterAvatarPicker::populateNearMe, LLLocalSpeakerMgr::updateSpeakerList, > as well as the LLPanelPeople::updateNearbyList that was originally mentioned > in the Jira. The region avatars lists only contain valid position data when > the avatars are below 1024m. The comment that mentions about retrieving > uuids is based on the function, not the current uses. No current calls in > the code are done with the avatar_ids argument NULL. Duplicates in the > returned list need to be suppressed. > > > This addresses bug VWR-17050. > http://jira.secondlife.com/browse/VWR-17050 > > > Diffs > ----- > > indra/newview/llworld.cpp 691e3941d950 > > Diff: http://codereview.secondlife.com/r/132/diff > > > Testing > ------- > > Simple testing in sandboxes of this patch at 20m and 2000m heights with and > without avatars nearby. Tested with varying the NearMeRange to insure it > does not show avatars beyond the range. Testers need to understand that > RenderFarClip has an impact on the avatars that are actually in the viewer > object list, so setting NearMeRange to a great distance at high altitude > won't necessarily add avatars to the list. Basically if you can see the > avatar and its within NearMeRange, the avatar should be in the nearby avatar > list in the people panel. > > > Thanks, > > Twisted > >
_______________________________________________ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
