Uh oh!
There was an error while loading. Please reload this page.
Rename dppl to dppy - #42
Conversation
PokhodenkoSA
left a comment
There was a problem hiding this comment.
Please, update with main branch. It will fix CI.
1e-to
commented
Dec 4, 2020
I'll tested it with IntelPython/dpctl#201 Ran 140 tests in 28.973s dpctl 0.3.8+100.gba65df7 pypi_0 pypi |
diptorupd
left a comment
There was a problem hiding this comment.
Apart from minor changes. LGTM!
Uh oh!
There was an error while loading. Please reload this page.
| from .device_init import * | ||
| else: | ||
| raise ImportError("Importing dppl failed") | ||
| raise ImportError("Importing dppy failed") |
There was a problem hiding this comment.
numba-dppy instead of dppy. Our extension package is called numba-dppy.
@PokhodenkoSA I have merged master and made couple of changes to let numba-dppy build with dpctl/master. For CI, do we need to tag a new RC candidate for dpctl and merge it to gold/2021? |
| run: | ||
| - python | ||
| - numba >=0.51 | ||
| - cffi |
| build_requires = ["cython"] | ||
| install_requires = [ | ||
| "numba", | ||
| "cffi", |
| # from numba_dppy.tests.dppy import * | ||
PokhodenkoSA
left a comment
There was a problem hiding this comment.
It seems that branch is not up tp date with main branch.
Co-authored-by: etotmeni <elena.totmenina@intel.com> Co-authored-by: Diptorup Deb <diptorup.deb@intel.com>
Closes the issue #8
Files in which it was not possible to rename, due to connection with dpCtl:
Locally tested:
Ran 140 tests in 28.366s
OK (skipped=26, expected failures=1)