kudkudak / common-sense-prediction

Common sense prediction using DL.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Unit test human evaluation

kudkudak opened this issue · comments

Add unit test, just to err on the safe side

This should test both the distance calculation, ordering, bucketing, and mixing. All parts independently and jointly.

I am not sure if compute_distances and compute_distances_fast are actually equivalent. There is some issue with buckets having incorrect triplets (e.g. close ones)? Extremely weird and worrying.

Idea: compute_distances_fast with batch_size=1 and batch_size=100.