Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | move reset code out of HandleMoveToTargetUpdate() so that we only call it ↵ | Justin Clark-Casey (justincc) | 2011-08-04 | 2 | -17/+13 | |
| | | | | where needed instead of passing in a flag | |||||
* | Make SIGNIFICANT_MOVEMENT SP constant a top-level property so that other ↵ | Justin Clark-Casey (justincc) | 2011-08-04 | 2 | -6/+8 | |
| | | | | classes can use it. | |||||
* | When the NPC reaches within the SIGNIFICANT_CLIENT_MOVEMENT distance of the ↵ | Justin Clark-Casey (justincc) | 2011-08-04 | 3 | -3/+7 | |
| | | | | | | | target, move it directly to the target. This makes the movement exact. Regression test changed to check avatar reaches exact target. Also has the nice side effect of making NPC animations continue to work after the first movement (which wasn't working). However, avatar still pauses in mid-stride | |||||
* | extend npc move test to check a second movement | Justin Clark-Casey (justincc) | 2011-08-04 | 2 | -3/+22 | |
| | ||||||
* | eliminate pre-existing unused SP.StopMovement() | Justin Clark-Casey (justincc) | 2011-08-04 | 1 | -8/+1 | |
| | ||||||
* | Rename HandleMoveToPositionUpdate() to HandleMoveToTargetUpdate() for ↵ | Justin Clark-Casey (justincc) | 2011-08-04 | 2 | -7/+7 | |
| | | | | consistency. Improve method doc. | |||||
* | eliminate a reset position flag by using functionally equivalent ↵ | Justin Clark-Casey (justincc) | 2011-08-04 | 1 | -3/+4 | |
| | | | | DCFlagKeyPressed | |||||
* | Stop calling the SP.ResetMoveToTarget() code if a target is not actually set | Justin Clark-Casey (justincc) | 2011-08-03 | 1 | -6/+16 | |
| | ||||||
* | remove move to duck walk compensation - no longer required. | Justin Clark-Casey (justincc) | 2011-08-03 | 2 | -13/+17 | |
| | | | | extends npc move to regression test to check stop after sufficient sim updates | |||||
* | rename NPC.Autopilot to NPC.MoveToTarget internally. Add method doc to ↵ | Justin Clark-Casey (justincc) | 2011-08-03 | 4 | -4/+34 | |
| | | | | INPCModule | |||||
* | extend move test to check avatar is moving in the right direction after ↵ | Justin Clark-Casey (justincc) | 2011-08-03 | 1 | -0/+13 | |
| | | | | setting a move target | |||||
* | extend move test to check one beat of the simulator without actually asking ↵ | Justin Clark-Casey (justincc) | 2011-08-03 | 2 | -5/+15 | |
| | | | | the npc to move. | |||||
* | Add passing but incomplete NPC move regression test | Justin Clark-Casey (justincc) | 2011-08-03 | 1 | -47/+25 | |
| | ||||||
* | stop avatar service being set up in NPC TestCreate() - it's no longer used | Justin Clark-Casey (justincc) | 2011-08-03 | 1 | -6/+48 | |
| | ||||||
* | refactor: rename the move to position methods to move to target to be ↵ | Justin Clark-Casey (justincc) | 2011-08-03 | 4 | -40/+16 | |
| | | | | consistent with terminology used by scene object part and elsewhere | |||||
* | get rid of vestigal move to parameters | Justin Clark-Casey (justincc) | 2011-08-03 | 10 | -41/+28 | |
| | ||||||
* | enable the NPC module for its regression test | Justin Clark-Casey (justincc) | 2011-08-03 | 1 | -0/+2 | |
| | ||||||
* | correct config comments mistakes | Justin Clark-Casey (justincc) | 2011-08-03 | 2 | -2/+1 | |
| | ||||||
* | Put config to enable disable [NPC] module. | Justin Clark-Casey (justincc) | 2011-08-03 | 3 | -4/+17 | |
| | | | | Default is disabled. You will need to explicitly enable to toy with this. | |||||
* | Do a partial fix/implementation of OSSL osNpcMoveTo() | Justin Clark-Casey (justincc) | 2011-08-03 | 4 | -67/+109 | |
| | | | | | | Avatar moves and stops. However, will stop in mid stride. And if the move to position is in the air, avatar will continue to make vain and quite hilarious attempts to take off (but never doing so). Clearly more work is needed. | |||||
* | Improve z axis move to/autopilot so the avatar does alternative ↵ | Justin Clark-Casey (justincc) | 2011-08-03 | 2 | -12/+30 | |
| | | | | | | crouch/huzzah when walking along the ground Moving a flying avatar to a ground point doesn't yet land the avatar. This may or may not be the best thing | |||||
* | Implement move to/autopilot for z axis movement as well. | Justin Clark-Casey (justincc) | 2011-08-03 | 2 | -17/+40 | |
| | | | | This is jerky (an artifact of the way it's being done, I think), but it's better than on implementation. | |||||
* | get autopilot/go here to work immediately. | Justin Clark-Casey (justincc) | 2011-08-03 | 1 | -3/+7 | |
| | | | | This works with viewer 1.23.5 and so in theory should work with libopenmetaverse. | |||||
* | refactor: Move update_movement_flag and the final check inside the ↵ | Justin Clark-Casey (justincc) | 2011-08-03 | 1 | -5/+5 | |
| | | | | m_allowMovement if in HandleAgentUpdate() since it's logically only ever used there | |||||
* | refactor: move out code from HandleAgentUpdate() which processes updates to ↵ | Justin Clark-Casey (justincc) | 2011-08-03 | 1 | -98/+116 | |
| | | | | | | move to a set position Also comment out the really spammy log message I accidentally left in on the last commit. | |||||
* | Fix Flotsam cache so it will use the disk cache if the memory cache is enabled | Robert Adams | 2011-08-02 | 1 | -2/+2 | |
| | ||||||
* | Partially fix autopilot/go here | Justin Clark-Casey (justincc) | 2011-08-02 | 7 | -119/+99 | |
| | | | | | | | This now works again except that it requires a click or avatar mvmt to get going This is because the ScenePresence.HandleAgentUpdate() method doesn't trigger until the client does something significant, at which point autopilot takes over. Even clicking is enough to trigger. This will be improved presently. | |||||
* | If GetRegionByName can't match something in the local db, then search the ↵ | Justin Clark-Casey (justincc) | 2011-08-02 | 1 | -6/+27 | |
| | | | | | | hypergrid if that functionality has been enabled. This should fix the problem today where old style HG addresses (e.g. "hg.osgrid.org:80:Vue-6400") stopped working since 8c3eb324c4b666e7abadef4a714d1bd8d5f71ac2 | |||||
* | remove ancient late 2008 cruft that handles the situation where the ↵ | Justin Clark-Casey (justincc) | 2011-08-02 | 1 | -10/+1 | |
| | | | | | | GetRegionsByName used to not be implemented/returned null. It's impossible that anybody is still running this since the infrastructure has changed massively since that time. | |||||
* | Change GridService.GetRegionByName() to only return info if there is an ↵ | Justin Clark-Casey (justincc) | 2011-08-02 | 3 | -3/+12 | |
| | | | | | | | exact region name match, unlike GetRegionsByName() This should fix the first part of http://opensimulator.org/mantis/view.php?id=5606, and maybe 5605. Thanks to Melanie for helping with this. | |||||
* | Revert "In GridService, have GetRegionByName() call GetRegionsByName() with ↵ | Justin Clark-Casey (justincc) | 2011-08-02 | 1 | -5/+4 | |
| | | | | | | | | a max return of 1 instead of duplicating code." This reverts commit 8d33a2eaa10ed75146f45cca4d6c19ac814d5fee. Better fix will be along in a minute | |||||
* | In GridService, have GetRegionByName() call GetRegionsByName() with a max ↵ | Justin Clark-Casey (justincc) | 2011-08-02 | 1 | -4/+5 | |
| | | | | | | return of 1 instead of duplicating code. This also fixes the problem where this method would not return a hypergrid region, unlike GetRegionsByName() | |||||
* | Get rid of AvatarAppearance.Owner to simplify the code. | Justin Clark-Casey (justincc) | 2011-08-02 | 15 | -53/+38 | |
| | | | | This is not used for anything - appearances are always properties of objects with ids (ScenePresence, AgentCircuitData) and just has the potential to get out of sync when the appearance is cloned. | |||||
* | Get osNpcCreate appearance working with avatars that are currently in the scene. | Justin Clark-Casey (justincc) | 2011-08-01 | 7 | -13/+93 | |
| | | | | | | Had to stop using AvatarService for now since it doesn't store baked texture IDs (which is why this was failing). Also failing because cloning appearance was also cloning the AvatarApperance.Owner field, which we weren't then changing. Extended TestCreate() to check this. | |||||
* | Move common gemo/agent map name code into CreateGeom() | Justin Clark-Casey (justincc) | 2011-08-01 | 1 | -11/+5 | |
| | | | | Fix build break. | |||||
* | Instead of preserving old name in geom_name_map in change size or shape, use ↵ | Justin Clark-Casey (justincc) | 2011-08-01 | 1 | -7/+3 | |
| | | | | | | the Name property instead. This is equivalent since the prim 'name' is never changed. In fact, this propery is never used for prims | |||||
* | comment out unused code in OdeScene.TriCallback() | Justin Clark-Casey (justincc) | 2011-08-01 | 1 | -12/+12 | |
| | ||||||
* | On geom removal, remove the name from the OdeScene.geom_name_map too | Justin Clark-Casey (justincc) | 2011-08-01 | 1 | -0/+1 | |
| | ||||||
* | remove the unused ODEPrim.prev_geom field | Justin Clark-Casey (justincc) | 2011-08-01 | 1 | -3/+0 | |
| | ||||||
* | When an ODE geom is removed (as when a non-phantom prim is deleted, resized ↵ | Justin Clark-Casey (justincc) | 2011-08-01 | 1 | -0/+2 | |
| | | | | | | or shape changed, also remove the OdeScene.actor_name_map entry pointing to the phys actor This is to stop a small memory leak over time when prims are deleted or phantom-toggled | |||||
* | refactor: centralize prim geom removal code from four places to one | Justin Clark-Casey (justincc) | 2011-08-01 | 3 | -43/+48 | |
| | ||||||
* | minor: add note to RemovePrimThreadLocked() to the effect that it contrary ↵ | Justin Clark-Casey (justincc) | 2011-08-01 | 2 | -1/+4 | |
| | | | | to the summary, it is being called from within Simulate() lock (OdeLock) | |||||
* | If a prim changes size or shape, add actor to _parent_scene.actor_name_map ↵ | Justin Clark-Casey (justincc) | 2011-08-01 | 1 | -1/+4 | |
| | | | | | | | | with new prim_geom key, as the old one becomes invalid. This resolves http://opensimulator.org/mantis/view.php?id=5603 where changing size or shape would stop collision_start being fired in a running script. In both this and existing code we are not removing old actors from actor_name_map when the existing prim_geom is removed, which leads to a small memory leak over time. This needs to be fixed. | |||||
* | minor: indentation correction | Justin Clark-Casey (justincc) | 2011-08-01 | 1 | -17/+13 | |
| | ||||||
* | Comment out SOP logging message I accidentally left in | Justin Clark-Casey (justincc) | 2011-07-31 | 1 | -1/+1 | |
| | ||||||
* | refactor: split out ninja joint part of SOP.DoPhysicsPropertyUpdate() so ↵ | Justin Clark-Casey (justincc) | 2011-07-31 | 1 | -71/+86 | |
| | | | | that we don't have to look at it if it's not relevant | |||||
* | In the packetpool, if we encounter a data block that somehow wasn't ↵ | Justin Clark-Casey (justincc) | 2011-07-31 | 1 | -6/+11 | |
| | | | | retrieved via GetDataBlock() then create a new stack instead of throwing an exception | |||||
* | Remove _mesh field since the mesh data no longer needs to be stored after ↵ | Justin Clark-Casey (justincc) | 2011-07-31 | 1 | -49/+16 | |
| | | | | | | it's initially used. This may improve memory usage for regions using mesh and sculpts, though I suspect that it doesn't address the current memory leak. | |||||
* | refactor: Remove argument to pass in an initial mesh to OdePrim since this ↵ | Justin Clark-Casey (justincc) | 2011-07-31 | 2 | -27/+5 | |
| | | | | | | is no longer required and it prevents removal of the _mesh field (which is only used temporarily) If passing in a mesh becomes important again in the future then this can be reinstated. | |||||
* | minor: correct method doc for last commit | Justin Clark-Casey (justincc) | 2011-07-31 | 1 | -1/+1 | |
| |