Skip to content

Conversation

@sorinsuciu-msft
Copy link
Contributor

No description provided.

@ChrisHMSFT
Copy link

@kennyhsu5 saw this was approved. One quick question -- we don't have merge rights. are you able to merge or do we need another approval? thanks!

@kennyhsu5
Copy link
Collaborator

@kennyhsu5 saw this was approved. One quick question -- we don't have merge rights. are you able to merge or do we need another approval? thanks!

I think I can merge it. Just trying to get more clarity on one final comment/issue first.

@kennyhsu5
Copy link
Collaborator

@sorinsuciu-msft Can you double check the unit tests are passing with this PR?

I'm seeing test_file_upload and test_engine_search_url_composition_azure_no_operation failures when trying to incorporate this PR into our CI.

Reverting it here since CI seem to be passing fine without this change.

cgayapr pushed a commit to cgayapr/openai-python that referenced this pull request Dec 14, 2024
* Add support for fine-tunning and files using the Azure API. * Small changes + version bumps * Version bump after merge * fix typo * adressed comments
cgayapr pushed a commit to cgayapr/openai-python that referenced this pull request Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants