the code is working, the test fails because different platforms give slightly different results, floating point issues. I can release the threshold in the test, but prefer to find better solution.
On Mon, Oct 21, 2013 at 11:21 AM, Ben Webb ben@salilab.org wrote:
> On 10/21/13 11:16 AM, Barak Raveh wrote: > >> perhaps remove those SAXS failed tests from the release branch? It is >> nice if users get a fail free release (so they can know if they have >> real installation problems) >> > > I don't see what that buys you - the non-working code is still there. If > someone manages to get as far as running the tests while still having "real > installation problems" most likely all of them will fail anyway - that > should be obvious. > > > Ben > -- > ben@salilab.org http://salilab.org/~ben/ > "It is a capital mistake to theorize before one has data." > - Sir Arthur Conan Doyle > ______________________________**_________________ > IMP-dev mailing list > IMP-dev@salilab.org > https://salilab.org/mailman/**listinfo/imp-devhttps://salilab.org/mailman/listinfo/imp-dev >