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

Test petclaw with Python kernel and multiple processes #16

Closed
ketch opened this issue May 26, 2011 · 2 comments
Closed

Test petclaw with Python kernel and multiple processes #16

ketch opened this issue May 26, 2011 · 2 comments
Assignees

Comments

@ketch
Copy link
Member

ketch commented May 26, 2011

I don't think it will work as coded right now. It may not be worth supporting this combination anyway, but if not we should check for it and throw a helpful exception.

@ghost ghost assigned amal-ghamdi May 26, 2011
@ahmadia
Copy link
Member

ahmadia commented Jun 19, 2011

I'm surprised that this doesn't work. Where are we breaking our orthogonality in assumptions between the PETSc interactions and Fortran kernels? I think if we are not willing to fully support the Python kernels we should remove them.

ketch added a commit that referenced this issue Jun 21, 2011
so that they work in parallel.
Also change 'F' to order='F' in allocation of 1D numpy arrays.
@ketch
Copy link
Member Author

ketch commented Jun 21, 2011

The issue was that grid.n was being used to determine loop limits. Now grid.n is the size of the whole grid, while grid.ng is the size of the processor's global vector. It is fixed.

It is quite possible that there will be features in PyClaw that we're not willing to support in PetClaw, though I would like to avoid that. PyClaw has a design goal of being used for teaching and rapid prototyping, which makes the Python kernels important.

@ketch ketch closed this as completed Jun 21, 2011
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

3 participants