MASON-INTEREST-L Archives

May 2011

MASON-INTEREST-L@LISTSERV.GMU.EDU

Options: Use Monospaced Font
Show Text Part by Default
Show All Mail Headers

Message: [<< First] [< Prev] [Next >] [Last >>]
Topic: [<< First] [< Prev] [Next >] [Last >>]
Author: [<< First] [< Prev] [Next >] [Last >>]

Print Reply
Subject:
From:
Sean Luke <[log in to unmask]>
Reply To:
MASON Multiagent Simulation Toolkit <[log in to unmask]>
Date:
Wed, 11 May 2011 10:04:54 -0400
Content-Type:
text/plain
Parts/Attachments:
text/plain (47 lines)
Hi Ben.  The sim.util.distribution implementations are actually  
slightly modified versions of original code from the now-defunct COLT  
project at CERN -- and with a statement to that effect [basically "we  
didn't make it so we can't vouch for other people's crummy code, but  
this might be useful for you" :-)  ].  This incorrect documentation is  
from them.

If you're concerned about the documentation, you should see the magic  
numbers in their code.  :-(

We can go over their documentation but it's going to be a long time in  
the queue.  But if YOU have time to prod at it, I will gladly fix  
every single error you find in that documentation.  I've fixed the  
ChiSquare doc bug.

[Sorry that this is a variant of the classic irritating Linux  
developers' response "well if you want a feature YOU write it" --  
which I hate.]

Sean

On May 11, 2011, at 9:41 AM, Ben Stabile wrote:

> I've been going over the "sim.util.distribution" implementations and  
> I've noticed that the code comments (JavaDoc) has several  
> inconsistencies. I know that the online docs don't include these  
> comments (yet?),  but if you DO generate the docs someone should  
> review the descriptions of parameter boundaries.
>
> For example, in ChiSquare class documentation (line 19) the  
> following is found:
>
>  * Valid parameter ranges: <tt>freedom &gt; 0</tt>.
> In fact, the parameter must be greater than or equal to 1.0 or an  
> ArgumentException will be thrown in "setState(double freedom)" (line  
> 142).
>
> Other inconsistencies exist elsewhere, but I'm just pointing this  
> one out so you can be aware that a thorough review should be  
> conducted if you plan on publishing the class docs for these  
> distributions. Otherwise, you could end up with quite a flurry of  
> newsgroup queries, or worse, a lot of broken code. ;-)
>
> Regards,
>
> Ben S.

ATOM RSS1 RSS2