chore(llmobs): [MLOB-2093] remove deprecated parameters arg from annotate method

[ncybul]

Deprecates the parameters arg for the LLMObs annotation method in favor of metadata.

I verified this change by running the following code:

@workflow
def run():
    LLMObs.annotate(
        input_data="hello",
        output_data="bye",
        parameters={"example": True},
    )
    return None

run()

Which produced the following error as expected: TypeError: LLMObs.annotate() got an unexpected keyword argument 'parameters'.

Replacing parameters with metadata resolves the issue:

Checklist

• PR author has checked that all the criteria below are met
• The PR description includes an overview of the change
• The PR description articulates the motivation for the change
• The change includes tests OR the PR description describes a testing strategy
• The PR description notes risks associated with the change, if any
• Newly-added code is easy to change
• The change follows the library release note guidelines
• The change includes or references documentation updates if necessary
• Backport labels are set (if applicable)

Reviewer Checklist

• Reviewer has checked that all the criteria below are met
• Title is accurate
• All changes are related to the pull request's stated goal
• Avoids breaking API changes
• Testing strategy adequately addresses listed risks
• Newly-added code is easy to change
• Release note makes sense to a user of the library
• If necessary, author has acknowledged and discussed the performance implications of this PR as reported in the benchmarks PR comment
• Backport labels are set in a manner that is consistent with the release branch maintenance policy

[github-actions]

CODEOWNERS have been resolved as:

releasenotes/notes/remove-llmobs-parameters-annotation-97406f9cece3fac4.yaml  @DataDog/apm-python
ddtrace/llmobs/_constants.py                                            @DataDog/ml-observability
ddtrace/llmobs/_llmobs.py                                               @DataDog/ml-observability
tests/llmobs/_utils.py                                                  @DataDog/ml-observability
tests/llmobs/test_llmobs.py                                             @DataDog/ml-observability
tests/llmobs/test_llmobs_decorators.py                                  @DataDog/ml-observability
tests/llmobs/test_llmobs_service.py                                     @DataDog/ml-observability

[datadog-dd-trace-py-rkomorn]

Datadog Report

Branch report: nicole-cybul/remove-parameter-arg-from-annotate
Commit report: dee2c2d
Test service: dd-trace-py

✅ 0 Failed, 130 Passed, 1468 Skipped, 4m 45.52s Total duration (36m 17.1s time saved)

[pr-commenter]

Benchmarks

Benchmark execution time: 2025-01-28 18:35:17

Comparing candidate commit f67b03e in PR branch nicole-cybul/remove-parameter-arg-from-annotate with baseline commit 48c6547 in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 394 metrics, 2 unstable metrics.

[Kyle-Verhoog]

just remember to change the base branch to 3.x-staging 😄