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

Some SFINCS models cannot be used to define the domain for Delft-FIAT model builder (missing region.geojsn file) #171

Open
2 tasks done
kathrynroscoe opened this issue Oct 21, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@kathrynroscoe
Copy link
Collaborator

Preliminary Checklist

  • I have searched the existing issues list for similar reports and found none. If you did find a similar issue, please add a comment to the existing issue instead of opening a new one.
  • I have verified that the issue exists in the latest version of DelftDashboard and its dependencies. Please update to the latest version and check if the issue still exists.

Severity Level

Medium

Steps to reproduce

In the Delft-FIAT model-builder, choose SFINCS model for your model domain and then select a SFINCS model folder that was built either (1) before hydromt-sfincs made the region.geojsn file or (2) was not created with hydromt-sfincs. It will not load the domain because it cannot find the region.geojsn file.

Current behaviour

Older SFINCS models or those not built with hydromt-sfincs cannot be used to define the Delft-FIAT model domain

Desired behaviour

When you choose an older SFINCS models or one not built with hydromt-sfincs, it will use the existing SFINCS files to generate a region.geojsn file, and then use this for the Delft-FIAT model domain.

Additional context

No response

@kathrynroscoe kathrynroscoe added the bug Something isn't working label Oct 21, 2024
@LuukBlom LuukBlom assigned GundulaW and unassigned roeldegoede Dec 5, 2024
@GundulaW
Copy link
Collaborator

GundulaW commented Dec 5, 2024

Three simple lines of code to get the model region immediately from a SFINCS model:

from hydromt_sfincs import SfincsModel
sf_model = SfincsModel(root=model_root, mode="r+")
sf_model.region

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants