Skip to content
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

Hallacy/v23 #116

Merged
merged 7 commits into from
Aug 24, 2022
Merged

Hallacy/v23 #116

merged 7 commits into from
Aug 24, 2022

Conversation

hallacy
Copy link
Collaborator

@hallacy hallacy commented Aug 19, 2022

  • Adds support for url timeouts with the request_timeout flag
  • Adds logging of the request_id
  • Wandb import warning only occurs if the wandb package isn't installed AND user requests the wandb tools via the CLI. Previously a warning would appear if the wandb package wasn't installed for all CLI commands.
  • Drops specifying the numpy version
  • Bumps version to 0.23.0

fps7806 and others added 7 commits August 19, 2022 13:52
@hallacy hallacy requested a review from hponde August 19, 2022 21:03
@hallacy hallacy merged commit 3edecbe into main Aug 24, 2022
@hallacy hallacy deleted the hallacy/v22 branch August 24, 2022 03:06
techthiyanes pushed a commit to techthiyanes/openai-python that referenced this pull request Oct 2, 2023
cgayapr pushed a commit to cgayapr/openai-python that referenced this pull request Dec 14, 2024
cgayapr pushed a commit to cgayapr/openai-python that referenced this pull request Dec 14, 2024
* overload output type depending on stream literal (openai#142)

* Bump to v22

* [numpy] change version (openai#143)

* [numpy] change version

* update comments

* no version for numpy

* Fix timeouts (openai#137)

* Fix timeouts

* Rename to request_timeout and add to readme

* Dev/hallacy/request timeout takes tuples (openai#144)

* Add tuple typing for request_timeout

* imports

* [api_requestor] Log request_id with response (openai#145)

* Only import wandb as needed (openai#146)

Co-authored-by: Felipe Petroski Such <felipe@openai.com>
Co-authored-by: Henrique Oliveira Pinto <hponde@openai.com>
Co-authored-by: Rachel Lim <rachel@openai.com>
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.

4 participants