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.

Sign up for GitHub

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

Jump to bottom

node id’s between connection not in attribute table #288

Closed
SnippenE opened this issue Jun 7, 2023 · 5 comments · Fixed by #328
Closed

node id’s between connection not in attribute table #288

SnippenE opened this issue Jun 7, 2023 · 5 comments · Fixed by #328
Labels
documentation Improvements or additions to documentation QGIS Ribasim QGIS plugin
Milestone
1. basic hydro fu...

Comments

@SnippenE
Copy link

SnippenE commented Jun 7, 2023 •
edited by visr
Loading

Vraag Noortje Romeijn (Witteveen en Bos):

Voor TKI NHI ben ik bezig om Ribasim te verkennen. Ik heb hier een paar vragen over:

Ik heb de voorbeelden gevolgd op deltares.github.io/ribasim en dat lukt. Nu probeer ik zelf een Ribasim model met behulp van de QGIS plugin op te zetten. Bij het maken van edges tussen de nodes komen de node id’s van de nodes waartussen ik een connectie wil hebben niet in de attribute table terecht (terwijl ik zie dat dit bij het voorbeeld model wel het geval is). Als ik het model vervolgens wil runnen, krijg ik een error waarin een verwijzing staat naar simple graphs en missing. Ik denk dat dit met elkaar te maken heeft. Hoe kan ik dit oplossen?

Is er ergens een overzicht beschikbaar van wat de nodes precies inhouden en welke informatie je minimaal aan de nodes moet meegeven om een werkend model te krijgen? Ik vraag me ook af hoe het zit met beginvoorwaarden. Het lijkt alsof dit nergens wordt meegegeven in het voorbeeld model.

Ik hoor het graag.

Vriendelijke groet,

N. Romeijn (Noortje)
Projectingenieur Watermanagement | Wateroverlast

[email protected]

Witteveen+Bos
@SnippenE SnippenE added this to Ribasim Jun 7, 2023
@SnippenE SnippenE converted this from a draft issue Jun 7, 2023
@SnippenE SnippenE added documentation Improvements or additions to documentation bug labels Jun 7, 2023
@SnippenE SnippenE added the QGIS Ribasim QGIS plugin label Jun 7, 2023
@SnippenE
Copy link
Author

SnippenE commented Jun 7, 2023 •
edited
Loading

related to #252 ?

@SouthEndMusic
Copy link
Collaborator

Over de beginvoorwaarden: je kan de begin storages meegeven met Basin / state. Dat is op dit moment geen verplichte input, in de Julia core wordt bij het ontbreken voor alle basins een storage van 1.0 m^3 aangenomen. @visr heeft aangegeven dat het wellicht goed is dat dit verplichte input wordt. Wellicht is het ook intuitiever om begin level aan te geven in plaats van begin storage?

@SouthEndMusic
Copy link
Collaborator

Over het overzicht van de nodes: usage geeft aan welke data bij welk node type hoort (het lijkt me inderdaad goed om hierbij aan te geven wat verplicht is en wat niet). equations geeft een theoretische achtergrond per node type.

@gijsber
Copy link
Contributor

gijsber commented Jun 8, 2023

begin level is inderdaad vanuit de hydrologische gebruiker gewenst

@visr
Copy link
Member

visr commented Jun 8, 2023

Bij het maken van edges tussen de nodes komen de node id’s van de nodes waartussen ik een connectie wil hebben niet in de attribute table terecht.

Dit is inderdaad een probleem, die horen daar wel te staan. De plugin vult deze waardes zelf in op basis van het begin en eindpunt van de lijnen. Hier een voorbeeld waar ik zojuist een nieuwe edge heb getekend maar nog steeds in edit mode zit, de attributen zijn nog niet ingevuld:

image

En nadat ik op het potlood heb geklikt om uit edit mode te gaan, zijn ze ingevuld:

image

Er is wel een bug nog moet worden opgelost, wat wellicht het probleem was hier. Als je wijzigingen opslaat met het rechtericoon hier, ipv het linker icoon, dan wordt er niets automatisch ingevuld:

image

@SnippenE SnippenE moved this from To do to Sprint backlog in Ribasim Jun 8, 2023
@SouthEndMusic SouthEndMusic mentioned this issue Jun 8, 2023
Closed
@github-project-automation github-project-automation bot moved this from Sprint backlog to ✅ Done in Ribasim Jun 9, 2023
@SouthEndMusic SouthEndMusic mentioned this issue Jun 9, 2023
Closed
@visr visr reopened this Jun 13, 2023
@visr visr mentioned this issue Jun 16, 2023
Merged
@visr visr closed this as completed in #328 Jun 16, 2023
visr added a commit that referenced this issue Jun 16, 2023
@visr
add data tables to gpkg_contents (#328)
9accd62 
Fixes #291

The QGIS plugin uses this metadata table to get the table names. Since
switching from GDAL to SQLite for writing data tables, the data tables
were not registered in gpkg_contents anymore. The GeoPackage
[spec](http://www.geopackage.org/spec131/#attributes) says this needs to
be done, but GDAL itself is more lenient, still showing the data tables
without registering them.

So now the gpkg_contents of basic.gpkg looks like this:

<img width="734" alt="image"
src="https://github.com/Deltares/Ribasim/assets/4471859/0e8bf370-4cad-4c0a-844e-3a3cb6b15ef4">

Fixes #288 

Meant to create a separate PR for this, but
bd4aae4 fixes two more issues. One is
that clicking save before stopping editing did not trigger the old
`afterCommitChanges`, using `editingStopped` works in both cases. The
other is that the new column edge_type is now also filled in
automatically with "flow". There is #301 for control in QGIS. To fix
that this function also needs to be updated.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation Improvements or additions to documentation QGIS Ribasim QGIS plugin
Projects
Ribasim
Archived in project
Milestone
1. basic hydro functionality (physica...
Development

Successfully merging a pull request may close this issue.

add data tables to gpkg_contents Deltares/Ribasim
4 participants
@visr @gijsber @SouthEndMusic @SnippenE