Skip to content

Fix/tauric api contract - #8

Open
nguyenhieutrung97 wants to merge 2 commits into
TauricResearch:mainfrom
nguyenhieutrung97:fix/tauric-api-contract
Open

nguyenhieutrung97 wants to merge 2 commits into
TauricResearch:mainfrom
nguyenhieutrung97:fix/tauric-api-contract

Conversation

@nguyenhieutrung97

Copy link
Copy Markdown

Bug

All data_market_* tools fail on every call (quotes/prices/news → "failed").

Root cause

The Tauric data API contract changed after v0.1.1 shipped:

  • history no longer takes period (only start/end/interval, ≤366 days/call)
  • news takes start/end instead of since/until
  • bars are served in data, batch quotes in a data list
  • change is now {absolute, percent}; quotes are stamped with price_time

Fix

  • pricing: turn period/session windows into dates client-side; page back
    via coverage.continue_end; read bars from data (old bars kept)
  • quotes: parse the data list of quote nodes (old shape kept); map the
    new change object; as_of from price_time
  • news: send start/end
  • articles: Tauric ids are strings now; renumber tolerates them

Verified live

quotes (ETH/BTC/NVDA), prices (1mo bars), news search — all return data.
Our ETH daily job: 6/6 tool calls OK, was 0/8 before.

The data tools failed on every call: the server now takes concrete
start/end windows instead of a 'period' preset on history (at most 366
days a call), a 'start'/'end' pair instead of 'since'/'until' on news,
serves bars and batch quotes in a 'data' container, nests the day
change as {absolute, percent} and stamps quotes with 'price_time'.

- pricing: turn period and session counts into dates client-side, page
  back by coverage.continue_end as before, read bars from data or bars
- quotes: parse the data list of quote nodes (old quotes map kept)
- news: send start/end
- articles: Tauric ids are strings now; renumber tolerates them

Verified live: quotes (ETH/BTC/NVDA), prices (1mo bars), news — all
return data; the ETH daily job's run went 6/6 tool calls OK.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant