Skip to content

Commit

Permalink
Dont serialize result (#81)
Browse files Browse the repository at this point in the history
* Don't serialize result

* lint
  • Loading branch information
jakethekoenig authored May 27, 2024
1 parent 880f105 commit 7ab7f9f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ packages=["spice"]

[project]
name = "spiceai"
version = "0.3.8"
version = "0.3.9"
license = {text = "Apache-2.0"}
description = "A Python library for building AI-powered applications."
readme = "README.md"
Expand Down
1 change: 1 addition & 0 deletions spice/spice.py
Original file line number Diff line number Diff line change
Expand Up @@ -288,6 +288,7 @@ def _log_response(self, response: SpiceResponse, name: Optional[str] = None):
full_name = f"{base_name}_{self._cur_logged_names[base_name]}.json"
self._cur_logged_names[base_name] += 1
response_dict = dataclasses.asdict(response)
response_dict.pop("_result", "") # May not be serializable
response_json = json.dumps(response_dict, cls=MessagesEncoder)

logging_dir = self.logging_dir / self._cur_run
Expand Down

0 comments on commit 7ab7f9f

Please sign in to comment.