Skip to content

Refactor overpass endpoint selection order#956

Merged
louis-e merged 3 commits intomainfrom
feat/overpass-arnis-priority
Apr 21, 2026
Merged

Refactor overpass endpoint selection order#956
louis-e merged 3 commits intomainfrom
feat/overpass-arnis-priority

Conversation

@louis-e
Copy link
Copy Markdown
Owner

@louis-e louis-e commented Apr 21, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 21, 2026 12:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Refactors the Overpass API endpoint selection order in fetch_data_from_overpass, introducing an Arnis-hosted Overpass endpoint and a probabilistic “probe” of an official endpoint before proceeding with the normal retry sequence.

Changes:

  • Add https://api.arnismc.com/overpass/api/interpreter and attempt it before shuffled official/fallback endpoints.
  • Introduce a 25% chance “probe” request to a random official Overpass server before the normal sequence.
  • Replace the prior concatenated shuffled server list with an explicit request_plan that tracks fallback vs non-fallback for retry delays/logging.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/retrieve_data.rs
Comment thread src/retrieve_data.rs Outdated
@github-actions
Copy link
Copy Markdown

⏱️ Benchmark run finished in 2m 35s
🏗️ Generation time: 19s (excl. data fetching)
🧠 Peak memory usage: 1733 MB

📈 Compared against baseline: 20s
🧮 Delta: 135s
🔢 Commit: ba095d1

🟢 Generation time is unchanged.

📅 Last benchmark: 2026-04-21 12:37:58 UTC

You can retrigger the benchmark by commenting retrigger-benchmark.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/retrieve_data.rs Outdated
@louis-e louis-e merged commit 00d02df into main Apr 21, 2026
2 checks passed
@louis-e louis-e deleted the feat/overpass-arnis-priority branch April 21, 2026 12:54
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.

2 participants