feat: add namespace explorer and stats - #143
osamashabih6960 wants to merge 1 commit into
Conversation
|
Implemented namespace explorer and statistics support. Changes include:
Testing:
|
codeforstartups
left a comment
There was a problem hiding this comment.
Thanks @osamashabih6960 — a namespace explorer is genuinely useful (issue #128/#131), and the tests pass. Two blockers before this can merge:
- Lint fails —
ruff checkreportsI001(unsorted imports) insrc/dynavec/client.py. Runruff check --fix src tests. client.pychurn is far too large to review safely — the file goes from 574 → 1373 lines (+1059/−251), with every method rewritten. A namespace-explorer feature should be additive (newnamespace_stats()/explorer methods + themodels.py/stores/dynamodb.pyadditions), not a near-total rewrite of the core client. Please scope this down to just the new functionality and avoid reformatting/restructuring unrelated code — otherwise regressions can hide in areas the tests don't cover. If a formatter caused the churn, please revert the unrelated reformatting.
Happy to re-review quickly once it's a focused diff. The models.py + stores/dynamodb.py parts look like the right shape.
|
Hlo sir ple merged my requests |
|
👋 To unblock this one, two things:
The |
|
👋 @osamashabih6960 — checking in on this one. Its been a couple of weeks and
Could you rebase onto latest |
Summary
Adds namespace exploration and statistics support.
Changes
namespace()NamespaceStatsmodelTesting