Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Sep 11, 2024

Conversation

hawk9821
Copy link
Contributor

@hawk9821 hawk9821 commented Sep 10, 2024

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
@Hisoka-X
Copy link
Member

Thanks @hawk9821

@hailin0 hailin0 merged commit 65c14aa into apache:dev Sep 11, 2024
7 checks passed
@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