[Users] Changes to Multipole

Ian Hinder ian.hinder at aei.mpg.de
Sat Jun 12 09:24:39 CDT 2010


On 12 Jun 2010, at 15:44, Frank Loeffler wrote:

> Hi,
>
> On Sat, Jun 12, 2010 at 03:07:37PM +0200, Ian Hinder wrote:
>> I would like to clean up some aspects of the
>> API now before the release.  If there are no objections, I will make
>> these changes so that they can be tested as part of the release  
>> process.
>
> As far as I can see (please correct me if I am wrong), Multipole  
> currently
> neither has own testsuites, nor is it used by another testsuite in the
> ET.
> This is bad, because it means that the thorn is probably not very
> well tested. On the other hand, it also means that there are not less
> reasons not to change something with this thorn.

Indeed.  As mentioned in the call on Monday, I am already working on  
regression tests for this thorn, as I also consider them important.   
The thorn was fairly well tested when it was written and has not seen  
much development since then.

> I am in favor of the changes you suggest, especially because it is a
> cleanup of the API which means less changes between this and the next
> release, and the thorn is not that old in general as well.
>
> I would however strongly suggest to include a few, but at least one,
> testsuite as well. We are on a tight schedule with the testing and  
> will
> fully start testing on Monday. Do you think you could have this  
> cleanup
> finished by then? We really would need to have those changes  
> included in
> the testing if they should be included in the release.

Yes, that should be possible.

-- 
Ian Hinder
ian.hinder at aei.mpg.de



More information about the Users mailing list