Commit e01914f
authored
Fix incorrectly uploaded RetroKNN weights (#91)
As noticed by @panpiort8, the default USPTO50K-trained RetroKNN weights
did not reproduce the USPTO50K accuracy reported in our paper, only
getting to ~50% instead of ~55%. It turns out that the weights got mixed
up when uploading from our internal storage to figshare. In short, the
RetroKNN weights consist of LocalRetro weights and an activation store
for retrieval, but the store present in the checkpoint was generated
using a _different_ checkpoint of LocalRetro than the one it was
uploaded with. After fixing this, the accuracy is back to its expected
~55% range.1 parent e7097bf commit e01914f
File tree
3 files changed
+6
-2
lines changed- docs
- syntheseus/reaction_prediction/inference
3 files changed
+6
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
10 | 14 | | |
11 | 15 | | |
12 | 16 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
16 | | - | |
| 16 | + | |
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
8 | | - | |
| 8 | + | |
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
0 commit comments