Skip to content

Conversation

keviddles
Copy link

@okhat
Copy link
Collaborator

okhat commented Jul 31, 2025

Hmm, what's a more fundamental fix? What was trusting remote code replaced with?

(OR at least let's not pin an exact version. For example pin anything strictly < 4.0)

@lhoestq
Copy link

lhoestq commented Aug 11, 2025

Hi ! I updated the hotpot_qa dataset to make it compatible with datasets>=4 here: https://huggingface.co/datasets/hotpotqa/hotpot_qa/discussions/9

it should be all good now :)

I also pinged the authors of PolyAI/banking77 to update their datasets at https://huggingface.co/datasets/PolyAI/banking77/discussions/6

@okhat
Copy link
Collaborator

okhat commented Aug 11, 2025

Amazing, thank you @lhoestq !!

@okhat
Copy link
Collaborator

okhat commented Aug 12, 2025

I suppose we should edit our HotPotQA class then to remove that kwarg

@lhoestq
Copy link

lhoestq commented Aug 12, 2025

Yes otherwise you'll get some warnings that this kwarg should be removed (it won't error out though since the dataset doesn't have remote code anymore anyways, and it will only show a warning)

okhat added a commit that referenced this pull request Sep 1, 2025
okhat added a commit that referenced this pull request Sep 1, 2025
cameronraysmith pushed a commit to cameronraysmith/dspy that referenced this pull request Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants