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

Fix: Allow Issue Matching across org/repos #61

Merged

Conversation

sshivaditya2019
Copy link
Collaborator

Resolves #60

  • Allows Issue Matching outside repos and orgs.
  • New pg func for handling the issue matching

@sshivaditya2019
Copy link
Collaborator Author

src/handlers/issue-matching.ts Show resolved Hide resolved
supabase/config.toml Outdated Show resolved Hide resolved
current_id: currentId,
query_embedding: embedding,
threshold,
top_k: 5,
Copy link
Member

Choose a reason for hiding this comment

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

Why 5?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It could be any value. Should I add this to config instead ?

Copy link
Member

Choose a reason for hiding this comment

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

Probably not but was just curious why you chose 5 but based on brief research it seems appropriate for our usecase.

@@ -114,6 +114,14 @@ export function createMockAdapters(context: Context) {

return [];
}),
findSimilarIssuesToMatch: jest.fn(async (params: { markdown: string; threshold: number; currentId: string }) => {
if (params.currentId === "task_complete") {
return [{ id: "similar3", similarity: 0.98 }];
Copy link
Member

Choose a reason for hiding this comment

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

Why similar3

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It doesn't matter, it could be any value since we are mocking the GraphQL response anyway, otherwise it would filtered by the node_id

@sshivaditya2019 sshivaditya2019 merged commit 23a5562 into ubiquity-os-marketplace:development Dec 18, 2024
2 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.

Issue Matching Fails, with invalid payload object.
3 participants