-
Notifications
You must be signed in to change notification settings - Fork 41
Redo infiltration #330
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
Draft
Leynse
wants to merge
24
commits into
main
Choose a base branch
from
redo_infiltration
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Redo infiltration #330
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
beb9f1a
- Initial commit, working on make a generic read_netcdf_quadtree_gene…
Leynse 68e1493
- Working version for storage volume nc with fixed mapping to SFINCS …
Leynse 245c1a1
- Add error when wanted variable was not found in file and directly stop
Leynse d3db40e
- Add netinfiltrationfile, netinftype variables
Leynse 4353274
- Implement netcdf reader option for all infiltration options
Leynse 378e27e
- Add stop_sfincs check on combination check of infiltration input ty…
Leynse a2f67b4
- Constant uniform infiltration is allowed for both grid types
Leynse c654c00
- Edit check - still possible that precip is forced but qinf=0
Leynse 36a5aa8
- Change names to infiltrationfile and infiltrationtype
Leynse 30a0cb6
- Move whole subroutine 'initialize_infiltration' from sfincs_domain.…
Leynse 0dc0720
- Add check whether specified infiltrationtype is correct
Leynse 2a1a974
- Added version for reading in netcdf quadtree manningfile (#271)
Leynse 29a145c
Merge branch '268-make-generic-quadtree-nc-input-reader' of https://g…
Leynse 167387b
- Bump date
Leynse a0ed4a0
Progress with redoing continuity logic, adding new infiltration flavo…
keesnederhoff cf6a78d
Included more output options
keesnederhoff 3300914
- clean repo
Leynse fe8cf95
Merge branch 'redo_infiltration' of https://github.com/Deltares/SFINC…
Leynse bb61661
submit progress
keesnederhoff 8c91945
Merge branch 'redo_infiltration' of https://github.com/Deltares/SFINC…
keesnederhoff f1c425b
included loss function in bucket model
keesnederhoff 2a07fe8
ongoing progress
keesnederhoff a5a25df
Merge branch 'main' into redo_infiltration
maartenvanormondt 462dfd1
Merge branch 'redo_infiltration' of https://github.com/Deltares/SFINC…
maartenvanormondt File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,40 @@ | ||
|
|
||
| ------------ Welcome to SFINCS ------------ | ||
|
|
||
| @@@@@ @@@@@@@ @@ @@ @@ @@@@ @@@@@ | ||
| @@@ @@@ @@@@@@@ @@ @@@ @@ @@@@@@@ @@@ @@@ | ||
| @@@ @@ @@ @@@ @@ @@ @@ @@@ | ||
| @@@@@ @@@@@@ @@ @@@@@@ @@ @@@@@ | ||
| @@@ @@ @@ @@ @@@ @@ @@ @@@ | ||
| @@@ @@@ @@ @@ @@ @@ @@@@@@ @@@ @@@ | ||
| @@@@@ @@ @@ @@ @ @@@@ @@@@@ | ||
|
|
||
| .............. | ||
| ......:@@@@@@@@:...... | ||
| ..::::..@@........@@.:::::.. | ||
| ..:::::..@@..::..::..@@.::::::.. | ||
| .::::::..@@............@@.:::::::. | ||
| .::::::..@@..............@@.:::::::. | ||
| .::::::::..@@............@@..::::::::. | ||
| .:::::::::...@@.@..@@..@.@@..::::::::::. | ||
| .:::::::::...:@@@..@@..@@@:..:::::::::.. | ||
| ............@@.@@..@@..@@.@@............ | ||
| ^^^~~^^~~^^@@..............@@^^^~^^^~~^^ | ||
| .::::::::::@@..............@@.:::::::::. | ||
| .......:.@@.....@.....@....@@.:....... | ||
| .::....@@......@.@@@.@....@@.....::. | ||
| .:::~@@.:...:.@@...@@.:.:.@@~::::. | ||
| .::~@@@@@@@@@@.....@@@@@@@@@~::. | ||
| ..:~~~~~~~:.......:~~~~~~~:.. | ||
| ...................... | ||
| .............. | ||
|
|
||
| ------------------------------------------ | ||
|
|
||
| Build-Revision: $Rev: v2.3.1 mt. Faber+branch-redo-infiltration | ||
| Build-Date: $Date: 2026-03-19 | ||
|
|
||
| ------ Preparing model simulation -------- | ||
|
|
||
| Reading input file ... | ||
| Error : SFINCS input file "sfincs.inp" not found! SFINCS has stopped! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| <?xml version="1.0" encoding="UTF-8"?> | ||
| <VisualStudioUserFile ProjectCreator="Intel Fortran" Keyword="Console Application" Version="11.0" ProjectIdGuid="{84DFC66B-C675-4F50-8A9A-62E7E483ACDD}"> | ||
| <Configurations> | ||
| <Configuration Name="Debug|x64" WorkingDirectory="d:\software\SFINCS\test_model\infiltration_testbed\05_bucket\local\"> | ||
| </Configuration> | ||
| <Configuration Name="Release|x64" WorkingDirectory="d:\software\SFINCS\test_model\infiltration_testbed\05_bucket\local\"> | ||
| </Configuration> | ||
| </Configurations></VisualStudioUserFile> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| <?xml version="1.0" encoding="UTF-8"?> | ||
| <VisualStudioUserFile ProjectType="typeStaticLibrary" ProjectCreator="Intel Fortran" Keyword="Static Library" Version="11.0" ProjectIdGuid="{53E1EF5D-DBAE-46DC-B269-8CA4B8EF9883}"> | ||
| <Configurations> | ||
| <Configuration Name="Debug|x64" WorkingDirectory="d:\software\SFINCS\test_model\infiltration_testbed\05_bucket\local\"> | ||
| </Configuration> | ||
| <Configuration Name="Release|x64" WorkingDirectory="d:\software\SFINCS\test_model\infiltration_testbed\05_bucket\local\"> | ||
| </Configuration> | ||
| </Configurations></VisualStudioUserFile> |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Important clarification - netcdf infiltration ONLY works for quadtree models, while the old binary options ONLY work for regular models. So it is not just backward compatibility here