Skip to content

#1292 updated flows at pape and millwood to represent the actual dire…#1333

Open
wael6954 wants to merge 1 commit intomasterfrom
1292-fixing-flows-with-incorrect-directionality
Open

#1292 updated flows at pape and millwood to represent the actual dire…#1333
wael6954 wants to merge 1 commit intomasterfrom
1292-fixing-flows-with-incorrect-directionality

Conversation

@wael6954
Copy link
Copy Markdown
Contributor

the flow goems at millwood and pape have an incorrect directionality for which the southbound flows are drawn pointing towards the north

@wael6954
Copy link
Copy Markdown
Contributor Author

image

this is another instance of which I am unsure if I need to switch the flow geoms because it is possible that the flow measures for bikes going the wrong way.

Copy link
Copy Markdown
Member

@Nate-Wessel Nate-Wessel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we've strayed from this a bit, but my original intention for the flow geoms is that they would have their first node over the actual sensor and the second node pointing in the direction of travel measured. Simply reversing these would swap the nodes and put the first node in the wrong location.

This is documented here.

IMO, if these flows are reversed, they should just be manually modified via QGIS's node tool.

@wael6954
Copy link
Copy Markdown
Contributor Author

@Nate-Wessel so are the flows I showed in the screenshot incorrect?

I also dont know how to do it in QGIS, but would be open to learn.

@Nate-Wessel
Copy link
Copy Markdown
Member

so are the flows I showed in the screenshot incorrect?

You tell me. I thought that's what this issue was about?

@gabrielwol gabrielwol added the Ecocounter Permanent bicycle count stations. Also use "Permanent Sensors" label label Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ecocounter Permanent bicycle count stations. Also use "Permanent Sensors" label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Eco-counter - Fill in missing site and flow information for new and existing counters

3 participants