search

verbal-autopsy-software / Tariff

R package of Tariff method for VA
0 stars 4 forks source link

Code incorrectly sets top tariffs to zero #16

Closed mjutras-mitre closed 5 years ago

mjutras-mitre commented 5 years ago

The algorithm describes taking the top 40 Tariffs, but this code seems to be setting the top 40 Tariffs to zero thereby only looking at 41 to the end

https://github.com/verbal-autopsy-software/Tariff/blob/2324381cf330fc82f6022a950e7579fdee413271/Tariff/R/Tariff.r#L235-L251

richardli commented 5 years ago

5ac9ff922e9c47f45c9a3fe47eb693edc78bb498

Bug fixed. Thank you. Silver lining is that use.top is default to FALSE.