close
Skip to content

[Feature][Transforms-V2] llm trans support field projection#7621

Merged
hailin0 merged 1 commit into
apache:devfrom
hawk9821:llm_projection
Sep 11, 2024
Merged

[Feature][Transforms-V2] llm trans support field projection#7621
hailin0 merged 1 commit into
apache:devfrom
hawk9821:llm_projection

Conversation

@hawk9821

@hawk9821 hawk9821 commented Sep 10, 2024

Copy link
Copy Markdown
Contributor

Purpose of this pull request

support for specifying which columns of incoming data should be used as input to the LLM, reducing token consumption and eliminating the impact of irrelevant data

Does this PR introduce any user-facing change?

no

How was this patch tested?

UT: LLMRequestJsonTest#testOpenAIProjectionRequestJson
E2E: TestLLMIT#testLLMWithOpenAIColumns

Check list

@hawk9821 hawk9821 force-pushed the llm_projection branch 3 times, most recently from 91a0854 to c5075f7 Compare September 10, 2024 05:44
Comment thread docs/en/transform-v2/llm.md Outdated
@Hisoka-X

Copy link
Copy Markdown
Member

Thanks @hawk9821

@hailin0 hailin0 merged commit 65c14aa into apache:dev Sep 11, 2024
@Hisoka-X Hisoka-X changed the title [Future][Transforms-V2] llm trans support field projection [Feture][Transforms-V2] llm trans support field projection Sep 11, 2024
@Hisoka-X Hisoka-X changed the title [Feture][Transforms-V2] llm trans support field projection [Feature][Transforms-V2] llm trans support field projection Sep 11, 2024
@hawk9821 hawk9821 deleted the llm_projection branch September 12, 2024 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants