billiard icon indicating copy to clipboard operation
billiard copied to clipboard

NameError on Pool.shrink()

Open mschwoer opened this issue 2 years ago • 4 comments

On calling Pool.shrink() I get a NameError: name 'values' is not defined (version: 4.0.2).

Seems like values() has been introduced quite a while ago (https://github.com/celery/billiard/blob/88ff851b9f97fdba497e6ad7062f76265795f1af/billiard/pool.py), but the import from .five has been removed down the line.

System

  • python 3.8 on MacOS 11.6.8

Steps to reproduce

conda create --name myenv python=3.8
conda activate myenv
pip install billiard==4.0.2
python

in the REPL, type

import billiard
p = billiard.Pool()
p.shrink(1)

mschwoer avatar Aug 12 '22 12:08 mschwoer

https://github.com/celery/billiard/blob/88ff851b9f97fdba497e6ad7062f76265795f1af/billiard/pool.py#L1285 this line right?

auvipy avatar Aug 14 '22 07:08 auvipy

yes I can reproduce it

auvipy avatar Aug 14 '22 07:08 auvipy

https://github.com/celery/billiard/blob/88ff851b9f97fdba497e6ad7062f76265795f1af/billiard/pool.py#L1310

auvipy avatar Aug 14 '22 07:08 auvipy

related https://github.com/celery/billiard/pull/374 please try the pr and let me know

auvipy avatar Aug 14 '22 07:08 auvipy

LGTM

mschwoer avatar Aug 16 '22 07:08 mschwoer