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

Aperta Adatta ai principianti
  • Qualcosa non va?
#1,912 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 3 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
84/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
cpp
Ambito
testing

Direzione di ricerca

Leggi ReadH5OinaDataTest.cpp, in particolare le righe 1308–1354 e gli altri casi .OocStoreContract in quel file; confronta le posizioni X/Y delle relative fixture con le dimensioni raster e il passo di ciascuna intestazione. Aggiorna la fixture della coda del batch in modo che descriva una griglia completa di 65537 × 1, mantenendo i valori speciali della tupla finale. Esegui OrientationAnalysisOocStoreContracts in una build out-of-core e conferma che il test viene superato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
C++
Stelle
18
Fork
15
Merge medio
3g 35m
PR unite (30g)
25

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di BlueQuartzSoftware/simplnx

Tutte le issue di BlueQuartzSoftware/simplnx

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.