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

shp_to_inp coordinates for outfall are missing #15

Closed
fcfunke opened this issue Jul 27, 2018 · 6 comments
Closed

shp_to_inp coordinates for outfall are missing #15

fcfunke opened this issue Jul 27, 2018 · 6 comments
Assignees

Comments

@fcfunke
Copy link

fcfunke commented Jul 27, 2018

I'm using shp_to_inp to create an input file out of my shapefiles (subcatchments, conduits, junctions, outfall).
It is working for all except the outfall where coordinates in swmm are missing.

My impression: Coordinates for junctions are saved but the outfall is not.
list_of_sections[["coordinates"]] <- junction
(Zeile 226 in https://github.com/dleutnant/swmmr/blob/master/R/shp_to_inp.R)

thank you for help

@dleutnant
Copy link
Owner

@DoeringA Any idea?

@dleutnant dleutnant assigned dleutnant and DoeringA and unassigned dleutnant Jul 27, 2018
@dickinsonre
Copy link

If i may interject, you probably don't loop over the whole Range of Nodes [Junctions, Outfall, Dividers and Storage] when you make the Coordinate section

@fcfunke
Copy link
Author

fcfunke commented Jul 27, 2018

It works if I include the outfall (Out1) in both path_point and path_outfall and remove it from $junction afterwards.
example1<-shp_to_inp(
path_polygon = file.path(directory,outsub),
path_point = file.path(directory,outnod),
path_outfall = file.path(directory,outout),
path_line = file.path(directory,outcon),
subcatchment_typologies = subcatchment_typologies,
infiltration = infiltration)

example1$junction <- example1$junction[example1$junction$Name != "Out1", ]
write_inp(example1,input)

@dleutnant
Copy link
Owner

Is this issue still unsolved? Otherwise please consider to close and comment. Thanks!

@DoeringA
Copy link
Collaborator

In the current version the coordinate section is only made for junction nodes. As dickinsonre commented, looping over the whole range of nodes still needs to be implemented. Currently, coordinates for other than junction nodes, must be added to the inp object manually (inp$coordinates).

@dleutnant
Copy link
Owner

#39

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

4 participants