Skip to content

DEPS: update setuptools min version #42304

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

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
@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dependencies Required and optional dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants