docs(quote): refresh option Greek descriptions on SecurityCalcIndex - #586
Merged
Merged
Conversation
Update the delta/gamma/theta/vega/rho field docs on SecurityCalcIndex (calc_indexes) across all six SDK layers to match the Calc Index interface documentation. theta is now documented as a per-day value — the server divides the raw annualized value by 365, so the returned theta is already daily (previously the docs said the raw value was annualized by 252 and had to be divided by 252). vega/rho keep the /100 note. Doc comments only; the C header and Node index.d.ts are regenerated. No field or behaviour changes.
This was referenced Sep 3, 2026
Merged
hogan-yuan
marked this pull request as draft
September 3, 2026 10:50
hogan-yuan
marked this pull request as ready for review
September 16, 2026 06:54
hogan-yuan
added a commit
to longbridge/developers
that referenced
this pull request
Sep 16, 2026
…1241) Update the delta / gamma / theta / vega / rho response-field descriptions on the Calc Index (calc_indexes) interface across all three languages (zh-CN / zh-HK / en). Key change: **theta now documents the per-day semantics** — the raw value has been divided by 365 on the server to convert to a daily value (previously the docs incorrectly said divide by 100). delta/gamma get full descriptions; vega/rho reworded to the /100 form. Source of truth: the 期权希腊字母 spec table. Matches the SDK (longbridge/openapi#586), MCP, and CLI changes.
hogan-yuan
added a commit
to longbridge/longbridge-mcp
that referenced
this pull request
Sep 16, 2026
…ation (#142) ## Changes - **calc_indexes**: the Calc Index API's theta was fixed server-side to return a per-day value (raw annualized value divided by 365 on the server), so calc_indexes no longer divides theta by 252 — that would double-normalize. Extracted the Greek normalization into `normalize_greeks` (vega/100, rho/100; theta as-is). - **option_quote**: the `option_quote` interface does not carry Greeks (they come from the Calc Index interface), so it previously showed the Greek fields but returned no data. It now also fetches delta/gamma/theta/vega/rho via `calc_indexes` for the same symbols and merges them into each quote (best-effort — a Greek lookup failure leaves the quotes unchanged). Matches longbridge/openapi#586 and the CLI change. build + clippy pass. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
hogan-yuan
added a commit
to longbridge/longbridge-terminal
that referenced
this pull request
Sep 16, 2026
…y value) (#315) The Calc Index API's theta was fixed server-side to return a per-day value (the raw annualized value is now divided by 365 on the server), so the CLI must no longer divide it by 252 — doing so under-reported theta. Format theta with `fmt_decimal` like the other raw Greeks; vega/rho keep their /100 normalization. Removed the now-unused `fmt_decimal_div252` helper. Matches longbridge/openapi#586 and the MCP change. build + clippy pass. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update the delta / gamma / theta / vega / rho field descriptions on
SecurityCalcIndex(calc_indexes) across all six SDK layers to match the Calc Index interface docs.Key change: theta is now a per-day value — the server divides the raw annualized theta by 365, so the returned value is already daily. Previously the docs said the raw value was annualized (×252) and had to be divided by 252. vega/rho keep the /100 note.
Doc comments only; C header and Node
index.d.tsregenerated. No field or behaviour changes. clippy --all --all-features and cargo fmt --check pass.🤖 Generated with Claude Code