Azure-Samples / azure-search-openai-demo

A sample app for the Retrieval-Augmented Generation pattern running in Azure, using Azure AI Search for retrieval and Azure OpenAI large language models to power ChatGPT-style and Q&A experiences.
https://azure.microsoft.com/products/search
MIT License
6.02k stars 4.12k forks source link

Incompatible issue for latest Python version 3.12 #1169

Open singloudly90 opened 9 months ago

singloudly90 commented 9 months ago

Please provide us with the following information:

This issue is for a: (mark with an x)


- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [x] regression (a behavior that used to work and stopped in a new release)
```Understand requirement txt is written:# This file is autogenerated by pip-compile with Python 3.11
My enterprise latest python version is 3.12,  does this repos intend to update with latest python version?
thank you very much.

### Minimal steps to reproduce
>Downgrade python to 3.11 from 3.12

### Any log messages given by the failure
>check.warn(importable)
      copying multidict\__init__.pyi -> build\lib.win-amd64-cpython-312\multidict
      copying multidict\py.typed -> build\lib.win-amd64-cpython-312\multidict
      running build_ext
      building 'multidict._multidict' extension
      error: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for multidict
Failed to build frozenlist multidict
ERROR: Could not build wheels for frozenlist, multidict, which is required to install pyproject.toml-based projects

### Expected/desired behavior
>Process successful

### OS and Version?
> Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

### azd version?
> run `azd version` and copy paste here.

### Versions
>3.12

### Mention any other details that might be useful

> ---------------------------------------------------------------
> Thanks! We'll be in touch soon.
mattgotteiner commented 9 months ago

Hello @singloudly90 ,

Please track #752

Thanks, Matt

pamelafox commented 8 months ago

Dug into this with @tonybaloney, the issue is that aiohttp module requires multidict which does not have a 3.12 wheel yet, and your computer lacks a compiler for making that wheel.

We can either hope that multidict releases a 3.12 wheel, or we can move our aiohttp calls to httpx.