Skip to content

Conversation

Robinlovelace
Copy link
Member

No description provided.

@Robinlovelace Robinlovelace linked an issue Jul 25, 2025 that may be closed by this pull request
2 tasks
@Robinlovelace
Copy link
Member Author

Update here: got the elevation for Brussels region.

image

Code: see commit above.

@Robinlovelace
Copy link
Member Author

Cycle net for 6km study area:

image

@Robinlovelace
Copy link
Member Author

Robinlovelace commented Jul 25, 2025

You can reproduce this with

gh pr checkout 68
# Open and run the code in vignettes/verification.Rmd

@Robinlovelace
Copy link
Member Author

@Robinlovelace
Copy link
Member Author

You can skip the DEM download step by downloading this and putting it in a new dems folder for testing: https://github.com/ropensci/slopes/releases/download/v1.0.1/Copernicus_DSM_COG_10_N50_00_E004_00_DEM.tif

@Robinlovelace Robinlovelace requested a review from temospena July 25, 2025 13:36
@Robinlovelace
Copy link
Member Author

Results for Leeds: looking pretty good.

image

@Robinlovelace
Copy link
Member Author

Preliminary results for Brussels: https://github.com/ropensci/slopes/releases/download/v1.0.1/cycle_net_slopes_brussels.html

I sense that this substantially overestimates the slopes. Next step: explore and apply smoothing function. Any feedback before then welcome.

Here's what it should look like:

image

Raw data:

https://github.com/ropensci/slopes/releases/download/v1.0.1/cycle_net_slopes_brussels.gpkg

@temospena
Copy link
Collaborator

I as able to reproduce it

image

@temospena
Copy link
Collaborator

Maybe we can revisit this validation / benchmark working paper :)
https://github.com/U-Shift/Declives-RedeViaria/blob/main/Benchmark.md

@Robinlovelace
Copy link
Member Author

Great idea Rosa, I'm well up for that.

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.

Additional validation datasets/tests
2 participants