relProd uses buggy implementation

Issue #5 resolved
Guy Van den Broeck created an issue

relProd_rec is known to have bugs and indeed does not terminate on my benchmark relProd_rec_ORG is known to be correct but slower? It makes more sense for relProd to use relProd_rec_ORG.

Comments (9)

  1. Arash Vahidi (private) repo owner

    Maybe relProd_rec is suspected to have a corner-case bug is a better description. But I don't think I have the code that triggered this issue anymore.

    If you have a benchmark that can reproduce the bug I will be able to look into it.

  2. Arash Vahidi (private) repo owner

    Guy, just a reminder that if you have code that triggers an error I would very much like to see it.

  3. Guy Van den Broeck reporter

    Sorry, the code is part of a large unpublished system and it's difficult to extract the bug from it.

  4. Arash Vahidi (private) repo owner

    After some internal changes in 110, this issue can no longer be reproduced.

  5. Log in to comment