feat: implement basic sanitation for miner selection

masa-finance / masa-bittensor

Bittensor Subnet Config

https://masa.ai

MIT License

0 stars 0 forks source link

feat: implement basic sanitation for miner selection #112

Open juanmanso opened 1 week ago

juanmanso commented 1 week ago

Related issues

Closes #92

Notes

Implementation heavily inspired by SN 15 - Blockchain Insight's own implementation of miner selection function (link)

hide-on-bush-x commented 1 week ago

Found a bug here, will fix it and once confirmed that it works ill merge it

juanmanso commented 3 days ago

Found a bug here, will fix it and once confirmed that it works ill merge it

Found this logging bug, let me know if it's the same you were already looking at @hide-on-bush-x

juanmanso commented 3 days ago

Validator code fixed (tested on twitts endpoint) with pushed code, gonna figure out miner issue shortly

Validator run

Miner run with error

juanmanso commented 1 day ago

As shown on the image below, it can be seeing that the validator is ignoring the local miner, which is the expected behavior. Now, we need to tweak this to toggle the selection so we can test locally

Ping uids result

Local set of validator/miners