Skip to content
This repository was archived by the owner on Sep 18, 2024. It is now read-only.

feat: add a ping and model manager to allow the validator to grab the closest miner #73

Closed
wants to merge 6 commits into from

Conversation

GuiBibeau
Copy link
Collaborator

@GuiBibeau GuiBibeau commented Apr 16, 2024

This implements a miner manager class in validators. Essentially this allows validators to know which miners are the closest to them for a given model. Allowing to give a proximity in responses when receiving requests.

Description of the pull request

Changes made

  • Service mesh receives a new endpoint.
  • Remove the need to grab the closest miner on each validator call received.
  • Keep a map of all miners and their available miners.

Related issues

Testings done

Manual with subnet on and off.

Screenshots (if any)

Checklist

  • I have written tests
  • My code does not produce new errors
  • I gave myself a code review before asking others.

Copy link

vercel bot commented Apr 16, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
akeru-akeru-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 17, 2024 1:33pm

Copy link
Collaborator

@TommieNg TommieNg left a comment

Choose a reason for hiding this comment

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

Just a heads-up that I'm not familiar with Python. Did my best to spot any errors, but for the logic part, you might want to loop in someone more Python experience.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants