-
-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Add kernel-tuner and python-constraint2 #24420
Changes from 6 commits
4d35086
220b8d2
56e6f43
580d294
062b828
a24c0ab
52be0d1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,76 @@ | ||
{% set name = "kernel-tuner" %} | ||
{% set version = "1.0.0b6" %} | ||
|
||
package: | ||
name: {{ name|lower }} | ||
version: {{ version }} | ||
|
||
source: | ||
url: https://pypi.io/packages/source/{{ name[0] }}/kernel_tuner/kernel_tuner-{{ version }}.tar.gz | ||
sha256: 8ec7cac41f1eec84e54bf2452008c905fad90279949e477355fb9eda054b1beb | ||
|
||
build: | ||
script: {{ PYTHON }} -m pip install . -vv --no-deps --no-build-isolation | ||
number: 0 | ||
skip: True # [win or py>311] | ||
|
||
requirements: | ||
host: | ||
- pip | ||
- poetry-core >=1.7.0 | ||
- python | ||
- setuptools >=67.7.2 | ||
run: | ||
- jsonschema | ||
- numpy >=1.22.2,<2 | ||
- python-constraint2 >=2.0.0b3,<3 | ||
- pandas >=2.0.0,<3 | ||
- python | ||
- scikit-learn >=1.0.2,<2 | ||
- scipy >=1.10.1,<2 | ||
- xmltodict | ||
|
||
test: | ||
commands: | ||
- pip check | ||
- pytest | ||
imports: | ||
- kernel_tuner | ||
requires: | ||
- mock | ||
- pep440 | ||
- pip | ||
- pytest >=7.4 | ||
- pytest-cov | ||
- tomli | ||
source_files: | ||
- pyproject.toml | ||
- test | ||
- README.rst | ||
|
||
about: | ||
home: https://github.com/kerneltuner/kernel_tuner | ||
summary: 'An easy to use CUDA/OpenCL kernel tuner in Python' | ||
description: | | ||
Kernel Tuner simplifies the software development of optimized and | ||
auto-tuned GPU programs, by enabling Python-based unit testing of GPU code | ||
and making it easy to develop scripts for auto-tuning GPU kernels. This | ||
also means no extensive changes and no new dependencies are required in the | ||
kernel code. The kernels can still be compiled and used as normal from any | ||
host programming language. | ||
|
||
Kernel Tuner provides a comprehensive solution for auto-tuning GPU | ||
programs, supporting auto-tuning of user-defined parameters in both host | ||
and device code, supporting output verification of all benchmarked kernels | ||
during tuning, as well as many optimization strategies to speed up the | ||
tuning process. | ||
license: Apache-2.0 | ||
license_file: LICENSE | ||
doc_url: https://kerneltuner.github.io | ||
|
||
extra: | ||
recipe-maintainers: | ||
- benvanwerkhoven | ||
- bouweandela | ||
- fjwillemsen | ||
- isazi |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,56 @@ | ||
{% set name = "python-constraint2" %} | ||
{% set version = "2.0.0b4" %} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We prefer to have releases on conda-forge, can you please submit one of these? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure if I understand the comment, my apologies. Do you mean a release that is not a beta version? If the issue is indeed that it is a beta release, the reason for doing it like this is the following: version 0.x of kernel tuner does not include the source distribution on PyPI, so that is I why I opted for building the conda-forge package for version 1.x. However, that requires |
||
|
||
package: | ||
name: {{ name|lower }} | ||
version: {{ version }} | ||
|
||
source: | ||
url: https://pypi.io/packages/source/{{ name[0] }}/{{ name }}/python_constraint2-{{ version }}.tar.gz | ||
sha256: 2c8dd055f63b91ccd93d3145411648030a705306d18a78c3e8f739109c2e889d | ||
|
||
build: | ||
script: {{ PYTHON }} -m pip install . -vv --no-deps --no-build-isolation | ||
number: 0 | ||
skip: True # [py>311] | ||
|
||
requirements: | ||
build: | ||
- {{ compiler('c') }} | ||
host: | ||
- cython >=3.0.2 | ||
- pip | ||
- poetry-core >=1.7.0 | ||
- python | ||
- setuptools >=67.7.2 | ||
run: | ||
- python | ||
|
||
test: | ||
commands: | ||
- pip check | ||
- pytest --ignore tests/test_constraint.py --ignore tests/test_compilation.py | ||
imports: | ||
- constraint | ||
requires: | ||
- pep440 | ||
- pip | ||
- pytest | ||
- pytest-cov | ||
- tomli | ||
source_files: | ||
- pyproject.toml | ||
- tests | ||
- README.rst | ||
|
||
about: | ||
home: https://github.com/python-constraint/python-constraint | ||
summary: A module for efficiently solving CSPs (Constraint Solving Problems) over finite domains. | ||
license: BSD-2-Clause | ||
license_file: LICENSE | ||
doc_url: http://python-constraint.github.io/python-constraint | ||
|
||
extra: | ||
recipe-maintainers: | ||
- bouweandela | ||
- fjwillemsen |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like it can be noarch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried that initially but the tests fail on Windows (see 580d294 and here), so after discussing with the authors of the package in KernelTuner/kernel_tuner#128 (comment), I disabled the windows build to avoid disappointing Windows users by allowing them to install something that doesn't work. Is this the right way to do it? Or can we make the package
noarch: python
but still skip the Windows build?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can make it
noarch: python
but add arun
dependency on__unix
. That would avoid building the package for every Python/OS combination.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed this as suggested in 52be0d1, but now the windows build is failing again. Did I miss something?