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

Remove tile_extract from config when creating stats #4722

Merged
merged 2 commits into from
May 7, 2024

Conversation

chrstnbwnkl
Copy link
Contributor

Issue

When running valhalla_add_predicted_traffic using a config that has an entry for both mjolnir.tile_dir and mjolnir.tile_extract, the summary (if requested) will be created from the tar, not the tile directory. The predicted traffic is added to the tile directory though, so according to the summary, no speeds were added. This PR just removes the mjolnir.tile_extract before creating the summary, so it reads from the tile directory instead.

Tasklist

  • Add tests
  • Add #fixes with the issue number that this PR addresses
  • Update the docs with any new request parameters or changes to behavior described
  • Update the changelog
  • If you made changes to the lua files, update the taginfo too.

@kevinkreiser kevinkreiser merged commit 1594fe9 into master May 7, 2024
8 of 9 checks passed
@kevinkreiser kevinkreiser deleted the cb-fix-pred-traffic-stats branch May 7, 2024 19:50
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.

None yet

2 participants