Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0008657opensim[REGION] Scripting Enginepublic2020-02-08 06:452020-02-11 10:35
Reportertampa 
Assigned To 
PrioritylowSeveritytweakReproducibilityalways
Statuspatch feedbackResolutionopen 
PlatformOSOS Version
Product Version0.9.1.0 
Target VersionFixed in Version 
Summary0008657: New default script
DescriptionGiven that we have a modular system in OpenSim for the use of physics, scripts and so on I thought it may be a good idea to supply that information to users in a more "in your face" way.

I edited the default script to supply a ton more information to the user such as the script engine, physics engine and simulator version. The complete script also gives information about the region performance, time and even provides a handy link back to the project page.

As the script contains ossl functions, I wanted to get some feedback on that. The functions used are enabled by default and open to use for anyone so the likelihood of them returning an error is slim to none.

Patch is attached and built against current master code, confirmed working as intended.
Additional Informationdefault
{
    state_entry()
    {
        llSay(0, "\nScript running on "
        + osGetScriptEngineName()
        + " with "
        + osGetPhysicsEngineName()
        + " "
        + osGetPhysicsEngineType()
        + " Physics Engine \nRunning Simulator version: "
        + osGetSimulatorVersion()
        + "\nusing: "
        + (string)osGetSimulatorMemoryKB()
        + " Kilobytes of memory"
        + "\nRegion FPS: " + (string)llGetRegionFPS()
        + " | Time Dilation: " + (string)llGetRegionTimeDilation()
        + " | Physics FPS: "
        + (string)llList2Float(osGetRegionStats(), STATS_PHYSICS_FPS)
        + "\nScript LPS: "
        + (string)llList2Float(osGetRegionStats(), STATS_SCRIPT_LPS)
        + " | Active Scripts: "
        + (string)llList2Float(osGetRegionStats(), STATS_ACTIVE_SCRIPTS)
        + " | Primitives: "
        + (string)llList2Float(osGetRegionStats(), STATS_TOTAL_PRIMS)
        + "\n You are: " + llKey2Name(llGetOwner())
        + " on Region: " + llGetRegionName()
        + " Time is: " + llGetTimestamp() + " Epoch: " + llGetUnixTime()
        + "\nThank you for using OpenSimulator! http://opensimulator.org/" [^]);
    }
}
TagsNo tags attached.
Git Revision or version number
Run ModeStandalone (1 Region) , Standalone (Multiple Regions) , Grid (1 Region per Sim) , Grid (Multiple Regions per Sim)
Physics EngineBulletSim, ubODE
Script EngineXEngine, YEngine
EnvironmentMono / Linux64
Mono Version6.x
Viewer
Attached Filespatch file icon Add-to-script-library.patch [^] (1,892 bytes) 2020-02-08 07:29 [Show Content]

- Relationships

-  Notes
(0036181)
UbitUmarov (administrator)
2020-02-08 06:51

Sorry No
that is a lot of useless information for a default script.
(0036182)
tampa (reporter)
2020-02-08 06:55

In the same tune just "script running" tells the user nothing, I suppose the performance information is a bit much, but telling the user which script and physics are in use makes sense to me given the caveats each have
(0036183)
aiaustin (developer)
2020-02-08 07:24
edited on: 2020-02-08 07:25

Keep the new script text as it is now, short and sweet. Easily selected to delete when you replace it with the intended content.

Your script might be better suited to being in the Library as a sample "Get System and Region Info".

When I compiled it on XEngine (on OSGrid 18-Dec-2019 code) by the way it grumbled about not being able to convert floats to strings. But I did not look at the cause.

(0036184)
tampa (reporter)
2020-02-08 07:26

I noticed that, strange on older versions it works only recent versions running X it complains, just two typecasts though for regionfps and time dilation
(0036185)
UbitUmarov (administrator)
2020-02-08 07:29

as i said on irc
a better idea is to add new usefull things on opensim library scripts
(0036204)
tampa (reporter)
2020-02-10 14:46

I added a patch for that instead
(0036205)
piusnoel (reporter)
2020-02-11 09:10

I like the idea of aiaustin: Your script might be better suited to being in the Library as a sample "Get System and Region Info".
(0036206)
UbitUmarov (administrator)
2020-02-11 10:35

the patch is only part of the work, and it is not correct

- Issue History
Date Modified Username Field Change
2020-02-08 06:45 tampa New Issue
2020-02-08 06:45 tampa File Added: New-default-script.patch
2020-02-08 06:45 tampa Status new => patch feedback
2020-02-08 06:51 UbitUmarov Note Added: 0036181
2020-02-08 06:55 tampa Note Added: 0036182
2020-02-08 07:24 aiaustin Note Added: 0036183
2020-02-08 07:25 aiaustin Note Edited: 0036183 View Revisions
2020-02-08 07:26 tampa Note Added: 0036184
2020-02-08 07:27 tampa Additional Information Updated View Revisions
2020-02-08 07:27 tampa File Deleted: New-default-script.patch
2020-02-08 07:29 UbitUmarov Note Added: 0036185
2020-02-08 07:29 tampa File Added: Add-to-script-library.patch
2020-02-10 14:46 tampa Note Added: 0036204
2020-02-11 09:10 piusnoel Note Added: 0036205
2020-02-11 10:35 UbitUmarov Note Added: 0036206


Copyright © 2000 - 2012 MantisBT Group
Powered by Mantis Bugtracker