-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error while Evaluating #29
Comments
That's because the test set of the MIND large dataset has no labels, so the format of the test file is not correct. See #11 and msnews/MIND#8. ... and it seems that there is nothing we can do 😢 |
@yusanshi. Does it mean the labels were not released by the authors of dataset/task? |
And if the case is that it was not released? How do we evaluate? Thanks |
Exactly.
Please see https://msnews.github.io/ and https://competitions.codalab.org/competitions/24122#participate. Basically you need to upload the inference result to the online evaluation platform. To generate the evaluation results in required file format, you should make some changes to the code. In the earlier version of the repo there are some code that may be helpful: news-recommendation/src/evaluate.py Lines 259 to 262 in fe11cef
|
Thanks. I am going to try that out |
There is still an issue with the model
|
Looks like you're using the old version code? Please use the latest code, i.e., the newest commit in master branch. Or I don't know which code you're using so I can't give any suggestions. |
|
Thanks @yusanshi. I have done that already. I am working on writing the results to file as suggested by you and according to the standard specified by the organizers but I seem to be lost.
in the evaluate file |
Since we have no labels so the following code makes no sense: news-recommendation/src/evaluate.py Lines 244 to 257 in fe11cef
Simply removing them should be OK. |
Hello @yusanshi,
There is an issue with indexing in the evaluate file precisely line 262:
This issue happens on MIND large dataset
Thanks
The text was updated successfully, but these errors were encountered: