Project

General

Profile

Development #38913

visualisations: retourner 0 si pas de valeur pour une mésure en pourcentage d'une visualisation

Added by Serghei Mihai 7 months ago. Updated 7 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
13 Jan 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Et non None, comme aujourd'hui.

0001-visualisation-return-0.0-for-None-measures-in-json_d.patch View (845 Bytes) Benjamin Dauvergne, 13 Jan 2020 11:27 AM

0001-visualisation-return-0.0-for-None-measures-in-json_d.patch View (835 Bytes) Benjamin Dauvergne, 13 Jan 2020 11:28 AM

0001-visualisation-return-0.0-for-None-measures-in-json_d.patch View (1.02 KB) Benjamin Dauvergne, 13 Jan 2020 03:17 PM

0001-visualisation-return-0.0-for-None-measures-in-json_d.patch View (1.38 KB) Benjamin Dauvergne, 13 Jan 2020 03:42 PM

0001-visualisation-return-0.0-for-None-measures-in-json_d.patch View (1.29 KB) Benjamin Dauvergne, 13 Jan 2020 03:50 PM

0001-tests-add-test-on-json_data-for-None-measures-38913.patch View (1.96 KB) Benjamin Dauvergne, 14 Jan 2020 05:55 PM

0002-engine-force-join-to-right-when-looking-for-members-.patch View (3.1 KB) Benjamin Dauvergne, 14 Jan 2020 05:55 PM

0003-visualisation-return-0.0-for-None-measures-in-json_d.patch View (2.05 KB) Benjamin Dauvergne, 14 Jan 2020 05:55 PM

Associated revisions

Revision f2b6ab95 (diff)
Added by Benjamin Dauvergne 7 months ago

tests: add test on json_data for None measures (#38913)

Revision 5b72a4be (diff)
Added by Benjamin Dauvergne 7 months ago

engine: force join to right when looking for members (#38913)

Revision 5473870d (diff)
Added by Benjamin Dauvergne 7 months ago

visualisation: return 0.0 for None measures in json_data (#38913)

History

#2 Updated by Benjamin Dauvergne 7 months ago

  • Assignee set to Benjamin Dauvergne

#3 Updated by Benjamin Dauvergne 7 months ago

#5 Updated by Serghei Mihai 7 months ago

J'aurais mis le if value is None avant les if isinstance.
Un petit test sinon?

#6 Updated by Valentin Deniaud 7 months ago

J'étais tombé dessus en parallèle et ça avait donné #38847. Ce n'est pas poussé, il faudra penser à le fermer si le fix côté bijoe est jugé préférable et suffisant.

#7 Updated by Serghei Mihai 7 months ago

Pour moi le souci doit être corrigé dans bijoe: le bon format doit être retourné en fonction de la mésure choisie dans la visualisation.

#11 Updated by Serghei Mihai 7 months ago

Au risque de passer pour un chieur, j'aurais aimé un test quand même.

#12 Updated by Benjamin Dauvergne 7 months ago

  • Status changed from Solution proposée to En cours

J'essayais d'être content de mon commit d'abord, je vais faire ça.

#13 Updated by Benjamin Dauvergne 7 months ago

Au passage je corrige encore un truc (forcage d'une jointure droite) sur
l'obtention des valeurs d'une dimension, parce qu'avec les données de test
telles quelles je n'arrivais pas à montrer le souci.

#14 Updated by Serghei Mihai 7 months ago

  • Status changed from Solution proposée to Solution validée

Go

#15 Updated by Valentin Deniaud 7 months ago

Je pousse quand même #38847 ou je ferme ?

#16 Updated by Serghei Mihai 7 months ago

Pour moi cette correction suffit.

#17 Updated by Benjamin Dauvergne 7 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 5473870dc2841b3f9714e88664b8811431466c69
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Mon Jan 13 11:26:50 2020 +0100

    visualisation: return 0.0 for None measures in json_data (#38913)

commit 5b72a4be6d2460b513ccd01f5dfd02e0ca13c779
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Tue Jan 14 17:34:38 2020 +0100

    engine: force join to right when looking for members (#38913)

commit f2b6ab958ae87b200a728577f1c39f232e98adf2
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Tue Jan 14 16:08:59 2020 +0100

    tests: add test on json_data for None measures (#38913)

#19 Updated by Frédéric Péters 7 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF