Closed dalgu90 closed 2 years ago
@dalgu90, forgot to mention one thing - please add the results for Fusion to the README in this PR itself
Thanks @abheesht17 for the detailed review. Let me fix them in the later commit (after our next meeting).
Thanks @abheesht17 for the detailed review. Let me fix them in the later commit (after our next meeting).
Sure! :)
Hey team. I reflected the comments except the ones with fixing the class/layer names.
Hello guys.
I ported the Fusion model. You can see the paper's hyper-parameters in the config files (
configs/fusion
). Paper: https://aclanthology.org/2021.findings-acl.184/ GitHub repo: https://github.com/machinelearning4health/FusionNOTE: In the paper, the performance on the MIMIC-III top-50 (equivalent to our top-50 old dataset) was made by training on the full dataset (with 5x examples). This dataset is equivalent to our new top-50 dataset, so we compared it below. The authors now share the results on the original top-50 dataset, which we compared in the MIMIC-III top-50 (old).
Here's the raw output of our implementation: