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

Merged
merged 38 commits into from
Jan 16, 2025
Merged

Improved fusion regression #2627

merged 38 commits into from
Jan 16, 2025

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?

@lintool lintool self-requested a review January 16, 2025 20:50
@lintool lintool merged commit 4609d0b into castorini:master Jan 16, 2025
1 check passed
@lintool lintool mentioned this pull request Jan 16, 2025
Merged
lintool added a commit that referenced this pull request Jan 16, 2025
@lintool
Tweak fusion regression - minor changes on top of #2627 (#2683)
0e7b63c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lintool lintool lintool approved these changes

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