CWorthy-ocean / roms-tools

Tools for setting up and running ROMS simulations
https://roms-tools.readthedocs.io
GNU General Public License v3.0
7 stars 3 forks source link

What discrete choices get made during setup? #9

Open TomNicholas opened 5 months ago

TomNicholas commented 5 months ago

The matlab tools have 6 separate steps, but that doesn't mean that the ROMS user actually needs to make a new choice at each of those steps. For example when you create the grid (step 1) you choose the location on the earth of the domain, but when you make the topography (step 2) you don't make a new choice, the topography just has to be for the region you chose in step 1.

We should try to fuse together different steps in this workflow so that the user is only required to do anything when there is an actual new choice to be made.

How many discrete choices are there? At least: 1) Choose the resolution, size, and location of the domain, 2) Choose the time resolution you want to run with (needed for tides + forcing data?), 3) Choose which data sources you want to use for the initial/boundary/forcing data

Are there any others? What about choice of processor decomposition?

@ubbu36

NoraLoose commented 1 month ago

For example when you create the grid (step 1) you choose the location on the earth of the domain, but when you make the topography (step 2) you don't make a new choice, the topography just has to be for the region you chose in step 1.

There is actually a new decision involved when adding the topography, namely which data source to use. The matlab code allows 8 different choices, see https://github.com/CWorthy-ocean/C-Star/blob/6dfebe86eba059f3732764fa8ff99d9cd412290a/setup_tools/setup_r2r_phys%2Bbgc/2.Make_topo/topo.m#L57-L80.

@sdbachman @TomNicholas Do we want to give the user the option to choose from different data sources for the topography? Probably yes... If so, we need to

TomNicholas commented 1 month ago

I feel like we discussed this a while ago and thought that in practice topography is unlikely to be something that anyone really cares about changing...

TomNicholas commented 1 month ago

That said an optional topography_source kwarg seems nice and neat, and it can just default to etopo5.