screenshot-to-code icon indicating copy to clipboard operation
screenshot-to-code copied to clipboard

Add AZURE support

Open cristianorevil opened this issue 1 year ago • 10 comments

This PR add the support for Azure OpenAI Services (Vision & DALLE-3)

Some considerations:

  • Azure parameters are managed only in .env file (not in web interface), I have updated the README file with instructions
  • On Azure the vision ad dalle-3 distributions must be in the same reource and same region
  • If in .env are present OPENAI and AZURE settings, OPENAI has the priority
  • The code probably need some refactor (i am not an high level python developer)

My tests with AZURE are ok, please test also OPENAI

cristianorevil avatar Dec 14 '23 11:12 cristianorevil

Apply Sweep Rules to your PR?

  • [ ] Apply: All new business logic should have corresponding unit tests.
  • [ ] Apply: Refactor large functions to be more modular.
  • [ ] Apply: Add docstrings to all functions and file headers.

I just need it, thanks for the contribution

aboutmydreams avatar Dec 22 '23 11:12 aboutmydreams

which works better, Azure or OpenAI?

wjurkowlaniec avatar Dec 23 '23 16:12 wjurkowlaniec

They are the same model. Should work the same.

abi avatar Dec 23 '23 17:12 abi

which works better, Azure or OpenAI?

you can upload customized data and set api rate limit in Azure.

aboutmydreams avatar Dec 25 '23 01:12 aboutmydreams

Great! I just want to know when will this PR be merged...

thuzhf avatar Jan 09 '24 09:01 thuzhf

Sorry for the slowness (catching up after the holidays). Should be merged some time next week.

abi avatar Jan 09 '24 16:01 abi

Sorry for the slowness (catching up after the holidays). Should be merged some time next week.

@abi Will this be merged in this week? Waiting for using it...

thuzhf avatar Jan 19 '24 06:01 thuzhf

@thuzhf Azure GPT 4 Vision has low token limits per minute so this hasn't been a priority for me. Have you tried cloning cristianorevil:azure-support and using it directly? You don't need to wait for this to be merged in to use it.

abi avatar Jan 19 '24 17:01 abi

Since stream requests are already supported and the issue with token limitations has been somewhat improved, can we support Azure now?

cobolbaby avatar Jul 03 '24 06:07 cobolbaby