Skip to content

DEPS: update setuptools min version #42304

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 23 commits into from
Aug 3, 2021

Conversation

fangchenli
Copy link
Member

@fangchenli fangchenli commented Jun 29, 2021

According to https://setuptools.readthedocs.io/en/latest/userguide/declarative_config.html, the min version of setuptools that support entry_points in setup.cfg is 51.0.0.

@jreback jreback added this to the 1.3 milestone Jun 29, 2021
@jreback jreback added the Dependencies Required and optional dependencies label Jun 29, 2021
@jreback
Copy link
Contributor

jreback commented Jun 29, 2021

cc @simonjayhawkins

@simonjayhawkins
Copy link
Member

are we not testing with the minimum setuptools?

@fangchenli
Copy link
Member Author

are we not testing with the minimum setuptools?

No, we are not.

@simonjayhawkins
Copy link
Member

in doc/source/development/contributing_environment.rst

Creating a Python environment (pip)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

If you aren't using conda for your development environment, follow these instructions.
You'll need to have at least the :ref:`minimum Python version <install.version>` that pandas supports. If your Python version
is 3.8.0 (or later), you might need to update your ``setuptools`` to version 42.0.0 (or later)
in your development environment before installing the build dependencies::

can be updated.

@jreback
Copy link
Contributor

jreback commented Jun 29, 2021

can you update the min pin in environment.yaml (and other places if needed)?

@jreback
Copy link
Contributor

jreback commented Jun 29, 2021

also this should be added to the wheel recipe @simonjayhawkins (practically this already works, but should add)

@simonjayhawkins
Copy link
Member

test failure unrelated

E           AssertionError: Caused unexpected warning(s): [('ResourceWarning', ResourceWarning("unclosed <ssl.SSLSocket fd=8, family=AddressFamily.AF_INET, type=SocketKind.SOCK_STREAM, proto=6, laddr=('10.1.1.54', 48258), raddr=('52.217.136.144', 443)>"), '/home/runner/work/pandas/pandas/pandas/io/sas/sas7bdat.py', 381)]

@simonjayhawkins
Copy link
Member

can you update the min pin in environment.yaml (and other places if needed)?

@fangchenli can you address this

@simonjayhawkins
Copy link
Member

also this should be added to the wheel recipe @simonjayhawkins (practically this already works, but should add)

pandas-feedstock or MacPython or both?

@fangchenli
Copy link
Member Author

can you update the min pin in environment.yaml (and other places if needed)?

We don't list setuptools in environment.yaml.

@simonjayhawkins simonjayhawkins added the Blocker Blocking issue or pull request for an upcoming release label Jun 29, 2021
@simonjayhawkins
Copy link
Member

labelled as blocker as we won't be changing min versions of dependencies in patch releases?

@jreback
Copy link
Contributor

jreback commented Jun 29, 2021

yeah i think we should just add this to environment.yaml. it is technically only needed if we use matplotlib so its not a required dep (so only add it where we have matplotlib, which is most places).

@simonjayhawkins
Copy link
Member

conda-incubator/setup-miniconda@v2 does not seem to like the setuptools pin, although I think there have been conda-incubator/setup-miniconda@v2 timeouts on other PRs

@simonjayhawkins
Copy link
Member

The version in pandas-dev on CI / Checks for master (https://github.com/pandas-dev/pandas/runs/2950514145) is

2021-06-30T09:08:01.2373051Z setuptools                49.6.0           py38h578d9bd_3    conda-forge

and creating dev env locally

my backport env (1.3.x)

(pandas-backport) simon@ProDesk:~/pandas (backport-of-pr-42318-on-1.3.x)$ conda list setuptools
# packages in environment at /home/simon/miniconda3/envs/pandas-backport:
#
# Name                    Version                   Build  Channel
setuptools                49.6.0           py38h578d9bd_3    conda-forge

my master env

(pandas-dev) simon@P340:~/pandas (master)$ conda list setuptools
# packages in environment at /home/simon/miniconda3/envs/pandas-dev:
#
# Name                    Version                   Build  Channel
setuptools                49.6.0           py38h578d9bd_3    conda-forge

According to https://setuptools.readthedocs.io/en/latest/userguide/declarative_config.html, the min version of setuptools that support entry_points in setup.cfg is 51.0.0.

hmm? our tests pass locally?

@simonjayhawkins
Copy link
Member

@jreback ok to release without this PR, or delay release to allow time for others to look at this? @pandas-dev/pandas-core

@jreback
Copy link
Contributor

jreback commented Jun 30, 2021

should
include this

@simonjayhawkins
Copy link
Member

simonjayhawkins commented Jun 30, 2021

should
include this

it appears can't create a dev env with conda with this pin

@bashtage
Copy link
Contributor

Could leave it out of the yml file and then do a pip install -r requirements-dev.txt (assuming it is in requirements-dev.txt). This will no op everything that is installed, but upgrade setuptools if below.

@simonjayhawkins
Copy link
Member

should
include this

it appears can't create a dev env with conda with this pin

can create a dev env locally from scratch using this pin. so maybe something related to conda-incubator/setup-miniconda@v2

creating a dev env from scratch without this pin, the only diff is

< setuptools=49.6.0=py38h578d9bd_3
---
> setuptools=52.0.0=py38h06a4308_0

so it's a mystery to me how conda chooses the version.

@jreback
Copy link
Contributor

jreback commented Jun 30, 2021

hmm yeah this should just work.

@lithomas1 lithomas1 removed the Blocker Blocking issue or pull request for an upcoming release label Jul 5, 2021
@simonjayhawkins simonjayhawkins modified the milestones: 1.3.1, 1.3.2 Jul 24, 2021
@simonjayhawkins
Copy link
Member

moving to 1.3.2

@jreback jreback merged commit 9ed9a65 into pandas-dev:master Aug 3, 2021
@lumberbot-app

This comment has been minimized.

@jreback
Copy link
Contributor

jreback commented Aug 3, 2021

thanks @fangchenli this is essentially a doc update at this point so backporting ok.

@jreback
Copy link
Contributor

jreback commented Aug 3, 2021

@meeseeksdev backport 1.3.x

@lumberbot-app

This comment has been minimized.

simonjayhawkins pushed a commit to simonjayhawkins/pandas that referenced this pull request Aug 5, 2021
simonjayhawkins added a commit that referenced this pull request Aug 5, 2021
Co-authored-by: Fangchen Li <fangchen.li@outlook.com>
@fangchenli fangchenli deleted the fix-setuptools-min-version branch August 17, 2021 04:17
feefladder pushed a commit to feefladder/pandas that referenced this pull request Sep 7, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Dependencies Required and optional dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants