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

Unable to run moderator on aiohttp 3.8.3 #197

Closed
likunshu opened this issue Nov 1, 2022 · 1 comment
Closed

Unable to run moderator on aiohttp 3.8.3 #197

likunshu opened this issue Nov 1, 2022 · 1 comment

Comments

@likunshu
Copy link

likunshu commented Nov 1, 2022

Running the example moderator with aiohttp=3.8.3 rise a error when POST:

ERROR:aiohttp.server:Error handling request
Traceback (most recent call last):
  File "/Users/kunshu/opt/anaconda3/envs/aio38/lib/python3.8/site-packages/aiohttp/web_protocol.py", line 433, in _handle_request
    resp = await request_handler(request)
  File "/Users/kunshu/opt/anaconda3/envs/aio38/lib/python3.8/site-packages/aiohttp/web_app.py", line 504, in _handle
    resp = await handler(request)
  File "/Users/Kunshu/Work/aiohttp-demos/demos/moderator/moderator/handlers.py", line 26, in moderate
    results = await run(self._executor, predict_probability, features)
RuntimeError: Task <Task pending name='Task-8' coro=<RequestHandler._handle_request() running at /Users/kunshu/opt/anaconda3/envs/aio38/lib/python3.8/site-packages/aiohttp/web_protocol.py:433> cb=[<TaskWakeupMethWrapper object at 0x103d47ca0>()]> got Future <Future pending cb=[_chain_future.<locals>._call_check_cancel() at /Users/kunshu/opt/anaconda3/envs/aio38/lib/python3.8/asyncio/futures.py:360]> attached to a different loop

While I tried to use aiohttp==3.7.4, it worked without the error.

By the way, the model example used rises a ModuleNotFoundError: No module named 'sklearn.linear_model.logistic' error with sciki-learn==1.1.2. Downgrade to scikit-learn==0.23.2 solves the problem.

@Dreamsorcerer
Copy link
Member

Fixing the tests in #196 (if possible).
Issue with newer sklearn is in #188.

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

No branches or pull requests

2 participants