Skip to content

Commit

Permalink
Update datasets requirement from <2.20,>=2.19 to >=2.20.0,<2.21 (#1330)
Browse files Browse the repository at this point in the history
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Daniel King <[email protected]>
Co-authored-by: Milo Cress <[email protected]>
Co-authored-by: Milo Cress <[email protected]>
  • Loading branch information
4 people authored Nov 6, 2024
1 parent 1f4de8f commit a16b93d
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@
'transformers>=4.43.2,<4.47',
'mosaicml-streaming>=0.9.0,<0.10',
'torch>=2.4.0,<2.4.1',
'datasets>=2.19,<2.20',
'datasets>=2.20.0,<2.21',
'fsspec==2023.6.0', # newer version results in a bug in datasets that duplicates data
'sentencepiece==0.2.0',
'einops==0.8.0',
Expand Down
9 changes: 9 additions & 0 deletions tests/eval/test_in_context_learning_datasets.py
Original file line number Diff line number Diff line change
Expand Up @@ -279,6 +279,7 @@ def test_update_generation_kwargs(
hf_loading_vars = {
'split': 'test',
'name': 'invoker',
'trust_remote_code': True,
}
hf_parsing_map = {'context': ['quas', 'wex', 'exort'], 'answer': ['spell']}
gen_kwargs = {'test_arg1': 1, 'test_arg2': 2}
Expand Down Expand Up @@ -370,6 +371,7 @@ def test_update_generation_kwargs_no_kwargs(
hf_loading_vars = {
'split': 'test',
'name': 'invoker',
'trust_remote_code': True,
}
hf_parsing_map = {'context': ['quas', 'wex', 'exort'], 'answer': ['spell']}

Expand Down Expand Up @@ -459,6 +461,7 @@ def test_construct_context(
hf_loading_vars = {
'split': 'test',
'name': 'invoker',
'trust_remote_code': True,
}
hf_parsing_map = {'context': ['quas', 'wex', 'exort'], 'answer': ['spell']}

Expand Down Expand Up @@ -513,6 +516,7 @@ def test_get_answer_from_example(
hf_loading_vars = {
'split': 'test',
'name': 'invoker',
'trust_remote_code': True,
}
hf_parsing_map = {'context': ['quas', 'wex', 'exort'], 'answer': ['spell']}

Expand Down Expand Up @@ -552,6 +556,7 @@ def test_fix_eos_on_preamble(tmp_path: Path):
hf_loading_vars = {
'split': 'test',
'name': 'invoker',
'trust_remote_code': True,
}
hf_parsing_map = {'context': ['quas', 'wex', 'exort'], 'answer': ['spell']}

Expand Down Expand Up @@ -592,6 +597,7 @@ def test_tokenize_example_with_tokenize_labels(
hf_loading_vars = {
'split': 'test',
'name': 'invoker',
'trust_remote_code': True,
}
hf_parsing_map = {'context': ['quas', 'wex', 'exort'], 'answer': ['spell']}

Expand Down Expand Up @@ -658,6 +664,7 @@ def test_tokenize_example_with_no_tokenize_labels(
hf_loading_vars = {
'split': 'test',
'name': 'invoker',
'trust_remote_code': True,
}
hf_parsing_map = {'context': ['quas', 'wex', 'exort'], 'answer': ['spell']}

Expand Down Expand Up @@ -2429,6 +2436,7 @@ def test_lm_spacing_dataloader(
[{
'split': 'test',
'name': 'juggernaut',
'trust_remote_code': True,
}],
)
@pytest.mark.parametrize(
Expand Down Expand Up @@ -2508,6 +2516,7 @@ def test_hf_dataloading_lm_dataloader(
[{
'split': 'test',
'name': 'invoker',
'trust_remote_code': True,
}],
)
@pytest.mark.parametrize(
Expand Down

0 comments on commit a16b93d

Please sign in to comment.