feat: Make calls to numtracker async

[tpoliaw]

No description provided.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.74%. Comparing base (0c00d37) to head (b6cb2d2).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1075   +/-   ##
=======================================
  Coverage   94.74%   94.74%           
=======================================
  Files          42       42           
  Lines        2741     2742    +1     
=======================================
+ Hits         2597     2598    +1     
  Misses        144      144           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[callumforrester]

@tpoliaw the fact that this isn't making any tests fail is concerning

[tpoliaw]

We're never relying on bluesky calling it in tests and the system tests don't use numtracker

[abbiemery]

Since we've moved on somewhat since this was opened: do we now use the call in system tests/is this tested, and if so does the change to make it async affect anything?

Could we either get this tested and in, or close this PR?

[tpoliaw]

Rebasing it on main still doesn't cause any tests to fail so I guess it works. This probably makes sense now that bluesky support async scan_id_sources.

Whether we still want to merge this without removing other uses of requests is a separate question.