https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342021-07-07T09:49:15ZRedmine Entr’ouvertChrono - Development #55424: statistiques : possibilité de regroupement par extra_datahttps://dev.entrouvert.org/issues/55424?journal_id=3156242021-07-07T09:49:15ZValentin Deniaud
<ul><li><strong>Fichier</strong> <a href="/attachments/55916">0001-api-filter-statistics-by-extra_data-55424.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/55916/0001-api-filter-statistics-by-extra_data-55424.patch">0001-api-filter-statistics-by-extra_data-55424.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>On pouvait déjà filtrer par la valeur de user_was_present, j'ai choisi d'étendre ça plutôt que de faire un autre filtre à côté, en me disant que la possibilité de combiner les deux était inutile (5 menus à la cantine + présent/absent/non renseigné aurait donné 15 point de données par jour).</p>
<p>À discuter, on ne peut pas juste exposer toutes les clés présentes dans extra_data, il y en aurait trop. Donc soit un nouveau champ à l'agenda « Regroupements pour les statistiques », soit se baser sur le champ « Booking check filters » qui renseigne déjà des clés de extra_data. Je suis parti sur l'option 2 pour faire au plus simple, ça devrait couvrir le cas d'usage Publik Famille.</p> Chrono - Development #55424: statistiques : possibilité de regroupement par extra_datahttps://dev.entrouvert.org/issues/55424?journal_id=3170462021-07-15T16:22:22ZPaul Marillonnet
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Ok, petite gymnastique mentale pour se mettre dans le bain, mais c’est joli :)<br />Du détail, mais j’en aurais profité pour inverser les deux <code>for</code> dans la double itération<br /><pre><code class="python syntaxhl"> <span class="k">for</span> <span class="n">bookings</span> <span class="ow">in</span> <span class="n">bookings_by_day</span><span class="p">.</span><span class="n">values</span><span class="p">():</span>
<span class="k">for</span> <span class="n">group</span><span class="p">,</span> <span class="n">data</span> <span class="ow">in</span> <span class="n">bookings_by_group</span><span class="p">.</span><span class="n">items</span><span class="p">():</span>
<span class="n">data</span><span class="p">.</span><span class="n">append</span><span class="p">(</span><span class="n">bookings</span><span class="p">.</span><span class="n">get</span><span class="p">(</span><span class="n">group</span><span class="p">))</span>
</pre><br />parce que c’est (je trouve) la façon intuitive dont on se représente ce que fait ce bout de code — d’abord on prend les réservations de chaque groupe, et on ajoute les données en jour par jour, non ?</p></code></pre> Chrono - Development #55424: statistiques : possibilité de regroupement par extra_datahttps://dev.entrouvert.org/issues/55424?journal_id=3174202021-07-19T11:49:58ZValentin Deniaud
<ul></ul><p>Paul Marillonnet a écrit :</p>
<blockquote>
<p>d’abord on prend les réservations de chaque groupe, et on ajoute les données en jour par jour, non ?</p>
</blockquote>
<p>Yep et ça induit même une petite simplification, poussé ça sur la branche.</p> Chrono - Development #55424: statistiques : possibilité de regroupement par extra_datahttps://dev.entrouvert.org/issues/55424?journal_id=3189022021-07-26T08:08:31ZValentin Deniaud
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>commit c0e2726e678f827ad26dff7dc2d61af7e08a154a
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Jul 7 11:30:56 2021 +0200
api: filter statistics by extra_data (#55424)</pre> Chrono - Development #55424: statistiques : possibilité de regroupement par extra_datahttps://dev.entrouvert.org/issues/55424?journal_id=3204572021-08-05T21:17:59ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul>