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

Fix TREC-COVID regressions and fine-tuning experiments #2431

Merged
merged 3 commits into from
Mar 29, 2024
Merged

Conversation

lintool
Copy link
Member

@lintool lintool commented Mar 29, 2024

Renamed SearchCollection parameters a while ago but never fixed in Python scripts for TREC-COVID regressions and fine-tuning experiments.

@lintool lintool requested a review from MXueguang March 29, 2024 01:00
Copy link

codecov bot commented Mar 29, 2024

Codecov Report

Attention: Patch coverage is 83.33333% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 66.88%. Comparing base (de73879) to head (fb02614).
Report is 2 commits behind head on master.

Files Patch % Lines
src/main/java/io/anserini/search/ScoredDocs.java 83.33% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2431      +/-   ##
============================================
- Coverage     66.88%   66.88%   -0.01%     
  Complexity     1410     1410              
============================================
  Files           208      208              
  Lines         12110    12112       +2     
  Branches       1487     1488       +1     
============================================
+ Hits           8100     8101       +1     
  Misses         3494     3494              
- Partials        516      517       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lintool lintool merged commit b91df2f into master Mar 29, 2024
3 checks passed
@lintool lintool deleted the bugfix branch March 29, 2024 11:03
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