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

Add a more convenient way to set aux arrays #15

Closed
ketch opened this issue May 23, 2011 · 0 comments
Closed

Add a more convenient way to set aux arrays #15

ketch opened this issue May 23, 2011 · 0 comments

Comments

@ketch
Copy link
Member

ketch commented May 23, 2011

Setting the aux array is somewhat different from setting q because boundary values must also be set on initialization. Right now the user is required to create a numpy array of the appropriate size and fill it in. Aron suggested providing a simpler interface that would only require the user to provide a function setaux(x, y, dx, dy) that takes arrays of cell centers and dimensions and returns an array of aux values for the cells. Then the grid would have a method that applies this function automatically using the coordinates of the grid. It was agreed that this would be helpful and should be supported, but without removing support for the current approach. The grid could also have a method to impose common "boundary conditions" to generate the ghost cell aux values (zero extrapolation and periodic).

@ketch ketch closed this as completed Jul 26, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant