Skip to content

feat(registry): add SN62 Ridges TaoMarketCap candle-chart data-artifact (#4069)#4994

Closed
ultrahighsuper wants to merge 1 commit into
JSONbored:mainfrom
ultrahighsuper:feat/sn62-ridges-candle-chart
Closed

feat(registry): add SN62 Ridges TaoMarketCap candle-chart data-artifact (#4069)#4994
ultrahighsuper wants to merge 1 commit into
JSONbored:mainfrom
ultrahighsuper:feat/sn62-ridges-candle-chart

Conversation

@ultrahighsuper

Copy link
Copy Markdown
Contributor

Summary

Registers SN62 Ridges's public OHLCV candle-chart dataset as a data-artifact surface — a no-auth JSON time-series of Ridges's alpha-token price/volume, a dataset the registry did not yet capture.

Surface

  • kind: data-artifact
  • url: https://api.taomarketcap.com/public/v1/subnets/62/candle-chart/
  • provider: taomarketcap (existing)
  • auth_required: false, public_safe: true, authority: community, review.state: community-submitted

Proof of ownership / authority

  • source_urls: the TaoMarketCap developer API docs and tensorplex-labs/subnet-docs/data/62/subnet.json (confirms SN62 Ridges's identity). api.taomarketcap.com is a provider-claimed on-chain indexer; the endpoint is listed in its public OpenAPI with an optional API key (no-auth access supported).

Verified live + public + safe

  • GET https://api.taomarketcap.com/public/v1/subnets/62/candle-chart/HTTP 200, application/json, ~224 KB, no auth.
  • Real payload: an array of hourly OHLCV candles, each scoped to subnet_id: 62, with period_name, timestamp, block_number, open/high/low/close, volume, start_volume, tao_price_usd/eur.
  • Public-safe: scanned the full body for SS58/base58 wallet or hotkey strings → zero matches; price/volume time-series only.

Non-duplication

Distinct from Ridges's existing surfaces — source-repo, the first-party subnet-api (/retrieval/top-agents) and openapi on agent-upload.ridges.ai. This is a different host and a different dataset: an on-chain price/volume time-series, which the first-party agent API does not provide. No open PR targets SN62.

Scope note (relative to #4069)

#4069 asks for SN62 Ridges's public data artifact. This standalone, machine-readable OHLCV price/volume dataset is exactly that, on a reliably-reachable indexer host.

Validation (local)

  • npm run validate:surface -- registry/subnets/ridges.json → passes (4 surfaces)
  • npm run scan:public-safety → passes
  • npm run validate:schemas → passes
  • npx prettier --check registry/subnets/ridges.json → clean
  • git diff --stat → single file, 19-line pure append (no top-level metadata or other surfaces touched)

Template Used

  • Subnet surface

Closes #4069

@superagent-security

Copy link
Copy Markdown

Superagent didn't find any vulnerabilities or security issues in this PR.

@codecov

codecov Bot commented Jul 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.94%. Comparing base (9a55b6f) to head (faccdf4).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4994   +/-   ##
=======================================
  Coverage   97.94%   97.94%           
=======================================
  Files         163      163           
  Lines       19440    19440           
  Branches     7385     7385           
=======================================
  Hits        19040    19040           
  Misses         53       53           
  Partials      347      347           
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@gittensory-orb gittensory-orb Bot added gittensor:feature Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier. gittensor:priority Maintainer-selected Gittensor priority — scores a 1.5x multiplier. Reserved for outstanding work. labels Jul 12, 2026
@gittensory-orb

gittensory-orb Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Caution

🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥

🛑 Gittensory review result - reject/close recommended

Review updated: 2026-07-12 22:50:18 UTC

1 file · 2 AI reviewers · 1 blocker · readiness 100/100 · CI green · clean

🛑 Suggested Action - Reject/Close

Review summary
This PR appends a single new `data-artifact` surface (SN62 Ridges TaoMarketCap OHLCV candle-chart) to the existing `registry/subnets/ridges.json` manifest, matching the netuid (62) and following the exact same shape as the already-merged SN73 MetaHash candle-chart surface in the reference context. It is single-file, append-only (no edits to top-level metadata, curation, or other surfaces), uses `authority: community` / `review.state: community-submitted`, and does not duplicate the subnet's existing first-party `subnet-api`/`openapi` surfaces on a different host (agent-upload.ridges.ai vs api.taomarketcap.com) or the native-chain-promoted `source_repo`/`website_url` fields.

Nits — 5 non-blocking
  • The `curation.gap_notes` array (unchanged in this diff) still lists "No verified standalone static data artifact yet." — worth a follow-up to update since this PR now fills that gap.
  • Trailing-slash URL `.../candle-chart/` is consistent with the sibling MetaHash entry but confirm TaoMarketCap's API is stable on trailing-slash vs non-trailing-slash paths.
  • Consider a small follow-up PR to prune the now-stale gap_notes entry in ridges.json's curation block.
  • registry/subnets/ridges.json:122 uses the generic TaoMarketCap documentation landing page as source evidence; a direct OpenAPI/spec URL or endpoint-specific documentation link would make the source proof stronger and less dependent on JS-rendered docs.
  • registry/subnets/ridges.json:123 links the mutable `main` branch for the Tensorplex subnet identity source; pinning this to a commit would make the evidence reproducible.

Why this is blocked

  • Registry surface review
📋 Copy for AI agents — paste into your coding agent
Fix the following blocker(s) from this PR review:

1. Registry surface review
Signal Result Evidence
Code review ❌ 1 blocker 2 reviewers, synthesized
Linked issue ✅ Linked #4069
Related work ✅ No active overlap found No same-issue or scoped active PR overlap found.
Change scope ✅ 20/20 Low review scope from cached public metadata (1 linked issue).
Validation posture ✅ 25/25 PR body includes validation/test evidence.
Contributor workload ✅ 10/10 Author activity: 56 registered-repo PR(s), 27 merged, 21 issue(s).
Contributor context ✅ Confirmed Gittensor contributor ultrahighsuper; Gittensor profile; 56 PR(s), 21 issue(s).
Gate result ❌ Blocking Repo-configured hard blocker found.
Linked issue satisfaction

Addressed
The PR adds exactly one data-artifact surface to registry/subnets/ridges.json pointing to a real, live, no-auth TaoMarketCap OHLCV endpoint scoped to subnet_id 62, with source_urls documenting the provider docs and the tensorplex subnet-docs identity confirmation, satisfying the issue's ask for a public data artifact for SN62.

Review context
  • Author: ultrahighsuper
  • Role context: outside_contributor
  • Public audience mode: oss maintainer
  • Lane context: Repository is configured for direct PR review.
  • Public profile languages: Python, JavaScript, Rust, TypeScript, Cuda, Ruby
  • Official Gittensor activity: 56 PR(s), 21 issue(s).
  • PR-specific overlap: none found.
Contributor next steps
  • Keep the PR focused and include validation evidence before maintainer review.
Signal definitions
  • Related work = same linked issue, overlapping active PRs, or title/path similarity.
  • Change scope = cached public metadata such as size labels, draft state, and review-burden hints.
  • Validation posture = whether the PR provides enough public validation/test evidence for maintainer review.
  • Contributor workload = public contributor activity and cleanup pressure, not a repo-wide quality failure.
  • Contributor context = public GitHub/Gittensor identity context; non-Gittensor status is not a blocker.
[BETA] Chat with Gittensory

Ask Gittensory a question about this PR directly in a comment — grounded only in the same cached, public-safe facts shown above, never a new claim.

  • @gittensory ask <question> answers contribution-quality Q&A with source citations and freshness.
  • @gittensory chat <question> answers in natural prose from cached decision-pack facts via local inference (maintainer/collaborator; read-only).
  • A plain-language @gittensory mention with a real question is routed to the closest matching read-only command automatically -- no exact syntax required.

Full command reference: https://gittensory.aethereal.dev/docs/gittensory-commands

🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed


💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →.

Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.

  • Re-run Gittensory review

@gittensory-orb

Copy link
Copy Markdown
Contributor

Gittensory is closing this pull request on the maintainer's behalf (Registry surface review). This is an automated maintenance action — to pursue this change, please open a new pull request with the issues resolved. Closed PRs may be analyzed later to improve review accuracy, but they are not automatically reopened or re-reviewed.

@gittensory-orb gittensory-orb Bot closed this Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gittensor:feature Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier. gittensor:priority Maintainer-selected Gittensor priority — scores a 1.5x multiplier. Reserved for outstanding work.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enrich SN62 Ridges — add public data artifact

1 participant