- 
                Notifications
    
You must be signed in to change notification settings  - Fork 3
 
# Use real topography data with GeophysicalModelGenerator #63
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
base: main
Are you sure you want to change the base?
Conversation
…tError if you don't choose your gridsizes the right way
…, convert_geo_1d, convert_geo_2d to create arbitrary topography data and make it accessible to TrixiBottomTopography.
          Changes
  | 
    
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I will take a look at this. Could you please restore files that do not need to change in this PR? For example, the .JuliaFormatter.toml file is deleted and some random files have been reformatted but not changed otherwise.
| @@ -0,0 +1,214 @@ | |||
| # Create and convert real topography data | |||
| In this section, it is explained how to get real topography data using GeophysicalModelGenerator.jl and make it accessible to TrixiBottomTopography. The necessary structure of the data for TrixiBottomTopography is explained in the `docs/src/conversion.md` file. In the file `docs/src/trixishallowwater_jl_geo_data_examples`, it is then explained how to properly run simulations with this data. In the following, we will consider the example file `create_convert_data_geo.jl`. | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please add links to Julia packages and add the .jl part to them, e.g.,
| In this section, it is explained how to get real topography data using GeophysicalModelGenerator.jl and make it accessible to TrixiBottomTopography. The necessary structure of the data for TrixiBottomTopography is explained in the `docs/src/conversion.md` file. In the file `docs/src/trixishallowwater_jl_geo_data_examples`, it is then explained how to properly run simulations with this data. In the following, we will consider the example file `create_convert_data_geo.jl`. | |
| In this section, it is explained how to get real topography data using [GeophysicalModelGenerator.jl](https://github.com/JuliaGeodynamics/GeophysicalModelGenerator.jl) and make it accessible to [TrixiBottomTopography.jl](https://github.com/trixi-framework/TrixiBottomTopography.jl). | |
| The necessary structure of the data for [TrixiBottomTopography.jl](https://github.com/trixi-framework/TrixiBottomTopography.jl) is explained in the `docs/src/conversion.md` file. In the file `docs/src/trixishallowwater_jl_geo_data_examples`, it is then explained how to properly run simulations with this data. In the following, we will consider the example file `create_convert_data_geo.jl`. | 
Could you please also update references to other doc files to real Documenter.jl/Markdown references that will be rendered as HTML links in the online docs?
| ```julia | ||
| # Get a first impression of the topography data | ||
| Topo, p, Topo_Cart = geo_topo_impression( | ||
| resolution="@earth_relief_01s", | ||
| lon_min=6.963880, | ||
| lon_max=6.978499, | ||
| lat_min=50.947861, | ||
| lat_max=50.957095 | ||
| ) | ||
| ``` | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the examples you demonstrate here do not run forever, could you please turn them into Documenter.jl named @example blocks so that they will be executed when building the online docs to ensure that everything still works?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it necessary to load such huge files into the git repository?
Changes
geo_data.jlthat shows how to use GeophysicalModelGenerator to load real topography and use it for simulations in TrixiBottomTopography.Testing
Notes
Ci.yml: Success
FormatCheck.yml: Failure
Documenter.yml: Failure
Looking for feedback on: