← Back to team overview

yade-dev team mailing list archive

Re: [Branch ~yade-dev/yade/trunk] Rev 2585: 1. Fix bug in VTKRecorder (Thanks, Vaclav)

 

Sorry it is my fault. I will fix it in a while. Bruno is right, tests
are useful.
Sorry again, Chiara

On 03/12/2010, Anton Gladky <gladky.anton@xxxxxxxxx> wrote:
> Bruno, please, no offense!
>
> It is just commented out with corresponding message in the code and comments
> in commit.
> It can be enabled very easy after fix.
>
>
> Anton
>
>
>
> On Fri, Dec 3, 2010 at 1:15 AM, Bruno Chareyre
> <bruno.chareyre@xxxxxxxxxxx>wrote:
>
>>
>> >   2. Cohesive-chain test disabled (please, fix it, who can...),
>> https://yade-dem.org/buildbot/builders/yade-full/builds/127
>>
>> Please guys, don't remove regression tests when they fail.
>> More logical would be to fix the code that makes the test fail (most
>> probably last
>> Chiara's commit), else why are we adding regression tests?
>>
>> Bruno
>>
>> _______________________________________________
>> Mailing list:
>> https://launchpad.net/~yade-dev<https://launchpad.net/%7Eyade-dev>
>> Post to     : yade-dev@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe :
>> https://launchpad.net/~yade-dev<https://launchpad.net/%7Eyade-dev>
>> More help   : https://help.launchpad.net/ListHelp
>>
>



Follow ups

References