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

Fixes related to meshkernelpy issues #234

Open
47 of 99 tasks
veenstrajelmer opened this issue Feb 7, 2023 · 1 comment
Open
47 of 99 tasks

Fixes related to meshkernelpy issues #234

veenstrajelmer opened this issue Feb 7, 2023 · 1 comment

Comments

@veenstrajelmer
Copy link
Collaborator

veenstrajelmer commented Feb 7, 2023

Next release:

Later:

Documentation:

  • update readme badges MeshKernelPy#178
  • Improve documentation of mesh2d_delete_small_flow_edges_and_small_triangles MeshKernelPy#161
  • create example in documentation for refinement based a polygon
  • create example in documentation for refinement based on bathymetry, including polygon (to avoid refinement on open boundary) >> more information required
  • add rgfgrid-based grid generation from hydrodynamic course as a meshkernel notebook (includes base grid, refinement, splines etc): n:\Projects\11209500\11209671\B. Measurements and calculations\01_Updated_course_materials\02_exercises\ (Exercise 1 to 4), base on stenderts notebook (p:\11210154-arem-noregret\hydro\modelbuilder\grid\original_curvilinear_UTM\grid_polygons.ipynb
  • meshkernel logo

Other:

Related issues:

  • Add model builder example notebook #217
  • meshkernel(py) issues in JIRA
  • possible to check orthogonality in mk? More grid validators/cleanups (modelbuilder example script)
  • possible to use connect_cells separately (without refining)?
  • polygon generation/editing, maybe already possible?
  • check TODO in meshkernel example script and modelbuilder example script
  • minimal interacter network is 1D: p:\dflowfm\maintenance\JIRA\06000-06999\06548\meshkernel_interp.py, can be used to interpolate bathy to?
  • do we need sphericalaccurate grids for dcsm/gtsm? These are not implemented yet (only spherical)
  • overview of RGFGRID functions

Workflows:

  • GTSM: global base grid, bathy+gradient refinement, cut landpart with ldb (incl drypoints), example script with some issues
  • 1D2D connected like GTSM+rivers
  • DCSM: bathy+polygonen refinement, cut landpart with ldb+depth (drypoints with matlab, details at JG?)
  • RMM: multiblock rivers, curvigrid generation based on splines, triangles in estuaries, squares in seapart. For the second, we need a to divide a polygon in equal length parts, part of meshkernel?
  • D-HYDRO course materials workflow
  • tutorial materials from manual
  • RGFGRID functions overview, manual edits like in edit>reg/irreg
  • create unittests of all workflows to make sure behaviour is tested and future changes do not result in undesired behaviour

DCSM steps:

  • Create base_grid (4 nm resolution) >> edge length now varies over latitude, inconvenient? spherical curvilinear_compute_rectangular_grid_on_extension does not result in grid with constant dx/dy MeshKernelPy#153 >> the original basegrid script seems to enforce a factor of 1.5 already also: p:\archivedprojects\11205259-004-dcsm-fm\GridGeneration\scripts\create_base_grids.m
  • delete with polygon (delete network)
  • delete with polygon (based on cell centers)
  • refine with polygon
  • set all z-values to -999 (fillvalue)
  • interp bathy from xyz
  • set missing z-values to certain value, if outside of polygon (actually to later delete cells on land, so easier method for that might also work)
  • delete nodes with z-values lower than specified value or missing value (only if this applies to all nodes of face) >> can be replaced with delete option that considers crossings of cell edges like in mesh2d_delete deletes too much cells MeshKernelPy#63 (comment)
  • remove small flowlinks (merge circumcenters) >> mesh2d_delete_small_flow_edges_and_small_triangles (merge circumcenters option? #812 (comment))
  • delete 1D/hanging links >> mesh2d_delete_hanging_edges
  • merge with additional grid (merge nodes on top of each other) >> mesh2d_merge_nodes
  • is not entirely solved by improving polygon deletion, since we have fjords in norway where emodnet has (wet) bathymetry but the landboundary does not include them
  • illegalcells necessary, or prevent high orthogonality after coastline cutting MeshKernelPy#150
  • original workflow documented in (is this the latest version?): p:\archivedprojects\11205259-004-dcsm-fm\GridGeneration\PolygonBasedRefinement\DCSM-FM_workflow_GridGeneration.txt
  • meshkernel alternative: p:\1204257-dcsmzuno\gridcreation_test_meshkernelpy\DCSM_grid_creation_meshkernel.py

Test GTSM-specific new features (JV):

Release 3.0.0:

mk 4.1.0 (released 2024-02-15):

Meshkernelpy 4.2.0:

meshkernel 5.0.1/5.0.2/6.0.0:

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

1 participant