Missing files

Question

Missing files

Closed this issue 3 years ago · 3 comments

Hi!

I was trying to reproduce your results and I found that there are missing files. Specifically, the f1.yml file is missing (but it is not a big deal), but the entire ResNetModel and ActualTPR folders are missing as well, which is a bit more cumbersome.

I am not sure if there are more missing files, but I would appreciate if you can add all missing files.

Best regards

Answer 1 · 2021-04-16T10:49:54.000Z

Hi Attached is the f1.yml ResNetModels is a code that I wrote to try also using resnet, instead of densnet. But eventually, I totally removed it from the code. Perhaps it has not been deleted from some places in the code and you face this error. Please remove the line that imports it or any line that may use resnet. It will not hurt at all. All results are based on densenet. Except for train, test and validation split that I was not allowed to load in the git hub the rest shout be fine.You may make your own 80-10-10 split (based on patient ID, no patient has to be shared between train-test and validation set). And use the code to train your own model with the same hyperparameters. As the test size is large you should be fine and the results should not be split sensitive. Also, ask Physionet about how you can get access to extended meta-data (race, insurance, etc.) I will add that file if there was any other missing let me know. Best Laleh

…

On Fri, Apr 16, 2021 at 5:16 AM Adrián Javaloy ***@***.***> wrote: Hi! I was trying to reproduce your results and I found that there are missing files. Specifically, the f1.yml file is missing (but it is not a big deal), but the entire ResNetModel folder is missing as well, which is a bit more cumbersome. I am not sure if there are more missing files, but I would appreciate if you can add all missing files. Best regards — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIYJR7TPM6RGW2HQQUCNWJLTI76APANCNFSM43BEW5WA> .

-- Best Laleh

Answer 2 · 2021-04-16T10:58:44.000Z

Thanks for the really quick response!

I realized that Resnet was not need and commented out the line importing it, and I got another error asking about a folder ActualTPR that is also missing. I haven't looked to deep into the code, so I am not sure whether that one is important or not.

Thanks!

Answer 3 · 2021-04-16T17:39:22.000Z

It is not a folder it is a .py file. It may refer to some files in the code which are related to data and I can not share them due to the data-sharing agreement. You need to download the data on your side and adapt the code to your setup. Best

…

On Fri, Apr 16, 2021 at 6:58 AM Adrián Javaloy ***@***.***> wrote: Thanks for the really quick response! I realized that Resnet was not need and commented out the line importing it, and I got another error asking about a folder ActualTPR that is also missing. I haven't looked to deep into the code, so I am not sure whether that one is important or not. Thanks! — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIYJR7QZ6HP5E7ZDYQ3BAJDTJAJ7FANCNFSM43BEW5WA> .

-- Best Laleh