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.

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

subview(X, ALL(), j) for 2-D LayoutRight View X: should it view a column? #5

Closed
mhoemmen opened this issue Apr 19, 2015 · 6 comments
Closed

Comments

@mhoemmen
Copy link
Contributor

I think it's actually viewing a row, which I find counterintuitive. Should it view a column instead? I'm only having this trouble for LayoutRight views.

@crtrott
Copy link
Member

crtrott commented Apr 20, 2015

Uhm it views a column as far as I see. But it does not return layout right, but LayoutStride.

@mhoemmen
Copy link
Contributor Author

Cool, just checking that this is what it's supposed to do :-) I'll add a test for this to the TpetraKernels multivector test. (The issue is that for our BLAS1 multivector kernels, the 1-D View version gives the right answer for LayoutLeft, but not for LayoutRight. So, it's not an urgent issue, but if we ever want LayoutRight multivectors in Tpetra, we'll need to figure it out.)

@crtrott
Copy link
Member

crtrott commented Apr 20, 2015

Ok there is/was a serious issue with LayoutStride. I think I fixed them and will push that in a minute. Sync to Trilinos will take a bit since I have to rebuild/run all of it.

@crtrott
Copy link
Member

crtrott commented Apr 21, 2015

Ok after pushing to Kokkos and in the middle of the Trilinos test/push cycle my machine died a horrible death (i/o bus error). Actually its not quite dead, but it doesn't let me reboot from the commandline (bus error). So the push to Trilinos has to wait till tomorrow I guess.

@crtrott
Copy link
Member

crtrott commented Apr 21, 2015

This thing is pushed and should be resolved. I close the issue, please reopen if you still have problems.

@crtrott crtrott closed this as completed Apr 21, 2015
@mhoemmen
Copy link
Contributor Author

Thanks Christian! :-D

crtrott added a commit that referenced this issue Jan 27, 2020
added CMake support for OpenMP target, CMake cleanup
masterleinad referenced this issue in masterleinad/kokkos Sep 23, 2020
Expanded on the isTriviallyCopyable checks
dalg24 added a commit that referenced this issue Aug 21, 2022
…cating_view

Check labels for container types for resize and realloc
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

No branches or pull requests

2 participants