-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
data must be str, not NoneType #244
Comments
I looked at .traj file
|
Thanks for the report, this was probably fixed in #242. Let me know if switching to the latest version didn't solve this! |
@klieret I'm using
|
Could you send me the full ommand that you ran? Also it looks like you're running from the |
In fact, on the latest |
Oh! I'm new to Docker, so I didn't know there was a matching docker container version. I seem to be using the latest version:
Edit I believe that I need to run Edit 2 The error does not occur again after running |
Looks like we can close this? Thanks @klieret |
Describe the issue
Hi I create a very simple repo and create an issue.
it showed the correct result but never save in prediction jsonl and model patch
Error:
INFO Found submission:
INFO Saved trajectory to trajectories/root/azure-gpt4__viciaky__test__default_from_url__t-0.00__p-0.95__c-3.00__install-1/viciaky__test-i1.traj
INFO Saved predictions to trajectories/root/azure-gpt4__viciaky__test__default_from_url__t-0.00__p-0.95__c-3.00__install-1/all_preds.jsonl
Traceback (most recent call last):
File "/app/run.py", line 144, in main
save_patch(traj_dir, instance_id, info)
File "/app/run.py", line 271, in save_patch
patch_output_file.write_text(model_patch)
File "/usr/local/lib/python3.9/pathlib.py", line 1283, in write_text
raise TypeError('data must be str, not %s' %
TypeError: data must be str, not NoneType
WARNING ❌ Failed on viciaky__test-i1: data must be str, not NoneType
Suggest an improvement to documentation
No response
The text was updated successfully, but these errors were encountered: