Skip to content
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

Move Sort Binning Operators into Kokkos namespace #421

Closed
crtrott opened this issue Sep 6, 2016 · 0 comments
Closed

Move Sort Binning Operators into Kokkos namespace #421

crtrott opened this issue Sep 6, 2016 · 0 comments
Assignees
@crtrott
Labels
Enhancement Improve existing capability; will potentially require voting
Milestone
Fall 2016

Comments

@crtrott
Copy link
Member

crtrott commented Sep 6, 2016

They are for no apparent reason in the impl namespace right now.
@crtrott crtrott added the Enhancement Improve existing capability; will potentially require voting label Sep 6, 2016
@crtrott crtrott added this to the Fall 2016 milestone Sep 6, 2016
@crtrott crtrott self-assigned this Sep 6, 2016
@crtrott crtrott changed the title Move Binning Operators into Kokkos namespace Move Sort Binning Operators into Kokkos namespace Sep 21, 2016
crtrott added a commit that referenced this issue Oct 3, 2016
@crtrott
Algorithms: moving BinOp into Kokkos namespace
5dd8156 
This addresses #421. Also renamed the BinOps from
DefaultBinOp1D and DefaultBinOp3D to BinOp1D and BinOp3D.

This breaks backward compatibility, but I don't think many people
use the sorting yet.
@crtrott crtrott closed this as completed Oct 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@crtrott crtrott

Labels
Enhancement Improve existing capability; will potentially require voting
Projects
None yet
Milestone
Fall 2016
Development

No branches or pull requests
1 participant
@crtrott