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

FlowBoundary is flow direction constraining #1424

Merged
merged 4 commits into from
Apr 24, 2024

Conversation

visr
Copy link
Member

@visr visr commented Apr 23, 2024

(cherry picked from commit e84ae3a)

Fixes #1423

@visr visr marked this pull request as draft April 23, 2024 17:09
@visr
Copy link
Member Author

visr commented Apr 23, 2024

Hmm CI doesn't like this. Perhaps this fixes #1423, but there are more tests to adopt it seems. @SouthEndMusic perhaps you want to take over this PR?

@SouthEndMusic
Copy link
Collaborator

Hmm CI doesn't like this. Perhaps this fixes #1423, but there are more tests to adopt it seems.

To solve the problem posed in #1423 you probably only need the change in utils.jl. The change in allocation_optim.jl can be kept here, that one makes sure that if the outflow edge of a UserDemand node is the reverse of its inflow edge, they are still always kept separate in the allocation flow output.

I did quite some digging to understand the change you took over in allocation_init.jl. I still don't really understand it. It is about how sources in the main network are handled, and for some reason I thought it was a good idea to treat sources in the main network as main-to-subnetwork connections. However, the consequences of this are quite complex and probably not as desired. I suspect that it comes from an observation that the capacity of sources in the main network is not taken into account properly, but this is not the way to solve it. We can make an issue for that now, or wait until we can inspect allocation flows in QGIS.

@SouthEndMusic SouthEndMusic marked this pull request as ready for review April 24, 2024 07:25
@SouthEndMusic SouthEndMusic merged commit 6c266af into main Apr 24, 2024
24 checks passed
@SouthEndMusic SouthEndMusic deleted the flow-boundary-direction branch April 24, 2024 07:25
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

Successfully merging this pull request may close these issues.

Allocation test failure on main
2 participants