V22 - Venez tester sur la beta SVP !

a marqué ce sujet comme résolu.

Il faudrait ptet plus de rigueur en QA en anticipant les cas chiants ? (typiquement pour les mentions, il y a plein de petits tests qui passe pas mais qui aurait pu être vu avant je pense)

Eskimon

Joker sur celle là, je pense qu’ il y a pas eu de PR pour ça, c’est juste une évolution côté zMarkdown qu’on a décidé de tester :) (et du coup, vhf l’a laissé passé sans QA :'( )

C’est typiquement à ce genre de tickets que je pensais en proposant cette idée. Cela dit, je n’ai pas assez suivi le développement récent pour savoir si ça aiderait vraiment. Mais je confirme que quand j’étais DTC, y’avait envion 50 % de tickets qu’on aurait pu déployer continuement, et 20 % qui nécessitaient soit des interventions manuelles, soit qui étaient suffisamment casse-gueule pour qu’on les QA longuement sur la bêta – les 30 % restant, c’est les tickets créés en QA les 20 % de gros tickets.

SpaceFox

Et je me répète, puisque personne à réagi là bas, que la release actuelle est encore une preuve que c’est assez casse gueule. Je vois assez difficilement comment faire passer des PR en prod et d’autres non sans générer des situations bizarres ou le-bug-est-en-prod-mais-réglé-par-une-PR-qui-casse-tout-donc-on-doit-attendre-la-prochaine-release ou autres. Une autre solution, c’est de mettre les PR "chiantes" en attente pour la release, mais alors, c’est rebase permanents et compagnie (d’ailleurs, bonne chance à celui qui va s’occuper de la ZEP-13 ;) ). Sauf si vous avez une autre idée en tête :)

+0 -0

(et du coup, vhf l’a laissé passé sans QA :'( )

pierre_24

Sur le coup, l’attention me touche mais c’est faux. :)

La QA a été faite quand on a ajouté le paramètre, sur la PR qui ajoute le paramètre et le comportement qui va avec le paramètre. Tu noteras également que la PR dont tu donnes le lien est sur dev, pas sur release. Elle est postérieure à la beta actuelle.

Et je vais pas balancer la personne qui a fait la QA de la PR implémentant les notifications de ping dans notre codebase, parce que ça sert à rien et que je lui en veux pas. (Faut pas trop rejeter la faute quand même, si tu veux mon avis.)

+1 -0

J’ai une petite remarque concernant le nouveau moteur de recherche : Par exemple, je cherche ’Nazara’, je trouve le bon résultat, pas de problème mais quand je tape ’nazara’, je trouve les même résultats mais rangés très différemment, je comprends pas trop la logique, c’est sensible à la casse mais j’ai pourtant les mêmes résultats que quand j’écris avec une majuscule sauf que c’est pas ordonné pareille. ^^

+2 -0

T’as tout à fait raison : nazara vs Nazara. Ce qui est bizarre, parce que Elasticsearch est configuré chez nous pour être case insensitive (sauf rares exceptions). D’ailleurs, à priori, Elasticsearch comprend la même chose:

tokens utilisés par Elasticsearch (à droite) à partir de "Nazara" et "nazara"

Je vais essayer de creuser l’histoire ce soir si j’ai vhf à mes côtés :-)

+0 -0

Rien me frappe spécialement:

  1
  2
  3
  4
  5
  6
  7
  8
  9
 10
 11
 12
 13
 14
 15
 16
 17
 18
 19
 20
 21
 22
 23
 24
 25
 26
 27
 28
 29
 30
 31
 32
 33
 34
 35
 36
 37
 38
 39
 40
 41
 42
 43
 44
 45
 46
 47
 48
 49
 50
 51
 52
 53
 54
 55
 56
 57
 58
 59
 60
 61
 62
 63
 64
 65
 66
 67
 68
 69
 70
 71
 72
 73
 74
 75
 76
 77
 78
 79
 80
 81
 82
 83
 84
 85
 86
 87
 88
 89
 90
 91
 92
 93
 94
 95
 96
 97
 98
 99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
{
  "detail": {
    "custom_analyzer": true,
    "charfilters": [
      {
        "name": "html_strip",
        "filtered_text": [
          "Nazara nazara"
        ]
      }
    ],
    "tokenizer": {
      "name": "custom_tokenizer",
      "tokens": [
        {
          "token": "Nazara",
          "start_offset": 0,
          "end_offset": 6,
          "type": "word",
          "position": 0,
          "bytes": "[4e 61 7a 61 72 61]",
          "positionLength": 1
        },
        {
          "token": "nazara",
          "start_offset": 7,
          "end_offset": 13,
          "type": "word",
          "position": 1,
          "bytes": "[6e 61 7a 61 72 61]",
          "positionLength": 1
        }
      ]
    },
    "tokenfilters": [
      {
        "name": "lowercase",
        "tokens": [
          {
            "token": "nazara",
            "start_offset": 0,
            "end_offset": 6,
            "type": "word",
            "position": 0,
            "bytes": "[6e 61 7a 61 72 61]",
            "positionLength": 1
          },
          {
            "token": "nazara",
            "start_offset": 7,
            "end_offset": 13,
            "type": "word",
            "position": 1,
            "bytes": "[6e 61 7a 61 72 61]",
            "positionLength": 1
          }
        ]
      },
      {
        "name": "protect_c_language",
        "tokens": [
          {
            "token": "nazara",
            "start_offset": 0,
            "end_offset": 6,
            "type": "word",
            "position": 0,
            "bytes": "[6e 61 7a 61 72 61]",
            "positionLength": 1
          },
          {
            "token": "nazara",
            "start_offset": 7,
            "end_offset": 13,
            "type": "word",
            "position": 1,
            "bytes": "[6e 61 7a 61 72 61]",
            "positionLength": 1
          }
        ]
      },
      {
        "name": "french_elision",
        "tokens": [
          {
            "token": "nazara",
            "start_offset": 0,
            "end_offset": 6,
            "type": "word",
            "position": 0,
            "bytes": "[6e 61 7a 61 72 61]",
            "positionLength": 1
          },
          {
            "token": "nazara",
            "start_offset": 7,
            "end_offset": 13,
            "type": "word",
            "position": 1,
            "bytes": "[6e 61 7a 61 72 61]",
            "positionLength": 1
          }
        ]
      },
      {
        "name": "french_stop",
        "tokens": [
          {
            "token": "nazara",
            "start_offset": 0,
            "end_offset": 6,
            "type": "word",
            "position": 0,
            "bytes": "[6e 61 7a 61 72 61]",
            "positionLength": 1
          },
          {
            "token": "nazara",
            "start_offset": 7,
            "end_offset": 13,
            "type": "word",
            "position": 1,
            "bytes": "[6e 61 7a 61 72 61]",
            "positionLength": 1
          }
        ]
      },
      {
        "name": "french_keywords",
        "tokens": [
          {
            "token": "nazara",
            "start_offset": 0,
            "end_offset": 6,
            "type": "word",
            "position": 0,
            "bytes": "[6e 61 7a 61 72 61]",
            "keyword": false,
            "positionLength": 1
          },
          {
            "token": "nazara",
            "start_offset": 7,
            "end_offset": 13,
            "type": "word",
            "position": 1,
            "bytes": "[6e 61 7a 61 72 61]",
            "keyword": false,
            "positionLength": 1
          }
        ]
      },
      {
        "name": "french_stemmer",
        "tokens": [
          {
            "token": "nazara",
            "start_offset": 0,
            "end_offset": 6,
            "type": "word",
            "position": 0,
            "bytes": "[6e 61 7a 61 72 61]",
            "keyword": false,
            "positionLength": 1
          },
          {
            "token": "nazara",
            "start_offset": 7,
            "end_offset": 13,
            "type": "word",
            "position": 1,
            "bytes": "[6e 61 7a 61 72 61]",
            "keyword": false,
            "positionLength": 1
          }
        ]
      }
    ]
  }
}

(edit: c’est long, hein ?)

+0 -0

J’ai eu le temps de tout tester comme je l’avais dit, voici les résultats. :)

Sur ce sujet
  • Décalage du span sur le nombre d’abonnées : OK sur PC, mais pas sur mobile (testé avec le mode développeur de Chromium)
  • Bloc des unes sur fond bleu au lieu du blanc : OK
  • Majuscule ayant un impact sur les résultats : impact toujours présent
Sur GitHub
  • #4216 : pour la prochaine version
  • #4217 : n’est en fait pas un bug, mais le comportement voulu, on peut réfléchir à ce comportement, mais rien d’urgent pour cette version
  • #4215 : le correctif n’est pas déployé mais a été mergé, et la QA est OK
  • #4201 : OK
+4 -0

Majuscule ayant un impact sur les résultats : impact toujours présent

Ma faute. Faut que je tombe sur victor sur IRC et que je lui demande d’exécuter des requêtes en mode ?explain. Je vais tenter de faire ça ce soir (au pire je poste les requêtes et victor fait ça quand il a le temps).

pierre_24

Tu peux faire un ticket et on fait passer ça sur la prochaine, c’est pas du tout bloquant pour la release. :)

Sinon pour les requêtes, IRC, MP, email, ici, c’est égal pour moi. Ce genre de truc n’est compréhensible que par toi, et exécutable présentement que par moi, du coup… ;)

[EDIT] Et un grand merci à GCodeur, j’oubliais presque ! Grace à toi la release est désormais imminente, y’aura pas de nouvelle RC.

+2 -0

Bienvenue sur la v22. La MEP a été pénible mais on y est arrivé. Au menu : java OOM - mysql qui crash puis refuse de démarrer - des configs manquantes - des mi-migrations.

Bref, ça, c’est fait. :D

+4 -0
Connectez-vous pour pouvoir poster un message.
Connexion

Pas encore membre ?

Créez un compte en une minute pour profiter pleinement de toutes les fonctionnalités de Zeste de Savoir. Ici, tout est gratuit et sans publicité.
Créer un compte