-
Notifications
You must be signed in to change notification settings - Fork 97
Add the Content Summarization Experiment base #136
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
Merged
jeffpaul
merged 27 commits into
WordPress:develop
from
dkotter:feature/content-summarization-experiment
Jan 13, 2026
Merged
Changes from 1 commit
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
b9917b5
Add the base Summarization Experiment code
dkotter 82bdbcf
Add the base Summarization Ability code
dkotter eb8e4e6
Add the system instructions for the Summarization Ability
dkotter 7544a40
Modify a bit our post context Ability and helper function to return '…
dkotter 47df6e4
Add tests
dkotter 72a97e0
Revert changes to the get-post-details ability and instead remove emp…
dkotter da233f9
Merge branch 'develop' into feature/content-summarization-experiment
dkotter 8bd5d6a
Allow passing in extra data when our system instructions are loaded
dkotter 6824c04
Allow choosing the summary length, from short, medium or long and pas…
dkotter 009f2c0
Dynamically set the expected return length depending on the passed in…
dkotter b56d200
Update tests
dkotter d186657
Pull the content out of the context so we can separate those a bit mo…
dkotter 97ef7f9
Change from a post_id field to a context field. Only add context to o…
dkotter fd1a81c
Update tests
dkotter 42834c7
Merge branch 'develop' into feature/content-summarization-experiment
dkotter a7a64e7
Merge branch 'develop' into feature/content-summarization-experiment
dkotter f720320
Merge branch 'develop' into feature/content-summarization-experiment
jeffpaul 04c543a
Merge branch 'develop' into feature/content-summarization-experiment
dkotter 5a49eae
Merge branch 'feature/content-summarization-experiment' of github.com…
dkotter 2b7b217
Merge branch 'develop' into feature/content-summarization-experiment
dkotter 00fdd9b
Turn off sniff
dkotter c97bd2e
Use xml to deliminate content and context instead of markdown and tri…
dkotter 0ebf538
Fix Plugin Check warnings
dkotter e5f4411
Remove direct access prevention code from helpers.php as it seems to …
dkotter bacc79f
Merge branch 'develop' into feature/content-summarization-experiment
jeffpaul 6e11966
Merge branch 'develop' into feature/content-summarization-experiment
jeffpaul 70f9280
Merge branch 'develop' into feature/content-summarization-experiment
dkotter File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Fix Plugin Check warnings
- Loading branch information
commit 0ebf5389a465b981f83dd2fe0fed137d90c3e0ff
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should clarify the data structure for the model to expect, and make it clear that the
contentis being summarized, while thecontextshould be considered.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense. I've pulled the content out of the context now so those are more obviously separated: d186657
We'll also likely want to make this same change for the Title Generation Experiment but I can do that in a separate PR