Skip to content

Conversation

@ASCE-D
Copy link
Contributor

@ASCE-D ASCE-D commented Nov 20, 2025

No description provided.

@ASCE-D ASCE-D requested a review from Dsantra92 November 20, 2025 14:39
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 20, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch codegen-pipeline

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@Dsantra92 Dsantra92 left a comment

Choose a reason for hiding this comment

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

Minor changes requested

commit = r["commit_id"]
prob = r["problem_id"]
mapping.setdefault(repo, []).append((commit, prob))
if task == "codegen":

Choose a reason for hiding this comment

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

Instead of using task, can you just check if it's URL or repo slug?

- base_dir/worktrees/<repo_name>/<worktree_name>
batch_no: number of batch copies to create per (commit,problem)
max_workers: number of threads for parallel repo processing
task: "qa" for legacy format, "codegen" for SWE-bench format

Choose a reason for hiding this comment

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

"qa" is not legacy format.

Do NOT include any text outside the JSON object.
"""

correctness = GEval(

Choose a reason for hiding this comment

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

modify it as qa correcteness?

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots

See analysis details on SonarQube Cloud

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