Skip to content

add missing docstrings to print_writeout and get_sample_size in evaluator_utils.py#3910

Open
RavSinghChandan wants to merge 1 commit into
EleutherAI:mainfrom
RavSinghChandan:docs/add-missing-docstrings-evaluator-utils
Open

add missing docstrings to print_writeout and get_sample_size in evaluator_utils.py#3910
RavSinghChandan wants to merge 1 commit into
EleutherAI:mainfrom
RavSinghChandan:docs/add-missing-docstrings-evaluator-utils

Conversation

@RavSinghChandan

Copy link
Copy Markdown

print_writeout and get_sample_size in lm_eval/evaluator_utils.py are public functions used during evaluation but have no docstrings, making their purpose and expected inputs unclear without reading the implementation.

  • print_writeout – logs the context prompt and request string for the first document in a task, useful for debugging.
  • get_sample_size – resolves a fractional or absolute sample limit into a concrete document count.

Added Google-style docstrings with Args and Returns sections matching the style used for find_test_root and run_task_tests already in this file.

No logic changes.

@CLAassistant

CLAassistant commented Jul 4, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@RavSinghChandan

Copy link
Copy Markdown
Author

@googlebot I signed it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants