Return-Path: Delivered-To: apmail-commons-dev-archive@www.apache.org Received: (qmail 36650 invoked from network); 28 Aug 2010 22:25:56 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 28 Aug 2010 22:25:56 -0000 Received: (qmail 45333 invoked by uid 500); 28 Aug 2010 22:25:55 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 45162 invoked by uid 500); 28 Aug 2010 22:25:54 -0000 Mailing-List: contact dev-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Commons Developers List" Delivered-To: mailing list dev@commons.apache.org Received: (qmail 45154 invoked by uid 99); 28 Aug 2010 22:25:54 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 28 Aug 2010 22:25:54 +0000 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests=FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of phil.steitz@gmail.com designates 209.85.212.43 as permitted sender) Received: from [209.85.212.43] (HELO mail-vw0-f43.google.com) (209.85.212.43) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 28 Aug 2010 22:25:47 +0000 Received: by vws8 with SMTP id 8so4059812vws.30 for ; Sat, 28 Aug 2010 15:25:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from :user-agent:mime-version:to:subject:references:in-reply-to :x-enigmail-version:content-type:content-transfer-encoding; bh=iN86CT5FfPEiYl+l3YSCI3MP1h0yb3Zn4ENMoRirTbM=; b=ww2zEkW43cQTkY4fsYGUS7/iYa13+7h62Pe7GDrprIWIS/OeUPY1tchNp44jydARjh I9P8/YKwuPrKjtSxnv1N53NbqbVOpwgh2IVc9I5qCl7x2GJET6qBxfcFEw68j97UfJzA Nt7ATqRSPUDr989AmKbpRG7l20dmBTD7uUtZs= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:user-agent:mime-version:to:subject:references :in-reply-to:x-enigmail-version:content-type :content-transfer-encoding; b=RR/O6Rk0CkmGj8dXktZVE2AbUue9aHM/fHenqzZIBAd4+Ni5/e3o6gU5aAmDN9BURU TwwfmlRGzO8eBcxDA+3FlAJB+OBed/IQcrpcJyQwcYgCxdsRuBeqYYC8h0DxS4mw5VBY gTtbB0WfQevaCMYJJb3pTBv6NZgQUZrfotwHM= Received: by 10.220.122.151 with SMTP id l23mr1687828vcr.162.1283034326266; Sat, 28 Aug 2010 15:25:26 -0700 (PDT) Received: from phil-steitzs-macbook-pro.local (c-68-44-120-111.hsd1.de.comcast.net [68.44.120.111]) by mx.google.com with ESMTPS id v11sm3304135vbb.11.2010.08.28.15.25.24 (version=SSLv3 cipher=RC4-MD5); Sat, 28 Aug 2010 15:25:25 -0700 (PDT) Message-ID: <4C798CD3.7040407@gmail.com> Date: Sat, 28 Aug 2010 18:25:23 -0400 From: Phil Steitz User-Agent: Thunderbird 2.0.0.24 (Macintosh/20100228) MIME-Version: 1.0 To: Commons Developers List Subject: Re: [math] unit tests for FastMath References: <4C78546C.9020805@free.fr> In-Reply-To: <4C78546C.9020805@free.fr> X-Enigmail-Version: 0.96.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Luc Maisonobe wrote: > Hi all, > > I have started integrating the FastMath class from MATH-375. I have not > committed anything for now as I am first fixing numerous checkstyle > errors (more than 300) and wanted to have one clean commit to simplify > patching both trunk for 3.0 and branch 2.X. > > One less obvious problem is unit tests. The current tests (which I > converted to Junit 4) depend on a few external classes that themselve > depend on a LGPL library, and this library doesn seem to be available in > maven repositories. This imply that we cannot include the library in the > release, and we cannot either use it as an optional dependency for tests. > > I would suggest to completely remove this library and create tests > simply by storing reference values in files that we would read. It would > be similar to the existing tests for random values or stats. The file > could of course be created using dfp or any other reference high > precision package (having several different reference sources would be > better IMHO). We would remove the random aspect of the tests, which may > or may not be a problem. Sounds like a reasonable approach. Can you explain a little more what the tests do and what you have in mind in terms of data coverage? In particular, what are the "random aspects" of the tests? I know the answer here is really RTFP (p = patch ;) but hey, I am a little lazy this evening. Phil > > Luc > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > For additional commands, e-mail: dev-help@commons.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org For additional commands, e-mail: dev-help@commons.apache.org