Skip to content

PRICEDISC, PRICEMAT, YIELDDISC, YIELDMAT, DISC, RECEIVED, INTRATE#57

Closed
BrianHung wants to merge 4 commits into
mainfrom
gkg48s-codex/implement-excel-functions-for-bond-calculations
Closed

PRICEDISC, PRICEMAT, YIELDDISC, YIELDMAT, DISC, RECEIVED, INTRATE#57
BrianHung wants to merge 4 commits into
mainfrom
gkg48s-codex/implement-excel-functions-for-bond-calculations

Conversation

@BrianHung

Copy link
Copy Markdown
Owner

Summary

  • implement discount-oriented financial functions
  • expose new functions in parsing and docs
  • document functions as available
  • add tests for new functionality

Testing

  • cargo test

https://chatgpt.com/codex/tasks/task_e_6889d5a1dc9c832785af85e40b2304cb

@BrianHung BrianHung changed the title Add discount bond financial functions PRICEDISC, PRICEMAT, YIELDDISC, YIELDMAT, DISC, RECEIVED, INTRATE Jul 31, 2025
@BrianHung BrianHung marked this pull request as ready for review July 31, 2025 18:43
@BrianHung

Copy link
Copy Markdown
Owner Author

cursor review
make sure there are comprehensive tests for each function, functions are ticked as available, and properly documented with their own page stub

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@BrianHung

Copy link
Copy Markdown
Owner Author

cursor review
make sure there are comprehensive tests for each function, functions are ticked as available, and properly documented with their own page stub

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no bugs!


@BrianHung

Copy link
Copy Markdown
Owner Author

Consolidating and closing in favor of #54.

@BrianHung BrianHung closed this Jul 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex ready to merge PRs ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant