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

Mismatch between PyPi and conda-forge versions #105

Closed
valeriupredoi opened this issue Nov 7, 2022 · 7 comments
Closed

Mismatch between PyPi and conda-forge versions #105

valeriupredoi opened this issue Nov 7, 2022 · 7 comments

Comments

@valeriupredoi
Copy link

Hey guys, am sure you are aware of this, hence I may sound like Dr Obvious, but heads up xgboost is at 1.7 on c-f and 1.7.1 on pypi, this is tripping Readthedocs builds for us - many thanks for your good work! 🍺

@hcho3
Copy link
Contributor

hcho3 commented Nov 7, 2022

Can you fix your build so that the mismatch doesn't break it? Typically there is some delay between PyPI and Conda releases.

@jakirkham
Copy link
Member

Would also add anyone is welcome to submit a PR to update the Conda package

@valeriupredoi
Copy link
Author

@hcho3 we have now, but it wasn't too straightforward since I had to dig out some new feature in the Readthedocs build configuration that averts the default pip install in eager mode - the root cause of the issue - RTD's pip installation was updating packages even if there was no need to do that; if you guys run into this issue too, use jobs in the RTD's config eg https://github.com/ESMValGroup/ESMValTool/pull/2913/files - @jakirkham absolutely, mate, I was planning on doing a PR here if you guys didn't beat me to it 😁 At any rate, cheers for your good work here, and apols if I came across annoying 😃

@jakirkham
Copy link
Member

Gotcha maybe it is worth raising an issue with RTD about adding --no-deps or similar to pip. This would allow bypassing the pip install of the dependencies (especially when this was already done with Conda). Just a thought 🙂

@valeriupredoi
Copy link
Author

Absolutely! Afraid there is one readthedocs/readthedocs.org#8890 - alas, closed - they invoke some constraints they can't work against to do that. At least now there is configuration magic, to avoid it 🪄

@jakirkham
Copy link
Member

That seems somewhat different since that is discussing how pip solves as opposed to simply skipping dependencies. So would suggest a new issue.

Think it may help also if you share your use case. Namely installing with Conda and then pip installing the source. They may be more open to discussion when realizing two different package managers are trying to do the same thing and clobbering each other.

@valeriupredoi
Copy link
Author

@jakirkham I totally took you up on that idea, and opened this one readthedocs/readthedocs.org#9727 - cheers for suggesting, and thanks for your comments and suggestions here, mate 🍺

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

3 participants