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

Make sure pipenv-based development workflow works #116

Closed
lukasheinrich opened this issue Apr 14, 2018 · 7 comments
Closed

Make sure pipenv-based development workflow works #116

lukasheinrich opened this issue Apr 14, 2018 · 7 comments
Assignees

Comments

@lukasheinrich
Copy link
Contributor

Description

We support a number of extras like the various tensorbackends, but as far as I know none of them are mutually exclusive and all of them are needed for running the unit tests.

We should therefore make sure pip install -e[develop] installs all necessary packages to run the tests. I think also pytest-benchmark is missing (@kratsg mentioned that)

@matthewfeickert matthewfeickert self-assigned this Apr 14, 2018
@lukasheinrich
Copy link
Contributor Author

hm -- maybe this is an issue with my local installation since that's exactly what we do in travis.

@matthewfeickert
Copy link
Member

Hm, this seems odd as pytest-benchmark[histogram] is in the developlist. I agree that we should make sure that everything is installed though with it.

@matthewfeickert
Copy link
Member

matthewfeickert commented Apr 14, 2018

We can also see from Travis Build 366 that the benchmarks are getting run when a PR is merged into master, so it is being setup correctly by pip install -e .[develop] in the CI.

@lukasheinrich
Copy link
Contributor Author

@matthewfeickert I think the issue might be that we manually upgrade pytest in the travis file. If we need a specific version probably pip install should pick up the right one

concretely what happens for me is that with pipenv

pipenv install -e[develop]

it still picks up a pytest outside of the pipenv, and that pytest wil not be able to import dependencies installed in the virtualenv

@lukasheinrich lukasheinrich changed the title Make sure tests run when pyhf is installed with pip install -e .[develop] Make sure pipenv-based development workflow works Apr 14, 2018
@matthewfeickert
Copy link
Member

Ah okay. Sorry, I haven't been using pipenv so I haven't noticed this. Let me setup a new environment with it and see what is the minimum package versions to make things work.

@lukasheinrich
Copy link
Contributor Author

closing this can't even reproduce this locally :-p

@matthewfeickert
Copy link
Member

Ah okay, so all is working well then?

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