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

rename KERNEL_* fields in inparam_advanced #41

Open
martinvandriel opened this issue Nov 23, 2015 · 1 comment
Open

rename KERNEL_* fields in inparam_advanced #41

martinvandriel opened this issue Nov 23, 2015 · 1 comment

Comments

@martinvandriel
Copy link
Contributor

Given that most AxiSEM users outside the developers use AxiSEM to compute Instaseis databases and have not heard about kernels, we might want to rename these fields. What about INSTASEIS_ * instead, as all Kernel users know Instaseis, but not the other way around?

@tnissen
Copy link

tnissen commented Nov 24, 2015

Agreed, but we should keep in mind that over short or long the kernel
capability will be added too, and we should avoid yet another renaming
then. Maybe something generic, such as WAVEFIELDS_* ?

On 11/23/15 13:04, Martin van Driel wrote:

Given that most AxiSEM users outside the developers use AxiSEM to
compute Instaseis databases and have not heard about kernels, we might
want to rename these fields. What about INSTASEIS_ * instead, as all
Kernel users know Instaseis, but not the other way around?


Reply to this email directly or view it on GitHub
#41.

Tarje

<>--<>--<>--<>--<>--<>
Dept. of Earth Sciences
Oxford University
South Parks Road
Oxford OX1 3AN; UK
tel: +44 1865 282149
web: seis.earth.ox.ac.uk
<>--<>--<>--<>--<>--<>

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