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

Expose some internal options as configuration options #627

Open
david-zwicker opened this issue Nov 28, 2024 · 1 comment
Open

Expose some internal options as configuration options #627

david-zwicker opened this issue Nov 28, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@david-zwicker
Copy link
Member

We currently support some special options for differential operators, including conservative versions for spherical grids and extra checks for vectorial operators. These options are currently controlled via additional arguments to the factory functions, which is difficult to expose to the user in general settings. It might be much better to expose the settings globally via the configuration mechanism. This approach is generalizable for other options, but it has the slight disadvantage that we loose fine-grained control, e.g., it will then be impossible to choose different settings for different use-cases of the operators – one way to circumvent this (and to also keep backward compatibility) would be to change the default value of the options to None, indicating that they should be read from the configuration.

@david-zwicker david-zwicker added the enhancement New feature or request label Nov 28, 2024
@david-zwicker
Copy link
Member Author

Similar ideas would also allow us to support different stencils, e.g., a 9-point stencil for 2d Cartesian coordinates, which would then only be enabled in this very specific case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant