<html><head><style type="text/css"><!-- DIV {margin:0px;} --></style></head><body><div style="font-family:times new roman, new york, times, serif;font-size:12pt"><div style="font-family: times new roman,new york,times,serif; font-size: 12pt;">I'm perfectly fine with the output format changing. I never wrote up a full specification or DTD, and for a while I was considering using LLSD formatted output. I was also considering writing a generic data marshalling framework since right now our data collector classes are mainly "copy this variable from this object and put it in this XML tag" iterated over a collection. But I was having doubts about how generic it could be while still being useful.<br><br>P.S. you'll also notice that you can automatically load new collectors into the DataSnapshot module by creating a class derived from IDataSnapshotProvider and implementing the required methods. Also the class has to be in the same assembly. Eventually I wanted
to have other modules be able to add to the DataSnapshot output as well, but since we haven't stabilized the output format just yet I don't think it's a good idea to do so right now.<br><br><div style="font-family: times new roman,new york,times,serif; font-size: 12pt;">----- Original Message ----<br>From: Justin Clark-Casey <jjustincc@googlemail.com><br>To: opensim-dev@lists.berlios.de<br>Sent: Sunday, April 6, 2008 1:30:01 PM<br>Subject: Re: [Opensim-dev] DataSnapshot patch in Mantis<br><br>
Charles Krinke wrote:<br>> Diva and Kmeisthax have created and submitted a patch to Mantis. It is <br>> Mantis#889.<br>><br>> I would appreciate it if Mw, Lbsa or Sdague could give a glance at it <br>> before we commit it.<br>><br>Well, I know you didn't ask my opinion, but I'm going to give it to you <br>anyway :-)<br><br>+1<br><br>Having read Diva's e-mail and briefly looked through the code, I support <br>the idea that this patch should be included as a core module. The <br>exposure of region information directly to the internet could enhance <br>integration with lots of other services, as well as allowing facilities <br>such as search. From a casual look, it seems that the module is written <br>with this generic aim in mind, rather than for search specific <br>functionality.<br><br>However, I would just like to say that I think that by inserting this <br>code directly into the OpenSim codebase, this gives developers
license - <br>whether they are core or by way of accepted patch - to change its <br>output, possibly at short notice. Since the aim of exposing region data <br>is generic, we may come upon a better way of representing output, <br>particularly at this alpha stage. We might feel that it would be better <br>to change this generic module directly, rather than establish another <br>parallel module.<br><br>If this happened, it would be up to people making use of the module to <br>adapt to the changes, rather than for a lot of negotiation to happen up <br>front before any changes are made.<br><br>I don't necessarily think this is likely, but I believe it is a <br>possibility that Diva and Kmesithax need to be aware of. I think this <br>is one of the costs associated with making this a core module (as well <br>as the necessity of non-core developers changing it by patch).<br><br>If Diva and Kmesithax are happy to live with that, then I, for
one, <br>welcome and commend their efforts :-)<br><br>-- <br>justincc<br>Justin Clark-Casey<br><br>_______________________________________________<br>Opensim-dev mailing list<br><a ymailto="mailto:Opensim-dev@lists.berlios.de" href="mailto:Opensim-dev@lists.berlios.de">Opensim-dev@lists.berlios.de</a><br><a href="https://lists.berlios.de/mailman/listinfo/opensim-dev" target="_blank">https://lists.berlios.de/mailman/listinfo/opensim-dev</a><br></div><br></div></div><br>
<hr size=1>You rock. That's why Blockbuster's offering you <a href="http://us.rd.yahoo.com/evt=47523/*http://tc.deals.yahoo.com/tc/blockbuster/text5.com">one month of Blockbuster Total Access</a>, No Cost.</body></html>