ToArrowRecordBatches accumulates arrays from previous batches, producing invalid record batches
I maintainer di solito rispondono entro 2 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 62/100
Direzione di ricerca
Il bug si trova in ToArrowRecordBatches() in src/Microsoft.Data.Analysis/DataFrame.Arrow.cs, dove la lista arrays è dichiarata al di fuori del ciclo di suddivisione in batch e non viene mai reimpostata. Leggi quel ciclo, quindi verifica la riproduzione rispetto a DataFrame.FromArrowRecordBatch. È completato quando ogni batch contiene solo i propri array di colonna e il batch 1 della riproduzione viene reimportato come frame di una riga e una colonna contenente 42. Il test da estendere si trova nei test Microsoft.Data.Analysis per i round-trip Arrow.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
DataFrame.ToArrowRecordBatches() accumulates arrays from previous batches. A one-column DataFrame exports its second record batch with one schema field but two arrays, including the first batch's stale array.
This invalid batch cannot be imported using DataFrame.FromArrowRecordBatch(), which throws ArgumentOutOfRangeException.
Reproduced on Microsoft.Data.Analysis 0.23.0, Apache.Arrow 14.0.2, Windows x64, .NET 10. The reproduction uses only public, high-level DataFrame APIs and is independent of Arrow string null-count bugs (https://github.com/dotnet/machinelearning/issues/7800 and https://github.com/dotnet/machinelearning/issues/7801).
Reproduction
Create a console app, add Microsoft.Data.Analysis version 0.23.0, and run:
using System;
using System.Linq;
using Microsoft.Data.Analysis;
var numbers = new Int64DataFrameColumn("numbers");
numbers.AppendMany(42, 268_304_384);
var frame = new DataFrame(numbers);
var batches = frame.ToArrowRecordBatches();
foreach (var (i, batch) in batches.Index())
{
Console.WriteLine(
$"Batch {i}: rows={batch.Length}; fields={batch.Schema.FieldsList.Count}; arrays={batch.Arrays.Count()}; array lengths=[{string.Join(", ", batch.Arrays.Select(array => array.Length))}]");
try
{
var roundTrip = DataFrame.FromArrowRecordBatch(batch);
Console.WriteLine($"Reimported batch {i}: rows={roundTrip.Rows.Count}; columns={roundTrip.Columns.Count}");
}
catch (ArgumentOutOfRangeException e)
{
Console.WriteLine($"Reimporting batch {i}: {e.GetType().Name}: {e.Message}");
}
}
Requires slightly over 2 GB for the numeric buffers and validity bitmaps. The row count naturally creates two numeric buffers and forces two record batches.
Actual output
Batch 0: rows=268304383; fields=1; arrays=1; array lengths=[268304383]
Reimported batch 0: rows=268304383; columns=1
Batch 1: rows=1; fields=1; arrays=2; array lengths=[268304383, 1]
Reimporting batch 1: ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index')
Expected behavior
Each batch should contain exactly one array matching its single schema field. Batch 1 should contain only its one-row array, and reimport should produce a one-row, one-column DataFrame containing 42.
Cause
DataFrame.ToArrowRecordBatches() creates the arrays list outside its batching loop and continually appends each batch's arrays without resetting that list.
Each record batch should receive a fresh per-batch collection containing only the current batch's column arrays. Also ensure that continuing enumeration cannot mutate an already yielded batch.
- Lingua principale
- C#
- Stelle
- 9.4k
- Fork
- 2k
- Merge medio
- 2g 23h
- PR unite (30g)
- 40
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di dotnet/machinelearning
-
LlamaTokenizer encodes some runs of repeated characters differently from SentencePieceForse già presa @Laurianti l’ha presa 17 giorni fa. Apertauntriaged
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
dotnet/machinelearning#7751 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
BertTokenizer merges words separated only by `\n`, `\t` or `\r`Forse già presa @rosebyte l’ha presa 25 giorni fa. Apertaarea-Tokenizers green
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
dotnet/machinelearning#7724 · 2 commenti · 1 assegnatario ·
I maintainer di solito rispondono entro 2 giorni
-
Comment in example code not matching docstring for TimeSeriesCatalog.DetectSeasonality MethodApertaarea-TimeSeries documentation untriaged
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
dotnet/machinelearning#7205 ·
I maintainer di solito rispondono entro 2 giorni
-
area-ONNX documentation Priority:3
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
dotnet/machinelearning#5894 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
Add a .vsconfig fileApertaarea-Infrastructure Priority:2
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
dotnet/machinelearning#3260 · 2 commenti · 1 reazione ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di dotnet/machinelearning
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
qew21/Genshin-Subtitles#45 ·
-
bug priority: P2
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
space-wizards/RobustToolbox#7158 ·
I maintainer di solito rispondono entro 1 giorno
-
Parameterless constructor is not a must-provided by user when a struct has a field initializerAperta:watch: Not Triaged dotnet-csharp/svc fundamentals/subsvc
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Wrong values of Kernel32.FileShare.Delete and Kernel32.FileAttributes.PosixSemanticsForse già presa @ElectLive l’ha presa oggi. Apertabug good first issue help wanted
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 83/100
ForNeVeR/TruePath#241 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno