Support dashed arguments for benchmark args

[sjmonson]

Summary

For all arguments in BenchmarkGenerativeTextArgs support dashed versions of the name during validation to match the CLI. Additionally alias the argument data_request_formatter to request_type.

Details

Alias are configured so that if both options are specified, the original name takes precedence. E.g. the order of precedence is max_seconds > max-seconds.

For data_request_formatter the request type alias takes precedence. E.g. request_type > request-type > data_request_formatter. Changed this because it caused precedence issues with CLI arg. Order is now data_request_formatter > data-request-formatter > request_type > request-type.

Test Plan

The following scenario file can be used to test. Modify which fields are uncommented to test precedence.

target: <hostname>
#request_type: text_completions
#request-type: text_completions
data_request_formatter: text_completions
profile: concurrent
rate: 2
max_seconds: 30
data:
  prompt_tokens: 256
  output_tokens: 128

Related Issues

• Resolves #

---

• "I certify that all code in this PR is my own, except as noted below."

Use of AI

• Includes AI-assisted code completion
• Includes code generated by an AI application
• Includes AI-generated tests (NOTE: AI written tests should have a docstring that includes ## WRITTEN BY AI ##)

[jaredoconnell]

This seems fine. This may cause some fragmentation in usage, but this doesn't seem like something that would be a maintenance burden.