Skip to content

PERMUT, PERMUTATIONA#60

Open
BrianHung wants to merge 22 commits into
mainfrom
codex/implement-excel-functions-fact,-combin,-combina,-permut,-per
Open

PERMUT, PERMUTATIONA#60
BrianHung wants to merge 22 commits into
mainfrom
codex/implement-excel-functions-fact,-combin,-combina,-permut,-per

Conversation

@BrianHung

Copy link
Copy Markdown
Owner

Summary

  • implement factorial and combinatorics functions
  • wire them into function registry and evaluation
  • test FACT, COMBIN, COMBINA, PERMUT and PERMUTATIONA

Testing

  • cargo test --lib --quiet

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

@BrianHung BrianHung changed the title Add combinatorial math functions FACT, COMBIN, COMBINA, PERMUT and PERMUTATIONA Jul 31, 2025
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 BrianHung added the ready to merge PRs ready to merge label Jul 31, 2025
@BrianHung BrianHung changed the title FACT, COMBIN, COMBINA, PERMUT and PERMUTATIONA fact, combin, combina, permut, permutationa Aug 6, 2025
@BrianHung BrianHung changed the title fact, combin, combina, permut, permutationa PERMUT, PERMUTATIONA Feb 1, 2026
@BrianHung BrianHung force-pushed the codex/implement-excel-functions-fact,-combin,-combina,-permut,-per branch from 455bbfd to e5bddaa Compare February 9, 2026 09:36
- Add MAX_N (1e15) and MAX_K (1e6) early return for #NUM! on huge args
- Check result.is_infinite() in loop and return #NUM!
- Update combina_comprehensive to expect #NUM! for COMBINA(0,3) to match
  existing COMBIN_COMBINA.xlsx round-trip test
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.

5 participants