Skip to content

Conversation

@mingjie-intel
Copy link
Contributor

@mingjie-intel mingjie-intel commented Nov 2, 2022

Add conda build for linux and windows. Both build instructions and workflow to upload it to anaconda servers where added.

Test upload could be found here:
https://anaconda.org/dppy/dpbench

Installation process is as simple as:

conda create -n dpbench-conda-test dpbench -c dppy/label/dev -c conda-forge -c intel -c nodefaults --override-channels
conda activate dpbench-conda-test
dpbench -a run
dpbench report

Works both on linux and windows

Copy link

@diptorupd diptorupd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed we need a conda-package workflow along with the conda recipe to verify that the recipe works. We can borrow what we are doing in dpctl or numba-dpex. The workflow should build the package, test it, and the upload to dppy/label/dev.

@adarshyoga adarshyoga removed the request for review from DenisScherbakov April 4, 2023 04:24
@adarshyoga adarshyoga assigned adarshyoga and unassigned ZzEeKkAa May 3, 2023
@adarshyoga adarshyoga marked this pull request as draft May 3, 2023 16:36
@ZzEeKkAa ZzEeKkAa assigned ZzEeKkAa and unassigned adarshyoga Jun 21, 2023
@ZzEeKkAa ZzEeKkAa force-pushed the package branch 17 times, most recently from e83dc51 to 883260d Compare June 23, 2023 23:11
@ZzEeKkAa ZzEeKkAa force-pushed the package branch 17 times, most recently from f5a868e to 61973e6 Compare June 28, 2023 18:48
@ZzEeKkAa ZzEeKkAa requested a review from diptorupd June 28, 2023 21:29
@ZzEeKkAa ZzEeKkAa dismissed diptorupd’s stale review June 28, 2023 21:30

Changes applied

@ZzEeKkAa ZzEeKkAa marked this pull request as ready for review June 28, 2023 21:30
@ZzEeKkAa ZzEeKkAa enabled auto-merge June 28, 2023 21:30
@ZzEeKkAa ZzEeKkAa merged commit 7d8f730 into main Jun 29, 2023
@ZzEeKkAa ZzEeKkAa deleted the package branch June 29, 2023 16:16
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

Successfully merging this pull request may close these issues.

4 participants