-
Notifications
You must be signed in to change notification settings - Fork 32
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
nicknames, metric parameters, empty inputs, and memory leaks #18
Conversation
Also now support for #12 |
… wrapper around cpp module which converts alternative formats to trec_eval format.
hmmm, gotta look into the failing build. |
1) Fixed issue with empty qrels on some platforms 2) Exposed the values nicknames expand to and moved logic to wrapper 3) Some cleanupq
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Can you also bump the version in setup.py
?
done |
Thanks @seanmacavaney! I want to take another closer look at this later, before pushing out a new release to PyPi. However, I'll merge this now so that it's installable by pointing pip to the repository. |
No description provided.