Yeah I don't care what base class the actual link/etc code goes in. Just meant there should be some seperation of the levels. So I was actually thinking of SceneObjectGroup as part of the InnerScene level and not at this point really concerned which class has which functions. <br><br>And with SceneObjectPart I believe the general opinion of just about everyone is that we should get rid of it and just have SceneObjects. <span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);"></span><br><br><b><i>"Frisby, Adam" <adam@deepthink.com.au></i></b> wrote:<blockquote class="replbq" style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px;">    <meta http-equiv="Content-Type" content="text/html; charset=us-ascii"> <meta name="Generator" content="Microsoft Word 12 (filtered medium)"> <style> <!--  /* Font Definitions */  @font-face  {font-family:"Cambria Math";  panose-1:2 4 5 3 5 4 6 3 2 4;}
 @font-face  {font-family:Calibri;  panose-1:2 15 5 2 2 2 4 3 2 4;} @font-face  {font-family:Tahoma;  panose-1:2 11 6 4 3 5 4 4 2 4;}  /* Style Definitions */  p.MsoNormal, li.MsoNormal, div.MsoNormal  {margin:0cm;  margin-bottom:.0001pt;  font-size:12.0pt;  font-family:"Times New Roman","serif";} a:link, span.MsoHyperlink  {mso-style-priority:99;  color:blue;  text-decoration:underline;} a:visited, span.MsoHyperlinkFollowed  {mso-style-priority:99;  color:purple;  text-decoration:underline;} p  {mso-style-priority:99;  mso-margin-top-alt:auto;  margin-right:0cm;  mso-margin-bottom-alt:auto;  margin-left:0cm;  font-size:12.0pt;  font-family:"Times New Roman","serif";} span.EmailStyle18  {mso-style-type:personal-reply;  font-family:"Calibri","sans-serif";  color:#1F497D;} .MsoChpDefault  {mso-style-type:export-only;} @page Section1  {size:612.0pt 792.0pt;  margin:72.0pt 72.0pt 72.0pt 72.0pt;} div.Section1  {page:Section1;} --> </style> <!--[if gte mso 9]><xml> 
 <o:shapedefaults v:ext="edit" spidmax="1026" /> </xml><![endif]--><!--[if gte mso 9]><xml>  <o:shapelayout v:ext="edit">   <o:idmap v:ext="edit" data="1" />  </o:shapelayout></xml><![endif]-->   <div class="Section1">  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);">I actually think that linking/etc probably doesn’t belong in scene.<o:p></o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);"><o:p> </o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);">Rather – it belongs in SceneObjectGroup – I actually think that SceneObjectPart doesn’t belong in Scene either.<o:p></o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family:
 "Calibri","sans-serif"; color: rgb(31, 73, 125);"><o:p> </o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);">Eg:<o:p></o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);"><o:p> </o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);">InnerScene -> IEntity[]<o:p></o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);"><o:p> </o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);">IEntity[] += SceneObjectGroup<o:p></o:p></span></div>  <div
 class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);"><o:p> </o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);">SceneObjectGroup Parts[];<o:p></o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);"><o:p> </o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);">We do the conversions/etc in LLClientView and other clientstacks which support a IClientPrimitive interface.<o:p></o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);"><o:p> </o:p></span></div>  <div class="MsoNormal"><span
 style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);">Adam<o:p></o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);"><o:p> </o:p></span></div>  <div class="MsoNormal"><span style="font-size: 11pt; font-family: "Calibri","sans-serif"; color: rgb(31, 73, 125);"><o:p> </o:p></span></div>  <div style="border-style: none none none solid; border-color: -moz-use-text-color -moz-use-text-color -moz-use-text-color blue; border-width: medium medium medium 1.5pt; padding: 0cm 0cm 0cm 4pt;">  <div>  <div style="border-style: solid none none; border-color: rgb(181, 196, 223) -moz-use-text-color -moz-use-text-color; border-width: 1pt medium medium; padding: 3pt 0cm 0cm;">  <div class="MsoNormal"><b><span style="font-size: 10pt; font-family: "Tahoma","sans-serif";"
 lang="EN-US">From:</span></b><span style="font-size: 10pt; font-family: "Tahoma","sans-serif";" lang="EN-US"> opensim-dev-bounces@lists.berlios.de [mailto:opensim-dev-bounces@lists.berlios.de] <b>On Behalf Of </b>Michael Wright<br> <b>Sent:</b> Thursday, 6 November 2008 8:01 AM<br> <b>To:</b> opensim-dev@lists.berlios.de<br> <b>Subject:</b> Re: [Opensim-dev] Renaming innerscene<o:p></o:p></span></div>  </div>  </div>  <div class="MsoNormal"><o:p> </o:p></div>  <div class="MsoNormal">Well parts of the Linking /attaching code would need to be in there. The code to actually connect the prims. But the actually handling of the client commands and anything above the actual connect commands should be in the Region level. Like with attachments if we update the attachment db list, that should be handled in the Region level. <br> <br> Same with Rezzing a object. There would be a base command in the InnerScene to add a new prim(s), but the handling of the
 Client command, and the fetching of the Prim data from Inventory would be a higher level function. <br> <br> So I just mean we should keep the very base level of functionality related to the 3D objects in the InnerScene. And anything above that should be in a high level. <br> <br> <b><i>Justin Clark-Casey <jjustincc@googlemail.com></i></b> wrote:<o:p></o:p></div>  <div class="MsoNormal">Michael Wright wrote:<br> > Well I think the plan/design was always meant to be based on functional <br> > units. Region/current Scene was never meant to be a metadata wrapper <br> > around InnerScene. It was meant to contain high level functions, at the <br> > region level. Like rezzing a object/ derezzing a object. Linking <br> > objects, attaching objects to a Avatar. I wouldn't class those as base <br> > scene level functions. And they can be seperated however we want.<br> <br> So what kind of things would be in InnerScene/SceneGraph, if not things link
 linking/delink and attachment/detachment <br> (both of which are there at the moment)? I'm not that familiar with exactly what one would normally expect to be in a <br> scene graph, coming as I do from an enterprise programming environment.<br> <br> > <br> > I just firmly believe we should have a base layer that has the <br> > functionality that is focused on the 3D and sending/receiving clients <br> > related to the movement of objects in the scene.<br> > <br> > */Justin Clark-Casey /* wrote:<br> > <br> > Heh, InnerScene has always confused me too - interesting to find out<br> > that nobody remembers what it was for (as Alan<br> > alluded to, this shows the value of writing embedded class level<br> > code documentation ;)<br> > <br> > I agree with Melanie - I feel that ultimately a split into<br> > functional units would be better than having two classes, one<br> > containing all region related information and one providing
 scene<br> > services. Especially if the outer class just wraps<br> > the inner classes' methods (but I suspect that wouldn't be the plan).<br> > <br> > I assume that the conceptual difference between scene and a region<br> > is that the first provides the core simulation<br> > services while the latter consists of metadata (e.g. which region<br> > the scene represents). If there was a separate region<br> > class, I'm not convinced that there would be very much in it, beyond<br> > what is already in RegionInfo.<br> > <br> > <br> > Melanie wrote:<br> > > As one who has spent a lot of time in InnerScene with the local<br> > > dragons, I see that in lots of cases, even having it InnerScene at<br> > > all complicates things horrendously.<br> > > There are actions that change between Scene and InnerScene 4 times<br> > > to get the job done. That could be done much more efficiently if<br> > > those methods
 were combined and then split up into functional units,<br> > > rather than "3d" scoped.<br> > ><br> > > One flow I found was like this:<br> > ><br> > > Caller calls Scene:A<br> > > Scene:A calls InnerScene:A<br> > > Scene:A calls Scene:B<br> > > Scene:A calls InnerScene:C<br> > > InnerScene:C calls Scene:C<br> > > Scene:A returns.<br> > ><br> > > This is overcomplex and baffling to new coders. I am not surprised<br> > > that many people choose not to touch it.<br> > ><br> > > Needless to say, I removed the above flow and made it into a single<br> > > call into InnerScene. It was part of attachment rezzing. When i did<br> > > that, I streamlined that flow.<br> > ><br> > > I believe InnerScene and Scene should not be separate, or not be<br> > > separated along the line they are separated now.<br> > ><br> > > On the name, I don't really care.<br>
 > ><br> > > +-0 there.<br> > ><br> > > Melanie<br> > ><br> > ><br> > > Frisby, Adam wrote:<br> > >> +1'ing,<br> > >><br> > >> I had no idea what InnerScene was for myself.<br> > >><br> > >> Adam<br> > >><br> > >> From: opensim-dev-bounces@lists.berlios.de<br> > [mailto:opensim-dev-bounces@lists.berlios.de] On Behalf Of Michael<br> > Wright<br> > >> Sent: Thursday, 6 November 2008 5:22 AM<br> > >> To: opensim-dev@lists.berlios.de<br> > >> Subject: [Opensim-dev] Renaming innerscene<br> > >><br> > >> A long time ago, we started the process of separating the 3d<br> > scene handling code into its own class, rather than having it mixed<br> > in with more region level code, like rezzing/inventory handling.<br> > >><br> > >> It was always planned to renamed the InnerScene and Scene<br> > classes, once
 this separation of the 3d scene graph code was<br> > completed. The original plan being that Scene became Region (or<br> > something like that) and InnerScene changed to Scene.<br> > >><br> > >> It might be a bit to much work to rename the Scene class at this<br> > stage. But what are everyone thoughts on renaming InnerScene to<br> > something like SceneGraph.<br> > >><br> > >> I think the InnerScene class is one of the more confusing areas<br> > as a lot new coders aren't really aware what design role it is meant<br> > to play.<br> > >><br> > >><br> > >><br> > >><br> > >><br> > >><br> > ------------------------------------------------------------------------<br> > >><br> > >> _______________________________________________<br> > >> Opensim-dev mailing list<br> > >> Opensim-dev@lists.berlios.de<br> > >>
 https://lists.berlios.de/mailman/listinfo/opensim-dev<br> > > _______________________________________________<br> > > Opensim-dev mailing list<br> > > Opensim-dev@lists.berlios.de<br> > > https://lists.berlios.de/mailman/listinfo/opensim-dev<br> > ><br> > <br> > <br> > -- <br> > justincc<br> > Justin Clark-Casey<br> > http://justincc.wordpress.com<br> > _______________________________________________<br> > Opensim-dev mailing list<br> > Opensim-dev@lists.berlios.de<br> > https://lists.berlios.de/mailman/listinfo/opensim-dev<br> > <br> > <br> > <br> > ------------------------------------------------------------------------<br> > <br> > _______________________________________________<br> > Opensim-dev mailing list<br> > Opensim-dev@lists.berlios.de<br> > https://lists.berlios.de/mailman/listinfo/opensim-dev<br> <br> <br> -- <br> justincc<br> Justin Clark-Casey<br>
 http://justincc.wordpress.com<br> _______________________________________________<br> Opensim-dev mailing list<br> Opensim-dev@lists.berlios.de<br> https://lists.berlios.de/mailman/listinfo/opensim-dev<o:p></o:p></div>  <div class="MsoNormal"><o:p> </o:p></div>  <div>  <o:p></o:p></div>  </div>  </div>  _______________________________________________<br>Opensim-dev mailing list<br>Opensim-dev@lists.berlios.de<br>https://lists.berlios.de/mailman/listinfo/opensim-dev<br></blockquote><br><p>