forked from conda-forge/xgboost-feedstock
-
Notifications
You must be signed in to change notification settings - Fork 3
Merge conda-forge/main
into rapidsai/main
(pt. 13)
#65
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
jakirkham
merged 136 commits into
rapidsai:main
from
jakirkham-feedstocks:merge_upstream_13
Jul 26, 2024
Merged
Merge conda-forge/main
into rapidsai/main
(pt. 13)
#65
jakirkham
merged 136 commits into
rapidsai:main
from
jakirkham-feedstocks:merge_upstream_13
Jul 26, 2024
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
jakirkham
commented
Jul 26, 2024
- Upgrade to XGBoost 2.1.0 conda-forge/xgboost-feedstock#167
…nda-forge-pinning 2024.06.28.19.57.36
…nda-forge-pinning 2024.06.28.19.57.36
automerged PR by conda-forge/automerge-action
This directory doesn't exist in XGBoost 2.1.0.
Co-authored-by: jakirkham <[email protected]>
…nda-forge-pinning 2024.06.29.20.13.13
…orge-pinning 2024.06.29.20.13.13
…nda-forge-pinning 2024.06.29.20.13.13
…nda-forge-pinning 2024.06.29.20.13.13
These precede the `libmamba` solver becoming the default in Conda. Very likely they are unneeded now (and may be causing issues). So drop them to see if it helps. Can always readd one-by-one if needed.
…nda-forge-pinning 2024.07.10.00.49.07
automerged PR by conda-forge/automerge-action
…nda-forge-pinning 2024.07.14.16.05.21
…nda-forge-pinning 2024.07.16.14.39.10
In some cases `pin_subpackage` was only added to `requirements/run`. In others it may be in `requirements/host` too. This consistently adds it to both. May help build tools recognize the dependency order when starting builds (and thus more consistently create working test environments).
Co-authored-by: jakirkham <[email protected]>
…nda-forge-pinning 2024.07.24.15.58.55
…nda-forge-pinning 2024.07.24.15.58.55
…nda-forge-pinning 2024.07.24.15.58.55
Co-authored-by: jakirkham <[email protected]>
Upgrade to XGBoost 2.1.0
The linter does not like having `noarch: python` at the top-level with selectors. Rewriting the recipe without selectors is a substantial lift and not worth it given the move towards rattler-build. However if we don't have `noarch` or a `skip` at the top-level, a full matrix of Python versions is built out for `xgboost` (despite these packages being redundant). To clean this up, we skip any Python version except the minimum one that XGBoost supports. This ensure we build for exactly one Python. To get the syntax right we need to use Jinja. Unfortunately that means we need to consolidate our other `skip`s into this one, which includes the Windows CUDA 11.8 skip. So we do so and try to make the syntax as pleasing as we can within the constraints.
…nda-forge-pinning 2024.07.25.23.42.20
automerged PR by conda-forge/automerge-action
hcho3
approved these changes
Jul 26, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.