Closed yujiali closed 9 years ago
It seems like there was a bug in the routine for adding decoys, thanks for finding it @yujiali! It was adding location i
instead of i + 1
as a decoy, which caused locations to show up twice. I believe the changes in https://github.com/facebook/bAbI-tasks/commit/7a37fb55550bdaa1cc6eafa480e132aeb78b136e should fix it; let me know if you think it's still incorrect.
Looks like the problem is fixed, thanks!
I tried to generate some task data for question 19 with the following default command:
In the resulting file I got, I observed some conflicting facts, for example
1 and 4 cannot hold at the same time. Another example is
Here 3 and 4 are confusing. And one more:
In this one there are two paths from bedroom to garden, (1) using fact 2; (2) using 3 and 4. The generated ground truth answer ignored the more direct solution, and the two solutions are actually conflicting, as fact 2 says garden is north of the bedroom but fact 3 and 4 together imply that garden is south of the bedroom.
Is this a common problem?