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

Azure OpenAI integration, SSL Verification on/off toggle, and retry scheduling update #112

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sungdukyu
Copy link

This PR includes updates across multiple files to add Azure OpenAI support. I have tested the changes with both launch_scientist.py and individual command-line calls for generate_ideas.py, perform_writeup.py, and iclr_analysis.py. Aider integration has been verified, and everything appears to be functioning correctly.

Major Updates:

  • Azure OpenAI Service Integration: Added support for OpenAI models via Azure (currently only gpt4o models). Other Azure-supported models can be added down the road.
  • SSL Certificate Verification: Introduced the --no-verify-ssl argument to disable SSL certificate verification for Azure OpenAI models. While this is currently implemented only for Azure, it can be extended to other APIs (OpenAI and Anthropic) since they all use httpx clients.

Minor Updates:

  • Retry Scheduling: Adjusted retry strategy for Semantic Scholar API queries (search_for_paper in generate_ideas.py). Switched from exponential backoff to constant with jitter for improved efficiency, avoiding excessive hang time.

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.

1 participant