Skip to content

Submission/conpara v216#178

Open
MohamedMady19 wants to merge 2 commits into
liamdugan:mainfrom
MohamedMady19:submission/conpara-v216
Open

Submission/conpara v216#178
MohamedMady19 wants to merge 2 commits into
liamdugan:mainfrom
MohamedMady19:submission/conpara-v216

Conversation

@MohamedMady19

Copy link
Copy Markdown

No description provided.

@MohamedMady19

Copy link
Copy Markdown
Author

Hi @liamdugan, please feel free to process this when possible.

@MohamedMady19

Copy link
Copy Markdown
Author

Hi @liamdugan, We kindly remind you to process this submission as it's necessary for our paper. Thank you!

@github-actions

Copy link
Copy Markdown

It looks like this eval run failed. Please check the workflow logs to see what went wrong, then push a new commit to your PR to rerun the eval.

@MohamedMady19

Copy link
Copy Markdown
Author

@liamdugan , the evaluation is failing due to a GitHub Actions security restriction on pull_request_target with fork PRs:

"Refusing to check out fork pull request code from a pull_request_target' workflow"

This is not a problem with our submission files. The fix requires adding allow-unsafe-pr-checkout: true to the actions/checkout step in leaderboard-submission.yml, or alternatively merging and running evaluation from main.

Our previous PRs (#172) went through fine — this appears to be a recent GitHub Actions policy change. Happy to help debug if needed. Thank you!

I would very much appreciate merging #172 with v2.14b to the leaderboard until this is solved. Thank you.

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.

1 participant