TEST: Read H5OINA out-of-core batch-tail fixture is not a valid raster grid

Abierto Apto para principiantes
  • ¿Algo no está bien?
#1,912 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 3 días

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
84/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
cpp
Área
testing

Línea de trabajo

Lee ReadH5OinaDataTest.cpp, especialmente las líneas 1308–1354 y los demás casos .OocStoreContract de ese archivo; compara las posiciones X/Y de sus fixtures con las dimensiones ráster y el paso de cada encabezado. Actualiza la fixture del final del lote para que describa una cuadrícula completa de 65537 × 1, conservando los valores especiales de la tupla final. Ejecuta OrientationAnalysisOocStoreContracts en una compilación out-of-core y confirma que la prueba pasa.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Summary

An out-of-core test fixture for Read Oxford Instruments Data (.h5oina) does not describe a valid raster grid, so the test fails with EbsdLib 3.1.3. This test is not in #1908 because the filter has a V&V report.

Location

ReadH5OinaDataTest.cpp#L1308-L1354, test case "genuine HDF5 batch-tail and slab oracle" (tag [.OocStoreContract]).

Problem

The fixture describes a 65537 x 1 grid, but it writes the same X value (0.5) and Y value (0.25) for every point except the last one:

    scan.x.assign(k_ScanTuples, scanIdx == 0 ? 0.5F : 1.5F);
    scan.y.assign(k_ScanTuples, scanIdx == 0 ? 0.25F : 1.25F);

EbsdLib 3.1.3 now builds the scan order from the X/Y positions (BuildScanGridOrderFromPositions, used since #1812). It checks that the positions cover the grid in the header. So execute fails:

ReadH5OinaDataTest.cpp:1354: FAILED: simplnxCheckedResult.valid() ... '-8970 : Attempting to read scan '1' from file '.../read_h5oina_vv_ooc_batch_tail.h5oina' produced an error from the 'H5OINAReader' class.
  Error Code: -101105
  Message: H5OINAReader Error: The 'X'/'Y' datasets of scan '1' do not describe a complete 65537 x 1 raster grid: the X/Y positions span 1 x 1 cells (X 0.5 to 0.75 step 1, Y 0.25 to 0.5 step 1) but the header describes a 65537 x 1 grid.'

Reproduced on macOS with the DREAM3D-NX preset NX-OOC-Qt69-Vtk96-Rel, test OrientationAnalysisOocStoreContracts, run serially.

Why CI does not see it

The test case is hidden and runs only in the out-of-core contracts target. PR #1905 registers that target again in DREAM3D-NX out-of-core builds. After #1905 merges, this test will fail there.

Suggested fix

Write real raster positions in the fixture: x[i] = x0 + i * xStep and y[i] = y0 for a 65537 x 1 grid, with the header step equal to xStep. Keep the special values of the last tuple for the batch-tail check, but not a special position. Check that the other .OocStoreContract cases in this file also use real raster positions. Run OrientationAnalysisOocStoreContracts in an out-of-core build.

Found during review of branch topic/correct_feature_am_updates.

Lenguaje dominante
C++
Estrellas
18
Forks
15
Merge medio
3 d 15 h
PR fusionados (30 d)
27

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de BlueQuartzSoftware/simplnx

Todos los issues de BlueQuartzSoftware/simplnx

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.