Skip to content

Use the latest prompt from the evaluator - #1099

Merged
zetter-rpf merged 1 commit into
mainfrom
update-prompt
Nov 10, 2026
Merged

zetter-rpf merged 1 commit into
mainfrom
update-prompt

Conversation

@zetter-rpf

Copy link
Copy Markdown
Contributor

Status

What's changed?

This is from the combination of:

https://github.com/RaspberryPiFoundation/slm-evaluation/blob/main/corpus/prompt.yml and
https://github.com/RaspberryPiFoundation/slm-evaluation/blob/main/corpus/guidelines.md

We'll likely want to change the prompt again, but this will let us test with the longer prompt that we have run the evaluation against earlier.

@cla-bot cla-bot Bot added the cla-signed label Nov 10, 2026
@zetter-rpf
zetter-rpf marked this pull request as ready for review November 10, 2026 12:35
@macroscopeapp

macroscopeapp Bot commented Nov 10, 2026

Copy link
Copy Markdown

Approvability

Verdict: Would Approve

Macroscope's review found this PR approvable — The prompt now gives more explicit guidance for concise, age-appropriate debugging hints without revealing fixes. The existing feature-gated access, inference integration and data handling remain unchanged.

Needs human approval because:

  • approvability is in preview mode for this repository — Macroscope would have approved this PR outside preview mode

You can add or adjust custom eligibility rules. Learn more.

@github-actions

Copy link
Copy Markdown

Test coverage

94.18% line coverage reported by SimpleCov.
Run: https://github.com/RaspberryPiFoundation/editor-api/actions/runs/42817086100

@raspberrypiherokubot
raspberrypiherokubot temporarily deployed to editor-api-p-update-pro-xe8yyx November 10, 2026 12:40 Inactive
@zetter-rpf
zetter-rpf merged commit 792ff96 into main Nov 10, 2026
10 checks passed
@zetter-rpf
zetter-rpf deleted the update-prompt branch November 10, 2026 15:03

This branch was previously deployed

1 inactive deployment
editor-api-p-update-pro-xe8yyx — aefc4f88 Deployed Nov 10, 2026 by raspberrypiherokubot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants