Projet

Général

Profil

Development #38913

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

Ajouté par Serghei Mihai il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
13 janvier 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Et non None, comme aujourd'hui.


Fichiers

Révisions associées

Révision f2b6ab95 (diff)
Ajouté par Benjamin Dauvergne il y a environ 4 ans

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

Révision 5b72a4be (diff)
Ajouté par Benjamin Dauvergne il y a environ 4 ans

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

Révision 5473870d (diff)
Ajouté par Benjamin Dauvergne il y a environ 4 ans

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

Historique

#2

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

  • Assigné à mis à Benjamin Dauvergne
#3

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

#5

Mis à jour par Serghei Mihai il y a environ 4 ans

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

#6

Mis à jour par Valentin Deniaud il y a environ 4 ans

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

Mis à jour par Serghei Mihai il y a environ 4 ans

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

Mis à jour par Serghei Mihai il y a environ 4 ans

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

#12

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

  • Statut changé de Solution proposée à En cours

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

#13

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

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

Mis à jour par Serghei Mihai il y a environ 4 ans

  • Statut changé de Solution proposée à Solution validée

Go

#15

Mis à jour par Valentin Deniaud il y a environ 4 ans

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

#16

Mis à jour par Serghei Mihai il y a environ 4 ans

Pour moi cette correction suffit.

#17

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

  • Statut changé de Solution validée à 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

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF