aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/OpenSim/Region/Framework/Scenes (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Ignore whitespace when reading serialized XML objects.Justin Clark-Casey (justincc)2014-08-294-188/+192
| | | | | | This was previously effectively being done by XmlDocument in the multiple passes through the XML. This change tells XmlReader to ignore whitespace. This also means changing arguments to use XmlReader instead of XmlTextReader (a descendent of XmlReader) directly. XmlReader.Create() has been the recommend way to create XML readers since .NET 2.0 as per MS SDK and is the only way to specific ignore whitespace settings.
* minor: Eliminate more unnecessary code copying individual parameters for ↵Justin Clark-Casey (justincc)2014-08-281-17/+19
| | | | Vector3 copying in Scene - this is not necessary as Vector3 is a value type
* On code section that rezzes single objects and attachments, reduce CPU use ↵Justin Clark-Casey (justincc)2014-08-283-69/+123
| | | | | | | | by reading asset XML a single time with a stream reader rather than multiple times. Reading large XML documents (e.g. complex attachments) is CPU expensive - this must be done as few times as possible (preferably just once). Reading these documents into XmlDocument is also more resource intensive than using XmlTextReader, as per Microsoft's own publication "Improve .NET Application Performance and Scalability" Optimization of other cases will follow if this change is successful.
* Don't allow update timer to invoke another scene update if the previous is ↵Justin Clark-Casey (justincc)2014-08-261-1/+10
| | | | still active.
* Fix frame times when updating scene on timer.Justin Clark-Casey (justincc)2014-08-261-15/+17
|
* Implement experimental non-default mechanism to update scene via a timer ↵Justin Clark-Casey (justincc)2014-08-262-35/+62
| | | | | | | | rather than a persistent thread with sleep. This is to see if an inaccuracy in sleep times under load is responsible for increase in frame times even when there is spare time still available. Can currently only be activated by setting "debug scene set update-on-timer true". Can be switched between timer and thread with sleep updates whilst the scene is running.
* If a user moves back in sight of a child region before the agent has been ↵Justin Clark-Casey (justincc)2014-08-151-6/+3
| | | | closed on teleport, don't unnecessarily resend all avatar and object data about that region.
* On teleport to a region that already has a child agent established (e.g. a ↵Justin Clark-Casey (justincc)2014-08-151-32/+43
| | | | | | neighbour) don't resend all the initial avatar and object data again. This is unnecessary since it has been received (and data continues to be received) in the existing child connection.
* Make RootTerseUpdatePeriod and ChildTerseUpdatePeriod configurable in ↵Justin Clark-Casey (justincc)2014-08-141-0/+3
| | | | | | | | | | [InterestManagement] in OpenSim.ini for experimental purposes. If n > 1 for RootTerseUpdatePeriod only every n terse update is actually sent to observers on same region, unless velocity is effectively zero (to stop av drift). If n > 1 for ChildTerseUpdatePeriod only every n terse update is sent to observers in other regions, unless velocity is effectively zero. Defaults are same as before (all packets are sent). Tradeoff is reduction of UDP traffic vs fidelity of observed av mvmt. Increasing n > 1 leads to jerky observed mvmt immediateley for root, though not on child, where experimentally have gone to n = 4 before jerkiness is noticeable.
* Make some existing reprioritization values changeable outside the scene for ↵Justin Clark-Casey (justincc)2014-08-141-15/+18
| | | | | | test purposes, and use more consise property syntax. No functional change.
* On entity transfer of scene presence, replace polling sleep in ↵Justin Clark-Casey (justincc)2014-08-131-14/+13
| | | | | | | SP.WaitForUpdateAgent() with a triggered event instead. Rapid polls are more expensive than triggered events (several polls vs one trigger) and may be problematic on heavily loaded simulators where many threads are vying for processor time. A triggered event is also slightly quicker as there is no maximum 200ms wait between polls.
* Remove redundant origin region lock in SP.CompleteMovement()Justin Clark-Casey (justincc)2014-08-131-6/+1
| | | | This is already going to be correctly set by WaitForUpdateAgent() earlier on in that method, which is always called where a callback to the originating region is required.
* Change RootRotationUpdateTolerance from 0.01 to 0.1 in code as well.Justin Clark-Casey (justincc)2014-08-081-1/+1
|
* Fixed crash when using Allowed/Denied Viewers, and the viewer's name is ↵Oren Hurvitz2014-08-061-2/+2
| | | | | | shorter than one of the test strings This fixes http://opensimulator.org/mantis/view.php?id=7294
* Add RootPositionUpdateTolerance, RootRotationUpdateTolerance, and ↵Justin Clark-Casey (justincc)2014-08-061-0/+7
| | | | | | | RootVelocityUpdateTolerance parameters to [InterestManagement] in OpenSimDefaults.ini These govern when AgentUpdates are sent to observers on position, rotation and velocity changes to an avatar (including the avatar themselves). Higher values reduce AgentUpdate traffic but at a certain level will degrade smoothness of avatar and perceived avatar movement.
* refactor: Rename recent new Client*UpdateTolerance to Root*UpdateTolerance ↵Justin Clark-Casey (justincc)2014-08-062-13/+13
| | | | for better accuracy and consistency with other similar parameters
* Implement "scene debug set root-upd-per" for dropping 1 in N root agent ↵Justin Clark-Casey (justincc)2014-07-292-2/+24
| | | | | | | updates except to originator For experimental purposes. Also corrects a previous bug where each terse update sent was counted rather than each set of terse updates to agents.
* Add debug mechanism for only sending 1 in N AgentUpdate packets to child agents.Justin Clark-Casey (justincc)2014-07-292-1/+18
| | | | | | Allows experiments in manually reducing updates under heavy load. Activated by "debug scene set client-upd-per" console command. In a simple test, can send as few as every 4th update before observed movement starts becoming disturbingly rubber-banded.
* Make it possible to change avatar position update, rotation and velocity ↵Justin Clark-Casey (justincc)2014-07-292-8/+22
| | | | | | | tolerances on the fly. This is done via "debug scene set client-pos-upd, client-rot-upd, client-vel-upd". For testing purposes.
* Allow the "debug scene set physics false|true" command to work when ↵Justin Clark-Casey (justincc)2014-07-291-20/+43
| | | | | | bulletsim physics is running in a separate thread. This will also allow the "disable physics" setting in the region debug viewer dialog to work in this circumstance.
* Add a "debug scene set child-repri <double>" command that allows child ↵Justin Clark-Casey (justincc)2014-07-292-3/+9
| | | | | | | | reprioritization distance to be changed on the fly. This governs when child agent position changes are sent to neighbouring regions. Corresponding config parameter is ChildReprioritizationDistance in [InterestManagement] in OpenSim.ini For test purposes.
* Fix recent regression in 3c6becd5 where login or hg login to variable sized ↵Justin Clark-Casey (justincc)2014-07-221-1/+0
| | | | | | | | | regions failed with outdated simulator message. I forgot that a null 'their version' would not be passed over the wire and ends up as an empty string instead (like older simulators). So instead pass through the correct simulator protcol version instead (SIMULATOR/0.3) when querying from login or hg login. Also removes a debug console write for agent limit accidentally left in for the same commit. Relates to mantis 7276
* On login and first HG entrance to a foreign grid, perform query access ↵Justin Clark-Casey (justincc)2014-07-211-0/+1
| | | | checks before proceeding.
* Fixed: after a Hypergrid teleport, attachments often either disappear, or ↵Oren Hurvitz2014-07-211-10/+13
| | | | | | | | | | | appear both on the avatar AND as in-world objects. Another manifestation of this bug is that after a Hypergrid teleport, when you click on one of the avatar's attachments the object doesn't show its name. This means that the viewer knows the attachment is there, but the simulator does not. The problem was caused by treating Hypergrid teleports as if they're Logins (because the teleport flag ViaLogin is enabled). This may fix: http://opensimulator.org/mantis/view.php?id=7238 This may fix: http://opensimulator.org/mantis/view.php?id=7220
* Fixed problems if an avatar tries to cross regions when the previous cross ↵Oren Hurvitz2014-07-212-136/+134
| | | | | | hasn't completed yet This caused the client to stop responding, and even the simulators to have problems. The solution is to disallow crossing before the previous cross has completed.
* Minor: changed "existant" to "existent"Oren Hurvitz2014-07-211-1/+1
|
* Close streams immediately when we finish using themOren Hurvitz2014-07-213-72/+80
|
* Call RemoveScriptInstance when removing from inventoryDev Random2014-07-141-2/+2
|
* On logout, delay the removal of AgentCircuitData until the very end, because ↵Diva Canto2014-07-041-5/+3
| | | | that data structure contains important information about the agent that may be needed by modules.
* This fixes the bug related to rebooting neighboring varregions while avatars ↵Diva Canto2014-07-041-10/+14
| | | | are logged in; the avies would not see the region anymore until they relogged. Same problem as before: inconsistent calculation of scope.
* It turns out that child agent management has had a bug for a while: there ↵Diva Canto2014-07-041-1/+3
| | | | was an inconsistency in the scope between opening and closing child agents in neighboring regions. For opening (in EnableChildAgents), the region's DrawDistance was being used; for closing (in IsOUtsideView) , the viewer's (SP) DrawDistance was being used. This fixes this inconsistency, therefore eliminating bugs observed in TPs between, at least, neighboring varregions.
* Temporarily disable new regression test TestLifecycle() until I have a ↵Justin Clark-Casey (justincc)2014-07-031-1/+1
| | | | | | chance to fix it This was working for me locally because it was still picking up all my *.ini config files, which jenkins won't have in its test env
* Re-enabled TestLifecycle regression test logging so I can get some idea of ↵Justin Clark-Casey (justincc)2014-07-031-1/+1
| | | | why it's failing on jenkins but not locally
* Actually call Close() for shared region modules when the simulator is being ↵Justin Clark-Casey (justincc)2014-07-022-0/+250
| | | | | | shutdown. Adds regression test for this case.
* minor: Change default max phys prim size in code to match OpenSimDefaults.iniJustin Clark-Casey (justincc)2014-06-281-1/+1
|
* minor: change allow script crossings default in code to true in order to ↵Justin Clark-Casey (justincc)2014-06-281-1/+1
| | | | match OpenSimDefaults.ini
* Revert "Let HG users be gods too, if they have local permission to be so."Diva Canto2014-06-091-4/+7
| | | | | | | HG gods are not safe at this point. It's better to disallow this until they can be made safe. This reverts commit e86c765be3b0d94c94ff1c5f15a3949ecc857627.
* Let HG users be gods too, if they have local permission to be so.Diva Canto2014-06-091-7/+4
|
* Send multiple terrain patches per terrain update packet if terrainRobert Adams2014-06-011-2/+5
| | | | | draw distance optimization is enabled. Makes terrain editting a lot snappier.
* Modifications to debugging printouts. No functional changes.Robert Adams2014-05-311-9/+5
|
* varregion: More tweeking to only sending patches within avatar draw distance.Robert Adams2014-05-311-9/+10
| | | | Still has problems with child avatars.
* varregion: send terrain patches from where the avatar outward if the parameterRobert Adams2014-05-311-1/+6
| | | | | | [Terrain]SendTerrainUpdatesByViewDistance=true. This tracks which patches have been sent to each client and outputs the patches as the avatar moves.
* minor: rename velocidyDiff -> velocityDiffJustin Clark-Casey (justincc)2014-05-301-2/+2
|
* minor: Add some commented out logging to ↵Justin Clark-Casey (justincc)2014-05-301-0/+8
| | | | ScenePresence.SendTerseUpdateToAllClients() which is extremely helpful when investigating presence update triggers.
* When saving an OAR in "Publish" mode, also discard Group informationOren Hurvitz2014-05-261-3/+10
|
* Prevent login to a region if the Telehub or Landing Point are in a banned parcelOren Hurvitz2014-05-251-4/+33
|
* If one is sitting on a child with an unset camera-eye and so using one set ↵Justin Clark-Casey (justincc)2014-05-231-3/+7
| | | | | | in a root prim, the focus should remain on the root prim. Matches behaviour just tested on the Linden grid.
* Add any camera at compensation for sat upon child prims to any existing ↵Justin Clark-Casey (justincc)2014-05-231-1/+1
| | | | camera-at value, rather than replace.
* Compensate camera-at and camera-eye for child prim rotation when sitting on ↵Justin Clark-Casey (justincc)2014-05-231-0/+12
| | | | child prim with camera-eye set
* If the root prim has a camera-at or camera-eye setting and a sat upon child ↵Justin Clark-Casey (justincc)2014-05-221-0/+7
| | | | | | prim does not, use the root prim offsets. This matches behaviour just tested on the Linden Lab grid.