Skip to content

Conversation

ddeville
Copy link
Contributor

@ddeville ddeville commented Jan 12, 2023

Fixes #141

Co-authored-by: andrewpeng@openai.com

Co-authored-by: andrewpeng@openai.com <Andrew Peng>
@ddeville ddeville requested a review from athyuttamre January 12, 2023 19:34
@ddeville ddeville merged commit e5b7d1a into main Jan 13, 2023
@ddeville ddeville deleted the damien/debug-level branch January 13, 2023 01:06
jhallard added a commit that referenced this pull request Jun 5, 2023
* [EZ] Remove sort-by-key when rendering JSON responses

The order of items in the API responses is intentional, and this order is clobbered by the rendering of `OpenAIObject`. This change removes the alphabetic sort
of response keys

* [EZ] Add unit test for new case

* Lint/fmt fixes

* Bleh isort

* Remove `internal_metrics` field
cgayapr pushed a commit to cgayapr/openai-python that referenced this pull request Dec 14, 2024
…ai#178)

Co-authored-by: andrewpeng@openai.com <Andrew Peng>

Co-authored-by: Andrew Peng <apeng@berkeley.edu>
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.

Too verbose logs
3 participants