Skip to content

Use llm for clustering#1302

Merged
bohou-aryn merged 1 commit into
mainfrom
llmcluster
May 15, 2025
Merged

Use llm for clustering#1302
bohou-aryn merged 1 commit into
mainfrom
llmcluster

Conversation

@bohou-aryn
Copy link
Copy Markdown
Collaborator

This use llm to form groups by sampling through the dataset, at the same time reuse the previous llm clustering method to find group.

"""


class AggregateCollect(Node):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What's the difference between this and AggregateCount?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

that's for collectting all the values and merge together as a string.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Okay, can merge once the tests pass.

@bohou-aryn bohou-aryn force-pushed the llmcluster branch 2 times, most recently from 7583da1 to 0ee5f31 Compare May 15, 2025 21:33
This use llm to form groups by sampling through the dataset, at the same
time reuse the previous llm clustering method to find group.
@bohou-aryn bohou-aryn merged commit b91c2d9 into main May 15, 2025
11 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants