Consolidate Chinese content and add financial analysis modules #2
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
| for period in range(1, periods + 1): | ||
| cash_flow = coupon if period < periods else coupon + face_value | ||
| pv = cash_flow / denom ** period | ||
| total += period * (period + 1) * pv | ||
| return total / (price * denom * denom) |
There was a problem hiding this comment.
convexity() accumulates period-based weights but returns total / (price * denom * denom) without dividing by frequency**2. Macaulay convexity in years requires scaling by f² because t counts coupon periods, so when frequency ≠ 1 (e.g., semiannual bonds with frequency=2) this function overstates convexity by frequency²—about 4× too high in that case—leading to materially incorrect interest-rate sensitivity estimates.
Useful? React with 👍 / 👎.
Summary
Chinese_Content.mdfile for centralized referencefinancial_programspackage with option pricing, portfolio analytics, and fixed income utilitiesTesting
https://chatgpt.com/codex/tasks/task_e_68e7bc9c13e48321ab862b5097863187