[ML] Failure in CBoostedTreeTest/testMseIncrementalForTargetDrift
- Lenguaje dominante
- C++
- Estrellas
- 157
- Forks
- 67
- Merge medio
- 17 h 52 min
- PR fusionados (30 d)
- 20
Descripción
The following error was seen on the `feature/incremental-training` branch for a PR build of #2032:
```
[Exception] - critical check errorDecreaseOnNew > 2.0 * errorIncreaseOnOld has failed [0 <= 0]
== [File] - CBoostedTreeTest.cc
== [Line] -918
```
The test could be trivially fixed by changing the assertion on line 918 from `>` to `>=`. However, this comment makes me think this might be a sign of a worse problem with the test:
```
// By construction, the prediction error for the old training data must
// increase because the old and new data distributions overlap and their
// target values disagree. However, we should see proportionally a much
// larger reduction in the new training data predcition error.
```
If the test is supposed to be constructed such that the prediction errors for the old and new data distributions cannot both be 0 then something is wrong in the test setup code.
Another thing to note is that this failure only occurred once out of several CI runs. So this test is not deterministic for a given platform/architecture. At one time we were always using the same random number generator seed for ml-cpp unit tests so that behaviour was consistent. If we are no longer doing that then we need to get the test code to print the random number generator seed for each run so that problems like this can be debugged.
Guía de contribución
Línea de trabajo
Comienza con CBoostedTreeTest.cc alrededor de la línea 918 y reproduce testMseIncrementalForTargetDrift en la rama feature/incremental-training o en el build de PR #2032. Comprueba si la configuración de la prueba puede producir cero errores para ambas distribuciones y si las ejecuciones utilizan una semilla aleatoria reproducible. Se considera terminado cuando el fallo está explicado y la prueba es determinista, con una assertion que coincida con su invariante prevista.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- machine-learning, testing-qa
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100