Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0008866opensim[GRID] Inventory Servicepublic2021-02-14 00:392021-03-21 13:41
Assigned To 
PlatformallOperating SystemallOperating System Versionall
Product Version0.9.1.0 
Target VersionFixed in Version 
Summary0008866: missing examples
DescriptionPlease add ossl example scripts. [^]
This is a mirror of the wiki.
Format: Explanation, First executable script, All other scripts as a remark.
TagsNo tags attached.
Git Revision or version number
Run Mode Grid (Multiple Regions per Sim)
Physics EngineBulletSim
Script EngineYEngine
EnvironmentMono / Linux64
Mono Version6.x
Attached Filestxt file icon script.patch.txt [^] (536 bytes) 2021-03-15 08:14 [Show Content]

- Relationships

-  Notes
djphil (reporter)
2021-02-14 00:50
edited on: 2021-02-14 00:52

Your idea is good but your presentation of the scripts is less.
If you base it on the presentation of the scripts I made for you on the opensim wiki it seems clearer / readable / understandable to new users.

Of course this is just my humble opinion.

ref: [^]

Manni (reporter)
2021-02-14 01:45

I don't even understand what they're trying to tell me.
tampa (reporter)
2021-02-14 02:46

Lacking in-line comments describing what each part does, inconsistent indentation, some are overly complex, some use functions in unsafe manners. I only looked at about a dozen, but they need a bit of work still.

Also, if you want to contribute please provide patches against master directly.
djphil (reporter)
2021-02-14 03:03
edited on: 2021-02-14 03:05

What I'm trying to say is that: [^]
Full of comments, 3 scripts in 1, it is not very presentable, clear, understandable.

I don't know if, as tampa says, full of comments on each line would make the script more readable.
Personally all the commented lines give me a headache. I always delete them to have a clear and pleasant script.

It might be better to put a simple link to the referente wiki page. The wiki is more suitable for deepening explanations.
I think you don't learn to script by reading the comments in a script but by reading the documentation.

piusnoel (reporter)
2021-02-14 06:21

In my opinion this is something a grid administrator could add to the script library if wanted, but not something that should be added in general.

I agree that the current library is relatively poor, but most of the scripts contained therein are useful and easy to understand.

The first place I'm going to search for script examples is usually in the wikis, but not in the script libraries provided in the viewer.
tampa (reporter)
2021-02-14 06:51

Think explanations of things can be better done in viewer. Been thinking about perhaps making each function in the script window have a right click open to "open help page" which brings up the relevant wiki page in a browser window in the viewer. Unfortunately not something we can add into OpenSim, but that would probably be best given it is in line with how, at least Windows, does help pages and such. Think everyone has once accidentally clicked on those before.

The script library is not well known to common users either. Think most either look toward the wikis or straight for examples on forums or such stuff. It's so far down the inventory list most never see it.
Luisillo_Contepomi (reporter)
2021-02-14 08:57

Thanks for showing us a repository for the scripting library. (The wiki is not much help. It's very difficult to find items to enhance the library).
djphil (reporter)
2021-02-14 08:59

The OSSL wiki is simple, just go to the right page.
@ [^]
Manni (reporter)
2021-02-15 03:05

So you mean the documented alternative scripts in an extra script, but that would increase the number of scripts extremely.
In addition, they mean exchanging the sparse documentation for a link to the OSSL Wiki, so that you have to constantly switch to the Wiki.

I did not find a patch of this size appropriate because copying it into the source code is much easier.

Of course, this can be done better, but it takes a lot of working hours on the part of the dev team because the source code is complicated and hardly documented.

I have now brought in 600 hours of work and tested this on 7 grids with positive feedback from many people.

Please note.
Beginners: Don't know what OSSL is, so you won't find any scripts or the wiki.
Anyone looking for OSSL in the Wiki will first come to the page on how to insert a new function with C#. I myself didn't know what OSSL was and where to find it for a long time.

Beginners: Can't understand anything from the source code if it is not sufficiently documented.

Many thanks to djphil for the many scripts that were added.
Manni (reporter)
2021-03-15 08:11

cd /opt/opensim
git clone [^]
cp -r /opt/opensim/opensim-ossl-example-scripts/assets /opt/opensim/bin/assets
cp -r /opt/opensim/opensim-ossl-example-scripts/inventory/ScriptsLibrary /opt/opensim/bin/inventory

#Win10 PowerShell
cd C:/opensim
git clone [^]
cp -r /opensim/opensim-ossl-example-scripts/assets /opensim/bin/assets
cp -r /opensim/opensim-ossl-example-scripts/inventory/ScriptsLibrary /opensim/bin/inventory
Ferd Frederix (reporter)
2021-03-21 13:41

Thanks, Manni, this looks extremely useful. I have been doing this too, and will send a patch to you.

- Issue History
Date Modified Username Field Change
2021-02-14 00:39 Manni New Issue
2021-02-14 00:50 djphil Note Added: 0037563
2021-02-14 00:52 djphil Note Edited: 0037563 View Revisions
2021-02-14 01:45 Manni Note Added: 0037564
2021-02-14 02:46 tampa Note Added: 0037565
2021-02-14 03:03 djphil Note Added: 0037566
2021-02-14 03:04 djphil Note Edited: 0037566 View Revisions
2021-02-14 03:05 djphil Note Edited: 0037566 View Revisions
2021-02-14 06:21 piusnoel Note Added: 0037568
2021-02-14 06:51 tampa Note Added: 0037569
2021-02-14 08:57 Luisillo_Contepomi Note Added: 0037570
2021-02-14 08:59 djphil Note Added: 0037571
2021-02-15 03:05 Manni Note Added: 0037572
2021-03-15 08:11 Manni Note Added: 0037668
2021-03-15 08:14 Manni File Added: script.patch.txt
2021-03-21 13:41 Ferd Frederix Note Added: 0037682

Copyright © 2000 - 2012 MantisBT Group
Powered by Mantis Bugtracker