RE-Net icon indicating copy to clipboard operation
RE-Net copied to clipboard

bug in model.predict() with s and s.item()

Open JuliaGast opened this issue 3 years ago • 1 comments

This (l.232 ff):

            for s, prob_s in zip(subjects, prob_subjects):
                if s in s_done:
                    continue
                else:
                    s_done.add(s)

and this:

            for o, prob_o in zip(objects, prob_objects):
                if o in o_done:
                    continue
                else:
                    o_done.add(o)

does not return a true for the if-statement. because e.g., tensor(53) is not equal to tensor(53) - these are different hashs. Instead, you'd have to modify it to

            for s, prob_s in zip(subjects, prob_subjects):
                if s.item() in s_done:
                    continue
                else:
                    s_done.add(s.item())

(and same for objects)

JuliaGast avatar May 23 '22 15:05 JuliaGast

Thanks for the bug! But it does not change the inference.

woojeongjin avatar Aug 17 '22 23:08 woojeongjin