#32618: Aggregation documentation should mention conditional aggregation
--------------------------------------+------------------------------------
Reporter: Simon Willison | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Documentation | Version: 3.2
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
--------------------------------------+------------------------------------
Changes (by Mariusz Felisiak):
* needs_docs: 1 => 0
* type: Bug => Cleanup/optimization
* easy: 0 => 1
* stage: Unreviewed => Accepted
Comment:
There is also an example in the
[https://docs.djangoproject.com/en/3.1/topics/db/aggregation/#cheat-sheet
Cheat sheet]. A small reference should be enough, e.g.
{{{
diff --git a/docs/topics/db/aggregation.txt
b/docs/topics/db/aggregation.txt
index cc6310052a..2e882e2a1a 100644
--- a/docs/topics/db/aggregation.txt
+++ b/docs/topics/db/aggregation.txt
@@ -354,7 +354,7 @@ authors with a count of highly rated books::
>>> Author.objects.annotate(num_books=Count('book'),
highly_rated_books=highly_rated)
Each ``Author`` in the result set will have the ``num_books`` and
-``highly_rated_books`` attributes.
+``highly_rated_books`` attributes. See also :ref:`conditional-
aggregation`.
.. admonition:: Choosing between ``filter`` and ``QuerySet.filter()``
}}}
What do you think?
--
Ticket URL: <https://code.djangoproject.com/ticket/32618#comment:2>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/064.4b5f2dd3156c9dc662eb56ed6c69a8bd%40djangoproject.com.
