Skip to content

requirements.txt doesn't contain test dependencies #185

Closed

Description

@amueller

we need jupyter_client and maybe nbconvert I think.

Activity

  1. mfeurer commented on Nov 11, 2016

    @mfeurer
    Collaborator

    Shouldn't these rather be added as test dependencies to setup.py?

  2. amueller commented on Nov 11, 2016

    @amueller
    ContributorAuthor

    yeah probably. I think the current setup is a bit strange. Maybe get rid of requirements.txt and just declare everything in setup.py?

  3. mfeurer commented on Nov 11, 2016

    @mfeurer
    Collaborator

    requirements.txt is actually parsed by setup.py, so there is no duplication. Is this a weird setup? You've got more experience on how to install python packages properly, I'm usually happy when it somehow works ;) But I'm not opposed to learn how to do it better.

  4. amueller commented on Nov 23, 2016

    @amueller
    ContributorAuthor

    I think we don't need a requirements.txt and should encode everything in the setup.py.

  5. mfeurer commented on Nov 24, 2016

    @mfeurer
    Collaborator

    From my experience, this would require the user to first install the dependencies of the dependencies prior to being able to install the package. Is that correct?

  6. amueller commented on Nov 28, 2016

    @amueller
    ContributorAuthor

    no. the existence of a requirements.txt doesn't change anything for dependency resolution.

  7. mfeurer commented on Nov 28, 2016

    @mfeurer
    Collaborator

    True, but since a requirements.txt is installed via pip, it installs one package after the other. If the order is correct, this works out nicely. I wouldn't know how to do this with a setup.py.

  8. amueller commented on Nov 28, 2016

    @amueller
    ContributorAuthor

    pip entirely ignores requirements.txt, unless you do pip -r requirements.txt after you downloaded the package. If you do pip install openml it will not have any influence.

  9. mfeurer commented on Nov 28, 2016

    @mfeurer
    Collaborator

    Sorry for being too unspecific, that's what I actually meant. Does that somehow work with only a setup.py as well?

  10. added this to the pip-release milestone on Mar 19, 2018
  11. self-assigned this
    on Apr 3, 2018
  12. ArlindKadra commented on Apr 8, 2018

    @ArlindKadra
    Member

    There is a lot of controversy regarding when to use requirements.txt and install_requires in setup.py. From the python and pip docs they suggest to keep them separate, use concrete dependencies in the requirements.txt and abstract ones in the setup.py. It is suggested not to mix them up together as a good programming practice.
    Also from the pip and python documentation, I found this info regarding the requirements file:
    "Note that you should not rely on the items in the file being installed by pip in any particular order.".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions