Skip to content

feat: add comment in dbeaver pr 37564 - #8

Open
EmptyRabbit wants to merge 1 commit into
alibaba:mainfrom
EmptyRabbit:feat/add-positive-sample
Open

feat: add comment in dbeaver pr 37564#8
EmptyRabbit wants to merge 1 commit into
alibaba:mainfrom
EmptyRabbit:feat/add-positive-sample

Conversation

@EmptyRabbit

@EmptyRabbit EmptyRabbit commented Aug 5, 2026

Copy link
Copy Markdown

When using this evaluation set to verify the agent, we discovered an issue where the PR data was not included, affecting the accuracy of the evaluation. Therefore, we hope this case can be added.

PR: dbeaver/dbeaver#37564
Human comment: DBNResource.java‎ comment on line R428
https://github.com/dbeaver/dbeaver/pull/37564/changes/BASE..bc3ac7bb47092a934b344791d8aa3a1c41e6c8f7
img_v3_02149_49bbc576-c099-4d42-9516-a286eba4bfag

@EmptyRabbit EmptyRabbit changed the title feat: add comment in dbeaver mr 37564 feat: add comment in dbeaver pr37564 Aug 5, 2026
@EmptyRabbit EmptyRabbit changed the title feat: add comment in dbeaver pr37564 feat: add comment in dbeaver pr 37564 Aug 5, 2026
@EmptyRabbit
EmptyRabbit force-pushed the feat/add-positive-sample branch from df6e185 to 665a815 Compare August 5, 2026 15:10
@CLAassistant

CLAassistant commented Aug 5, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@stay-foolish-forever

Copy link
Copy Markdown
Contributor

Thanks for the report and for putting together this case!

You're right that a missing human comment in the evaluation data can skew results. We'll verify this case against the source PR (dbeaver/dbeaver#37564, DBNResource.java line R428) shortly, and once we've confirmed the omission we'll merge this PR.

Appreciate you taking the time to dig into the dataset and contribute back.

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.

3 participants