-
Notifications
You must be signed in to change notification settings - Fork 72
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
redeclare examples as integration tests #87
Conversation
c081833
to
3a48ff6
Compare
dfe5eb7
to
4acdeae
Compare
I have additionally added a vector addition single file example without additional includes using a single accelerator and without time measurement. |
Could you please split this pull request one for move examples to integration tests and one for the first easy example. It is easier to review. |
{ | ||
// Calculate the number of elements to compute in this thread. | ||
// The result is uniform for all but the last thread. | ||
auto const elems(threadElemExtent + alpaka::math::min(acc, static_cast<unsigned long long>(0u), static_cast<unsigned long long>(numElements-(threadFirstElemIdx+threadElemExtent)))); |
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.
independent of the point that we should move this in a separate pull request:
Might be we should create a alpaka trait/function for this line.
4acdeae
to
5fb7e17
Compare
The request has been split. |
5fb7e17
to
ce9ea1d
Compare
eval "${2}" | ||
fi | ||
|
||
cd "${oldPath}" |
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.
missing end of file newline
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.
Fixed.
after the missing EOF is added I can merge this pull request |
ce9ea1d
to
6140cc8
Compare
redeclare examples as integration tests
As has been noted in #77 the current examples are not really easy examples but represent more or less integration tests. Therefore they have been moved to
test/integ
.