fix: humanize Charts pair 24h volume and TWAP for mixed decimals #1088
No reviewers
Labels
No labels
agent:fix_bugfix
agent:fix_conflicts
agent:fix_security
agent:gap_analysis
agent:implement
agent:implement
agent:implement
agent:open_issues
agent:ready
agent:research
agent:security_audit
agent:verify
architecture
backend
blocker:hybrid
blocker:launch
blocker:limit-orders
blocker:v2
block:log_only
block:security
bug
ci
contracts
correctness
deploy
dev
devops
docs
documentation
duplicate
e2e
enhancement
epic
feature
frontend
functional-completion
gas
good first issue
governance
help wanted
high-risk
hooks
hybrid
indexer
infra
infrastructure
integrators
invalid
launch-blocker
limit-orders
localnet
localterra
low priority
missing-implementation
needs-design
ops
performance
priority
high
priority
medium
product
qa
QA
question
ready
ready
research
scripts
security
security-hardening
smartcontracts
tech-debt
testing
ux
UX
v2
verification
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
code/cl8y-dex-terraclassic!1088
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "fix/564-charts-vol-twap-human"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
volume_base/volume_quotewith the pair row’s decimals (formatPairStatsVolume) and shows Vol (USD) from indexervolume_usd.raw × 10^(d0−d1)+formatPairPrice), not compactTon the raw Decimal. USD OHLC usesformatPairPrice(no compactT)./chartsand/tradescales quote (else base) by pair-leg decimals. Indexer JSON units stay raw.Closes #564
Test plan
make verify-issue-564/chartsUST1/USTR: Vol (UST1) hundreds-not-millions; Vol (USTR)KnotT; TWAP ~111 class not111T—; idle 0 trades →$0/tradeUST1/USTR volume pane is not 1e18 scaleadded 17 commits
main94278ff6- Merge origin/main into fix/564-charts-vol-twap-human.Compare with previous version
mentioned in commit
e9a6270966mentioned in issue #564
mentioned in issue #565
mentioned in issue #573