Skip to content

[issue-6025] [P-SDK] fix: mapping callables not reading task output#6026

Merged
alexkuzmik merged 2 commits into
comet-ml:mainfrom
kriogenia:kriogenia/issue-6025-fix-mapping-callables-not-reading-task-output
Apr 24, 2026
Merged

[issue-6025] [P-SDK] fix: mapping callables not reading task output#6026
alexkuzmik merged 2 commits into
comet-ml:mainfrom
kriogenia:kriogenia/issue-6025-fix-mapping-callables-not-reading-task-output

Conversation

@kriogenia
Copy link
Copy Markdown
Contributor

Details

Changes create_scoring_input to pass the whole mapping_inputs to callables so
they can be used to complex extractions from the task outputs.

Change checklist

Issues

AI-WATERMARK

AI-WATERMARK: [no]

Testing

Adapted the unit test to also check that it works for task_output

Documentation

No need

@github-actions github-actions Bot added python Pull requests that update Python code tests Including test files, or tests related like configuration. Python SDK labels Apr 1, 2026
@kriogenia kriogenia force-pushed the kriogenia/issue-6025-fix-mapping-callables-not-reading-task-output branch from 9930f52 to ac45d85 Compare April 1, 2026 12:15
@kriogenia kriogenia changed the title [issue-6025] fix: mapping callables not reading task output [issue-6025] [P-SDK] fix: mapping callables not reading task output Apr 1, 2026
@kriogenia kriogenia marked this pull request as ready for review April 1, 2026 12:15
@kriogenia kriogenia requested a review from a team as a code owner April 1, 2026 12:15
@kriogenia kriogenia force-pushed the kriogenia/issue-6025-fix-mapping-callables-not-reading-task-output branch from ac45d85 to 3f86579 Compare April 1, 2026 12:16
@kriogenia kriogenia force-pushed the kriogenia/issue-6025-fix-mapping-callables-not-reading-task-output branch from 3f86579 to bb705a5 Compare April 1, 2026 13:20
Copy link
Copy Markdown
Collaborator

@alexkuzmik alexkuzmik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution! Sorry, it took a long time. The team was focused on Opik 2.0 release.
Approved.

@alexkuzmik alexkuzmik merged commit 9eb4137 into comet-ml:main Apr 24, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Python SDK python Pull requests that update Python code tests Including test files, or tests related like configuration.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: create_scoring_functions not using task_output with callables

2 participants