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

Dfs2 - static items (bathymetry) #677

Open
stkistner opened this issue Mar 21, 2024 · 6 comments · May be fixed by #682
Open

Dfs2 - static items (bathymetry) #677

stkistner opened this issue Mar 21, 2024 · 6 comments · May be fixed by #682
Labels
enhancement New feature or request

Comments

@stkistner
Copy link

Problem

When reading certain Dfs2's, e.g. model outputs from the FM egine the file often contains a static item that describes the bathymetry. This item is a perquisite in some of the processing tools or connected models (e.g. directional wave analysis or MIKE MA). When processing these files with mikeio the data is discarded, which naturally leads to problems.

Solution

Add an option to read/write the static bathy item

Alternatives considered

Using mikecore I can get around this. But we like to use mikeio for simplicity. I haven't seen any other issues related to this.

@ecomodeller ecomodeller added the enhancement New feature or request label Mar 21, 2024
@ecomodeller
Copy link
Member

Is bathymetry the only example of where a static item is used (in practice), or are there other examples?

@stkistner
Copy link
Author

Is bathymetry the only example of where a static item is used (in practice), or are there other examples?

Bathymetry is the only example that I'm aware of.

@ecomodeller
Copy link
Member

ecomodeller commented Mar 21, 2024 via email

@jsmariegaard
Copy link
Member

The FM engine can output dfs2 in a gridded overlay. That is not a legacy output. I don't know, however, whether it contains any static bathymetry 🤔

@stkistner
Copy link
Author

I extracted a Dfs2 this morning from a Dfsu, and that did extract the bathy data too.

But I know this is for 1 out of 100 applications. In the meantime I've managed to re-add the bathy using mikecore, but a a mikeio implementation would be nice 😉

@ecomodeller
Copy link
Member

Let's add the bathymetry as an optional attribute to the Grid2D data structure.

@ecomodeller ecomodeller linked a pull request Mar 25, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants