-
-
Notifications
You must be signed in to change notification settings - Fork 407
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
feat: add support for product quantity and packaging weights in search graphs #8963
Conversation
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.
LGTM, thanks.
return ($title, $unit, $unit2, $allow_decimals); | ||
} | ||
|
||
sub display_scatter_plot ($graph_ref, $products_ref) { |
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.
a POD would be good !
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.
adding one
Co-authored-by: Alex Garel <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #8963 +/- ##
==========================================
+ Coverage 38.16% 46.01% +7.84%
==========================================
Files 59 64 +5
Lines 19079 19818 +739
Branches 4653 4798 +145
==========================================
+ Hits 7282 9119 +1837
+ Misses 10720 9512 -1208
- Partials 1077 1187 +110
... and 33 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Co-authored-by: Alex Garel <[email protected]>
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.
Thanks very much for the Eco-Score, in addition to NOVA
Kudos, SonarCloud Quality Gate passed! |
Discussed in last packagings call: now that we have packaging weights, we could create graphs (histograms or scatter plots) using the packaging weights, so that it's easy to see distributions and view the corresponding products.
Part of