Skip to content

Commit

Permalink
feat: Added windows support by replacing pytrec-eval with pytrec-eval…
Browse files Browse the repository at this point in the history
…-terrier (#292)

* ci: Added windows to test suite

* feat: Changed to pytrec-eval-terrier to add support for windows installs
  • Loading branch information
KennethEnevoldsen authored Apr 1, 2024
1 parent 39d005b commit fc0e105
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
5 changes: 4 additions & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,10 @@ jobs:
matrix:
os: [ubuntu-latest] #, macos-latest, windows-latest]
python-version: ["3.8", "3.9", "3.10"]

include:
# Add Windows with Python 3.8 only to avoid tests taking too long
- os: windows-latest
python-version: "3.8"
# This allows a subsequently queued workflow run to interrupt previous runs
concurrency:
group: "${{ github.workflow }}-${{ matrix.python-version}}-${{ matrix.os }} @ ${{ github.ref }}"
Expand Down
2 changes: 1 addition & 1 deletion docs/mmteb/points.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

| GitHub | Total points | New dataset | New task | Dataset annotations | (Bug)fixes | Running Models | Review PR | Paper Writing | Ideation | Coordination |
| ----------------- | ------------ | ----------- | -------- | ------------------- | ---------- | -------------- | -------- | -------------- | -------- | ------------- |
| KennethEnevoldsen | | 38 | | | | | | | | |
| KennethEnevoldsen | | 38 | | | 1 | | | | | |
| x-tabdeveloping | | 2 | | | | | | | | |

Note that coordination and ideation is not included in the total points, but is used to determine first and last authors.
Expand Down
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ dependencies = [
"torch",
"tqdm",
"rich",
"pytrec_eval",
"pytrec-eval-terrier>=0.5.6",
"pydantic",
"typing_extensions",
"eval_type_backport",
Expand Down

0 comments on commit fc0e105

Please sign in to comment.