-
Notifications
You must be signed in to change notification settings - Fork 98
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
switch problem parameters to a script #234
Comments
correction: this is not true -- all parameters are scalars. |
we should reuse
The other design consideration is whether we want this to be something separate from now we deal with extern, or if they can all be in the same module? |
It turns out that there are not array parameters, but rather, the |
I would have this script create a new module called |
as part of this, we should consider eliminating the from Weiqun: in inputs, you would do:
then to read it as a string that Fortran can deal with:
|
actually... we don't need to do anything new. We could just have the problems have a |
The automates the creation of the probdata variables, the namelist for reading them, setting them as managed for CUDA, and adds the ability to output the values to a file (like job_info) This feature is opt-in. You need to set USE_PROB_PARAMS in your GNUmakefile and then define the problem parameters in a file _prob_params in the problem directory. For problems that opt-in, the values of the problem parameters are now output into the job_info file. closes #234 closed #619
We should list the problem-specific parameters in a file like we do with the extern parameters (like
_parameters
), and then parse this at compile time to build the probin module and the namelist read mechanism called fromprobinit()
.This will be cleaner and easier to generalize to the GPUs. We should switch all problems to this system. This is similar to what Maestro currently does. We can build from the infrastructure that exists for extern already.
The text was updated successfully, but these errors were encountered: