Skip to content
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

replacing pytrec_eval with pytrec-eval-terrier #175

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

archersama
Copy link
Contributor

When replacing pytrec_eval with pytrec-eval-terrier, several installation issues of pytrec_eval can be effectively addressed. These issues include, but are not limited to:

  1. Compilation Failure: The need for compilation during installation often leads to failures, as discussed in this Stack Overflow post: https://stackoverflow.com/questions/74444484/pytrec-library-not-able-to-install-on-my-system-raising-error.
  2. Path Issues on Windows: Installation failures on Windows due to path-related problems, as documented in this GitHub issue: Pip install error on windows 10, possible hardcoded path cvangysel/pytrec_eval#32.

By using pytrec-eval-terrier, these installation problems can be effectively mitigated, as demonstrated by the solution provided in this comment: cvangysel/pytrec_eval#32 (comment).

@thakur-nandan thakur-nandan merged commit 8f8384e into beir-cellar:main Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants