-
Notifications
You must be signed in to change notification settings - Fork 2
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
#26: Zoltan2: MatrixAdapter refactor API #49
#26: Zoltan2: MatrixAdapter refactor API #49
Conversation
This PR is almost ready for review; I just want to run it using the |
Since the |
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.
looks good - please fix the new lines issue
packages/zoltan2/core/src/input/Zoltan2_TpetraCrsMatrixAdapter.hpp
Outdated
Show resolved
Hide resolved
packages/zoltan2/core/src/input/Zoltan2_XpetraCrsMatrixAdapter.hpp
Outdated
Show resolved
Hide resolved
I believe the changes will break NGA-develop branch. Please rebase with recent changes to NGA-develop and fix compilation errors |
See e066b2f for reference on how to deal with Weights getters (we want to support both 2D and 1D versions - #14 (comment)) |
a4dc732
to
1429716
Compare
1429716
to
adbbb49
Compare
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.
It passes local tests with the new API. Thanks @cwschilly!
Fixes #26 and adds TpetraCrsMatrixAdapter