ToArrowRecordBatches accumulates arrays from previous batches, producing invalid record batches
Maintainer thường phản hồi trong vòng 2 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 62/100
Hướng nghiên cứu
Lỗi nằm ở ToArrowRecordBatches() trong src/Microsoft.Data.Analysis/DataFrame.Arrow.cs, nơi danh sách arrays được khai báo bên ngoài vòng lặp chia lô và không bao giờ được đặt lại. Hãy đọc vòng lặp đó, sau đó đối chiếu bản tái hiện với DataFrame.FromArrowRecordBatch. Hoàn thành khi mỗi lô chỉ chứa các mảng cột của chính nó và lô 1 của bản tái hiện được nhập lại thành một frame một hàng, một cột chứa 42. Bài kiểm tra cần mở rộng nằm trong các bài kiểm tra Microsoft.Data.Analysis cho các vòng round-trip Arrow.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- C#
- Star
- 9.4k
- Fork
- 2k
- Merge trung bình
- 3 ngày 8 giờ
- Pull request đã merge (30 ngày)
- 41
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của dotnet/machinelearning
-
LlamaTokenizer encodes some runs of repeated characters differently from SentencePieceCó thể đã có người làm @Laurianti đã nhận 15 ngày trước. Đang mởuntriaged
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
dotnet/machinelearning#7751 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
BertTokenizer merges words separated only by `\n`, `\t` or `\r`Có thể đã có người làm @rosebyte đã nhận 23 ngày trước. Đang mởarea-Tokenizers green
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
dotnet/machinelearning#7724 · 2 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Comment in example code not matching docstring for TimeSeriesCatalog.DetectSeasonality MethodĐang mởarea-TimeSeries documentation untriaged
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
dotnet/machinelearning#7205 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
area-ONNX documentation Priority:3
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
dotnet/machinelearning#5894 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Add a .vsconfig fileĐang mởarea-Infrastructure Priority:2
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
dotnet/machinelearning#3260 · 2 bình luận · 1 reaction ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của dotnet/machinelearning
Issue tương tự
-
python triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
microsoft/semantic-kernel#14615 ·
Maintainer thường phản hồi trong vòng 3 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
fboucher/2d6-dungeon-app#245 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày
-
7.0
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
unoplatform/uno.extensions#3253 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Sector-Vestige/Sector-Vestige#476 ·
Maintainer thường phản hồi trong vòng 1 ngày