RTX icon indicating copy to clipboard operation
RTX copied to clipboard

#2565 #2563

Open bazarkua opened this issue 3 months ago • 2 comments

Added validation to KPSelector now only query healthy KPs, we now skip any KP whose /meta_knowledge_graph doesn’t return HTTP 200.

Added SPOKE to a hardcoded blacklist (because (CI and PROD endpoints were returning inconsistent results). in Expand/smartapi.py BLACKLISTED_KPS = {'infores:spoke'} and propagated kps_excluded_by_black_list through the cache. KPSelector logs “Blacklisted by ARAX (KP is unstable)”.

Linting/type hygiene: Minor edits to satisfy mypy/ruff in touched modules; no behavior changes beyond the validation/blacklist logic.

bazarkua avatar Sep 22 '25 04:09 bazarkua

@edeutsch Thanks for suggestion, you're right after Googling the definitions of the blacklist and blocklist, it's more appropriate to use blocklist. From my understanding last commit https://github.com/RTXteam/RTX/commit/1f4d2259ec3b5cb2ecff31da6754f23819cc42ba supposed to be temporary? I can add 'infores:automat-robokop' to the block list in my next commit in the same way I did for 'infores:spoke' in the smartapi.py

bazarkua avatar Sep 22 '25 18:09 bazarkua

@edeutsch Thanks for suggestion, you're right after Googling the definitions of the blacklist and blocklist, it's more appropriate to use blocklist.

thanks

From my understanding last commit 1f4d225 supposed to be temporary?

correct.

I can add 'infores:automat-robokop' to the block list in my next commit in the same way I did for 'infores:spoke' in the smartapi.py

not yet, but perhaps later this week yes. Pending discussion on Wednesday AHM.

edeutsch avatar Sep 22 '25 18:09 edeutsch