Skip to content

Commit

Permalink
Fix typo kwargs=>hf_kwargs (#853)
Browse files Browse the repository at this point in the history
* Fix kwargs typo, should be hf_kwargs

* update test
  • Loading branch information
irenedea authored Jan 10, 2024
1 parent ddba5c8 commit d822630
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion llmfoundry/data/finetuning/dataloader.py
Original file line number Diff line number Diff line change
Expand Up @@ -190,7 +190,7 @@ def build_finetuning_dataloader(cfg: DictConfig,
max_seq_len=cfg.dataset.max_seq_len,
preprocessing_fn=preprocessing_fn,
tokenizer=tokenizer,
hf_kwargs=cfg.dataset.get('kwargs', {}))
hf_kwargs=cfg.dataset.get('hf_kwargs', {}))

# Ensure dataset is large enough.
if cfg.drop_last:
Expand Down
2 changes: 1 addition & 1 deletion tests/data/test_dataloader.py
Original file line number Diff line number Diff line change
Expand Up @@ -326,7 +326,7 @@ def test_finetuning_dataloader_safe_load(hf_name: str,
'decoder_only_format': True,
'shuffle': True,
'safe_load': True,
'kwargs': {
'hf_kwargs': {
'revision': hf_revision
}
},
Expand Down

0 comments on commit d822630

Please sign in to comment.