togethercomputer / together-python

The Official Python Client for Together's API
https://pypi.org/project/together/
Apache License 2.0
24 stars 5 forks source link

Fix missing keys in report_dict for _check_jsonl #93

Closed mryab closed 4 months ago

mryab commented 4 months ago

Due to the changes in https://github.com/togethercomputer/together-python/pull/73, there is now a separate report_dict that is created in _check_jsonl. The end of the function checks for multiple keys in that dictionary that are not created by default. This PR fixes the problem by checking for the presence of the key in the dictionary instead