Skip to content

Commit

Permalink
Fix domain field can be None
Browse files Browse the repository at this point in the history
  • Loading branch information
jkawamoto committed Jul 23, 2024
1 parent 9a28bb7 commit 2a4cdde
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion sacrebleu/utils.py
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import itertools
import json
import os
import re
Expand Down Expand Up @@ -548,7 +549,8 @@ def filter_subset(systems, test_sets, langpair, origlang, subset=None):
from .dataset.wmt_xml import WMTXMLDataset
if isinstance(dataset, WMTXMLDataset):
fields = dataset._unwrap_wmt21_or_later(rawfile)
for doc_origlang, doc_domain in zip(fields['origlang'], fields['domain']):
domains = fields['domain'] if 'domain' in fields else itertools.repeat(None)
for doc_origlang, doc_domain in zip(fields['origlang'], domains):
if origlang is None:
include_doc = True
else:
Expand Down

0 comments on commit 2a4cdde

Please sign in to comment.