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.

Sign up for GitHub

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

Jump to bottom

Improved fusion regression #2627

Open
wants to merge 38 commits into
base: master
Choose a base branch
from
Open

Improved fusion regression #2627

wants to merge 38 commits into from

Conversation

Stefan824
Copy link
Contributor

Improved fusion regression by:

  1. Modified fusion yaml files to include regression yamls and run-name
  2. Adjusted fuse-regression script accordingly
  3. Added error handling for missing run-files

If well, should address #2619

cadurosar and others added 30 commits January 26, 2024 10:57
@cadurosar
add fusion
1cb5ea1
@cadurosar
Merge branch 'castorini:master' into master
fe7e529
@DanielKohn1208
added cadurosar's code via a copy + paste and made changes to match p…
c626a7c 
…yserini convention
@DanielKohn1208
merged cadurosar's code
4d702d0
@DanielKohn1208
moved FuseRuns
7db24a9
@DanielKohn1208
merged cadurosar's code with modifications
e7efd1b
@DanielKohn1208
added run fusion to match pyserini implementation
ebd8ed4
@Stefan824
added fusion feature
1b398af
@Stefan824
modified arguments; added test cases
27b44df
@Stefan824
modified TrecRun class code style
72e6e06
@Stefan824
added comment
5f7ec35
@Stefan824
deleted test file from previous version
509049c
@Stefan824
Added dependency for junit test
39f62a9
@Stefan824
resolved formatting; merged trectools module to fusion
37e89fa
@Stefan824
remove unused test cases
54c74b4
@Stefan824
removed unused test files
32e13c2
@Stefan824
Merge remote-tracking branch 'origin/master' into add-fusion
6c648f7
@Stefan824
added fusion regression script paired with two yaml test files
a9d7804
@Stefan824
added md for test
e049e48
@Stefan824
add cmd on test instruction
bd0ce76
@Stefan824
removed abundant dependency
17ceb49
@Stefan824
revert unecessary change
6f550b1
@Stefan824
resolved a minor decoding issue
f4644e1
@Stefan824
added a yaml that is based on regression test run results
0ea8369
@Stefan824
added doc for test2
ec57e96
@Stefan824
typo
042b678
@Stefan824
changed name for test yamls
f2b6f4c
@Stefan824
second attempt to revert src/main/resources/regression/beir-v1.0.0-ro…
d94c0f9 
…bust04.flat.yaml back
@Stefan824
fixed precision and added run_origins for fusion yaml
f5871b9
@Stefan824
removed two yamls that use runs not from current regression experiments
b7961f3
@lintool
Copy link
Member

lintool commented Nov 8, 2024

@Stefan824 can you please fix the conflict?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Stefan824 @lintool @cadurosar @DanielKohn1208