-
Notifications
You must be signed in to change notification settings - Fork 266
PHPLIB-1736: Add $scoreFusion stage
#1811
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
base: v2.x
Are you sure you want to change the base?
Conversation
| { | ||
| public function testExample(): void | ||
| { | ||
| $pipeline = new Pipeline(); |
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'll bet 20 Bonusly points that the Copilot agent can generate the PHP aggregation on the first try.
Thank you for contributing the PHP library.
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.
That was a positive comment.
In fact, this file is generated, but it is a frame in which we have to write the code with the PHP aggregation builder. This allows us to validate the functioning of the aggregation builder and check the DX.
The tests/Builder/Stage has many examples of this.
The translation logic is very systematic, which means that an AI agent can easily guess what to write based on the expected extended JSON.
This is a suggestion that you complete this piece of code, but we can also take care of it.
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.
Pushed a commit 🤞
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## v2.x #1811 +/- ##
============================================
- Coverage 87.72% 81.68% -6.05%
- Complexity 3195 3198 +3
============================================
Files 424 425 +1
Lines 6296 6305 +9
============================================
- Hits 5523 5150 -373
- Misses 773 1155 +382
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
5ba6a7c to
dc813d1
Compare
Co-authored-by: Jérôme Tamarelle <jerome@tamarelle.net>
Merging this PR will:
$scoreFusionas per its documentation.Context: Working on COMPASS-8939 adding auto-complete support for
$scoreFusionto mongosh.