Skip to content

Skip tests #120

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 48 commits into from
Dec 17, 2020
Merged

Skip tests #120

merged 48 commits into from
Dec 17, 2020

Conversation

reazulhoque
Copy link
Contributor

This PR skips certain tests on some architecture.

@reazulhoque
Sum example
f5ea456
@reazulhoque
Moved from infer_type, lower_builtin to overload
1d346cd
@reazulhoque
Added two level module name functions
3dd9f24
@reazulhoque
Remove cython generated file
0003586
@reazulhoque
Module name fix for moving to new extension
1f85af9
@reazulhoque
Incomplete linalg.eig implementation
a41b9a2
@reazulhoque
Merge branch 'main' into feature/rewrite_pass_to_rename_functions
0c9dfcb
@reazulhoque
Updated dppl to dppy
4cd5c82
@reazulhoque
Updted all dppl to dppy and moved rewrite_numpy_function_pass to it's…
55ff896 
… own file
@reazulhoque
Import module at correct locations
687a52a
@reazulhoque
Added comments
0fc597a
@reazulhoque
Added test and updated comments
35c22ef
@reazulhoque
Revert unneeded changes
ab646b6
@reazulhoque
Update Eigen implementation
10b90f1
@reazulhoque
Separate the implementations into their own category files
a10d90d
@reazulhoque
merge main
d5ba664
@reazulhoque
Fix float32 precision
aa96e28
@reazulhoque
Added test for eigen
67663d7
@reazulhoque
Added typed pass to rename ndarray.function_name
d6eae93
@reazulhoque
Added test for ndarray.sum
66974a1
@reazulhoque
Merge main
6165572
@reazulhoque
Started inital port
1b78336
@reazulhoque
Add min, amin, ndarray.min
fb03df8
@reazulhoque
mean WIP
8fced17
@reazulhoque
Add median
13e7574
@reazulhoque
Add argsort, argmax, argmin
7ffb550
@reazulhoque
Added cov
59f95b8
@reazulhoque
Added dot
e43cf17
@reazulhoque
Addd matmul
f5b76b7
@reazulhoque
Merge main
742932e
@reazulhoque
Copy link
Contributor Author

Tested with the following dependencies locally:

dpctl: d157ee0b1a9ba88d0f3b226bd069af2dd9a56b9a
Numba (patched): 864790061aa7548ac2b7ebcf448354def849e657
dpnp: 0.4.0
Result:

----------------------------------------------------------------------
Ran 157 tests in 71.137s
OK (skipped=2, expected failures=1)

@PokhodenkoSA PokhodenkoSA mentioned this pull request Dec 16, 2020
Closed
PokhodenkoSA
PokhodenkoSA reviewed Dec 16, 2020
View reviewed changes
PokhodenkoSA
PokhodenkoSA reviewed Dec 16, 2020
View reviewed changes
PokhodenkoSA
PokhodenkoSA reviewed Dec 16, 2020
View reviewed changes
PokhodenkoSA
PokhodenkoSA approved these changes Dec 16, 2020
View reviewed changes
Copy link
Contributor

@PokhodenkoSA PokhodenkoSA left a comment

Choose a reason for hiding this comment

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

Function could be named better. And one duplication. After fixing this PR could be merged.

@PokhodenkoSA PokhodenkoSA merged commit c0a17bc into IntelPython:main Dec 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@diptorupd diptorupd Awaiting requested review from diptorupd

1 more reviewer

@PokhodenkoSA PokhodenkoSA PokhodenkoSA approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@PokhodenkoSA PokhodenkoSA

Labels

3 - Waiting: author 4 - Ready to merge

Projects

None yet

Milestone

MS141 (0.12.0)

Development

Successfully merging this pull request may close these issues.

2 participants

@reazulhoque @PokhodenkoSA