Python icon indicating copy to clipboard operation
Python copied to clipboard

Add doctests and refactor Monte Carlo estimation functions

Open akiels opened this issue 8 months ago • 1 comments

Contributes to #9943

This pull request refactors and improves the implementation of the Monte Carlo estimation functions:

  • Refactors functions to return values instead of printing, improving testability.
  • Adds missing type hints.
  • Renames function function_to_integrate to the more descriptive semicircle_function.
  • Clarifies docstrings.
  • Adds doctests for each function:
    • pi_estimator
    • area_under_curve_estimator
    • area_under_line_estimator_check
    • pi_estimator_using_area_under_curve

All doctests pass using python3 -m doctest -v path/to/file.py

akiels avatar May 01 '25 10:05 akiels

UNSUBSCRIBE

---- Replied Message ---- | From | @.> | | Date | 05/01/2025 18:09 | | To | TheAlgorithms/Python @.> | | Cc | Subscribed @.***> | | Subject | [TheAlgorithms/Python] Add doctests and refactor Monte Carlo estimation functions (PR #12703) |

Contributes to #9943

This pull request refactors and improves the implementation of the Monte Carlo estimation functions:

Refactors functions to return values instead of printing, improving testability. Adds missing type hints. Renames function function_to_integrate to the more descriptive semicircle_function. Clarifies docstrings. Adds doctests for each function: pi_estimator area_under_curve_estimator area_under_line_estimator_check pi_estimator_using_area_under_curve

All doctests pass using python3 -m doctest -v path/to/file.py

You can view, comment on, or merge this pull request online at:

https://github.com/TheAlgorithms/Python/pull/12703

Commit Summary bcce276 Refactor Monte Carlo estimation functions and add doctest 87a5587 cleanup: Remove unused script stub File Changes

(1 file)

Mmaths/monte_carlo.py (192) Patch Links: https://github.com/TheAlgorithms/Python/pull/12703.patch https://github.com/TheAlgorithms/Python/pull/12703.diff

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.***>

alex2998668174 avatar May 02 '25 08:05 alex2998668174